-
Notifications
You must be signed in to change notification settings - Fork 79
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
Initial commit to add barbican to openstack-operator #531
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: vakwetu The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/b396023ed4ca4587a2a6ec0b289ac156 ❌ openstack-k8s-operators-content-provider FAILURE in 10m 30s |
the integration looks good to me. best to land the barbican PR and then check the failing jobs here |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/83bdefc74d5c4a148790eab8af5e2069 ❌ openstack-k8s-operators-content-provider FAILURE in 8m 01s |
@vakwetu: The following tests failed, say
Full PR test history. Your PR dashboard. 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. I understand the commands that are listed here. |
PR needs rebase. 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. |
new pull request created with different branch |
This is WIP right now. We need to land a change to update barbican to use the override route first. Then we won't have to do some gymnastics to get the right image of barbican in place and we can test properly.