opendevreview | sean mooney proposed openstack/nova-specs master: Add spec for external cpu power managment https://review.opendev.org/c/openstack/nova-specs/+/821419 | 00:03 |
---|---|---|
opendevreview | sean mooney proposed openstack/nova-specs master: add per process healthcheck spec https://review.opendev.org/c/openstack/nova-specs/+/821279 | 00:19 |
opendevreview | Steve Baker proposed openstack/nova master: Allow remaining in in DELETING for ironic cleaning https://review.opendev.org/c/openstack/nova/+/813729 | 01:09 |
opendevreview | Ghanshyam proposed openstack/nova master: Re-add python 3.6 functional testing https://review.opendev.org/c/openstack/nova/+/821636 | 01:27 |
opendevreview | Ghanshyam proposed openstack/os-vif master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/os-vif/+/819204 | 02:20 |
opendevreview | Ghanshyam proposed openstack/placement master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/placement/+/819206 | 02:24 |
opendevreview | Ghanshyam proposed openstack/osc-placement master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/osc-placement/+/819203 | 02:26 |
opendevreview | Ghanshyam proposed openstack/os-traits master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/os-traits/+/819205 | 02:27 |
opendevreview | Ghanshyam proposed openstack/python-novaclient master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/python-novaclient/+/819208 | 02:28 |
gmann | gibi: 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 |
opendevreview | Wenping Song proposed openstack/nova master: Fill the AcceleratorRequestBindingFailed exception msg info https://review.opendev.org/c/openstack/nova/+/817326 | 05:54 |
*** hemna0 is now known as hemna | 07:38 | |
opendevreview | Wenping Song proposed openstack/nova master: Fill the AcceleratorRequestBindingFailed exception msg info https://review.opendev.org/c/openstack/nova/+/817326 | 08:27 |
opendevreview | Federico Ressi proposed openstack/nova master: Debug Nova APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | 08:45 |
bauzas | good spec review day, everyone | 08:55 |
bauzas | this starts with a core ping for my own easy repproval spec https://review.opendev.org/c/openstack/nova-specs/+/821376 | 08:56 |
bauzas | gibi : ^ | 08:56 |
gibi | on it | 09:35 |
* gibi is a bit distracted this morning probably needs a second coffee | 09:35 | |
gibi | bauzas: fast approved | 09:37 |
* bauzas sends hugs | 09:38 | |
* bauzas faces an unexpected delay for looking at specs due to some other issue but will start soon | 09:38 | |
opendevreview | Merged openstack/nova-specs master: Repropose boot a vm with unaddressed port https://review.opendev.org/c/openstack/nova-specs/+/821376 | 09:55 |
gibi | sean-k-mooney: I've left questions in the external-cpu-management spec | 10:01 |
gibi | bauzas: I skip the update userdata spec for now as you had strong opposition about the use case. | 10:04 |
bauzas | gibi: I'll revise this spec given they provided a new PS | 10:05 |
gibi | bauzas: ack | 10:06 |
gibi | brinzhang0: left feedback in the OWNER_ spec | 10:18 |
brinzhang0 | gibi:uii:ack, we will review later, thanks | 10:20 |
brinzhang0 | s/uii;//g | 10:20 |
brinzhang0 | bauzas,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 |
bauzas | brinzhang0: yes, will do today | 10:31 |
brinzhang0 | thanks | 10:32 |
gibi | brinzhang0: I try to get to it tomorrow | 11:26 |
brinzhang0 | gibi: cool thanks | 11:28 |
gibi | bauzas: I'm done with the first round of spec reviews | 12:10 |
gibi | bauzas: could you please look at https://review.opendev.org/q/topic:any-traits-support today | 12:10 |
gibi | bauzas: I will try to ask melwitt to look at it too | 12:10 |
gtrevisan | Hello 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 outbrito | 12:22 | |
opendevreview | Balazs Gibizer proposed openstack/os-traits master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/os-traits/+/819205 | 12:25 |
gibi | gtrevisan: 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 branches | 12:26 |
opendevreview | mitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments. https://review.opendev.org/c/openstack/nova/+/820935 | 12:55 |
opendevreview | mitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments. https://review.opendev.org/c/openstack/nova/+/820935 | 12:58 |
gibi | gmann, 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#L593 | 14:15 |
sean-k-mooney | not that im aware of we can proably get it from fastener yes that is proably the best option | 14:18 |
sean-k-mooney | but good point on it needing to be an inter process lock | 14:18 |
sean-k-mooney | i forgot about that | 14:19 |
gibi | we cannot get fastener :/ https://review.opendev.org/c/openstack/nova/+/813114 | 14:19 |
gibi | eventlet is not compatible | 14:19 |
sean-k-mooney | ah ok | 14:19 |
sean-k-mooney | asyncio has some lock | 14:19 |
sean-k-mooney | maybe the standard lib has one we could use | 14:19 |
gibi | I will check | 14:19 |
sean-k-mooney | although if it does not we might need to look at the taging approch instead | 14:20 |
gibi | worst case I can try to re-implement it based on olso's inter process lock | 14:20 |
sean-k-mooney | yes that is an option too | 14:20 |
sean-k-mooney | the 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 start | 14:21 |
bauzas | gibi: +2 for both placement specs | 14:30 |
gibi | bauzas: awesome, thanks | 14:30 |
bauzas | gibi 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 | |
gibi | I think the that part is automatic when the patch merges | 14:33 |
bauzas | last spec but not the least to review https://review.opendev.org/c/openstack/nova-specs/+/821279/2/specs/yoga/approved/per-process-healthchecks.rst | 15:29 |
bauzas | but I have our nova meeting to prepare :) | 15:29 |
bauzas | reminder : nova meeting in 30 mins, the last of year 2021 ;) | 15:29 |
gibi | \o/ | 15:34 |
bauzas | gibi: do you know if elodilles is around ? | 15:47 |
bauzas | or already in some kinda french-like vacations ? | 15:47 |
gibi | hm I think so | 15:47 |
gibi | I mean he should be around | 15:47 |
bauzas | ok | 15:47 |
elodilles | let me search for him | 15:48 |
elodilles | o:) | 15:48 |
bauzas | elodilles: if you have time and be around, please just update your stable branches topic ;) | 15:48 |
elodilles | oh, meeting time! | 15:48 |
bauzas | elodilles: this is fine, I'm also sometimes schizophrenic ;) | 15:48 |
elodilles | sorry, i completely lost in time :S (i was off yesterday and today felt like Monday to me :S) | 15:49 |
bauzas | not yet talking about me with the 3rd person | 15:49 |
elodilles | :D | 15:50 |
bauzas | elodilles: no worries, should be a quick meeting | 15:50 |
bauzas | no specless bp approval ask :) | 15:50 |
gibi | elodilles: I suggest to use the royal "we" instead that did not give your schizophrenia away that easily | 15:51 |
bauzas | gibi: oh btw. melwitt is now on PTO until next year | 15:53 |
gibi | bauzas: thanks | 15:53 |
* bauzas just verified our internal agenda | 15:53 | |
gibi | then I need to find somebody else to approve the placement spec | 15:53 |
bauzas | yep | 15:53 |
bauzas | not sure we have a quorum tho | 15:54 |
bauzas | but, | 15:54 |
bauzas | I'm OK if you ask for an exception | 15:54 |
elodilles | gibi: I'll inform elodilles about this and we'll see :D | 15:54 |
bauzas | gibi: your specs are simple and don't have any upgrade concerns | 15:54 |
gibi | elodilles: better :), now drop elodilles too and then it will be perfect | 15:54 |
bauzas | sean-k-mooney and me are OK with them | 15:54 |
bauzas | so in case you'd like to have your specs be merged in the other week after the first week, fine for me | 15:55 |
gibi | bauzas: 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-mooney | yes im ok with the placment specs form gibi | 15:55 |
gibi | and if I fail then I will ask for an exception on the second week | 15:56 |
sean-k-mooney | my internal calls for the day have finally finsihed so i will be spending more time reviewing specs for the rest of the day | 15:57 |
bauzas | #startmeeting nova | 16:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
opendevmeet | The meeting name has been set to 'nova' | 16:00 |
bauzas | hey everyone who is not yet on PTO :p | 16:00 |
gibi | o/ | 16:00 |
gmann | o/ | 16:00 |
bauzas | let'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_meeting | 16:01 |
elodilles | o/ | 16:01 |
bauzas | #topic Bugs (stuck/critical) | 16:02 |
bauzas | #info No Critical bug | 16: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/BugTriage | 16:02 |
bauzas | #link https://storyboard.openstack.org/#!/project/openstack/placement 25 open stories (+0 since the last meeting) in Storyboard for Placement | 16:02 |
bauzas | any bug in particular to discuss ? | 16:03 |
gibi | - | 16:03 |
bauzas | nice | 16: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 |
bauzas | crickets for me | 16:04 |
bauzas | nothing to tell about gate bugs or placement jobs | 16:04 |
bauzas | both work | 16:04 |
bauzas | anything to discuss about our gate ? | 16:05 |
gibi | - | 16:05 |
bauzas | nice too | 16:05 |
bauzas | #topic Release Planning | 16:05 |
gibi | I was more preoccupied with the neutron gate this week | 16:05 |
bauzas | #info Yoga-2 is due Jan 6th | 16:05 |
bauzas | #link https://releases.openstack.org/yoga/schedule.html#y-2 | 16:05 |
bauzas | #info 2nd spec review day happens today on Dec 14th | 16:06 |
bauzas | so we have like 6 or 7 specs to look at | 16:06 |
bauzas | we only merged one spec which was a fast reapproval | 16:06 |
bauzas | some are quite in a good direction but maybe we will have not enough time for merging them *before* yoga-2 | 16:06 |
gibi | yeah if things are not landing this week then it will be hard | 16:07 |
gibi | to land them without cores :D | 16:07 |
bauzas | if people don't disagree, I'll propose an exceptional spec approval deadline exception during our first meeting of 2022 | 16:07 |
bauzas | exception process* | 16:07 |
gibi | sure, lets see what we will have open then | 16:08 |
bauzas | yup | 16:08 |
bauzas | not a reason to not look at the open specs | 16:08 |
bauzas | but just saying that one week more shouldn't be a problem | 16:08 |
* bauzas wonders btw. why the TC accepted a milestone so close to holidays | 16:09 | |
sean-k-mooney | yes that shoudl be workable solong as we dont let it drag out too long | 16:09 |
bauzas | either way, this is what it is | 16:09 |
bauzas | anything to discuss about this ? | 16:09 |
sean-k-mooney | well not all project have a m2 spec freeze | 16:09 |
sean-k-mooney | so it may be less impactful on others | 16:09 |
sean-k-mooney | not form me | 16:10 |
gmann | yeah, its project to project | 16:10 |
bauzas | sean-k-mooney: you're right, this is maybe us who haven't considered it correctly when we thought about Yoga-2 as a spec freeze | 16:10 |
bauzas | anyway, not a problmem | 16:10 |
bauzas | we'll see how things go after new year | 16: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%252B1 | 16: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 +2 | 16:11 |
bauzas | thanks gibi btw. | 16:11 |
gibi | thanks bauzas | 16:11 |
gmann | I am ok with that, just 1 comment i added to make it clear on non-core adding +1 | 16:11 |
gibi | the current list of review prio looks OK to me | 16:11 |
bauzas | about the unified limits series, I was confused with what to review but this is fixed | 16:11 |
gmann | but that is ok to add in followup also, not blocker | 16:12 |
bauzas | after discussing it with melwitt internally | 16:12 |
bauzas | gmann: ack, will look | 16:12 |
bauzas | I really want us to merge the unified limits series, this has been overdue for too long | 16:12 |
sean-k-mooney | i agree | 16:12 |
bauzas | so 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-mooney | unified limist assumign its readay i think would be a big win and release highlight | 16:13 |
gibi | there is a list of patches on the yoga-python-testing that we need to look at I will add them to priorities | 16:14 |
bauzas | any series or patches that people would want to highlight to the team for reviews ? | 16:14 |
sean-k-mooney | gmann: on a related note do we have a nova spec for the chagne in RBAC direction and what that means for nova | 16:14 |
bauzas | heh, jinxed by gibi | 16:14 |
bauzas | gibi: patches ? | 16:14 |
gibi | yep | 16:14 |
gibi | gmann has a list of them | 16:14 |
bauzas | gibi: I mean, links ? | 16:14 |
gmann | yeah, this is list for auditing all policy #link https://wiki.openstack.org/wiki/Nova/rbac | 16:15 |
gibi | https://review.opendev.org/c/openstack/python-novaclient/+/819208 | 16:15 |
gibi | https://review.opendev.org/c/openstack/os-traits/+/819205 | 16:15 |
gibi | https://review.opendev.org/c/openstack/osc-placement/+/819203 | 16:15 |
gibi | https://review.opendev.org/c/openstack/nova/+/821636 | 16:15 |
gibi | https://review.opendev.org/c/openstack/placement/+/819206 | 16:15 |
gmann | and we can do audit in wiki page in one shot or in code changes also. | 16:15 |
gmann | gibi: thanks | 16:15 |
bauzas | gibi: I marked the nova and placement ones with the R-P flag | 16:16 |
gmann | sean-k-mooney: that was for RBAC one, and we have BP also for that #link https://blueprints.launchpad.net/nova/+spec/policy-defaults-refresh-2 | 16:16 |
bauzas | sean-k-mooney: we said on a meeting yes for a specless BP approval | 16:16 |
gmann | yeah | 16:16 |
bauzas | (for the policy thing) | 16:16 |
sean-k-mooney | bauzas: ok | 16:17 |
sean-k-mooney | normally i think we shoudl treath policy like an api change | 16:17 |
sean-k-mooney | and use a spec for it but we can proceed without one for now | 16:17 |
bauzas | this is the defaults, but meh | 16:17 |
gmann | I have pushed SYSTEM reader->system admin on top of dansmith server policy changes. and I will be pushing more changes this week | 16:17 |
sean-k-mooney | bauzas: well to me policy change are interop issues | 16:17 |
bauzas | sean-k-mooney: I don't disagree | 16:18 |
sean-k-mooney | so change default is a breaking change but lest not repoen that | 16:18 |
bauzas | but for users, those are transparent if defaults correctly scoped | 16:18 |
gmann | well policy change does not require spec always. we have fixed (new or default change) many as bug also | 16:18 |
gmann | yeah, as it is overall change in RBAC, and as described in goal we have way that users will be broken by default | 16:19 |
gmann | and migration path and plan for them to new policy in next release | 16:19 |
gmann | *user will not be broken | 16:19 |
gmann | sorry for typo. :) | 16:19 |
sean-k-mooney | just 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 cycle | 16:20 |
sean-k-mooney | i will review the wiki you provided offline | 16:20 |
gmann | yeah, specially we are isloating the system form project resource and no system reader in yoga | 16:20 |
sean-k-mooney | ack | 16:20 |
gmann | sean-k-mooney: thanks, wiki audit will be helpful | 16:21 |
bauzas | I think we also said that the TC would document this | 16:21 |
bauzas | the policy changes, I mean | 16:22 |
bauzas | I even wrote this in the BP when I approved | 16:22 |
sean-k-mooney | its in the tc spec to some degree | 16:22 |
gmann | basically 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-1 | 16:22 |
bauzas | that's my point | 16:22 |
sean-k-mooney | i would like to see a nova game plan for how to aling to that | 16:22 |
sean-k-mooney | which is why i ask about a nova spec | 16:22 |
sean-k-mooney | but if most of that work will happen next cycle im fine with the wiki and specless blueprint for this cycle | 16:22 |
bauzas | that's why we said we can start to digest and see whether we need to document more the changes in the spec | 16:22 |
bauzas | right | 16:23 |
bauzas | moving on ? | 16:23 |
sean-k-mooney | +1 | 16:23 |
gmann | yeah, its there in TC goal | 16:23 |
bauzas | #topic Stable Branches | 16:23 |
bauzas | stable gates are not blocked | 16:23 |
bauzas | shit | 16:23 |
bauzas | elodilles: it's you who should tell it | 16:23 |
elodilles | :) | 16:24 |
elodilles | so they are not blocked | 16:24 |
gibi | :D | 16:24 |
elodilles | (as far as I've seen it before the meeting with a quick double-check) | 16:24 |
gmann | i thought it is very blocked and bauzas typo :) | 16:24 |
elodilles | though on some branches (wallaby, victoria?) patches need many rechecks to be able to merge | 16:24 |
bauzas | interesting dystopîa | 16:24 |
bauzas | between gmann and elodilles | 16:25 |
elodilles | gmann: hopefully not o:) | 16:25 |
gmann | \o/ | 16:25 |
elodilles | or are they? :-o | 16:25 |
bauzas | so, what to say ? I guess we need to double check | 16:27 |
bauzas | :) | 16:27 |
elodilles | i haven't seen any blocking issue, but i'm now confused :) | 16:28 |
elodilles | gmann: did i miss something? | 16:28 |
gmann | ah no, I have not seen too. I think recheck is more needed that is uit | 16:29 |
gmann | it | 16:29 |
gmann | I saw in other project stable/train but not nova | 16:29 |
elodilles | oh, thanks | 16:30 |
elodilles | cool for nova then | 16:30 |
bauzas | I guess we can reasonable move on as nothing will change during those days I guess | 16:30 |
bauzas | nothing actionable I mean, | 16:31 |
bauzas | but who knows | 16:31 |
elodilles | ++ | 16:31 |
bauzas | #topic Sub/related team Highlights | 16:31 |
bauzas | skipping libvirt's point, as lyarwood is on PTO | 16:31 |
bauzas | #topic Open discussion | 16:31 |
bauzas | aaaand that's it, we consumed all the items | 16:32 |
bauzas | last call for anything anyone | 16:32 |
gibi | happy PTO everyone! :D | 16:32 |
sean-k-mooney | o/ | 16:32 |
gibi | I hope I will be bored at the end of my PTO | 16:33 |
gmann | \o | 16:33 |
gmann | gibi: more beer can company you :) | 16:34 |
gibi | :) | 16:34 |
bauzas | happy new year for everyone indeed | 16:35 |
bauzas | safe travels for those who visit family or do leisure | 16:35 |
bauzas | safe beers for those who stay at home | 16:36 |
gmann | you too. thanks | 16:36 |
bauzas | and hopefully, make a wish for 2022 eventually be the time for seeing us together :) | 16:36 |
bauzas | that being said, let's call it a year. | 16:37 |
gibi | \o/ | 16:37 |
bauzas | thanks | 16:37 |
bauzas | #endmeeting | 16:37 |
opendevmeet | Meeting ended Tue Dec 14 16:37:53 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:37 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2021/nova.2021-12-14-16.00.html | 16:37 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2021/nova.2021-12-14-16.00.txt | 16:37 |
opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2021/nova.2021-12-14-16.00.log.html | 16:37 |
opendevreview | mitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments. https://review.opendev.org/c/openstack/nova/+/820935 | 16:37 |
elodilles | bauzas: thanks \o/ & same to you :) & sorry :S | 16:40 |
opendevreview | Martin Kopec proposed openstack/nova master: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/790519 | 16:44 |
*** beekneemech is now known as bnemec | 17:08 | |
*** efried1 is now known as efried | 22:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!