| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-18 | |||
| 09:49:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 09:49:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 09:49:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 10:03:45 | opendevreview | Elod Illes proposed openstack/nova stable/xena: Add missing condition https://review.opendev.org/c/openstack/nova/+/843821 | |
| 10:35:52 | opendevreview | Elod Illes proposed openstack/nova stable/wallaby: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/821978 | |
| 10:53:40 | opendevreview | Rajesh Tailor proposed openstack/nova master: Remove return from rpc cast https://review.opendev.org/c/openstack/nova/+/845335 | |
| 11:37:43 | opendevreview | Rajesh Tailor proposed openstack/nova master: Update Availability zone doc page https://review.opendev.org/c/openstack/nova/+/846463 | |
| 11:54:48 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 11:54:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 11:54:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 11:54:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 11:54:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: mypy-squas it back https://review.opendev.org/c/openstack/nova/+/846464 | |
| 11:55:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Basics for PCI Placement reporting https://review.opendev.org/c/openstack/nova/+/846187 | |
| 11:55:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 11:55:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 11:55:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 11:55:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 12:04:10 | opendevreview | Merged openstack/nova stable/wallaby: libvirt: Abort live-migration job when monitoring fails https://review.opendev.org/c/openstack/nova/+/837320 | |
| 12:45:25 | opendevreview | Merged openstack/nova stable/xena: Add missing condition https://review.opendev.org/c/openstack/nova/+/843821 | |
| 13:17:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 13:17:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 13:17:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 13:17:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: squash to first dev_spec https://review.opendev.org/c/openstack/nova/+/846465 | |
| 13:17:11 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 14:03:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: Basics for PCI Placement reporting https://review.opendev.org/c/openstack/nova/+/846187 | |
| 14:03:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 14:03:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 14:03:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 14:03:38 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 14:03:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 16:48:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 16:51:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename [pci]passthrough_whitelist to device_spec https://review.opendev.org/c/openstack/nova/+/843834 | |
| 16:51:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename exception.PciConfigInvalidWhitelist to PciConfigInvalidSpec https://review.opendev.org/c/openstack/nova/+/843861 | |
| 16:51:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename whitelist in tests https://review.opendev.org/c/openstack/nova/+/843862 | |
| 16:51:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Basics for PCI Placement reporting https://review.opendev.org/c/openstack/nova/+/846187 | |
| 16:51:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extend device_spec with resource_class and traits https://review.opendev.org/c/openstack/nova/+/846218 | |
| 16:51:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 16:51:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 16:51:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 16:51:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 16:51:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 18:18:09 | opendevreview | Rajesh Tailor proposed openstack/nova stable/yoga: Fix typos in help messages https://review.opendev.org/c/openstack/nova/+/846260 | |
| #openstack-nova - 2022-06-19 | |||
| 03:48:01 | opendevreview | norman shen proposed openstack/nova master: Clear connection info if vol disconnected https://review.opendev.org/c/openstack/nova/+/845995 | |
| #openstack-nova - 2022-06-20 | |||
| 07:49:58 | Uggla | Good morning | |
| 09:27:27 | stephenfin | Morning. Can someone look at this real simple unit test-only patch for me? https://review.opendev.org/c/openstack/nova/+/802893 | |
| 09:29:52 | sean-k-mooney | nova manage | |
| 09:29:57 | sean-k-mooney | sure ill take a look | |
| 09:31:19 | sean-k-mooney | oh its a followup to https://review.opendev.org/c/openstack/nova/+/800855 | |
| 09:31:51 | sean-k-mooney | ah yes i remember that | |
| 09:32:07 | stephenfin | thanks :) | |
| 09:39:01 | sean-k-mooney | +2 a minor unrelated formating chnage is included but im fine with that | |
| 10:03:14 | opendevreview | Rajesh Tailor proposed openstack/nova master: Remove unnecessary if condition https://review.opendev.org/c/openstack/nova/+/844418 | |
| 15:20:05 | opendevreview | Merged openstack/nova master: tests: Add test for bug #1936278 https://review.opendev.org/c/openstack/nova/+/802893 | |
| 16:22:44 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: Fix verifying all the alloc requests from a multi-create https://review.opendev.org/c/openstack/nova/+/846786 | |
| #openstack-nova - 2022-06-21 | |||
| 03:49:51 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/846876 | |
| 03:56:49 | opendevreview | OpenStack Proposal Bot proposed openstack/python-novaclient master: Imported Translations from Zanata https://review.opendev.org/c/openstack/python-novaclient/+/846880 | |
| 07:40:44 | bauzas | gibi: you maybe missed https://review.opendev.org/c/openstack/nova/+/846786 | |
| 07:42:52 | gibi | bauzas: I think what you did is what we discussed. I would add some testing around the for:else: and the breaks but otherwise I agree with the direction | |
| 07:43:17 | bauzas | gibi: yup, of course, this is a WIP b/c I would add a UT | |
| 07:44:07 | gibi | so then that WIP looks OK to me | |
| 09:16:05 | elodilles | bauzas sean-k-mooney : can you please review the release patches (whether you see something that requires MINOR version bump, etc). I don't think it's worth to wait more time, as every week we'll have new patches + gate is sometimes slow and days are passing o:) | |
| 09:16:57 | elodilles | bauzas sean-k-mooney : of course if you see any patch that is a MUST to have in any of the release, then we can wait. but otherwise i think it's better to release now o:) | |
| 09:17:54 | elodilles | bauzas sean-k-mooney : and release managers won't review the release patches until PTL or release liaisons haven't reviewed the patches ;) | |
| 09:18:55 | sean-k-mooney[m] | actully i ment to update the patches | |
| 09:19:01 | sean-k-mooney[m] | to use the new sha last week | |
| 09:19:33 | sean-k-mooney[m] | most of the patches i wanted to wait for have alredy landed last week | |
| 09:19:50 | sean-k-mooney[m] | we just need to repin the patches to point to the currnent tip of the stable branches | |
| 09:21:02 | sean-k-mooney[m] | elodilles give me 10 mins to grab a coffee and ill go do that when i get back and +1 them | |
| 09:21:31 | elodilles | sean-k-mooney[m]: i've updated the patches with the latest hashes :) | |
| 09:21:50 | elodilles | sean-k-mooney[m]: no need to hurry as i also will go to have lunch ;) | |
| 09:22:38 | elodilles | so enjoy your coffee, and review the patches some time later today o:) | |
| 09:22:48 | sean-k-mooney[m] | oh ok thanks i stilll ment to do this last week so before i start anything else ill review them | |
| 09:23:28 | bauzas | elodilles: ack, will look | |
| 09:23:44 | bauzas | gibi: reopened https://bugs.launchpad.net/nova/+bug/1890244 fwiw | |
| 09:23:45 | elodilles | sean-k-mooney[m] bauzas: ++, thanks in advance! \o/ | |
| 09:26:29 | bauzas | gibi: I'm curious how you didn't went able to reproduce the bug in https://bugs.launchpad.net/nova/+bug/1890244/comments/1 | |
| 09:26:31 | gibi | bauzas: so probably CONF.workarounds.disable_group_policy_check_upcall was the trick why I was not able to reproduce it | |
| 09:26:42 | bauzas | gibi: hah | |
| 09:27:06 | bauzas | gibi: but IMHO we default to False | |
| 09:27:22 | gibi | then I dont know | |
| 09:28:11 | gibi | bauzas: did you reproduced it? | |
| 09:28:14 | bauzas | I need to respin my devstack env | |
| 09:28:42 | bauzas | gibi: no, I only triaged it and we have a downstream BZ that was related | |
| 09:28:49 | bauzas | https://bugzilla.redhat.com/show_bug.cgi?id=2099279 | |
| 09:29:01 | bauzas | the code is pretty simple to look at | |
| 09:30:35 | bauzas | oh wait, you could be right, shit. | |
| 09:31:00 | bauzas | when we populate the ReqSpec, we rehydrate the fields and we can set the group value to None if the group was deleteed | |
| 09:32:04 | bauzas | https://github.com/openstack/nova/commit/94fd36f0582c5dbcf2b9886da7c7bf986d3ad5d1 is super old | |
| 09:35:35 | gibi | I still cannot reproduce | |
| 09:35:35 | gibi | https://paste.opendev.org/show/bXPvHPVJ7FfPgjfhTsSv/ | |
| 09:37:46 | bauzas | gibi: then I don't understand | |
| 09:37:54 | bauzas | gibi: I verified master | |
| 09:38:07 | gibi | the above repro trial I run on master too | |
| 09:38:16 | bauzas | gibi: when we evacuate, we regenerate the old RequestSpec record | |
| 09:38:20 | gibi | I don't have a multinode train env | |
| 09:38:30 | bauzas | and then, we set the hints | |
| 09:39:57 | bauzas | gibi: can you look at your reqspec record and see whether you have the hints dict ? | |
| 09:40:06 | gibi | sure | |
| 09:41:07 | gibi | https://paste.opendev.org/show/baqepgHm3PhuQ9VhvfWi/ | |
| 09:41:29 | gibi | there is the group hint in the dfb | |