| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-04-18 | |||
| 16:46:43 | sean-k-mooney | assuming nothign happens btween now and then | |
| 16:47:01 | bauzas | I'd rather prefer to ask rosmaita again and the tc about the reno tooling things | |
| 16:47:22 | bauzas | to see whether we could forward port the deletion notes to B | |
| 16:47:30 | bauzas | brian's proposal was quite ok to me | |
| 16:47:45 | bauzas | but the tooling has to be correctly documented | |
| 16:48:01 | dansmith | that doesn't change anything about the decision, just the mechanics of the notice, to be clear | |
| 16:48:27 | bauzas | yup, that's why I'm still afraid of removing anything in B | |
| 16:48:37 | dansmith | deprecating? | |
| 16:48:37 | bauzas | the plan isn't sold yet | |
| 16:48:50 | dansmith | we can't remove these in B because we didn't warn about it | |
| 16:48:58 | bauzas | removing anything, and by extension deprecating (in this case) | |
| 16:49:01 | sean-k-mooney | ya like if we really wanted to we could just move the 2023.1 git tag so that 27.0.0 has the release note btu that has other problems so i dont really wnat to go there | |
| 16:49:16 | dansmith | no. | |
| 16:49:21 | sean-k-mooney | bauzas: ya not sure where removal is coming form in b | |
| 16:49:25 | bauzas | sean-k-mooney: well, we can't do this or then the distros would hate us | |
| 16:49:27 | sean-k-mooney | that is not what is beign proposed | |
| 16:49:37 | bauzas | sorry, I meant the general case | |
| 16:49:40 | sean-k-mooney | bauzas: hehe i know | |
| 16:50:03 | sean-k-mooney | but if we really wanted to get it in the 27.0.0 release notes im sure that possibel | |
| 16:50:29 | bauzas | I'm saying I'm still quite unhappy with any change proposing to deprecate or remove any thing in B, including in particular the hyperv driver that we speak | |
| 16:50:43 | sean-k-mooney | anyway i think we can likely move forward with the patch and waith for it to go out in 2024.1 | |
| 16:51:01 | sean-k-mooney | then see whtat the state is for 2024.2 and if its still functioanl at that point or not | |
| 16:51:31 | bauzas | dansmith: I haven't read the tc agenda for today, but do you think they will discuss the slurp things we discussed before the PTG ? | |
| 16:51:32 | sean-k-mooney | bauzas: i find it very unreasoonble to disallow deprecateion or removals in B | |
| 16:52:04 | sean-k-mooney | but i agree that we can only remove thing in B if they were deprecated in A or older | |
| 16:52:04 | bauzas | sean-k-mooney: my PTG concern is still legit, we haven't still a solid plan on forward-porting the notes to C | |
| 16:52:12 | dansmith | bauzas: neither have I so I dunno | |
| 16:53:02 | sean-k-mooney | sure but i dont think we should block the removal of deprecated fucntionalty on resolvign it | |
| 16:54:38 | dansmith | tbh, I'm not getting the problem here | |
| 16:54:52 | dansmith | we should backport a reno to 2023.1 about the deprecation, | |
| 16:55:14 | dansmith | we should have another one in 2024.1 for such a large thing | |
| 16:55:30 | dansmith | we should have a deprecation log message in 2023.1, 2023.2, 2024.1 | |
| 16:55:40 | dansmith | I think bauzas' concern is about remembering to do the extra reno in 2024.1, right? | |
| 16:55:41 | sean-k-mooney | yes bauzas and i swtiched topic slightly to talke about deprecation/removals in B in general | |
| 16:55:50 | sean-k-mooney | yes | |
| 16:55:50 | bauzas | dansmith: correct | |
| 16:55:57 | bauzas | dansmith: your plan looks good then | |
| 16:56:26 | bauzas | provided we don't forget to mention the deprecation in 2024.1 | |
| 16:56:32 | dansmith | sean-k-mooney: well, you seemed to go back to the specific with the "block removing on resolving" | |
| 16:56:43 | sean-k-mooney | well in 2024 we would be annouching that we intend to remvoe it effectivly | |
| 16:56:49 | sean-k-mooney | if that what we decied at that point | |
| 16:57:00 | sean-k-mooney | were as right now we are just saying its untested | |
| 16:57:13 | bauzas | sean-k-mooney: dansmith: I apologize, I mixed two problems | |
| 16:57:14 | dansmith | bauzas: to be honest, the removal of it is not that important, IMHO | |
| 16:57:21 | sean-k-mooney | and deprecated but no plan to remove it in any partical release | |
| 16:57:22 | bauzas | don't disagree | |
| 16:57:26 | bauzas | dansmith: ^ | |
| 16:57:29 | dansmith | bauzas: saying it's deprecated is a lot more important.. we can remove it when it becomes an _actual_ problem | |
| 16:57:35 | bauzas | true | |
| 16:57:46 | bauzas | ok, looks like we're settled then | |
| 16:57:48 | dansmith | which, if it's been long enough, doesn't matter when we do it | |
| 16:57:51 | bauzas | I can vote | |
| 16:59:05 | sean-k-mooney | anyway i need to go to another meeting now | |
| 16:59:25 | sean-k-mooney | bauzas: as an fyi i plan to submit the patch to remove the AZ filter next week when i have time to write it | |
| 16:59:44 | bauzas | ack | |
| 17:00:08 | sean-k-mooney | it was ment to be done like in xena or yoga. i have a version somewhere but im jsut going to do it from scratch | |
| 17:01:58 | bauzas | sean-k-mooney: you know that by merging this without having a tooling in place, we basically leave myself to be responsible of the release note forward-porting, right? :) | |
| 17:02:14 | bauzas | (just saying this here as well, as a side discussion occurs somewhere else) | |
| 17:09:46 | bauzas | https://review.opendev.org/c/openstack/nova/+/863910 is sent to the gate | |
| 17:12:15 | dansmith | so, | |
| 17:12:30 | dansmith | marking it as experimental back then made more sense than I think it does now | |
| 17:12:38 | dansmith | (I realize I'm +2 on that) | |
| 17:13:14 | dansmith | I think we do need to mark it as actually deprecated to be clean here, because we know it has no maintenance horizon, the dependent library is abandoned and in danger, etc | |
| 17:18:18 | sean-k-mooney | dansmith: we are technially doing both | |
| 17:18:27 | sean-k-mooney | its deprecated in the relesae notes | |
| 17:18:41 | dansmith | no it's not | |
| 17:18:42 | sean-k-mooney | but we use experimantal in the logs becasue we did not want to call it deprecated for reasons | |
| 17:18:52 | dansmith | https://review.opendev.org/c/openstack/nova/+/863910/1/releasenotes/notes/hyperv-experimental-antelope-372e18a05cafc295.yaml | |
| 17:18:59 | dansmith | it says "may be removed" but no | |
| 17:19:03 | dansmith | not "deprecated" | |
| 17:19:08 | sean-k-mooney | deprecations: | |
| 17:19:15 | sean-k-mooney | its in the deprecation section | |
| 17:19:25 | dansmith | meh | |
| 17:20:25 | dansmith | it's too vague at this point.. we should be saying it's deprecated and will be removed | |
| 17:20:46 | dansmith | the log is probably even more important than the reno | |
| 17:21:24 | dansmith | 2% of deployments are using hyperv...we really should not leave any room for misinterpreting what we know is going to happen | |
| 17:23:12 | sean-k-mooney | ok im fine with refining it but i was ok with the fact it was in the deprecation section to track that it was deprecated. | |
| 17:55:54 | opendevreview | Merged openstack/nova stable/yoga: Unify placement client singleton implementations https://review.opendev.org/c/openstack/nova/+/858997 | |
| 18:17:14 | opendevreview | Merged openstack/nova stable/yoga: Avoid n-cond startup abort for keystone failures https://review.opendev.org/c/openstack/nova/+/858998 | |
| 19:23:16 | opendevreview | sean mooney proposed openstack/nova master: add hypervisor version weigher https://review.opendev.org/c/openstack/nova/+/880231 | |
| 19:25:37 | opendevreview | sean mooney proposed openstack/nova master: add hypervisor version weigher https://review.opendev.org/c/openstack/nova/+/880231 | |
| #openstack-nova - 2023-04-19 | |||
| 09:09:58 | opendevreview | Jorge San Emeterio proposed openstack/nova master: Have host look for CPU controller of cgroupsv2 location. https://review.opendev.org/c/openstack/nova/+/873127 | |
| 11:45:40 | opendevreview | Tobias Urdin proposed openstack/nova master: Fix wrong nova-manage command in upgrade check https://review.opendev.org/c/openstack/nova/+/880819 | |
| 13:16:16 | auniyal | sean-k-mooney, bauzas, gibi, dansmith, melwitt, artom and other cores can you please have look on these patches https://etherpad.opendev.org/p/release-liaison-PatchesToReview | |
| 13:16:23 | auniyal | some of them are good to merge | |
| 13:26:49 | opendevreview | Merged openstack/nova master: hyperv: Mark driver as experimental https://review.opendev.org/c/openstack/nova/+/863910 | |
| 13:36:03 | artom | auniyal, I'm not a core :) | |
| 14:41:12 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2023.2 Bobcat https://review.opendev.org/c/openstack/nova/+/876447 | |
| 15:07:46 | IPO | Hello All. xena will be on EM soon. Could you please review https://review.opendev.org/c/openstack/cinder/+/876474 ? | |
| 15:08:36 | IPO | Sorry, this one: https://review.opendev.org/c/openstack/nova/+/829804 | |
| 15:14:44 | elodilles | IPO: nova has already released its final xena release, and actually the xena-em transition patch is also about to merge. (note that patches can be merged after the transition as well, just upstream releases cannot be done anymore) | |
| 15:36:08 | IPO | I see. Thank you for info. So is there any chance to merge ? | |
| 15:39:51 | IPO | I mean "note that patches can be merged after the transition as well" cover only security patches ? | |
| 15:41:24 | elodilles | IPO: every bug fix can be merged after the Extended Maintenance transition as well - until the branch will be xena-eol tagged (End-of-Life) and deleted | |
| 15:42:40 | IPO | Thanks for comment ! | |
| 15:42:47 | elodilles | no problem | |
| 17:08:42 | sean-k-mooney | bauzas: thanks for the review | |
| 17:09:04 | sean-k-mooney | ill fix the docs issues but propably tomorrow im getting a headache so i think ill call it a day | |
| #openstack-nova - 2023-04-20 | |||
| 07:43:43 | lajoskatona | bauzas: Hi, I opened a bug against Neutron/Nova/Tempest to track what we do with tests like the ones under tempest/api/compute/security_groups | |
| 07:44:01 | lajoskatona | bauzas: here's the bug: https://bugs.launchpad.net/neutron/+bug/2017023 | |