| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-20 | |||
| 10:43:08 | kashyap | sean-k-mooney: That's because, the second check evaluates both model _and_ flags. So that is useful. | |
| 10:43:09 | sean-k-mooney | if you wer only removing the first check but keeping the second i was ok with that being done unconditionally | |
| 10:43:15 | kashyap | sean-k-mooney: Aaah | |
| 10:43:16 | kashyap | I see | |
| 10:43:26 | sean-k-mooney | right | |
| 10:43:35 | sean-k-mooney | i wanted to keep the second correct check | |
| 10:43:37 | kashyap | sean-k-mooney: So okay ... that is also a possibility | |
| 10:43:53 | sean-k-mooney | so im ok with approving your currnt patch or you can just delete the first check | |
| 10:43:58 | sean-k-mooney | its up to you | |
| 10:43:59 | kashyap | sean-k-mooney: I can remove the first one unconditionally and keep the second under the workaround | |
| 10:44:11 | sean-k-mooney | i dont think you need the workaround | |
| 10:44:22 | sean-k-mooney | if you just delete the second one | |
| 10:44:25 | sean-k-mooney | *first one | |
| 10:44:45 | kashyap | Right. | |
| 10:44:49 | sean-k-mooney | but if you want to future proof then sure you can put the remain check under a workaroudn | |
| 10:44:57 | sean-k-mooney | anyway let me knwo which you would prefer | |
| 10:45:06 | sean-k-mooney | the curernt patch https://review.opendev.org/c/openstack/nova/+/870794/6/nova/virt/libvirt/driver.py | |
| 10:45:16 | sean-k-mooney | or just delete the first check | |
| 10:45:17 | kashyap | sean-k-mooney: Yes, I want to future-proof it a bit by keeping the 2nd part under the workaround | |
| 10:45:28 | kashyap | We can't test all possible CPU models on all hardware | |
| 10:45:34 | kashyap | This check can prove to be useful | |
| 10:45:36 | kashyap | sean-k-mooney: How about this: | |
| 10:45:42 | sean-k-mooney | ok gibi ^ does that work for you | |
| 10:45:46 | kashyap | remove the first check and keep the second one under the workaround. | |
| 10:46:05 | sean-k-mooney | sure the second check is not under the workaround in the current patch by the way | |
| 10:46:14 | sean-k-mooney | but im ok with what you are suggesting | |
| 10:46:16 | kashyap | sean-k-mooney: Yes, yes. I know :) | |
| 10:46:45 | bauzas | kashyap: shall I hold reviewing ? | |
| 10:47:27 | kashyap | bauzas: Yes, pls | |
| 10:47:48 | bauzas | k | |
| 10:47:54 | bauzas | anyway, I need to do gym | |
| 11:24:53 | gibi | sean-k-mooney, kashyap: removing the first check unconditionally and making the second check conditionally (and by default executed) works for me | |
| 11:25:09 | kashyap | gibi: Nod; reworking ... the unit test | |
| 11:42:54 | artom | bauzas, hey, so after gym, would https://review.opendev.org/c/openstack/nova/+/869812 be good to go? | |
| 11:49:22 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | |
| 11:49:22 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: At start-up rework compareCPU() usage with a workaround https://review.opendev.org/c/openstack/nova/+/870794 | |
| 11:49:28 | kashyap | gibi: sean-k-mooney: --^ | |
| 11:52:10 | kashyap | gibi: (I've also readjusted the wording where appropriate from the old patch) | |
| 11:53:09 | kashyap | (Lost the +W on the 2nd patch) | |
| 12:37:09 | gibi | kashyap: done. I'm happy that we found a solution that is acceptable to all involved parties | |
| 12:38:41 | kashyap | gibi: Yes :) | |
| 12:38:44 | kashyap | Thank you! | |
| 12:51:19 | gibi | folks https://review.opendev.org/c/openstack/tempest/+/870974 just merged to skip the test case that causes OOM on the gate, so it should be easier now to land patches | |
| 12:51:20 | kashyap | gibi: Ahh, I still have to fix 3-4 unit tests; just noticed. :( | |
| 12:51:35 | gibi | kashyap: no worries, ping me and I can re-review | |
| 12:51:46 | kashyap | Thanks | |
| 13:22:07 | opendevreview | Merged openstack/nova master: Split ignored_tags in stats.py https://review.opendev.org/c/openstack/nova/+/867978 | |
| 13:25:22 | opendevreview | Rajesh Tailor proposed openstack/nova master: Handle InstanceInvalidState exception https://review.opendev.org/c/openstack/nova/+/861738 | |
| 13:33:43 | opendevreview | Elod Illes proposed openstack/nova stable/xena: nova-live-migration tests not needed for Ironic https://review.opendev.org/c/openstack/nova/+/855469 | |
| 14:18:18 | bauzas | artom: verifying now | |
| 14:20:23 | bauzas | artom: sending to the gate | |
| 14:22:33 | artom | Cheers! | |
| 15:15:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Factor out a mixin class for candidate aware filters https://review.opendev.org/c/openstack/nova/+/854929 | |
| 15:16:36 | kashyap | (It's failing as a result of my: https://review.opendev.org/c/openstack/nova/+/870794) | |
| 15:18:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Show candidate combinatorial explosion by dev number https://review.opendev.org/c/openstack/nova/+/855885 | |
| 15:28:11 | gibi | kashyap: noted the issue in https://review.opendev.org/c/openstack/nova/+/870794/7#message-a9e217530aaca46763329444a8776bc1eb1948d8 | |
| 15:29:08 | kashyap | gibi: Duuh, I thought I was careful w/ the spaces and still messed | |
| 15:29:46 | kashyap | Thanks for the eagle eyes, /me goes to fix... | |
| 15:35:34 | 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 | |
| 15:36:07 | 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 | |
| 15:38:10 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | |
| 15:38:10 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: At start-up rework compareCPU() usage with a workaround https://review.opendev.org/c/openstack/nova/+/870794 | |
| 15:43:21 | kashyap | Hmm, one more UT to debug (unrelated to the formatting issue). | |
| 15:57:58 | opendevreview | Dan Smith proposed openstack/nova master: Pass service ref to init_host(), if exists https://review.opendev.org/c/openstack/nova/+/863916 | |
| 15:57:58 | opendevreview | Dan Smith proposed openstack/nova master: Add virt/node module for stable uuids https://review.opendev.org/c/openstack/nova/+/863915 | |
| 15:57:59 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 15:57:59 | opendevreview | Dan Smith proposed openstack/nova master: Add get_available_node_uuids() to virt driver https://review.opendev.org/c/openstack/nova/+/863917 | |
| 15:58:00 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 15:58:00 | opendevreview | Dan Smith proposed openstack/nova master: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 16:07:18 | opendevreview | Dan Smith proposed openstack/nova master: Make resource tracker use UUIDs instead of names https://review.opendev.org/c/openstack/nova/+/863919 | |
| 16:07:18 | opendevreview | Dan Smith proposed openstack/nova master: Persist existing node uuids locally https://review.opendev.org/c/openstack/nova/+/863918 | |
| 16:07:19 | opendevreview | Dan Smith proposed openstack/nova master: WIP: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 16:18:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Factor out a mixin class for candidate aware filters https://review.opendev.org/c/openstack/nova/+/854929 | |
| 16:44:21 | opendevreview | David Hill proposed openstack/nova master: Change maximum value of pcie-ports to 64 https://review.opendev.org/c/openstack/nova/+/871305 | |
| 16:47:42 | gibi | bauzas: fyi, all the mandatory pieces of bp/pci-device-tracking-in-placement has been landed. So the bp can be set completed in A. | |
| 16:48:21 | gibi | there are two open patches https://review.opendev.org/q/topic:bp/pci-device-tracking-in-placement+status:open one is a small refactor that might or might not land. The other is just a demonstrator that we have a "limitation" | |
| 16:51:52 | sean-k-mooney | congrats | |
| 16:58:15 | gibi | It would have been impossible without sean-k-mooney's and stephenfin's help. thank you! | |
| 17:12:00 | opendevreview | David Hill proposed openstack/nova master: Change maximum value of pcie-ports to 64 https://review.opendev.org/c/openstack/nova/+/871305 | |
| 17:35:03 | opendevreview | David Hill proposed openstack/nova master: Change maximum value of pcie-ports to 32 https://review.opendev.org/c/openstack/nova/+/871305 | |
| #openstack-nova - 2023-01-21 | |||
| 00:22:09 | opendevreview | melanie witt proposed openstack/nova master: [WIP] add healthcheck manager to manager base https://review.opendev.org/c/openstack/nova/+/827844 | |
| 00:22:09 | opendevreview | melanie witt proposed openstack/nova master: [WIP] add initial healthcheck support https://review.opendev.org/c/openstack/nova/+/825015 | |
| 00:22:10 | opendevreview | melanie witt proposed openstack/nova master: [WIP] add healthcheck utils and constants https://review.opendev.org/c/openstack/nova/+/829469 | |
| 00:22:10 | opendevreview | melanie witt proposed openstack/nova master: [WIP] add healthcheck tracker to nova context https://review.opendev.org/c/openstack/nova/+/829468 | |
| 00:22:11 | opendevreview | melanie witt proposed openstack/nova master: add healthcheck endpoint to proxy commands https://review.opendev.org/c/openstack/nova/+/830703 | |
| 05:39:22 | opendevreview | Merged openstack/nova master: Microversion 2.94: FQDN in hostname https://review.opendev.org/c/openstack/nova/+/869812 | |
| 15:57:35 | jpic | hi all, any idea why this VM won't boot anymore after being resized from 32 to 64 vCPU? It seems to complain about memory but the hypervisor seems to have way enough, boot logs: https://dpaste.org/b5Cis | |
| #openstack-nova - 2023-01-23 | |||
| 05:12:59 | opendevreview | Amit Uniyal proposed openstack/nova stable/zed: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871413 | |
| 05:14:37 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871414 | |
| 05:15:43 | opendevreview | Amit Uniyal proposed openstack/nova stable/xena: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871415 | |
| 05:16:41 | opendevreview | Amit Uniyal proposed openstack/nova stable/wallaby: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871416 | |
| 05:17:58 | opendevreview | Amit Uniyal proposed openstack/nova stable/victoria: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871417 | |
| 05:19:41 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871419 | |
| 06:09:36 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871444 | |
| 06:26:41 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Improving logging at '_allocate_mdevs'. https://review.opendev.org/c/openstack/nova/+/871444 | |
| 07:52:13 | opendevreview | Rajesh Tailor proposed openstack/nova master: Handle InstanceInvalidState exception https://review.opendev.org/c/openstack/nova/+/861738 | |
| 09:47:50 | opendevreview | Rajesh Tailor proposed openstack/nova master: Handle InstanceInvalidState exception https://review.opendev.org/c/openstack/nova/+/861738 | |
| 10:00:28 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 10:00:28 | 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 | |
| 10:07:58 | bauzas | gibi: so, wants me to move https://blueprints.launchpad.net/nova/+spec/pci-device-tracking-in-placement to Implemented so , | |