| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-02-28 | |||
| 09:44:01 | sean-k-mooney | i dont think it need to be in nova sycle highlights | |
| 09:44:15 | sean-k-mooney | i think it shoudl go out in the overall release prelude | |
| 09:44:29 | bauzas | this is anyway too late for the cycle highlights | |
| 09:44:36 | bauzas | I'll just mention it in the prelude | |
| 09:44:48 | sean-k-mooney | we should ahve one section that covers all the secure rbac work for all projects | |
| 09:44:57 | bauzas | and I'll actually look at the generated relnotes to see if I missed anything else | |
| 09:45:26 | bauzas | sean-k-mooney: don't disagreed on your point for the cycle highlights | |
| 09:45:34 | bauzas | this is more a cross-project effort | |
| 09:45:45 | bauzas | but let's mention the nova related implications in the prelude | |
| 09:45:59 | sean-k-mooney | sure | |
| 09:46:13 | sean-k-mooney | it should also be in the general release notes in the upgrades section | |
| 09:46:13 | gibi | +1 | |
| 09:46:31 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/866218/13/releasenotes/notes/enable-enforce-scope-and-new-defaults-14db8c75b263b599.yaml | |
| 09:46:47 | bauzas | I'm pretty sure the marketing folks will handle that info correctly despite not being said in our cycle highlights :) | |
| 09:47:09 | sean-k-mooney | i would have preferd not to have it in the cycle highlights anyway | |
| 09:47:55 | sean-k-mooney | as i said i think its better to comunitcate this in a cross project way since it really needs to be enabled in a corrdenated way | |
| 09:48:13 | sean-k-mooney | with that said we really do need to complete the service role this cycle | |
| 09:48:47 | sean-k-mooney | to keep on track with the comunity goal schduel | |
| 09:49:15 | opendevreview | Maksim Malchuk proposed openstack/nova stable/xena: Fix to implement 'pack' or 'spread' VM's NUMA cells https://review.opendev.org/c/openstack/nova/+/829804 | |
| 09:49:20 | sean-k-mooney | im going to revert the logging patch now https://review.opendev.org/c/openstack/nova/+/873584 | |
| 09:49:34 | bauzas | sean-k-mooney: yes about the service role, but gmann still has a -W on it | |
| 09:49:34 | sean-k-mooney | bauzas: unless you object? | |
| 09:50:07 | bauzas | sean-k-mooney: nope, go for it, it's in the list-of-patches-to-merge-before-next-day | |
| 09:50:30 | sean-k-mooney | cool its in the gate | |
| 09:51:57 | sean-k-mooney | this is the base of the first offical SLU release | |
| 09:52:04 | sean-k-mooney | do we need to mentiaon anything beyond that | |
| 09:52:17 | sean-k-mooney | we dont offically support it form yoga | |
| 09:52:31 | sean-k-mooney | that was a test of our tooling/process | |
| 09:52:45 | sean-k-mooney | so while it should work i would not adivse people to use it | |
| 09:53:34 | sean-k-mooney | there wasnt an expecation taht all project support Yoga to Antelope | |
| 10:02:01 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add the 2023.1 Antelope prelude section https://review.opendev.org/c/openstack/nova/+/875380 | |
| 10:21:47 | stephenfin | sean-k-mooney: https://review.opendev.org/c/openstack/oslo.db/+/875627 that will fix the annoying warnings you've been seeing in the functional tests | |
| 10:37:16 | sean-k-mooney | nice unfortuhnetly that wont be in 2023.1 at this point | |
| 10:37:50 | stephenfin | yeah, we'll have to backport it | |
| 10:40:13 | sean-k-mooney | i assume once we cut rc1 tomrrow you would like your SQLA2.0 seriese to land sooner rather then later | |
| 10:40:39 | sean-k-mooney | i.e. droping the legacy migration and the other patches you have up | |
| 10:40:52 | stephenfin | yes, please. get it in early | |
| 10:41:06 | sean-k-mooney | ya that is what i was thinking | |
| 10:41:33 | sean-k-mooney | ill try and do a pass of it next week | |
| 10:43:24 | sean-k-mooney | conf.get_location(key, group=group.name).location == | |
| 10:43:26 | sean-k-mooney | cfg.Locations.opt_default | |
| 10:43:36 | sean-k-mooney | so i tought here was a cleaner way to do that | |
| 10:44:03 | sean-k-mooney | either usign in or a method to check it the option was set from the default | |
| 10:44:32 | stephenfin | There could be. I'm simply not aware of it if so | |
| 10:45:11 | sean-k-mooney | i have not used it personally but i vaguely recall it form the docs | |
| 10:45:29 | sean-k-mooney | the patch looks fine to me but i just want to check if that is a thing quickly | |
| 10:46:42 | sean-k-mooney | stephenfin: ok no you are doing it the way you are ment too https://docs.openstack.org/oslo.config/latest/reference/locations.html | |
| 10:46:47 | sean-k-mooney | that is what i was recalling | |
| 10:47:01 | sean-k-mooney | i tought there was an is_default() or similar | |
| 10:47:22 | stephenfin | yeah, I explicitly wanted to raise a warning if the application (i.e. nova) was setting it too | |
| 10:47:28 | stephenfin | since that's still wrong | |
| 10:47:42 | sean-k-mooney | actully you could use is_user_controlled | |
| 10:48:11 | sean-k-mooney | ah | |
| 10:48:23 | sean-k-mooney | ok so you want to also support set_default and set_override | |
| 10:48:37 | sean-k-mooney | so is_user_controlled is not correct | |
| 10:48:38 | stephenfin | I want to also raise for the two of those, yes | |
| 10:48:43 | stephenfin | exactly | |
| 10:50:58 | sean-k-mooney | cool captured that in the review an dset Backport-Candidate on it | |
| 10:52:46 | sean-k-mooney | im debating if having Backport-Candidate would be useful for nova et al | |
| 10:53:02 | sean-k-mooney | is it useful for oslo? | |
| 10:54:13 | stephenfin | not really, no | |
| 10:54:57 | sean-k-mooney | ingeneral if the patch is not intentioanlly written to be backportable it wont be | |
| 10:54:59 | stephenfin | The biggest problem is lack of reviewers, not lack of things to review. Also, we don't tend to make many changes nowadays so there aren't many backports | |
| 10:55:30 | sean-k-mooney | so if its just set after the fact its probaly less useful | |
| 10:55:38 | sean-k-mooney | ya reviews are a problem everywhere | |
| 10:55:59 | sean-k-mooney | although oslo with all its repos proably have it wrose then most | |
| 11:46:16 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Have schema for 'lock' action be applied to all microversions. https://review.opendev.org/c/openstack/nova/+/875653 | |
| 13:00:34 | opendevreview | Samuel Kunkel proposed openstack/nova master: fix: amd-sev handle missing img properties https://review.opendev.org/c/openstack/nova/+/874264 | |
| 13:03:18 | opendevreview | Amit Uniyal proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | |
| 13:07:21 | opendevreview | Amit Uniyal proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | |
| 13:09:20 | ratailor_ | #openstack-nova can I get reviews for these patches https://review.opendev.org/q/Ia738a0972b050f549f446c85171d3f33e60ada4f and https://review.opendev.org/q/Id4c8c5f3b32985ac7d3d7c833b82e0876f7367c1 ? | |
| 13:20:53 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Have schema for 'lock' action be applied to all microversions. https://review.opendev.org/c/openstack/nova/+/875653 | |
| 13:21:23 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Have schema for 'lock' action be applied to all microversions. https://review.opendev.org/c/openstack/nova/+/875653 | |
| 13:43:42 | lowercase | stephenfin: I looked into our next enviornment, which also contains a build_requests table with no columns. | |
| 13:43:52 | lowercase | in an identical state to the dev | |
| 13:45:33 | opendevreview | Amit Uniyal proposed openstack/nova stable/wallaby: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/790447 | |
| 13:46:01 | lowercase | and i just checked one of my production databases, also include a build_requests in the same state. select * from build_requests; Empty set (0.000 sec) | |
| 13:52:30 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/845748 | |
| 13:52:41 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/845753 | |
| 13:52:48 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/845754 | |
| 13:59:49 | opendevreview | Sylvain Bauza proposed openstack/nova master: Update to the PTL guide https://review.opendev.org/c/openstack/nova/+/875730 | |
| 14:05:09 | opendevreview | Sylvain Bauza proposed openstack/nova master: Update to the PTL guide https://review.opendev.org/c/openstack/nova/+/875730 | |
| 14:46:00 | bauzas | elodilles: I'm done with the meeting wikipage, feel free to amend it anytime you want | |
| 14:47:55 | elodilles | bauzas: ack, thanks | |
| 14:48:15 | elodilles | bauzas: well, actually, it is done o:) | |
| 14:48:35 | elodilles | no need to update :X | |
| 14:48:51 | bauzas | elodilles: I'll name you Barry Allen | |
| 14:50:53 | elodilles | just a sec, have to google this. :S | |
| 14:51:00 | dansmith | bauzas: why remove the grenade-slip-level job for bobcat? it's not required, but we might as well keep it.. is it because it hasn't yet been updated for zed->bobcat? | |
| 14:54:00 | bauzas | dansmith: good question I was about getting a coffee before pinging you to ask you what you would prefer | |
| 14:54:27 | dansmith | personally I'd rather leave it in place as much as we can, as I think it just adds extra coverage | |
| 14:54:39 | dansmith | however, I need to look at it again, | |
| 14:54:59 | bauzas | dansmith: okay, so we would leave non-voting in a non-SLURP branch and just make it voting after Bobcat RC1 ? | |
| 14:55:14 | dansmith | as it might be that we need to change it so that it tests the right releases, which we can't do until after rc of course | |
| 14:55:34 | dansmith | I would leave it voting personally | |
| 14:55:49 | bauzas | even for like B>D ? | |
| 14:56:10 | dansmith | yeah, I mean, | |
| 14:56:24 | dansmith | are you thinking we'll run into issues with deprecations? | |
| 14:56:50 | dansmith | unless it's something specifically in that test I think we'll be okay.. we've had it enabled for a while and never found any problems in nova (IIRC) | |
| 14:56:52 | bauzas | well, | |