| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-30 | |||
| 15:29:44 | sean-k-mooney | i think the code is still correct | |
| 15:29:48 | dansmith | that has nothing to do with my concern that we don't know if we're doing it or not | |
| 15:29:59 | sean-k-mooney | but we maybe jsut missing a test for that edge case | |
| 15:30:16 | dansmith | it's not an edge case, it's what we were *always* doing before | |
| 15:30:32 | sean-k-mooney | yes which was incorect | |
| 15:30:35 | dansmith | and if it wasn't problematic for andrewbonney's situation, this patch wouldn't be needed | |
| 15:30:42 | sean-k-mooney | we shoudl never have been doing it on unbind | |
| 15:30:44 | sean-k-mooney | only on detach | |
| 15:31:10 | sean-k-mooney | in both the internal and external dns case we were privously doing it unconditionally | |
| 15:31:21 | sean-k-mooney | now both path only do it if its a detach | |
| 15:32:23 | dansmith | none of that has anything to do with the tests I'm quoting.. I'm talking about the _unit_ test that checks that this method does what it should based on the various inputs, and there's now a conditional that we don't (AFAIK) verify does the right thing | |
| 15:34:07 | sean-k-mooney | detach defautls to true. so test_unbind_ports_reset_dns_name_by_admin whih is the next test calls unbiend iwth it set to ture | |
| 15:34:26 | dansmith | right, and where's the assert that reset_dns was called? | |
| 15:34:30 | sean-k-mooney | and the new test passes detach=false | |
| 15:34:37 | sean-k-mooney | it never existed before | |
| 15:34:41 | dansmith | I KNOW THAT | |
| 15:35:06 | sean-k-mooney | my point is the lack fo that assert is not related to this patch | |
| 15:35:18 | sean-k-mooney | we can add it | |
| 15:35:32 | dansmith | before, there was no question.. it was clearly a gap in the assertion, but at least there was no way for it to not be run.. now there *is* a way for it not to be run, and the test doesn't assert that it is called when we expect it to | |
| 15:37:11 | sean-k-mooney | sure we can add the assert. i have no issue with that | |
| 15:39:38 | sean-k-mooney | we can proably check this in the funtional test as well. we jsut need to replace shelve with detach | |
| 15:43:15 | dansmith | that's probably something we should be doing anyway.. making sure that this gets called properly in the high-level op that it purports to change | |
| 15:43:39 | sean-k-mooney | we had very minimal test coveage wehn thsi was added to nova in the first place https://github.com/openstack/nova/commit/997d8f516cee99b4e16429d13ca5cf7fc05166aa | |
| 15:43:49 | sean-k-mooney | so its as good a time to improve that as any | |
| 15:46:06 | sean-k-mooney | well actully this funciton was added in https://github.com/openstack/nova/commit/1b797f6f7e99fdef380340d6fe29e4004be48781 later | |
| 15:53:58 | opendevreview | Dan Smith proposed openstack/nova master: Don't reset port dns_name when shelving instances https://review.opendev.org/c/openstack/nova/+/919961 | |
| 15:54:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server migrations API https://review.opendev.org/c/openstack/nova/+/956143 | |
| 15:54:50 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server tags API https://review.opendev.org/c/openstack/nova/+/956144 | |
| 15:54:51 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server metadata APIs https://review.opendev.org/c/openstack/nova/+/956145 | |
| 18:18:24 | sean-k-mooney | gmaan: i added my +2 and some other comments. i think im fine to refiene the docs/release not in a follow up but holding +w so you can read the comments | |
| 18:18:47 | sean-k-mooney | if no one else provides feedback ill add +w tomorrow or others are freee to do so | |
| 18:18:56 | gmaan | sean-k-mooney: checking | |
| 18:23:15 | sean-k-mooney | gibi: dan adressed there testing comment in https://review.opendev.org/c/openstack/nova/+/919961 woudl you mind looking at that again tomorow when your aroudn and reappoving it if your still happy. i just recheked it because of a kernel panic so hopefully it will be green in the morning | |
| 18:29:00 | gmaan | sean-k-mooney: replied, 'i think im fine to refiene the docs/release not in a follow up' - > you mean to you are ok to do it in follow up | |
| 18:29:28 | gmaan | ohk I think 'release not' -> 'release note' got it now | |
| 18:29:36 | gmaan | fixing those in follow up | |
| 18:29:40 | sean-k-mooney | ya im fine with a follow up | |
| 18:29:45 | gmaan | cool | |
| 18:29:59 | sean-k-mooney | you can respin if you want but i think tis udnerstandable just the wording could be clearer | |
| 19:27:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document native threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 19:27:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 19:27:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 19:27:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 19:28:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 19:28:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 19:28:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 19:28:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 19:28:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI]Make nova-tox-py312-threading voting https://review.opendev.org/c/openstack/nova/+/955791 | |
| 19:28:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 19:28:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | |
| 19:28:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | |
| 19:28:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | |
| 19:28:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | |
| 19:28:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 19:28:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document native threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 19:28:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 19:28:35 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 19:28:35 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 19:28:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 19:28:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 19:28:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 19:28:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| 19:28:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: [CI]Make nova-tox-py312-threading voting https://review.opendev.org/c/openstack/nova/+/955791 | |
| 19:28:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: [vncproxy]Handle ssl.wrap_socket removal in py312 https://review.opendev.org/c/openstack/nova/+/955915 | |
| 19:28:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make RBD Tpool usage conditional https://review.opendev.org/c/openstack/nova/+/956089 | |
| 19:28:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make libvirt Tpool proxying conditional https://review.opendev.org/c/openstack/nova/+/956090 | |
| 19:28:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix ProviderTree copying with threading Lock https://review.opendev.org/c/openstack/nova/+/956091 | |
| 19:28:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]Further categorization of disabled unit tests https://review.opendev.org/c/openstack/nova/+/956092 | |
| 19:28:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 19:29:41 | gibi | dansmith: sean-k-mooney: ^^ fixed the doc patch and rebased on recent master | |
| 19:31:02 | opendevreview | Ghanshyam proposed openstack/nova master: Fix doc comment on manager role change https://review.opendev.org/c/openstack/nova/+/956189 | |
| 19:33:30 | gmaan | sean-k-mooney: stephenfin ^^ fixing doc comments | |
| 19:33:36 | sean-k-mooney | gibi: cool ill take a look soon | |
| 19:33:45 | gibi | thanks | |
| 19:34:06 | sean-k-mooney | im going to try an wrap up soon but if i dont get to it before i finsih ill start with it in the morning | |
| 19:48:08 | dansmith | hoo boy | |
| 19:55:51 | sean-k-mooney | gibi: i left a question on https://review.opendev.org/c/openstack/nova/+/953436 but up to thre im +2 im goign to swap to gmaan's doc followup and call it a day | |
| 20:56:13 | opendevreview | Ghanshyam proposed openstack/nova master: Fix doc comment on manager role change https://review.opendev.org/c/openstack/nova/+/956189 | |
| 21:28:20 | opendevreview | Merged openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/953063 | |
| 22:02:07 | opendevreview | Merged openstack/nova master: Don't reset port dns_name when shelving instances https://review.opendev.org/c/openstack/nova/+/919961 | |
| #openstack-nova - 2025-07-31 | |||
| 07:14:40 | opendevreview | Andrew Bonney proposed openstack/nova stable/2025.1: Don't reset port dns_name when shelving instances https://review.opendev.org/c/openstack/nova/+/956216 | |
| 07:22:01 | sp-bmilanov | sean-k-mooney hi, maybe offtopic, but I just saw the "Assisted-By: Claude-Code" trailer at https://review.opendev.org/c/openstack/nova-specs/+/953940 ... I don't know if you've discussed it somewhere, but can you share a bit about your experience writing the spec and how you used Claude-Code? What were you most pleased with / most frustrated with? Do you have any advice if anyone else wants to use an LLM for writing spe | |
| 07:22:01 | sp-bmilanov | nts? Not looking to use something at the moment personally, but I am curious about the experience of others (even more so in the context of OpenStack) | |
| 10:24:37 | opendevreview | Ian Watson proposed openstack/nova master: Add migration conditional for resource tracker https://review.opendev.org/c/openstack/nova/+/955690 | |
| 11:03:36 | opendevreview | Stephen Finucane proposed openstack/nova master: Fix doc comment on manager role change https://review.opendev.org/c/openstack/nova/+/956189 | |
| 12:10:35 | gboutry | Hello Nova, I'm looking for arm64 live migration capability, and it looks like it's currently not support: https://docs.openstack.org/nova/latest/user/support-matrix.html#operation_live_migrate | |
| 12:10:57 | gboutry | Can someone confirm this docs is up to date and this is unsupported? | |
| 12:18:45 | opendevreview | Andrew Bonney proposed openstack/nova stable/2025.1: Don't reset port dns_name when shelving instances https://review.opendev.org/c/openstack/nova/+/956216 | |
| 12:22:44 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | |
| 12:22:44 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | |
| 12:22:45 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | |
| 12:23:12 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (1/6) https://review.opendev.org/c/openstack/nova/+/956234 | |
| 13:09:31 | stephenfin | incoming | |
| 13:09:40 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with images APIs https://review.opendev.org/c/openstack/nova/+/956102 | |
| 13:09:40 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Separate volume, snapshot and volume attachments https://review.opendev.org/c/openstack/nova/+/952347 | |
| 13:09:41 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use valid UUIDs for cinder resources https://review.opendev.org/c/openstack/nova/+/952935 | |
| 13:09:41 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only apply "soft" additionalProperties validation to requests https://review.opendev.org/c/openstack/nova/+/952936 | |
| 13:09:42 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348 | |
| 13:09:42 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | |
| 13:09:43 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | |