| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-22 | |||
| 16:31:23 | bauzas | but people can start looking at it after the meeting if they want | |
| 16:31:40 | bauzas | this is just, I don't want this branch holding because of one single job | |
| 16:31:40 | elodilles | gmann: ack | |
| 16:32:06 | bauzas | gmann: elodilles: I'd appreciate some DNM patches to make sure we don't hit this every change | |
| 16:33:15 | bauzas | looks we discuss all the thingies by now | |
| 16:33:20 | bauzas | discussed* | |
| 16:33:25 | bauzas | can we move ? | |
| 16:33:30 | gmann | did recheck on 828413, let's see | |
| 16:33:54 | bauzas | gmann: ++ | |
| 16:34:11 | elodilles | yes, thanks, let's move on | |
| 16:35:15 | bauzas | last topic then | |
| 16:35:21 | bauzas | #topic Open discussion | |
| 16:35:24 | bauzas | I have one | |
| 16:35:32 | bauzas | (bauzas) Upgrade our minimum service check https://review.opendev.org/c/openstack/nova/+/833440 | |
| 16:35:50 | bauzas | takashi gently provided a changes for bumping our min version support | |
| 16:36:10 | bauzas | before merging it, I'd like to make sure all people here agree on it | |
| 16:36:24 | dansmith | so one thing we might want to consider, | |
| 16:36:32 | bauzas | (that said, there is a grenade issue on its change, so even with +Wing it...) | |
| 16:36:48 | dansmith | is a PTG topic about the check (and the problems with it that we didn't foresee) to see if there's any better way we could or should be doing that whole thing | |
| 16:36:56 | dansmith | and just punt on the patch until we have that discussion | |
| 16:37:34 | bauzas | I already opened a PTG topic | |
| 16:37:46 | bauzas | I'll add the service check in it | |
| 16:37:52 | dansmith | okay | |
| 16:39:00 | bauzas | just done | |
| 16:39:07 | bauzas | people agree with this plan ? | |
| 16:39:22 | bauzas | either way, as said the change itself has grenade issues that need to be fixed | |
| 16:39:43 | bauzas | and I don't see any reason for rushing on it being merged | |
| 16:39:49 | bauzas | we have the whole zed timeframe for this | |
| 16:40:05 | elodilles | (grenade issue might be because devstack does not have yet stable/yoga) | |
| 16:40:34 | elodilles | (so that should be OK in 1 or 2 days) | |
| 16:40:40 | bauzas | we haven't released stable/yoga | |
| 16:40:53 | bauzas | this will be done on next Wed | |
| 16:41:12 | bauzas | elodilles: but yeah, sounds this | |
| 16:41:30 | elodilles | ++ | |
| 16:41:31 | gmann | yeah, we should do that soon, neutron face same ssue. | |
| 16:41:41 | gmann | elodilles: I will discuss in release channel | |
| 16:41:48 | elodilles | gmann: ack | |
| 16:43:18 | bauzas | ok, I guess we're done then | |
| 16:43:50 | artom | Oh, can we chat about https://review.opendev.org/c/openstack/nova/+/833453? | |
| 16:43:53 | bauzas | #agreed let's hold https://review.opendev.org/c/openstack/nova/+/833440 until we correctly discuss this at the PTG | |
| 16:44:52 | artom | Really only bringing it up here because, as a periodic, we'd have to check up on the status, presumably here | |
| 16:44:56 | artom | Here == the meeting | |
| 16:45:13 | bauzas | artom: yeah, that's my point | |
| 16:45:24 | bauzas | we already do a few checks during the gate topic | |
| 16:45:45 | bauzas | but I wonder whether that wouldn't be better if we could agree on this at the PTG | |
| 16:46:19 | EugenMayer | is it possible to set the flavor of an instance manually using the api? | |
| 16:46:49 | EugenMayer | Oh - sorry. Still meeting time. Ignore me. | |
| 16:46:55 | artom | bauzas, doesn't seem controversial, but OK :) | |
| 16:47:07 | bauzas | artom: yup, I don't disagree | |
| 16:47:24 | bauzas | do people have concerns with adding a periodic check on whitebox ? | |
| 16:47:29 | artom | I guess the downside is CI resource usage, but... one nightly job seems OK? | |
| 16:47:58 | bauzas | I heard news of some CI resource shortage, but I'm not in the TC | |
| 16:48:10 | artom | Yet ;) | |
| 16:48:19 | bauzas | dansmith: gmann: can we just add a periodic job without being concerned ? | |
| 16:48:20 | artom | dansmith said someone is pulling out | |
| 16:48:31 | artom | (phrasing </archer>) | |
| 16:48:39 | dansmith | periodic is probably not a big deal I would imagine | |
| 16:48:54 | dansmith | I think we're going to need to trim down nova's per-patch jobs too, as it's getting pretty heavy | |
| 16:49:40 | bauzas | yeah, I don't think this is a big thing if we add a periodic | |
| 16:49:58 | gmann | yeah, and periodic also we can see if daily or weekly? | |
| 16:49:58 | bauzas | dansmith: adding a PTG topic about it fwiw | |
| 16:50:56 | bauzas | tbh, the only matter is how much we'll check its status and that will be weekly (during the team meeting) | |
| 16:51:04 | gmann | bauzas: artom along with periodic, add in experimental pipeline too for manual trigger. that helps to avoid adding it in check/gate pipeline if anyone want to run maually | |
| 16:51:38 | artom | bauzas, yep, no point in making it daily if we're only checking the status weekly | |
| 16:51:44 | artom | gmann, ack, can do | |
| 16:51:45 | gmann | +1 | |
| 16:51:57 | dansmith | yeah daily seems excessive | |
| 16:53:07 | bauzas | artom: update this change with the weekly period time and mention in the commit msg we'll need to verify it during weekly meetings | |
| 16:53:43 | bauzas | look at the placement ones | |
| 16:53:50 | artom | Oh yeah! | |
| 16:53:56 | gmann | artom: https://github.com/openstack/placement/blob/master/.zuul.yaml#L64 | |
| 16:54:00 | gmann | yeah | |
| 16:54:15 | bauzas | this is another pipeline IIRC | |
| 16:54:15 | artom | Hah, that was easy | |
| 16:54:20 | sean-k-mooney | by the way i think weekly jobs in general suit use better as we can review them in the weekly meeting | |
| 16:54:32 | sean-k-mooney | if we have a nightly we proably wont look at it every day | |
| 16:54:38 | bauzas | oh yeah | |
| 16:54:51 | bauzas | I just hope this meeting won't transform into some CI meeting | |
| 16:54:57 | chateaulav | artom: nova zuul has an example of weekly periodic now | |
| 16:55:02 | bauzas | if we start adding more periodics | |
| 16:55:22 | artom | I mean, feel free to nack the idea entirely :) | |
| 16:55:29 | sean-k-mooney | bauzas: well it should just be (are they green no we shoudl look at X after the meeting) | |
| 16:55:31 | artom | I'll obviously try to debate/convince you | |
| 16:55:37 | bauzas | artom: nah, I like the idea, I just want us to buy it | |
| 16:55:47 | artom | But if we think whitebox doesn't bring value to Nova CI, let's just not do it :) | |
| 16:56:04 | bauzas | we're approaching meeting's end time | |
| 16:56:14 | artom | End times are nigh | |
| 16:56:18 | bauzas | any other item to mention before we close ? | |
| 16:56:19 | sean-k-mooney | :) | |
| 16:56:33 | sean-k-mooney | ah i actully had two blueprints i wanted to raise | |
| 16:56:47 | sean-k-mooney | we defered updating the defaults for allcoation ratios | |
| 16:57:00 | bauzas | sean-k-mooney: oh I forgot to mention I changed Launchpad to reflect zed as the active series | |
| 16:57:16 | sean-k-mooney | shall we proceed with that or discuss at ptg | |
| 16:57:29 | sean-k-mooney | also kasyaps blueprint for usign the new libvirt apis | |
| 16:57:35 | bauzas | we're a bit short in time for reapproving specless bps by now | |
| 16:57:37 | sean-k-mooney | can we retarget both to zed | |
| 16:57:45 | sean-k-mooney | ack | |
| 16:57:52 | sean-k-mooney | we can disucss it next week or at ptg | |
| 16:57:52 | bauzas | but we can look at them during next meeting | |
| 16:58:03 | bauzas | well, Zed is open | |