| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-05-14 | |||
| 05:45:20 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose specs for ephemeral encryption https://review.opendev.org/c/openstack/nova-specs/+/907654 | |
| 07:19:59 | gibi | bauzas: I'm not sure you saw it in the last weeks meeting log that we discussed https://blueprints.launchpad.net/nova/+spec/shared-security-groups as a specless bp. The cores present had no objection to approve it but we agreed to wait for you to have a final word on it and approve it if it is OK to you too | |
| 07:20:52 | gibi | relevant meeting log https://blueprints.launchpad.net/nova/+spec/shared-security-groups | |
| 07:20:53 | bauzas | gibi: I haven't yet looked at last week's meeting | |
| 07:21:00 | gibi | https://meetings.opendev.org/meetings/nova/2024/nova.2024-05-07-16.00.log.html#l-34 | |
| 07:21:09 | bauzas | btw. today is a specs review day | |
| 07:21:36 | bauzas | gibi: ack, I'll accept it then | |
| 07:21:40 | gibi | thanks | |
| 09:52:34 | opendevreview | Merged openstack/nova stable/2024.1: [ironic] Fix rebooting instance https://review.opendev.org/c/openstack/nova/+/918180 | |
| 09:57:51 | opendevreview | Merged openstack/nova-specs master: Repropose per-process-healthchecks for Dalmatian https://review.opendev.org/c/openstack/nova-specs/+/919050 | |
| 09:57:53 | opendevreview | Merged openstack/nova-specs master: Update and Re-propose "Allow Manila shares to be directly attached to an instance when using libvirt" for Dalmatian https://review.opendev.org/c/openstack/nova-specs/+/913997 | |
| 10:02:39 | sahid | o/ | |
| 11:13:59 | opendevreview | sean mooney proposed openstack/nova master: use futerist for RBD calls https://review.opendev.org/c/openstack/nova/+/917962 | |
| 11:34:23 | gibi | dansmith: when you are around I'm wondering if you were part of the userdata discussion during the PTG as I see discrepancies here https://review.opendev.org/c/openstack/nova-specs/+/863884/7#message-e1f200505bbf81d0b9a254c4634f6fdc477871dd | |
| 11:56:37 | sean-k-mooney | im kind of having a hard time reading and parsing sepcs today so i think im oging to stop unless some one asks me specific for input on somethign | |
| 12:03:41 | opendevreview | Konrad Gube proposed openstack/nova master: Use Cinder's os-extend_volume_completion volume action. https://review.opendev.org/c/openstack/nova/+/873560 | |
| 12:13:05 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Stateless firmware support https://review.opendev.org/c/openstack/nova-specs/+/908297 | |
| 13:01:47 | opendevreview | Merged openstack/nova-specs master: libvirt: Stateless firmware support https://review.opendev.org/c/openstack/nova-specs/+/908297 | |
| 13:24:45 | opendevreview | Stephen Finucane proposed openstack/placement master: Add placement.wsgi.api module https://review.opendev.org/c/openstack/placement/+/919569 | |
| 13:35:03 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: AMD SEV-ES support https://review.opendev.org/c/openstack/nova-specs/+/907702 | |
| 13:40:11 | ykarel | Hi reported https://bugs.launchpad.net/nova/+bug/2065689 and https://bugs.launchpad.net/nova/+bug/2065693, please check if in case those are something known already | |
| 13:41:35 | dansmith | bauzas: gibi: can you guys land this quick? it's blocking a devstack performance improvement: https://review.opendev.org/c/openstack/nova/+/918689 | |
| 13:41:41 | dansmith | it's trivial | |
| 13:42:15 | dansmith | gibi: we had some discussion about it yeah, I can have a look | |
| 13:42:41 | bauzas | dansmith: ack | |
| 13:48:53 | dansmith | gibi: hmm, yeah I feel like some context is missing from the PTG summary.. I thought there was a different conclusion reached about what to do, but now I'm not sure what that is.. maybe bauzas remembers? | |
| 13:58:13 | bauzas | gibi: dansmith: I think we said the owner provided the right version eventually about our concerns | |
| 14:08:55 | gibi | dansmith: if you are OK to use the RPC cast to trigger the regeneration of the config drive instead of having a flag in sysmeta and do the regeneration at spawn then I'm OK with it. But reading back on the comments I felt that you regen at spawn idea was not discussed at the PTG | |
| 14:12:43 | dansmith | gibi: no, it can't be a cast, right? | |
| 14:13:22 | opendevreview | Stephen Finucane proposed openstack/placement master: Integrate pre-commit https://review.opendev.org/c/openstack/placement/+/919579 | |
| 14:13:22 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Remove envdir overrides https://review.opendev.org/c/openstack/placement/+/919578 | |
| 14:13:23 | opendevreview | Stephen Finucane proposed openstack/placement master: pre-commit: Add sphinx-lint https://review.opendev.org/c/openstack/placement/+/919580 | |
| 14:13:35 | gibi | I think it is propsed as a cast now, and the compute sets the result of the instance action based on the success / failure of the regen | |
| 14:14:32 | gibi | the nova-api can check the compute version but should not wait for the regen to happen via an RPC call | |
| 14:14:37 | gibi | as regen takes time | |
| 14:14:54 | opendevreview | sean mooney proposed openstack/nova master: add initial healthcheck support https://review.opendev.org/c/openstack/nova/+/825015 | |
| 14:14:55 | opendevreview | sean mooney proposed openstack/nova master: add healthcheck tracker to nova context https://review.opendev.org/c/openstack/nova/+/829468 | |
| 14:14:55 | opendevreview | sean mooney proposed openstack/nova master: add healthcheck manager to manager base https://review.opendev.org/c/openstack/nova/+/827844 | |
| 14:14:56 | opendevreview | sean mooney proposed openstack/nova master: [WIP] add iniital health check to compute agent https://review.opendev.org/c/openstack/nova/+/906911 | |
| 14:14:56 | opendevreview | sean mooney proposed openstack/nova master: add healthcheck endpoint to proxy commands https://review.opendev.org/c/openstack/nova/+/830703 | |
| 14:14:57 | opendevreview | sean mooney proposed openstack/nova master: [WIP] add libvirt connection healtcheck https://review.opendev.org/c/openstack/nova/+/907424 | |
| 14:15:03 | dansmith | okay but then we've told the http client "yes, we will do that thing" even though it might show up at the compute that has no idea what to do with it | |
| 14:16:09 | gibi | if the service version of the compute is old the the API can fail the instance action, but if the service version is new enough then the api can cast down to the compute to ask for the regen | |
| 14:16:23 | gibi | (except for shelved_offloaded instances) | |
| 14:16:51 | gibi | at least this is how I understand the PTG notes and the spec now | |
| 14:19:17 | dansmith | okay I haven't re-read it this morning yet | |
| 14:20:26 | bauzas | I'll try to review it too today | |
| 14:20:36 | bauzas | but that's an old spec | |
| 14:22:09 | gibi | it was revised around the PTG so the content is fairly fresh | |
| 14:22:41 | opendevreview | Stephen Finucane proposed openstack/placement master: Add placement.wsgi.api module https://review.opendev.org/c/openstack/placement/+/919569 | |
| 14:22:42 | opendevreview | Stephen Finucane proposed openstack/placement master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/placement/+/919582 | |
| 14:25:27 | gibi | Does anybody remember why we name our db contraints like uniq_key_pairs0user_id0name It feels like there is a reason for that specific structure but I never come accross the explanation | |
| 14:25:33 | gibi | https://github.com/openstack/nova/blob/7096423b343ffce9622fd078fc2b3a87fd3386f7/nova/db/api/models.py#L308-L310 | |
| 14:31:50 | dansmith | yeah I think there was a reason, but it might have just been "consistency" | |
| 14:45:18 | bauzas | indeed | |
| 14:56:32 | gibi | OK, then I ask Uggla_ to follow the pattern just for consistency | |
| 14:56:51 | gibi | here https://review.opendev.org/c/openstack/nova/+/912518/2/nova/db/main/models.py#767 | |
| 15:03:42 | Uggla_ | gibi, ok | |
| 15:04:01 | stephenfin | fyi the sqlalchemy 2.x bump *finally* merged, so if you want to free up some resources we can merge these now https://review.opendev.org/c/openstack/nova/+/915562 https://review.opendev.org/c/openstack/placement/+/915568 | |
| 15:10:04 | ykarel | #topic Stable branches | |
| 15:10:09 | ykarel | all good | |
| 15:10:18 | ykarel | #topic Stadium projects | |
| 15:10:22 | ykarel | all green | |
| 15:10:30 | ykarel | #topic Rechecks | |
| 15:10:44 | ykarel | 6/14 bare rechecks | |
| 15:10:53 | frickler | ykarel: wrong channel? | |
| 15:11:04 | ykarel | ahh right | |
| 15:11:07 | ykarel | thx frickler | |
| 15:16:40 | opendevreview | Gorka Eguileor proposed openstack/nova master: Support os-brick specific lock_path https://review.opendev.org/c/openstack/nova/+/849328 | |
| 15:32:46 | bauzas | nova meeting in 30 mins | |
| 15:33:31 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: Remove template files from non-empty directories https://review.opendev.org/c/openstack/nova-specs/+/919597 | |
| 15:56:29 | bauzas | gibi: shit, I forgot I'll need to travel to the French OpenInfra Day in Paris on next Tuesday | |
| 15:56:52 | bauzas | so I couldn't be able to run the meeting | |
| 15:59:13 | gibi | bauzas: I will be off on next Tuesday | |
| 15:59:50 | bauzas | no worries, I'll ask in our meeting today | |
| 16:01:17 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:17 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:17 | opendevmeet | Meeting started Tue May 14 16:01:17 2024 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:17 | bauzas | #startmeeting nova | |
| 16:01:21 | bauzas | hey folks | |
| 16:01:44 | tkajinam | o/ | |
| 16:01:46 | elodilles | o/ | |
| 16:02:04 | gibi | o/ | |
| 16:02:04 | fwiesel | o/ | |
| 16:02:42 | Uggla_ | o/ | |
| 16:02:50 | bauzas | okay, let's softly start | |
| 16:02:58 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:03:07 | bauzas | #topic Bugs (stuck/critical) | |
| 16:03:11 | bauzas | #info No Critical bug | |
| 16:03:16 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:03:21 | bauzas | anything important to tell about bugs ? | |
| 16:03:58 | bauzas | looks not | |
| 16:04:03 | bauzas | #topic Gate status | |
| 16:04:07 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:04:10 | bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:04:15 | bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | |
| 16:04:20 | bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:04:25 | bauzas | #info Please try to provide meaningful comment when you recheck | |
| 16:05:06 | tkajinam | just fyi. global u-c was updated and sqlalchemy was bumped to 2.0. I don't expect any problems with core services since sqlahcmely 2.0 has been tested for some time, but we may need to pay some attention to gate. | |
| 16:05:50 | bauzas | okay, all of the perodics are green, even the nova-emulation one | |