Monday, 2022-10-03

abdiOpenStack 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
opendevreviewAmit Uniyal proposed openstack/nova master: Adds check if resized to swap zero  https://review.opendev.org/c/openstack/nova/+/85733908:06
opendevreviewSahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state  https://review.opendev.org/c/openstack/nova/+/85838309:32
opendevreviewSahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state  https://review.opendev.org/c/openstack/nova/+/85838409:32
auniyalHi gibi 09:45
auniyalI have a doubt in this comment, can you please confirm, the changes 09:45
auniyalhttps://review.opendev.org/c/openstack/nova/+/791135/709:45
auniyallast comment in this patch09:46
gibiauniyal: 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 block09:55
sean-k-mooneygibi: no10:10
sean-k-mooneygibi: post live migration is after we are allowed to rollback10:10
gibisorry10:10
gibiI mixed source det10:10
gibidest10:10
sean-k-mooneyoh ok10:11
gibiso it is not enough to set the instance.host but need to set the instance.node10:11
gibito the dest10:11
sean-k-mooneyyes10:11
gibibut then auniyal you have to look up the dest node10:11
sean-k-mooneywell your running on the test at this point10:12
auniyalisn't dest node is same as dest host ? or dest10:12
sean-k-mooneyso you just need to set it im not sure you need to look it up10:12
sean-k-mooneyauniyal: for libvirt yes10:12
sean-k-mooneyin general no10:12
sean-k-mooneyactully no10:12
sean-k-mooneyeven for libvirt they can be different10:13
sean-k-mooneyalthough i think that owuld be an error10:13
sean-k-mooney   node_name = compute_node.hypervisor_hostname10:15
sean-k-mooneywhere as instance.host = CONF.host10:15
auniyalqueston, 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-mooneyby default those shoudl be the same but if you manually set [DEFAULT]/host10:16
sean-k-mooneythen they can be differnt10:16
sean-k-mooneynode is the name of the comptue node10:16
sean-k-mooneyhost is the name of the compute service10:16
sean-k-mooneyfor non clustered drivers like libvirt they are normally the same10:16
sean-k-mooneyfor clustered dirvers like ironic where 1 compute service manages many baremetal servers10:17
sean-k-mooneythey are differnet10:17
sean-k-mooneyyou can look up the name like this 10:18
sean-k-mooneyhttps://github.com/openstack/nova/blob/1025c9879341d44db33c4cc501435364dd185a9e/nova/compute/manager.py#L9263-L926710:18
auniyalack10:19
opendevreviewAndre Aranha proposed openstack/nova master: Replace Centos 8 jobs for Centos 9  https://review.opendev.org/c/openstack/nova/+/85827211:48
opendevreviewAndre Aranha proposed openstack/nova master: Remove the periodic Centos 8 job  https://review.opendev.org/c/openstack/nova/+/85827211:49
opendevreviewAndre Aranha proposed openstack/nova stable/yoga: Test setting the nova job to centos-9-stream  https://review.opendev.org/c/openstack/nova/+/86008712:02
opendevreviewMaksim Malchuk proposed openstack/nova stable/xena: Fix to implement 'pack' or 'spread' VM's NUMA cells  https://review.opendev.org/c/openstack/nova/+/82980412:39
*** dasm|off is now known as dasm12:59
atmarkHello. Getting this error when trying to temporarily increase allocation_ratio in placement via CLI14:48
atmarkJSON 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.014:48
gibiatmark: either define the total value in --resource too, or use --amend to instruct the client to only change the allocation_ratio16:14
atmarkgibi: --amend works, thanks 17:19
gibiatmark: :)17:44
*** dasm is now known as dasm|off21:16

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!