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.

Sign up for GitHub

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

Jump to bottom

RestAPI issue: New VM from a template with set 'resize_and_pin_numa' CPU policy is created with 'None' CPU policy #633

Closed
polinaag opened this issue Sep 5, 2022 · 0 comments · Fixed by #649
Closed

RestAPI issue: New VM from a template with set 'resize_and_pin_numa' CPU policy is created with 'None' CPU policy #633

polinaag opened this issue Sep 5, 2022 · 0 comments · Fixed by #649
Assignees
@smelamud
Labels
bug virt

Comments

@polinaag
Copy link

polinaag commented Sep 5, 2022

  1. Create a template with the CPU policy = 'resize_and_pin_numa', pinned to host. The template xml is attached.
  2. Create a new VM from this template by rest API

POST https://{{host}}/ovirt-engine/api/vms/

vm_from_resize_template

golden_env_mixed_1


resize_and_pin_template

actual result: the VM is created with None CPU policy although the template is set with 'resize_and_pin_numa' policy
At the same time if to create such a VM in web UI it will be created with resize policy as expected.

expected result: VM must be created with the 'resize_and_pin_numa' policy
resize_and_pin_template.log
@smelamud smelamud self-assigned this Sep 12, 2022
smelamud added a commit to smelamud/ovirt-engine that referenced this issue Sep 13, 2022
@smelamud
restapi: Use cpuPinningPolicy from template
a17852a 
Include cpuPinningPolicy when extracting data from a VM template for
creating a VM through REST API.

Fixes oVirt#633

Change-Id: I139d3c9c00a005610eebe172272a45b3f37ace27
Signed-off-by: Shmuel Melamud <smelamud@redhat.com>
@smelamud smelamud mentioned this issue Sep 13, 2022
Merged
smelamud added a commit to smelamud/ovirt-engine that referenced this issue Sep 14, 2022
@smelamud
restapi: Use additional fields from template
21c721b 
Include cpuPinningPolicy, cpuPinning, parallelMigrations fields when
extracting data from a VM template for creating a VM through REST API.

Fixes oVirt#633

Change-Id: I139d3c9c00a005610eebe172272a45b3f37ace27
Signed-off-by: Shmuel Melamud <smelamud@redhat.com>
ljelinkova pushed a commit that referenced this issue Sep 15, 2022
@smelamud @ljelinkova
restapi: Use additional fields from template
df63bc6 
Include cpuPinningPolicy, cpuPinning, parallelMigrations fields when
extracting data from a VM template for creating a VM through REST API.

Fixes #633

Change-Id: I139d3c9c00a005610eebe172272a45b3f37ace27
Signed-off-by: Shmuel Melamud <smelamud@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@smelamud smelamud

Labels
bug virt
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

restapi: Use cpuPinningPolicy from template smelamud/ovirt-engine
3 participants
@smelamud @mz-pdm @polinaag