| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-17 | |||
| 12:22:07 | sean-k-mooney | i chatted to bauzas about this a bit downstream and we wer eunsure how you and other stabel cores would feel about when it comes to backporting | |
| 12:22:30 | sean-k-mooney | elodilles: is it ok to do both in one patch or woudl you prefer we did not backport the ci change | |
| 12:22:56 | sean-k-mooney | i woudl prefer to backport both and if we are backportign both i would prefer to have it be in one patch | |
| 12:23:45 | sean-k-mooney | my curent plan is to set heal_instance_info_cache_interval=0 in nova-next | |
| 12:36:54 | opendevreview | John Garbutt proposed openstack/nova master: Ironic nodes with instance reserved in placement https://review.opendev.org/c/openstack/nova/+/864773 | |
| 12:39:29 | johnthetubaguy | gibi sean-k-mooney I have added a release note and the workaround config, slightly more intrusive, but not by much I guess. | |
| 12:40:37 | sean-k-mooney | ack most of the way though the spec not much feeback so far beyond what is already there | |
| 12:40:59 | sean-k-mooney | just got to the nova manage command | |
| 12:48:47 | bauzas | sean-k-mooney: elodilles: yup, I just wondered if we were ok for backporting a .zuul file :) | |
| 12:49:57 | sean-k-mooney | im pretty sure i have dont that before | |
| 12:50:11 | sean-k-mooney | backported a job change we defintly did it to fix the train gate | |
| 13:01:19 | elodilles | sean-k-mooney bauzas : as far as i remember, traditionally, doc change backports were not accepted, but i think it is OK to backport them. about the CI I'm a bit hesitant, though. but it depends on the change, i would say | |
| 13:01:25 | gibi | johnthetubaguy: left reply in the pci re-schedule bugfix | |
| 13:01:47 | gibi | johnthetubaguy: that single instance.refresh() feels strange to me | |
| 13:01:58 | sean-k-mooney | elodilles: it littrally is disablelng a perodic task | |
| 13:02:36 | sean-k-mooney | elodilles: we perodicaly heal the network info cache but that in practic should not be required as neutron tells us when something chagnes | |
| 13:02:46 | sean-k-mooney | elodilles: so the ci change is setting one config value to 0 | |
| 13:03:04 | sean-k-mooney | in one job nova-next | |
| 13:03:04 | johnthetubaguy | gibi: agreed, its crazy that fixes so much, drops all the transient changes before a save, roughly. I want more of that claims stuff in the functional test I think | |
| 13:03:25 | elodilles | sean-k-mooney: so it won't be a new CI job, but a config value change in 1(?) job as I understand then | |
| 13:03:46 | sean-k-mooney | yes just one addtionall config override in the nova-next job | |
| 13:03:52 | gibi | johnthetubaguy: if the PciDevice.instance_uuid field is already update to point to this instance then simply resetting the instance with instance.refresh() cannot be enough | |
| 13:03:58 | sean-k-mooney | to set the existing config option to 0 instead of the defautl 60 | |
| 13:04:02 | elodilles | sean-k-mooney: and you state that it won't introduce any instability o:) | |
| 13:04:24 | sean-k-mooney | well thats why we want to have it runing in ci | |
| 13:04:29 | sean-k-mooney | but i dont belive it will | |
| 13:04:53 | elodilles | maby let's see it first in master banch then :) | |
| 13:05:01 | elodilles | * maybe | |
| 13:05:25 | sean-k-mooney | ok so two patches one for docs change to corect the help text adn seperate one for zuul | |
| 13:05:30 | sean-k-mooney | i can do that | |
| 13:05:43 | elodilles | sean-k-mooney: yep, that sounds good | |
| 13:09:45 | sean-k-mooney | cool ill do that then thanks | |
| 13:36:16 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: functional: Change order of two classes https://review.opendev.org/c/openstack/nova/+/864672 | |
| 13:36:17 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: functional: Rework '_delete_server' https://review.opendev.org/c/openstack/nova/+/864721 | |
| 13:36:18 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: functional: Unify '_build_minimal_create_server_request' implementations https://review.opendev.org/c/openstack/nova/+/864713 | |
| 13:36:19 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Extend NeutronFixture to allow live migration with ports https://review.opendev.org/c/openstack/nova/+/864900 | |
| 14:04:49 | opendevreview | Danylo Vodopianov proposed openstack/nova master: Napatech SmartNIC support https://review.opendev.org/c/openstack/nova/+/859577 | |
| 14:09:39 | opendevreview | Danylo Vodopianov proposed openstack/os-vif master: MTU support for DPDK port added https://review.opendev.org/c/openstack/os-vif/+/859574 | |
| 14:11:34 | opendevreview | John Garbutt proposed openstack/nova master: Ironic nodes with instance reserved in placement https://review.opendev.org/c/openstack/nova/+/864773 | |
| 14:16:26 | opendevreview | Merged openstack/nova stable/ussuri: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/862603 | |
| 14:36:52 | opendevreview | Merged openstack/nova stable/ussuri: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/862604 | |
| 14:37:09 | opendevreview | Merged openstack/nova master: DOC update remote console access https://review.opendev.org/c/openstack/nova/+/860687 | |
| 14:47:24 | johnthetubaguy | gibi: I don't think the PCI device objects are getting saved in time, I am trying to prove that in the functional test at the moment | |
| 15:11:05 | opendevreview | Merged openstack/nova master: Replace "db archive" with "db archive_deleted_raws" https://review.opendev.org/c/openstack/nova/+/847963 | |
| 15:15:58 | opendevreview | sean mooney proposed openstack/nova stable/yoga: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/855022 | |
| 15:15:59 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/855023 | |
| 15:16:00 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Remove double mocking https://review.opendev.org/c/openstack/nova/+/855024 | |
| 15:16:01 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/855025 | |
| 15:16:02 | opendevreview | sean mooney proposed openstack/nova stable/yoga: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/855026 | |
| 15:16:04 | opendevreview | sean mooney proposed openstack/nova stable/yoga: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/855035 | |
| 15:16:30 | sean-k-mooney | gibi: finally got around to fixing the commit messge on the double mocking pathc. i also rebased to the tip of stable yoga | |
| 15:16:59 | sean-k-mooney | im goint to try backporting that to xena and wallaby now | |
| 15:20:45 | gibi | sean-k-mooney: I'm +2 on https://review.opendev.org/q/topic:bug%252F1970467 | |
| 15:28:49 | sean-k-mooney | ok including https://review.opendev.org/c/openstack/nova/+/829974 fixes it and that applies cleanly | |
| 16:24:34 | opendevreview | John Garbutt proposed openstack/nova master: Functional test test_boot_reschedule_with_proper_pci_device_count https://review.opendev.org/c/openstack/nova/+/760354 | |
| 16:24:35 | opendevreview | John Garbutt proposed openstack/nova master: Fix PCI passthrough race on reschedule (refresh) https://review.opendev.org/c/openstack/nova/+/710848 | |
| 16:30:32 | opendevreview | John Garbutt proposed openstack/nova master: Fix PCI passthrough race on reschedule (refresh) https://review.opendev.org/c/openstack/nova/+/710848 | |
| 16:34:48 | johnthetubaguy | gibi: I double checked with mgoddard, it turns out he found either patch will fix the issue, we just get to pick which one to merge. The functional tests seem to suggest mark is totally correct, i.e. either patch will fix it. I prefer the instance.refresh() one myself. | |
| 16:58:41 | gibi | johnthetubaguy: I'm wondering when we add the pci dev to the instance.pci_devices list. If we do that during the claim, then claim.abort should be the one to clean that up | |
| 18:51:16 | opendevreview | sean mooney proposed openstack/nova stable/xena: Fix migration with remote-managed ports & add FT https://review.opendev.org/c/openstack/nova/+/864931 | |
| 18:51:17 | opendevreview | sean mooney proposed openstack/nova stable/xena: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/864932 | |
| 18:51:18 | opendevreview | sean mooney proposed openstack/nova stable/xena: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/864933 | |
| 19:08:23 | opendevreview | sean mooney proposed openstack/nova stable/xena: Remove double mocking https://review.opendev.org/c/openstack/nova/+/864934 | |
| 19:08:24 | opendevreview | sean mooney proposed openstack/nova stable/xena: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/864935 | |
| 19:08:25 | opendevreview | sean mooney proposed openstack/nova stable/xena: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/864936 | |
| 19:08:26 | opendevreview | sean mooney proposed openstack/nova stable/xena: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/864937 | |
| 20:30:38 | opendevreview | Ghanshyam proposed openstack/nova master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/nova/+/861111 | |
| 20:31:21 | opendevreview | Ghanshyam proposed openstack/nova master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/nova/+/861111 | |
| 20:39:05 | opendevreview | Ghanshyam proposed openstack/placement master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/placement/+/861471 | |
| 20:41:01 | opendevreview | Ghanshyam proposed openstack/placement master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/placement/+/861471 | |
| 20:47:26 | opendevreview | Ghanshyam proposed openstack/os-traits master: Update python classifier for python 3.10 https://review.opendev.org/c/openstack/os-traits/+/861466 | |
| 20:57:37 | opendevreview | Ghanshyam proposed openstack/python-novaclient master: Update python classifier for python 3.10 https://review.opendev.org/c/openstack/python-novaclient/+/861469 | |
| 21:01:58 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/osc-placement/+/861470 | |
| 21:09:23 | opendevreview | Ghanshyam proposed openstack/os-vif master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/os-vif/+/861468 | |
| 21:48:45 | opendevreview | Ghanshyam proposed openstack/nova master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/nova/+/861111 | |
| #openstack-nova - 2022-11-18 | |||
| 01:50:17 | opendevreview | Ghanshyam proposed openstack/os-vif master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/os-vif/+/861468 | |
| 03:21:07 | opendevreview | Jorhson Deng proposed openstack/nova master: Optimize the small pagesize in numa_fit_instance_to_host https://review.opendev.org/c/openstack/nova/+/864812 | |
| 03:50:43 | gmann | sean-k-mooney: if you have time tomorrow, can you check openstack-tox-functional-ovs-with-sudo job failure on jammy - https://review.opendev.org/c/openstack/os-vif/+/861468 | |
| 10:13:43 | opendevreview | Sylvain Bauza proposed openstack/nova master: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/850673 | |
| 10:13:44 | opendevreview | Sylvain Bauza proposed openstack/nova master: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/838976 | |
| 11:44:17 | opendevreview | sean mooney proposed openstack/nova stable/xena: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/864932 | |
| 11:44:18 | opendevreview | sean mooney proposed openstack/nova stable/xena: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/864933 | |
| 11:44:19 | opendevreview | sean mooney proposed openstack/nova stable/xena: Remove double mocking https://review.opendev.org/c/openstack/nova/+/864934 | |
| 11:44:20 | opendevreview | sean mooney proposed openstack/nova stable/xena: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/864935 | |
| 11:44:21 | opendevreview | sean mooney proposed openstack/nova stable/xena: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/864936 | |
| 11:44:22 | opendevreview | sean mooney proposed openstack/nova stable/xena: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/864937 | |
| 12:12:07 | opendevreview | sean mooney proposed openstack/nova stable/xena: refactor: remove duplicated logic https://review.opendev.org/c/openstack/nova/+/864932 | |
| 12:12:08 | opendevreview | sean mooney proposed openstack/nova stable/xena: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/864933 | |
| 12:12:09 | opendevreview | sean mooney proposed openstack/nova stable/xena: Remove double mocking https://review.opendev.org/c/openstack/nova/+/864934 | |
| 12:12:10 | opendevreview | sean mooney proposed openstack/nova stable/xena: Remove double mocking... again https://review.opendev.org/c/openstack/nova/+/864935 | |
| 12:12:12 | opendevreview | sean mooney proposed openstack/nova stable/xena: Add compute restart capability for libvirt func tests https://review.opendev.org/c/openstack/nova/+/864936 | |
| 12:12:14 | opendevreview | sean mooney proposed openstack/nova stable/xena: enable blocked VDPA move operations https://review.opendev.org/c/openstack/nova/+/864937 | |
| 15:04:15 | opendevreview | Rodolfo Alonso proposed openstack/nova master: Bump minimum version of os-vif to 3.1.0 https://review.opendev.org/c/openstack/nova/+/865031 | |
| 15:08:54 | opendevreview | Sylvain Bauza proposed openstack/nova master: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/838976 | |
| 15:08:54 | opendevreview | Sylvain Bauza proposed openstack/nova master: Reproducer for bug 1951656 https://review.opendev.org/c/openstack/nova/+/850673 | |
| 15:08:55 | opendevreview | Sylvain Bauza proposed openstack/nova master: Deprecate mdev creation and hardfail on reboot when missing. https://review.opendev.org/c/openstack/nova/+/864418 | |
| 15:09:10 | bauzas | sean-k-mooney: based on your input ^ | |
| 15:09:36 | sean-k-mooney | cool ill take a look shortly | |
| 15:09:50 | sean-k-mooney | by the way backporting to/past xena is a pain | |
| 15:10:01 | sean-k-mooney | due to vmware/sudo-junko | |