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

roles: hosted_engine_setup: Adding the -n flag to qemu-img command #682

Merged
merged 1 commit into from
Feb 28, 2023

Conversation

saksham-oracle
Copy link
Contributor

@saksham-oracle saksham-oracle commented Feb 17, 2023

Issues:

  1. Desired Engine Disk size for hosted engine VM is not reflected and always 38GB.

Fix:

  1. Adding the -n flag to use the same flags used by vdsm to ensure that we get consistent results compatible with volumes created by vdsm. Vdsm now uses -n flag for all qemu-img convert calls "https://gerrit.ovirt.org/c/vdsm/+/109784".
    https://bugzilla.redhat.com/show_bug.cgi?id=1828888#c5

Signed-off-by: Saksham Srivastava saksham.sa.srivastava@oracle.com

@arachmani
Copy link
Member

Code LGTM, please add a changelog and a commit message with "Signed-off-by" in order to make CI happy.

@arachmani arachmani changed the title Adding the -n flag to qemu-img command roles: hosted_engine_setup: Adding the -n flag to qemu-img command Feb 19, 2023
@mwperina mwperina requested a review from mnecas February 21, 2023 13:25
@mnecas
Copy link
Member

mnecas commented Feb 21, 2023

+1
The Signed-off-by needs to be in the commit-msg; please git commit --amend -s git push -f.
And add the changelog as https://github.com/oVirt/ovirt-ansible-collection/blob/master/changelogs/fragments/684-ovirt_cluster_info-fix-example-patter.yml

@saksham-oracle saksham-oracle force-pushed the qemu-img-change branch 2 times, most recently from c304fc3 to 557b030 Compare February 28, 2023 12:08
1. Desired Engine Disk size for hosted engine VM is not reflected and always 38GB.

Fix:

1. Adding the -n flag to use the same flags used by vdsm to ensure that we get consistent results compatible with volumes created by vdsm. Vdsm now uses -n flag for all qemu-img convert calls "https://gerrit.ovirt.org/c/vdsm/+/109784".
https://bugzilla.redhat.com/show_bug.cgi?id=1828888#c5

Signed-off-by: Saksham Srivastava <saksham.sa.srivastava@oracle.com>
Copy link
Member

@arachmani arachmani left a comment

Choose a reason for hiding this comment

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

LGTM

@mnecas mnecas merged commit 56f8f8a into oVirt:master Feb 28, 2023
@mnecas mnecas mentioned this pull request Mar 3, 2023
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