Skip to content

[release-4.4] Bug 1813851: OpenStack: start using image import when possible#3297

Closed
Fedosin wants to merge 4 commits intoopenshift:release-4.4from
Fedosin:image_import_4_4
Closed

[release-4.4] Bug 1813851: OpenStack: start using image import when possible#3297
Fedosin wants to merge 4 commits intoopenshift:release-4.4from
Fedosin:image_import_4_4

Conversation

@Fedosin
Copy link
Contributor

@Fedosin Fedosin commented Mar 16, 2020

Now we use legacy image uploading that doesn't support image conversion. It leads to the fact that we upload qcow2 images to Ceph backend, which doesn't support this format:
https://access.redhat.com/solutions/2434691

By using the Image Import mechanism we make sure that all uploaded images will be automatically converted to Raw for Ceph (and only for Ceph) backends.

If the image import mechanism is not available we fallback to the legacy uploading.

cherry-picked from: #3162

Now we use Terraform to create images in Glance. This commit starts
using direct Gophercloud calls for this.
@openshift-ci-robot openshift-ci-robot added the bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. label Mar 16, 2020
@openshift-ci-robot
Copy link
Contributor

@Fedosin: This pull request references Bugzilla bug 1813851, which is invalid:

  • expected dependent Bugzilla bug 1806143 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but it is ON_QA instead

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

[release-4.4] Bug 1813851: OpenStack: start using image import when possible

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.

@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign wking
You can assign the PR to them by writing /assign @wking in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Fedosin added 3 commits March 16, 2020 13:16
This commit updates gophercloud library and sets the version to
"0.8.0", because "0.9.0" contains a breaking change.
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Mar 16, 2020

@Fedosin: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/e2e-libvirt bb8ebf9 link /test e2e-libvirt
ci/prow/e2e-openstack bb8ebf9 link /test e2e-openstack
ci/prow/e2e-aws-fips bb8ebf9 link /test e2e-aws-fips

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

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.

@Fedosin
Copy link
Contributor Author

Fedosin commented Mar 17, 2020

/hold

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Mar 17, 2020
@Fedosin Fedosin closed this Mar 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants