| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-05-10 | |||
| 20:55:16 | opendevreview | Tony Breeds proposed openstack/nova master: [DNM] Testing docs bump with new Sphinx https://review.opendev.org/c/openstack/nova/+/919280 | |
| 20:57:52 | opendevreview | Tony Breeds proposed openstack/placement master: [DNM] Testing docs bump with new Sphinx https://review.opendev.org/c/openstack/placement/+/919312 | |
| #openstack-nova - 2024-05-13 | |||
| 05:45:11 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 05:45:12 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 05:45:12 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 05:45:13 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 05:45:13 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 05:45:14 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 05:45:14 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 05:45:15 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 05:45:15 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 05:45:17 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 05:45:17 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 05:45:19 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 05:45:19 | opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | |
| 05:45:21 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 05:45:21 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 05:45:23 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 05:45:23 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 05:45:25 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 05:45:25 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 05:45:27 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 05:45:27 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 05:45:29 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 05:45:29 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 08:03:59 | sahid | o/ | |
| 08:04:01 | sahid | hello | |
| 08:04:13 | sahid | any chance to make progress for thar one? https://review.opendev.org/c/openstack/nova/+/896512 | |
| 08:04:42 | sahid | it's waiting for while now without any progress | |
| 08:06:58 | bauzas | sahid: your change is on the review queue for sure, so it will be reviewed | |
| #openstack-nova - 2024-05-14 | |||
| 05:23:04 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose specs for ephemeral encryption https://review.opendev.org/c/openstack/nova-specs/+/907654 | |
| 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: tox: Remove envdir overrides https://review.opendev.org/c/openstack/placement/+/919578 | |
| 14:13:22 | opendevreview | Stephen Finucane proposed openstack/placement master: Integrate pre-commit https://review.opendev.org/c/openstack/placement/+/919579 | |
| 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 manager to manager base https://review.opendev.org/c/openstack/nova/+/827844 | |
| 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:56 | opendevreview | sean mooney proposed openstack/nova master: add healthcheck endpoint to proxy commands https://review.opendev.org/c/openstack/nova/+/830703 | |
| 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: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 | |