| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-12-21 | |||
| 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 | |
| 16:13:45 | opendevreview | Aaron S proposed openstack/nova master: Add further workaround features for qemu_monitor_announce_self https://review.opendev.org/c/openstack/nova/+/867324 | |
| 16:19:17 | opendevreview | Aaron S proposed openstack/nova master: Add further workaround features for qemu_monitor_announce_self https://review.opendev.org/c/openstack/nova/+/867324 | |
| 18:02:02 | gmann | gibi: pinged sdks core in channel to merge that patch but if that does not happen before ensure-tox unpin tox then we can make sdk job as n-v for now and proceed | |
| 18:02:11 | gmann | let's see | |
| 18:02:33 | gmann | sdks fix needed for many other projects also as they run sdk job including devstack | |
| 18:30:13 | opendevreview | Aaron S proposed openstack/nova master: Add further workaround features for qemu_monitor_announce_self https://review.opendev.org/c/openstack/nova/+/867324 | |
| 18:31:18 | opendevreview | Aaron S proposed openstack/nova master: Add further workaround features for qemu_monitor_announce_self https://review.opendev.org/c/openstack/nova/+/867324 | |
| 18:36:17 | opendevreview | Aaron S proposed openstack/nova master: Add further workaround features for qemu_monitor_announce_self https://review.opendev.org/c/openstack/nova/+/867324 | |
| #openstack-nova - 2022-12-23 | |||
| 08:36:34 | Etua | Hello, I'd like to create and test some new filters and weights for the scheduler. Ideally, I would mimic an existing OpenStack deployment I have access to. I already installed placement component, to which the scheduler would connect and now I'd like to isolate it. I'm not sure how to approach it, as scheduler is part of Nova and not a stand-alone component as placement. | |
| 08:36:39 | Etua | My intention is to initially fill placement's database with information about the infrastructure, set up the scheduler and then query it for consecutive requests (linear, no need for locks if that changes anything) to log the decisions it would make with modified weights/filters and the state of the system in-between requests. | |