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

Release rancher-turtles 0.4.0 #166

Merged
merged 5 commits into from
Nov 27, 2024
Merged

Release rancher-turtles 0.4.0 #166

merged 5 commits into from
Nov 27, 2024

Conversation

hardys
Copy link
Contributor

@hardys hardys commented Nov 14, 2024

This aligns with the 0.13 upstream Rancher turtles release, and updates CAPM3 to 1.7.2

@hardys hardys marked this pull request as draft November 14, 2024 18:15
@hardys hardys marked this pull request as ready for review November 19, 2024 16:17
@hardys hardys requested a review from alknopfler November 19, 2024 16:17
alknopfler
alknopfler approved these changes Nov 20, 2024
View reviewed changes
Copy link
Contributor

@alknopfler alknopfler left a comment

Choose a reason for hiding this comment

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

LGTM

@hardys
Copy link
Contributor Author

hardys commented Nov 20, 2024

I found an issue with the airgap CM's - we're pulling the upstream images so will revert this to draft while I work on a fix

@hardys hardys marked this pull request as draft November 20, 2024 08:32
@hardys hardys force-pushed the turtles_013 branch 2 times, most recently from 5c02479 to e320e5b Compare November 21, 2024 17:36
@hardys
Copy link
Contributor Author

hardys commented Nov 21, 2024

The image/version issues should now be resolved:

> for n in rke2-bootstrap-system rke2-control-plane-system capi-system capm3-system rancher-turtles-system; do kubectl get deployment -n $n -o yaml | grep image: ; done
          image: registry.rancher.com/rancher/cluster-api-provider-rke2-bootstrap:v0.8.0
          image: registry.rancher.com/rancher/cluster-api-provider-rke2-controlplane:v0.8.0
          image: registry.rancher.com/rancher/cluster-api-controller:v1.7.7
          image: registry.opensuse.org/isv/suse/edge/clusterapi/containers/images/cluster-api-provider-metal3:1.7.2
          image: registry.opensuse.org/isv/suse/edge/clusterapi/containers/images/ip-address-manager:1.7.2
          image: registry.rancher.com/rancher/cluster-api-operator:v0.14.0
          image: registry.rancher.com/rancher/rancher/turtles:v0.13.0

@hardys hardys marked this pull request as ready for review November 21, 2024 17:48
@hardys hardys requested a review from fdegir November 22, 2024 10:05
@hardys
rancher-turtles: update CAPM3 to 1.7.2
0110410
@hardys
rancher-turtles-airgap-resources: Align with 0.13 updates
421a3ad
@hardys
rancher-turtles: override versions/images in values
c814a5d 
Currently rancher-turtles seems to install latest upstream
CAPI core, so override in the values to ensure we pin and
use the downstream image
@hardys
make charts/html
c2a70f4
@hardys
Copy link
Contributor Author

hardys commented Nov 26, 2024

@fdegir and @Kristian-ZH found an issue testing in SV, now updated with a fix

@hardys hardys mentioned this pull request Nov 26, 2024
Merged
fdegir
fdegir approved these changes Nov 26, 2024
View reviewed changes
Copy link

@fdegir fdegir left a comment

Choose a reason for hiding this comment

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

This passed SV!

@hardys hardys merged commit 2bc4ad0 into suse-edge:main Nov 27, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fdegir fdegir fdegir approved these changes

@alknopfler alknopfler alknopfler approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hardys @fdegir @alknopfler