| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-05-26 | |||
| 13:41:39 | sean-k-mooney | if we want to make it release independent then sure | |
| 13:43:58 | opendevreview | sean mooney proposed openstack/os-vif master: update job template to zed https://review.opendev.org/c/openstack/os-vif/+/843432 | |
| 13:44:04 | sean-k-mooney | actully some typos fixed ^ | |
| 13:44:20 | sean-k-mooney | ill leave it for now for ye to review but if you want me to change anything i can | |
| 13:44:49 | sean-k-mooney | stephenfin: by the way your dont happen to be a bindep core? | |
| 13:45:15 | gibi | sean-k-mooney: so we say py 3.10 is supported, which probably works, but I'm not sure the overall Zed release will go out with py3.10 supportr | |
| 13:45:41 | sean-k-mooney | gibi: debain and ubuntu ship yoga on 3.10 | |
| 13:46:05 | sean-k-mooney | so zed will be released on at least 3.10 by them | |
| 13:46:27 | sean-k-mooney | its technially not in the testing runtimes but we have tests using it so i think it will be fine | |
| 13:46:27 | gibi | yeah, hence my statement that it probably works without problem | |
| 13:46:43 | gibi | OK, I'm convinced +@ | |
| 13:46:44 | gibi | +2 | |
| 13:46:53 | sean-k-mooney | i can drop that from the list if you like untile we have a 22.04 based job | |
| 13:47:01 | sean-k-mooney | but i could also do that as a follow up | |
| 13:47:16 | sean-k-mooney | i.e. add a 22.04 job | |
| 13:47:18 | gibi | no, it is OK, we run py3.10 on os-vif so we know it works | |
| 13:47:27 | sean-k-mooney | cool | |
| 13:49:21 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: DNM: Testing live migration with local attach https://review.opendev.org/c/openstack/nova/+/843146 | |
| 14:50:17 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/843336 | |
| 14:50:18 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Fix LM rollback w/o multi port bindings extension https://review.opendev.org/c/openstack/nova/+/843337 | |
| 14:58:23 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Reproduce live migration rollback w/o multi port bindings error https://review.opendev.org/c/openstack/nova/+/843338 | |
| 14:58:24 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Fix LM rollback w/o multi port bindings extension https://review.opendev.org/c/openstack/nova/+/843339 | |
| 15:34:59 | kashyap | gibi: Hey, before I go out on PTO I can't get this to completion this week :-( - https://review.opendev.org/q/topic:bp%252Fcpu-selection-with-hypervisor-consideration | |
| 15:35:53 | gibi | kashyap: is there anything I can help with there? | |
| 15:36:56 | kashyap | gibi: Stephen has a bunch of valid comments; need to address those and get it in. | |
| 15:37:05 | gibi | ack | |
| 15:38:29 | kashyap | gibi: Also, one more is the separate config class to parse the "cpu" bit from `virsh domcapabilitles` | |
| 15:38:53 | gibi | yeah I remember | |
| 15:38:57 | gibi | that | |
| 15:39:14 | kashyap | Also have to address your own good comments on this patch: https://review.opendev.org/c/openstack/nova/+/762330 | |
| 15:40:40 | kashyap | gibi: Ah, forgot that I already have the patch here for: https://review.opendev.org/c/openstack/nova/+/838191 (libvirt/config: Parse the 'cpu' element in domainCapabilities) | |
| 15:40:45 | kashyap | (Modulo test) | |
| 15:41:27 | gibi | yeah only test needed | |
| 15:41:44 | kashyap | Actually, no that's not it...it's not adding a separate config class there | |
| 15:42:34 | kashyap | gibi: From IRC log | |
| 15:42:46 | kashyap | (Of this channel): | |
| 15:42:47 | kashyap | 17:13 < kashyap> gibi: To tie up the lose end on caps vs domCaps -- the guidance from the libvirt folks is (a) no, we can't treat the caps == domCaps w/ 'host-model' mode; and (b) we should use domCaps wherever possible. | |
| 15:42:51 | kashyap | 17:13 < kashyap> gibi: So that means, we should introduce a new config object | |
| 15:42:54 | kashyap | 17:13 < gibi> kashyap: ack, make sense | |
| 15:43:19 | gibi | hehh, I'm just following you guid here :) | |
| 15:43:30 | kashyap | So, to sum up: | |
| 15:44:14 | kashyap | (1) address your + Stephen's comments; (2) add the separate config object class to parse "cpu" element from domCaps, and work that in "properly" | |
| 15:44:45 | kashyap | I could definitely use some help, if you have some time for this! I keep duking around this and never quite managing it fully | |
| 15:45:57 | gibi | I cannot promise I can jump on this right away but I keep it in mind | |
| 15:46:15 | gibi | but thanks for summing it up it helps keeping the context | |
| 17:44:21 | opendevreview | Merged openstack/os-vif master: update job template to zed https://review.opendev.org/c/openstack/os-vif/+/843432 | |
| 18:41:43 | opendevreview | Rajat Dhasmana proposed openstack/nova-specs master: Repropose volume backed server rebuild spec https://review.opendev.org/c/openstack/nova-specs/+/840155 | |
| 19:28:14 | opendevreview | Miguel Lavalle proposed openstack/os-vif master: Delete trunk bridges to avoid race with Neutron https://review.opendev.org/c/openstack/os-vif/+/841499 | |
| #openstack-nova - 2022-05-27 | |||
| 00:05:09 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: WIP: Persist connection_info during live migration https://review.opendev.org/c/openstack/nova/+/843554 | |
| 00:31:46 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: WIP: Persist connection_info during live migration https://review.opendev.org/c/openstack/nova/+/843554 | |
| 00:31:47 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: DNM: Testing live migration with local attach https://review.opendev.org/c/openstack/nova/+/843146 | |
| 00:38:33 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: DNM: Testing live migration with local attach https://review.opendev.org/c/openstack/nova/+/843146 | |
| 00:38:34 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: WIP: Persist connection_info during live migration https://review.opendev.org/c/openstack/nova/+/843554 | |
| 02:04:53 | opendevreview | Ghanshyam proposed openstack/nova stable/victoria: DNM: testing Tempest pin in stable/victoria https://review.opendev.org/c/openstack/nova/+/843301 | |
| 04:25:57 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add missing condition https://review.opendev.org/c/openstack/nova/+/843565 | |
| 04:27:49 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos https://review.opendev.org/c/openstack/nova/+/843127 | |
| 10:32:38 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos https://review.opendev.org/c/openstack/nova/+/843127 | |
| 10:50:09 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos https://review.opendev.org/c/openstack/nova/+/843127 | |
| 11:54:28 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix duplicates keys https://review.opendev.org/c/openstack/nova/+/843605 | |
| 12:10:09 | opendevreview | Merged openstack/nova master: Fix extending non LUKSv1 encrypted volumes https://review.opendev.org/c/openstack/nova/+/836064 | |
| 14:02:58 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Run encrypted volume tests in nova-live-migration-ceph https://review.opendev.org/c/openstack/nova/+/843621 | |
| 14:03:12 | artom | sean-k-mooney, if I've done this correct, ^^ should reproduce on master | |
| 14:03:17 | artom | Without any workarounds | |
| 15:57:53 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Run encrypted volume tests in nova-live-migration https://review.opendev.org/c/openstack/nova/+/843621 | |
| 18:54:19 | opendevreview | Ghanshyam proposed openstack/os-vif master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/os-vif/+/840020 | |
| 19:07:18 | artom | sean-k-mooney, so, I can't reproduce it on master, and I think that's because https://review.opendev.org/c/openstack/nova/+/804230/3/nova/compute/manager.py fixed it | |
| 19:07:26 | artom | And I even reviewed it, too | |
| 19:07:43 | artom | You may be done for the day, feel free to ignore this | |
| 19:08:01 | artom | I'm using (potentially absent) you as a rubber duck (again) | |
| 19:09:42 | artom | Lemme try backporting that to stable/wallaby instead of my WIP thing | |
| 19:19:25 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/843678 | |
| 19:19:26 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843679 | |
| 19:19:27 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843680 | |
| 19:19:42 | sean-k-mooney | artom: o/ | |
| 19:19:51 | sean-k-mooney | artom: im not around for long | |
| 19:20:04 | sean-k-mooney | but ill look quickly | |
| 19:20:36 | sean-k-mooney | artom: ah yes cool | |
| 19:21:10 | sean-k-mooney | and there is a functional repoducer https://review.opendev.org/c/openstack/nova/+/804229/3 | |
| 19:21:45 | sean-k-mooney | artom: ya so i think that is a better approch vs stable only | |
| 19:22:15 | artom | For sure | |
| 19:22:18 | sean-k-mooney | artom: nice find | |
| 19:22:48 | artom | :) I tried on master with iscsi + cryptsetup, which should have hit the same problem | |
| 19:22:50 | artom | It didn't | |
| 19:22:56 | artom | So I went git log diving | |
| 19:23:08 | artom | Luckily, Lee's commit message was well written and was easy to find. | |
| 19:23:15 | sean-k-mooney | so thats in xena so ya wallby to trait i guess | |
| 19:23:35 | sean-k-mooney | *to train | |
| 19:23:43 | sean-k-mooney | for the backports | |
| 19:24:29 | artom | We're going to burn out the SRT team (well, our 1 Nova dude Amit) in a matter of days | |
| 19:24:42 | artom | Like, he'll need therapy | |
| 19:24:44 | artom | I almost do | |
| 19:25:15 | sean-k-mooney | it hopefuly wont be too bad of a backpor but the fact we ned py27 suport for trian is a pain | |
| 19:25:49 | sean-k-mooney | even wallaby seam kind of old to me honestly | |
| 19:26:02 | artom | The code itself is mostly OK | |
| 19:26:13 | sean-k-mooney | anyway im going ot finish up for the weekend but ill check it on monday | |
| 19:26:24 | artom | But the price to pay for all the func tests improvements since train is backports being a PITA | |
| 19:26:53 | sean-k-mooney | ya | |
| 19:59:25 | opendevreview | melanie witt proposed openstack/nova master: Update [scheduler] limit_tenants_to_placement_aggregate config help https://review.opendev.org/c/openstack/nova/+/843684 | |
| 20:48:33 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/843678 | |
| 20:48:34 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843679 | |