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

use quay.io qm - kvm #661

Merged
merged 2 commits into from
Dec 12, 2024
Merged

use quay.io qm - kvm #661

merged 2 commits into from
Dec 12, 2024

Conversation

dougsland
Copy link
Collaborator

@dougsland dougsland commented Nov 14, 2024

PR is base for #609

It creates the howto upload containerized kvm to
https://quay.io/organization/qm-images

@dougsland
Copy link
Collaborator Author

I was working/discussing with @Yarboa about this one.
cc @aesteve-rh

Copy link
Collaborator

@Yarboa Yarboa left a comment

Choose a reason for hiding this comment

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

I am not sure I am following the use case,
Is it containerized vm inside QM., correct?
So I think it obsoletes the previous work done for kvm, isnt it?
I think README should be changed, also, do you want to maintain both cases?

rpm/qm.spec Outdated Show resolved Hide resolved
@dougsland
Copy link
Collaborator Author

dougsland commented Nov 17, 2024

I am not sure I am following the use case, Is it containerized vm inside QM., correct?

You are correct.

So I think it obsoletes the previous
work done for kvm, isnt it? I think README should be changed, also, do you want to maintain both cases?

Yes, as soon it works we need to update README.md and remove the previous manual steps.

@dougsland dougsland changed the title [WIP]: use quay.io qm - kvm use quay.io qm - kvm Nov 20, 2024
subsystems/kvm/ContainerFile Outdated Show resolved Hide resolved
subsystems/kvm/ContainerFile Outdated Show resolved Hide resolved
@Yarboa
Copy link
Collaborator

Yarboa commented Dec 2, 2024

Please review #670
Rebase and lets merge this one.

Signed-off-by: Douglas Schilling Landgraf <[email protected]>
Signed-off-by: Yariv Rachmani <[email protected]>
@Yarboa
Copy link
Collaborator

Yarboa commented Dec 11, 2024

@dougsland
I am removing the image download and guestfs-tools
Once image created,
We should download image, virt-cust and podman cp image qcow2 image

Current image goes up to 3.5GB it is too big

@dougsland
Copy link
Collaborator Author

@dougsland I am removing the image download and guestfs-tools Once image created, We should download image, virt-cust and podman cp image qcow2 image

Current image goes up to 3.5GB it is too big

I trust on you, just update the PR or create a new one with your ideas.

To reduce size and time od contianer quay.io/qm-images/kvm
Build, qcow2 images is manipulated on host

Signed-off-by: Yariv Rachmani <[email protected]>
@dougsland dougsland merged commit cac7300 into main Dec 12, 2024
13 checks passed
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.

2 participants