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

updated k8s-deps to 0.27.4 and controller runtime to 0.15.0 #51

Merged
merged 2 commits into from
Aug 30, 2023

Conversation

vardhaman22
Copy link
Contributor

@vardhaman22 vardhaman22 commented Aug 17, 2023

related issue: rancher/rancher#41840

for updating controller-runtime changes from #48 are added.

Verified

This commit was signed with the committer’s verified signature.
zkochan Zoltan Kochan
Copy link
Member

@jiaqiluo jiaqiluo left a comment

Choose a reason for hiding this comment

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

We need to bump dependencies in controller-runtime/go.mod as well

Copy link
Member

@kinarashah kinarashah left a comment

Choose a reason for hiding this comment

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

Since rancher will be updated to use capi v1.5.0, we'd also need to update corresponding controller-runtime version in lasso. https://github.com/kubernetes-sigs/cluster-api/blob/v1.5.0/go.mod#L46

#48 has partial update to v0.14.0 for reference

@vardhaman22 vardhaman22 force-pushed the k8s-127-lasso branch 2 times, most recently from 444757c to ec98d91 Compare August 19, 2023 01:07
@vardhaman22 vardhaman22 changed the title updated k8s-deps to 0.27.4 updated k8s-deps to 0.27.4 and controller runtime to 0.15.0 Aug 19, 2023
Copy link
Member

@jiaqiluo jiaqiluo left a comment

Choose a reason for hiding this comment

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

LGTM

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
also updated k8s deps to 0.27.4
@vardhaman22 vardhaman22 removed the request for review from snasovich August 22, 2023 18:12
@kinarashah kinarashah merged commit d684fde into rancher:master Aug 30, 2023
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.

None yet

3 participants