opendevreview | melanie witt proposed openstack/nova stable/train: Adapt websocketproxy tests for SimpleHTTPServer fix https://review.opendev.org/c/openstack/nova/+/866201 | 00:02 |
---|---|---|
*** akekane is now known as abhishekk | 04:59 | |
opendevreview | Wenping Song proposed openstack/nova master: Get only resolved arqs instead of filter all arqs https://review.opendev.org/c/openstack/nova/+/866291 | 06:12 |
opendevreview | Manuel Bentele proposed openstack/nova master: libvirt: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova/+/828675 | 08:44 |
opendevreview | Merged openstack/nova master: Adds regression functional test for 1980720 https://review.opendev.org/c/openstack/nova/+/861357 | 08:45 |
opendevreview | Merged openstack/nova stable/train: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/864168 | 10:25 |
jsanemet | hello | 10:54 |
jsanemet | could i get a review for this spec? | 10:54 |
jsanemet | https://review.opendev.org/c/openstack/nova-specs/+/865432 | 10:54 |
jsanemet | thanks | 10:54 |
opendevreview | Merged openstack/nova stable/train: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/864169 | 11:37 |
opendevreview | Rajat Dhasmana proposed openstack/nova stable/wallaby: [stable-only] Use os-brick from source in wallaby https://review.opendev.org/c/openstack/nova/+/866326 | 11:49 |
*** akekane is now known as abhishekk | 13:07 | |
*** dasm|off is now known as dasm | 13:46 | |
opendevreview | Manuel Bentele proposed openstack/nova master: libvirt: Add configuration options to set SPICE compression settings https://review.opendev.org/c/openstack/nova/+/828675 | 13:55 |
tobias-urdin | do we support live migration of instances with pinned pcpus? perhaps sean-k-mooney can shime in | 14:08 |
tobias-urdin | imo should be supported since train, but always better to ask i guess | 14:11 |
sean-k-mooney | tobias-urdin: yes but it depend on the release | 14:11 |
bauzas | we indeed do | 14:12 |
sean-k-mooney | https://specs.openstack.org/openstack/nova-specs/specs/train/implemented/numa-aware-live-migration.html | 14:12 |
sean-k-mooney | the numa aware live migration feature added the ablity to recaluate and update the xml for the destination host | 14:12 |
sean-k-mooney | tobias-urdin: before that you could only live migrate if the same cpus were free on the dest and that was kind of a hack | 14:13 |
sean-k-mooney | so before train really only cold migration | 14:13 |
sean-k-mooney | after train live migration should work properly | 14:13 |
sean-k-mooney | tobias-urdin: the same is true for hugepages. it was done in the same feautre | 14:14 |
sean-k-mooney | tobias-urdin: there is still one unfixed bug | 14:14 |
sean-k-mooney | live migrating between hosts with different vcpu_pin_sets or different cpu_shared_sets | 14:14 |
sean-k-mooney | for non numa instnace is still technically broken | 14:15 |
opendevreview | Merged openstack/nova stable/yoga: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/855023 | 14:15 |
opendevreview | Merged openstack/nova stable/yoga: Remove double mocking https://review.opendev.org/c/openstack/nova/+/855024 | 14:15 |
tobias-urdin | ack, so if nodes are identical in terms of config (pin sets) it shouldn't be a problem | 14:15 |
opendevreview | Merged openstack/nova stable/yoga: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/855025 | 14:15 |
sean-k-mooney | we do not update the cpus for floating instnaces until you hard reboot the vm | 14:15 |
opendevreview | Merged openstack/nova stable/yoga: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/855026 | 14:15 |
opendevreview | Merged openstack/nova stable/victoria: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/863903 | 14:15 |
sean-k-mooney | tobias-urdin: for pinend vms the pin sets can be differnt and it shoudl not be a problem | 14:16 |
sean-k-mooney | in trian+ | 14:16 |
tobias-urdin | ack, not sure I understand the bug tho | 14:18 |
sean-k-mooney | the but is if you are not using cpu pinning, and you have set cpu_share_set or vcpu_pin_set | 14:19 |
sean-k-mooney | if that is not the same on all hosts floating vms will float over the wrong cores after migrating | 14:20 |
sean-k-mooney | the numa aware migration feature only updates the cpus if the vm has a numa toplogy | 14:20 |
sean-k-mooney | if its a normal vm with no numa info it does not update the cpus | 14:20 |
sean-k-mooney | and if you have eiterh vcpu_pin_set defiend or cpu_shared_set then the floating vms will be soft pinned to float over those cores | 14:21 |
tobias-urdin | ah I see, thanks, that shouldn't be an issue on our part then. thanks as always! | 14:21 |
sean-k-mooney | so this bug has been there since like icehouse wehn vcpu_pin_set was added as a config option | 14:21 |
sean-k-mooney | we have just never fixed it as it need an rpc change to pass back the requried info | 14:22 |
tobias-urdin | ack | 14:45 |
*** blarnath is now known as d34dh0r53 | 15:06 | |
atmark | hello, looking at resume_state_on_host_boot() in https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4150-L4172, I like to add OR condition in https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4164 such that it also ignores if the VM has property `NoRestart=True' e.g. `if state in ignored_states or instance_properties = | 15:32 |
atmark | 'NoRestart=True':`. Appreciate any pointers on how can I accomplish this. | 15:32 |
bauzas | sean-k-mooney: gibi found a good issue in your stable/xena series for VDPA https://review.opendev.org/c/openstack/nova/+/864932 | 15:54 |
bauzas | sean-k-mooney: just update the commit msg and we're all cool | 16:00 |
sean-k-mooney | bauzas: yes i think i have seen that | 16:01 |
sean-k-mooney | and i think its adressed in the train version | 16:01 |
sean-k-mooney | but ill need to look at it again | 16:01 |
sean-k-mooney | i know i have to respine it for something and redo the train oen as a result | 16:01 |
opendevreview | sean mooney proposed openstack/nova stable/xena: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/864932 | 18:14 |
opendevreview | sean mooney proposed openstack/nova stable/xena: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/864933 | 18:14 |
opendevreview | sean mooney proposed openstack/nova stable/xena: Remove double mocking https://review.opendev.org/c/openstack/nova/+/864934 | 18:14 |
opendevreview | sean mooney proposed openstack/nova stable/xena: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/864935 | 18:14 |
opendevreview | sean mooney proposed openstack/nova stable/xena: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/864936 | 18:14 |
opendevreview | sean mooney proposed openstack/nova stable/xena: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/864937 | 18:14 |
sean-k-mooney | bauzas: once ^ are merged ill redo the wallaby ones to update teh cherry-picked lines | 18:15 |
sean-k-mooney | i just updated the commit messages in the first two patches the rest are the same just rebased | 18:16 |
*** EugenMayer42 is now known as EugenMayer4 | 19:14 | |
opendevreview | Merged openstack/nova stable/yoga: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/855035 | 19:25 |
*** dasm is now known as dasm|off | 22:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!