abdi | OpenStack Nova Team. I filed the following bug against tempest: https://bugs.launchpad.net/tempest/+bug/1989232. However, it has not got a lot of attention from the tempest side. My ongoing investigation suggests nova may be the culprit. Could you review my last posting on the bug and comment? | 03:13 |
---|---|---|
opendevreview | Amit Uniyal proposed openstack/nova master: Adds check if resized to swap zero https://review.opendev.org/c/openstack/nova/+/857339 | 08:06 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | 09:32 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | 09:32 |
auniyal | Hi gibi | 09:45 |
auniyal | I have a doubt in this comment, can you please confirm, the changes | 09:45 |
auniyal | https://review.opendev.org/c/openstack/nova/+/791135/7 | 09:45 |
auniyal | last comment in this patch | 09:46 |
gibi | auniyal: what I ment there is that you have to restore the state of the instance object to point to the source host. And that needs not just the instance.host but also the instnace.node to be set properly. You can copy the instance current node at the start of the post_live_migration_at_destination call in a local variabla and then use that to restore instance.node in the finally block | 09:55 |
sean-k-mooney | gibi: no | 10:10 |
sean-k-mooney | gibi: post live migration is after we are allowed to rollback | 10:10 |
gibi | sorry | 10:10 |
gibi | I mixed source det | 10:10 |
gibi | dest | 10:10 |
sean-k-mooney | oh ok | 10:11 |
gibi | so it is not enough to set the instance.host but need to set the instance.node | 10:11 |
gibi | to the dest | 10:11 |
sean-k-mooney | yes | 10:11 |
gibi | but then auniyal you have to look up the dest node | 10:11 |
sean-k-mooney | well your running on the test at this point | 10:12 |
auniyal | isn't dest node is same as dest host ? or dest | 10:12 |
sean-k-mooney | so you just need to set it im not sure you need to look it up | 10:12 |
sean-k-mooney | auniyal: for libvirt yes | 10:12 |
sean-k-mooney | in general no | 10:12 |
sean-k-mooney | actully no | 10:12 |
sean-k-mooney | even for libvirt they can be different | 10:13 |
sean-k-mooney | although i think that owuld be an error | 10:13 |
sean-k-mooney | node_name = compute_node.hypervisor_hostname | 10:15 |
sean-k-mooney | where as instance.host = CONF.host | 10:15 |
auniyal | queston, what is diff between host and node, till now, I thought the instance.host is where compute service is running, (node is as general we say in any network tree/mapping (machine)) | 10:16 |
sean-k-mooney | by default those shoudl be the same but if you manually set [DEFAULT]/host | 10:16 |
sean-k-mooney | then they can be differnt | 10:16 |
sean-k-mooney | node is the name of the comptue node | 10:16 |
sean-k-mooney | host is the name of the compute service | 10:16 |
sean-k-mooney | for non clustered drivers like libvirt they are normally the same | 10:16 |
sean-k-mooney | for clustered dirvers like ironic where 1 compute service manages many baremetal servers | 10:17 |
sean-k-mooney | they are differnet | 10:17 |
sean-k-mooney | you can look up the name like this | 10:18 |
sean-k-mooney | https://github.com/openstack/nova/blob/1025c9879341d44db33c4cc501435364dd185a9e/nova/compute/manager.py#L9263-L9267 | 10:18 |
auniyal | ack | 10:19 |
opendevreview | Andre Aranha proposed openstack/nova master: Replace Centos 8 jobs for Centos 9 https://review.opendev.org/c/openstack/nova/+/858272 | 11:48 |
opendevreview | Andre Aranha proposed openstack/nova master: Remove the periodic Centos 8 job https://review.opendev.org/c/openstack/nova/+/858272 | 11:49 |
opendevreview | Andre Aranha proposed openstack/nova stable/yoga: Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/860087 | 12:02 |
opendevreview | Maksim Malchuk proposed openstack/nova stable/xena: Fix to implement 'pack' or 'spread' VM's NUMA cells https://review.opendev.org/c/openstack/nova/+/829804 | 12:39 |
*** dasm|off is now known as dasm | 12:59 | |
atmark | Hello. Getting this error when trying to temporarily increase allocation_ratio in placement via CLI | 14:48 |
atmark | JSON does not validate: 'total' is a required property Failed validating 'required' in schema['properties']['inventories']['patternProperties']['^[A-Z0-9_]+$']: | 14:48 |
atmark | ./openstack resource provider inventory set c80259e6-67fb-47a0-b04f-364b2f2b2969 --resource MEMORY_MB:allocation_ratio=1.0 | 14:48 |
gibi | atmark: either define the total value in --resource too, or use --amend to instruct the client to only change the allocation_ratio | 16:14 |
atmark | gibi: --amend works, thanks | 17:19 |
gibi | atmark: :) | 17:44 |
*** dasm is now known as dasm|off | 21:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!