Friday, 2022-05-27

opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: WIP: Persist connection_info during live migration  https://review.opendev.org/c/openstack/nova/+/84355400:05
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: WIP: Persist connection_info during live migration  https://review.opendev.org/c/openstack/nova/+/84355400:31
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: DNM: Testing live migration with local attach  https://review.opendev.org/c/openstack/nova/+/84314600:31
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: DNM: Testing live migration with local attach  https://review.opendev.org/c/openstack/nova/+/84314600:38
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: WIP: Persist connection_info during live migration  https://review.opendev.org/c/openstack/nova/+/84355400:38
opendevreviewGhanshyam proposed openstack/nova stable/victoria: DNM: testing Tempest pin in stable/victoria  https://review.opendev.org/c/openstack/nova/+/84330102:04
opendevreviewRajesh Tailor proposed openstack/nova master: Add missing condition  https://review.opendev.org/c/openstack/nova/+/84356504:25
opendevreviewRajesh Tailor proposed openstack/nova master: Fix typos  https://review.opendev.org/c/openstack/nova/+/84312704:27
*** bhagyashris is now known as bhagyashris|ruck05:08
opendevreviewRajesh Tailor proposed openstack/nova master: Fix typos  https://review.opendev.org/c/openstack/nova/+/84312710:32
opendevreviewRajesh Tailor proposed openstack/nova master: Fix typos  https://review.opendev.org/c/openstack/nova/+/84312710:50
opendevreviewRajesh Tailor proposed openstack/nova master: Fix duplicates keys  https://review.opendev.org/c/openstack/nova/+/84360511:54
opendevreviewMerged openstack/nova master: Fix extending non LUKSv1 encrypted volumes  https://review.opendev.org/c/openstack/nova/+/83606412:10
opendevreviewArtom Lifshitz proposed openstack/nova master: WIP: Run encrypted volume tests in nova-live-migration-ceph  https://review.opendev.org/c/openstack/nova/+/84362114:02
artomsean-k-mooney, if I've done this correct, ^^ should reproduce on master14:03
artomWithout any workarounds14:03
opendevreviewArtom Lifshitz proposed openstack/nova master: WIP: Run encrypted volume tests in nova-live-migration  https://review.opendev.org/c/openstack/nova/+/84362115:57
opendevreviewGhanshyam proposed openstack/os-vif master: Drop lower-constraints.txt and its testing  https://review.opendev.org/c/openstack/os-vif/+/84002018:54
artomsean-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 it19:07
artomAnd I even reviewed it, too19:07
artomYou may be done for the day, feel free to ignore this19:07
artomI'm using (potentially absent) you as a rubber duck (again)19:08
artomLemme try backporting that to stable/wallaby instead of my WIP thing19:09
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: fake: Ensure need_legacy_block_device_info returns False  https://review.opendev.org/c/openstack/nova/+/84367819:19
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545  https://review.opendev.org/c/openstack/nova/+/84367919:19
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved  https://review.opendev.org/c/openstack/nova/+/84368019:19
sean-k-mooneyartom: o/19:19
sean-k-mooneyartom: im not around for long19:19
sean-k-mooneybut ill look quickly19:20
sean-k-mooneyartom: ah yes cool19:20
sean-k-mooneyand there is a functional repoducer https://review.opendev.org/c/openstack/nova/+/804229/319:21
sean-k-mooneyartom: ya so i think that is a better approch vs stable only19:21
artomFor sure19:22
sean-k-mooneyartom: nice find19:22
artom:) I tried on master with iscsi + cryptsetup, which should have hit the same problem19:22
artomIt didn't19:22
artomSo I went git log diving19:22
artomLuckily, Lee's commit message was well written and was easy to find.19:23
sean-k-mooneyso thats in xena so ya wallby to trait i guess19:23
sean-k-mooney*to train19:23
sean-k-mooneyfor the backports19:23
* artom already hates backporting stuff to stable/train, and 17 isn't even out yet19:24
artomWe're going to burn out the SRT team (well, our 1 Nova dude Amit) in a matter of days19:24
artomLike, he'll need therapy19:24
artomI almost do19:24
sean-k-mooneyit hopefuly wont be too bad of a backpor but the fact we ned py27 suport for trian is a pain19:25
sean-k-mooneyeven wallaby seam kind of old to me honestly19:25
artomThe code itself is mostly OK19:26
sean-k-mooneyanyway im going ot finish up for the weekend but ill check it on monday19:26
artomBut the price to pay for all the func tests improvements since train is backports being a PITA19:26
sean-k-mooneyya19:26
opendevreviewmelanie witt proposed openstack/nova master: Update [scheduler] limit_tenants_to_placement_aggregate config help  https://review.opendev.org/c/openstack/nova/+/84368419:59
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: fake: Ensure need_legacy_block_device_info returns False  https://review.opendev.org/c/openstack/nova/+/84367820:48
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545  https://review.opendev.org/c/openstack/nova/+/84367920:48
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved  https://review.opendev.org/c/openstack/nova/+/84368020:48
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545  https://review.opendev.org/c/openstack/nova/+/84367921:09
opendevreviewArtom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved  https://review.opendev.org/c/openstack/nova/+/84368021:09

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