| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-11-21 | |||
| 16:50:45 | fwiesel | sean-k-mooney: The main concern is here more that we have to run arbitrary code, so we want it a bit more isolated. | |
| 16:51:16 | sean-k-mooney | fwiesel: yep form past expeince runing a third party ci for nova/neutron | |
| 16:51:24 | sean-k-mooney | geting it approval to run it is non trivial | |
| 16:51:30 | bauzas | fwiesel: grandchild: you can see in the above link how many builds we run | |
| 16:51:38 | bauzas | for the 'check' pipeline | |
| 16:52:33 | bauzas | and we can't unfortunately only test changes that modify the virt driver | |
| 16:52:40 | grandchild | (zuul link throws 500 in fetching the jobs ;) ) | |
| 16:53:17 | bauzas | because we could regress on vmwareapi by a change not related to the driver | |
| 16:53:22 | grandchild | but i'd wager, like fwiesel said, isolation is more of an issue than load. | |
| 16:53:35 | grandchild | bauzas: sure, we'd run the whole pipeline, we are aware of that | |
| 16:53:41 | bauzas | for example, say someone modifies the virt API and forgets to modify vmwareapi module | |
| 16:54:06 | bauzas | grandchild: cool, I just wanted to be aware | |
| 16:54:07 | sean-k-mooney | grandchild: well you dont need to run the full pipeline | |
| 16:54:21 | sean-k-mooney | you only need 1 job that runs on all changes to a subet of direcotries | |
| 16:54:52 | sean-k-mooney | ideally any change the touches the compute and virt modules | |
| 16:54:54 | bauzas | sean-k-mooney: that's my point | |
| 16:54:58 | sean-k-mooney | and ideally som others | |
| 16:54:58 | grandchild | sean-k-mooney: sorry if the terminology is off -- i meant the check for the whole codebase | |
| 16:55:13 | bauzas | sean-k-mooney: I don't really know which modules they should only check | |
| 16:55:47 | bauzas | but we could try to run something small first | |
| 16:55:48 | sean-k-mooney | well ideally every change but at least every chagne to the compute and virt modules | |
| 16:55:55 | bauzas | without removing the experimental tag | |
| 16:56:09 | bauzas | and see how this goes first | |
| 16:56:23 | bauzas | before adding more checks to the 3rd party pipeline | |
| 16:56:37 | sean-k-mooney | sof feature freeze is feb 29th | |
| 16:56:50 | sean-k-mooney | i had wanted the removals to happen before m1 | |
| 16:57:07 | sean-k-mooney | i could be convinced to wait till around m2 for ci or very early febuary | |
| 16:57:16 | grandchild | that sounds fair actually, 3 months to get something on its feet. might be achievable. | |
| 16:57:31 | bauzas | here is what I can propose | |
| 16:57:40 | grandchild | but like fwiesel said, a week for an estimate from us. | |
| 16:57:42 | gibi | (if we remove the the driver from master the driver is be on the stable branches) | |
| 16:57:42 | fwiesel | We could also turn it the other way around. You remove it, and we re-add it, when you are convinced it is fine. | |
| 16:57:49 | bauzas | grandchild: fwiesel: would you be able to attend our meeting on a weekly basis ? | |
| 16:57:57 | grandchild | bauzas: definitely | |
| 16:58:07 | bauzas | okay, here is my proposal then | |
| 16:58:09 | fwiesel | fwiesel: Yes | |
| 16:58:19 | fwiesel | bauzas: Yes | |
| 16:58:24 | bauzas | we could defer the removal until caracal-2 | |
| 16:58:50 | bauzas | and every week, I'd add a topic in our meeting about your 3rd party CI | |
| 16:59:07 | bauzas | you could then tell us where you are and what you need | |
| 16:59:51 | bauzas | and if before end of December, we can't really see anything coming, then we would remove the virt driver | |
| 17:00:04 | fwiesel | Fair enough. | |
| 17:00:05 | grandchild | that sounds fine to me, fwiesel? | |
| 17:00:10 | grandchild | k | |
| 17:00:19 | bauzas | any other opinions but me ? | |
| 17:00:32 | gibi | sound OK to me too | |
| 17:00:33 | bauzas | I'm just a code herder, I'm not a L | |
| 17:00:44 | sean-k-mooney | i can defer writing the remaining patches until after milestone 2 | |
| 17:00:53 | sean-k-mooney | i have the patch to remove the driver already done | |
| 17:01:13 | sean-k-mooney | but i can wait for the api/object changes until we see if the ci turns up by milestone 2 | |
| 17:01:18 | bauzas | okay, then I can write the action item | |
| 17:01:35 | bauzas | sean-k-mooney: I haven't checked, but surely the main patch isn't merged yet, right ? | |
| 17:01:44 | sean-k-mooney | ill -w the removal patch for now | |
| 17:01:49 | sean-k-mooney | its not but i wanted it to be | |
| 17:01:54 | bauzas | okay, sounds a plan | |
| 17:01:58 | sean-k-mooney | i was goign to ask for review today | |
| 17:02:09 | bauzas | and we'll see on a weekly basis how this goes | |
| 17:02:16 | bauzas | lemme write the action | |
| 17:02:53 | bauzas | #agreed we defer the vmwareapi driver removal until caracal-2 | |
| 17:03:29 | bauzas | #agreed fwiesel and/or grandchild will report on a weekly basis about their efforts on running a 3rd party CI for vmwareapi | |
| 17:03:53 | bauzas | #action bauzas to create a meeting topic in order to discuss the efforts | |
| 17:03:55 | grandchild | ftr: jkulik is not in the office today, but also in the mix from us :) | |
| 17:04:03 | bauzas | sure | |
| 17:04:20 | bauzas | sean-k-mooney: I appreciate your understanding about the deferral | |
| 17:04:35 | sean-k-mooney | its ok ill ask ye to review other patches instead :P | |
| 17:04:38 | grandchild | sean-k-mooney: we do too! thanks :) | |
| 17:04:38 | bauzas | grandchild: fwiesel: I aslo appreciate your efforts in trying to bond with us | |
| 17:04:53 | bauzas | let it be going | |
| 17:05:02 | bauzas | and we'll see during the next weeks how things go | |
| 17:05:12 | bauzas | we're waaaay overlate | |
| 17:05:15 | sean-k-mooney | before we finish the meeting | |
| 17:05:20 | sean-k-mooney | i did have a topic for this week | |
| 17:05:22 | bauzas | shoot, but quick | |
| 17:05:25 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/899381 | |
| 17:05:36 | sean-k-mooney | basically does that need a specless blueprint or bug | |
| 17:06:02 | sean-k-mooney | tl;dr libvirt can now tell use how many sev instnace we can create | |
| 17:06:09 | sean-k-mooney | we have a optional config option to do that | |
| 17:06:25 | sean-k-mooney | this patch will just auto detect the amount and report it | |
| 17:06:43 | sean-k-mooney | it was deferd form the orgianl spec because we need libvirt to be modifed to enable this | |
| 17:06:45 | bauzas | I'm fine with approving it as specless but telling them to create a spec if during the implementation reviews we find some design concerns | |
| 17:07:02 | sean-k-mooney | well its actully more or less ready to merge | |
| 17:07:18 | sean-k-mooney | i have one nit and wanted to answer this question before +2ing it | |
| 17:07:21 | bauzas | given the change is already there, it's fine to review it now and ask for a spec if really needed | |
| 17:07:26 | sean-k-mooney | and stephenfin previously +2'd it | |
| 17:07:31 | bauzas | they wouldn't be impacted by the spec freeze | |
| 17:07:35 | bauzas | which is later | |
| 17:07:51 | bauzas | any controversial opinions on it being specless ? | |
| 17:08:15 | bauzas | looks not | |
| 17:08:29 | bauzas | we'll need tkajinam to create a blueprint | |
| 17:08:33 | sean-k-mooney | ok lets ask tkajinam to file a specless blueprint for this then and review it as normal once done | |
| 17:08:38 | sean-k-mooney | ya | |
| 17:08:42 | sean-k-mooney | ok that was it | |
| 17:08:45 | gibi | look OK to me | |
| 17:08:52 | bauzas | #agreed https://review.opendev.org/c/openstack/nova/+/899381 could be specless | |
| 17:09:21 | bauzas | #action tkajinam to create a blueprint and report it to bauzas so that he can late-approve it given above ^$ | |
| 17:09:33 | bauzas | okay, we're definitely overlate | |
| 17:09:40 | bauzas | really sorry about that | |
| 17:09:42 | bauzas | thanks all | |
| 17:09:46 | bauzas | #endmeeting | |
| 17:09:47 | opendevmeet | Meeting ended Tue Nov 21 17:09:46 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |