Thursday, 2023-06-01

« Wednesday, 2023-05-31 Index Friday, 2023-06-02 »
TheJuliabase os?00:06
NobodyCamubuntu2200:07
TheJuliawhat is happening?00:07
NobodyCamI don't have a centos 9 in this env00:07
TheJuliaeh, centos9 would be more headache, ubuntu just generally works00:07
NobodyCamdies at neutron...00:08
TheJuliahow so?00:08
NobodyCamunable to create tenant network00:08
NobodyCamtrying a older conf now..00:09
NobodyCamgoogling I'm not sure which options to enable / disable00:09
TheJuliawhat is the error you get when you try to create the tenant network?00:09
NobodyCamrunning now00:11
NobodyCamIs there a newer version of this: https://docs.openstack.org/ironic/latest/contributor/ironic-multitenant-networking.html00:13
TheJuliaI mean, the doc is likely out of date00:13
TheJuliause the config from zuul.d/ironic-jobs.yaml00:13
NobodyCam;)00:13
NobodyCam++ have tried ... will try again :)00:14
TheJuliaokay00:27
TheJuliai forgot how intertwined network fwapping was00:27
NobodyCamis there a way to fire the jobs build local.conf 00:27
NobodyCamsadly yea00:27
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: service steps  https://review.opendev.org/c/openstack/ironic/+/88478300:28
TheJuliaI'm going to have to get ^^^ sorted and split it most likely00:28
TheJuliahere CI job, here ci job00:28
TheJuliahttps://6bf08761604833c0eeb6-bc69b84197ebb41a6ac7b0fa05f62021.ssl.cf2.rackcdn.com/883062/7/check/ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode/6a957e7/controller/logs/_.localrc_auto.txt00:30
NobodyCamNice.... Thank you!00:30
JayFI'm going to put changing the linked doc to ^ basically how to steal CI job configs to get a working devstack into my todo list for tomorrow00:31
TheJuliaJayF: sounds like a good plan00:31
JayFthat is no guarantee it will get done00:32
JayFbut it's written down in a place I will look :D 00:32
NobodyCamJayF: \o/ woo hoo00:32
NobodyCamokay going to call it a day... 00:35
opendevreviewJulia Kreger proposed openstack/ironic-tempest-plugin master: CI: Add RBAC specific tempest jobs for main branch exec  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/88231200:35
NobodyCamhave a great night all00:35
opendevreviewJulia Kreger proposed openstack/ironic-tempest-plugin master: rbac - Fix vif_attach expected return values  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/88303200:35
*** dmellado17 is now known as dmellado104:55
rpittaugood morning ironic! o/07:08
opendevreviewVerification of a change to openstack/bifrost master failed: Reduce multiple tasks to task with loop  https://review.opendev.org/c/openstack/bifrost/+/87718608:25
opendevreviewMerged openstack/ironic master: Add to Redfish hardware inventory collection  https://review.opendev.org/c/openstack/ironic/+/88178310:25
adam-metal3hello Ironic!11:00
adam-metal3FYI, I don't know if this was discussed in the Ironic community but In the Metal3 MTM it was discussed with a few of you, I have proposed this and I am implementing it on a fork , just wanted to let you know to avoid duplicating the work, https://bugs.launchpad.net/ironic-python-agent/+bug/202194711:02
dtantsurJayF: FYI ^^11:21
opendevreviewMahnoor Asghar proposed openstack/python-ironicclient master: Add support for node inventory  https://review.opendev.org/c/openstack/python-ironicclient/+/88383311:47
opendevreviewMahnoor Asghar proposed openstack/python-ironicclient master: Add support for node inventory  https://review.opendev.org/c/openstack/python-ironicclient/+/88383312:06
iurygregorygood morning ironic12:07
opendevreviewMahnoor Asghar proposed openstack/python-ironicclient master: Fix minor spelling errors and incorrect help text  https://review.opendev.org/c/openstack/python-ironicclient/+/88504012:15
opendevreviewMerged openstack/bifrost master: Reduce multiple tasks to task with loop  https://review.opendev.org/c/openstack/bifrost/+/87718612:58
TheJuliaadam-metal3: I think that is a no-brainer for us to accept into ipa's code base, really. A fork would just create a night mare later (as forks do)13:18
TheJuliaadam-metal3: I mean, if you *really* just don't want to use the existing url format for it13:20
TheJuliaCould I get another review on https://review.opendev.org/c/openstack/ironic/+/883581 ?13:30
iurygregorysure13:30
iurygregorywow green13:30
iurygregory\o/13:30
TheJuliagrenade failed on one pass of the xena patch, unrelated failure, rechecked. I'd like to get them merged so I can proceed backporting13:31
iurygregory++13:32
iurygregorychecking the diff now13:32
TheJuliaidentical for the two minus CI on wallaby13:33
TheJulia... unrelated the worl hates us because tempest is not branched13:34
iurygregorytruth13:34
rpittauthe fact that tempest is not branched has a lot of ramifications (sorry)13:35
TheJulias/worl/world/13:36
opendevreviewVerification of a change to openstack/ironic stable/xena failed: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88358013:36
* TheJulia wonders where that logic is held13:37
JayFadam-metal3: I agree with Julia. JFDI13:47
JayF 13:47
TheJulia++13:47
JayFadam-metal3: please please don't ever feel like you have to fork one of our projects to get a feature you need13:47
TheJulia... a lot of us have had to deal with the pain of forks, so we actively try to avoid it  too13:48
TheJuliaPAIN13:48
adam-metal3JayF TheJulia:We have permanent forks (even unchanged forks) in the Nordix github project as simetimes we have to provide code for downstream faster than OSS processes would allow it my intention is to usptream everything I cna :D14:02
adam-metal3but I am very happy to hear that you like the proposal :D14:02
dtantsurTheJulia: an argument against username/password in the URL: they're not masked this way14:03
dtantsura new argument, say image_password, will be automatically masked IIRC14:03
TheJuliauhhhhhh I'm not so sure about that, honestly14:05
TheJuliaI thought we checked 14:05
TheJuliabut ETOOMANYTHREADS right now14:05
dtantsurI'm quite sure we mask everything with "password" or "token" in it, but dunno about URLs14:05
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic master: Add DB API for Firmware and Object  https://review.opendev.org/c/openstack/ironic/+/88306214:06
adam-metal3I am not an expert but I thought basic auth is only initiated if the "auth" argument is supplied with some value in the session's request function, is it so that I can put it into the url also?14:08
dtantsurI haven't checked it, I must admit. We use a separate argument for Redfish, for example.14:08
iurygregoryseems like HPE CI is having problems, "Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset."14:09
TheJuliaadam-metal3: python requests, specifically recognizes the age old "url form" and then uses it if supplied14:24
TheJuliawell, I think it is actually urllib, but yeah14:25
TheJuliait is somewhere there and we've confirmed it does work14:25
adam-metal3TheJulia: ohh wow thanks good to know14:26
TheJuliabut as dtantsur has mentioned, it might not be masked14:26
TheJuliaI think it is if you api query it, I just don't remember at this point14:26
TheJuliaa separate parameter with "password" in the name definitely is masked automatically14:27
TheJuliaso overall, what your proposing is a good thing14:27
adam-metal3cool14:33
opendevreviewOpenStack Release Bot proposed openstack/ironic-inspector bugfix/11.5: Update .gitreview for bugfix/11.5  https://review.opendev.org/c/openstack/ironic-inspector/+/88505814:47
opendevreviewMahnoor Asghar proposed openstack/python-ironicclient master: Add support for node inventory  https://review.opendev.org/c/openstack/python-ironicclient/+/88383314:50
opendevreviewOpenStack Release Bot proposed openstack/ironic-python-agent bugfix/9.5: Update .gitreview for bugfix/9.5  https://review.opendev.org/c/openstack/ironic-python-agent/+/88505914:59
opendevreviewMahnoor Asghar proposed openstack/python-ironicclient master: Add support for node inventory  https://review.opendev.org/c/openstack/python-ironicclient/+/88383315:09
opendevreviewMerged openstack/python-ironicclient master: Fix minor spelling errors and incorrect help text  https://review.opendev.org/c/openstack/python-ironicclient/+/88504015:11
JayFTheJulia: do you intend to fix cinder issue for T/U/V?15:15
JayFTheJulia: and/or does it exist? Only asking for purposes of if it's "done" for me to update the OSSA15:16
TheJuliadunno, community opinion there?15:16
JayFI have users on V, I wouldn't hate it getting patched...15:16
TheJuliaoh, it definitely exists15:16
TheJuliaI can take it to Train then, it won't be the end of the world.15:16
JayFbut I also think T/U/V, with upstream hat on, are nearing the need to be EOL'd15:16
JayFwhich in a weird way is more reason to fix it15:17
TheJuliayeah15:17
JayFensure the final state of those branches is working15:17
TheJuliayeah15:17
opendevreviewJulia Kreger proposed openstack/ironic stable/victoria: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88501615:18
TheJuliamanual backports after that :)15:19
opendevreviewJulia Kreger proposed openstack/ironic stable/ussuri: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88506315:23
NobodyCamgood morning Openstack folks15:24
opendevreviewJulia Kreger proposed openstack/ironic stable/ussuri: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88506315:24
iurygregoryhey NobodyCam o/15:24
TheJuliaGüten Morgen15:25
opendevreviewJulia Kreger proposed openstack/ironic stable/train: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88506515:26
TheJuliano... idea... how stable/train is going to look15:26
iurygregory*pain*15:26
TheJuliayes, pain15:28
TheJuliahttps://www.youtube.com/watch?v=I_keWS1i3RA15:29
TheJuliaI <3 that movie15:29
rpittauTheJulia: then you'll appreciate https://boardgamegeek.com/boardgame/316750/princess-bride-adventure-book-game15:43
opendevreviewVerification of a change to openstack/ironic stable/wallaby failed: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88358115:43
TheJuliaseriously CI why you hate me15:53
rpittaugood night! o/16:00
TheJuliagoodnight16:00
opendevreviewVerification of a change to openstack/ironic stable/wallaby failed: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88358116:09
opendevreviewVerification of a change to openstack/ironic stable/xena failed: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88358017:52
opendevreviewVerification of a change to openstack/ironic stable/xena failed: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88358018:01
opendevreviewJulia Kreger proposed openstack/ironic master: DNM Enable OVN  https://review.opendev.org/c/openstack/ironic/+/88508718:09
opendevreviewJulia Kreger proposed openstack/ironic master: DNM Enable OVN  https://review.opendev.org/c/openstack/ironic/+/88508719:09
TheJuliaturns out you need to actually put the change on a job which runs19:17
TheJuliaJayF: w/r/t some of the older branch grenade jobs, perhaps... deep networking issues are a rabbithole for use19:21
JayFTheJulia: what's the context on the OVN change? 19:22
JayFjust curious19:22
TheJuliaApparently upstream OVN now supports v6 dhcp... apparently19:22
TheJuliaso in theory, we could switch to using it19:22
TheJulia(and I'm sick and tired of dhcp not working)19:22
JayFoh, that's nice19:23
TheJulia(and supposidly v4 dhcp should just work... so time for proof to be acquired)19:25
JayFwhat do we use instead, now?19:26
JayFOVS? 19:26
sschmittI was looking at the OVN baremetal PXE at one point but didn't get to finishing. I did see the dhcp options being set correctly from what I could tell19:28
TheJuliaOVS19:32
TheJuliasschmitt: for a time OVN ignored any packet which came off of the physical port :\ *tableflip*19:33
sschmittwell that's neat19:34
opendevreviewJulia Kreger proposed openstack/ironic master: DNM Enable OVN  https://review.opendev.org/c/openstack/ironic/+/88508719:34
TheJulia... yeah, it was so awesome19:34
opendevreviewMerged openstack/ironic-lib stable/train: Remove now-dead CoreOS job, non-passing jobs  https://review.opendev.org/c/openstack/ironic-lib/+/88472220:15
opendevreviewMerged openstack/ironic stable/xena: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88358021:29
*** JayF is now known as Guest196221:37
*** JasonF is now known as JayF21:37
iurygregoryhappy dance xena is fixed21:41
opendevreviewJulia Kreger proposed openstack/ironic-python-agent-builder master: Add libaio engine for fio on Centos/Rhel9  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/88510021:58
TheJuliawoot22:07
opendevreviewMerged openstack/ironic stable/wallaby: Fix Cinder Integration fallout from CVE-2023-2088  https://review.opendev.org/c/openstack/ironic/+/88358122:16
iurygregoryWOOT22:38
iurygregory\o/22:38
TheJuliaSweet22:39
TheJuliaNow the rest!22:39
NobodyCamiurygregory: \0 howdy22:45
iurygregorydoing good =)22:47
iurygregoryfighting with stevedore to understand why is not loading the extension I added 22:49
NobodyCamI'm fighting dev stack... well neutron23:19
NobodyCam`Unable to create the network. No tenant network is available for allocation.`23:19
JayFNobodyCam: are you, perhaps, running this in a libvirt VM? 23:22
JayFNobodyCam: I had some problems with the default libvirtd VM network being the same as the one bifrost wants to use in that dev environment23:23
JayFI have no idea whatsoever if that can impact devstack; I suspect not; but it might give you an idea where to look next.23:23
NobodyCam+++ oh good pointer... I have esxi, qemu, and a bm host I am testing on23:26
« Wednesday, 2023-05-31 Index Friday, 2023-06-02 »

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!