Skip to content
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

Add support to use pre-built pod VM image for Azure #460

Merged
merged 2 commits into from
Sep 24, 2024

Conversation

bpradipt
Copy link
Contributor

No description provided.

This commit fixes shellcheck warnigns related to
quotes, variable misspelling etc

Signed-off-by: Pradipta Banerjee <pradipta.banerjee@gmail.com>
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 16, 2024
Copy link

openshift-ci bot commented Sep 16, 2024

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Copy link
Contributor

@snir911 snir911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall, added few comments

config/peerpods/podvm/azure-podvm-image-handler.sh Outdated Show resolved Hide resolved
config/peerpods/podvm/azure-podvm-image-handler.sh Outdated Show resolved Hide resolved
config/peerpods/podvm/azure-podvm-image-handler.sh Outdated Show resolved Hide resolved
@bpradipt bpradipt force-pushed the raw-image branch 4 times, most recently from f96c565 to ba9f985 Compare September 18, 2024 14:33
Allows creating image version from pre-built qcow2/raw/vhd image.
The pre-built image is available in a container image and
the details needs to be specified in PODVM_IMAGE_URI
in azure-podvm-image-cm.yaml.

Azure accepts 1MB aligned VHD image only,
so a qcow2 or raw will be converted to the same before uploading.

qemu-img is used for the conversion.

eg:
oci::quay.io/openshift_sandboxed_containers/azure-podvm-image:latest::/image/podvm.vhd
oci::quay.io/openshift_sandboxed_containers/azure-podvm-image:latest::/podvm.qcow2

Signed-off-by: Pradipta Banerjee <pradipta.banerjee@gmail.com>

podvm-builder: Add qemu-img package

qemu-img will be used to verify the pre-built image as well
as convert it to different format as needed

Signed-off-by: Pradipta Banerjee <pradipta.banerjee@gmail.com>
@bpradipt bpradipt marked this pull request as ready for review September 18, 2024 15:21
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 18, 2024
@bpradipt bpradipt requested a review from snir911 September 18, 2024 15:21
@openshift-ci openshift-ci bot requested review from gkurz and pmores September 18, 2024 15:21
Copy link

openshift-ci bot commented Sep 18, 2024

@bpradipt: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/sandboxed-containers-operator-e2e 391be5c link false /test sandboxed-containers-operator-e2e
ci/prow/check 391be5c link false /test check

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-sigs/prow repository. I understand the commands that are listed here.

Copy link
Contributor

@snir911 snir911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks!, added one question

@bpradipt bpradipt requested a review from littlejawa September 23, 2024 13:35
Copy link
Contributor

@littlejawa littlejawa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
Thanks @bpradipt

@bpradipt bpradipt merged commit e8d9a78 into openshift:devel Sep 24, 2024
2 of 4 checks passed
@bpradipt bpradipt deleted the raw-image branch September 24, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants