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.

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

[WIP] Road to version 9.0: Updated go version to 1.22 & Fixed Breaking Changes ✨ #1037

Open
wants to merge 32 commits into
base: version-9.0
Choose a base branch
from

Conversation

ansh-devs
Copy link

@ansh-devs ansh-devs commented Jul 22, 2024

Changes

  • Updated GO version to 1.22.
  • Updated deprecated WebhookServer And CacheNamespace configurations to use the updated controller-runtime.
  • Refactored configMapResource in jenkinsController.
  • Makefile Indentation fixed.
  • Nix Builds unresolved dependencies fixed. (Now checks are successfully passing).
  • Updated golangci-lint version to golangci-lint@v1.55.0.
  • Updated Mocks for pkg/controller/jenkins/client/jenkins.go to support new changes.
  • Update E2E tests and make them compatible with new changes.

Pending changes

  • Update HELM tests and make them compatible with new changes.

NOTE

  • As of now only one change has been made in .github/workflows that is to exclude go.sum from Codespell

View Patch

@brokenpip3
Copy link
Collaborator

wow great work in few days so far! 👏 🚀

let's remove the prealloc linter from the golanci-lint config temporary so we can see what is the status of the e2e tests :)

Checking the bats logs I saw some issues but let's make sure that the e2e tests are working first, I can help in case of issues since I have the deep knowledge of those tests, so let me know in case you will face something odd :)

@ansh-devs
Copy link
Author

ansh-devs commented Jul 23, 2024

Sure, Iet's go with that

@brokenpip3
Copy link
Collaborator

I see great progress :)

to fix the latest issues can you update the controller gen version?
Taking a quick look should be something like:

-KUSTOMIZE_VERSION ?= v3.8.7
-CONTROLLER_TOOLS_VERSION ?= v0.9.2
+KUSTOMIZE_VERSION ?= v5.3.0
+CONTROLLER_TOOLS_VERSION ?= v0.14.0

we probably also need to change the envtest version and installation method but it's not critical now

@ansh-devs
Copy link
Author

I see great progress :)

to fix the latest issues can you update the controller gen version? Taking a quick look should be something like:

-KUSTOMIZE_VERSION ?= v3.8.7
-CONTROLLER_TOOLS_VERSION ?= v0.9.2
+KUSTOMIZE_VERSION ?= v5.3.0
+CONTROLLER_TOOLS_VERSION ?= v0.14.0

we probably also need to change the envtest version and installation method but it's not critical now

seems to be an type check errors issue among packages I will look into it !

@ansh-devs
Copy link
Author

@brokenpip3 Have you checked it

@brokenpip3
Copy link
Collaborator

ehi @ansh-devs unfortunately no, I still need to do it.
I usually try to cover one or two weekends to work on oss projects but so far I was not able to do it, 'cause life.
I will try to do my best in the following weeks, don't worry, soon or later I will put the proper effort like I did in the past 2 years :)
I'm sorry if your PRs is blocked by me but right now I can't do more than this.

@ansh-devs
Copy link
Author

Sure take your time😃

@ansh-devs
Copy link
Author

Hi @brokenpip3, are there any breaking changes due to upgrade ?

@brokenpip3
Copy link
Collaborator

No no nothing :)
I was just bumping to exclude other issues for the tests, I'm positive that soon I will have one day to dedicate to find the failures and unblock this PR and the migration epic :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants