| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-16 | |||
| 16:28:58 | bauzas | that's always a good idea (to release os-traits release way earlier than milestone-3) | |
| 16:29:06 | bauzas | ditto with os-vif if required | |
| 16:29:41 | tkajinam | ok, that's nice. :-) | |
| 16:30:08 | bauzas | once the spec is approved, I don't see any reason to hold a os-traits patch | |
| 16:30:36 | bauzas | tkajinam: accordingly, please ping me if you want once your both specs are validated | |
| 16:30:47 | sean-k-mooney | i dont see why not | |
| 16:31:10 | bauzas | I just can't remember the state of both specs :) | |
| 16:31:22 | bauzas | so I'll first look whether they're accepted or not | |
| 16:31:54 | tkajinam | the SEV-ES work still needs spec approval but the other two have specs approved | |
| 16:32:14 | tkajinam | I'll update the status after the meeting | |
| 16:32:46 | bauzas | thanks | |
| 16:32:59 | bauzas | (yeah I remembered about the firmware one) | |
| 16:33:15 | sean-k-mooney | im not sure if we need the es trait | |
| 16:33:27 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | |
| 16:33:32 | sean-k-mooney | didnt we say we would use a seperate resouce class for that | |
| 16:33:45 | bauzas | for SEV-ES ? | |
| 16:33:50 | sean-k-mooney | yep | |
| 16:34:03 | sean-k-mooney | oh no we didnt | |
| 16:34:10 | sean-k-mooney | its the same resouce class | |
| 16:34:12 | tkajinam | sean-k-mooney, we agreed using trait, not resource class to distinguish SEV and SEV-ES, IIUC | |
| 16:34:13 | sean-k-mooney | just with a triat | |
| 16:34:18 | sean-k-mooney | it been a while since i looked | |
| 16:34:19 | tkajinam | yes | |
| 16:34:25 | sean-k-mooney | ya ok | |
| 16:34:54 | bauzas | yeah https://etherpad.opendev.org/p/nova-dalmatian-ptg#L498 | |
| 16:35:04 | bauzas | (I was looking at the etherpad) | |
| 16:35:24 | bauzas | we didn't wanted to differentiate all the SEV supports | |
| 16:35:37 | bauzas | s/supports/types | |
| 16:35:46 | bauzas | anything else to discuss ? | |
| 16:36:07 | sean-k-mooney | not on this topic at least not form me | |
| 16:36:39 | tkajinam | nothing else from me | |
| 16:41:13 | bauzas | sorry, I looked at other stuff | |
| 16:41:19 | bauzas | thanks folks | |
| 16:41:23 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-16-16.00.log.html | |
| 16:41:23 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-16-16.00.txt | |
| 16:41:23 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-16-16.00.html | |
| 16:41:23 | opendevmeet | Meeting ended Tue Jul 16 16:41:23 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:41:23 | bauzas | #endmeeting | |
| 16:41:33 | fwiesel | thank you too, bye | |
| 16:41:35 | elodilles | thanks o/ | |
| 16:42:10 | tkajinam | thanks ! | |
| 16:43:01 | tkajinam | bauzas, if you have a moment, could you approve https://blueprints.launchpad.net/nova/+spec/libvirt-stateless-firmware ? its spec was already merged https://review.opendev.org/c/openstack/nova-specs/+/908297 | |
| 16:43:10 | tkajinam | I remember we discussed the missing bp link which was fixed by https://review.opendev.org/c/openstack/nova-specs/+/920612 | |
| 16:43:26 | bauzas | tkajinam: as I said, I'll update Launchpad for all approved specs | |
| 16:43:49 | bauzas | I was on PTO last 10 days, so I'll both check the open specs but also the new merged ones | |
| 16:44:11 | bauzas | (and then update the status etherpad for adding series of the accepted specs) | |
| 16:44:13 | tkajinam | ah, ok. I'll recheck it after that "batch update" | |
| 16:48:28 | opendevreview | Merged openstack/os-traits master: Add traits for TPM models https://review.opendev.org/c/openstack/os-traits/+/909107 | |
| 19:07:15 | melwitt | thanks sean-k-mooney and bauzas for the spec reviews 🙏 | |
| #openstack-nova - 2024-07-17 | |||
| 02:03:37 | opendevreview | melanie witt proposed openstack/nova master: nova-manage: Add flavor scanning to migrate_to_unified_limits https://review.opendev.org/c/openstack/nova/+/924110 | |
| 05:04:40 | ykarel | y | |
| 05:06:38 | opendevreview | melanie witt proposed openstack/nova master: Add config option [quota]strict_unified_limits https://review.opendev.org/c/openstack/nova/+/924025 | |
| 05:29:30 | opendevreview | melanie witt proposed openstack/nova master: Add config option [quota]strict_unified_limits https://review.opendev.org/c/openstack/nova/+/924025 | |
| 07:46:11 | opendevreview | melanie witt proposed openstack/nova master: Add config option [quota]strict_unified_limits https://review.opendev.org/c/openstack/nova/+/924025 | |
| 09:46:45 | bauzas | hmmm, looks like we have a problem with the promote job https://specs.openstack.org/openstack/nova-specs/specs/2024.2/approved/ | |
| 09:46:48 | bauzas | frickler: ^ | |
| 09:47:12 | frickler | looking | |
| 09:47:29 | bauzas | frickler: https://review.opendev.org/q/project:openstack/nova-specs+status:merged as you can see, we merged more specs | |
| 09:47:56 | bauzas | also, I don't see the new ones in https://specs.openstack.org/openstack/nova-specs/specs/2024.2/ | |
| 09:50:38 | frickler | bauzas: yes, noonedeadpunk reported a similar issue yesterday. this may need a zuul fix which is still pending I think | |
| 09:52:05 | frickler | yes, same issue, see https://zuul.opendev.org/t/openstack/build/6a6f5c3fff4840c29d8f9b23b3a5280a | |
| 09:54:56 | bauzas | frickler: okay, thjank | |
| 09:55:00 | bauzas | thanks* | |
| 10:31:33 | opendevreview | Shu Juan Zhang proposed openstack/nova master: Fix the 409 error false alarm for instance resize https://review.opendev.org/c/openstack/nova/+/924318 | |
| 10:31:36 | opendevreview | zhou zhong proposed openstack/nova master: nova-manage: modify image properties in request_spec https://review.opendev.org/c/openstack/nova/+/924319 | |
| 10:32:54 | opendevreview | zhou zhong proposed openstack/nova master: nova-manage: modify image properties in request_spec https://review.opendev.org/c/openstack/nova/+/924319 | |
| 11:43:01 | opendevreview | zhou zhong proposed openstack/nova master: nova-manage: modify image properties in request_spec https://review.opendev.org/c/openstack/nova/+/924319 | |
| 12:01:37 | opendevreview | zhou zhong proposed openstack/nova master: nova-manage: modify image properties in request_spec https://review.opendev.org/c/openstack/nova/+/924319 | |
| 12:05:13 | opendevreview | zhou zhong proposed openstack/nova master: nova-manage: modify image properties in request_spec https://review.opendev.org/c/openstack/nova/+/924319 | |
| 12:52:46 | opendevreview | zhou zhong proposed openstack/nova master: nova-manage: modify image properties in request_spec https://review.opendev.org/c/openstack/nova/+/924319 | |
| 13:08:40 | sean-k-mooney | bauzas: on the flip side the next time the promote job runs it will promote all prior content too | |
| 13:08:57 | sean-k-mooney | bauzas: so its not really that much of a concern | |
| 14:53:25 | bauzas | can some nova-core look at this simple fix ? https://review.opendev.org/c/openstack/nova/+/914761 | |
| 14:57:12 | opendevreview | Merged openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | |
| 15:29:39 | jlejeune | bauzas: hello, I just see your comment on my small fix in unit tests: https://review.opendev.org/c/openstack/nova/+/924148, yesterday you told me a launchpad bug report is useless as it's just internal changes, do I really need to create one for that ? | |
| 15:39:48 | bauzas | jlejeune: because when you asked me, I wasn't knowing it was due to a specific enviroment usager | |
| 15:39:51 | bauzas | usage* | |
| 15:40:05 | bauzas | hence now I'd like to exactly know the exception we get | |
| 15:40:43 | bauzas | but if you want, just reply to me in the gerrit patch with an example of the exception we return, and then I'll tell you to add a larger comment then on our patch | |
| 15:41:13 | bauzas | but like I said, we try to not `except Exception` unless this is really needed | |
| 15:49:17 | jlejeune | yeah yeah I understand that of course, like I said I've juste copied/pasted what someone else did on other patchs for the same exception :) | |
| 15:49:36 | jlejeune | *on other tests | |
| 15:50:53 | jlejeune | I will check what's the exact exception, it raised when calling subprocess.execute() function | |
| 16:01:23 | bauzas | jlejeune: thanks :) | |
| 16:29:35 | bauzas | can someone just ping me quickly ? that's for a test | |
| 16:29:46 | frickler | bauzas: ping | |
| 16:29:59 | bauzas | frickler: thanks | |
| 20:21:08 | opendevreview | ribaudr proposed openstack/nova master: Amend DB model add a unique constraint. https://review.opendev.org/c/openstack/nova/+/912518 | |
| 20:21:08 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 20:21:09 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 20:21:09 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 20:21:10 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 20:21:10 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 20:21:11 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 20:21:12 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 20:21:12 | 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 | |
| 20:21:14 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 20:21: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 | |
| 20:21:16 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares https://review.opendev.org/c/openstack/nova/+/860287 | |
| 20:21:16 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |