-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Mapo build prep #24617
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
Mapo build prep #24617
Conversation
ci-operator/config/openshift/machine-api-provider-openstack/.config.prowgen
Outdated
Show resolved
Hide resolved
...ft/machine-api-provider-openstack/openshift-machine-api-provider-openstack-release-4.10.yaml
Outdated
Show resolved
Hide resolved
|
Clarifying conversation occured here: https://coreos.slack.com/archives/CEKNRGF25/p1639507206317300 This will be squashed down to one commit |
|
/label tide/merge-method-squash |
|
And now the both will do it for us |
|
thanks for all the help @petr-muller |
|
/assign @mandre |
|
/lgtm |
|
/retest |
|
Bootstrap failed and there's no must-gather. Is this because we're missing openshift/machine-api-provider-openstack#13? |
|
Edit: on second thoughts, it's weird since we're not using MAPO until after bootstrap machine has completed. It must be something else that broke it. Edit 2: potentially fixed with openshift/installer#5484 |
|
/retest |
|
This time it passed bootstrap \o/ |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: EmilienM, iamemilio, mandre The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/hold cancel |
|
@iamemilio: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. I understand the commands that are listed here. |
|
@iamemilio: Updated the following 3 configmaps:
DetailsIn 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. |
Gets the MAPO CI Operator config in shape to be hooked up to ART.