Monday, 2022-02-28

opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036803:59
*** amoralej|off is now known as amoralej07:24
*** bhagyashris_ is now known as bhagyashris|ruck07:56
opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036808:18
opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036808:36
elodillesbauzas: hi, if you have time, could you please review this WA patch for stable/wallaby? https://review.opendev.org/c/openstack/nova/+/83065609:28
bauzaselodilles: sure, looking10:09
elodillesthanks o/10:29
opendevreviewribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs(db+object)  https://review.opendev.org/c/openstack/nova/+/83119310:44
opendevreviewribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs(manila abstraction)  https://review.opendev.org/c/openstack/nova/+/83119410:44
opendevreviewBalazs Gibizer proposed openstack/nova stable/xena: Add service version check workaround for FFU  https://review.opendev.org/c/openstack/nova/+/83117411:14
gibimelwitt: thanks for proposing the fix for the DB fixture. I left a comment. I haven't had time to dig deep into the problem so maybe my comment is not valid11:23
gibibauzas: there are two housekeeping patches for nova https://review.opendev.org/c/openstack/nova/+/831075 https://review.opendev.org/c/openstack/nova/+/831077 we need these before RC1 12:50
gibiand there will be one more related housekeeping but after RC1 to do this for Yoga->Zed  https://review.opendev.org/c/openstack/nova/+/80993212:52
sean-k-mooneyah the rpc alias12:59
sean-k-mooneygibi: unified limist has mostly merged did that not ahve a microverion in it?13:00
gibiit did not13:00
sean-k-mooneyhuh ok13:00
gibiunified limits is a configurable limits driver13:00
gibi* quota driver13:00
gibiso there is no meaningful way to ask the old behavior with an old microversion 13:01
gibias the behavior is config driven13:01
sean-k-mooneyack that makes sense13:02
sean-k-mooneyi tought we were going to have one  at one point but i understand now why. did we disucss having one if we revometed the proxy apis?13:02
gibiI'm not aware of such discussion13:03
sean-k-mooneyok i just tought we discussed one at one point but maybe the out come was it did not make sense for the reasons you gave above13:03
sean-k-mooneyanyway in that case both of those patches look fine to me. did you want bauzas specificly to approve them or will i just review them and send them on there way13:04
gibihm, the PTL guide says that the RPC alias is more appropriate just before RC113:07
gibiso lets wait a bit13:07
sean-k-mooneyack13:08
gibiI think the microversion doc is OK to go in as that is only documentation13:08
sean-k-mooneyi have +2'd them but ill leave teh +w to bauzas 13:08
gibiack, thanks13:09
opendevreviewMerged openstack/nova stable/wallaby: workarounds: Add libvirt_disable_apic  https://review.opendev.org/c/openstack/nova/+/80562813:12
*** dasm|off is now known as dasm|rover13:33
opendevreviewTakashi Kajinami proposed openstack/nova master: Add oslo.limit options to generated nova.conf  https://review.opendev.org/c/openstack/nova/+/83110114:22
bauzassean-k-mooney: gibi: looking at them14:36
bauzasdone14:38
bauzasyuval: could you please update https://review.opendev.org/c/openstack/nova/+/830817 soon with sean-k-mooney's comment ?15:00
opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036815:00
yuvalhey yes, right away15:01
bauzasyuval: thanks15:02
bauzassean-k-mooney: your help is appreciated on https://review.opendev.org/c/openstack/nova/+/82837215:02
bauzassean-k-mooney: looks to me good but I could have missed something15:02
opendevreviewyuval proposed openstack/nova master: Lightos driver release note  https://review.opendev.org/c/openstack/nova/+/83081715:08
yuvalbauzas: ^updated15:09
bauzasyuval: great, thanks15:09
opendevreviewRajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild  https://review.opendev.org/c/openstack/nova/+/83121915:10
chateaulavbauzas: thanks very much!16:05
opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036816:09
opendevreviewRajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild  https://review.opendev.org/c/openstack/nova/+/83121916:09
opendevreviewRajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances  https://review.opendev.org/c/openstack/nova/+/83088316:09
opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036816:14
opendevreviewRajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild  https://review.opendev.org/c/openstack/nova/+/83121916:16
opendevreviewRajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances  https://review.opendev.org/c/openstack/nova/+/83088316:16
opendevreviewMerged openstack/nova master: Update compute rpc version alias for yoga  https://review.opendev.org/c/openstack/nova/+/83107716:32
opendevreviewMerged openstack/nova master: doc: mark the max microversion for yoga  https://review.opendev.org/c/openstack/nova/+/83107516:32
opendevreviewDan Smith proposed openstack/nova master: Add grenade-skip-level irrelevant-files config  https://review.opendev.org/c/openstack/nova/+/83122916:34
dansmithgmann: ^16:34
gmanndansmith: ack16:36
gibibauzas: did we missed the cycle highlight deadline for yoga?17:01
bauzasorly ?17:01
bauzasman, I missed them17:02
* bauzas will rush writing some17:02
gibihttp://lists.openstack.org/pipermail/openstack-discuss/2022-February/027361.html17:02
gibime too17:02
bauzasyeah the ff trampled me under the bus17:03
* bauzas should amend the Nova PTL doc to mention it17:03
bauzasgibi: it was under my radar as you always proposed some17:03
gibigood point17:03
gibi(the PTL doc update)_17:03
bauzaspriority #1, I'm working on the cycle highlights17:04
bauzasgibi: anything in particular you'd like to see mentioned ?17:04
bauzasI was about to mention the policy thing, unified limits, the ARM support17:04
gibiunified limits so people can try and provide feedback, 17:04
bauzasyeah17:04
bauzasthis one is obvious17:04
gibiyeah, policy and emulation both good to mention too17:04
bauzasdon't know if the lightbits os thingy is marked on the cinder side17:05
bauzaselodilles: I apologize about missing the cycle highlights, writing such things as we speak17:06
bauzasdiablo_rojo isn't in the chat17:06
bauzassean-k-mooney: around ? could you tell me a one-liner for https://review.opendev.org/c/openstack/nova-specs/+/787458/14/specs/yoga/approved/integration-with-off-path-network-backends.rst ?17:27
* bauzas writes the cycle highlights17:27
sean-k-mooneyall of the nova part is done17:28
bauzascan we say it's a cycle highlight ?17:29
bauzassean-k-mooney: still around ?17:50
sean-k-mooneyyes17:50
sean-k-mooneyoh sec17:51
sean-k-mooneysorry was talking to artom and kashyap on a call17:52
sean-k-mooneybauzas: ill quick chekc if the neutron part is merged17:52
sean-k-mooneybut yes i think it should be a release highlight17:52
sean-k-mooneyi assume you are working on the release prelude17:52
sean-k-mooneybauzas: the neutron part is tracked under this RFE bug https://review.opendev.org/q/topic:bug%252F193215417:54
sean-k-mooneyand it is all merged17:54
sean-k-mooneyas of 5 days ago17:54
* bauzas tries to summarize in one line 17:54
bauzasthat's for the cycle highlight17:54
bauzasnot the prelude yet17:54
sean-k-mooney"In the Yoga release support was added for network backends that leverage smartnics to offload the contolplane form the host server"17:55
sean-k-mooney"this enables increased securtiy by removing the contol plane form the host server and reduced overhaed by leveraging the cpu and ram resouces on modern smart nics"17:56
sean-k-mooneybauzas: somethign like that ^17:56
bauzasthanks17:57
elodillesbauzas: no problem, i think we are still in time with the cycle highlights :)18:00
* bauzas rushes to upload them18:02
bauzasone last bullet to add18:02
bauzasgibi: if you're still there, any placement cycle hightlight worth to mention ?18:10
bauzashonestly, don't think so, any-traits is not really something for endusers18:11
sean-k-mooneywell it depends18:11
sean-k-mooneyany traits is something for project but perhaps unified limites18:11
sean-k-mooneyi assume that will already be covered18:12
sean-k-mooneyi guess unifed limits is more a nova/keystone highlight18:12
sean-k-mooneyeven if it is useing placment18:12
bauzassean-k-mooney: gibi: gmann: melwitt: chateaulav: dmitriis: your reviews are worthy https://review.opendev.org/c/openstack/releases/+/831241 18:13
chateaulavbauzas: Awesome!18:14
bauzassean-k-mooney: sure, my context was only for placement cycle highlights 18:14
bauzaschateaulav: thanks for your hard work and dedication18:15
bauzaschateaulav: even if I had no time to review your changes :)18:15
chateaulavlol18:15
opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036818:16
gmannbauzas: thanks for that, checked policy one and almost looks good. 1 comment to add doc link https://review.opendev.org/c/openstack/releases/+/831241/1/deliverables/yoga/nova.yaml#2018:20
gmannfeel free to rephrase the line to minimum text or so18:20
bauzasgmann: given we're late, I prefer to leave it like this but I can rephrase the wording for the prelude section18:20
gmannbauzas: sure, that works too as this is just highlights and people do/expected to check the prelude and releasenote. 18:22
bauzas++18:22
gmann+1 on patch18:22
bauzasthanks18:24
opendevreviewRajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild  https://review.opendev.org/c/openstack/nova/+/83121918:25
opendevreviewRajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild  https://review.opendev.org/c/openstack/nova/+/83121918:25
opendevreviewRajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances  https://review.opendev.org/c/openstack/nova/+/83088318:26
bauzaslo18:30
bauzasok, zuul failed, me respins18:31
bauzassean-k-mooney: updated https://review.opendev.org/c/openstack/releases/+/83124118:35
sean-k-mooneyack thanks ill look again once i hit send on this email18:37
opendevreviewGhanshyam proposed openstack/nova master: Fix unit test for oslo.context 4.1.0  https://review.opendev.org/c/openstack/nova/+/83124418:39
bauzas++18:39
* bauzas has to disappear, dinner time18:39
sean-k-mooneybauzas: have you seen https://review.opendev.org/c/openstack/requirements/+/829599/18:40
sean-k-mooneyim goign to see what is required to fix not so far its just one unit tes18:41
gmannsean-k-mooney: bauzas pushed the fix https://review.opendev.org/c/openstack/nova/+/83124418:41
sean-k-mooneyoh ok18:41
gmannfunctional test are all good, yeah it is just one test18:41
sean-k-mooneyok cool 18:41
sean-k-mooneyill repond to the mail i just sent and add your change then review it18:42
gmannohk, sure18:42
sean-k-mooneybauzas: one final nit https://review.opendev.org/c/openstack/releases/+/831241/2/deliverables/yoga/nova.yaml#1918:48
sean-k-mooneyother then that i think im +1 on that18:48
sean-k-mooneybasicaly its not clear to me if you intened to say operator are not encuraged to use the new scopes until we switch to it next cycle or they are encured to enable it before we enabel it by default18:50
sean-k-mooney""operators are18:50
sean-k-mooney    encouraged to start using it until18:50
sean-k-mooney"""18:50
sean-k-mooneyis ambiguous18:50
*** artom__ is now known as artom19:02
opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036819:18
opendevreviewRajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild  https://review.opendev.org/c/openstack/nova/+/83121919:18
opendevreviewRajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances  https://review.opendev.org/c/openstack/nova/+/83088319:18
bauzassean-k-mooney: thanks for the good comment, rebased.19:49
elodillesmelwitt: when you'll have some time, than could you also review this stable/wallaby patch to make wallaby gate a bit more stable? o:) https://review.opendev.org/c/openstack/nova/+/83065620:59
opendevreviewRajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild  https://review.opendev.org/c/openstack/nova/+/83121923:00
opendevreviewRajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild  https://review.opendev.org/c/openstack/nova/+/82036823:00
opendevreviewRajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances  https://review.opendev.org/c/openstack/nova/+/83088323:00

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