| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-12-21 | |||
| 10:32:42 | bauzas | Uggla: done https://review.opendev.org/c/openstack/nova/+/868088 | |
| 10:48:52 | Uggla | bauzas, thx I'm gonna have a look | |
| 10:49:13 | bauzas | Uggla: if you need help, I have a few hours left before end of my working year :) | |
| 10:49:44 | bauzas | Uggla: IIRC, we maybe also have this pattern in the unittests | |
| 10:58:50 | opendevreview | Stephen Finucane proposed openstack/nova master: DNM: Identify why internal_info' isn't being set. https://review.opendev.org/c/openstack/nova/+/868306 | |
| 14:16:05 | opendevreview | ribaudr proposed openstack/nova master: Adds a repoducer for post_live_migration_at_destination failures https://review.opendev.org/c/openstack/nova/+/868088 | |
| 14:16:06 | opendevreview | ribaudr proposed openstack/nova master: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/868089 | |
| 14:24:00 | Uggla | bauzas, if you can have a look ^ before leaving. | |
| 14:26:13 | frickler | lp broke the 2M mark? cool | |
| 14:38:52 | opendevreview | Jean-Sébastien Bevilacqua proposed openstack/nova master: Add Lustre support to nova https://review.opendev.org/c/openstack/nova/+/853786 | |
| 15:03:48 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/c/openstack/nova/+/670213 | |
| 15:03:49 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for volume_target.* https://review.opendev.org/c/openstack/nova/+/867790 | |
| 15:03:49 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_provision_state https://review.opendev.org/c/openstack/nova/+/867791 | |
| 15:03:50 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_power_state https://review.opendev.org/c/openstack/nova/+/867792 | |
| 15:03:50 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.vif* https://review.opendev.org/c/openstack/nova/+/867793 | |
| 15:03:51 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining network operations https://review.opendev.org/c/openstack/nova/+/867794 | |
| 15:03:51 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining ironic driver calls https://review.opendev.org/c/openstack/nova/+/867795 | |
| 15:03:52 | opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for deprecated baremetal proxy API https://review.opendev.org/c/openstack/nova/+/867796 | |
| 15:03:52 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove Ironic client wrapper https://review.opendev.org/c/openstack/nova/+/867797 | |
| 15:03:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Remove unnecessary aliasing https://review.opendev.org/c/openstack/nova/+/867798 | |
| 15:03:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Use real SDK objects in tests https://review.opendev.org/c/openstack/nova/+/867799 | |
| 15:07:27 | stephenfin | gibi: Not sure you're still around but if you are, did I get to everything PCI-in-placement related yesterday? | |
| 15:10:50 | gibi | stephenfin: I'm off officially, but here right now | |
| 15:10:58 | gibi | I saw your feedback on the PCI series | |
| 15:11:05 | gibi | thanks for the review | |
| 15:11:42 | gibi | and yes you covered the whole thing | |
| 15:11:54 | gibi | about the refactor of the filters | |
| 15:12:05 | gibi | I think I can pull that patch out of the series to the top | |
| 15:12:17 | gibi | and we can get back to that next year with Sean | |
| 15:12:57 | stephenfin | if it's not too much work, that would be helpful. Doesn't need to happen until next year since Sean won't review til then either | |
| 15:13:04 | gibi | yeah | |
| 15:13:23 | gibi | I will see how much I can progress on this in the next couple of hours | |
| 15:14:05 | gibi | we could merge couple of patches that has Sean's +2 already after I pulled out the refactor | |
| 15:14:10 | gibi | I think that worth the effort now | |
| 15:14:28 | gibi | the rest needs Sean's review next year yes | |
| 15:14:50 | stephenfin | sounds good | |
| 15:40:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Store allocated RP in InstancePCIRequest https://review.opendev.org/c/openstack/nova/+/854121 | |
| 15:40:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Func test for PCI in placement scheduling https://review.opendev.org/c/openstack/nova/+/854122 | |
| 15:40:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support cold migrate and resize with PCI tracking in placement https://review.opendev.org/c/openstack/nova/+/854247 | |
| 15:40:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support evacuate with PCI in placement https://review.opendev.org/c/openstack/nova/+/854615 | |
| 15:40:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support unshelve with PCI in placement https://review.opendev.org/c/openstack/nova/+/854616 | |
| 15:40:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same host resize with PCI in placement https://review.opendev.org/c/openstack/nova/+/854441 | |
| 15:40:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Test reschedule with PCI in placement https://review.opendev.org/c/openstack/nova/+/854626 | |
| 15:40:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Test multi create with PCI in placement https://review.opendev.org/c/openstack/nova/+/854663 | |
| 15:40:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow enabling PCI scheduling in Placement https://review.opendev.org/c/openstack/nova/+/854924 | |
| 15:40:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Follow up for the PCI in placement series https://review.opendev.org/c/openstack/nova/+/855654 | |
| 15:40:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename _to_device_spec_conf to _to_list_of_json_str https://review.opendev.org/c/openstack/nova/+/855648 | |
| 15:40:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce PCI pool filtering bug https://review.opendev.org/c/openstack/nova/+/855649 | |
| 15:40:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Strictly follow placement allocation during PCI claim https://review.opendev.org/c/openstack/nova/+/855650 | |
| 15:40:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP for the scheduler part of PCI in placement https://review.opendev.org/c/openstack/nova/+/862876 | |
| 15:40:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Split ignored_tags in stats.py https://review.opendev.org/c/openstack/nova/+/867978 | |
| 15:40:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Factor out base class for candidate aware filters https://review.opendev.org/c/openstack/nova/+/854929 | |
| 15:42:29 | gibi | stephenfin: moved the patch and FUPped the your comments in https://review.opendev.org/c/openstack/nova/+/854121 | |
| 15:43:00 | gibi | I think the bottom two patches of the series now has +2 from you and Sean and I only rebased those so I will plug the +A there | |
| 15:51:46 | gibi | the rest needs to wait for next year | |
| 15:52:07 | opendevreview | Balazs Gibizer proposed openstack/placement master: DNM force latest tox https://review.opendev.org/c/openstack/placement/+/868354 | |
| 15:55:43 | opendevreview | Balazs Gibizer proposed openstack/os-vif master: DNM force latest tox https://review.opendev.org/c/openstack/os-vif/+/868357 | |
| 15:59:05 | opendevreview | Balazs Gibizer proposed openstack/python-novaclient master: DNM force latest tox https://review.opendev.org/c/openstack/python-novaclient/+/868358 | |
| 16:02:50 | opendevreview | Balazs Gibizer proposed openstack/osc-placement master: DNM force latest tox https://review.opendev.org/c/openstack/osc-placement/+/868359 | |
| 16:35:38 | opendevreview | Ghanshyam proposed openstack/nova stable/yoga: DNM force latest tox https://review.opendev.org/c/openstack/nova/+/868283 | |
| 16:46:16 | opendevreview | ribaudr proposed openstack/nova master: Adds a repoducer for post_live_migration_at_destination failures https://review.opendev.org/c/openstack/nova/+/868088 | |
| 16:46:16 | opendevreview | ribaudr proposed openstack/nova master: Fix bug 2000069 https://review.opendev.org/c/openstack/nova/+/868089 | |
| 16:52:26 | opendevreview | Merged openstack/nova master: [doc] Fix BFV Rescue trait name in microversion https://review.opendev.org/c/openstack/nova/+/852543 | |
| 18:16:31 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/862967 | |
| 18:16:32 | opendevreview | Artom Lifshitz proposed openstack/nova master: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/862964 | |
| 19:20:11 | opendevreview | Merged openstack/nova master: Store allocated RP in InstancePCIRequest https://review.opendev.org/c/openstack/nova/+/854121 | |
| 19:20:19 | opendevreview | Merged openstack/nova master: Func test for PCI in placement scheduling https://review.opendev.org/c/openstack/nova/+/854122 | |
| #openstack-nova - 2022-12-22 | |||
| 07:57:32 | opendevreview | Justas Poderys proposed openstack/nova-specs master: VirtIO PackedRing Configuration support https://review.opendev.org/c/openstack/nova-specs/+/868377 | |
| 08:29:46 | pslestang | Hello! | |
| 08:30:09 | pslestang | Would you have some times to review this patchset? https://review.opendev.org/c/openstack/nova/+/867832 | |
| 08:30:43 | pslestang | I already have one from sean-k-mooney[m] but I need some more opinion | |
| 12:08:48 | opendevreview | Aaron S proposed openstack/nova master: Add further workaround features for qemu_monitor_announce_self https://review.opendev.org/c/openstack/nova/+/867324 | |
| 14:51:09 | opendevreview | Balazs Gibizer proposed openstack/placement master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/placement/+/868418 | |
| 14:59:14 | opendevreview | yatin proposed openstack/nova master: [DNM] Test lower tb cache https://review.opendev.org/c/openstack/nova/+/868419 | |
| 15:00:50 | opendevreview | Balazs Gibizer proposed openstack/placement master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/placement/+/868418 | |
| 15:09:15 | opendevreview | Balazs Gibizer proposed openstack/os-vif master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/os-vif/+/868420 | |
| 15:12:00 | opendevreview | Balazs Gibizer proposed openstack/python-novaclient master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/python-novaclient/+/868421 | |
| 15:15:36 | opendevreview | Balazs Gibizer proposed openstack/osc-placement master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/osc-placement/+/868422 | |
| 15:20:46 | gibi | bauzas, stephenfin, sean-k-mooney[m]: here is fixes for the tox4 switch for compute related projects: https://review.opendev.org/q/topic:tox4 nova will be green once https://review.opendev.org/c/openstack/openstacksdk/+/867827 lands | |
| 15:22:55 | sean-k-mooney[m] | thanks im expicting that to be relitivly minor for nova et al | |
| 15:27:00 | sean-k-mooney[m] | gibi the main thing im not sure about is why skipdist=true is being moved out of the default section and only added in bindep | |
| 15:27:22 | gibi | sean-k-mooney[m]: skipdist and usedevelop does not work together any more afaik | |
| 15:27:45 | gibi | https://github.com/tox-dev/tox/issues/2730 | |
| 15:28:43 | gibi | sean-k-mooney[m]: btw if you are around then the os-vif patch still failing in the functional target but I'm not sure why | |
| 15:29:45 | sean-k-mooney[m] | there is a patch from gman that i commeted on | |
| 15:29:48 | sean-k-mooney[m] | i have not had a chance to look into it fully but its on my todo list for my first week back | |
| 15:30:09 | sean-k-mooney[m] | https://review.opendev.org/c/openstack/os-vif/+/861468/4#message-68b5695d34afe144c7789f48fc9271fd8c19e303 | |
| 15:30:51 | sean-k-mooney[m] | gibi its unrelated to your change | |
| 15:31:00 | gibi | OK, thanks | |
| 15:31:12 | sean-k-mooney[m] | also https://review.opendev.org/c/openstack/nova/+/868419 is interesting | |
| 15:31:33 | gibi | sean-k-mooney[m]: also good news, stephenfin reviewed the PCI series all the way up. We landed couple of patches from the bottom where you had +2 too | |
| 15:32:01 | sean-k-mooney[m] | excelent that is also on my todo list for when i get back | |
| 15:43:28 | sean-k-mooney[m] | anyway i skimmed over the seriese and it looks like stephen is happy with most of the rest of the series. | |
| 15:44:32 | sean-k-mooney[m] | so hopefully we can get this merged early next year thanks for prepareing the series the way you did it helps | |
| 15:45:44 | sean-k-mooney[m] | im going to go back to my audio book but you can ping me via this matrix account if anything comes up that needs my attention. o/ | |
| 15:47:14 | gibi | enjoy o/ | |
| 15:49:46 | opendevreview | Balazs Gibizer proposed openstack/placement master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/placement/+/868418 | |
| 15:51:03 | opendevreview | Balazs Gibizer proposed openstack/os-vif master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/os-vif/+/868420 | |
| 15:51:47 | opendevreview | Balazs Gibizer proposed openstack/python-novaclient master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/python-novaclient/+/868421 | |
| 15:53:04 | opendevreview | Balazs Gibizer proposed openstack/python-novaclient master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/python-novaclient/+/868421 | |