| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-21 | |||
| 10:21:35 | sean-k-mooney | but its not the default | |
| 10:21:59 | gibi | I can simply promote my cell conductor to a super one by adding the api db settings to it | |
| 10:22:10 | sean-k-mooney | yep that would work | |
| 10:22:21 | gibi | but this really feels like something that upstream we dont test hence dont really support | |
| 10:22:37 | gibi | anyhow I do the propomotion and retry | |
| 10:22:38 | sean-k-mooney | well we do have supprot to test this in devstack | |
| 10:22:46 | gibi | do we have jobs testing it? | |
| 10:22:51 | sean-k-mooney | i dobt it | |
| 10:22:56 | gibi | we have code, we dont have test :D | |
| 10:23:09 | sean-k-mooney | ill check with codesearch | |
| 10:23:13 | sean-k-mooney | and see if anything enables it | |
| 10:23:30 | gibi | anyhow I go and do the promotion now and retry. I still believe that nova handles the missing group gracefully | |
| 10:24:20 | sean-k-mooney | so kolla-ansible disable the upcall out of the box i didnt get any hits form tripleo or pupet so i assuem they have it enabled | |
| 10:24:32 | sean-k-mooney | ooo actully would work since it does not supprot super conductors | |
| 10:25:10 | sean-k-mooney | well if you do multi cell its usign the cell1 conductor as the super conductor | |
| 10:25:18 | sean-k-mooney | so it would work only in cell1 | |
| 10:26:04 | gibi | :) | |
| 10:26:04 | gibi | intersting setup | |
| 10:26:19 | sean-k-mooney | https://opendev.org/openstack/grenade/src/branch/master/.zuul.yaml#L188= | |
| 10:26:31 | sean-k-mooney | so our grenade job apprently uses singel conductor mode | |
| 10:27:12 | sean-k-mooney | so we could renable the upcall check there if we wanted | |
| 10:27:52 | sean-k-mooney | gibi: could we consider changing the default for this or removing the call eventually | |
| 10:27:58 | gibi | yeah but there we dont do evac testing | |
| 10:28:19 | gibi | sean-k-mooney: I don't know what is the startegy I think originally we wanted to get rid of all the upcall | |
| 10:28:22 | gibi | s | |
| 10:28:36 | sean-k-mooney | yes we did | |
| 10:28:42 | sean-k-mooney | we currently only have 2 i think | |
| 10:28:42 | gibi | but apperantly we never went there and replaced them with calls pushing down the necessary daya | |
| 10:28:45 | gibi | data | |
| 10:29:13 | sean-k-mooney | we have the track_instance_changes config option ot push data to the schduler | |
| 10:29:22 | sean-k-mooney | and the group policy check upcall | |
| 10:32:20 | gibi | OK now I can reproduce the issue on master https://paste.opendev.org/show/btK22SLw4UH6oDDu5wKw/ | |
| 10:32:23 | gibi | we go this way https://github.com/openstack/nova/blob/ebe08834f311e8e22bfd9685d7e6e91dab967382/nova/compute/manager.py#L1733 | |
| 10:32:37 | gibi | and there we dont have the except branch | |
| 10:32:48 | gibi | not like in L1741 | |
| 10:33:37 | sean-k-mooney | yep | |
| 10:33:53 | sean-k-mooney | that what i was saying downstream | |
| 10:34:04 | gibi | now we have proof :) | |
| 10:34:15 | sean-k-mooney | we jut need to move the try to the top of the if | |
| 10:34:21 | gibi | bauzas: ^^ I go re-triage the upstream bug | |
| 10:34:39 | sean-k-mooney | so https://github.com/openstack/nova/blob/ebe08834f311e8e22bfd9685d7e6e91dab967382/nova/compute/manager.py#L1738= need to move to https://github.com/openstack/nova/blob/ebe08834f311e8e22bfd9685d7e6e91dab967382/nova/compute/manager.py#L1718= | |
| 10:35:18 | sean-k-mooney | it shoudl be easy to prove that with a unit test too | |
| 10:39:57 | gibi | sean-k-mooney, bauzas: I updated the upstream bug. I agree that we should fix this as suggested ^^. | |
| 10:40:16 | gibi | but now I go get some lunch and go back to the pci tracking patches | |
| 10:42:33 | sean-k-mooney | gibi: do you want me to just submit a unit test repoducer and then move the try | |
| 10:42:41 | gibi | sean-k-mooney: sure you can go | |
| 10:42:59 | gibi | I'm not assigning the bug to me :) | |
| 11:25:47 | opendevreview | sean mooney proposed openstack/nova master: ignore deleted server groups in validation https://review.opendev.org/c/openstack/nova/+/847001 | |
| 11:25:47 | opendevreview | sean mooney proposed openstack/nova master: add repoducer test for bug 1890244 https://review.opendev.org/c/openstack/nova/+/847000 | |
| 11:25:56 | sean-k-mooney | bauzas: gibi ^ | |
| 11:54:39 | 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:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Basics for PCI Placement reporting https://review.opendev.org/c/openstack/nova/+/846187 | |
| 11:54:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject mixed VF rc and trait config https://review.opendev.org/c/openstack/nova/+/846436 | |
| 11:54:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject PCI dependent device config https://review.opendev.org/c/openstack/nova/+/846435 | |
| 11:54:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject devname based device_spec config https://review.opendev.org/c/openstack/nova/+/846466 | |
| 11:54:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Ignore PCI devs with physical_network tag https://review.opendev.org/c/openstack/nova/+/846219 | |
| 11:54:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support [pci]device_spec reconfiguration https://review.opendev.org/c/openstack/nova/+/846470 | |
| 11:54:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Stop if tracking is disable after it was enabled before https://review.opendev.org/c/openstack/nova/+/847009 | |
| 11:58:01 | opendevreview | Rajesh Tailor proposed openstack/nova stable/wallaby: Add missing condition https://review.opendev.org/c/openstack/nova/+/847011 | |
| 11:58:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extra tests for remote managed dev spec https://review.opendev.org/c/openstack/nova/+/844626 | |
| 11:58:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add more test coverage for devname base dev spec https://review.opendev.org/c/openstack/nova/+/844625 | |
| 11:58:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PciAddressSpec descendants to call super.__init__ https://review.opendev.org/c/openstack/nova/+/844565 | |
| 11:58:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Unparent PciDeviceSpec from PciAddressSpec https://review.opendev.org/c/openstack/nova/+/844491 | |
| 11:58:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Poison /sys access via various calls in test https://review.opendev.org/c/openstack/nova/+/844627 | |
| 11:58:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove dead code from PhysicalPciAddress https://review.opendev.org/c/openstack/nova/+/844628 | |
| 11:58:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove unused PF checking from get_function_by_ifname https://review.opendev.org/c/openstack/nova/+/845775 | |
| 11:58:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Clean up mapping input to address spec types https://review.opendev.org/c/openstack/nova/+/845765 | |
| 11:58:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix type annotation of pci.Whitelist class https://review.opendev.org/c/openstack/nova/+/845780 | |
| 11:58:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move __str__ to the PciAddressSpec base class https://review.opendev.org/c/openstack/nova/+/845781 | |
| 12:07:18 | gibi | sean-k-mooney, bauzas: I consider the first (first and the half:D) batch of patches of pci tracking in placement ready for review. I posted a rewiew guide to the ML for help http://lists.openstack.org/pipermail/openstack-discuss/2022-June/029156.html | |
| 12:10:02 | gibi | sean-k-mooney: I'm +2 on the evac server group fix | |
| 12:10:06 | gibi | bauzas: easy win ^^ | |
| 12:13:58 | sean-k-mooney | :) i see it in my inbox ill try and take a look at them this week | |
| 12:14:59 | sean-k-mooney | """This covers the first 4 sub chapters of Proposed Change""" | |
| 12:15:35 | sean-k-mooney | have i said how happy i am that you volenterred to work on this :) | |
| 12:17:19 | gibi | yes you have :) | |
| 12:17:48 | gibi | actually after the initial shock, this batch of patches was a nice coding excersize | |
| 12:24:48 | opendevreview | Rajesh Tailor proposed openstack/nova master: Remove unnecessary if condition https://review.opendev.org/c/openstack/nova/+/844418 | |
| 12:42:18 | bauzas | gibi: sean-k-mooney: sorry, was doing other things, will look above | |
| 12:42:29 | gibi | bauzas: no worries | |
| 12:59:47 | bauzas | sean-k-mooney: thanks for having worked on it, it was an easy peasy | |
| 13:01:08 | sean-k-mooney | yep simple fix | |
| 13:02:35 | bauzas | sean-k-mooney: just -1 because of the try clause https://review.opendev.org/c/openstack/nova/+/847001 | |
| 13:03:14 | sean-k-mooney | i will have to duplicate the try | |
| 13:03:39 | sean-k-mooney | the only place that its not needed for is the schdueler_hints.get | |
| 13:04:26 | sean-k-mooney | i can duplicate it but i think this is cleaner but whatever ye prefer | |
| 13:14:27 | opendevreview | sean mooney proposed openstack/nova master: ignore deleted server groups in validation https://review.opendev.org/c/openstack/nova/+/847001 | |
| 13:14:51 | sean-k-mooney | bauzas: ^ | |
| 13:31:36 | bauzas | sean-k-mooney: +2d with a comment on the relnote (sorry missed it at first look) | |
| 14:04:14 | bauzas | gibi: updated the agenda for today's meeting | |
| 14:04:18 | gibi | bauzas: thanks | |
| 14:04:25 | bauzas | gibi: thanks for chairing it until I arrive | |
| 14:04:46 | bauzas | Uggla: missed to review your patches, yet another day with other prios | |
| 14:19:24 | bauzas | but I'll back around the meeting time | |
| 14:36:20 | opendevreview | Rajesh Tailor proposed openstack/nova master: Remove unnecessary if condition https://review.opendev.org/c/openstack/nova/+/844418 | |
| 14:59:50 | Uggla | bauzas, no pb. | |
| 15:00:19 | Uggla | bauzas, sean-k-mooney, please have a quick look at https://review.opendev.org/c/openstack/nova/+/845581/1/nova/compute/api.py and just tell me what you prefer. Longer code with logs (current version) vs shorter one with less logs (proposed by Gibi). If you prefer Gibi's proposal, I will refactor and you will be able to do the review with an updated version. | |
| 15:02:23 | Uggla | bauzas, sean-k-mooney, gibi can we have a look at https://review.opendev.org/c/openstack/nova-specs/+/833669/7..9/specs/zed/approved/libvirt-virtiofs-attach-manila-shares.rst#b94 to get rid of this point tomorrow morning ? | |
| 15:03:44 | sean-k-mooney | Uggla: i think shorter | |
| 15:03:53 | sean-k-mooney | just looking at https://review.opendev.org/c/openstack/nova/+/845581/1/nova/compute/api.py quickly | |