| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-12-12 | |||
| 09:13:14 | frickler | gibi: nice, thx | |
| 10:25:23 | opendevreview | Stephen Finucane proposed openstack/nova master: trivial: Remove legacy API artifact https://review.opendev.org/c/openstack/nova/+/937377 | |
| 10:34:34 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Allow min/max_version arguments to response https://review.opendev.org/c/openstack/nova/+/936364 | |
| 10:34:34 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Allow min/max_version arguments to expected_errors https://review.opendev.org/c/openstack/nova/+/936363 | |
| 10:34:35 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Adjust validation helpers for a single-method future https://review.opendev.org/c/openstack/nova/+/936365 | |
| 10:34:35 | opendevreview | Stephen Finucane proposed openstack/nova master: trivial: Remove legacy API artifact https://review.opendev.org/c/openstack/nova/+/937377 | |
| 10:34:36 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 10:34:36 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Stop using wsgi.Controller.api_version to switch between API versions https://review.opendev.org/c/openstack/nova/+/936366 | |
| 10:34:37 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 10:34:37 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| 10:34:38 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 10:34:39 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use UUIDs in TestOpenStackClient https://review.opendev.org/c/openstack/nova/+/937013 | |
| 10:34:39 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | |
| 10:34:41 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 10:34:41 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 10:34:43 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 10:34:43 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 10:34:45 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server IPs APIs https://review.opendev.org/c/openstack/nova/+/937608 | |
| 10:34:45 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 11:10:28 | opendevreview | Amit Uniyal proposed openstack/nova stable/2024.2: Update Nova bdm with updated swap info https://review.opendev.org/c/openstack/nova/+/937610 | |
| 11:12:49 | opendevreview | Amit Uniyal proposed openstack/nova stable/2024.1: Update Nova bdm with updated swap info https://review.opendev.org/c/openstack/nova/+/937611 | |
| 11:15:59 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.2: Update Nova bdm with updated swap info https://review.opendev.org/c/openstack/nova/+/937612 | |
| 11:28:04 | jkulik | hey folks, I just stumbled over this RR https://review.opendev.org/c/openstack/nova/+/678958 - it seems to have been accepted with workflow +1 in 2019 but never got merged | |
| 11:30:06 | jkulik | We had at the time partly applied these patches downstream as we had problems with `safe_connect`'s retry logic, so I would guess patch is still useful | |
| 11:50:55 | frickler | jkulik: that change could not merge because the parent wasn't merged. according to the discussion there, it looks like a different fix might have been implemented? maybe try to rebase the patch if you think it still would be useful? | |
| 11:59:50 | opendevreview | ribaudr proposed openstack/nova-specs master: WIP: Migrate VFIO devices using kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/937615 | |
| 12:00:07 | jkulik | ok, thanks for the feedback | |
| 12:01:32 | opendevreview | Artem Vasilyev proposed openstack/nova master: Fix unit tests on macOS https://review.opendev.org/c/openstack/nova/+/937616 | |
| 12:11:04 | artem_vasilyev | Hi everyone, I encountered failing unit tests in Nova on macOS. I've fixed these issues by mocking the platform check and filesystem access. I'm not sure if I should open a bug in Launchpad for this — happy to do so if needed. For now, I’d appreciate it if someone could review my patch: https://review.opendev.org/c/openstack/nova/+/937616 | |
| 12:44:54 | gibi | melwitt: bauzas: ^^ this is a simple unit test isolation fix. I'm +2 but we need a second core to land it. | |
| 12:49:24 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add debug log for os-brick multipathing https://review.opendev.org/c/openstack/nova/+/937557 | |
| 13:38:05 | opendevreview | Merged openstack/nova master: Fix unit tests on macOS https://review.opendev.org/c/openstack/nova/+/937616 | |
| 13:38:52 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 13:38:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 13:38:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| 13:38:54 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | |
| 13:38:54 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 13:38:55 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 13:38:55 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use UUIDs in TestOpenStackClient https://review.opendev.org/c/openstack/nova/+/937013 | |
| 13:38:56 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 13:38:57 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 13:38:57 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 13:38:59 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server IPs APIs https://review.opendev.org/c/openstack/nova/+/937608 | |
| 13:38:59 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 16:07:01 | opendevreview | Balazs Gibizer proposed openstack/placement master: DNM: test with breadth-first in tempest https://review.opendev.org/c/openstack/placement/+/937274 | |
| 16:07:01 | opendevreview | Balazs Gibizer proposed openstack/placement master: Add round-robin candidate generation strategy https://review.opendev.org/c/openstack/placement/+/936832 | |
| 17:19:05 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: DNM: test dependency on devstack-plugin-ceph changes https://review.opendev.org/c/openstack/nova/+/918957 | |
| 18:05:31 | opendevreview | Pavlo Shchelokovskyy proposed openstack/python-novaclient master: Adapt for api 2.87 in volume resource https://review.opendev.org/c/openstack/python-novaclient/+/937642 | |
| 20:34:57 | opendevreview | ribaudr proposed openstack/nova-specs master: Migrate VFIO devices using kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/937615 | |
| 20:35:52 | opendevreview | ribaudr proposed openstack/nova-specs master: Enable VFIO devices with kernel variant drivers https://review.opendev.org/c/openstack/nova-specs/+/936407 | |
| 20:39:56 | opendevreview | ribaudr proposed openstack/nova master: [POC] Add managed flag to pci device spec https://review.opendev.org/c/openstack/nova/+/937649 | |
| 20:51:36 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy service and manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| 20:55:35 | gmann | sean-k-mooney[m]: bauzas stephenfin gibi melwitt ^^ I am little late on service/manager role spec but it is up for review. This will be my main focus area after holidays | |
| 21:02:57 | opendevreview | Alan Bishop proposed openstack/nova-specs master: Show swap_progress in attachment details https://review.opendev.org/c/openstack/nova-specs/+/937485 | |
| 21:36:15 | opendevreview | Merged openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| #openstack-nova - 2024-12-13 | |||
| 03:10:31 | opendevreview | Masahito Muroi proposed openstack/nova master: Use boot_roles_count and boot_role_ |
|
| 03:45:21 | opendevreview | Merged openstack/nova master: Reports instance events to the DB regarding attaching and detaching a share https://review.opendev.org/c/openstack/nova/+/927088 | |
| 08:26:14 | opendevreview | Merged openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 10:15:56 | gibi | bauzas: sean-k-mooney[m]: I still need to a confirmation from the core team about the response schema in https://review.opendev.org/c/openstack/nova-specs/+/936140/5#message-ca2738479859956903f463f8ad76d8f8aa568ca8 as this is not a dict of string but a lot more nuanced schema | |
| 10:16:13 | gibi | *need to get a confirmatin | |
| 10:17:25 | bauzas | gibi: fwiw, hints were always intended to be as much generic as possible, hence the AdditionalProperties be set to True | |
| 10:17:52 | bauzas | I was assuming that the response would accordingly be as much generic as possible too | |
| 10:18:22 | bauzas | we could consider the response schema to be equivalenbt | |
| 10:22:34 | gibi | do we have examples in other places in our API where we return an unspecified json blob? | |
| 10:23:09 | gibi | does the sdk/client folks will be able to handle this unspecified output with a meaningful UX? | |
| 10:23:31 | gibi | I'm not against doing this I'm just affraid of the consequnces | |
| 10:23:48 | gibi | especially the yet unknown consequences | |
| 10:31:16 | gibi | my quick search did not show any other req/rsp schema def with additionalProperties:true | |
| 10:31:37 | gibi | stephenfin: ^^ you might be interested | |
| 10:40:20 | opendevreview | Merged openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 10:40:27 | opendevreview | Merged openstack/nova master: Manila shares admin guide documentation https://review.opendev.org/c/openstack/nova/+/871642 | |
| 10:40:34 | opendevreview | Merged openstack/nova master: Refactor test_server_shares: Mock in Base Class and trait verification https://review.opendev.org/c/openstack/nova/+/935861 | |
| 13:03:00 | gibi | Uggla: ^^ that concludes the merging of the manila series, the only outstaning patch is the API schema refactor which has a -1 from stephen | |
| 13:05:57 | opendevreview | ribaudr proposed openstack/nova master: Refactor response schemas for share API https://review.opendev.org/c/openstack/nova/+/936656 | |
| 14:13:51 | opendevreview | Pavlo Shchelokovskyy proposed openstack/python-novaclient master: Adapt for api 2.89 in volume resource https://review.opendev.org/c/openstack/python-novaclient/+/937642 | |
| 17:48:40 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Fix: Rebuild volume backed server with snapshot image https://review.opendev.org/c/openstack/nova/+/916409 | |
| 18:30:07 | opendevreview | Artem Vasilyev proposed openstack/nova master: Fix functional tests and mypy linter on macOS https://review.opendev.org/c/openstack/nova/+/937727 | |
| 23:07:27 | opendevreview | Ghanshyam proposed openstack/nova master: [Trivial] Fix the typo error https://review.opendev.org/c/openstack/nova/+/937744 | |
| #openstack-nova - 2024-12-14 | |||
| 13:59:16 | opendevreview | James Page proposed openstack/nova master: Add RISCV64 QEMU emulation support. https://review.opendev.org/c/openstack/nova/+/889137 | |
| #openstack-nova - 2024-12-16 | |||
| 07:09:45 | artem_vasilyev | Hi, another changeset - this time fixing functional tests and linters for macOS. Please take a look and review, thanks! https://review.opendev.org/c/openstack/nova/+/937727 | |
| 11:51:34 | stephenfin | gibi: replied. I think you're right and we should be a little more specific | |
| 12:23:25 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 12:23:25 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use UUIDs in TestOpenStackClient https://review.opendev.org/c/openstack/nova/+/937013 | |
| 12:23:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 12:23:26 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 12:23:27 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 12:23:27 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 12:23:28 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server IPs APIs https://review.opendev.org/c/openstack/nova/+/937608 | |
| 12:38:29 | gibi | stephenfin: thanks! | |
| 12:40:36 | opendevreview | Merged openstack/osc-placement master: Update bug tracker url https://review.opendev.org/c/openstack/osc-placement/+/908221 | |
| 12:42:03 | opendevreview | Merged openstack/os-traits master: Drop unnecessary 'x' bit from doc config file https://review.opendev.org/c/openstack/os-traits/+/932879 | |
| 15:24:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 15:24:14 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 15:24:21 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| #openstack-nova - 2024-12-17 | |||
| 04:32:15 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy service and manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| 04:33:53 | gmann | melwitt: added the list of APIs proposed for the manager role ^^, please check. | |
| 04:33:54 | gmann | bauzas ^^ I added a note in commit msg as well in spec about "service role proposal is re-proposed and manager role proposal is new in this cycle" | |
| 05:07:18 | opendevreview | Ghanshyam proposed openstack/nova-specs master: Propose API policy service and manager role spec https://review.opendev.org/c/openstack/nova-specs/+/937650 | |
| 06:36:36 | opendevreview | DongHun, Kim proposed openstack/nova master: Improve instance shutdown logic for graceful shutdown https://review.opendev.org/c/openstack/nova/+/937840 | |
| 08:51:23 | opendevreview | DongHun, Kim proposed openstack/nova master: Improve instance shutdown logic for graceful shutdown https://review.opendev.org/c/openstack/nova/+/937840 | |