| opendevreview | Merged openstack/nova stable/wallaby: Avoid unbound instance_uuid var during delete https://review.opendev.org/c/openstack/nova/+/828839 | 02:20 |
|---|---|---|
| opendevreview | Merged openstack/nova stable/xena: Retry in CellDatabases fixture when global DB state changes https://review.opendev.org/c/openstack/nova/+/840949 | 02:21 |
| opendevreview | Merged openstack/nova stable/wallaby: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833435 | 02:21 |
| opendevreview | Merged openstack/nova stable/train: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/853546 | 02:55 |
| bauzas | good morning Nova | 06:34 |
| gibi | o/ | 07:48 |
| opendevreview | Slawek Kaplonski proposed openstack/nova master: WIP Don't provide MTU value in metadata service if DHCP is enabled https://review.opendev.org/c/openstack/nova/+/855664 | 08:46 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Bump compute version and check shares support https://review.opendev.org/c/openstack/nova/+/850499 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.power_on_error notification https://review.opendev.org/c/openstack/nova/+/852084 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.power_off_error notification https://review.opendev.org/c/openstack/nova/+/852278 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/852085 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | 08:47 |
| 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 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/854824 | 08:47 |
| opendevreview | ribaudr proposed openstack/nova master: Change microversion to 2.94 https://review.opendev.org/c/openstack/nova/+/852088 | 08:47 |
| stephenfin | bauzas: sean-k-mooney: Can one of you look at this? https://review.opendev.org/c/openstack/placement/+/855862 | 09:47 |
| bauzas | stephenfin: oh sure | 09:47 |
| sean-k-mooney | stephenfin: i tought gibi had a patch for this | 09:49 |
| stephenfin | that is gibi's patch :) | 09:50 |
| sean-k-mooney | oh that is gibis patch | 09:50 |
| sean-k-mooney | yep bauzas has already hit is so its gran but ya i talked to gibi about that approch so i was already happy with that direction | 09:50 |
| bauzas | I'm not super happy with the cryptic use of SQLA but I'm ok with the patch | 09:51 |
| sean-k-mooney | stephenfin: for placement at least this was jsut an warnign i dont thinik anything was broken | 09:51 |
| sean-k-mooney | cryptic? | 09:52 |
| sean-k-mooney | oh thet fact we sometime have the rows and somethimes have named tuples (dicts before) | 09:52 |
| sean-k-mooney | we shoudl fix mixing the type in the cache but that a spereate latent bug | 09:53 |
| bauzas | sean-k-mooney: I said cryptic because you really need to understand SQLA internals | 09:55 |
| bauzas | this is no longer really an ORM | 09:55 |
| sean-k-mooney | you dont really | 09:55 |
| sean-k-mooney | they are issuing deprecation warnings | 09:55 |
| sean-k-mooney | that tell you exactly what to fix | 09:55 |
| sean-k-mooney | because they are trying to do a 2.0 release | 09:56 |
| sean-k-mooney | so the are getting ready to drop legacy interfaces | 09:56 |
| bauzas | well, I don't want to discuss that much about it, but I understand your point | 09:56 |
| opendevreview | Merged openstack/placement master: Make us compatible with oslo.db 12.1.0 https://review.opendev.org/c/openstack/placement/+/855862 | 11:15 |
| artom | Do we have a antelope PTG etherpad already? | 13:50 |
| * artom has a thing | 13:50 | |
| gibi | yes | 13:50 |
| gibi | https://etherpad.opendev.org/p/nova-antelope-ptg | 13:50 |
| JayF | Heya, only one more patch left in the Ironic driver fixes stuff (I still have the CI one that's getting backported... but when these land that becomes less important). https://review.opendev.org/c/openstack/nova/+/821352 this puts the last fix in stable/train | 18:10 |
| elodilles | JayF: nova-tox-validate-backport job failed -> can you re-apply the cherry pick from the stable/ussuri patch? o:) | 18:14 |
| JayF | oh, whoops, I didn't realize that was non-voting, I was just checking ironic job and voting jobs | 18:14 |
| JayF | I'll absolutely fix, ty | 18:14 |
| opendevreview | Jay Faulkner proposed openstack/nova stable/train: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821352 | 18:15 |
| JayF | I'll be out of your hair soon ;) this is the last ironic driver fix in Nova in really a long time, until the design issue currently at the spec stage gets fixed | 18:15 |
| elodilles | JayF: no problem & thanks for the update ;) +2'd | 18:21 |
| melwitt | we need sean-k-mooney to come back to ^ as well, I haven't been reviewing it bc I was the one who uploaded it (the backport) originally | 18:26 |
| *** __ministry is now known as Guest2212 | 19:01 | |
| fungi | finest nova folk... further reviews of https://review.opendev.org/850003 would be much appreciated, in order to avoid releasing zed with a known security bug and creating more backport work | 20:02 |
| melwitt | I can take a look at it later today, thanks for the link | 20:06 |
| *** dasm is now known as dasm|off | 21:11 | |
| sean-k-mooney | fungi: melwitt +wd | 21:54 |
| sean-k-mooney | also reviewed the ironic one | 21:55 |
| sean-k-mooney | fungi: we are going to bring up that issue with neutron in the PTG too hopefully we can block vnic_type chages for bound port going forward in there api but we will start the backport of that once RC1 is out | 21:56 |
| JayF | thanks :D | 21:57 |
| sean-k-mooney | no worries :) i needed to do one thing i had left before i finsihed for the night got to drop again soon but hopefully that will land soon | 21:59 |
| opendevreview | Merged openstack/nova stable/train: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821352 | 22:17 |
| fungi | sean-k-mooney: thanks for the update! | 22:22 |
| sean-k-mooney | fungi: unfortunetly that failed https://review.opendev.org/c/openstack/nova/+/849985 i think its conflicted with the pci serise which merged since it was writen | 22:43 |
| fungi | oh, oof | 22:44 |
| sean-k-mooney | its proably not that hard to reslove so ill see if we can still get this in zed | 22:44 |
| fungi | there's still time yes. and then we'll want backports of course, but one step at a time | 22:44 |
| fungi | at least it's already public, so the only real looming deadline is not releasing zed with it if we can help that | 22:45 |
| sean-k-mooney | yep | 22:45 |
| sean-k-mooney | ill check with gibi when they are onlien and pull this locally in the morning and see whast happening | 22:45 |
| sean-k-mooney | actuly its failiing in a test that its not modifying | 22:48 |
| sean-k-mooney | fungi: so those tests pass locally for me | 22:54 |
| sean-k-mooney | im going to recheck it and see but we might have some uninited shareing i dont think this is related to this patch | 22:55 |
| fungi | interesting. i'll try to keep an eye on it, just wanted to make sure it was on someone's radar | 22:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!