| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-12-07 | |||
| 16:16:32 | bauzas | so people could provide new revisions after this day and cores could merge them after | |
| 16:16:34 | kashyap | bauzas: Nod. Okay, you can move on to reviw prio | |
| 16:16:38 | kashyap | sean-k-mooney: I'm off from 14Dec myself | |
| 16:16:40 | bauzas | I mean, before Dec 20 | |
| 16:17:18 | bauzas | but, that said, I'm not against moving the spec freeze deadline after Jan 6th if people really want but, | |
| 16:17:38 | bauzas | keep in mind we already have a good number of accepted bps that we need to review | |
| 16:17:40 | sean-k-mooney | lets see if there is a need after m2 | |
| 16:18:01 | gibi | yeah, we can get back to this on 11th of Jan meeting | |
| 16:18:14 | sean-k-mooney | ya i think we likely will have enough on our plate but we can assess on the 11th | |
| 16:18:21 | bauzas | for the moment, those are the BPs we have for Yoga https://blueprints.launchpad.net/nova/yoga | |
| 16:18:32 | bauzas | but I need to add new BPs that specs were merged during this week | |
| 16:19:21 | sean-k-mooney | yep i also likly need to file 1-2 blueprints for heathchecks for one | |
| 16:19:32 | sean-k-mooney | ill try and do that soon | |
| 16:19:32 | bauzas | actually https://review.opendev.org/q/project:openstack/nova-specs+is:merged tells me that's it for the moment | |
| 16:19:41 | bauzas | so Launchpad is OK | |
| 16:19:47 | sean-k-mooney | ack | |
| 16:20:33 | bauzas | but ok, let's revisite this on Jan 4th and Jan 11th to see whether folks want some exceptions | |
| 16:20:59 | bauzas | this said, I have a point to discuss in this meeting about holidays, btw. | |
| 16:21:14 | bauzas | (in the open discussion section) | |
| 16:21:54 | bauzas | do people agree with it ? we will at least see what's going on with specs after the review day | |
| 16:22:33 | gmann | +1 | |
| 16:22:35 | sean-k-mooney | yep +1 | |
| 16:22:50 | bauzas | ok, moving on | |
| 16:23:15 | bauzas | #topic Review priorities | |
| 16:23:38 | bauzas | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement)+label:Review-Priority%252B1 | |
| 16:23:45 | bauzas | #link https://review.opendev.org/c/openstack/nova/+/816861 bauzas proposing a documentation change for helping contributors to ask for reviews | |
| 16:24:25 | bauzas | I'd appreciate a second core for https://review.opendev.org/c/openstack/nova/+/815373 | |
| 16:24:53 | bauzas | also, | |
| 16:25:04 | gibi | bauzas: I will check that | |
| 16:25:10 | bauzas | no update coming from https://review.opendev.org/c/openstack/nova/+/790519 since more than one month, can we consider to remove the prio ? | |
| 16:25:46 | gibi | yepp drop it from prio | |
| 16:25:59 | gibi | we can add it back once the dependecies are sorted out | |
| 16:26:33 | bauzas | yeah this is related to the centos9 devstack effort, right? | |
| 16:26:50 | gibi | im not sure | |
| 16:27:01 | bauzas | from adalee's last comment, looks like it | |
| 16:27:29 | bauzas | lyarwood: around ? | |
| 16:27:32 | gibi | ahh I see, yes | |
| 16:27:41 | lyarwood | yup /me reads | |
| 16:27:44 | bauzas | lyarwood: you could have more context than me on this FIPS job | |
| 16:28:00 | lyarwood | I've not touched the review in a while | |
| 16:28:21 | bauzas | lyarwood: well, looks like there are some deps issues | |
| 16:28:45 | lyarwood | kk well it's on Ade to push it forward tbh | |
| 16:30:04 | bauzas | agreed | |
| 16:30:11 | bauzas | so, I'll drop the review-prio flag | |
| 16:30:19 | bauzas | but I want to make sure there is progress on it | |
| 16:30:47 | bauzas | hopefully my gerrit update will remind Ade there is work to do | |
| 16:31:26 | gmann | there are few tempest one (in the series of those deps) also needs to merged before | |
| 16:31:36 | gmann | which is under review | |
| 16:32:16 | bauzas | yeah looks like there are deps | |
| 16:32:26 | bauzas | quite a massive work | |
| 16:32:38 | gmann | and this is goal proposal in case to know broader pic #link https://review.opendev.org/c/openstack/governance/+/816587 | |
| 16:32:51 | bauzas | haha | |
| 16:32:53 | bauzas | gtk | |
| 16:33:50 | bauzas | thanks gmann | |
| 16:34:12 | bauzas | anyway, I think we discuss about all the subject, next one | |
| 16:34:19 | bauzas | discussed* (dang) | |
| 16:34:30 | bauzas | oh, my change | |
| 16:34:42 | bauzas | https://review.opendev.org/c/openstack/nova/+/816861 is ready for reviews | |
| 16:34:50 | bauzas | I dropped the whole "I'm the owner, help me" thing | |
| 16:35:28 | bauzas | so, this is now only about "I'm non-core and I like this change, so I'd appreciate if cores could review it given I commit myself to review it too" | |
| 16:35:56 | bauzas | good opportunity for review visibility, just sayin' | |
| 16:35:57 | sean-k-mooney | ya i think that is a good chagne | |
| 16:36:03 | gibi | bauzas: I will get back to that too | |
| 16:37:07 | bauzas | thanks | |
| 16:37:17 | sean-k-mooney | if we are more or less happy with it i can start drafting the project config change for it but ill waith until after the spec review day next week to work on it | |
| 16:37:34 | bauzas | the idea is to give visibility on review time for non-cores and giving them trust | |
| 16:37:59 | gmann | I think this is good. and if every non core adding RP as +1 on their patches then we can re-iterate it | |
| 16:38:08 | bauzas | so, if you're non-core and you'd appreciate some visibility on review time, this is for you | |
| 16:38:42 | sean-k-mooney | i think this is more or less like +1 ing your own patch and we can treat it as such | |
| 16:38:58 | gmann | yeah, 'yes i reviewed it and it looks important to me so can core have a look' | |
| 16:39:01 | sean-k-mooney | e.g. you should in general avoid it but you may still want to do it in some cases | |
| 16:39:11 | bauzas | sean-k-mooney: I punted this use case | |
| 16:39:20 | gmann | may be good to mention in that doc for more clarity | |
| 16:39:38 | sean-k-mooney | bauzas: ya that is ok | |
| 16:39:39 | bauzas | comments are then appreciated on the change itself :) | |
| 16:39:54 | bauzas | next one | |
| 16:39:56 | bauzas | #topic Stable Branches | |
| 16:40:05 | bauzas | elodilles: your dog. | |
| 16:40:26 | elodilles | well, this will be short as not so much happening around stable branches | |
| 16:40:34 | elodilles | * stable gates should be OK (though no patches have been merged in the last couple of days) | |
| 16:40:45 | elodilles | that's it ^^^ | |
| 16:41:06 | bauzas | huzzah | |
| 16:41:28 | bauzas | I remember I promised some reviews | |
| 16:41:37 | bauzas | but lacked my homework | |
| 16:42:11 | bauzas | ok, moving on then | |
| 16:42:18 | bauzas | #topic Sub/related team Highlights | |
| 16:42:22 | bauzas | Libvirt (lyarwood) | |
| 16:42:27 | bauzas | nothing special to mention ? | |
| 16:42:43 | lyarwood | Nothing from me this week, FWIW we will need a new owner for this point in the future. | |
| 16:42:49 | lyarwood | I'll talk about that more later | |
| 16:43:03 | bauzas | you're right | |
| 16:43:12 | bauzas | or, | |
| 16:43:22 | bauzas | not sure we still need to have a subteam | |
| 16:43:37 | bauzas | given afaik only lee and me work on this :) | |
| 16:43:38 | lyarwood | Either or, it's up to the remaining team | |
| 16:43:46 | bauzas | who's teamed up ? | |
| 16:44:12 | bauzas | anyway, we could make a call at next meeting | |
| 16:44:15 | lyarwood | I mean the wider Nova team ;) | |
| 16:45:10 | bauzas | ahah | |
| 16:46:43 | bauzas | anyway, last topic | |