| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-06 | |||
| 16:56:18 | bauzas | the infection increases. | |
| 16:56:23 | elodilles | these were the main points :X | |
| 16:57:33 | bauzas | well, I'd say stable/train can be a concern to me | |
| 16:57:40 | bauzas | for others, well, ExtendedMaintenance | |
| 16:58:23 | bauzas | so, the ones who care should just raise a hand | |
| 16:58:39 | bauzas | but we're approaching the top of the hour | |
| 16:59:03 | bauzas | #topic Open discussion | |
| 16:59:09 | bauzas | anything anyone for last min ? | |
| 16:59:37 | bauzas | guess not | |
| 16:59:42 | bauzas | thanks all | |
| 16:59:44 | bauzas | #endmeeting | |
| 16:59:44 | opendevmeet | Meeting ended Tue Sep 6 16:59:44 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:59:44 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-06-16.00.html | |
| 16:59:44 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-06-16.00.txt | |
| 16:59:44 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-09-06-16.00.log.html | |
| 16:59:52 | elodilles | thanks bauzas o/ | |
| 17:00:34 | elodilles | and you saved 16 seconds from the meeting :-o amazing! | |
| 17:01:36 | gibi | and it is gone :D | |
| 17:01:41 | bauzas | we're asked to reduce our electricity | |
| 17:01:51 | bauzas | I just save a few kWh | |
| 17:07:11 | elodilles | :] | |
| 17:07:41 | gibi | fyi I added a topic generic to the PTG etherpad about catching design issues eariler in the cycle, please add your ideas there | |
| 17:07:52 | gibi | * generic topic | |
| 17:26:27 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Move specs to implemented https://review.opendev.org/c/openstack/nova-specs/+/856173 | |
| 18:25:11 | opendevreview | Merged openstack/nova master: Add missing descriptions in HACKING.rst https://review.opendev.org/c/openstack/nova/+/853054 | |
| 18:33:41 | opendevreview | Merged openstack/python-novaclient master: MV 2.93 - Add support to rebuild boot volume https://review.opendev.org/c/openstack/python-novaclient/+/827163 | |
| 21:59:32 | JayF | These two Ironic driver changes are ready for review into stable/train -- Ironically, they have +2s from two different folks https://review.opendev.org/c/openstack/nova/+/853546 https://review.opendev.org/c/openstack/nova/+/821352 | |
| 22:13:45 | opendevreview | Takashi Natsume proposed openstack/nova master: Add a hacking rule for the setDaemon method https://review.opendev.org/c/openstack/nova/+/854653 | |
| #openstack-nova - 2022-09-07 | |||
| 02:20:52 | opendevreview | Merged openstack/nova stable/wallaby: Avoid unbound instance_uuid var during delete https://review.opendev.org/c/openstack/nova/+/828839 | |
| 02:21:00 | 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:05 | opendevreview | Merged openstack/nova stable/wallaby: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833435 | |
| 02:55:53 | opendevreview | Merged openstack/nova stable/train: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/853546 | |
| 06:34:50 | bauzas | good morning Nova | |
| 07:48:54 | gibi | o/ | |
| 08:46:43 | 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:47:11 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 08:47:11 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 08:47:12 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 08:47:12 | 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:13 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 08:47:13 | opendevreview | ribaudr proposed openstack/nova master: Bump compute version and check shares support https://review.opendev.org/c/openstack/nova/+/850499 | |
| 08:47:14 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 08:47:14 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 08:47:15 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 08:47:16 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 08:47:16 | opendevreview | ribaudr proposed openstack/nova master: Add instance.power_on_error notification https://review.opendev.org/c/openstack/nova/+/852084 | |
| 08:47:18 | opendevreview | ribaudr proposed openstack/nova master: Add instance.power_off_error notification https://review.opendev.org/c/openstack/nova/+/852278 | |
| 08:47:18 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/852085 | |
| 08:47:20 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 08:47:20 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 08:47:22 | 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:22 | 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:24 | opendevreview | ribaudr proposed openstack/nova master: Change microversion to 2.94 https://review.opendev.org/c/openstack/nova/+/852088 | |
| 09:47:34 | stephenfin | bauzas: sean-k-mooney: Can one of you look at this? https://review.opendev.org/c/openstack/placement/+/855862 | |
| 09:47:51 | bauzas | stephenfin: oh sure | |
| 09:49:58 | sean-k-mooney | stephenfin: i tought gibi had a patch for this | |
| 09:50:10 | stephenfin | that is gibi's patch :) | |
| 09:50:11 | sean-k-mooney | oh that is gibis patch | |
| 09:50:47 | 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:51:51 | bauzas | I'm not super happy with the cryptic use of SQLA but I'm ok with the patch | |
| 09:51:54 | sean-k-mooney | stephenfin: for placement at least this was jsut an warnign i dont thinik anything was broken | |
| 09:52:10 | sean-k-mooney | cryptic? | |
| 09:52:36 | sean-k-mooney | oh thet fact we sometime have the rows and somethimes have named tuples (dicts before) | |
| 09:53:01 | sean-k-mooney | we shoudl fix mixing the type in the cache but that a spereate latent bug | |
| 09:55:06 | bauzas | sean-k-mooney: I said cryptic because you really need to understand SQLA internals | |
| 09:55:24 | bauzas | this is no longer really an ORM | |
| 09:55:43 | sean-k-mooney | you dont really | |
| 09:55:51 | sean-k-mooney | they are issuing deprecation warnings | |
| 09:55:57 | sean-k-mooney | that tell you exactly what to fix | |
| 09:56:05 | sean-k-mooney | because they are trying to do a 2.0 release | |
| 09:56:23 | sean-k-mooney | so the are getting ready to drop legacy interfaces | |
| 09:56:54 | bauzas | well, I don't want to discuss that much about it, but I understand your point | |
| 11:15:06 | opendevreview | Merged openstack/placement master: Make us compatible with oslo.db 12.1.0 https://review.opendev.org/c/openstack/placement/+/855862 | |
| 13:50:31 | artom | Do we have a antelope PTG etherpad already? | |
| 13:50:50 | gibi | yes | |
| 13:50:58 | gibi | https://etherpad.opendev.org/p/nova-antelope-ptg | |
| 18:10:42 | 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:14:18 | elodilles | JayF: nova-tox-validate-backport job failed -> can you re-apply the cherry pick from the stable/ussuri patch? o:) | |
| 18:14:53 | JayF | oh, whoops, I didn't realize that was non-voting, I was just checking ironic job and voting jobs | |
| 18:14:57 | JayF | I'll absolutely fix, ty | |
| 18:15:15 | 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:58 | 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:21:06 | elodilles | JayF: no problem & thanks for the update ;) +2'd | |
| 18:26:47 | 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 | |
| 20:02:07 | 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:06:22 | melwitt | I can take a look at it later today, thanks for the link | |
| 21:54:49 | sean-k-mooney | fungi: melwitt +wd | |
| 21:55:01 | sean-k-mooney | also reviewed the ironic one | |
| 21:56:20 | 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:57:10 | JayF | thanks :D | |
| 21:59:16 | 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 | |
| 22:17:14 | opendevreview | Merged openstack/nova stable/train: Ignore plug_vifs on the ironic driver https://review.opendev.org/c/openstack/nova/+/821352 | |
| 22:22:44 | fungi | sean-k-mooney: thanks for the update! | |
| 22:43:35 | 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:44:21 | fungi | oh, oof | |
| 22:44:24 | sean-k-mooney | its proably not that hard to reslove so ill see if we can still get this in zed | |
| 22:44:44 | fungi | there's still time yes. and then we'll want backports of course, but one step at a time | |
| 22:45:06 | 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:25 | sean-k-mooney | yep | |
| 22:45:48 | sean-k-mooney | ill check with gibi when they are onlien and pull this locally in the morning and see whast happening | |