Monday, 2020-03-30

*** ociuhandu has joined #openstack-ironic01:06
openstackgerritMerged openstack/ironic stable/stein: Retry agent get_command_status upon failures  https://review.opendev.org/71543801:09
*** ociuhandu has quit IRC01:09
*** ociuhandu has joined #openstack-ironic01:10
*** ociuhandu has quit IRC01:20
*** ociuhandu has joined #openstack-ironic01:21
*** ociuhandu has quit IRC01:25
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: list ports by portgroups  https://review.opendev.org/71575001:39
*** Goneri has quit IRC01:48
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: add api, list ports detail by portgroups  https://review.opendev.org/71575101:51
*** ricolin_ has joined #openstack-ironic02:22
*** ricolin_ has quit IRC02:31
*** ociuhandu has joined #openstack-ironic02:34
*** ociuhandu has quit IRC02:39
openstackgerritMerged openstack/ironic-python-agent master: Fix GPT partition tables after agent writes contents  https://review.opendev.org/71479802:52
*** jdandrea has quit IRC03:04
*** mkrai has joined #openstack-ironic03:27
*** ociuhandu has joined #openstack-ironic04:14
*** ociuhandu has quit IRC04:24
*** ociuhandu has joined #openstack-ironic04:24
*** ociuhandu has quit IRC04:30
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: list ports by portgroups  https://review.opendev.org/71575004:30
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: add api, list ports detail by portgroups  https://review.opendev.org/71575104:41
*** ociuhandu has joined #openstack-ironic05:24
*** ijw has joined #openstack-ironic05:31
iurygregorygood morning Ironic o/05:31
*** ijw_ has quit IRC05:32
*** ociuhandu has quit IRC05:34
*** ociuhandu has joined #openstack-ironic05:35
*** ociuhandu has quit IRC05:40
*** ociuhandu has joined #openstack-ironic05:44
*** ociuhandu has quit IRC05:54
*** ociuhandu has joined #openstack-ironic05:56
*** ociuhandu has quit IRC06:00
arne_wiebalckGood morning, iurygregory and ironic!06:22
iurygregoryarne_wiebalck, o/06:22
openstackgerritVerification of a change to openstack/ironic-python-agent failed: Rescan after restarting the md device  https://review.opendev.org/70663206:30
*** ociuhandu has joined #openstack-ironic06:35
*** ociuhandu has quit IRC06:40
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Add UEFI support  https://review.opendev.org/69615606:44
*** ociuhandu has joined #openstack-ironic07:05
*** ociuhandu has quit IRC07:09
*** dougsz has joined #openstack-ironic07:12
*** rachit7 has joined #openstack-ironic07:13
*** ociuhandu has joined #openstack-ironic07:19
*** ociuhandu has quit IRC07:25
*** rpittau|afk is now known as rpittau07:34
rpittaugood morning ironic! o/07:34
iurygregoryrpittau, o/07:39
rpittauhey iurygregory :)07:39
*** ociuhandu has joined #openstack-ironic07:52
*** ociuhandu has quit IRC07:57
*** lucasagomes has joined #openstack-ironic08:00
*** ociuhandu has joined #openstack-ironic08:01
*** tkajinam has quit IRC08:30
*** derekh has joined #openstack-ironic08:33
*** dtantsur|afk is now known as dtantsur08:34
dtantsurmorning ironic08:34
rpittauhey dtantsur :)08:38
dtantsurhey-hey! how was the weekend over there?08:38
rpittauvery quiet, full of pizza, pasta and alcohol :)08:40
dtantsurnot bad :) any opportunity for running? or no longer?08:41
rpittaufor now I'm not running, I don't want to risk injuries or a cold, at least until things don't settle down a little, hospitals in ile de france are overwhelmed08:43
rpittauapartment workout is fine, we don't have anyone leaving in the apartment below ours, so we can jump freely :D08:44
rpittaus/leaving/living08:44
rpittaudtantsur: how was over there ?08:44
openstackgerritDmitry Tantsur proposed openstack/ironic master: Bump hacking to 2.0.0  https://review.opendev.org/71560208:51
dtantsurheh, yeah we're also sticking to home workouts for now08:51
dtantsuroverall, things don't seem bad. but sitting at home is getting on nerves slowly.08:52
dtantsurthe weekend was mostly puzzle&cuddling :)08:52
openstackgerritRiccardo Pittau proposed openstack/ironic master: Remove future usage  https://review.opendev.org/71587908:53
rpittaudtantsur: I was checking the hacking change too! :D didn't see your patch, I'll review it08:53
dtantsurheh08:53
dtantsurrpittau: you can take ironic-inspector if you'd like to08:54
rpittauin general it's not too bad here, just worried about flour shortage08:55
rpittaudtantsur: sure :)08:55
dtantsurwe mostly have troubles with toilette paper :D08:55
rpittauhere that was bad at the very beginning, but now looks better :P08:56
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP] Use openstacksdk to interact with swift  https://review.opendev.org/71407408:56
*** k_mouza has joined #openstack-ironic08:59
*** rcernin has quit IRC09:00
*** ociuhandu has quit IRC09:03
openstackgerritDmitry Tantsur proposed openstack/ironic-lib master: Bump hacking to 2.0.0  https://review.opendev.org/71588209:05
dtantsurrpittau: ^^^ I'm checking libraries since they need a release soon09:05
rpittaudtantsur: alright, I will check the sushy side09:05
rpittaudtantsur: do we need to change lower-constraints ?09:06
dtantsurrpittau: I think hacking is an exception on constraints09:06
rpittauok, maybe remove it ?09:06
* dtantsur checks ironicclient09:06
dtantsuroh, we have it there?09:06
rpittauyeah09:07
rpittauthat's why I was asking09:07
dtantsurokay, updating for now09:07
openstackgerritDmitry Tantsur proposed openstack/ironic-lib master: Bump hacking to 2.0.0  https://review.opendev.org/71588209:07
rpittauI bumped it in ironic-inspector09:07
rpittauok09:07
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Bump hacking to 2.0.0  https://review.opendev.org/71588409:07
openstackgerritDmitry Tantsur proposed openstack/ironic master: Bump hacking to 2.0.0  https://review.opendev.org/71560209:07
rpittaummm weird no change in inspector09:08
*** alexmcleod has joined #openstack-ironic09:08
dtantsurw00t?09:08
rpittauyep09:08
dtantsursame in metalsmith, it doesn't even have hacking capped09:08
rpittau¯\_(ツ)_/¯09:08
dtantsurironicclient is... ugh09:09
*** ociuhandu has joined #openstack-ironic09:09
rpittauheh was not that lucky with sushy09:11
*** ociuhandu has quit IRC09:13
dtantsurI have a feeling that hacking<2.0 was ignoring most of standard checks.. so many things popped up09:17
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Bump hacking to 2.0.0  https://review.opendev.org/71588609:18
rpittauyep09:23
openstackgerritRiccardo Pittau proposed openstack/sushy master: Bump hacking to 2.0.0  https://review.opendev.org/71588809:24
*** uzumaki has joined #openstack-ironic09:24
uzumakimorning ironic o/09:25
dtantsurhey uzumaki09:27
rpittauhey uzumaki :)09:27
uzumakihow's it going? dtantsur rpittau09:27
rpittauwe make it go :)09:27
dtantsursurviving slowly :)09:27
uzumakirpittau, that sounds cool! especially given the fact that we all work on ironic, which does make things go ;)09:28
rpittausometimes we need to kick things though to make them go :D09:29
uzumakidtantsur, ah, same here, I really wish for things to get back to normal though..09:29
uzumakirpittau, not sure what exactly is this "kicking" but, nevermind xD09:29
rpittauuzumaki: ok, more like Fonzie bumping-jukebox style :)09:30
dtantsurback to normal is a dream, yes09:30
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: [WIP] Wholedisk+Uefi job  https://review.opendev.org/71445309:30
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: [WIP] Wholedisk+Uefi job  https://review.opendev.org/71445309:32
openstackgerritDmitry Tantsur proposed openstack/ironic-lib master: Bump hacking to 3.0.0  https://review.opendev.org/71588209:34
uzumakidtantsur, I... hope not :/09:36
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Bump hacking to 3.0.0  https://review.opendev.org/71588609:36
uzumakirpittau, not aware of Fonzie and his jukebox, so just watching it on youtube.. lol09:37
rpittauuzumaki: that's basic culture :P09:37
uzumakiwell, apologies for not being at par with the comtemporary culture, rpittau09:38
* dtantsur has no clue either09:38
uzumakiwell, it's a sitcom "The Fonz"09:38
uzumakithe guy has this thing where he slaps the jukebox to get it working09:38
rpittauit's from a 70s early 80s sitcom that was very famous in Italy09:39
openstackgerritDmitry Tantsur proposed openstack/ironic master: Bump hacking to 3.0.0  https://review.opendev.org/71560209:39
uzumakirpittau, given that information, well, not exactly contemporary for me, me being born in the 90s09:40
openstackgerritRiccardo Pittau proposed openstack/sushy-tools master: Bump hacking to 3.0.0  https://review.opendev.org/71589109:41
*** tkajinam has joined #openstack-ironic09:41
uzumakianybody has any idea if devstack supports making a cleaning network? I don't want to have to manually create it, and install network-baremetal09:42
openstackgerritRiccardo Pittau proposed openstack/sushy master: Bump hacking to 3.0.0  https://review.opendev.org/71588809:45
uzumakiis there a way to install the neutron-ironic-plugin through devstack itself?09:45
openstackgerritRiccardo Pittau proposed openstack/sushy-tools master: Bump hacking to 3.0.0  https://review.opendev.org/71589109:45
dtantsuruzumaki: things pretty much work out of box when it comes to VMs, but I'm not entirely sure if you hit any problems with a BM setup09:46
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Bump hacking to 3.0.0  https://review.opendev.org/71588409:46
uzumakiwell, my node can't seem to PXE boot with the IPA when it reboots09:46
openstackgerritRiccardo Pittau proposed openstack/sushy-cli master: Bump hacking to 3.0.0  https://review.opendev.org/71589909:51
*** uzumaki has quit IRC09:52
openstackgerritRiccardo Pittau proposed x/sushy-oem-idrac master: Bump hacking to 3.0.0  https://review.opendev.org/71590009:53
*** uzumaki has joined #openstack-ironic09:54
uzumakican i configure my IRC client to automatically fetch the chat logs for some x time from when I login?09:55
dtantsuruzumaki: in IRC there is no server-side storage of logs, unfortunately09:57
dtantsursome of us, myself included, are setting up their own IRC proxies to achieve that09:58
uzumakidtantsur, urghh.. that sucks, well I was wondering if anybody said anything on my question after your reply about VMs09:58
uzumakiI looked at the whiteboard and saw a whitepaper in progress, which looked really nice. Would anybody mind an extra set of hands on it? or eyes?10:01
arne_wiebalckuzumaki: of course not, have a look and make suggestions10:02
arne_wiebalckuzumaki: do you have the link?10:02
uzumakiI think I do, let me check10:03
* uzumaki sees The Future (Julia) and wonders is this declares the future to be Julia, or a future Julia10:09
dtantsur:D10:11
*** mkrai has quit IRC10:14
openstackgerritAndreas Jaeger proposed openstack/bifrost master: Update hacking for Python3  https://review.opendev.org/71561710:18
*** ociuhandu has joined #openstack-ironic10:21
dtantsurrpittau: are you doing IPA or should I?10:26
*** mkrai has joined #openstack-ironic10:27
openstackgerritDmitry Tantsur proposed openstack/ironic master: Fix the remaining hacking issues  https://review.opendev.org/71590710:29
dtantsurthis ^^ was pretty annoying, but I hope it's worth it10:29
*** mkrai has quit IRC10:36
*** mkrai_ has joined #openstack-ironic10:36
rpittaummm again connection issues?10:37
rpittaudtantsur: I can take care of it10:37
dtantsurthx!10:37
rachit7Hi dtantsur rpittau I have added you as a reviewer on my patch. Please add this patch into your review list. https://review.opendev.org/#/c/710852/10:38
patchbotpatch 710852 - ironic - DRAC: Fix a failure to create virtual disk bug - 5 patch sets10:38
dtantsurok10:38
rachit7thanks :)10:39
openstackgerritHarald Jensås proposed openstack/ironic master: Doc - IPv6 Provisioning  https://review.opendev.org/71531310:42
openstackgerritHarald Jensås proposed openstack/ironic master: Doc - IPv6 Provisioning  https://review.opendev.org/71531310:44
*** uzumaki has quit IRC10:50
*** dmellado has quit IRC10:50
*** yaawang has joined #openstack-ironic10:53
*** rpittau is now known as rpittau|bbl10:56
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent master: Bump hacking to 3.0.0  https://review.opendev.org/71591310:56
*** uzumaki has joined #openstack-ironic11:01
openstackgerritDmitry Tantsur proposed openstack/ironic master: Document deploy_boot_mode and boot_option for standalone deployments  https://review.opendev.org/71147211:43
arne_wiebalckTheJulia: I changed the partition size to comply with the ppc prepboot partition size as suggested by Tony ... and the Zuul was gracious to the patch: https://review.opendev.org/#/c/696156/11:48
patchbotpatch 696156 - ironic-python-agent - Software RAID: Add UEFI support - 9 patch sets11:48
arne_wiebalckdtantsur: iurygregory ^^11:49
dtantsurnice!11:49
iurygregoryarne_wiebalck, zuul is helping you a lot, what kind of pact you made with zuul? XD11:50
arne_wiebalckiurygregory: I speak to it.11:50
iurygregoryarne_wiebalck, =OOOO11:50
arne_wiebalckiurygregory: check my recheck :-D11:50
*** uzumaki has quit IRC11:50
iurygregoryhehehe nice11:51
*** mkrai_ has quit IRC11:51
openstackgerritMerged openstack/bifrost master: Update hacking for Python3  https://review.opendev.org/71561712:01
*** rh-jelabarre has joined #openstack-ironic12:04
*** dmellado has joined #openstack-ironic12:06
dtantsuroh, nice: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=7b9b816f4b9a12:20
*** uzumaki has joined #openstack-ironic12:21
*** uzumaki has quit IRC12:35
*** mkrai_ has joined #openstack-ironic12:43
*** jdandrea has joined #openstack-ironic12:45
*** rpittau|bbl is now known as rpittau12:46
*** ociuhandu has quit IRC12:48
*** uzumaki has joined #openstack-ironic12:59
*** ociuhandu has joined #openstack-ironic13:00
gudrutis2Hello Ironic. What is the preferred way to install pip packages ir IPA image? `package-installs.yaml` or script in `install.d`?13:02
dtantsurgudrutis2: diskimage-builder has a -p argument if I recall correctly13:03
dtantsurpackage-installs is preferred, but won't work with pip, I suspect13:03
uzumakiprimary suspects when node doesn't boot with ramdisk during cleaning?13:04
*** ociuhandu has quit IRC13:05
openstackgerritRiccardo Pittau proposed openstack/ironic-tempest-plugin master: Bump hacking to 3.0.0  https://review.opendev.org/71593813:06
*** ociuhandu has joined #openstack-ironic13:10
*** rh-jlabarre has joined #openstack-ironic13:12
*** irclogbot_0 has quit IRC13:13
*** rachit7 has quit IRC13:13
*** zbitter has quit IRC13:15
*** rh-jelabarre has quit IRC13:15
*** sziviani has quit IRC13:15
*** zaneb has joined #openstack-ironic13:16
*** sziviani has joined #openstack-ironic13:16
gudrutis2dtantsur: using script and using  $DIB_PYTHON_PIP is the prefered way as I see13:17
*** irclogbot_1 has joined #openstack-ironic13:18
openstackgerritRiccardo Pittau proposed openstack/sushy-tools master: Bump hacking to 3.0.0  https://review.opendev.org/71589113:18
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP: In-band deploy steps  https://review.opendev.org/69877013:18
dtantsurTIL13:18
openstackgerritRiccardo Pittau proposed openstack/sushy-tools master: Bump hacking to 3.0.0  https://review.opendev.org/71589113:19
*** mkrai_ has quit IRC13:21
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876813:21
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830913:21
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: WIP: In-band deploy steps  https://review.opendev.org/69877013:22
*** cdearborn has joined #openstack-ironic13:24
*** tzumainn has joined #openstack-ironic13:26
*** ociuhandu has quit IRC13:27
*** dking_desktop has joined #openstack-ironic13:30
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Bump hacking to 3.0.0  https://review.opendev.org/71594313:36
*** Goneri has joined #openstack-ironic13:40
*** rloo has joined #openstack-ironic13:48
rpiosouzumaki: Have you been able to deploy?13:48
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Wholedisk+Uefi job  https://review.opendev.org/71445313:51
*** khansa has joined #openstack-ironic13:52
openstackgerritDerek Higgins proposed openstack/ironic master: WIP ipv6 jobs  https://review.opendev.org/61316913:53
uzumakirpioso, well, my server has been PXE booting on the wrong NIC port all along (facepalm) fixing that now13:54
rpiosouzumaki: No worries. It's a common baremetal issue.13:55
openstackgerritMerged openstack/ironic master: Remove future usage  https://review.opendev.org/71587914:03
*** dmellado has quit IRC14:04
*** dmellado has joined #openstack-ironic14:05
*** Goneri has quit IRC14:11
*** Goneri has joined #openstack-ironic14:14
*** beekneemech is now known as bnemec14:15
*** TxGirlGeek has joined #openstack-ironic14:21
*** rloo has quit IRC14:22
*** rloo has joined #openstack-ironic14:23
openstackgerritMerged openstack/ironic master: Document deploy_boot_mode and boot_option for standalone deployments  https://review.opendev.org/71147214:24
TheJuliarpioso: I would appreciate it if you could take a look at https://review.opendev.org/#/c/710751/.  You have a -1 on the patch and I feel like the question you raised has been answered. This is non-client library release freeze week for the openstack project, so we need to move forward or the patch will need to wait14:27
patchbotpatch 710751 - sushy - Add `set_system_boot_options` method - 4 patch sets14:27
openstackgerritRiccardo Pittau proposed openstack/sushy master: [trivial] add reason why we skip W503 in pep8 check  https://review.opendev.org/71595414:29
openstackgerritJulia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local  https://review.opendev.org/49211414:30
rpiosoTheJulia: We've been testing it.14:30
rpiosocdearborn: ^^^14:30
*** jerry has quit IRC14:31
*** jerry1 has joined #openstack-ironic14:31
openstackgerritRiccardo Pittau proposed openstack/ironic-tempest-plugin master: Bump hacking to 3.0.0  https://review.opendev.org/71593814:32
TheJuliarpioso: Okay, You realize that we're not going to be able to realistically wait on manual testing results beyond the next couple of days?14:32
iurygregoryTheJulia, good morning o/ , do you want a doc with the list of the tests for each job or only an update in https://docs.openstack.org/ironic-tempest-plugin/latest/usage.html saying that we run the tests in two jobs instead of one (I'm going to update the old job names in usage)14:32
TheJuliaWe'll need to confirm with etingof that he didn't have any issues testing against the servers we have in the lab14:32
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Bump hacking to 3.0.0  https://review.opendev.org/71594314:33
*** khansa has quit IRC14:33
TheJuliaiurygregory: something abstract would be good, I think so we don't HAVE to revisit the doc with every job14:33
TheJuliadtantsur: of course I thought https://review.opendev.org/#/c/492114/ had merged somewhere in the last few weeks :(14:33
patchbotpatch 492114 - ironic - Change [deploy]/default_boot_option to local - 28 patch sets14:33
dtantsurouch14:34
dtantsurgood morning TheJulia14:34
dtantsurTheJulia: let's wait for the CI (fingers crossed) and move on with it14:34
*** ociuhandu has joined #openstack-ironic14:34
rpiosoTheJulia: Ack14:35
*** mkrai has joined #openstack-ironic14:35
TheJuliadtantsur: yeah, it has passed it a half dozen times easily, but the standalone raid jobs have been problematic :(14:35
TheJulialol https://038bc1dad3c7113252fa-ce8f61360280bb66a516430c13b1c7a2.ssl.cf1.rackcdn.com/706632/11/check/ironic-standalone-ipa-src/f368035/testr_results.html14:35
iurygregoryI would ask arne_wiebalck to use the recheck magic on zuul14:37
TheJuliait is the raid setup timeout14:37
TheJuliawell14:37
TheJulianot that, but the command timeout14:37
TheJuliathe command is a sync call14:37
TheJuliathe steps take >60 seconds in test VMs :(14:37
TheJuliawhich explains the multiple commands14:37
TheJuliaUGH14:37
TheJuliaThat is why it fails14:37
TheJuliawell, one of the multiple reasons14:38
iurygregoryouch more than 60sec? D:14:38
TheJuliaYeah, 46-74 seconds is what I was seeing when digging through logs last week14:38
dtantsurwow14:38
TheJuliathis past week feels like land of bugs for agent<->ironic communication14:39
dtantsurbugs, bugs everywhere14:39
openstackgerritRiccardo Pittau proposed openstack/sushy-tools master: Bump hacking to 3.0.0  https://review.opendev.org/71589114:42
TheJuliayup, agent logs are returning >60 seconds to show failed14:42
TheJuliaYay! an after meeting bug hunt14:43
*** kaifeng_ has joined #openstack-ironic14:43
openstackgerritRiccardo Pittau proposed openstack/networking-generic-switch master: Bump hacking to 3.0.0  https://review.opendev.org/71597014:49
*** ociuhandu has quit IRC14:52
accakksHi everyone, I'm Aakanksha,  computer science student from India and Outreachy applicant interested in project  "Add Time Series Support to Ironic Inspector Data Collections".14:52
accakksCurrently reading up, completing installations etc. Hope to make my first contribution very soon.14:52
openstackgerritRiccardo Pittau proposed openstack/metalsmith master: Bump hacking to 3.0.0  https://review.opendev.org/71597114:52
*** iurygregory has quit IRC14:54
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: Bump hacking to 3.0.0  https://review.opendev.org/71594314:55
openstackgerritRiccardo Pittau proposed openstack/networking-generic-switch master: Bump hacking to 3.0.0  https://review.opendev.org/71597014:56
*** iurygregory has joined #openstack-ironic14:56
dtantsurhi accakks, welcome14:56
rpittauhey accakks :)14:57
TheJuliaWelcome accakks!14:59
arne_wiebalckHey accakks ;)14:59
rpiosoWelcome to ironic, accakks!14:59
iurygregoryo/15:00
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Mar 30 15:00:39 2020 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
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
kaifeng_o/15:00
rpittauo/15:00
erbarro/15:00
rpioso\o15:00
mgoddard\o15:00
arne_wiebalcko/15:01
ajyao/15:01
iurygregoryo/15:01
rlooo/15:01
cdearborno/15:01
TheJuliaGood morning everyone!15:01
TheJuliaOur agenda can be found on the wiki as always!15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
bdoddo/15:01
TheJulia#topic Announcements / Reminders15:01
rajiniro/15:01
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:01
dtantsuro/15:01
TheJulia#info Upcoming breaking changes in pip, and test-requirements.txt not installed anymore in devstack.15:02
TheJulia#link http://lists.openstack.org/pipermail/openstack-discuss/2020-March/013642.html15:02
openstackgerritDmitry Tantsur proposed openstack/ironic master: Bump hacking to 3.0.0  https://review.opendev.org/71560215:02
openstackgerritDmitry Tantsur proposed openstack/ironic master: Fix the remaining hacking issues  https://review.opendev.org/71590715:02
rpittauthis might (hopefully not) cause issues in CI15:02
TheJulia#info This week is the week of the non-client library release freeze15:03
accakkso/15:03
dtantsurnot installing test-requirements is probably good news15:03
mgoddardTheJulia: do we need releases?15:03
TheJuliaThis means sushy and ironic-lib need to be released for the cycle this week.15:03
dtantsurand metalsmith if needed15:03
rpittauit is, just keep all the possible eyes on the CI for possible breakage15:03
dtantsurand does anyone know if sushy-cli is considered a client library?15:03
TheJuliamgoddard: I think holding off at the moment, ther eis a patch we want to get in on sushy15:03
TheJuliadtantsur: client15:03
dtantsurokay, so one more week15:04
TheJuliadtantsur: or a tool15:04
dtantsurironic-lib will only need the hacking patch15:04
TheJulia++15:04
rpittaudtantsur: client15:04
dtantsurmetalsmith is likely fine, but needs double-checking15:04
mgoddardwith a bit of luck, this tells us: https://docs.openstack.org/ironic/latest/contributor/releasing.html15:04
TheJulia#info Next week is client libraries!15:04
mgoddardironic-lib, metalsmith, sushy15:04
rlooso nice to have documentation :)15:05
iurygregory++15:05
TheJuliaAlso next week, I need to write cycle highlights. :(15:05
dtantsurTheJulia: need help with that?15:05
TheJuliaso merge anything good and noteworthy! :)15:05
TheJuliadtantsur: Should be relatively easy. :)15:05
dtantsurokay, ping me if anything is needed :)15:05
TheJuliaDoes anyone have anything else to announce or remind us of this week?15:06
dtantsurreminder: be nice to yourself and wash your hands15:06
TheJulia++15:06
iurygregoryit's ptl season on openstack-discuss15:07
TheJulia#info Everyone is under stress: Be nice to them, yourself, and wash your hands!15:07
TheJuliaYes, nominations close tomorrow I believe15:07
*** yoctozepto has quit IRC15:07
TheJulia(I'm still pondering, if anyone is wondering)15:08
dtantsurWe have been wondering, yes :)15:08
TheJulia:)15:08
iurygregoryyup15:08
*** yoctozepto has joined #openstack-ironic15:08
rloothx for pondering TheJulia!15:08
TheJulia:)15:08
TheJuliaheh15:08
TheJuliawe should move on!15:08
*** ab-a has quit IRC15:08
iurygregory++15:09
TheJulia#topic Review action items from the previous meeting.15:09
*** openstack changes topic to "Review action items from the previous meeting. (Meeting topic: ironic)"15:09
TheJuliaWe had two action items, one to send out a doodle for the baremetal sig whitepaper call, and another iurygregory was going to follow-up with HPE ilo CI team.15:09
TheJuliaI believe both were done15:09
iurygregoryyup =)15:09
dtantsurwe need a follow-up for the whitepaper now?15:10
iurygregory3 of 5 jobs at least are reporting logs15:10
TheJuliadtantsur: why yes, we do!15:10
TheJulia#action We need to meet again this week to re-visit the whitepaper.15:10
TheJulia#undo15:10
openstackRemoving item from minutes: #action We need to meet again this week to re-visit the whitepaper.15:10
TheJulia#action TheJulia to send out another doodle for the baremetal whitepaper15:10
TheJuliaAre we good to proceed to subteam status reports?15:11
*** ab-a has joined #openstack-ironic15:11
iurygregory++15:13
TheJulia#topic Review subteam status reportss15:13
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:13
TheJuliaStarting at line 25215:13
*** openstack changes topic to "Review subteam status reportss (Meeting topic: ironic)"15:13
TheJuliaarne_wiebalck: I don't remember, were you or someone from cern going to write something regarding conductor groups performance and all ?15:13
arne_wiebalckTheJulia: yes, once we are finished15:13
TheJuliak15:13
arne_wiebalckTheJulia: we will go with 500 nodes per group15:14
arne_wiebalckTheJulia: but we still have to do the split15:14
arne_wiebalckTheJulia: I will update the etherpad once we have more datat15:14
arne_wiebalckdata15:14
TheJuliaarne_wiebalck: awesome15:15
TheJuliadtantsur: How are you feeling w/r/t deploy steps this cycle?15:15
dtantsurI'm trying :)15:15
dtantsurI feel quite confident we can land the foundational bits15:15
TheJuliaAwesome15:15
dtantsurI'm less sure about the actual final split of the deploy.deploy step15:15
TheJuliaThat is a lot to try and lan right now :\15:16
TheJulialand15:16
dtantsurexactly15:16
TheJuliaetingof is out today so no update on dhcp-less deployments but it looks like he has been focusing on rebasing/updating this past week so that is good15:16
arne_wiebalckretirement is done I think and can be removed15:16
TheJulia++15:17
dtantsur* can be retired!15:17
arne_wiebalcklol15:17
TheJuliaI was just going to ask15:17
TheJulialol15:17
arne_wiebalckUEFI s/w RAID needs some more reviews to finally merge15:17
TheJulia++15:17
TheJuliaAny core reviewers have 20 minutes, please review the UEFI s/w RAID patch. It also conflicts with dtantsur's device selection patch, so the sooner ONE of them merges, the faster we can get the other merged.15:18
TheJuliaiurygregory, rpioso: anything left on the python3 task list ?15:20
TheJuliaor should we consider that done for the cycle at line 36115:20
rpittauTheJulia: sushy-tools six removal15:20
iurygregoryI was about to say =)15:20
rpittauand tenks15:20
rpittauI added a comment on the etherpad15:20
rpittauand rechecked the patch15:20
TheJuliaAt this point, I suspect six removal can be punted to next cycle15:21
rpittaufor sushy-tools we probably can move forward15:21
rpittauok15:21
TheJuliayeah, if the review is in place, put it on the patches to review this week15:21
TheJuliafor sushy-tools15:21
rpittauerm s/sushy-tools/tenks15:21
TheJuliaoh wait15:21
TheJuliait is not branched, so that is going to break people15:21
TheJuliaNext cycle is sounding much better15:21
rpittauyeah, I meant tenks :)15:21
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Split Tempest tests between ipmi and redfish  https://review.opendev.org/71204115:21
TheJuliamgoddard: do you feel okay with that moving forward with tenks?15:22
TheJuliaWhile waiting, next item is the grenade zuul3 changes. iurygregory Are we still waiting on devstack/grenade changes to merge?15:23
iurygregoryTheJulia, yes15:23
TheJulia*sigh*15:23
iurygregoryluigi still doing some work on the grenade side15:23
iurygregorythis will be a goal for V so I assume they will merge in the begin o the V cycle ...15:23
TheJuliaThank you everyone for the recent CI improvement patches and bug fixes by the way.15:24
TheJuliaI'm good, is everyone else good with proceeding?15:25
mgoddardTheJulia: sorry, had to step away for parenting15:25
mgoddardAs tenks is not branched, and we use it in environments without good py3 support I think we should keep six around for now15:25
mgoddardProceed :)15:25
*** ociuhandu has joined #openstack-ironic15:26
*** ab-a has quit IRC15:26
TheJuliamgoddard: okay, thanks15:27
tzumainnTheJulia, hi! regarding node ownership - I'm having difficulty with the suggested revisions to one of the patches; I was hoping that I'd be able to talk to someone about them at some point... ?15:27
tzumainnI left comments on the patch as well15:27
TheJuliatzumainn: How about a little later in our meeting?15:27
TheJuliaor after?15:28
TheJulia#topic Deciding on Priorities for the week15:28
*** openstack changes topic to "Deciding on Priorities for the week (Meeting topic: ironic)"15:28
tzumainnTheJulia, that would be great, thanks!15:28
tzumainnafter would be fine15:28
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:28
TheJuliaLine 12215:28
TheJuliaLots of merged things, so give me a moment to clean those up15:29
TheJuliaWhile doing so, could someone check the client libraries for open patches that make sense and add them to the list?15:29
*** ab-a has joined #openstack-ironic15:30
iurygregorysomething like https://review.opendev.org/#/q/status:open+project:openstack/sushy ?15:30
TheJuliaI was thinking python-ironicclient python-ironic-inspector-client15:30
TheJuliaI checked sushy and I think we already have the one listed15:31
*** mkrai has quit IRC15:32
*** dmellado has quit IRC15:32
TheJuliaplease add a little bit of a description to the item when adding links15:32
TheJuliadtantsur: any more deploy steps patches for the list?15:32
iurygregoryironic-inspector-client has 0 patches open15:33
dtantsurTheJulia: everything is WIP atm15:33
*** jgwentworth is now known as melwitt15:33
kaifeng_the inspection scope may need a client patch, but not up yet15:34
*** ociuhandu has quit IRC15:34
TheJuliakaifeng_: I thought it didn't....15:34
TheJuliabut it has been a while since I reviewed that one last15:34
TheJuliaLooks like we got a lot merged last week15:34
*** iurygregory has quit IRC15:35
dtantsureven I'm not sure, and I wrote most of it :)15:35
*** ociuhandu has joined #openstack-ironic15:35
*** iurygregory has joined #openstack-ironic15:35
TheJuliaIs everyone good with the list of patches there? Are there any patches we're missing?15:36
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Split Tempest tests between ipmi and redfish  https://review.opendev.org/71204115:36
dtantsurlooks ok15:36
iurygregoryyeah15:37
TheJuliacdearborn: rpioso: fyi, https://review.opendev.org/#/c/710327/ is on the list under the idrac driver and it is a sushy-oem-idrac patch, so ironic-core can't approve it to merge.15:37
patchbotpatch 710327 - x/sushy-oem-idrac - Export System Configuration Added code for exporti... - 1 patch set15:37
TheJuliaShall we move on?15:38
dtantsuryep15:38
iurygregoryyes15:38
TheJuliaMoving on then!15:39
TheJulia#topic Discussion15:39
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:39
TheJuliaFirst and only listed topic of the week on the agenda is I believe rpittau's15:39
* TheJulia hands the microphone to rpittau 15:39
*** dmellado has joined #openstack-ironic15:39
*** mkrai has joined #openstack-ironic15:40
TheJuliaI guess I should do my best rpittau impression!15:40
iurygregorypackage lost...15:40
TheJuliaSo we've long been having an issue with the native OVMF package in bionic where TinyIPA machines freeze/fail to boot.15:40
TheJulia#link https://bugs.launchpad.net/ubuntu/+source/edk2/+bug/182172915:41
openstackLaunchpad bug 1821729 in edk2 (Ubuntu Cosmic) "UEFI in ovmf package causes kernel panic" [Undecided,Invalid]15:41
TheJuliaFrom my understanding, the the belief on the OVMF maintainer is that it is tinycore's kernel that is at fault for the crash15:41
TheJuliawhich means we need to figure out our forward path15:42
TheJuliaI think rpittau is hoping that going to tinycore 11 will fix this, but I think it is time we discuss as a community what our next step should/could be for UEFI testing15:43
TheJulia#link https://review.opendev.org/#/c/711647/15:43
patchbotpatch 711647 - ironic-python-agent-builder - [WIP] Use tinycore 11 - 4 patch sets15:43
openstackgerritMerged openstack/sushy master: Bump hacking to 3.0.0  https://review.opendev.org/71588815:43
dtantsurwill it help that we're switching the CI to DIB?15:43
TheJuliaI'm personally wondering if we just need to move UEFI to dib based...15:43
TheJuliadtantsur: likely!15:44
dtantsurproblem solved? :)15:44
TheJuliaSo it all may become moot in the end15:44
iurygregory++ for dib15:44
*** ab-a has quit IRC15:45
*** rajinir has quit IRC15:45
*** donnyd has quit IRC15:45
*** andrein has quit IRC15:45
*** vdrok has quit IRC15:45
*** rpittau has quit IRC15:45
*** Anticimex has quit IRC15:45
TheJuliaOkay then, well that seems to solve it, if there are no objections I think it is time to move to Open Discussion15:45
*** ab-a has joined #openstack-ironic15:45
*** rajinir has joined #openstack-ironic15:45
*** donnyd has joined #openstack-ironic15:45
*** andrein has joined #openstack-ironic15:45
*** vdrok has joined #openstack-ironic15:45
*** rpittau has joined #openstack-ironic15:45
*** Anticimex has joined #openstack-ironic15:45
*** tepper.freenode.net changes topic to "Deciding on Priorities for the week (Meeting topic: ironic)"15:45
TheJuliayay for netsplits15:45
iurygregorywow15:45
dtantsurOo15:45
TheJuliaIt is a thing15:46
TheJulia#topic Open Discussion15:46
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:46
TheJuliatzumainn: o/15:46
TheJuliaif you want to discuss the issue your encountering?15:46
tzumainnTheJulia, hi! so I'm having issues with the suggested revisions on https://review.opendev.org/#/c/708678/15:46
patchbotpatch 708678 - ironic - Use auth values from neutron conf when managing Ne... - 2 patch sets15:46
rpittausplit all the things15:46
rpittaudid we get to talk about the ovmf issue ?15:47
iurygregoryrpittau, we did15:47
TheJuliarpioso: yeah, we did, the feeling is dib will fix everything15:47
rpittauehm without me? :D15:47
tzumainnfor context, the suggested changes were to use openstacksdk, and to further split apart the port update operation to limit which fields were being updated through the admin15:47
iurygregorythe ping probably got lost =D15:47
TheJuliarpioso: you netsplit15:47
iurygregoryrpittau, tldr switch to dib XD15:47
TheJuliatzumainn: well, the fact openstacksdk doesn't support updating the mac is problematic and a solid reason to hold off on trying to use it for now15:48
rpiosoTheJulia: I'm not rpittau :-)15:48
TheJuliarpioso: doh!15:48
rpiosorpittau: o/15:48
*** accakks has quit IRC15:48
tzumainnTheJulia, okay! the second issue makes me wonder if it makes sense to isolate the update_port calls into a common/neutron.py method, and then... somehow checking that the user has the appropriate permissions on the affected network?15:49
*** mkowalski[b] has quit IRC15:49
tzumainnbecause essentially checking field-by-field, operation-by-operation feels delicate to me, like a single accident could cause unintended exposure15:50
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Add ironic-standalone-redfish  https://review.opendev.org/71204215:50
TheJuliaThat does seem problematic. hjensas, you know the neutron stuff really well. Any thoughts here because we don't want to become an engine to permit unauthorized port bindings to networks peopel shouldn't be touching because we need to update just a couple of the fields as an privilged account and then likely should step our access back down for the final requested action15:53
tzumainnthe reason I think that doing this field-by-field doesn't make sense, is that it looks like some of the update operations only update privileged fields15:54
TheJuliaI guess I'm sitting here wondering how port binding was working when we passed the context previouslly, but then the mac still required us to do it directly.... or was this a feature that only ever worked with Nova was infront of ironic?15:55
TheJuliaor the user was an admin user?15:55
*** hjensas has quit IRC15:57
*** dsneddon has joined #openstack-ironic15:58
TheJuliaheh, well there goes the hope on that thought16:00
TheJuliaI guess we're done with the meeting for now, tzumainn I think this discussion is going to continue.16:00
tzumainnTheJulia, that's fine, thanks!16:00
TheJulia#endmeeting16:02
*** 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."16:02
openstackMeeting ended Mon Mar 30 16:02:12 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:02
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-03-30-15.00.html16:02
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-03-30-15.00.txt16:02
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-03-30-15.00.log.html16:02
TheJuliaSo yeah, they wer ealways admins16:05
TheJuliaugh16:05
tzumainnTheJulia, I'm going to try a refactored solution that I *think* might address the security concerns, it might make it easier to see what I'm trying to do (and for people to tell me what I might be doing wrong)16:05
TheJulia++16:06
tzumainnTheJulia, I didn't get a chance to ask, but if the node lessee patch gets merged it would require an update to python-ironicclient - should that be listed somewhere?16:06
TheJuliaThat sounds like a good idea. I suspect that maybe checking if the project is sane and matches then being able to proceed is sufficent16:06
TheJuliatzumainn: we have a list of patches to review on the ironic etherpad16:06
TheJuliahttps://etherpad.openstack.org/p/IronicWhiteBoard16:07
tzumainnTheJulia, yep - I haven't added the python-ironicclient yet because the node lessee patch hasn't merged, so I haven't submitted the client patch16:07
tzumainnbut I wasn't positive from the discussion whether not-yet-submitted patches should be listed somewhere... ?16:07
TheJuliatzumainn: you'll want to get that client patch up relatively soon, fwiw16:08
tzumainnTheJulia, should I submit it before the node lessee patch is merged?16:08
TheJuliatzumainn: yes16:08
tzumainnTheJulia, okay, I'll do that and put it on the etherpad - thanks!16:08
TheJuliaJust add a Depends-On label16:08
tzumainngotcha16:09
TheJulia++16:09
TheJuliaerr, that was for another message in the wrong window16:09
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM] test ovmf manual upgrade  https://review.opendev.org/69230916:10
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM] test ovmf default package with dib  https://review.opendev.org/71203316:10
rpittauTheJulia, iurygregory I already tested ovmf with DIB ^^ with not good results, I rebased it just to give it another change16:11
rpittaus/change/chance16:11
iurygregoryrpittau, ouch D:16:11
rpittauanyway, I'll give it another try locally tomorrow, time to go zombie hunting (groceries shopping)16:12
rpittaugood night! o/16:12
*** rpittau is now known as rpittau|afk16:12
iurygregorygn rpittau|afk16:12
*** mkrai has quit IRC16:15
openstackgerritTzu-Mainn Chen proposed openstack/python-ironicclient master: Add node lessee  https://review.opendev.org/71601116:17
*** lucasagomes has quit IRC16:17
openstackgerritMerged openstack/sushy-cli master: Bump hacking to 3.0.0  https://review.opendev.org/71589916:17
openstackgerritJulia Kreger proposed openstack/ironic master: Set agent command timeout to 120 for standalone jobs  https://review.opendev.org/71601316:23
arne_wiebalckbye, everyone o/16:33
TheJuliao/16:33
*** hjensas has joined #openstack-ironic16:39
*** Anticimex has quit IRC16:43
*** Anticimex has joined #openstack-ironic16:44
*** mkowalski[b] has joined #openstack-ironic16:48
openstackgerritJulia Kreger proposed openstack/ironic master: Extend install_bootloader command timeout  https://review.opendev.org/71602116:49
TheJuliaso hopefully, this will help sort out more of our fun issues around software raid testing in ci16:56
uzumakimorning TheJulia o/16:57
dtantsurTheJulia: we should make install_bootloader and asynchronous deploy step16:58
dtantsureventually16:58
TheJuliadtantsur: yeah...16:59
dtantsurTheJulia: re https://review.opendev.org/#/c/716021/ do we want to update the description of the configuration option to mention that some commands may use a larger timeout?16:59
patchbotpatch 716021 - ironic - Extend install_bootloader command timeout - 1 patch set16:59
TheJulianot an end of the cycle change right now with how many issues we hit in that area16:59
dtantsurwell, we don't have in-band deploy steps yet at all :)16:59
TheJuliadtantsur: I was thinking of that right after I uploaded it16:59
TheJuliaI can alway sofllow-up16:59
dtantsurif you update it, I can quickly re-review before I leave for a day16:59
TheJuliadtantsur: I can do it in a couple minutes16:59
dtantsurcool17:00
openstackgerritJulia Kreger proposed openstack/ironic master: Extend install_bootloader command timeout  https://review.opendev.org/71602117:04
TheJuliadtantsur: done17:05
dtantsurthanks, +217:06
dtantsurtime to go now, see you tomorrow17:06
*** dtantsur is now known as dtantsur|afk17:06
dtantsur|afko/17:06
TheJuliaGoodnight17:07
*** derekh has quit IRC17:09
*** alexmcleod has quit IRC17:12
*** k_mouza has quit IRC17:21
*** ociuhandu has quit IRC17:24
openstackgerritJulia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local  https://review.opendev.org/49211417:30
openstackgerritJulia Kreger proposed openstack/ironic master: Add warning for deprecated get_default_boot_option  https://review.opendev.org/71603017:30
*** ociuhandu has joined #openstack-ironic17:59
*** k_mouza has joined #openstack-ironic18:19
*** k_mouza has quit IRC18:20
*** kaifeng_ has quit IRC18:32
*** ociuhandu has quit IRC18:48
*** ociuhandu has joined #openstack-ironic19:25
*** ociuhandu has quit IRC19:29
*** dougsz has quit IRC19:44
openstackgerritChristopher Dearborn proposed openstack/ironic master: Fix issue where server fails to reboot  https://review.opendev.org/71449119:55
*** k_mouza has joined #openstack-ironic20:20
*** k_mouza has quit IRC20:25
*** cdearborn has quit IRC20:54
*** rcernin has joined #openstack-ironic21:56
*** k_mouza has joined #openstack-ironic22:27
*** k_mouza has joined #openstack-ironic22:28
*** k_mouza has quit IRC22:33
openstackgerritTzu-Mainn Chen proposed openstack/ironic master: Use auth values from neutron conf when managing Neutron ports  https://review.opendev.org/70867823:25
*** rh-jlabarre has quit IRC23:53
*** rloo has quit IRC23:54

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!