-
Notifications
You must be signed in to change notification settings - Fork 87
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
🐛 Fixing bmo version for clusterctl tests #1596
🐛 Fixing bmo version for clusterctl tests #1596
Conversation
d72b357
to
3e45648
Compare
metal3-e2e-clusterctl-upgrade-test-main |
/test metal3-e2e-clusterctl-upgrade-test-main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lentzi90 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
3e45648
to
98a67fb
Compare
d95a6bd
to
1ab98e6
Compare
/test metal3-e2e-clusterctl-upgrade-test-main |
/hold |
1ab98e6
to
fec0452
Compare
fec0452
to
6cdce9d
Compare
0823e1d
to
33308f4
Compare
33308f4
to
3016d87
Compare
@mboukhalfa , @kashifest please review. |
cd7dec8
to
996b510
Compare
/test metal3-e2e-clusterctl-upgrade-test-main |
996b510
to
03be7f1
Compare
/test metal3-e2e-clusterctl-upgrade-test-main |
/test metal3-centos-e2e-integration-test-main |
/test metal3-ubuntu-e2e-integration-test-main |
/test metal3-centos-e2e-integration-test-main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits.
test/e2e/data/ironic-deployment/overlays/release-24.1/kustomization.yaml
Outdated
Show resolved
Hide resolved
test/e2e/data/ironic-deployment/overlays/release-24.1/kustomization.yaml
Outdated
Show resolved
Hide resolved
test/e2e/data/ironic-deployment/overlays/release-latest/ironic_bmo_configmap.env
Outdated
Show resolved
Hide resolved
test/e2e/data/ironic-deployment/overlays/release-latest/kustomization.yaml
Outdated
Show resolved
Hide resolved
test/e2e/data/ironic-deployment/overlays/release-latest/kustomization.yaml
Outdated
Show resolved
Hide resolved
03be7f1
to
207f77f
Compare
@tuminoid fixed the improvements. please check. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two nits still to go.
207f77f
to
66a1ef2
Compare
Signed-off-by: adil ghaffar <muhammad.adil.ghaffar@est.tech>
66a1ef2
to
1620771
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/test metal3-e2e-clusterctl-upgrade-test-main |
/override metal3-centos-e2e-integration-test-main metal3-ubuntu-e2e-integration-test-main clusterctl tests to be run on the final code, held until it passes |
@tuminoid: Overrode contexts on behalf of tuminoid: metal3-centos-e2e-integration-test-main, metal3-ubuntu-e2e-integration-test-main In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/unhold |
What this PR does / why we need it:
clustetctl tests were using the wrong version of bmo and ironic before the upgrade. This PR fixes that by checking out bmo repo to right supported branch before installing bmo and ironic.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #