| *** mhen_ is now known as mhen | 02:46 | |
| opendevreview | Merged openstack/placement master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/placement/+/970068 | 02:57 |
|---|---|---|
| opendevreview | Merged openstack/python-novaclient master: Install pcre packages only in doc jobs https://review.opendev.org/c/openstack/python-novaclient/+/973846 | 03:05 |
| opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | 03:28 |
| opendevreview | Ghanshyam proposed openstack/nova master: PoC: Graceful shutodwn of nova services https://review.opendev.org/c/openstack/nova/+/967261 | 05:29 |
| opendevreview | Michel Nederlof proposed openstack/nova master: Add RBD XML update functionality for migration process https://review.opendev.org/c/openstack/nova/+/974032 | 09:10 |
| opendevreview | Johannes Kulik proposed openstack/nova master: Attaching a volume returns HTTP 202 https://review.opendev.org/c/openstack/nova/+/971068 | 09:37 |
| opendevreview | Michel Nederlof proposed openstack/nova master: Add RBD XML update functionality for migration process https://review.opendev.org/c/openstack/nova/+/974032 | 09:58 |
| opendevreview | Merged openstack/python-novaclient master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/python-novaclient/+/970066 | 10:24 |
| gibi | stephenfin: thanks I reviewed the series and left some comments but overall looks good | 10:34 |
| stephenfin | gibi: ty! | 10:36 |
| stephenfin | gibi: I responded to one of the comments on https://review.opendev.org/c/openstack/nova/+/973977. Would appreciate your thoughts before I push a new revision | 10:49 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Replace objects from typing with literals https://review.opendev.org/c/openstack/nova/+/973977 | 10:56 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Bump hacking https://review.opendev.org/c/openstack/nova/+/973978 | 10:56 |
| opendevreview | Johannes Kulik proposed openstack/nova master: Attaching a volume returns HTTP 202 https://review.opendev.org/c/openstack/nova/+/971068 | 11:16 |
| gibi | stephenfin: thanks for the reply I agree with your final assesment that we should not change the second caller as the two variable can be independently None | 12:13 |
| gibi | I will set my votes accordingy :) | 12:13 |
| gibi | OK all good from my perspective now | 12:15 |
| *** ralonsoh_ is now known as ralonsoh | 12:26 | |
| opendevreview | Esra Ozkan proposed openstack/nova master: Fix Concurrent VM Live Migrate - Volume Backup Error https://review.opendev.org/c/openstack/nova/+/973750 | 13:07 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota class sets API https://review.opendev.org/c/openstack/nova/+/956140 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota sets API https://review.opendev.org/c/openstack/nova/+/956141 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for migrations API https://review.opendev.org/c/openstack/nova/+/956142 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server migrations API https://review.opendev.org/c/openstack/nova/+/956143 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server tags API https://review.opendev.org/c/openstack/nova/+/956144 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server metadata APIs https://review.opendev.org/c/openstack/nova/+/956145 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | 14:58 |
| 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 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (5/6) https://review.opendev.org/c/openstack/nova/+/956239 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (6/6) https://review.opendev.org/c/openstack/nova/+/956240 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server shares APIs https://review.opendev.org/c/openstack/nova/+/956266 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | 14:58 |
| opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Preparatory work for use of TypedDict in schema validation https://review.opendev.org/c/openstack/nova/+/974075 | 14:58 |
| stephenfin | gmaan: I think I've responsed to all your questions and fixed the things I needed to fix on the above ^ | 14:59 |
| gibi | Im delayed from the eventlet sync | 15:31 |
| bauzas | gibi: I can't join but I'll look at the doc | 15:42 |
| sean-k-mooney | gibi: stephenfin: can you make some time to review https://review.opendev.org/c/openstack/nova/+/973149 and https://review.opendev.org/c/openstack/os-vif/+/971231 the initall neturon patch has merged the rest will depend on thse mergign and a release of os-vif all the patches are tested end ot end by https://review.opendev.org/c/openstack/neutron/+/972820 | 15:42 |
| gmaan | stephenfin: ack | 15:54 |
| opendevreview | Merged openstack/nova master: Run mypy from tox https://review.opendev.org/c/openstack/nova/+/973975 | 15:58 |
| opendevreview | Merged openstack/nova master: typing: "Import" objects https://review.opendev.org/c/openstack/nova/+/973976 | 16:02 |
| opendevreview | sean mooney proposed openstack/os-vif master: Add TAP device pre-creation support for OVS/OVN https://review.opendev.org/c/openstack/os-vif/+/971231 | 16:08 |
| sean-k-mooney | the link to the nova status etherpad is missing in the topic | 16:12 |
| sean-k-mooney | was it remvoed intentionally? | 16:12 |
| sean-k-mooney | it should be https://etherpad.opendev.org/p/nova-2026.1-status | 16:13 |
| sean-k-mooney | instead of pointing at the ptg link | 16:13 |
| opendevreview | Merged openstack/nova master: typing: Replace objects from typing with literals https://review.opendev.org/c/openstack/nova/+/973977 | 16:26 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for quota sets API https://review.opendev.org/c/openstack/nova/+/956141 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for migrations API https://review.opendev.org/c/openstack/nova/+/956142 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server migrations API https://review.opendev.org/c/openstack/nova/+/956143 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server tags API https://review.opendev.org/c/openstack/nova/+/956144 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server metadata APIs https://review.opendev.org/c/openstack/nova/+/956145 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (1/3) https://review.opendev.org/c/openstack/nova/+/956231 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (2/3) https://review.opendev.org/c/openstack/nova/+/956232 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify servers views (3/3) https://review.opendev.org/c/openstack/nova/+/956233 | 17:03 |
| 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 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (2/6) https://review.opendev.org/c/openstack/nova/+/956236 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (3/6) https://review.opendev.org/c/openstack/nova/+/956237 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (4/6) https://review.opendev.org/c/openstack/nova/+/956238 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (5/6) https://review.opendev.org/c/openstack/nova/+/956239 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for servers APIs (6/6) https://review.opendev.org/c/openstack/nova/+/956240 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server shares APIs https://review.opendev.org/c/openstack/nova/+/956266 | 17:03 |
| opendevreview | Stephen Finucane proposed openstack/nova master: tests: Invert validation check https://review.opendev.org/c/openstack/nova/+/956241 | 17:03 |
| stephenfin | gmaan: Sorry, fixed that missing delete_query dict /o\ | 17:03 |
| opendevreview | Merged openstack/nova master: Bump hacking https://review.opendev.org/c/openstack/nova/+/973978 | 17:40 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.cmd, nova.conf, nova.console https://review.opendev.org/c/openstack/nova/+/705657 | 17:53 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to remaining top-level modules https://review.opendev.org/c/openstack/nova/+/705658 | 17:53 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Add hints to nova.virt, nova.virt.libvirt https://review.opendev.org/c/openstack/nova/+/974220 | 17:53 |
| opendevreview | Stephen Finucane proposed openstack/nova master: typing: Correct import issues https://review.opendev.org/c/openstack/nova/+/974221 | 17:53 |
| opendevreview | Stephen Finucane proposed openstack/nova master: mypy: Enable incremental checks https://review.opendev.org/c/openstack/nova/+/974222 | 17:53 |
| gmaan | stephenfin: thanks, network field still seems odd to me. I think we can keep that as commented field and client can handle it for older version if that needs to be supported. | 18:37 |
| sean-k-mooney | gmaan: i was kind of indiffernt on that which is why i didnt comment either way in the review so im happy to follow your lead on taht | 18:52 |
| sean-k-mooney | we have kept schemas for other removed or deprecated things so im fine with keeping it or not | 18:52 |
| sean-k-mooney | gmaan: what i woudl suggest is if we remove it do it in a followup in the end of the series | 18:53 |
| sean-k-mooney | well unless stephen is rebasign them again anyway for somethign else | 18:53 |
| gmaan | sean-k-mooney: yeah, sure I am ok in followup. did not want to rebase all things | 18:54 |
| gmaan | I think I commented about ok as followup but maybe in other change | 18:54 |
| opendevreview | Merged openstack/nova master: blueprint: iothreads-for-instances https://review.opendev.org/c/openstack/nova/+/939254 | 21:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!