| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-01 | |||
| 13:26:11 | sean-k-mooney | we normally did not dod that in the past however | |
| 13:26:21 | sean-k-mooney | so i think this is all good to go | |
| 13:26:31 | gibi | the trait based capability scheduling looks OK to me | |
| 13:26:45 | gibi | I think dansmith had issues using the capability trait outside of the scheduling | |
| 13:26:55 | dansmith | if we're already scheduling, then traits make plenty of sense | |
| 13:26:57 | dansmith | right | |
| 13:27:10 | sean-k-mooney | yep its in the existing pre filter | |
| 13:27:23 | gibi | bauzas: do you need my +2 or you will send it in? | |
| 13:27:39 | sean-k-mooney | before we started to do these check with placment we would have landed on the host and failed there with hypervior too old or similar | |
| 13:28:11 | bauzas | sorry folks was in 1:1 meeting | |
| 13:28:20 | bauzas | sean-k-mooney: thanks, will reply then with +2 | |
| 13:28:29 | bauzas | it was just an open thought | |
| 13:28:29 | gibi | ack, then I'm not needed there :) | |
| 13:28:51 | bauzas | I just want to make sure that operators know they need to upgrade all their hosts if so | |
| 13:29:03 | bauzas | but that's understandable | |
| 13:29:32 | sean-k-mooney | well its as you said | |
| 13:29:44 | sean-k-mooney | the dont actully have to but it will be capsity limited | |
| 13:30:12 | sean-k-mooney | but that kind of to be expected that you cant use new feature on old nodes | |
| 13:30:20 | bauzas | yup, agreed | |
| 13:30:31 | bauzas | at least with traits | |
| 13:30:36 | sean-k-mooney | well even without | |
| 13:30:54 | bauzas | without, we ask for compute service checks in general | |
| 13:31:00 | sean-k-mooney | yep | |
| 13:31:09 | sean-k-mooney | but that is why i said the traits patch chould be first | |
| 13:31:21 | bauzas | well, if both merge for Zed, I'm cool | |
| 13:31:32 | sean-k-mooney | and the other one secodn and why i said in the current order they need to merge togather | |
| 13:31:51 | bauzas | yes and no | |
| 13:32:14 | bauzas | you could merge the first, this is just you won't be able to get the feature until you upgrade all | |
| 13:32:14 | sean-k-mooney | bauzas: let me rephase i dont want to merge only one of thoes two patches | |
| 13:32:25 | dansmith | gibi: sean-k-mooney: are we FFEing the user_data stuff such that I should try to bang out the rest of the RPC stuff ASAP? | |
| 13:32:32 | bauzas | won't be able to *be sure* to get the feature | |
| 13:32:43 | bauzas | dansmith: eeek, context ? | |
| 13:32:47 | bauzas | oh, the rpc call | |
| 13:33:02 | bauzas | lemme just send to the gate ricolin's work | |
| 13:33:09 | sean-k-mooney | bauzas: that an di guess i need to rev my follow up patch with actul tests | |
| 13:33:45 | sean-k-mooney | dansmith: im not agasint doint that if you think you will have time | |
| 13:34:04 | sean-k-mooney | but that probaly the main RFE that i think FFE might make sense for | |
| 13:34:20 | dansmith | sean-k-mooney: I was expecting to see a rev of the patch to move the regen to after the instance is destroyed and fix the actual writing that was failing | |
| 13:34:30 | dansmith | so I hadn't rebased my RPC stuff yet | |
| 13:34:31 | sean-k-mooney | the only other one might be the PCI series ebut have nto looked at it for two days | |
| 13:34:40 | dansmith | but that has to happen first right? | |
| 13:34:52 | sean-k-mooney | oh i wrote a ptach to fix config drive | |
| 13:35:10 | dansmith | oh is that ths? https://review.opendev.org/c/openstack/nova/+/855351/1 | |
| 13:35:16 | sean-k-mooney | yes | |
| 13:35:41 | gibi | sean-k-mooney: you are +2 up until the healing patches in the PCI and that is the realistic target there. If stephenfin will have no time to review them today then I will ask for an FFE for that. We let the scheduling part slip in any case | |
| 13:35:51 | bauzas | the mutable userdata has API impact | |
| 13:35:52 | dansmith | okay but that would need to be squashed or go in front right? | |
| 13:35:59 | bauzas | but I'm OK with FFE'ing if needed | |
| 13:36:16 | dansmith | bauzas: it would also be about half not-very-reviewed code at this point based on the look of the configdrive patch | |
| 13:36:30 | dansmith | so not really "just didn't get reviewed in time" | |
| 13:36:49 | sean-k-mooney | dansmith: yes or they split the patch in to non config drive and config drive | |
| 13:37:20 | dansmith | that would mean two microversions for effectively the same thing | |
| 13:37:30 | sean-k-mooney | dansmith: i basically started my patch so they would have a refernce for what needed to be done | |
| 13:37:36 | dansmith | unless we do the config regen and rpc ahead of exposing int the api | |
| 13:37:39 | bauzas | dansmith: I think we reviewed it good, but we got a bone | |
| 13:37:59 | bauzas | so, I'm OK with giving more time to review that bone fix | |
| 13:38:06 | dansmith | bauzas: I'm saying the code that needs to be written to make it landable would probably double the actual code in the patch | |
| 13:38:09 | sean-k-mooney | dansmith: yep we coudl do that so intialy it would not be invokeable from the api btut he code would be in place | |
| 13:38:30 | dansmith | sean-k-mooney: yeah that seems better to me | |
| 13:38:32 | bauzas | dansmith: then, we need to take this extratime to balance the risks and maybe not merge it | |
| 13:38:52 | bauzas | or decide we only merge half the things | |
| 13:39:12 | sean-k-mooney | dansmith: so you could proably combin your rpc code into the pathc i started then we could flip the order of them | |
| 13:39:13 | gibi | we could merge the part that support user data update with non config drive instances. | |
| 13:39:26 | sean-k-mooney | but that also means we need to other pepoel to agree to review this | |
| 13:39:33 | sean-k-mooney | since you and i are basicaly out at that point | |
| 13:40:12 | gibi | I can review the patches (I will be around 18:00 CEST today but I can spend time early tomorrow too) | |
| 13:40:31 | gibi | * I will be around until 18:00 CEST today | |
| 13:40:47 | dansmith | sean-k-mooney: right, that's a problem too | |
| 13:41:21 | dansmith | sean-k-mooney: so a patch in front that adds the flag to the reboot call, then the patch to make it regeneratable, then the api patch to do it for both types would be cleanest I think | |
| 13:41:47 | sean-k-mooney | dansmith: ya that sounds like a plan | |
| 13:42:07 | sean-k-mooney | it keeps just one micorversion | |
| 13:42:57 | sean-k-mooney | i can go writeh the tests need to make the rebuild patch complete if we agree to proceed in this direction | |
| 13:43:57 | dansmith | sean-k-mooney: ack, I'll rebase my rpc stuff | |
| 13:44:00 | sean-k-mooney | dansmith: it would proably make sense for you to write your patch directly off master and i could rebase and flip the order when your done | |
| 13:44:14 | dansmith | sean-k-mooney: yeah | |
| 13:44:29 | bauzas | I'm OK with having 3 patches, the API one being the latest | |
| 13:44:48 | bauzas | and this being FFE, so we have 1 week to correctly look at all of this | |
| 13:47:50 | bauzas | elodilles: when is the exact deadline for cycle highlights ? | |
| 13:47:54 | bauzas | today or tomorrow ? | |
| 13:48:32 | bauzas | one day, the marketing team will understand how crazy it is to ask for feature docs deliveries at the FeatureFreeze (and even before) | |
| 13:48:45 | bauzas | while RC1 time would be so easier | |
| 13:49:15 | bauzas | this is just, I need to take 1 hour of my time for writing something while we're on fire | |
| 13:50:53 | sean-k-mooney | on the plus side tempest-integrated-compute forces config drive and that passed so at least that change in logic works for normal boots https://zuul.opendev.org/t/openstack/build/98cfa86b93974d79b1d43c29433d7e48/log/controller/logs/etc/nova/nova-cpu_conf.txt#17 | |
| 13:52:26 | sean-k-mooney | i also checked that locally but nice to see it in ci too. | |
| 13:53:33 | elodilles | bauzas: i think there is no strict deadline for today, we will merge it tomorrow if it will be ready only around that time :) | |
| 13:54:16 | sean-k-mooney | elodilles: i think the remidner was asking for it yesterday | |
| 13:54:47 | opendevreview | Justas Poderys proposed openstack/nova-specs master: Improve multiqueue network adapter support https://review.opendev.org/c/openstack/nova-specs/+/855514 | |
| 13:54:54 | sean-k-mooney | but i guess bauzas point was we might not know the full feature set until we hit FF | |
| 13:54:59 | bauzas | elodilles: surely there is no problem, but I wonder why such rush | |
| 13:55:01 | sean-k-mooney | or RC1 if we have FFEs | |
| 13:55:12 | bauzas | elodilles: I just replied to your email fwiw | |
| 13:55:27 | bauzas | I guess my point is not if I can deliver later | |
| 13:55:52 | bauzas | but rather asking for an official move to RC1 | |
| 13:56:10 | sean-k-mooney | elodilles: oh you just said this week | |
| 13:56:34 | bauzas | sean-k-mooney: https://releases.openstack.org/zed/schedule.html#cycle-highlights | |
| 13:56:59 | sean-k-mooney | yep | |
| 13:57:05 | sean-k-mooney | looking at it now | |
| 13:57:10 | bauzas | the schedule officially says the week of Zed-3 | |
| 13:57:25 | bauzas | this is non-trivial | |
| 13:57:45 | sean-k-mooney | i can understand that they might want to have them read for RC1 | |