Tuesday, 2021-12-14

opendevreviewsean mooney proposed openstack/nova-specs master: Add spec for external cpu power managment  https://review.opendev.org/c/openstack/nova-specs/+/82141900:03
opendevreviewsean mooney proposed openstack/nova-specs master: add per process healthcheck spec  https://review.opendev.org/c/openstack/nova-specs/+/82127900:19
opendevreviewSteve Baker proposed openstack/nova master: Allow remaining in in DELETING for ironic cleaning  https://review.opendev.org/c/openstack/nova/+/81372901:09
opendevreviewGhanshyam proposed openstack/nova master: Re-add python 3.6 functional testing  https://review.opendev.org/c/openstack/nova/+/82163601:27
opendevreviewGhanshyam proposed openstack/os-vif master: Updating python testing classifier as per Yoga testing runtime  https://review.opendev.org/c/openstack/os-vif/+/81920402:20
opendevreviewGhanshyam proposed openstack/placement master: Updating python testing as per Yoga testing runtime  https://review.opendev.org/c/openstack/placement/+/81920602:24
opendevreviewGhanshyam proposed openstack/osc-placement master: Updating python testing as per Yoga testing runtime  https://review.opendev.org/c/openstack/osc-placement/+/81920302:26
opendevreviewGhanshyam proposed openstack/os-traits master: Updating python testing classifier as per Yoga testing runtime  https://review.opendev.org/c/openstack/os-traits/+/81920502:27
opendevreviewGhanshyam proposed openstack/python-novaclient master: Updating python testing classifier as per Yoga testing runtime  https://review.opendev.org/c/openstack/python-novaclient/+/81920802:28
gmanngibi: sean-k-mooney this is what I was talking about, my test env is down so did not test that. I will check on morning if any error (feel free to update if you see any)04:42
opendevreviewWenping Song proposed openstack/nova master: Fill the AcceleratorRequestBindingFailed exception msg info  https://review.opendev.org/c/openstack/nova/+/81732605:54
*** hemna0 is now known as hemna07:38
opendevreviewWenping Song proposed openstack/nova master: Fill the AcceleratorRequestBindingFailed exception msg info  https://review.opendev.org/c/openstack/nova/+/81732608:27
opendevreviewFederico Ressi proposed openstack/nova master: Debug Nova APIs call failures  https://review.opendev.org/c/openstack/nova/+/80668308:45
bauzasgood spec review day, everyone08:55
bauzasthis starts with a core ping for my own easy repproval spec https://review.opendev.org/c/openstack/nova-specs/+/82137608:56
bauzasgibi : ^08:56
gibion it09:35
* gibi is a bit distracted this morning probably needs a second coffee09:35
gibibauzas: fast approved09:37
* bauzas sends hugs09:38
* bauzas faces an unexpected delay for looking at specs due to some other issue but will start soon09:38
opendevreviewMerged openstack/nova-specs master: Repropose boot a vm with unaddressed port  https://review.opendev.org/c/openstack/nova-specs/+/82137609:55
gibisean-k-mooney: I've left questions in the external-cpu-management spec10:01
gibibauzas: I skip the update userdata spec for now as you had strong opposition about the use case. 10:04
bauzasgibi: I'll revise this spec given they provided a new PS10:05
gibibauzas: ack10:06
gibibrinzhang0: left feedback in the OWNER_ spec10:18
brinzhang0gibi:uii:ack, we will review later, thanks10:20
brinzhang0s/uii;//g10:20
brinzhang0bauzas,gibi: do you have time to review the remove tenat_id series of patches? https://review.opendev.org/q/topic:%22bp%252Fremove-tenant-id%22+(status:open%20OR%20status:merged)10:30
bauzasbrinzhang0: yes, will do today10:31
brinzhang0thanks10:32
gibibrinzhang0: I try to get to it tomorrow11:26
brinzhang0gibi: cool thanks11:28
gibibauzas: I'm done with the first round of spec reviews 12:10
gibibauzas: could you please look at https://review.opendev.org/q/topic:any-traits-support today12:10
gibibauzas: I will try to ask melwitt to look at it too12:10
gtrevisanHello everyone, quick question. We have a bug currently in progress, which we've also evidenced in stable/ussuri. A fix was uploaded to it, which we believe should also apply to Ussuri. Should we suggest a cherry-pick of this fix to Ussuri as well? Is the Ussuri release still in extended maintenance?12:20
*** outbrito_ is now known as outbrito12:22
opendevreviewBalazs Gibizer proposed openstack/os-traits master: Updating python testing classifier as per Yoga testing runtime  https://review.opendev.org/c/openstack/os-traits/+/81920512:25
gibigtrevisan: you can propose backports to ussuri yes, but please note that our stable process requires to land the backport in order so first in the newer stable branches12:26
opendevreviewmitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments.  https://review.opendev.org/c/openstack/nova/+/82093512:55
opendevreviewmitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments.  https://review.opendev.org/c/openstack/nova/+/82093512:58
gibigmann, sean-k-mooney: do we have somewhere in our stack a inter process reader writer lock? I know fastener 0.16.0 has one but we are still on 0.14.1 https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L59314:15
sean-k-mooneynot that im aware of we can proably get it from fastener yes that is proably the best option14:18
sean-k-mooneybut good point on it needing to be an inter process lock14:18
sean-k-mooneyi forgot about that14:19
gibiwe cannot get fastener :/ https://review.opendev.org/c/openstack/nova/+/81311414:19
gibieventlet is not compatible14:19
sean-k-mooneyah ok 14:19
sean-k-mooneyasyncio has some lock14:19
sean-k-mooneymaybe the standard lib has one we could use14:19
gibiI will check14:19
sean-k-mooneyalthough if it does not we might need to look at the taging approch instead14:20
gibiworst case I can try to re-implement it based on olso's inter process lock14:20
sean-k-mooneyyes that is an option too14:20
sean-k-mooneythe semaintic i think we ould want is the writer would not start until all readers had finsihedn and when a writer is queued no new readers can start14:21
bauzasgibi: +2 for both placement specs14:30
gibibauzas: awesome, thanks14:30
bauzasgibi I guess I need to modify https://storyboard.openstack.org/#!/story/2005345 to tell that the task is merged :)14:31
bauzas(of course once we +W the spec)14:31
* bauzas goes looking at other specs :)14:33
gibiI think the that part is automatic when the patch merges14:33
bauzaslast spec but not the least to review https://review.opendev.org/c/openstack/nova-specs/+/821279/2/specs/yoga/approved/per-process-healthchecks.rst15:29
bauzasbut I have our nova meeting to prepare :)15:29
bauzasreminder : nova meeting in 30 mins, the last of year 2021 ;)15:29
gibi\o/15:34
bauzasgibi: do you know if elodilles is around ?15:47
bauzasor already in some kinda french-like vacations ?15:47
gibihm I think so15:47
gibiI mean he should be around15:47
bauzasok15:47
elodilleslet me search for him15:48
elodilleso:)15:48
bauzaselodilles: if you have time and be around, please just update your stable branches topic ;)15:48
elodillesoh, meeting time!15:48
bauzaselodilles: this is fine, I'm also sometimes schizophrenic ;)15:48
elodillessorry, i completely lost in time :S (i was off yesterday and today felt like Monday to me :S)15:49
bauzasnot yet talking about me with the 3rd person 15:49
elodilles:D15:50
bauzaselodilles: no worries, should be a quick meeting15:50
bauzasno specless bp approval ask :)15:50
gibielodilles: I suggest to use the royal "we" instead that did not give your schizophrenia away that easily15:51
bauzasgibi: oh btw. melwitt is now on PTO until next year15:53
gibibauzas: thanks15:53
* bauzas just verified our internal agenda15:53
gibithen I need to find somebody else to approve the placement spec15:53
bauzasyep15:53
bauzasnot sure we have a quorum tho15:54
bauzasbut,15:54
bauzasI'm OK if you ask for an exception15:54
elodillesgibi: I'll inform elodilles about this and we'll see :D15:54
bauzasgibi: your specs are simple and don't have any upgrade concerns15:54
gibielodilles: better :), now drop elodilles too and then it will be perfect 15:54
bauzassean-k-mooney and me are OK with them15:54
bauzasso in case you'd like to have your specs be merged in the other week after the first week, fine for me15:55
gibibauzas: OK, I will try to ping melwitt (as she is the closest to placement from rest of  the cores at the moment) during first week of Jan 15:55
sean-k-mooneyyes im ok with the placment specs form gibi15:55
gibiand if I fail then I will ask for an exception on the second week15:56
sean-k-mooneymy internal calls for the day have finally finsihed so i will be spending more time reviewing specs for the rest of the day15:57
bauzas#startmeeting nova16:00
opendevmeetMeeting started Tue Dec 14 16:00:00 2021 UTC and is due to finish in 60 minutes.  The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'nova'16:00
bauzashey everyone who is not yet on PTO :p16:00
gibio/16:00
gmanno/16:00
bauzaslet's do a quick meeting given I'm not sure we have a lot of quorum for this one :)16:01
bauzas#link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting16:01
elodilleso/16:01
bauzas#topic Bugs (stuck/critical) 16:02
bauzas#info No Critical bug16:02
bauzas#link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 24 new untriaged bugs (+0 since the last meeting)16:02
bauzas#help Nova bug triage help is appreciated https://wiki.openstack.org/wiki/Nova/BugTriage16:02
bauzas#link https://storyboard.openstack.org/#!/project/openstack/placement 25 open stories (+0 since the last meeting) in Storyboard for Placement 16:02
bauzasany bug in particular to discuss ?16:03
gibi-16:03
bauzasnice16:03
bauzas#topic Gate status 16:04
bauzas#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs 16:04
bauzas#link https://zuul.openstack.org/builds?project=openstack%2Fplacement&pipeline=periodic-weekly Placement periodic job status 16:04
bauzas#info Please look at the gate failures, file a bug, and add an  elastic-recheck signature in the opendev/elastic-recheck repo (example: https://review.opendev.org/#/c/759967)16:04
bauzascrickets for me16:04
bauzasnothing to tell about gate bugs or placement jobs16:04
bauzasboth work16:04
bauzasanything to discuss about our gate ?16:05
gibi-16:05
bauzasnice too16:05
bauzas#topic Release Planning 16:05
gibiI was more preoccupied with the neutron gate this week16:05
bauzas#info Yoga-2 is due Jan 6th16:05
bauzas#link https://releases.openstack.org/yoga/schedule.html#y-216:05
bauzas#info 2nd spec review day happens today on Dec 14th16:06
bauzasso we have like 6 or 7 specs to look at16:06
bauzaswe only merged one spec which was a fast reapproval16:06
bauzassome are quite in a good direction but maybe we will have not enough time for merging them *before* yoga-216:06
gibiyeah if things are not landing this week then it will be hard 16:07
gibito land them without cores :D16:07
bauzasif people don't disagree, I'll propose an exceptional spec approval deadline exception during our first meeting of 202216:07
bauzasexception process*16:07
gibisure, lets see what we will have open then16:08
bauzasyup16:08
bauzasnot a reason to not look at the open specs16:08
bauzasbut just saying that one week more shouldn't be a problem16:08
* bauzas wonders btw. why the TC accepted a milestone so close to holidays16:09
sean-k-mooneyyes that shoudl be workable solong as we dont let it drag out too long16:09
bauzaseither way, this is what it is16:09
bauzasanything to discuss about this ?16:09
sean-k-mooneywell not all project have a m2 spec freeze16:09
sean-k-mooneyso it may be less impactful on others16:09
sean-k-mooneynot form me16:10
gmannyeah, its project to project16:10
bauzassean-k-mooney: you're right, this is maybe us who haven't considered it correctly when we thought about Yoga-2 as a spec freeze16:10
bauzasanyway, not a problmem16:10
bauzaswe'll see how things go after new year16:10
bauzas#topic Review priorities 16:10
bauzas#link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement)+label:Review-Priority%252B116:11
bauzas#link https://review.opendev.org/c/openstack/nova/+/816861 bauzas proposing a documentation change for helping contributors to ask for reviews, needs a second +216:11
bauzasthanks gibi btw.16:11
gibithanks bauzas 16:11
gmannI am ok with that, just 1 comment i added to make it clear on non-core adding +116:11
gibithe current list of review prio looks OK to me16:11
bauzasabout the unified limits series, I was confused with what to review but this is fixed16:11
gmannbut that is ok to add  in followup also, not blocker 16:12
bauzasafter discussing it with melwitt internally16:12
bauzasgmann: ack, will look16:12
bauzasI really want us to merge the unified limits series, this has been overdue for too long16:12
sean-k-mooneyi agree16:12
bauzasso I'll continue to review, but given melwitt's on PTO, I guess the progress would be better after next year 16:13
sean-k-mooneyunified limist assumign its readay i think would be a big win and release highlight16:13
gibithere is a list of patches on the  yoga-python-testing that we need to look at I will add them to priorities16:14
bauzasany series or patches that people would want to highlight to the team for reviews ?16:14
sean-k-mooneygmann: on a related note do we have a nova spec for the chagne in RBAC direction and what that means for nova16:14
bauzasheh, jinxed by gibi16:14
bauzasgibi: patches ?16:14
gibiyep16:14
gibigmann has a list of them16:14
bauzasgibi: I mean, links ?16:14
gmannyeah, this is list for auditing all policy #link https://wiki.openstack.org/wiki/Nova/rbac16:15
gibihttps://review.opendev.org/c/openstack/python-novaclient/+/81920816:15
gibihttps://review.opendev.org/c/openstack/os-traits/+/81920516:15
gibihttps://review.opendev.org/c/openstack/osc-placement/+/81920316:15
gibihttps://review.opendev.org/c/openstack/nova/+/82163616:15
gibihttps://review.opendev.org/c/openstack/placement/+/81920616:15
gmannand we can do audit in wiki page in one shot or in code changes also.16:15
gmanngibi: thanks 16:15
bauzasgibi: I marked the nova and placement ones with the R-P flag16:16
gmannsean-k-mooney: that was for RBAC one, and we have BP also for that #link https://blueprints.launchpad.net/nova/+spec/policy-defaults-refresh-216:16
bauzassean-k-mooney: we said on a meeting yes for a specless BP approval16:16
gmannyeah16:16
bauzas(for the policy thing)16:16
sean-k-mooneybauzas: ok 16:17
sean-k-mooneynormally i think we shoudl treath policy like an api change16:17
sean-k-mooneyand use a spec for it but we can proceed without one for now16:17
bauzasthis is the defaults, but meh16:17
gmannI have pushed SYSTEM reader->system admin on top of dansmith  server policy changes. and I will be pushing more changes this week16:17
sean-k-mooneybauzas: well to me policy change are interop issues16:17
bauzassean-k-mooney: I don't disagree16:18
sean-k-mooneyso change default is a breaking change but lest not repoen that16:18
bauzasbut for users, those are transparent if defaults correctly scoped16:18
gmannwell policy change does not require spec always. we have fixed (new or default change) many as bug also16:18
gmannyeah, as it is overall change in RBAC, and as described in goal we have way that users will be broken by default16:19
gmannand migration path and plan for them to new policy in next release16:19
gmann*user will not be broken16:19
gmannsorry for typo. :)16:19
sean-k-mooneyjust to not consume all the time on this is the plan this cycle to walk back the work we had done and focus on identifying the work need with most of the new work happing next cycle16:20
sean-k-mooneyi will review the wiki you provided offline16:20
gmannyeah, specially we are isloating the system form project resource and no system reader in yoga16:20
sean-k-mooneyack16:20
gmannsean-k-mooney: thanks, wiki audit will be helpful 16:21
bauzasI think we also said that the TC would document this16:21
bauzasthe policy changes, I mean16:22
bauzasI even wrote this in the BP when I approved16:22
sean-k-mooneyits in the tc spec to some degree16:22
gmannbasically this is what we will do (phase1 ) in nova in yoga #link https://governance.openstack.org/tc/goals/selected/consistent-and-secure-rbac.html#phase-116:22
bauzasthat's my point16:22
sean-k-mooneyi would like to see a nova game plan for how to aling to that16:22
sean-k-mooneywhich is why i ask about a nova spec16:22
sean-k-mooneybut if most of that work will happen next cycle im fine with the wiki and specless blueprint for this cycle16:22
bauzasthat's why we said we can start to digest and see whether we need to document more the changes in the spec16:22
bauzasright16:23
bauzasmoving on ?16:23
sean-k-mooney+116:23
gmannyeah, its there in TC goal16:23
bauzas#topic Stable Branches 16:23
bauzasstable gates are not blocked16:23
bauzasshit16:23
bauzaselodilles: it's you who should tell it16:23
elodilles:)16:24
elodillesso they are not blocked16:24
gibi:D16:24
elodilles(as far as I've seen it before the meeting with a quick double-check)16:24
gmanni thought it is very blocked and bauzas typo :)16:24
elodillesthough on some branches (wallaby, victoria?) patches need many rechecks to be able to merge16:24
bauzasinteresting dystopîa16:24
bauzasbetween gmann and elodilles16:25
elodillesgmann: hopefully not o:)16:25
gmann\o/16:25
elodillesor are they? :-o 16:25
bauzasso, what to say ? I guess we need to double check16:27
bauzas:)16:27
elodillesi haven't seen any blocking issue, but i'm now confused :)16:28
elodillesgmann: did i miss something?16:28
gmannah no, I have not seen too. I think recheck is more needed that is uit16:29
gmannit16:29
gmannI saw in other project stable/train but not nova16:29
elodillesoh, thanks16:30
elodillescool for nova then16:30
bauzasI guess we can reasonable move on as nothing will change during those days I guess16:30
bauzasnothing actionable I mean,16:31
bauzasbut who knows16:31
elodilles++16:31
bauzas#topic Sub/related team Highlights 16:31
bauzasskipping libvirt's point, as lyarwood is on PTO16:31
bauzas#topic Open discussion 16:31
bauzasaaaand that's it, we consumed all the items16:32
bauzaslast call for anything anyone16:32
gibihappy PTO everyone! :D16:32
sean-k-mooneyo/16:32
gibiI hope I will be bored at the end of my PTO16:33
gmann\o16:33
gmanngibi: more beer can company you :)16:34
gibi:)16:34
bauzashappy new year for everyone indeed16:35
bauzassafe travels for those who visit family or do leisure16:35
bauzassafe beers for those who stay at home16:36
gmannyou too. thanks 16:36
bauzasand hopefully, make a wish for 2022 eventually be the time for seeing us together :)16:36
bauzasthat being said, let's call it a year.16:37
gibi\o/16:37
bauzasthanks16:37
bauzas#endmeeting16:37
opendevmeetMeeting ended Tue Dec 14 16:37:53 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:37
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2021/nova.2021-12-14-16.00.html16:37
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2021/nova.2021-12-14-16.00.txt16:37
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2021/nova.2021-12-14-16.00.log.html16:37
opendevreviewmitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments.  https://review.opendev.org/c/openstack/nova/+/82093516:37
elodillesbauzas: thanks \o/ & same to you :) & sorry :S16:40
opendevreviewMartin Kopec proposed openstack/nova master: Add check job for FIPS  https://review.opendev.org/c/openstack/nova/+/79051916:44
*** beekneemech is now known as bnemec17:08
*** efried1 is now known as efried22:43

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