openstack: add image for openstack ci#1466
openstack: add image for openstack ci#1466openshift-merge-robot merged 1 commit intoopenshift:masterfrom
Conversation
|
/lgtm |
|
/test e2e-aws |
There was a problem hiding this comment.
@abhinavdahiya In my local tests, the epel-release and rdo-release rpms were not available in the base image. I had to install them from https links.
There was a problem hiding this comment.
do you have log from failure...
did it look like #1456 (comment)
There was a problem hiding this comment.
It just didnt find the package. So it fails silently, but then the openstackclient is not installed.
|
/test e2e-aws |
|
@abhinavdahiya could we merge this? |
There was a problem hiding this comment.
Can this go in images/openstack/Dockerfile.ci or similar with an OWNERS file? The UPI file is here because we expect it to be cross-platform.
There was a problem hiding this comment.
yep that makes sense. thanks for the review
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinavdahiya, trown, wking 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 |
|
/test e2e-aws |
Adds a CI image for use by the openstack e2e job which has the openstack cli clients.