| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-30 | |||
| 16:16:32 | bauzas | I'm afraid I didn't had time to modify it | |
| 16:16:49 | bauzas | nothing to discuss then for this week | |
| 16:17:05 | bauzas | unless people want to have review priorities ? | |
| 16:17:46 | bauzas | fwiw, my main priority will be unified limits reviews | |
| 16:18:32 | bauzas | OK, next topic | |
| 16:18:39 | bauzas | #topic Stable Branches | |
| 16:18:48 | bauzas | I'm sitting down, listening to elodilles | |
| 16:18:52 | elodilles | #info stable gates are OK | |
| 16:19:00 | elodilles | stable/xena has ~3 merged bug fixes, maybe we can wait some more bug fix to land before we do a release | |
| 16:19:10 | elodilles | and that's it | |
| 16:20:12 | bauzas | I was triaging a bug related to the compute service delete issue, and I saw we hadn't yet merged https://review.opendev.org/c/openstack/nova/+/802847 | |
| 16:20:55 | bauzas | I'll look at other backports that are fixing bad bugs, and I'll try to review them | |
| 16:21:25 | elodilles | thanks in advance \o/ | |
| 16:22:01 | bauzas | well, no problem | |
| 16:22:08 | bauzas | moving on | |
| 16:22:16 | bauzas | #topic Sub/related team Highlights | |
| 16:22:24 | bauzas | Libvirt :lyarwood ? | |
| 16:22:33 | lyarwood | Nothing from me | |
| 16:22:36 | bauzas | cool | |
| 16:22:43 | bauzas | #topic Open discussion | |
| 16:22:48 | bauzas | (gmann) Specless BP approval request for RBAC community-wide goal | |
| 16:23:02 | bauzas | #link https://blueprints.launchpad.net/nova/+spec/policy-defaults-refresh-2 | |
| 16:23:12 | bauzas | gmann: around ? | |
| 16:23:51 | dansmith | I am, if you have questions | |
| 16:24:09 | bauzas | honestly, I don't think I have any concerns | |
| 16:24:32 | dansmith | I've been working on a patch to get servers back to the place where we want it, | |
| 16:24:33 | bauzas | maybe one about upgrades and what it means for operators that were modifying the policies | |
| 16:24:35 | dansmith | as an example people can work from | |
| 16:24:39 | dansmith | and it's very close | |
| 16:24:51 | bauzas | but this is just changing the defaults | |
| 16:24:54 | dansmith | gmann and johnthetubaguy[m] are mostly happy I think, just working out one more functional thing | |
| 16:25:19 | dansmith | bauzas: well, this is pretty much all about defaults anyway, | |
| 16:25:34 | dansmith | and nobody could really have rolled to the new ones yet anyway, so not an upgrade concern, IMHO | |
| 16:25:46 | dansmith | but the revised plan involves less change when they do upgrade | |
| 16:26:31 | bauzas | dansmith: I guess you're referring to https://review.opendev.org/c/openstack/governance/+/815158/20/goals/proposed/consistent-and-secure-rbac.rst as the revised plan ? | |
| 16:26:40 | dansmith | yep | |
| 16:27:24 | bauzas | ok, | |
| 16:27:45 | bauzas | this plan isn't yet sold but whatever it will be, nothing will really change from nova | |
| 16:28:02 | dansmith | well, things have to change in nova of course, | |
| 16:28:08 | bauzas | so as you said, I don't think there is any upgrade concern then | |
| 16:28:29 | bauzas | nothing will really change from a nova perspective if you prefer | |
| 16:28:37 | dansmith | but mostly just undoing some of the proposed stuff that hasn't been able to be realized yet.. stepping back from some of that stuff that we merged proactively | |
| 16:28:40 | bauzas | things have to change, but upgrades aren't a concern either way | |
| 16:29:00 | dansmith | much less of a concern than what they were, but of the stuff we're keeping, no real change, yeah | |
| 16:29:24 | dansmith | and keystone will go first which will help our upgrade be even less impactful than it was going to be, if we ever got past the big bubble we had going | |
| 16:29:48 | bauzas | to answer the original paperwork question, I think there is no controversy to tell it's a specless BP and we don't to document this as we already have https://review.opendev.org/c/openstack/governance/+/815158/20/goals/proposed/consistent-and-secure-rbac.rst | |
| 16:30:02 | bauzas | we don't need* to | |
| 16:30:13 | dansmith | ++ | |
| 16:30:44 | bauzas | that said, of course this work will need some release notes | |
| 16:30:54 | dansmith | obviously | |
| 16:30:55 | bauzas | to explain the changes to the operators | |
| 16:31:00 | bauzas | ok | |
| 16:31:04 | bauzas | anyone has other concerns N? | |
| 16:31:22 | bauzas | dang, I need to learn typing | |
| 16:31:58 | bauzas | (and that's what happens when you have a french keyboard with ? located near n and requiring shift) | |
| 16:32:08 | bauzas | anyway | |
| 16:32:41 | bauzas | #agreed https://blueprints.launchpad.net/nova/+spec/policy-defaults-refresh-2 accepted as a specless BP as the direction is already explained in https://review.opendev.org/c/openstack/governance/+/815158/ | |
| 16:32:50 | bauzas | moving on, last topic | |
| 16:33:12 | bauzas | (ganso) Raising awareness of vif_multiqueue_enabled in flavor work that is ready to be reviewed/merged | |
| 16:33:15 | bauzas | ganso: around ? | |
| 16:33:21 | ganso | o/ | |
| 16:33:34 | bauzas | #link https://blueprints.launchpad.net/nova/+spec/multiqueue-flavor-extra-spec | |
| 16:33:49 | ganso | so as the topic titles says: https://review.opendev.org/q/topic:%22bp%252Fmultiqueue-flavor-extra-spec%22+(status:open%20OR%20status:merged) | |
| 16:34:14 | ganso | we've discussed 2-3 weeks ago about this and that it could/may be specless, but it was approved to be specless ~6 months ago | |
| 16:34:20 | bauzas | ganso: nothing changed during the implementation phase requiring further discussion ? | |
| 16:34:45 | ganso | bauzas: as far as I know, nothing changed and the code is complete | |
| 16:35:05 | bauzas | the BP was previously approved as specless so I don't see problems approving it again providing there were no changes in design | |
| 16:35:08 | ganso | I rebased it and it is passing CI | |
| 16:35:13 | bauzas | (requiring further discussions) | |
| 16:36:11 | ganso | I'm pretty much shepherding this set of changes now, but the work was done by stephenfin | |
| 16:36:12 | bauzas | ganso: I guess you're taking over stephenfin's work ? | |
| 16:36:22 | ganso | yes | |
| 16:36:25 | bauzas | OK, that's crystal clear then | |
| 16:36:39 | bauzas | I don't have any problems reapproving it | |
| 16:36:49 | bauzas | anyone else disagreeing ? | |
| 16:36:49 | ganso | great =) | |
| 16:37:28 | bauzas | #agreed https://blueprints.launchpad.net/nova/+spec/multiqueue-flavor-extra-spec to approve it again as a specless BP for the yoga release cycle | |
| 16:37:50 | bauzas | we're at the end of the agenda, anything else to mention ? | |
| 16:38:15 | bauzas | I'm happy to say we were quick this time :) | |
| 16:38:24 | gibi | \o/ | |
| 16:38:30 | bauzas | if not, | |
| 16:38:34 | bauzas | #endmeeting* | |
| 16:38:34 | opendevmeet | Meeting ended Tue Nov 30 16:38:34 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:38:34 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2021/nova.2021-11-30-16.00.html | |
| 16:38:34 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2021/nova.2021-11-30-16.00.txt | |
| 16:38:34 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2021/nova.2021-11-30-16.00.log.html | |
| 16:38:49 | bauzas | man, I'm fed up with my typing issues | |
| 16:40:03 | bauzas | thanks all | |
| 16:40:22 | elodilles | thanks bauzas o/ | |
| 16:40:57 | bauzas | I'm a gross person, I haven't officially thanked you all at the end of the meeting | |
| 16:42:09 | elodilles | :D | |
| 17:07:13 | bauzas | gmann: can I put you assignee on https://blueprints.launchpad.net/nova/+spec/policy-defaults-refresh-2 ? | |
| 17:14:48 | opendevreview | Nicolas Parquet proposed openstack/nova master: Add v2.91 microversion, allowing @ and dot (.) characters in keypair name https://review.opendev.org/c/openstack/nova/+/781076 | |
| 18:05:52 | gmann | bauzas: sure, thanks | |
| 18:07:59 | opendevreview | Dan Smith proposed openstack/nova master: Revert project-specific APIs for servers https://review.opendev.org/c/openstack/nova/+/816206 | |
| 18:08:00 | opendevreview | Dan Smith proposed openstack/nova master: Make API fixture pass roles https://review.opendev.org/c/openstack/nova/+/819907 | |
| 18:08:26 | dansmith | gmann: this has the fixture change on top ^ so you can see it not working without it, and then working when we change that | |
| 18:08:42 | dansmith | I haven't done more digging on why this is required, but hoping it's something you can spot easily | |
| 18:08:53 | gmann | dansmith: ok, checking | |
| 18:08:54 | dansmith | I wonder if we're not really testing fake policy like we think in functional? | |