Monday, 2021-06-07

*** altlogbot_3 has joined #openstack-ironic00:13
*** altlogbot_3 has left #openstack-ironic00:13
*** nfedorov has quit IRC01:40
*** zaneb has joined #openstack-ironic02:28
*** zaneb has quit IRC02:46
*** paras333 has joined #openstack-ironic03:04
*** paras333 has quit IRC03:17
*** mgoddard has quit IRC03:33
*** mgoddard has joined #openstack-ironic03:33
*** paras333 has joined #openstack-ironic03:44
*** zaneb has joined #openstack-ironic03:47
*** zaneb has quit IRC03:58
*** paras333 has quit IRC03:59
*** paras333 has joined #openstack-ironic05:56
*** paras333 has quit IRC06:04
yoctozeptomorning06:17
iurygregorygood morning janders yoctozepto and Ironic o/06:43
yoctozeptoo/ iurygregory06:46
arne_wiebalckGood morning yoctozepto iurygregory and Ironic!06:55
yoctozeptoo/ arne_wiebalck06:56
* yoctozepto needs to learn this hello line06:56
jandershey yoctozepto iurygregory arne_wiebalck and Ironic o/06:57
arne_wiebalckGood morning janders o/06:57
yoctozeptoo/ janders06:57
* yoctozepto going for hello-full-mesh06:57
iurygregoryhey arne_wiebalck o/07:09
*** rpittau|afk is now known as rpittau07:26
rpittaugood morning ironic! o/07:26
yoctozeptoo/ rpittau07:31
rpittauhey yoctozepto :)07:31
iurygregorymorning rpittau o/07:37
rpittauhey iurygregory :)07:38
opendevreviewRadosław Piliszek proposed openstack/python-ironic-inspector-client master: [doc] Fix lldp_basic name  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/79478207:40
*** paras333 has joined #openstack-ironic08:02
*** paras333 has quit IRC08:10
jandershey rpittau o/08:35
*** sshnaidm|afk is now known as sshnaidm08:51
* dtantsur rubs eyes08:57
opendevreviewMerged openstack/python-ironic-inspector-client master: Expose more interface fields  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/79484308:58
*** sshnaidm has quit IRC08:58
*** sshnaidm has joined #openstack-ironic08:59
rpittauhey janders :)09:08
rpittaumorning dtantsur :)09:08
yoctozeptoIronic cores, do you normally accept backports of simple life-improving enhancements like the one in https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/794843 ?09:18
rpittauyoctozepto: technically that's a feature, so I'm afraid not09:20
yoctozeptorpittau: ok09:21
opendevreviewAija Jauntēva proposed openstack/ironic master: Redfish: Get only RAID controller's physical disks  https://review.opendev.org/c/openstack/ironic/+/79507409:37
opendevreviewMerged openstack/ironic-python-agent master: Ignore efi grub2-install failure  https://review.opendev.org/c/openstack/ironic-python-agent/+/79436309:47
*** paras333 has joined #openstack-ironic10:08
*** paras333 has quit IRC10:16
arne_wiebalckbye everyone, see you tomorrow o/11:25
yoctozeptobye arne_wiebalck11:26
janderssee you tomorrow Ironic o/11:35
*** Luzi has joined #openstack-ironic11:42
*** Luzi has quit IRC11:51
*** tosky has joined #openstack-ironic11:59
*** paras333 has joined #openstack-ironic12:14
*** paras333 has quit IRC12:22
opendevreviewMerged openstack/ironic stable/wallaby: Avoid unnecessary validation in boot interfaces  https://review.opendev.org/c/openstack/ironic/+/79044112:37
*** zaneb has joined #openstack-ironic12:39
*** Qianbiao has joined #openstack-ironic12:41
Qianbiaohello ironic12:42
rpittauhey Qianbiao :)12:45
opendevreviewMerged openstack/ironic-python-agent master: Burn-in: Add disk step  https://review.opendev.org/c/openstack/ironic-python-agent/+/79264412:46
Qianbiaohey rpittau o/12:47
Qianbiaohappy fresh week12:47
rpittauthanks!12:48
*** lmcgann has joined #openstack-ironic12:49
dtantsurwait, it's Monday again?12:50
*** Luzi has joined #openstack-ironic12:52
*** paras333 has joined #openstack-ironic12:56
dtantsurmgoddard: FYI https://review.opendev.org/c/openstack/bifrost/+/794177/ is marked as WIP and won't merge until it's changed13:05
opendevreviewDmitry Tantsur proposed openstack/ironic-lib stable/wallaby: utils.execute: log stdout and stderr even on failure  https://review.opendev.org/c/openstack/ironic-lib/+/79504513:06
opendevreviewDmitry Tantsur proposed openstack/ironic-lib stable/victoria: utils.execute: log stdout and stderr even on failure  https://review.opendev.org/c/openstack/ironic-lib/+/79504613:06
*** tzumainn has joined #openstack-ironic13:09
*** Mahnoor_Asghar has joined #openstack-ironic13:10
*** rloo has joined #openstack-ironic13:14
opendevreviewDmitry Tantsur proposed openstack/bifrost master: Reasonable error message if network_interface is not found  https://review.opendev.org/c/openstack/bifrost/+/79510613:17
TheJuliadtantsur: yes :(13:19
TheJuliaiurygregory: you going to be on for review jam?13:19
dtantsurle sigh13:19
dtantsurgood morning TheJulia13:19
TheJuliagood morning dtantsur13:19
iurygregoryTheJulia, today I probably won't but tomorrow I can join13:20
iurygregorygood morning TheJulia =)13:20
TheJuliaiurygregory: ack13:20
rpittaugood morning TheJulia :)13:21
opendevreviewDmitry Tantsur proposed openstack/bifrost master: Reasonable error message if network_interface is not found  https://review.opendev.org/c/openstack/bifrost/+/79510613:21
dtantsurokay, hopefully now people won't get confused ^^^13:21
TheJuliaSeems quiet in here today, did everyone get coffee?13:25
mgoddardah, thanks dtantsur13:26
* rpittau will get the second coffee i n3 minutes13:27
dtantsurTheJulia: does sushi count?13:27
*** sdanni has joined #openstack-ironic13:29
*** nfedorov has joined #openstack-ironic13:32
TheJuliasushi for breakfast?13:50
TheJuliaAnd our library name has forever ruined my brain's spelling of sushi. ;)13:50
rpittauit is indeed good13:50
*** Luzi has quit IRC13:51
TheJuliadtantsur: did you see the graph I posted from Friday?13:52
TheJuliabuhman: review some of my recent changes, I need to fix the topic though13:52
*** tzumainn has quit IRC13:57
dtantsurTheJulia: I was out on Friday, so likely no14:00
dtantsurand yes, sushi for breakfast!14:00
dtantsur(because we cooked so much yesterday that we had 2 good portions left for today)14:00
TheJuliasushi++14:00
TheJuliahttps://usercontent.irccloud-cdn.com/file/bCKTykY0/node-performance-all-patches.png14:00
TheJuliaReview jam in progress https://meetpad.opendev.org/ironic14:01
dtantsurvery nice!14:01
dtantsurI'll need to skip both review and the meeting today, sorry14:01
dtantsursuspecting I was the one to run it, could someone run it instead?14:01
dtantsur(I can hopefully do it next Monday then)14:01
TheJuliaI can run it14:01
dtantsurthx!14:01
TheJuliano problem14:02
* dtantsur back in a few hours14:02
*** kkillsfirst has joined #openstack-ironic14:02
TheJuliadtantsur: keep in mind that is with keeping a lot of the internal object work in place, *but* overhead wise it is not that much with what is left, the bulk is santization that remains as you can see with the final patch. It also shifts some of the overhead to upfront instead of constant stream of 1000+ queries to do the list. :(14:03
TheJuliaso the DBs should be *much* happier14:03
TheJuliaat least, in theory14:03
rpittauI think it was me running the meeting today14:05
*** tzumainn has joined #openstack-ironic14:05
opendevreviewMerged openstack/bifrost stable/victoria: Install DIB dependencies in bifrost-ironic-install role  https://review.opendev.org/c/openstack/bifrost/+/79417714:06
opendevreviewMerged openstack/ironic master: Move provision states documentation to the user guide  https://review.opendev.org/c/openstack/ironic/+/79394614:08
TheJuliarpittau: well, if your able/willing ? I can run it14:25
rpittauTheJulia: yeah, I will, I volunteered last time :)14:26
*** Mahnoor_Asghar has quit IRC14:34
TheJuliaokay14:48
*** stendulker has joined #openstack-ironic14:59
rpittau#startmeeting ironic15:00
opendevmeetMeeting started Mon Jun  7 15:00:13 2021 UTC and is due to finish in 60 minutes.  The chair is rpittau. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
iurygregoryo/15:00
opendevmeetThe meeting name has been set to 'ironic'15:00
TheJuliao/15:00
erbarro/15:00
stendulkero/15:00
rpittauHello everyone!15:00
ajyao/15:01
rpiosoo/15:01
TheJuliaGreetings15:01
rpittauDo we actually have the quorum?15:02
TheJuliauhh... that is a good question15:02
TheJulia7 counting yourself15:02
rpittau:/15:02
TheJulia815:02
TheJuliaso technically yes :\15:02
TheJuliabut it is borderline15:02
*** zbitter has joined #openstack-ironic15:02
TheJuliawell, zbitter appearing might make that 9 attendees15:02
iurygregorylol15:03
rpittau:)15:03
rpittaualright, let's do it!15:03
rlooo/15:03
TheJulia\o rloo15:03
rpittaurloo to the rescue15:03
rpittau#topic Announcements / Reminders15:03
rloo(sorry, doorbell rang...)15:03
rlooi mean, that's why i'm a bit late :)15:03
rpittau#info Xena-2 is on 15 July15:03
zbitterTheJulia: nope, just flaky network ;)15:04
rpittau#info last week we released Sprint 115:04
rpittauDoes anyone have anything else to announce or remind us of?15:04
iurygregoryIronic 18.0.0 =)15:04
iurygregoryis available15:04
rpittauyay \o/15:05
rloowhee15:05
TheJulia\o/15:05
rpittauseems no other announcements, moving forward15:06
Qianbiao\0/15:06
rpittau#topic Review action items from previous meeting15:06
rpittauwe don't have any action items from last time15:06
rpittaumoving on15:07
rpittau#topic Review subteam status reports15:07
rpittau#link https://etherpad.openstack.org/p/IronicWhiteBoard15:07
rpittaustarting from L6415:07
*** Mahnoor_Asghar has joined #openstack-ironic15:08
*** JayF has joined #openstack-ironic15:08
rpittauI think we can mark iscsi removal as done15:08
*** zaneb has quit IRC15:08
rpittaukaifeng and arne_wiebalck are not here, any updates on node error history?15:09
rpittauiurygregory: postpone privsep discussion to next time?15:09
iurygregoryrpittau, probably since Dmitry is not here15:09
rpittausounds good15:09
iurygregorywe can also discuss in the patch15:10
rpittausure15:10
rpittauiurygregory: anything new on virtual media visibility ?15:11
iurygregoryrpittau, still writting code WIP15:11
rpittauok15:11
rpittauteam status looks ok, anything else to add ?15:13
iurygregoryI think we can move on =)15:14
TheJuliaNothing status wise really15:14
rpittaualrighty15:14
rpittau#info Deciding on priorities for the coming week15:14
iurygregorytopic? =)15:14
TheJuliayes, topic :)15:14
rpittauerrr15:14
rpittaubad copy-paste15:14
TheJuliaheh15:15
rpittau#topic Deciding on priorities for the coming week15:15
TheJuliaSo I would really appreciate some of the db performance patches I've got up in review15:15
rpittaucurrent prio https://tinyurl.com/ironic-weekly-prio-dash15:15
TheJuliait makes getting node list *much* faster15:15
rpittausure!15:15
rpittauwe definitely don't have many priorities left15:16
iurygregoryyeah15:16
TheJuliaI'll add the hash tags then15:16
JayFTheJulia: is that whole line ready to merge? I reviewed them somewhat this weekend but didn't know you were ready to land 'em15:16
rpittauok, thanks15:16
ajyacan another core take a look at https://review.opendev.org/c/openstack/ironic/+/792088 ?15:16
TheJuliaJayF: close, I see some failures on the api patch but I've not had a chance to inspect them, I think it was +1'ed by zuul on the next to last change so it is likely stupidly minor15:17
rpittauajya: I added the tag in case we don't get to it today15:17
ajyarpittau: thanks15:17
TheJuliaoh, the actual api layer change, it was the sanitizer change that didn't :\15:17
TheJuliapep8/bandit figures15:18
* TheJulia blinks15:18
rpittaupep8 is unforgiving15:18
TheJuliait says it is complexity too15:18
TheJuliafigures15:18
TheJuliaI should have that sorted today15:19
rpittauok, we have I think a bit more space for priorities this week, anyone else has patches to add ?15:22
iurygregoryI would appreciate reviews in the event spec hehe15:23
iurygregoryit's already in the priority15:23
TheJulialooks like we may want to get our eyes on https://review.opendev.org/c/openstack/ironic/+/76800915:23
rpittauyup, ok15:23
iurygregoryTheJulia, ++15:23
rpittauI added that too15:23
TheJuliaI need to re-review it15:24
TheJuliaWe also need the upper layers15:24
TheJuliaWe have someone pushing forward node tags15:24
TheJuliawhich was added to the db ages ago15:24
TheJuliakaifeng posted a question noting we may need to re-evaluate need15:25
TheJuliawhich is avlid15:25
TheJuliavalid15:25
TheJuliaI think we can proceed15:27
rpittauyeah, we look good on priorities15:27
rpittau#topic Discussion15:27
rpittauno discussions this week, let's move on15:27
rpittau#topic Baremetal SIG15:28
iurygregoryBare Metal SIG will meet tomorrow Tue June 8, 2021, at 2pm UTC on zoom.15:28
rpittaureminder! Bare Metal SIG will meet tomorrow Tue June 8, 2021, at 2pm UTC on zoom15:28
iurygregory"Topic-of-the-day" presentation by Dmitry Tantsur (dtantsur) with an "Introduction to the Ironic Python Agent Builder"15:28
iurygregoryarne_wiebalck asked me to give the update =)15:28
rpittauok :)15:28
rpittauthat's all for the Baremetal SIG15:29
rpittau#topic RFE review15:29
rpittauwe don't have anything scheduled, but if anyone has RFEs to propose feel free to do it now15:30
rpittaualright, let's move on :)15:30
rpittau#topic Open Discussion15:30
rpittauarne_wiebalck had a topic for open discussion but he asked to postpone to next week15:31
rpittauanything else we want to discuss today ?15:31
TheJulianothing on the top of my head15:32
rpittauok! Last but not least....15:32
rpittau#topic Who is going to run the next meeting?15:32
TheJuliaI'll run it15:32
rpittauthanks TheJulia15:32
iurygregorytks TheJulia =)15:32
rpittauand thanks everyone!15:32
rpittau#endmeeting15:32
opendevmeetMeeting ended Mon Jun  7 15:32:46 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:32
opendevmeetMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-06-07-15.00.html15:32
opendevmeetMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-06-07-15.00.txt15:32
opendevmeetLog:            http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-06-07-15.00.log.html15:32
Qianbiaothanks rpittau15:33
rpittauhave a wonderful week everyone :)15:33
sdanniHey! hoping it's not too late... We would like to get more feedback on our security interface spec: https://review.opendev.org/c/openstack/ironic-specs/+/576718.15:34
sdannicomments and suggestions are welcomed!15:35
TheJuliao/ sdanni15:37
rpittausdanni: that's in the priorities, I'll tryu to get there during the week15:37
TheJuliasdanni: I think I commented on the let me know item you had15:37
TheJuliadriver_internal_info most likely15:37
TheJulia :)15:37
sdannilmcgann: anything to add?15:38
lmcgannnothing else at the moment15:41
*** Qianbiao has quit IRC15:53
rpittaubye! o/16:00
*** rpittau is now known as rpittau|afk16:01
lmcgannTheJulia: in regards to your comment on driver_internal_info, I think I answered your question, but let me know if there's anything I missed or am not understanding!16:02
TheJulialmcgann: okay, I'll try to cycle back to it after I finish my current administrative email16:07
*** stendulker_ has joined #openstack-ironic16:08
*** zbitter is now known as zaneb16:12
*** stendulker has quit IRC16:12
*** Mahnoor_Asghar has quit IRC16:16
*** jelabarre-rh has joined #openstack-ironic16:17
*** stendulker_ has quit IRC16:18
opendevreviewMerged openstack/bifrost master: Reasonable error message if network_interface is not found  https://review.opendev.org/c/openstack/bifrost/+/79510616:26
opendevreviewMerged openstack/ironic master: Improve agent_client logging  https://review.opendev.org/c/openstack/ironic/+/79399517:06
*** mjturek has joined #openstack-ironic17:12
*** bburns has joined #openstack-ironic17:20
*** bburns has quit IRC17:24
*** bburns has joined #openstack-ironic17:25
opendevreviewDmitry Tantsur proposed openstack/ironic stable/wallaby: Improve agent_client logging  https://review.opendev.org/c/openstack/ironic/+/79513517:53
*** kkillsfirst has quit IRC18:06
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/wallaby: Reasonable error message if network_interface is not found  https://review.opendev.org/c/openstack/bifrost/+/79513618:31
*** kkillsfirst has joined #openstack-ironic18:38
TheJuliagood morning kkillsfirst18:43
* dtantsur likes the nick18:45
kkillsfirstHello TheJulia18:46
TheJuliaI wasn't going to say anything, but I think it is awesome too.18:48
*** qwebirc7749 has joined #openstack-ironic18:48
*** qwebirc7749 has quit IRC18:48
dtantsurhave a good night folks!18:49
TheJuliao/18:51
TheJuliastevebaker: paras333: my laptop crashed  :(19:50
paras333TheJulia: that's ok we are anyways done with the call19:56
* TheJulia decides it is best to just get lunch20:02
*** rloo has quit IRC20:05
*** rloo has joined #openstack-ironic20:06
buhmanTheJulia: (you asked for reviews on db-perf-1) is moderate code golf welcome?20:09
TheJuliacode golf?20:15
TheJuliakeep in mind, I'm trying to make a series which is backportable20:15
*** zbitter has joined #openstack-ironic21:01
*** jlvillal has joined #openstack-ironic21:04
*** rloo has quit IRC21:07
*** rloo has joined #openstack-ironic21:07
jandersgood morning Ironic o/21:07
*** zaneb has quit IRC21:07
buhmanTheJulia: yeah; I was going to suggest changing the "if x in <list>" operations to set operations.21:50
TheJuliabuhman: eh, It won't make much of a performance difference in the grand scheme of things21:52
buhmanhttps://gist.github.com/buhman/c3d32950acb29a6a899f923838e7e7d0 those two are equivalent21:52
buhmanok21:52
*** tosky has quit IRC22:04
TheJuliabuhman: oh, yeah, even that far up in the stack... *shrugs*22:29
*** lmcgann has quit IRC22:40
stevebakerTheJulia: I can replicate the ipa python2.7 test failures, so I'll take a look at that22:57
TheJuliastevebaker: okay, I can also look at it tomorrow if my work laptop decides I. An be let in tomorrow :(22:58
stevebakercool22:58
* TheJulia goes back to exercising because the computers hate her22:58
*** kkillsfirst has quit IRC23:20
*** kkillsfirst has joined #openstack-ironic23:32
*** jelabarre-rh has quit IRC23:53

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