-
Notifications
You must be signed in to change notification settings - Fork 261
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
🌱 devstack: build OVN from source #2319
Conversation
✅ Deploy Preview for kubernetes-sigs-cluster-api-openstack ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
no failure, retrying |
no failure, retrying |
/cherry-pick release-0.11 release-0.10 |
@EmilienM: once the present PR merges, I will cherry-pick it on top of 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-sigs/prow repository. |
In this PR we'll control the version of OVN to align with the upstream devstack jobs building OVN and OVS from LTS versions. This is largely a workaround to address some timeout issues that we recently have and that with a controlled version of OVN we hope to reduce.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: EmilienM 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 |
no failure, retrying |
1 similar comment
no failure, retrying |
Is there any merit to switching to CentOS here? It looks like openvswitch 3.3 is available pre-built (and 3.1, 3.2, and 3.4). Or is it just that we need some patch which isn't included in a release yet? |
I think switching to another OS in our CI wouldn't be a terrible effort but not a quick workaround to get our CI back in stable state. We should consider that for the future though. |
no failure, retrying |
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
7 successful runs, I think it's safe to try that. |
That will be 8 successful runs 😋 |
@EmilienM: new pull request created: #2324 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-sigs/prow repository. |
What this PR does / why we need it:
In this PR we'll control the version of OVN to align with the upstream
devstack jobs building OVN and OVS from LTS versions.
This is largely a workaround to address some timeout issues that we
recently have and that with a controlled version of OVN we hope to
reduce.
issue #2320