Wednesday, 2021-10-06

TheJuliajanders: thanks. UGH01:36
jandersTheJulia: thank you. This may merge in time to make EMEA folks life easier :)01:39
TheJuliahopefully01:40
* TheJulia goes to inspect water damage01:40
opendevreviewMerged openstack/ironic bugfix/18.1: Disable Neutron firewall  https://review.opendev.org/c/openstack/ironic/+/81243702:50
*** pmannidi is now known as pmannidi|brb03:07
*** pmannidi|brb is now known as pmannidi03:18
*** pmannidi is now known as pmannidi|AFK04:23
jandersiurygregory: ^ merging does seem to fix CI for the changes I'm looking at04:33
jandersrecheck of https://review.opendev.org/c/openstack/ironic/+/812433 succeeded 04:33
janders\o/04:33
*** pmannidi|AFK is now known as pmannidi05:27
iurygregorygood morning Ironic o/06:03
iurygregoryjanders, awesome! yeah I saw that some of the failures were related the the neutron issue since we only merged after bugfix18.1 - stable/xena06:04
arne_wiebalckGood morning, iurygregory and Ironic!06:19
arne_wiebalckand janders ofc o/06:19
iurygregoryarne_wiebalck, o/06:22
jeevaopenstack baremetal port create 74:E6:E2:FB:AB:A6 --node 08ef471c-f7bc-4258-8e9c-2953ff50255406:48
jeevais that MAC the MAC of the IDRAC or NIC1 ?06:49
iurygregoryjeeva, should be the MAC of the NIC06:59
iurygregoryironic inspector can provide the information about the MAC address of the NICs in the node (if you are using)07:01
jeevayeah i assumed so thanks, but it wasn't 100% clear )07:24
iurygregoryTheJulia, good news I was able to find some of the problems in the PR https://github.com/OpenStackweb/ironic-website/pull/43 \o/ (I'm going to add a review)07:30
jeevai deployed ironic using kolla-ansible (and its relevent ironic guide)07:54
jeevabut when i go to openstack GUI > admin > system > ironic bare metal provisioning, i get "Error: Unable to retrieve Ironic nodes. __init__() got an unexpected keyword argument 'project_id'"07:54
jandershey iurygregory arne_wiebalck jeeva and Ironic o/07:55
iurygregoryI never used the ironic-ui, maybe we have some bugs .-.07:55
iurygregoryjanders, o/07:55
rpittaugood morning ironic! o/08:04
iurygregorygood morning rpittau o/08:04
rpittauhey iurygregory :)08:04
jeevai got the machine powers on via openstack cli08:08
jeevajust need to work on my PXE network issues now08:08
jeevadocumentation is lacking08:08
jeeva(on kolla ironic)08:08
jeevabunch of other settings i find that needs to be updated as well08:08
dtantsurmorning folks08:16
rpittaugood morning dtantsur :)08:17
dtantsuriurygregory: I've just realized we have a PTG day scheduled on the RH recharge day08:19
iurygregorydtantsur, yes we do =)08:20
iurygregoryI will talk with my manager to change the date of the recharge day (it's possible)08:20
* dtantsur moves the conversation downstream08:20
iurygregoryyeah =)08:20
rpittauisn't the RH recharge the 12 ?08:21
iurygregorynope :D08:23
opendevreviewDmitry Tantsur proposed openstack/ironic master: Demote three warning messages  https://review.opendev.org/c/openstack/ironic/+/81264308:54
jeevais there perhaps a cheatsheet of every single config that needs to change in openstack to get ironic working ?09:27
dtantsurjeeva: our install guide is probably the closest thing09:28
dtantsurnot in the cheatsheet format though09:28
dtantsurhttps://docs.openstack.org/ironic/latest/install/install.html09:28
jeevai use kolla09:43
jeevaand stuff like /etc/kolla/config/neutron/ml2_conf.ini configs doesn't show on the docs09:44
dtantsurI guess it's a question for mgoddard then09:45
*** hjensas is now known as hjensas|lunch09:58
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP add a helper for node-based periodics  https://review.opendev.org/c/openstack/ironic/+/81249510:00
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent master: Remove else/elif when not needed  https://review.opendev.org/c/openstack/ironic-python-agent/+/81210810:44
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP add a helper for node-based periodics  https://review.opendev.org/c/openstack/ironic/+/81249511:11
dtantsurcould someone tell me if I'm overengeneering it at this point ^^11:11
opendevreviewMerged openstack/ironic stable/xena: [iRMC] Avoid repeatedly resuming clean after creating raid configuration  https://review.opendev.org/c/openstack/ironic/+/81243111:32
opendevreviewMerged openstack/ironic stable/xena: [iRMC] Set polling after RAID is built  https://review.opendev.org/c/openstack/ironic/+/81243211:39
opendevreviewMerged openstack/ironic bugfix/18.1: [iRMC] Avoid repeatedly resuming clean after creating raid configuration  https://review.opendev.org/c/openstack/ironic/+/81243411:39
opendevreviewMerged openstack/ironic bugfix/18.1: [iRMC] Set polling after RAID is built  https://review.opendev.org/c/openstack/ironic/+/81243311:42
opendevreviewMerged openstack/ironic master: Add support for verify steps  https://review.opendev.org/c/openstack/ironic/+/80000111:42
ajyadtantsur: lgtm11:44
dtantsurthx!11:48
janderssee you tomorrow Ironic o/11:49
opendevreviewMerged openstack/ironic master: Make iDRAC management steps verify steps  https://review.opendev.org/c/openstack/ironic/+/80403211:59
opendevreviewMerged openstack/ironic bugfix/18.1: Fix iDRAC import configuration missing task handling  https://review.opendev.org/c/openstack/ironic/+/81231311:59
iurygregorybye janders o/12:21
opendevreviewMerged openstack/ironic bugfix/18.1: Fix iDRAC import configuration job with errors  https://review.opendev.org/c/openstack/ironic/+/81231212:22
opendevreviewMerged openstack/ironic bugfix/18.1: Add better error messages for invalid conf molds  https://review.opendev.org/c/openstack/ironic/+/81231112:22
opendevreviewMerged openstack/ironic stable/wallaby: Add better error messages for invalid conf molds  https://review.opendev.org/c/openstack/ironic/+/80950812:22
jeevaironic_dnsmasq_interface: "mellanox1.505"12:44
jeevaironic_cleaning_network: "mellanox1.505"12:44
jeevamust this be the name or UUID12:44
jeevabecause in troubleshooting i get this:12:44
jeeva| network    | False  | cleaning network with name or UUID mellanox1.505 was not found12:44
opendevreviewManuel Schönlaub proposed openstack/sushy master: Add revision for drive resource  https://review.opendev.org/c/openstack/sushy/+/81267613:19
TheJuliajeeve uuid is far more absolute, using names brings risk13:28
iurygregorygood morning TheJulia =)13:29
TheJuliagood morning iurygregory 13:55
opendevreviewDmitry Tantsur proposed openstack/ironic master: WIP add a helper for node-based periodics  https://review.opendev.org/c/openstack/ironic/+/81249514:54
dtantsurthis is shaping well IMO ^^14:54
dtantsuron this positive note I need to go, talk to you tomorrow14:54
TheJuliaHave a good evening14:54
TheJuliaiurygregory: I just tagged you on two wallaby releases, fwiw15:02
iurygregoryTheJulia, I just saw the email =) looking now15:03
jamesdentonHi TheJulia - I hear you might have some skills with OVN+Ironic+DHCP Agent15:05
TheJuliajamesdenton: rutro15:05
TheJuliajamesdenton: whats up?15:05
jamesdenton:)15:05
jamesdentonWell, I had a working LXB+Ironic deployment (OSA) and converted it to OVN. Works great. However, the Ironic piece is broken due to DHCP, and I'm trying to figure out how to properly integrate the legacy DHCP agent w/ OVN. I've looked a few templates on the Kolla and TripleO side and i don't think i'm missing anything, but maybe i am15:06
jamesdentonDHCP agent is creating the namespaces, and the tap interfaces are being added to br-int but the local vlan tag is 4095 and binding_failed on those. 15:07
jamesdentonNot sure what to expect15:07
TheJuliaso, AIUI, it is just inclusion of the agent on tripleo and it should work15:07
TheJuliaparas333_: hjensas: Can one of you confirm ^^^ ?15:08
jamesdentoni assume the ovn-controller service needs to be running, too, on the same node as the dhcp agent?15:09
TheJuliajamesdenton: dunno15:09
jamesdentonfair15:10
TheJuliaisent vlan tag 4095 a reserved vlan?15:12
paras333_jamesdenton: this is the template we have used https://github.com/openstack/tripleo-heat-templates/blob/698aabe28053b209f94964a34969a24a417047c2/environments/services/neutron-ovn-dvr-ha.yaml#L14 with ovn15:13
jamesdentonit's like.. the error vlan15:13
TheJuliayeah, sometimes used to wire through something as a vlan trunk in physical networking15:13
paras333_and we have removed this L14 if we are using BMAAS15:13
TheJuliamaybe line 14 needs a comment pointing out it is required for BMaaS15:14
jamesdentonyou get it here? https://github.com/openstack/tripleo-heat-templates/blob/698aabe28053b209f94964a34969a24a417047c2/environments/services/ironic-overcloud.yaml15:15
TheJuliatemplate processing order perhaps?15:18
jamesdentonWell, my deployment isn't TripleO-based, but I'm trying to compare to see what i'm missing. This might be an issue (locally) with migrated networks vs new networks. I'll keep messing with it. Thank you!15:26
TheJuliajamesdenton: let us know what you find15:27
jamesdentonwill do15:30
iurygregoryJayF, Berlin - Germany =)15:43
iurygregoryJune 7-9 15:43
JayFiurygregory: Nice! It is really sad I don't have reason to go -- my MIL is German15:59
iurygregory=(16:03
rpittaunever been to Berlin :)16:04
rpittaubye, good night! o/16:11
TheJuliaBerlin is an awesome city16:14
arne_wiebalck... to a level where you get into arguments about what to see/visit when time is limited :)16:21
arne_wiebalckbye everyone o/16:21
TheJuliaI will likely try to stay an extra week or longer. :)16:21
opendevreviewMerged openstack/ironic-python-agent master: Software RAID: Call udev_settle before creation  https://review.opendev.org/c/openstack/ironic-python-agent/+/81247016:22
arne_wiebalckgood idea ... time will still be limited, so be prepared!16:22
arne_wiebalckthis ^^ patch is a backport candidate I think16:23
* TheJulia dreams of the tasty pastries16:24
opendevreviewJulia Kreger proposed openstack/ironic-python-agent stable/xena: Software RAID: Call udev_settle before creation  https://review.opendev.org/c/openstack/ironic-python-agent/+/81269616:24
opendevreviewJulia Kreger proposed openstack/ironic-python-agent stable/wallaby: Software RAID: Call udev_settle before creation  https://review.opendev.org/c/openstack/ironic-python-agent/+/81269716:24
TheJuliaarne_wiebalck: down to wallaby reasonable?16:24
arne_wiebalckTheJulia: yes16:25
arne_wiebalckTheJulia: this will be our next release at least16:25
arne_wiebalckTheJulia: https://en.wikipedia.org/wiki/Berliner_(doughnut)16:25
TheJuliammmmmm16:25
TheJuliadoughnuts16:25
arne_wiebalckheh16:26
arne_wiebalckI guess I should not +2 my own patch.16:27
arne_wiebalckStrictly speaking, the relevant line is from jcosmao :-D16:27
TheJuliaeh, I think that is fine in this case16:35
iurygregoryI never been to Berlin also (and is just 8hrs by train) LOL, but it will take more than 8hrs for me because the Summit is only next year =( 16:46
TheJuliamore than 8 hours?!?16:47
TheJuliamoving?16:48
iurygregoryyup :D16:48
iurygregoryI won't be in CZ next year =)16:48
TheJuliaGoing home?16:48
iurygregoryyup =)16:48
TheJuliaawesome16:50
iurygregoryyeah :D 16:50
iurygregoryI will be remote 16:50
TheJuliaso... will you be taking czech beer recipes with you? :)17:03
iurygregorymaybe :D17:05
hjensasjamesdenton: re OVN and DHCP. You need the "legacy" DHCP agent running. Reason afik is that the distributed DHCP implementation in OVN does'nt wire up so that it listens on an external interface, so it would only respond to requests from VM's on a hypervisor. https://docs.openstack.org/neutron/latest/admin/archives/config-agents.html#configure-dhcp-agent17:09
hjensasjamesdenton: In TripleO we enable the Neutron DHCP Agent service when Ironic is enabled - https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/environments/services/ironic-overcloud.yaml#L1117:11
TheJuliaiurygregory: I think I've got https://github.com/OpenStackweb/ironic-website/pull/43 sorted17:13
TheJuliaiurygregory: thanks for finding the formatting issues. The having to escape a braket character is *fun*17:14
iurygregoryTheJulia, \o/ yay, yeah it was funny to debug using HTML comments XD17:15
iurygregoryI would add a <br> before "Please remember to update your agent, or you will be missing the following features: " so the text would start close to the second list.. 17:24
iurygregoryat least locally the page shows "Please remember to update your agent, or you will be missing the following features: " right after the item about deprecation warnings for Secure RBAC17:25
TheJuliaiurygregory: pushed17:29
iurygregoryTheJulia, you still need to have the empty line after the end of the list otherwise everything will look like a huge list 17:35
TheJuliaiurygregory: which list?17:36
iurygregoryL36 would be empty L37 the <br> L38 the "Please remember to update your agent, or you will be missing the following features: "17:36
TheJuliaOh!17:37
TheJuliaugh17:37
iurygregoryTheJulia, the first one about the features17:37
* TheJulia tweeks it and pushes again17:38
iurygregoryhttps://paste.opendev.org/show/809830/17:38
iurygregoryit looks like this at the moment basically ^17:38
TheJuliayeah17:38
TheJuliayou wouldn't think it would render that way based on everything17:38
* TheJulia waits for the new render to appear17:38
iurygregoryyup =(17:39
iurygregoryI need to thank my friend in Brazil that works at vexxhost , he helped me to get things working locally in my laptop so I could test :D17:40
TheJuliaI've got the builder working someplace....17:41
TheJuliaI think one of my VMs17:41
TheJuliaokay, maybe we need two br tags17:47
TheJuliabingo17:49
iurygregoryTheJulia, lgtm =D https://deploy-preview-43--ironicbaremetal.netlify.app/blog/18-2-release/17:54
TheJuliaAnd should be live shortly17:58
iurygregory\o/17:58
TheJulianow, next blog thing to work on17:58
jamesdentonhjensas Thank you for the info. I configured my environment correctly, but it seems since this was a migration (LXB->OVN) there was a little bit of cruft that needed to be handled. Namely, disabling/re-enabling DHCP on the subnet *and* restarting the nodes. That seems to have done the trick so far.18:00
iurygregoryI just noticed we still have 18.1 in the main page 18:07
iurygregoryoh ok cache problem :D18:07
TheJuliacache... the greatest problem.18:15
opendevreviewAde Lee proposed openstack/ironic master: Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773918:34
ade_leeTheJulia, fips changes as discussed ^^18:35
TheJuliaade_lee: awesome, I'll take a look. I did just ask in #openstack-qa to see if we could get someone to review the devstack fix steve pushed up18:38
ade_leeTheJulia, great - do we need a similar patch to ironic-inspector?18:38
TheJuliaade_lee: yeah, and metalsmith18:39
ade_leeTheJulia, ok - I'll put up similar over the next day or so18:42
TheJuliaade_lee: awesome18:42
opendevreviewManuel Schönlaub proposed openstack/sushy master: Add revision for drive resource  https://review.opendev.org/c/openstack/sushy/+/81267619:13
opendevreviewChris Krelle proposed openstack/ironic master: add snmp power action delay  https://review.opendev.org/c/openstack/ironic/+/80990219:18
TheJuliaNobodyCam: quick comment on the change, instead of assert_has_calls 19:31
NobodyCamLooks19:36
stevebakermorning20:28
opendevreviewChris Krelle proposed openstack/ironic master: add snmp power action delay  https://review.opendev.org/c/openstack/ironic/+/80990220:54
opendevreviewVerification of a change to openstack/ironic master failed: Demote three warning messages  https://review.opendev.org/c/openstack/ironic/+/81264321:02
opendevreviewChris Krelle proposed openstack/ironic master: add snmp power action delay  https://review.opendev.org/c/openstack/ironic/+/80990221:11
opendevreviewChris Krelle proposed openstack/ironic master: add snmp power action delay  https://review.opendev.org/c/openstack/ironic/+/80990221:41
opendevreviewMerged openstack/ironic-python-agent stable/xena: Software RAID: Call udev_settle before creation  https://review.opendev.org/c/openstack/ironic-python-agent/+/81269621:42
NobodyCamThank you to all for the reviews!21:49
NobodyCamhey hey stevebaker good morning21:49
stevebakerNobodyCam: hey hey!21:49
NobodyCam:)21:49
opendevreviewMerged openstack/ironic-python-agent stable/wallaby: Software RAID: Call udev_settle before creation  https://review.opendev.org/c/openstack/ironic-python-agent/+/81269721:50
TheJuliaNobodyCam: no problem22:00
NobodyCam:)22:00
opendevreviewJulia Kreger proposed openstack/ironic master: add snmp power action delay  https://review.opendev.org/c/openstack/ironic/+/80990222:02
TheJuliaNobodyCam: slightly different ^^22:03
TheJuliaso in theory, we could backport it if it was framed as a bugfix... just saying ;)22:03
NobodyCamhehehe22:04
NobodyCamI even talked to myself about the SNMP removal but managed to talk my self out of it. ;p22:04
NobodyCamThank you Julia!22:07
TheJuliaNobodyCam: if you look at the rendered output from the reno generation, you'll see why we do it that way22:08
NobodyCamAhh 22:10
NobodyCamyou people use things other then the command line to review documentation... that's crazy talk22:11
NobodyCamhehehe22:12
NobodyCamJ/K22:12
TheJuliaNobodyCam: I don't, I think dmitry does22:23
TheJuliaI've tried it but also hit bugs in it so I was kind of like... webui works for me22:23
TheJuliapersonally, i miss the old gerrit look which is honestly kind of enticing to me22:23
NobodyCam++++22:24
NobodyCambefore the opendev days22:24
TheJuliathat wasn't what caused it22:29
TheJuliaopenstack carried a really old customized gerrit ui for a long time22:29
NobodyCamahhh22:51
jandersgood morning Ironic o/23:21

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