-
Notifications
You must be signed in to change notification settings - Fork 253
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
✨ Update CI to use OpenStack Bobcat #1804
Conversation
✅ Deploy Preview for kubernetes-sigs-cluster-api-openstack ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I know Antelope is the latest stable version but I want to see how it works with Zed first (step by step). |
/test pull-cluster-api-provider-openstack-e2e-test |
8392e39
to
8650543
Compare
/test pull-cluster-api-provider-openstack-e2e-test |
/test pull-cluster-api-provider-openstack-e2e-test |
5da96ae
to
120fb0e
Compare
/test pull-cluster-api-provider-openstack-e2e-test |
c5a4522
to
fbce63d
Compare
/retest |
8858c78
to
7510378
Compare
it's green! let's wait for #1796 to merge. |
/hold cancel |
The tests work fine when we delete the bastion nodes, which make me think switching to bobcat highlight some resources issues in our jobs where some nodes goes into ERROR because "no valid host found" in Nova. |
I don't remember whether we have anything has impact on the performance, so seems your change (vcpu 4=>2) should be good |
2 vcpu should be fine for what we are doing, it is not like we stress the control plane with anything. |
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: EmilienM, 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 |
/lgtm |
@lentzi90 seems the CI not finished but it's merged, is the CI e2e test not a gate anymore (need pass in order to merge) |
Oh! The e2e test is a gate but not the e2e-full. I should have realized to put hold. Well I guess we will soon know if it works or not 🙂 |
Thanks for merging. Yes I'll keep an eye on that job, I run it nearly every day these days... :) |
/cherry-pick release-0.9 |
@EmilienM: #1804 failed to apply on top of branch "release-0.9":
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. |
What this PR does / why we need it:
Update devstack configuration to deploy OpenStack Bobcat with OVN on Ubuntu 22.04.