Monday, 2021-03-22

stevebakerhey00:08
*** rcernin has joined #openstack-ironic00:14
*** rcernin has quit IRC00:22
*** rcernin has joined #openstack-ironic00:22
*** tosky has quit IRC00:26
*** stevebaker has quit IRC02:15
*** JayF has quit IRC02:23
*** JayF has joined #openstack-ironic02:39
*** ricolin has joined #openstack-ironic03:08
*** rcernin has quit IRC03:22
*** mkrai has joined #openstack-ironic03:24
*** uzumaki has joined #openstack-ironic03:55
*** rcernin has joined #openstack-ironic04:02
*** rcernin has quit IRC04:07
*** mkrai has quit IRC04:10
*** mkrai has joined #openstack-ironic04:15
*** rcernin has joined #openstack-ironic04:48
*** rcernin has quit IRC04:48
*** rcernin has joined #openstack-ironic05:26
*** ricolin_ has joined #openstack-ironic05:52
*** anuradha1904 has joined #openstack-ironic06:02
arne_wiebalckGood morning, ironic!06:52
jandersgood morning arne_wiebalck o/06:58
arne_wiebalckhey  janders o/06:59
*** mkrai has quit IRC07:39
*** mkrai_ has joined #openstack-ironic07:39
*** ociuhandu has joined #openstack-ironic07:46
*** ociuhandu has quit IRC07:52
*** mkrai_ has quit IRC07:58
*** rpittau|afk is now known as rpittau08:19
rpittaugood morning ironic! o/08:19
rpittauif anyone has a moment I think we're good to go with this https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/78105208:22
*** rcernin has quit IRC08:23
*** uzumaki has quit IRC08:27
*** mkrai has joined #openstack-ironic08:28
*** ociuhandu has joined #openstack-ironic08:33
*** ociuhandu has quit IRC08:38
openstackgerritRiccardo Pittau proposed openstack/sushy stable/ussuri: Fix ExtendedInfo error handling for non-list item  https://review.opendev.org/c/openstack/sushy/+/78202908:42
*** tosky has joined #openstack-ironic08:49
*** ociuhandu has joined #openstack-ironic08:56
*** lucasagomes has joined #openstack-ironic08:56
*** rcernin has joined #openstack-ironic08:58
*** ociuhandu has quit IRC09:18
*** mkrai has quit IRC09:22
*** mkrai_ has joined #openstack-ironic09:22
*** derekh has joined #openstack-ironic09:27
*** moshiur has joined #openstack-ironic09:30
moshiurGood morning. rpittau: can we please move forward and merge this PR?: https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/77872609:32
rpittaumoshiur: good morning, as far I would like to since that is approved, we can't because the patch it depends on is still not merged09:34
moshiurHi TheJulia: is there anything that is blocking us to get merge the following PR?: https://review.opendev.org/c/openstack/diskimage-builder/+/77872309:35
moshiurrpittau: Thanks, I asked TheJulia.09:36
rpittaumoshiur: we don't  control the diskimage-builder project, you should ask in the #openstack-infra or in the #opendev channels09:36
*** rcernin has quit IRC09:37
moshiurok09:37
moshiurrpittau: I asked for review in #openstack-infra. lets see..09:43
*** ociuhandu has joined #openstack-ironic09:44
*** ociuhandu has quit IRC09:46
*** ociuhandu_ has joined #openstack-ironic09:46
arne_wiebalckrpittau: I am having trouble getting devstack going (with Ironic), tried with bionic and c8 so far ... any recommendations on what has some chances to succeed?09:51
*** viks____ has joined #openstack-ironic09:56
*** dtantsur|afk is now known as dtantsur10:06
dtantsurmorning ironic10:06
dtantsurmoshiur: actually, there is #openstack-dib specifically for diskimage-builder10:06
*** mkrai_ has quit IRC10:13
*** ricolin_ has quit IRC10:16
*** ociuhandu_ has quit IRC10:16
*** mkrai has joined #openstack-ironic10:16
*** ociuhandu has joined #openstack-ironic10:34
*** ociuhandu has quit IRC10:36
*** ociuhandu has joined #openstack-ironic10:37
rpittaud'oh! how did I forget that?10:37
*** ociuhandu has quit IRC10:37
*** ociuhandu has joined #openstack-ironic10:38
*** ociuhandu has quit IRC10:38
*** ociuhandu has joined #openstack-ironic10:38
*** ociuhandu has quit IRC10:38
rpittauarne_wiebalck: What kind of trouble? To be honest it's a while I don't run devstack on centos8, but at least with ussuri I don't remember any issue, bionic should work the same, but I suggest to run it on focal if possible, especially if victoria or wallaby.10:39
arne_wiebalckI'll try focal now10:40
arne_wiebalckthis kind of issue: https://stackoverflow.com/questions/63439723/error-in-installation-of-openstack-in-ubuntu10:40
*** ociuhandu has joined #openstack-ironic10:40
arne_wiebalck(on centos8)10:41
arne_wiebalckon bionic, there was a neutron issue when running stack.sh10:41
arne_wiebalckthis is all with the local.conf we have in our docs10:41
arne_wiebalckour == upstream ironic10:41
arne_wiebalckrpittau: and, this is master :)10:42
*** ociuhandu has quit IRC10:43
*** ociuhandu has joined #openstack-ironic10:44
rpittauarne_wiebalck: for master definitely with focal, was that centos8 stream?10:44
arne_wiebalckrpittau: no, centos10:44
arne_wiebalckrpittau: ok, let me try with focal ...10:44
rpittaummm I may try with centos8 stream, some packages may not have been updated with older centos10:45
arne_wiebalckthe devstack upstream docs recommend bionic, I think, that's why I started there10:49
rpittauarne_wiebalck: is that devstack docs or ironic docs ?10:49
arne_wiebalckrpittau: devstack10:49
* arne_wiebalck hopes he was not on some outdated page now ...10:49
arne_wiebalckhttps://docs.openstack.org/devstack/latest/ recommends bionic10:51
rpittauarne_wiebalck: no, it's actually the latest docs that suggest bionic, which is... weird10:51
rpittauwe stopped testing on bionic in ussuri AFAIR, from victoria the official testing OS is focal10:52
arne_wiebalckthanks!10:57
*** mkrai has quit IRC11:00
moshiurThanks dtantsur: I ask for review in #openstack-dib11:04
openstackgerritMerged openstack/ironic master: Add anaconda configuration and template  https://review.opendev.org/c/openstack/ironic/+/77216611:05
*** rcernin has joined #openstack-ironic11:13
*** ociuhandu has quit IRC11:18
*** rcernin has quit IRC11:18
*** uzumaki has joined #openstack-ironic11:21
openstackgerritDhuldev Valekar proposed openstack/ironic master: DRAC: Added redfish management clean steps  https://review.opendev.org/c/openstack/ironic/+/72159311:27
*** ociuhandu has joined #openstack-ironic11:28
*** ociuhandu has quit IRC11:30
*** ociuhandu has joined #openstack-ironic11:31
openstackgerritsonali bhausaheb borkar proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port  https://review.opendev.org/c/openstack/ironic/+/69332811:35
*** rcernin has joined #openstack-ironic11:44
*** monica_pardhi has joined #openstack-ironic11:45
*** abhishekyadav has joined #openstack-ironic11:47
abhishekyadav@mon11:47
iurygregorygood morning Ironic!11:52
*** rh-jelabarre has joined #openstack-ironic11:55
*** monica_pardhi has quit IRC11:55
janders good morning / afternoon rpittau dtantsur iurygregory o/11:56
*** mkrai has joined #openstack-ironic11:56
iurygregoryjanders, o/11:57
openstackgerritJacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites  https://review.opendev.org/c/openstack/ironic/+/74411711:57
iurygregoryjanders, dtantsur  hey regarding the way of setting MultiOpt, if oslo supports both ways we will be able to validate we don't support? .-.12:07
*** ociuhandu has quit IRC12:09
iurygregoryin cinder when they are using multiopt with dict it's possible to configure things that way https://docs.openstack.org/cinder/pike/contributor/replication.html#service-configuration12:09
openstackgerritDhuldev Valekar proposed x/sushy-oem-idrac master: Update management clean steps  https://review.opendev.org/c/x/sushy-oem-idrac/+/78221912:09
openstackgerritMerged openstack/ironic stable/ussuri: Enable swift temporary URLs in grenade and provide a good error message  https://review.opendev.org/c/openstack/ironic/+/77743012:29
*** rcernin has quit IRC12:31
*** rcernin has joined #openstack-ironic12:31
*** ociuhandu has joined #openstack-ironic12:39
*** jamesdenton has quit IRC12:48
*** jamesdenton has joined #openstack-ironic12:49
dtantsuriurygregory, janders, mmmmm... I'm not sure we should artificially restrict this feature, maybe just don't document it?12:56
dtantsurI mean, don't document setting multiple values per line (and use singular for the option name)?12:57
openstackgerritMerged openstack/ironic-python-agent stable/victoria: Fix root UUID for streamed partition images  https://review.opendev.org/c/openstack/ironic-python-agent/+/78181413:01
openstackgerritMerged openstack/ironic stable/train: DRAC: Fix a failure to create virtual disk  https://review.opendev.org/c/openstack/ironic/+/76971913:01
*** ociuhandu has quit IRC13:02
iurygregorydtantsur, I'm ok with it =)13:04
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Make baremetal --debug actually work  https://review.opendev.org/c/openstack/python-ironicclient/+/78110013:05
iurygregoryhabemus Backport Candidate label =D13:07
dtantsur\o/13:07
*** ociuhandu has joined #openstack-ironic13:08
* dtantsur uses13:08
iurygregorynow I will update my script to usse gerrit api to see if I can do some magic with it :D13:09
iurygregoryafter fixing some unit tests13:09
TheJuliagood morning13:11
*** rloo has joined #openstack-ironic13:11
dtantsurmorning TheJulia13:11
iurygregorygood morning TheJulia13:11
arne_wiebalckrpittau: With focal, I hit the issue mentioned in the stackoverflow link mentioned above (and the fix mentioned there seems to indeed fix this).13:12
arne_wiebalckrpittau: The local.conf from the Ironic docs did not manage to bring up devstack, the local.conf from the samples dir seems to work ok.13:12
TheJuliaarne_wiebalck: that is not terribly srprising :(13:14
arne_wiebalckTheJulia: the sample local.conf is pretty minimal I realise now13:14
arne_wiebalckTheJulia: why not?13:15
openstackgerritMerged openstack/ironic master: Use OOB inspection to fetch MACs for IB inspection  https://review.opendev.org/c/openstack/ironic/+/77439113:16
TheJuliaWhat is in the docs is kind of old and easy for people to miss13:16
jandersdtantsur iurygregory not restricting and not documenting sounds reasonable13:18
janderssee you tomorrow Ironic o/13:18
iurygregorybye janders o/13:18
jandersiurygregory dtantsur thanks heaps for your help with https://review.opendev.org/c/openstack/ironic/+/744117 today, much appreciated! :)13:18
janderscheers13:18
iurygregoryjanders, yw =)13:19
arne_wiebalckTheJulia: Do we have a newer version of how devstack for Ironic should be configured somewhere?13:20
iurygregoryarne_wiebalck, you can take the local.conf from the jobs :D13:20
iurygregoryit's not a minimal example, but it would help I would say13:21
arne_wiebalckiurygregory: Thanks! Could you point me to one you would deem a sensible starting point?13:22
rpittauarne_wiebalck: the example in the docs definitely needs updating13:23
arne_wiebalckrpittau: :-D13:23
iurygregoryarne_wiebalck, sure!13:23
arne_wiebalckrpittau: happy to do that once I got one running ;)13:23
*** tzumainn has joined #openstack-ironic13:25
rpittauI'm still at lunch, end of meal coffee takes time in Italy :)13:26
rpittauI'll be happy to review that, I'll check the jobs values later13:26
*** abhishekyadav has quit IRC13:27
openstackgerritDhuldev Valekar proposed x/sushy-oem-idrac master: Update management clean step  https://review.opendev.org/c/x/sushy-oem-idrac/+/78225413:32
TheJuliaarne_wiebalck: no, But13:34
TheJuliaarne_wiebalck: but I thinkI started heading into updating those docs and got stuck in a loop :)13:34
arne_wiebalckTheJulia: iurygregory made some suggestions, I'll see if I can get things going. If yes, we can circle back to the docs.13:35
iurygregorythe only thing is that it will be tied to a job definition hehe13:35
rpittauwell, you can make it generic13:39
iurygregoryyeah, after removing some variables and checking if it would work :D13:41
rpittaueheh yeah13:42
TheJuliaOne thing I noticed, etcd3 needs to be disabled13:43
TheJuliaIt also likely needs to be removed or fixed with devstack, but...yeah13:43
*** ociuhandu has quit IRC13:44
TheJuliaReview Jam in 15 minutes13:44
*** ociuhandu has joined #openstack-ironic13:46
*** ociuhandu has quit IRC13:46
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Refactor: use mounted from ironic-lib  https://review.opendev.org/c/openstack/ironic-python-agent/+/78225913:47
openstackgerritDhuldev Valekar proposed x/sushy-oem-idrac master: Update the resources to put idrac in known good state  https://review.opendev.org/c/x/sushy-oem-idrac/+/78225413:47
*** ociuhandu has joined #openstack-ironic13:49
*** ociuhandu has quit IRC13:50
*** ociuhandu has joined #openstack-ironic13:50
*** ociuhandu has quit IRC13:55
*** ociuhandu has joined #openstack-ironic13:58
*** ociuhandu has quit IRC14:00
*** ociuhandu has joined #openstack-ironic14:01
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Hardware manager call to load configuration from vmedia  https://review.opendev.org/c/openstack/ironic-python-agent/+/78226214:01
*** jhesketh has quit IRC14:02
*** jhesketh has joined #openstack-ironic14:04
*** rcernin has quit IRC14:05
*** ociuhandu_ has joined #openstack-ironic14:21
*** ociuhandu has quit IRC14:22
*** ociuhandu has joined #openstack-ironic14:23
*** lmcgann has joined #openstack-ironic14:24
*** ociuhand_ has joined #openstack-ironic14:24
*** ociuhandu has quit IRC14:24
*** ociuhandu has joined #openstack-ironic14:25
*** ociuhandu_ has quit IRC14:26
*** ociuhand_ has quit IRC14:28
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Support pre-built deploy/rescue ISO in Redfish  https://review.opendev.org/c/openstack/ironic/+/78226614:32
*** ociuhandu_ has joined #openstack-ironic14:33
*** ociuhandu has quit IRC14:34
*** ociuhand_ has joined #openstack-ironic14:34
*** lbragstad__ is now known as lbragstad14:36
*** ociuhandu_ has quit IRC14:38
*** kaifeng has joined #openstack-ironic14:48
*** dansmith has quit IRC14:53
rpittauajya, rpioso, are you going to cut a stable/wallaby for sushy-oem-idrac ? besides the new 2.0.0 version I mean14:54
*** frigo has joined #openstack-ironic14:54
TheJuliazer0c00l: Hey, when you around, I have a question w/r/t https://review.opendev.org/c/openstack/ironic/+/77589114:55
*** stendulker has joined #openstack-ironic14:56
*** Qianbiao has joined #openstack-ironic14:56
*** dansmith has joined #openstack-ironic14:57
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Mar 22 15:00:14 2021 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
TheJuliao/15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
iurygregoryo/15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
bfournieo/15:00
bdoddo/15:00
dtantsuro/15:00
arne_wiebalcko/15:00
ajyao/15:00
rpittauo/15:00
erbarro/15:00
rpioso\o15:00
TheJuliaOur agenda for this week can be found on the wiki.15:00
rlooo/15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
TheJulia#topic Announcements / Reminders15:01
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:01
Qianbiaoo/15:01
kaifengo/15:01
stendulkero/15:01
TheJulia#info This week is week R-3 in the Wallaby release schedule. Most projects will be cutting their RC1 and stable branches this week.15:01
*** frigo has quit IRC15:02
TheJulia#info Due to other projects releasing, we may encounter devstack changes which will may force us to release early. Currently we're hoping to release next week.15:02
TheJulia#info PTG has been scheduled and a number of topics for discussion are on the etherpad.15:03
TheJulia#link https://april2021-ptg.eventbrite.com/15:03
TheJulia#link https://etherpad.opendev.org/p/ironic-xena-ptg15:03
TheJuliaDoes anyone have anything to announce or remind us of this week?15:03
iurygregorywe have the Backport-Candidate label \o/15:04
iurygregoryit will probably help us track things we want to be sure to backport =)15:04
iurygregoryonly cores can set the label, so when reviewing a patch if you think we should backport add the +1 =)15:05
TheJulia#info We now have a backport-candidate label \o/15:05
*** frigo has joined #openstack-ironic15:05
*** moshiur has quit IRC15:05
TheJuliaiurygregory: Thanks for taking that on and getting the patches through Infra15:05
iurygregorynp o/15:05
TheJuliaLooks like we had no action items from last week.15:06
TheJuliaDoes anyone have anything else to announce?15:07
TheJuliaFWIW, I *may* try to take the week of R-1 or R-0 off, but have not solidified any plans as of yet.15:07
TheJuliaI guess the only other thing to mention is I believe daylight savings time in Europe is next week?15:08
rpittauyep15:09
TheJulia:(15:09
JayFAre there any outstanding patches that need review/merge targetted for the Wallaby release?15:09
rpittaumarch 28, end of this week15:09
TheJuliaJayF: I believe there are, but I think we'll get to that during our next segment15:09
TheJuliaWell, if there is nothing else to announce or remind ourselves of, then I believe we can proceed15:10
TheJulia#topic Review subteam status reports15:10
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:10
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:11
TheJuliaStarting around line 26015:11
TheJuliadtantsur: do you have anything w/r/t dhcp-less to merge this cycle?15:13
dtantsurTheJulia: I'm concentrated on glean at this moment15:14
TheJuliadtantsur: ack15:14
TheJuliarpittau: For the anaconda stuff since it seems to be merging, maybe we add the ironic-week-prio hashtag if not already present to everything except the very last patch in that chain?15:15
rpittauTheJulia: sounds good15:15
rpittaudone15:17
TheJuliaajya: thank you for noting the status of the config molds item15:17
ajyaTheJulia: yw15:17
TheJuliaOnward to priorities for the week?15:18
rpittaulet's15:18
TheJulia#topic Deciding on priorities for the coming week15:19
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:19
TheJuliaThe main question I want to ask during this is what is needed to release, and make sure we get our eyes on those items during the next few days15:20
TheJuliaAre there any new items that need to be considered?15:20
ajyathe ones from review jam15:20
TheJuliaI believe they already have the hashtag, but will check in a minute15:20
rpiosoTheJulia: We have a couple of oldie, but goodie changes, too: https://review.opendev.org/c/openstack/ironic/+/693328 and https://review.opendev.org/c/openstack/ironic/+/72159315:21
rpittauwell we need ironic, ironic-inspector, ironic-python-agent, and bifrost at least15:22
iurygregoryhttps://review.opendev.org/c/openstack/ironic/+/721593 - merge conflict15:22
iurygregoryand missing the hashtag ironic-week-prio (it will help to get visibility)15:22
ajyahttps://review.opendev.org/c/openstack/ironic/+/769928 https://review.opendev.org/c/openstack/ironic/+/759428 - these don't have hashtag15:23
ajyacan I add it myself?15:23
rpiosoiurygregory: May I add that tag?15:23
TheJuliaTags added15:24
iurygregoryajya, in the ones you are the owner/Uploader you can15:24
iurygregoryrpioso, ^ same15:24
iurygregoryif you think the patch is in good shape and ready for review =)15:24
ajyaok, I'll add updates as discussed in the review jam and then hashtag15:25
TheJuliaI was about to propose a bifrost change and loaded it and see it was already approved in the time between the list view and me loading the patch15:25
iurygregory*magic*15:25
rpittauno priority for release but we need this for the exporter https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/78192715:26
TheJuliaDoes https://review.opendev.org/c/openstack/ironic-python-agent/+/782259 make sense for priorites for this week?15:26
rpiosoTheJulia: Thank you for adding the tags :)15:26
JayFTheJulia: I'll review (and likely land) that right now-ish so may not be worth the paperwork :)15:26
iurygregoryrpittau, +W =) quick fix15:26
TheJuliaJayF: ack15:26
TheJulianothing else on IPA looks really pressing at the moment15:27
TheJuliaunless someone has something that is hiding from the list of last 50 patches15:27
rpittauthanks iurygregory15:27
TheJuliaI think https://review.opendev.org/c/openstack/ironic-inspector/+/770708 might make sense, but since they haven't done the release note can I get a volunteer?15:28
*** mkrai has quit IRC15:28
iurygregoryTheJulia, I can15:28
TheJuliaiurygregory: thanks15:29
iurygregorywill push an update today15:29
rpittaua quick one more https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/78105215:30
TheJuliaThat one was easy :)15:31
rpittau:)15:31
TheJuliaI found an important patch15:33
TheJuliahttps://review.opendev.org/c/openstack/ironic/+/780974\15:33
rpittauoh wow, that is indeed15:33
TheJuliaand https://review.opendev.org/c/openstack/ironic/+/78097315:33
rpittaucouple of comments to sort out15:34
TheJuliaWell, https://review.opendev.org/q/hashtag:%2522ironic-week-prio%2522+(status:open) looks pretty good :)15:35
TheJuliahttps://review.opendev.org/c/openstack/ironic/+/767432 <-- really needs to make 17 or I need to do more secure-rbac policy patches15:36
rpittauoh yeah, the depends on patch has merged, good15:37
TheJuliaWell, I think we're done with priorities and can proceed onward!15:38
dtantsuryep15:38
TheJuliaohh, we'll need to update the release mapping15:38
TheJuliaI can do that this week15:38
dtantsurand fix API versions list15:38
openstackgerritAija Jauntēva proposed openstack/ironic master: Add configuration mold storage  https://review.opendev.org/c/openstack/ironic/+/76992815:38
*** frigo has quit IRC15:38
TheJuliadtantsur: yup15:38
TheJuliaarne_wiebalck: anything for the Baremetal SIG this week?15:39
TheJuliaLooks like arne_wiebalck is away from his keyboard at the moment15:40
TheJuliaI guess we can proceed to Open Discussion15:40
TheJulia#topic Open Discussion15:40
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:40
TheJuliaDoes anyone have anything they would like to bring up this week?15:40
arne_wiebalckBare Metal SIG: NTR I think :)15:41
rpittaujust one thing, please keep an eye on the tinyipa jobs as they may require RAM increase for the recent tinycore 12.x update15:41
TheJulia#info please keep an eye on the tinyipa jobs as they may require RAM increase for the recent tinycore 12.x update15:41
rpittauthanks :)15:41
TheJuliaAnything else for this week?15:44
TheJuliaOr have the crickets arrived?15:44
iurygregoryprobably15:45
TheJuliaWell, Thanks everyone, and have a wonderful week!15:46
rpittauthanks!15:46
dtantsuro/15:46
Qianbiaothanks, have a good week15:46
arne_wiebalckthanks TheJulia15:46
iurygregory\o/15:46
TheJuliaFinal reminder, Add PTG topics!15:46
TheJulia:)15:47
TheJulia#endmeeting15:47
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list."15:47
openstackMeeting ended Mon Mar 22 15:47:04 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:47
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-22-15.00.html15:47
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-22-15.00.txt15:47
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-03-22-15.00.log.html15:47
*** stendulker has quit IRC15:48
rlooTheJulia: wrt https://review.opendev.org/c/openstack/ironic/+/767432. Thinking you want it addressed sooner rather than later. I commented...15:52
TheJuliarloo: its been deprecated for like ?5? years15:53
rlooTheJulia: did we deprecate it or say that it would still work for < 1.28 version? That's what I don't recall.15:54
rlooto be honest, now that i think about it, what's the diff wrt 'deprecate' and using microversions. think i need more caffeine...15:55
TheJuliadeprecate it... I think you led the last charge to rip it out :)15:55
* rloo looks in release notes...15:55
TheJuliaI don't think we ever committed to keeping it working for the API version <1.2815:56
rlooyeah, i think i did too. but my comment in the code seems to indicate that i thought we still needed to support it if < 1.28.15:56
TheJuliaBut even if we did, we quite literally cannot15:56
TheJuliaWe cannot because it bypasses RBAC checking for what could be a privileged node action15:56
rlooi don't see how RBAC in a newer microversion, is expected to 'work properly' with an older microversion.15:57
TheJuliaRBAC is not microversioned behavior, and is operator overridable via custom policy files.15:58
rlooi thought we up15:58
rlooed the microversino when we added rbac?15:58
TheJuliaAs a signal to indicate "capability should be present"15:58
TheJuliaThat will allow us to customize tempest tests15:58
*** Qianbiao has quit IRC15:59
TheJuliaBecause in the future, some of the tests will no longer be valid depending on the exact operating mode they are connecting with15:59
TheJuliaor need to use specific credentials15:59
JayFIt's worth calling attention to RBAC/policy having a separate deprecation path than API16:00
JayFthe old rules are in there, and will be for a deprecation period if enabled16:00
TheJuliaThe conundrum is using extra info is an end-run around it all.16:01
rlooi skimmed and don't see any mention of deprecating that in the release notes. going to do a more thorough search...16:03
TheJuliaso I have two paths... rip out the capability completely, or build rbac rules around it. Only two x/ namespace projects even still reference extra['vif_port_id']16:03
* iurygregory lunch time16:03
rlooam wondering if deleting this can wait another cycle. put a deprecation notice in wallaby.16:04
TheJuliacan't16:04
TheJuliaI either need to completely rip it out, or I need to put rbac rules in place16:04
rlooputting rbac rules in place is to say that ironic has 'rbac' completed?16:04
TheJuliahttps://docs.openstack.org/releasenotes/ironic/ocata.html#deprecation-notes16:05
rloo++ thx for finding that.16:05
rloook, i think we're good then.16:05
TheJuliarloo: Well, capability to enforce access controls for non-privilged users16:06
openstackgerritAija Jauntēva proposed openstack/ironic master: Add import, export configuration to idrac-redfish  https://review.opendev.org/c/openstack/ironic/+/75942816:06
TheJuliaowner/lessee provided some of that framework/capability, but rbac just allowed us to basically take it to the next level16:06
TheJuliajust... the glaring security issue in all of it is that field :)16:06
ajyarpittau: re sushy-oem-idrac wallaby branch - yes, there will be (cc rpioso)16:06
rpittauajya: thanks16:06
openstackgerritMerged openstack/ironic-prometheus-exporter master: Increase RAM for ironic vm  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/78192716:09
openstackgerritVerification of a change to openstack/bifrost failed: Relax format of checksum file  https://review.opendev.org/c/openstack/bifrost/+/77857816:15
TheJuliaiurygregory: I commented w/r/t netboot on https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/74144516:16
TheJuliafwiw16:16
*** anuradha1904 has quit IRC16:22
rpiosorpittau: We are likely going to create a 2.1.0. There could be a 2.1.1, too. Those would both be done before Wallaby ironic is released. Is there any reason why we should not wait to cut stable/wallaby until the dust settles?16:27
rpittaurpioso: not really, it was mainly a curiosity, that is an independent project so it's totally up to you. If it was for me I would try to follow the non-client libs release schedule for consistency with sushy, but it's not mandatory in the case of sushy-oem-idrac, in the end it really all depends if you have more features to add or not.16:32
*** penick has joined #openstack-ironic16:34
*** penick has quit IRC16:35
*** penick has joined #openstack-ironic16:36
rpiosorpittau: Thank you for clarifying :)16:42
TheJuliaajya: I spotted a minor s/web/http/ on https://review.opendev.org/c/openstack/ironic/+/769928. I also added more verbose comments for other reviewers16:42
* TheJulia is now on cat attempt to capture the microphone # 6 for today.16:42
*** ociuhand_ has quit IRC16:42
*** ociuhandu has joined #openstack-ironic16:43
ajyaTheJulia: thanks, will update16:43
iurygregoryTheJulia, looking now16:46
*** ociuhandu_ has joined #openstack-ironic16:55
ajyaTheJulia: for that retry thing, are there re-usable conf values or should I create new ones?16:55
TheJuliaIt was just a thought. I would just add to the molds section of the configuration16:56
*** ociuhandu has quit IRC16:58
*** ociuhandu_ has quit IRC16:59
zer0c00lTheJulia: pong17:04
TheJuliaquestion for you, I noticed you were adding arguments in the middle of kwargs on the anaconda pxe template changes17:05
TheJuliachange17:05
zer0c00li can move that kwarg to the end.17:05
TheJuliaAt the end is preferred17:05
zer0c00li thought ramdisk_boot and anaconda_boot go together.17:05
zer0c00lbut if it can cause problems - yes i can move it17:05
TheJuliaSince things like tests also just do value checking in order17:05
TheJuliayeah, in case someone did something like create_ramdisk(1,2,3,4,5,6,7,9)17:06
TheJuliainstead of actually defining the kw to associate17:06
zer0c00lok.17:07
*** lucasagomes has quit IRC17:07
dtantsurgoing to get some fresh air while it's still allowed17:09
dtantsuro/17:09
*** dtantsur is now known as dtantsur|afk17:09
rpittaudtantsur|afk: enjoy! :)17:09
TheJuliaugh :(17:09
TheJuliaenjoy!17:09
TheJuliaI take it germany is going into another hard lockdown?17:09
rpittauI think I'm going to get some fresh pizza, for the same reason :P17:09
TheJulia:(17:09
rpittauTheJulia: unfortunately yes, until end of April if I understood correctly17:10
TheJuliaugh17:10
*** penick has quit IRC17:11
rpittaubye everyone! o/17:11
*** rpittau is now known as rpittau|afk17:11
openstackgerritAija Jauntēva proposed openstack/ironic master: Add configuration mold storage  https://review.opendev.org/c/openstack/ironic/+/76992817:16
openstackgerritMerged openstack/python-ironicclient master: Fix --fields network_data  https://review.opendev.org/c/openstack/python-ironicclient/+/78111217:19
openstackgerritJulia Kreger proposed openstack/ironic master: Revise release notes for 17.0 release  https://review.opendev.org/c/openstack/ironic/+/78097317:19
openstackgerritMerged openstack/ironic-python-agent-builder master: Add a job to test tinyipa in ussuri  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/78105217:22
*** jamesdenton has quit IRC17:24
*** jamesdenton has joined #openstack-ironic17:24
openstackgerritJulia Kreger proposed openstack/ironic master: Revise release notes for 17.0 release  https://review.opendev.org/c/openstack/ironic/+/78097317:39
openstackgerritJulia Kreger proposed openstack/ironic master: Add prelude for Ironic 17.0.  https://review.opendev.org/c/openstack/ironic/+/78097417:39
openstackgerritJulia Kreger proposed openstack/ironic master: Version 17.0 mapping/doc updates  https://review.opendev.org/c/openstack/ironic/+/78230617:39
openstackgerritMerged openstack/ironic-prometheus-exporter master: Add Python3 xena unit tests  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/78184617:41
openstackgerritVerification of a change to openstack/ironic failed: Fix broken configdrive_use_object_store  https://review.opendev.org/c/openstack/ironic/+/77734918:18
arne_wiebalckbye everyone o/18:25
*** kaifeng has quit IRC18:27
*** dtantsur has joined #openstack-ironic18:34
*** dtantsur|afk has quit IRC18:41
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Automaticaly set cipher suite  https://review.opendev.org/c/openstack/ironic/+/77052618:47
openstackgerritArun S A G proposed openstack/ironic master: Add anaconda support in the pxe boot driver  https://review.opendev.org/c/openstack/ironic/+/77589118:59
openstackgerritArun S A G proposed openstack/ironic master: Validate the kickstart template and file before use  https://review.opendev.org/c/openstack/ironic/+/77744818:59
openstackgerritArun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat  https://review.opendev.org/c/openstack/ironic/+/77805818:59
openstackgerritArun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface  https://review.opendev.org/c/openstack/ironic/+/78039818:59
openstackgerritJulia Kreger proposed openstack/ironic master: Remove extra/vif_port_id  https://review.opendev.org/c/openstack/ironic/+/76743219:04
JayFTheJulia: are we expecting review jam to move with DST, or remain the same UTC time?19:16
TheJuliawith UTC19:16
JayFack ty19:16
TheJuliait makes my life less complicated then :)19:17
JayFTues 6pm UTC counting  DST becomes ... 11-11:5019:17
JayFoh hey, I put it in the calendar at utc19:18
JayFgo me19:18
iurygregoryTheJulia, quick question, for https://review.opendev.org/c/openstack/ironic-inspector/+/770708 the release note can be something like http://paste.openstack.org/show/803792/ ?19:21
iurygregoryI'm wondering if it should be a feature or just upgrade19:21
TheJuliaiurygregory: i would say just upgrades is fine19:24
TheJuliaits not really a feature19:24
iurygregoryack I will push o/19:24
TheJuliaiurygregory: wait19:24
iurygregoryok19:24
TheJuliaiurygregory: alow the operator to use the `ibmc` driver for19:25
TheJulia    introspection. "allow" and it is more allowing introspection to try and match the BMC address if no ports are defined.19:25
iurygregory"allowing introspection to try and match the BMC address if no ports are defined when using the `ibmc` driver."19:28
*** Anticime1 is now known as Anticimex19:29
TheJuliaYup, I think that works19:30
* iurygregory waits for the delay...19:31
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Add ibmc_address to ipmi_address_fields  https://review.opendev.org/c/openstack/ironic-inspector/+/77070819:31
iurygregorynot bad, 14sec19:31
*** thiagop has joined #openstack-ironic19:38
*** uzumaki has quit IRC19:53
TheJuliaheh20:08
tzumainnTheJulia, I noticed on the ptg topics list, you mention the ceph iscsi driver and note that there's a question of whether it works - is "working" something beyond boot-from-volume?20:12
TheJuliatzumainn: it just merged recently, and yeah, in part the question is does it work, does it have the right info, do we need to do something slightly different20:13
tzumainnTheJulia, ah, I can say that it worked in testing at the MOC20:13
TheJuliaso literal start with ceph volume, tell things to attach and magic happens and just works?20:14
tzumainnif the volume is defined in cinder, yeah20:15
TheJulia\o/20:15
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Add ibmc_address to ipmi_address_fields  https://review.opendev.org/c/openstack/ironic-inspector/+/77070820:24
*** rcernin has joined #openstack-ironic20:25
*** adrianc has quit IRC20:31
*** adrianc has joined #openstack-ironic20:32
*** ociuhandu has joined #openstack-ironic20:37
*** rcernin has quit IRC20:38
*** rcernin has joined #openstack-ironic21:03
*** gyee has joined #openstack-ironic21:04
*** thiagop is now known as outbrito21:08
*** ociuhandu has quit IRC21:15
*** rcernin has quit IRC21:17
*** derekh has quit IRC21:32
*** bdodd has quit IRC21:35
*** lmcgann has quit IRC21:36
*** bdodd has joined #openstack-ironic21:38
*** stevebaker has joined #openstack-ironic21:42
*** bdodd has quit IRC21:48
*** rcernin has joined #openstack-ironic21:51
*** rcernin has quit IRC21:51
*** bdodd has joined #openstack-ironic21:52
*** rcernin has joined #openstack-ironic21:52
*** uzumaki has joined #openstack-ironic21:57
*** ociuhandu has joined #openstack-ironic22:13
*** ociuhandu has quit IRC22:14
*** hoonetorg has quit IRC22:37
*** rloo has quit IRC23:34

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