| *** ministry is now known as __ministry | 00:54 | |
| opendevreview | melanie witt proposed openstack/nova master: DNM testing https://review.opendev.org/c/openstack/nova/+/925635 | 01:00 |
|---|---|---|
| *** bauzas_ is now known as bauzas | 01:12 | |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | 02:45 |
| opendevreview | melanie witt proposed openstack/nova master: DNM testing cont'd https://review.opendev.org/c/openstack/nova/+/925838 | 06:57 |
| opendevreview | melanie witt proposed openstack/nova master: DNM testing cont'd https://review.opendev.org/c/openstack/nova/+/925838 | 07:04 |
| melwitt | sean-k-mooney: just for illustrating how the fetch_func stuff is not used at all in this poc ^ (hopefully I didn't mess up in deleting) | 07:10 |
| *** bauzas_ is now known as bauzas | 07:36 | |
| gibi | melwitt: thanks for proposing the fix for the persistent mdev bug | 07:41 |
| *** bauzas_ is now known as bauzas | 08:57 | |
| ykarel | can someone check the zed backport https://review.opendev.org/c/openstack/nova/+/925749 | 10:06 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | 10:11 |
| elodilles | ykarel: clean cherry pick so +2+W'd it | 11:48 |
| ykarel | thx elodilles | 11:48 |
| gibi | ykarel: as that targets an unmaintained branch nova core team is not involved reviewing and landing it | 11:48 |
| gibi | but I guess elodilles is in the global unmaintained core team :) | 11:51 |
| ykarel | gibi, ack | 11:53 |
| sean-k-mooney | so i have debated this in the past with my self but i think the unmatined branches woudl benifit form a dedicated channel that is seperate form the proejct channels to cordinate unmatined branch reviews | 11:53 |
| elodilles | sean-k-mooney: there is a semi-official channel: #openstack-unmaintained o:) | 11:54 |
| sean-k-mooney | ack provided the unmaintained pings are not excessive i dont entirly object to havign them here too but we were very clear in teh resolution that core teams would not review patches too or genericallly maintiane the unmainted branches as part fo the resolution | 11:57 |
| elodilles | yepp | 11:58 |
| sean-k-mooney | so unless an unmainted branch maitiner wants input on a patch they are unsure about i woudl prefer to avoid the day to day work of maintainign them here | 11:58 |
| sean-k-mooney | elodilles: if you are unsure about somethign imporant liek a cve back port we will proably be fine with a quick skim over a patch you have prepared | 11:59 |
| sean-k-mooney | or if your trying to resolve a merge conflcit and we are not too busy but in general for can we review x i would prefer to keep that in #openstack-unmaintained | 12:00 |
| elodilles | sean-k-mooney: ACK, thanks in advance! o:) | 12:13 |
| * gibi don't even have +2 rights on unmaintaned/ | 12:23 | |
| gibi | (not that I want it) | 12:23 |
| sean-k-mooney | same i considerd puting my name forward but i have too much on my plate currently to really consier it | 12:24 |
| sean-k-mooney | i know many are in that both but im jsut relecting reality that i cant spend time on the unmainited branches at present | 12:25 |
| gibi | we set them unmaintaned as we don't want to / cannot maintain them. So it is fair to say that we dont care :) | 12:35 |
| *** tkajinam is now known as Guest2415 | 12:52 | |
| *** tkajinam is now known as Guest2416 | 13:00 | |
| *** ministry is now known as __ministry | 13:55 | |
| *** whoami-rajat_ is now known as whoami-rajat | 13:56 | |
| stephenfin | elodilles: Speaking of unmaintained, we have a variety of git-review proposal patches open against cliff, python-openstackclient and openstacksdk. Can you or someone abandon those since it's very unlikely anyone will use those branches for client tooling | 15:34 |
| opendevreview | melanie witt proposed openstack/nova master: libvirt: Remove node device XML validate flags https://review.opendev.org/c/openstack/nova/+/925826 | 16:34 |
| JayF | sean-k-mooney: & other cores; https://review.opendev.org/q/topic:ironic-guest-metadata is tested working in devstack, I wanted some review feedback before moving beyond this point into migration/tempest. I've asked Ironic cores to look at them as well. | 16:39 |
| sean-k-mooney | i was ment to loop back to those after the cve | 16:40 |
| JayF | I figured :). No real rush just sometime in the next few days would be nice | 16:40 |
| JayF | I'm pretty crushed right now too and wouldn't be able to do anything immediately with the feedback, but I know the queue is long so wanted to take my number ;) | 16:41 |
| sean-k-mooney | JayF: does the ironic patch depend on the nova one and is that tested in ci | 16:42 |
| sean-k-mooney | i.e. can we use that to demonstrate thsi end to end | 16:42 |
| sean-k-mooney | we do have results form https://zuul.opendev.org/t/openstack/build/0c7147a1aea541eb88ae387cb5487fe0 also | 16:43 |
| sean-k-mooney | which i guess we can reference im just not sure if the metadta will be visable in the ironic logs but i woudl hope we can see it properly there | 16:43 |
| JayF | sean-k-mooney: I tested it manually in devstack. automatic_lessee defaults to false in Ironic so it won't show up in CI until we explicitly test it; and I'm not going to add to the chain until I get some directional feedback (i.e. this is good if tested) | 16:43 |
| JayF | sean-k-mooney: mainly just trying to avoid a thing where I write 5 patches across 5 repos and get feedback to change a single word that takes an hour because of the stacking :P | 16:44 |
| JayF | I knew I shoulda done a quick+dirty demo video; CID did that for his recent Ironic API change and it was WONDERFUL as a reviewer to see | 16:45 |
| sean-k-mooney | ack we can likely turn it on in that job in a follwo up testing patch | 16:45 |
| sean-k-mooney | by the way | 16:45 |
| sean-k-mooney | Jul 25 20:34:42.508653 np0038062841 nova-compute[106292]: ERROR nova.network.neutron [req-a7169636-31a7-4d11-9017-a6865cd16b59 req-102a3eac-094a-42f8-a114-8ac7a841a33c service nova] [instance: 929bb573-a06a-4fa7-a77a-c643bd9655ce] The vnic_type of the bound port 88d554aa-8c7a-46ac-b54b-efeac135c643 has been changed in neutron from "normal" to "baremetal". Changing vnic_type of | 16:45 |
| sean-k-mooney | a bound port is not supported by Nova. To avoid breaking the connectivity of the instance please change the port vnic_type back to "normal". | 16:45 |
| sean-k-mooney | that is a sideffect of ironic doing somethign that is not allowed that related to a cve previously | 16:46 |
| JayF | Can you document that in a bug somewhere please? | 16:47 |
| sean-k-mooney | in this speicic case its not problematic form a secuirty point of view but is a side effect of the exisitng tech debt | 16:47 |
| sean-k-mooney | sure my guess is the job is not configure properly to set the tempest config option for the vnic_type to baremental | 16:47 |
| sean-k-mooney | speicicaly https://github.com/openstack/tempest/blob/master/tempest/config.py#L784-L791 | 16:48 |
| sean-k-mooney | if you need a prot type other the normal nova only supprot that by passing a precreated port as part of the server create request | 16:52 |
| sean-k-mooney | so ironic just like sriov ports required tempest to be configued to use the relevent vnic type in the job | 16:53 |
| sean-k-mooney | JayF: https://bugs.launchpad.net/ironic/+bug/2076265 os for the issue in the job config althoguh i think the larger issue of nova/ironic/neutron interaction is really a seprate problem | 17:08 |
| JayF | sean-k-mooney: I'd suggest sending an email to the mailing list proposing a joint PTG session then. It's not a topic I can address well in the moment as networking is far from my area of expertise in Ironic and I'd need to get more context | 17:09 |
| elodilles | stephenfin: i guess those are mainly the .gitreview patches. i'll rather check if we could merge them now. (i also don't think that we will get (many) backports to these projects, but at least the .gitreview file should be correct) | 17:10 |
| *** bauzas_ is now known as bauzas | 18:15 | |
| *** bauzas_ is now known as bauzas | 18:26 | |
| *** bauzas_ is now known as bauzas | 19:36 | |
| *** ministry is now known as __ministry | 20:06 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!