Monday, 2021-08-30

*** pmannidi is now known as pmannidi|AFK03:19
opendevreviewJacob Anders proposed openstack/ironic master: Add support for verify steps  https://review.opendev.org/c/openstack/ironic/+/80000105:53
arne_wiebalckGood morning janders and Ironic!06:03
jandersgood morning arne_wiebalck o/06:03
opendevreviewJacob Anders proposed openstack/ironic master: Add support for verify steps  https://review.opendev.org/c/openstack/ironic/+/80000106:06
iurygregorygood morning arne_wiebalck and Ironic o/06:28
jandershey iurygregory o/06:28
iurygregoryhey janders o/06:29
arne_wiebalckGood morning iurygregory o/06:32
opendevreviewJacob Anders proposed openstack/ironic master: Add support for verify steps  https://review.opendev.org/c/openstack/ironic/+/80000106:49
janders^ let's see what CI thinks of my tests :)06:50
jandersoff for an evening walk, back in a bit06:50
jandershow long did you guys stay off exercise after your covid vaccinations? I don't have much side effects any more (except for still feeling a little tired)06:52
iurygregoryprometheanfire, I haven't seen any issue with tox py38 for Ironic jobs related to the SQLAlchemy error that is reported in the patch you mentioned =(06:52
jandersjust being careful (it was pfizer, which is said to have super-rare cardio complications, that is the reason behind being careful)06:53
iurygregoryjanders, fyi astrazeneca has more cardio complications than pfizer =)06:53
arne_wiebalckjanders: I used this as an excuse for a week.06:54
iurygregoryarne_wiebalck, LOL06:54
arne_wiebalckiurygregory: :-D06:55
iurygregoryarne_wiebalck, I hope it was after the 2nd shot not after the 1st :D06:55
iurygregorythe nice thing I've discovered during the weekend is that the system here in CZ says I didn't get the vaccine :D06:56
iurygregoryso I don't have any certificate \o/06:56
arne_wiebalckiurygregory: any shot ofc 07:35
iurygregoryarne_wiebalck, hahaha best idea XD07:35
arne_wiebalckiurygregory: ai ... moving around I used my pass already a couple of times07:35
arne_wiebalckiurygregory: even to buy a sandwich and to sit down in the bakery I needed it07:36
iurygregoryyeah same here in CZ07:37
iurygregoryI need to figure out how to get the cert now hehe07:37
iurygregoryor I will need to keep doing antigen-test07:37
jandersthanks iurygregory arne_wiebalck07:41
jandersI may get back on my bike tomorrow07:41
opendevreviewTadeas Kot proposed openstack/python-ironicclient master: Add support for fields in drivers CLI  https://review.opendev.org/c/openstack/python-ironicclient/+/80597807:46
jeevanetwork question, if my openstack controllers are on the untagged MAAS PXE network, what is their place in ironic? To stay as is, or must they also be the same as the ironic PXE untagged VLAN ?08:20
opendevreviewAija Jauntēva proposed openstack/ironic master: Remove manager param for iDRAC OEM calls  https://review.opendev.org/c/openstack/ironic/+/79220008:46
dtantsurgood morning ironic, happy Monday09:27
dtantsurjanders: re exercising: I had my vaccine on Friday, went climbing on Monday09:28
iurygregorygood morning dtantsur 09:30
opendevreviewAija Jauntēva proposed openstack/ironic master: Add RAID mode conversion for idrac-redfish  https://review.opendev.org/c/openstack/ironic/+/80411209:36
opendevreviewAija Jauntēva proposed openstack/ironic master: Add clear foreign config to idrac-redfish RAID  https://review.opendev.org/c/openstack/ironic/+/80484810:10
dtantsurfolks, we've collected a few outstanding bifrost patches, could you take a look? https://review.opendev.org/q/project:openstack%252Fbifrost+status:open10:38
dtantsuriurygregory: are you looking into the sqlalchemy issue?10:42
dtantsur(I can do it as a warm-up for the week)10:42
iurygregorydtantsur, I was trying to reproduce the issue but no lucky yet10:43
dtantsuryeah, upgrading oslo.db to 11.0.0 doesn't seem enough10:44
iurygregoryyup 10:44
*** sshnaidm is now known as sshnaidm|off10:46
dtantsurprometheanfire: I'm starting to suspect a race or regression in oslo.db10:46
dtantsurpotentially in https://opendev.org/openstack/oslo.db/commit/22b44ee18b1585bc2943a29ddaa18051cb1344ed10:47
iurygregorydtantsur, maybe we should also raise this in #openstack-oslo to see if they have some ideas?11:02
dtantsuriurygregory: likely. could you please do it? I'm not feeling well.11:02
iurygregoryyup!11:03
iurygregorytake care =)11:03
opendevreviewJacob Anders proposed openstack/ironic master: Add support for verify steps  https://review.opendev.org/c/openstack/ironic/+/80000111:19
jandersthank you dtantsur - sounds like I may try getting on the bike tomorrow :)11:43
janderssee you tomrorow Ironic o/11:43
iurygregorybye janders o/11:46
opendevreviewMerged openstack/ironic master: Add better error messages for invalid conf molds  https://review.opendev.org/c/openstack/ironic/+/80522212:07
opendevreviewVerification of a change to openstack/ironic master failed: Minor formatting and doc changes to change boot mode feature commit.  https://review.opendev.org/c/openstack/ironic/+/80584112:15
opendevreviewAija Jauntēva proposed openstack/ironic master: Fix iDRAC import configuration job with errors  https://review.opendev.org/c/openstack/ironic/+/80543112:17
TheJuliagood morning13:07
arne_wiebalckHey TheJulia o/13:10
dtantsurmorning TheJulia 13:11
iurygregorygood morning TheJulia =)13:14
opendevreviewJulia Kreger proposed openstack/ironic master: Revert "Allow reboot to hard disk following iso ramdisk deploy."  https://review.opendev.org/c/openstack/ironic/+/80528413:21
opendevreviewVerification of a change to openstack/ironic master failed: Minor formatting and doc changes to change boot mode feature commit.  https://review.opendev.org/c/openstack/ironic/+/80584113:33
prometheanfiredtantsur: fun13:36
dtantsuryeah.. I don't quite get what is going on13:37
dtantsuriurygregory wanted to ask on #openstack-oslo13:37
iurygregoryI did =)13:38
iurygregorybut no answer yet13:39
iurygregoryI'm not sure who are the oslo.db experts to ping, so I just asked in the channel =)13:40
opendevreviewDmitry Tantsur proposed openstack/sushy-tools master: Move virtual media endpoint handlers to a separate blueprint  https://review.opendev.org/c/openstack/sushy-tools/+/80660314:35
iurygregorylmcgann, I just noticed that some of the patches for the keylime are in merge conflict, it would be good to fix before the Review Jam tomorrow =)14:36
lmcgannwill do!14:36
opendevreviewDmitry Tantsur proposed openstack/sushy-tools master: [WIP] Support uploading certificates for virtual media  https://review.opendev.org/c/openstack/sushy-tools/+/80625314:40
iurygregory#startmeeting ironic15:00
opendevmeetMeeting started Mon Aug 30 15:00:00 2021 UTC and is due to finish in 60 minutes.  The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'ironic'15:00
iurygregoryo/15:00
erbarro/15:00
iurygregoryHello everyone, welcome to our weekly meeting!15:00
arne_wiebalcko/15:00
ajyao/15:00
dtantsuro/15:00
rlooo/15:00
iurygregoryYou can find our agenda in the wiki15:00
iurygregory#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
rpioso\o15:00
TheJuliao/15:00
iurygregory#topic Announcements / Reminder15:01
iurygregory#info We have topics for the Review Jam tomorrow =)15:01
iurygregoryAnyone has something to announce or remind us?15:01
TheJuliaIury will be our glorius PTL for Yoga!15:02
dtantsur\o/15:02
iurygregory\o/15:02
rlooyay, many thanks iurygregory!!!15:02
iurygregorynp, I will do my best \o/15:03
TheJulia:)15:03
iurygregory#topic Review action items from previous meeting15:03
iurygregorysince we don't have any action items from the previous meeting, moving on15:04
iurygregory#topic Review subteam status reports15:04
iurygregory#link https://etherpad.opendev.org/p/IronicWhiteBoard15:04
iurygregorystarting on L6015:04
TheJuliaLooks like everyone is getting pulled downstream pretty hardcore15:06
iurygregoryyup, it happens from time to time =(15:07
dtantsurhey, I've produced a ton of bifrost and sushy-tools patches :D15:07
iurygregoryI think dtantsur will add a lot patches  for the Deciding on priorities for the coming week :D15:07
dtantsurwith great pleasure!15:07
TheJuliaSounds most excellent15:08
iurygregoryoh nice, we have a section for the Keylime work in the subteam status :D15:09
iurygregorydtantsur, TheJulia Driver structure and model doesn't have any updates for like 1month, can I just add "no-updates"? 15:10
TheJuliaYeah, I've been pulled downstream so much I haven't really had time to think about any of it15:10
dtantsuryes please15:11
iurygregoryack =)15:11
dtantsurI'll likely spend the rest of the cycle redfishing15:11
TheJuliadtantsur: is that like going fishing on a boat?!15:11
dtantsurI would love to try :)15:11
dtantsurbut my curse is figuring out TLS apparently15:12
iurygregoryI think dtantsur will become a chef :D15:12
dtantsurPrincipal Fugu Engineer15:12
TheJuliaso next time we're all in Germany, dtantsur can cook us fish :)15:12
iurygregoryFugu LOL15:12
TheJulialol15:12
dtantsurTheJulia: we have very nice sushi fish here thanks to a few authentic japanese shops15:13
TheJuliaOne day :)15:13
dtantsursigh15:13
TheJuliasigh++15:13
iurygregoryalmost dinner time and we are talking about food \o/15:13
iurygregorysounds about right :D15:13
iurygregoryI think we are good on the subteam status, moving on15:14
iurygregory#topic Deciding on priorities for the coming week15:14
ajyacan these be added - https://review.opendev.org/c/openstack/ironic/+/804112 https://review.opendev.org/c/openstack/ironic/+/804848 ?15:14
TheJuliaajya: is there a story for background on those?15:15
iurygregorymakes sense to me 15:15
iurygregoryyeah having a story RFE would be good =)15:16
dtantsurcan I just add everything non-WIP from https://review.opendev.org/q/owner:dtantsur%2540protonmail.com+status:open+(project:openstack/bifrost+OR+project:openstack/sushy-tools) ? 15:16
ajyaTheJulia: no, it's to get it closer to idrac-wsman behavior15:16
dtantsurI don't want to paste every patch here15:16
TheJuliaiurygregory: I'm not sure it is an rfe, but can't be sure without some more words somplace15:16
TheJuliadtantsur: sounds reasonable15:16
iurygregoryTheJulia, well they are with feature in the releasenotes thats why I said rfe hehe15:17
TheJuliaiurygregory: ahh, yeah15:17
arne_wiebalckI have added https://review.opendev.org/c/openstack/ironic/+/801425 (this is the fast track retry on failed BMC connections, seems to prevent fast track nodes from being switched off)15:17
iurygregorydtantsur, I think you can =)15:17
TheJuliaarne_wiebalck: random bmc unhappiness?15:18
arne_wiebalckTheJulia: I think UDP packet loss 15:18
ajyaI can create story with tasks for these patches with some background info, otherwise patches should be self-contained.15:19
TheJuliaI guess that is bound to happen on conductors with lots of nodes15:19
arne_wiebalckTheJulia: that is what I am thinking15:19
TheJuliaajya: sounds good, and also lets explicitly state if there is a backport intent15:19
arne_wiebalckTheJulia: and we do retries for power sync, but not on fast track15:19
TheJuliaarne_wiebalck: makes sense15:19
* arne_wiebalck should try moving to redfish to fix these instead of retrying ...15:20
TheJuliaeh, we've still seem bmc's go out for lunch for a moment15:20
TheJuliaso makes sense to retry anwyay15:20
iurygregoryajya, I think a story would be good 15:20
arne_wiebalckTheJulia: ++15:20
iurygregoryajya, feel free to add the hashtag in the patches you mentioned o/15:22
ajyaok, thanks15:22
iurygregoryanyone has more patches that needs review?15:23
TheJuliaNot on my end15:23
iurygregoryI think we are good with the amount of patches we added 15:24
iurygregorymoving on15:24
iurygregory#topic Discussion15:25
iurygregoryWe don't have any topics for Discussion in the agenda, but I will give a few minutes to see if anyone has something15:25
TheJuliaCould/should we just move to Open Discussion then?15:25
iurygregorymaybe, arne_wiebalck do you have something for the SIG?15:26
arne_wiebalckyes15:26
iurygregory#topic Baremetal SIG15:26
iurygregorygo ahead =)15:26
arne_wiebalckwe said we would do an operator feedback session next time15:26
arne_wiebalckthis is what we have on the etherpad atm15:26
arne_wiebalckIIRC, we said we would decide nearer the time if that would make sense15:27
arne_wiebalckgiven that there was also the openinfra live event15:27
TheJuliaYeah, we kind of didn't push that forward with holidays and the chance of getting something larger organized, yet that fell through15:27
arne_wiebalckright15:27
arne_wiebalckwhile I would like to have such a discussion I am not sure operators would show up by themselves15:28
arne_wiebalckgiven the attendance we had over the last months15:28
TheJuliaNo getting them to appear in a community setting seems to be one of the hardest things to do15:28
arne_wiebalckso, we could either reach out directly or skip and have a topic of the day15:29
arne_wiebalckthoughts?15:29
iurygregorywell, maybe we can try to reach out the osopsmeetup ?15:29
iurygregoryand if we know some operators we can try to ask if they are interested?15:30
arne_wiebalckI think they face similar issues with participation15:30
arne_wiebalckosops I mean15:30
jssfrwhat's an operator feedback session?15:30
arne_wiebalckjssfr: Ironic users come with their feedback, pain points, ideas, ...15:31
arne_wiebalckjssfr: Ironic users/operators/admins15:31
jssfroh15:31
jssfrnever heard of it :)15:31
TheJuliaiurygregory: I think it might make sense to engage them15:31
arne_wiebalckheh, that is the problem15:31
TheJuliaiurygregory: reach out and say "hey, we're willing to listen/chat" if interested15:31
TheJuliabut I don't know if that makes a baremetal SIG session15:31
iurygregoryright15:32
TheJuliaI'd <3 soemthing to helpf or planning with the next development cycle though15:32
arne_wiebalckTheJulia: well, we can try and offer to listen15:32
iurygregorywe can probably come up with something for the PTG?15:32
arne_wiebalckTheJulia: if noone comes, fine, at least we gave the chance15:32
iurygregoryor even a separate event15:32
iurygregoryarne_wiebalck, ++15:32
TheJuliayeah, true15:33
* TheJulia wonders if anyone has posted questions/comments on our existing videos15:33
arne_wiebalckok, so how about everyone reaches out and we see what happens?15:33
TheJuliasounds reasonable15:34
iurygregoryyeah, I agree15:34
arne_wiebalcklike, we know some operators/users :)15:34
arne_wiebalckok, let's try this then15:34
iurygregory#action everyone to reach out to know operators/users15:35
arne_wiebalckthis is for tomorrow in a week15:35
arne_wiebalckSep 7th15:35
arne_wiebalckI will send out a mail s well. 15:35
iurygregoryarne_wiebalck, ++15:35
arne_wiebalckthat is it from me for the SIG15:36
iurygregorytweet also :D15:36
arne_wiebalckyes, will do15:36
iurygregorytks arne_wiebalck !15:36
iurygregoryskipping RFE review since we don't have any15:36
iurygregory#topic Open Discussion15:36
TheJuliaI'm suspecting we should likely start discussing/planning the PTG15:37
arne_wiebalckI have something :)15:37
iurygregorygo ahead arne_wiebalck 15:37
iurygregoryTheJulia,++15:37
arne_wiebalckFor the burn-in we are interested in the log output in more or less real time.15:37
arne_wiebalckWe were thinking about having sth like fluentd on the image.15:38
arne_wiebalckThe other option would be to ship logs to the conductor and use the conductor logging.15:38
arne_wiebalckThe conductor gets the IPA logs eventually.15:38
arne_wiebalckNow, for fast track this is more tricky, since it will only do this at the end.15:39
TheJuliaI'm not opposed to it, but how would it work when the conductor is just a process in a container15:39
arne_wiebalckHow does this work now for the IPA logs when the conductor is in a container?15:40
arne_wiebalckMy thinking is to leverage the logging that should be in place for the conductor anyway.15:40
TheJuliaarne_wiebalck: just lives on a filesystem, likely a bind mounted filesystem15:40
arne_wiebalckUsing fluentd on the IPA pushes the issue downstream and to image building.15:41
TheJuliaindeed15:41
TheJuliaso realistically, it could be anywhere configured by an operator15:41
arne_wiebalckHaving logs flow into the conductor more regularly offers operators to use them even in fast track.15:41
TheJuliayeah15:42
arne_wiebalckImagine 100s of nodes in fast track, logs sent into the conductor, and  onwards to ES, for instance.15:43
arne_wiebalckWe have a mechanism to get the logs already.15:43
arne_wiebalckIt is just run only once and collects a lot of additional logsd.15:43
arne_wiebalcklogs15:43
TheJuliaI guess it is more an agent/image thing than anything else?15:43
arne_wiebalckYeah, you could see it that way. But configuring the IPA to push logs into a monitoring infra may be more difficult than the controllers which may do this anyway.15:44
arne_wiebalckI am trying to leverage the existing pipeline.15:45
TheJuliasounds good15:45
arne_wiebalckI would imagine fast track users want the IPA logs, no?15:46
arne_wiebalckAnd they may not want to wait until the IPA is fully done (which I think is when the logs are collected).15:46
arne_wiebalckAnyway, if anyone has thoughts on this, let me know :)15:47
TheJuliaIt makes sense to get them and if they are doing something like burn in it does also make sense15:47
TheJuliaAnd is really just more operator capable at that point15:47
arne_wiebalckYes, burn-in is the use case at hand.15:47
arne_wiebalckThe h/w colleagues would like to get the network speed results for their records for instance.15:48
arne_wiebalckThere is a logical difference between burn-in and benchmark, but it does not make sense to run a benchmark when the burn-in provides these values.15:48
TheJuliaindeed15:49
arne_wiebalckNo need to decide anything now, I was just thinking that sending logs more regularly may be sensible for fast track.15:50
arne_wiebalckIn general, not only burn-in.15:50
* arne_wiebalck stops now :)15:50
iurygregoryit may be sensible I would say, just wondering the overhead that can cause 15:51
arne_wiebalckiurygregory: yes, atm, we create an archive for a clean "cycle"15:52
arne_wiebalckiurygregory: we will need to avoid sending the same logs over and over again15:53
iurygregoryinteresting, I do like the idea15:53
iurygregorynext week we can talk about the PTG topics I would say since we only have 5minutes now15:54
TheJuliaSounds good15:54
iurygregorywe already have the etherpad https://etherpad.opendev.org/p/ironic-yoga-ptg15:55
iurygregoryfeel free to add topics =)15:55
iurygregory#topic Who is going to run the next meeting?15:55
iurygregorydo we have any volunteers?15:55
rloois next mon a holiday for folks? it is in canada and i think US.15:56
iurygregory*-* 06 Sep?15:56
iurygregorymaybe we can skip if people agree15:56
rloolabor day in US15:56
iurygregoryit's my bday so I give +1 to skip LOL15:57
rloothat's a really good reason to celebrate!15:57
iurygregoryyeah =)15:58
dtantsur++15:59
iurygregoryso we good to skip next weekly meeting?15:59
rloocrickets means... yes :D16:00
iurygregoryyeah :D16:00
iurygregoryI will send an email16:00
iurygregorytks everyone!16:00
iurygregory#endmeeting16:00
opendevmeetMeeting ended Mon Aug 30 16:00:26 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-30-15.00.html16:00
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-30-15.00.txt16:00
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-30-15.00.log.html16:00
arne_wiebalckthanks iurygregory !16:00
TheJuliaThanks iurygregory!16:00
ajyaTheJulia: when you have time, can you revisit this patch https://review.opendev.org/c/openstack/ironic/+/792200 that got unblocked by dependency release? Thanks.16:03
TheJuliaajya: done16:10
opendevreviewLeo McGann proposed openstack/ironic master: Adds Attestation Interface to Node  https://review.opendev.org/c/openstack/ironic/+/75583616:23
opendevreviewMerged openstack/ironic master: Minor formatting and doc changes to change boot mode feature commit.  https://review.opendev.org/c/openstack/ironic/+/80584116:51
arne_wiebalckbye everyone, see you tmrw o/16:56
opendevreviewVerification of a change to openstack/ironic master failed: Remove manager param for iDRAC OEM calls  https://review.opendev.org/c/openstack/ironic/+/79220017:39

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