Skip to content

Conversation

tbugfinder
Copy link
Contributor

No description provided.

@tbugfinder
Copy link
Contributor Author

Not sure about the current process. @mumoshu would you review?

@tbugfinder tbugfinder changed the title bump go from 1.17 to 1.19 and bump to helm 3.11.1 bump go from 1.18 to 1.19 and bump to helm 3.11.3 Apr 24, 2023
@tbugfinder
Copy link
Contributor Author

tbugfinder commented Apr 24, 2023

Merge conflicts were resolved and go.mod/go.sum updated.

Hm, now ending up in:

# helm.sh/helm/v3/pkg/kube
../../go-workspace/pkg/mod/helm.sh/helm/v3@v3.11.3/pkg/kube/client.go:98:12: cannot use cmdutil.NewFactory(getter) (value of type "k8s.io/kubectl/pkg/cmd/util".Factory) as type Factory in struct literal:
	"k8s.io/kubectl/pkg/cmd/util".Factory does not implement Factory (missing OpenAPIGetter method)
make: *** [build] Error 2

@mumoshu
Copy link
Collaborator

mumoshu commented Apr 29, 2023

@tbugfinder Thanks for rebasing!

now ending up in:

We probably need to bump k8s.io deps to v0.26.x as apparently those are required by helm 3.11.3

https://github.com/helm/helm/blob/66a969e7cc08af2377d055f4e6283c33ee84be33/go.mod#L38-L43

@tbugfinder tbugfinder force-pushed the security/update-go-mod branch from beb2cc1 to 0db3ad0 Compare April 29, 2023 15:47
@tbugfinder
Copy link
Contributor Author

I've pushed a less aggressive update to go.mod :-D

@tbugfinder
Copy link
Contributor Author

@mumoshu could the workflow run be approved?

@tehseenshahab
Copy link

@mumoshu Could you please merge this PR? Thanks

Copy link
Collaborator

@mumoshu mumoshu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks a lot for your patience and contribution @tbugfinder!!

@mumoshu mumoshu merged commit d6bed22 into databus23:master May 16, 2023
hexcsl pushed a commit to hexcsl/helm-diff that referenced this pull request Jul 3, 2023
* bump go from 1.17 to 1.19

* bump helm version and dependencies

* update circleci image

* update working directory to a writeable target

* update to helm v3.11.3

---------

Co-authored-by: tbugfinder <tbugfinder@online.ms>
hexcsl pushed a commit to hexcsl/helm-diff that referenced this pull request Jul 3, 2023
* bump go from 1.17 to 1.19

* bump helm version and dependencies

* update circleci image

* update working directory to a writeable target

* update to helm v3.11.3

---------

Co-authored-by: tbugfinder <tbugfinder@online.ms>
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.

3 participants