Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update dependencies #102

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Update dependencies #102

wants to merge 1 commit into from

Conversation

antoninbas
Copy link
Member

  • Bump Go from 1.19 to 1.23
  • Use UBI9 instead of UBI8 for operator base image
  • Bump some minor Go module dependencies
  • Adopt best practices in Dockerfile for building Go code
  • Fix typo in README

@antoninbas
Update dependencies
bbd804a 
* Bump Go from 1.19 to 1.23
* Use UBI9 instead of UBI8 for operator base image
* Bump some minor Go module dependencies
* Adopt best practices in Dockerfile for building Go code
* Fix typo in README

Signed-off-by: Antonin Bas <[email protected]>
@antoninbas antoninbas marked this pull request as draft September 13, 2024 22:35
@antoninbas
Copy link
Member Author

@ksamoray I'm think there is a better way to bring the operator up-to-date. Can you confirm that you use the operator-sdk CLI to create this project?

@ksamoray
Copy link
Contributor

@ksamoray I'm think there is a better way to bring the operator up-to-date. Can you confirm that you use the operator-sdk CLI to create this project?

Hi, I took over this after the project was already existing. @salv-orlando might know how this was initially set up.

@ksamoray
Copy link
Contributor

@antoninbas Code seems OK yet failing UTs, can you have a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ksamoray ksamoray ksamoray approved these changes

@salv-orlando salv-orlando Awaiting requested review from salv-orlando

Assignees
No one assigned
Labels
cla-not-required
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@antoninbas @ksamoray @vmwclabot