Wednesday, 2022-03-09

*** dasm is now known as dasm|out03:11
*** ministry is now known as __ministry03:50
ygk_12345hi all06:55
ygk_12345can we change a flavor from public to private ?06:55
ygk_12345can we convert a private flavor to public ?07:08
gibiygk_12345: I think you cannot change it to public, but you can add tenants to the access list to see the private flavor07:08
ygk_12345gibi: ok07:08
* bauzas is sleepy today, water supply outage since 6.30am08:26
bauzasfortunately, my coffee machine has water in its tank :)08:26
dmitriisgibi: o/ if you have time https://review.opendev.org/c/openstack/nova/+/82997408:34
rribaud__bauzas, hello, I hope it will be fixed soon.08:38
bauzasrribaud__: hopefully :crossed_fingers!08:39
* bauzas reconsiders to use the Isere river08:39
kashyapbauzas: Bonjour.  I have a soft -1 with a suggestion on the Yoga prelude.  :)08:40
bauzaskashyap: /me looks08:40
rribaud__bauzas, usually water maintenance people are reactive, that should not be long.08:40
bauzasrribaud__: actually, this is fixed by the last hour but the pressure is bad08:41
kashyapbauzas: No rush, just a phrasing part08:42
bauzasrevising it 08:43
bauzaskashyap: thanks for your comments, good explanation why we don't want it for production use08:46
kashyapNo prob08:46
opendevreviewSylvain Bauza proposed openstack/nova master: Add the Yoga prelude section  https://review.opendev.org/c/openstack/nova/+/83229208:47
bauzasthere it goes... 08:47
bauzasdansmith: gibi: amended the prelude based on dan's comment08:47
bauzaselodilles: today and tomorrow will be busy days for merging things08:49
bauzaselodilles: I guess we will only be able to amend the nova rc1 yaml by the end of our day 08:50
bauzas(tomorrow I mean)08:50
bauzashah, all water issues fixed, let me visit my shower :)08:53
* bauzas is back in 20 mins-ish08:53
opendevreviewribaudr proposed openstack/os-traits master: Attach Manila shares via virtiofs(traits)  https://review.opendev.org/c/openstack/os-traits/+/83276909:26
elodillesbauzas: ack, if the patch is -1'd then the release managers won't force-approve the patch without pinging you ;)09:38
bauzasyah I know09:38
bauzaselodilles: but I wanted to explain we need to merge a few changes before removing the -109:39
bauzasso it would be a bit late tomorrow09:39
elodillesbauzas: cool, roger :)09:39
bauzas(as we need to wait for the merge)09:39
opendevreviewribaudr proposed openstack/os-traits master: Add 'COMPUTE_STORAGE_VIRTIO_FS', 'COMPUTE_MEM_BACKING_FILE'  https://review.opendev.org/c/openstack/os-traits/+/83276909:39
elodillesi'll be around, so no problem :)09:39
elodillesand thanks for the heads-up09:40
bauzaselodilles: haha, https://www.youtube.com/watch?v=IC7pZSd8iRg09:41
elodillesyepp, that's classic ;)09:56
rribaud__hello, question, I have modified os-trait locally. What is the best way to use it with tox (nova) ?10:22
opendevreviewMerged openstack/nova master: Follow up for unified limits  https://review.opendev.org/c/openstack/nova/+/83091810:33
*** brinzhang_ is now known as brinzhang10:39
ygk_12345Folks, in which database is the nova flavor details stored ?11:27
ygk_12345for all the created flavors I mean11:27
ygk_12345any idea ?11:36
rribaud__ygk_12345, I think it is in nova_api  --> flavors table11:40
ygk_12345rribaud__: thanks for the info11:51
* rribaud__ finally manage to use my local os-traits by specifying it into requirements.txt and removing the tox constraints. Maybe there is a better solution but at least it works.12:31
opendevreviewribaudr proposed openstack/os-traits master: Add 'COMPUTE_STORAGE_VIRTIO_FS', 'COMPUTE_MEM_BACKING_FILE'  https://review.opendev.org/c/openstack/os-traits/+/83276912:35
sean-k-mooney1rribaud__: you just do .tox/py3/bin/pip install ../os-traits12:55
sean-k-mooney1so after you have created the tox env if you need a local os-traits just install it and run tox again12:56
sean-k-mooney1or run the command tox would run manuatlly 12:56
sean-k-mooney1.tox/py3/bin/stestr ...12:56
*** dasm|out is now known as dasm13:35
opendevreviewMerged openstack/nova master: Adds regression test for bug LP#1944619  https://review.opendev.org/c/openstack/nova/+/82184013:45
*** iurygregory_ is now known as iurygregory13:48
rribaud__sean-k-mooney1, thx. 13:53
rribaud__sean-k-mooney1++13:53
*** sean-k-mooney1 is now known as sean-k-mooney14:02
opendevreviewAlexey Stupnikov proposed openstack/nova master: Add functional tests to reproduce bug #1960412  https://review.opendev.org/c/openstack/nova/+/83001014:08
opendevreviewAlexey Stupnikov proposed openstack/nova master: Clean up when queued live migration aborted  https://review.opendev.org/c/openstack/nova/+/82857014:17
erlonbauzas: hey, in the pre_live_migration fix, what you mean with 'Can you please provide a FUP for this', to implement seans' suggestion?14:20
bauzaserlon: sorry, FUP means "a follow-up patch"14:27
bauzaserlon: that means I can +W this patch if you agree with providing a separate patch for modifying the reno file14:27
erlonbauzas: oh, ok, sure, Ill do and address the other comments too14:27
bauzaserlon: thanks14:27
bauzaserlon: we can merge this FUP before tomorrow14:28
*** hemna7 is now known as hemna14:28
bauzas(before RC1)14:28
erlonright, that was my only concern14:28
*** hemna5 is now known as hemna14:47
bauzaserlon: I don't yet see your FUP ?15:23
erlonbauzas: was in a meeting15:24
bauzaserlon: ok, just ping me when you send it 15:27
bauzasand then I'll +W the original patch15:27
opendevreviewErlon R. Cruz proposed openstack/nova master: Fix pre_live_migration rollback  https://review.opendev.org/c/openstack/nova/+/81532415:33
erlonbauzas: ^15:34
bauzasthanks15:34
bauzaserlon: oh you made a new revision15:35
bauzasanyway, will +2/+W it then15:35
bauzassean-k-mooney: ^15:35
erlonbauzas: I was wondering about that, that a new ps would trigger all CI runs15:36
bauzaserlon: it will15:36
erlonbauzas: how did I suppose to do to avoid that?15:36
bauzaserlon: but it should be OK 15:36
bauzaserlon: by providing a new change 15:36
bauzasor a patch15:36
erlonow, depended on that one15:36
erlongot it15:36
bauzasyup15:36
bauzasanyway, +Wd15:36
bauzasso the CI will verify it twice15:37
erlonok, it should be good for tomorrow15:37
bauzasshould be OK15:37
bauzasyes15:37
erlonthanks for helping with that15:37
sean-k-mooneyrribaud__: by the way do you prefer that nic or uggla15:45
sean-k-mooneyerlon: bauzas  i quickly reviewd it and +2'd it also15:46
sean-k-mooneyerlon: thanks for following up with us :)15:46
rribaud__sean-k-mooney, uggla one, it changes following a disconnect.15:46
*** rribaud__ is now known as Uggla15:46
sean-k-mooneycool15:47
sean-k-mooneyi havent fully memorised your name yet so was not sure which one you wanted to go by15:47
* Uggla recover me nickname15:48
UgglaI try to use rribaud on internal and Uggla external15:49
Ugglato make a clear diff and avoid to disclose internal stuff. :)15:49
sean-k-mooneyya some do that15:49
sean-k-mooneyi prefer to use the same for both15:49
Ugglabut for strange reason after disconnect, reconnect pick my internal one.15:50
bauzasgibi: I'm doing a bit of rounding about the RC1 bugs https://bugs.launchpad.net/nova/+bugs?field.tag=yoga-rc-potential17:08
bauzassean-k-mooney: ^17:08
bauzasI'm marking the according patches as review-prio17:08
bauzasfor the open bugs where no core hasn't reviewed yet, I'll descope them from the RC1 tag tomorrow morning17:09
* Uggla eating daifuku (moshi) + tea. This is so good !17:09
Ugglayuzu + lemon rules.17:09
opendevreviewribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve  https://review.opendev.org/c/openstack/python-novaclient/+/83165117:13
melwittbauzas: not related to RC1 but on bugs, I have a proposed change for a gate failure bug here if you could lmk if you see any issues with it https://review.opendev.org/c/openstack/nova/+/83108417:15
bauzasmelwitt: sure17:15
melwittty17:16
bauzasmelwitt: looked at your patch, needs my full brain for it17:44
melwittbauzas: np, it's not urgent17:44
bauzasmelwitt: will look at it once I'm rejuverated tomorrow morning :)17:45
bauzasrejuvenated* :)17:45
melwittthanks :)17:45
whoami-rajatdansmith, hey, the rebuild test passed \o/ https://review.opendev.org/c/openstack/tempest/+/83101817:47
opendevreviewmelanie witt proposed openstack/nova stable/ussuri: Reproduce bug 1953359  https://review.opendev.org/c/openstack/nova/+/82204717:50
opendevreviewmelanie witt proposed openstack/nova stable/ussuri: Extend the reproducer for 1953359 and 1952915  https://review.opendev.org/c/openstack/nova/+/82204817:50
opendevreviewmelanie witt proposed openstack/nova stable/ussuri: [rt] Apply migration context for incoming migrations  https://review.opendev.org/c/openstack/nova/+/82205017:51
*** dasm is now known as dasm|bbl18:26
elodillesmelwitt: could you please have a quick look at this stable/queens gate fix? periodic job is failing on it and it would be nice to make the error go away o:) https://review.opendev.org/c/openstack/nova/+/83032719:04
melwittelodilles: sure thing19:14
*** hemna6 is now known as hemna20:29
opendevreviewMerged openstack/nova master: reenable greendns in nova.  https://review.opendev.org/c/openstack/nova/+/83096621:47
opendevreviewMerged openstack/nova stable/queens: [stable-only] Fix sphinx doc job  https://review.opendev.org/c/openstack/nova/+/83032721:48
opendevreviewArtom Lifshitz proposed openstack/nova master: Update PCI requests in request spec on resize  https://review.opendev.org/c/openstack/nova/+/80604922:48
*** artom__ is now known as artom22:49

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