Monday, 2020-05-18

« Sunday, 2020-05-17 Index Tuesday, 2020-05-19 »
*** stevebaker has quit IRC00:08
*** ociuhandu has joined #openstack-ironic00:22
*** bdodd has joined #openstack-ironic00:22
*** ociuhandu has quit IRC00:29
openstackgerritBrin Zhang proposed openstack/ironic-lib master: Bump hacking min version to 3.0.1  https://review.opendev.org/72872200:37
openstackgerritBrin Zhang proposed openstack/ironic-inspector master: Bump hacking min version to 3.0.1  https://review.opendev.org/72872300:37
openstackgerritBrin Zhang proposed openstack/ironic master: Bump hacking min version to 3.0.1  https://review.opendev.org/72872400:39
*** threestrands has joined #openstack-ironic00:43
openstackgerritBrin Zhang proposed openstack/ironic-python-agent master: Bump hacking min version to 3.0.1  https://review.opendev.org/72872500:43
*** stevebaker has joined #openstack-ironic00:43
openstackgerritBrin Zhang proposed openstack/ironic-tempest-plugin master: Bump hacking min version to 3.0.1  https://review.opendev.org/72872600:44
openstackgerritwangwei1 proposed openstack/ironic-prometheus-exporter master: Add coverage configuration for tox  https://review.opendev.org/72839200:49
openstackgerritBrin Zhang proposed openstack/ironic-lib master: Bump hacking min version to 3.0.1  https://review.opendev.org/72872201:34
openstackgerritBrin Zhang proposed openstack/ironic master: Fix hacking min version to 3.0.1  https://review.opendev.org/72872401:37
openstackgerritBrin Zhang proposed openstack/ironic-tempest-plugin master: Fix hacking min version to 3.0.1  https://review.opendev.org/72872601:37
openstackgerritBrin Zhang proposed openstack/ironic-python-agent master: Bump hacking min version to 3.0.1  https://review.opendev.org/72872501:40
openstackgerritBrin Zhang proposed openstack/ironic-inspector master: Bump hacking min version to 3.0.1  https://review.opendev.org/72872301:41
*** Qianbiao has joined #openstack-ironic01:53
*** ricolin has joined #openstack-ironic02:06
openstackgerritQianBiao Ng proposed openstack/ironic master: Feature: Add raid configuration support for ibmc driver  https://review.opendev.org/72812302:09
*** Qianbiao is now known as Qianbiao|afk02:10
*** xiaolin has quit IRC02:38
*** Qianbiao|afk has quit IRC03:01
*** Qianbiao|afk has joined #openstack-ironic03:35
*** ociuhandu has joined #openstack-ironic03:49
*** ociuhandu has quit IRC03:59
*** ociuhandu has joined #openstack-ironic04:01
*** ociuhandu has quit IRC04:06
*** Qianbiao|afk has quit IRC05:02
*** ociuhandu has joined #openstack-ironic05:17
*** ociuhandu has quit IRC05:23
*** xiaolin has joined #openstack-ironic05:55
*** ricolin has quit IRC06:08
*** ricolin has joined #openstack-ironic06:17
*** ricolin_ has joined #openstack-ironic06:18
*** ricolin_ has quit IRC06:18
*** Qianbiao|afk has joined #openstack-ironic06:36
arne_wiebalckGood morning, ironic!06:49
openstackgerritQianBiao Ng proposed openstack/ironic master: Feature: Add raid configuration support for ibmc driver  https://review.opendev.org/72812306:49
*** ccstone has quit IRC06:50
*** ccstone has joined #openstack-ironic06:50
*** gkadam has joined #openstack-ironic07:04
*** gkadam has quit IRC07:05
*** yolanda has joined #openstack-ironic07:07
*** dougsz has joined #openstack-ironic07:08
*** iurygregory has joined #openstack-ironic07:17
iurygregorygood morning Ironic07:17
*** rpittau|afk is now known as rpittau07:29
rpittaugood morning ironic! o/07:29
*** cp- has quit IRC07:31
*** cp- has joined #openstack-ironic07:32
iurygregorymorning rpittau o/07:39
rpittauhey iurygregory :)07:39
*** lucasagomes has joined #openstack-ironic07:45
openstackgerritRiccardo Pittau proposed openstack/ironic master: Collect tftpd info from journald  https://review.opendev.org/71774507:46
gudrutis2Morning Ironic07:47
iurygregorymorning gudrutis207:47
iurygregoryrpittau, I thought we were already using hacking 3.0.1 in all projects no? .-.07:48
iurygregoryasking because of https://review.opendev.org/#/c/72872307:48
patchbotpatch 728723 - ironic-inspector - Bump hacking min version to 3.0.1 - 2 patch sets07:48
rpittauiurygregory: the current lowest version is 3.0.0, which is actually fine for us since we fixed all the issues07:49
iurygregoryrpittau, ok cool07:49
iurygregory=)07:49
rpittauit doesn't hurt to merge all those patches, but I don't see it as a priority right now07:50
iurygregoryyeah07:50
iurygregoryI was wondering if CI was broken hehehe07:50
rpittauit doesn't look so, I was actually surprised today to see all my patches with +1 from zuul! :)07:50
* iurygregory is doing recheck on some patches hehehe07:51
iurygregorywow networking-baremetal using openstacksdk =O07:52
rpittauyeah, harald got that working :)07:52
iurygregorysince this is trivial https://review.opendev.org/#/c/725918/ I will +A anyone against?07:53
patchbotpatch 725918 - ironic - [Trivial] Fix awkward mocking in redfish boot tests - 3 patch sets07:53
rpittauiurygregory: that's fine, go ahead :)07:53
hjensasrpittau: I'm looking into using tenacity now, and the opts. Do you know if we can do 'noauth' with openstacksdk?07:54
iurygregoryhjensas, I think we can07:54
rpittauhjensas: we should be able to, yeah07:54
iurygregorywhat can happen is that we are missing something07:55
iurygregoryat least there are noauth tests and ironic is tested =)07:55
hjensasok, I will dig into noauth and the other opts as well then.07:57
openstackgerritRiccardo Pittau proposed x/sushy-oem-idrac master: Update lower-constraints.txt  https://review.opendev.org/72726007:57
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Reduce timeout for tempest job  https://review.opendev.org/72863508:00
*** threestrands has quit IRC08:13
openstackgerritRiccardo Pittau proposed openstack/ironic stable/train: Revert "Add timeout and retries to JSON RPC client"  https://review.opendev.org/72878408:38
Qianbiao|afkhello ironic08:39
*** Qianbiao|afk is now known as Qianbiao08:39
QianbiaoHello, i find redfish Specification do ask add if-match header when send "put|patch" request.08:40
Qianbiaowhy sushy did not implement this feature.08:41
iurygregoryhey Qianbiao08:43
Qianbiaohi iurygregory o/08:43
iurygregorydo you have a link for that?08:43
iurygregoryI think I've found http://redfish.dmtf.org/schemas/DSP0266_1.6.1.html08:45
iurygregoryETags section?08:45
Qianbiaohttps://www.dmtf.org/sites/default/files/standards/documents/DSP0266_1.10.0.pdf08:45
*** sshnaidm|off has joined #openstack-ironic08:47
*** sshnaidm|off is now known as sshnaidm08:47
iurygregoryseems like we didn't have time to implement that I would say =)08:48
Qianbiao:)08:48
Qianbiaothis is wierd, if so, why sushy can access idrac, ilo, etc.08:49
iurygregorywell redfish has problems =) and also it will depend if the hardware vendor have this as required in their side also ( this is what I think)08:51
Qianbiaoiurygregory lol, first, I thought maybe ibmc did not follow the redfish specification.08:58
QianbiaoSo, i check the dmtf doc.08:58
iurygregoryQianbiao, well we have a lot of different troubles with different hardwares =)08:58
rpittauQianbiao: sushy actually has an implementation to retrieve and check the etag of the resource08:58
iurygregoryQianbiao, https://etherpad.opendev.org/p/ironic-redfish =)08:59
Qianbiaorpittau any doc08:59
rpittauQianbiao: https://opendev.org/openstack/sushy/src/branch/master/sushy/resources/settings.py#L14708:59
rpittauQianbiao: consider that the etag is not mandatory08:59
rpittauQianbiao: more info here https://docs.openstack.org/sushy/rocky/reference/api/sushy.resources.settings.html08:59
*** k_mouza has joined #openstack-ironic09:03
Qianbiaorpittau i did not dig into the code much. it seems that if redfish response a Redfish.Setting with etag requried.09:05
Qianbiaothen it will auto send etag when patch a resource?09:05
rpittauit will verify the etag, yes09:06
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: test list chassis with details  https://review.opendev.org/71186509:09
Qianbiao<rpittau> only several resources support this action. if we could add a global switch to like Sushy(), that would be great.09:13
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Reduce timeout for tempest job  https://review.opendev.org/72863509:21
*** uzumaki has quit IRC09:22
openstackgerritMerged openstack/ironic master: [Trivial] Fix awkward mocking in redfish boot tests  https://review.opendev.org/72591809:23
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: list port groups with detail  https://review.opendev.org/72880209:36
iurygregoryrpittau, hey if a person wants to update a volume connector the node can be in manage or should be in provide?09:36
* iurygregory thinks manage should be fine09:36
rpittauiurygregory: I believe manage is fine, I would have to double-check09:37
iurygregoryI was wondering if it's necessary to set the state since the functional tests in the openstacksdk are showing "openstack.exceptions.BadRequestException: BadRequestException: 400: Client Error for url: https://10.176.192.190/baremetal/v1/volume/connectors/deef5e16-632b-48ef-92a0-5b19f04ccb06, The requested action "volume connector update" can not be performed on node "8c0784cd-a1d4-4800-ba80-fa7c54107e27" while it is09:37
iurygregoryin state "None"."09:37
rpittauiurygregory: you have to specify the state of the node09:39
rpittauopenstack.exceptions.BadRequestException: BadRequestException: 400: Client Error for url: https://10.176.192.190/baremetal/v1/volume/connectors, Invalid input for field/attribute type. Value: 'None'. Mandatory field missing.09:39
iurygregoryrpittau, yeah I saw this ones too =) already add comments09:44
iurygregoryI'm checking test_volume_connector_patch and test_volume_connector_update09:44
iurygregoryso setting the provision state should be fine right?09:45
rpittauiurygregory: that should work09:46
*** k_mouza has quit IRC09:47
*** rpittau is now known as rpittau|bbl10:06
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: list port groups by node  https://review.opendev.org/72880710:09
*** k_mouza has joined #openstack-ironic10:11
*** Qianbiao has quit IRC10:11
*** Qianbiao has joined #openstack-ironic10:13
*** tkajinam has quit IRC10:15
*** bdodd_ has joined #openstack-ironic10:31
*** bdodd has quit IRC10:32
ajyaHi, what is this patch waiting for? Got stuck because of dependency that failed to merge at that time? https://review.opendev.org/#/c/727676/10:44
patchbotpatch 727676 - ironic - Remove iDRAC Redfish boot device known issue - 1 patch set10:44
iurygregoryajya, it's probably zuul causing problems =)10:47
iurygregoryI've put a recheck to trigger the jobs again10:47
ajyathanks, iurygregory10:47
iurygregoryyw10:48
iurygregoryoh the stable patches are not merged and maybe zuul thinks it can't merge since the change-Id is on Depends-on10:49
iurygregoryif it doesn't goes to gate after the recheck you can probably submit the patch again removing the Depends-On from the commit message10:50
ajyaoh, maybe, let's see10:50
iurygregoryajya, the patch is not in gate =(11:08
iurygregoryhttps://zuul.openstack.org/status if you put the number of the gerrit change it doesn't show =(11:08
iurygregorygerrit change is 72767611:08
*** dougsz has left #openstack-ironic11:22
ajyaiurygregory: ok, I'll discuss with rpioso whether to remove the dependency or wait for stables to be merged11:25
openstackgerrityuanliu proposed openstack/ironic master: rm duplicate word 'that'  https://review.opendev.org/72881311:25
iurygregoryajya, ok =)11:25
*** ociuhandu has joined #openstack-ironic11:28
*** ftarasenko has quit IRC11:30
openstackgerritMerged openstack/ironic master: Encapsulate auth_token middleware  https://review.opendev.org/72746611:31
*** Qianbiao has quit IRC11:36
*** Qianbiao has joined #openstack-ironic11:37
*** ociuhandu has quit IRC11:40
*** ociuhandu has joined #openstack-ironic11:59
*** ociuhandu has quit IRC12:08
*** rpittau|bbl is now known as rpittau12:12
*** rh-jelabarre has joined #openstack-ironic12:12
Qianbiaohello, i got a question for raid configuratino.12:18
*** bdodd_ is now known as bdodd12:18
Qianbiaowhen ibmc try to setup inband RAID controller, a restart is required.12:19
QianbiaoIs this workflow violate?12:19
openstackgerritMerged openstack/ironic stable/ussuri: Fix pep8  https://review.opendev.org/72768912:20
rpittauajya: I approved and rechecked the dependencies for that patch, they should merge soon12:32
ajyarpittau: thanks, fingers crossed12:33
rpittauajya: do you mind issuing a rebase on https://review.opendev.org/727676 ?12:37
patchbotpatch 727676 - ironic - Remove iDRAC Redfish boot device known issue - 1 patch set12:37
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Reduce timeout for tempest job  https://review.opendev.org/72863512:39
TheJuliagood morning everyone12:51
rpittaugood morning TheJulia :)12:51
iurygregorygood morning TheJulia =)12:53
*** ociuhandu has joined #openstack-ironic12:53
*** ociuhandu has quit IRC12:58
*** chihhsin has quit IRC13:01
*** ociuhandu has joined #openstack-ironic13:03
*** ociuhandu has quit IRC13:08
ajyarpittau: will rebase13:09
rpittauTheJulia: I issued a revert in stable/train because we will need to upgrade the min version of keystoneauth in ironic-staging-drivers to make it work there https://review.opendev.org/72878413:10
patchbotpatch 728784 - ironic (stable/train) - Revert "Add timeout and retries to JSON RPC client" - 1 patch set13:10
rpittauI have the feeling we should keep lower-constraints between ironic and ironic-staging-drivers the same13:10
rpittauthanks ajya13:10
rpiosoGood morning13:12
rpittauhey rpioso :)13:12
rpiosorpittau: o/13:12
*** jhesketh has quit IRC13:15
iurygregorymorning rpioso13:18
openstackgerritJulia Kreger proposed openstack/ironic master: Fix image type reference  https://review.opendev.org/72888413:21
*** jhesketh has joined #openstack-ironic13:22
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection  https://review.opendev.org/68397113:24
openstackgerritAija Jaunteva proposed openstack/ironic master: Remove iDRAC Redfish boot device known issue  https://review.opendev.org/72767613:25
openstackgerritJulia Kreger proposed openstack/ironic-inspector stable/ussuri: Treat endpoints with trailing slashes the same way as without them  https://review.opendev.org/72888513:25
*** mraineri has joined #openstack-ironic13:26
*** uzumaki has joined #openstack-ironic13:32
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection  https://review.opendev.org/68397113:34
*** jhesketh has quit IRC13:41
openstackgerritMerged openstack/ironic master: CI: add test-requirements to irrelevant-files  https://review.opendev.org/72842813:42
openstackgerritMerged openstack/ironic master: CI: raise DIB jobs memory to 3 GiB  https://review.opendev.org/72716913:42
openstackgerritMerged openstack/ironic master: RAID docs: fix an invalid json example  https://review.opendev.org/72869713:42
*** uzumaki has quit IRC13:43
*** uzumaki has joined #openstack-ironic13:44
openstackgerritMerged openstack/metalsmith master: Update tox.ini  https://review.opendev.org/72691513:46
*** ociuhandu has joined #openstack-ironic13:52
TheJuliastevebaker: it occurs to me that we should also add basic auth to ironic-inspector as well13:52
TheJuliarpittau: approved the revert13:55
rpittauTheJulia: thanks!13:56
*** ociuhandu has quit IRC13:56
*** ociuhandu has joined #openstack-ironic14:00
TheJuliastevebaker: and I wonder about jsonrpc...14:03
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection  https://review.opendev.org/68397114:19
*** ftarasenko has joined #openstack-ironic14:26
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Follow-up on iso boot spec  https://review.opendev.org/72891114:28
ftarasenkoHi, team! Can someone help me about building ironic image that supports raid? I've added mdadm package to my centos image, but currently it shows me grub and then fails to start. What do I miss? Train version.14:29
arne_wiebalckftarasenko: this is whole disk image?14:32
arne_wiebalckftarasenko: also, which boot mode uefi or bios?14:33
ftarasenkowhole disk, bios mode14:34
arne_wiebalckftarasenko: ironic with nova or stand-alone?14:34
ftarasenkowith nova14:34
arne_wiebalckftarasenko: did you set the rootfs uuid on the image?14:35
arne_wiebalckftarasenko: this is needed if your root fs is not the first partition in the image14:35
arne_wiebalckftarasenko: wait, seems this comes only with Ussuri14:37
ftarasenkoarne_wiebalck: no, I did not. do I correctly understand that I need to get this information from image?14:37
arne_wiebalckftarasenko: is the root fs on the first partition?14:37
arne_wiebalckftarasenko: it needs to be set as image metadata (and yes you would get it from the image)14:38
arne_wiebalckftarasenko: it needs to be set in Ussuri14:38
arne_wiebalckftarasenko: in Train we assume the root fs is on the first partition14:38
ftarasenkoarne_wiebalck: I'll mount image and check now. Do you recommend to upgrade Ironic to Ussuri?14:39
ftarasenkoI did not asked IPA for any partitioning, but maybe /boot it put there14:39
arne_wiebalckftarasenko: if there is another partition, I am not sure if the bootloader installation would have worked ... since you're instance is ACTIVE, correct?14:40
arne_wiebalcks/you're/your/14:40
ftarasenkoarne_wiebalck: correct. I have active instance, I've cleaned it and created raid before boot14:41
arne_wiebalckftarasenko: right14:41
arne_wiebalckftarasenko: the partition I was talking about is the one inside your centos image, not on the host14:42
arne_wiebalckftarasenko: and the node boots fine if you do not have a RAID configured?14:43
arne_wiebalckftarasenko: (to make sure the image is fine otherwise)14:43
*** ociuhandu has quit IRC14:43
ftarasenkoarne_wiebalck: correct, it boots fine without RAID. I'm checking now for image, trying to find suitable host14:44
*** ociuhandu has joined #openstack-ironic14:45
arne_wiebalckftarasenko: the above should read "and created raid before deploy" (not boot), right?14:46
openstackgerritBob Fournier proposed openstack/ironic master: New configuration parameter to use ipmitool retries  https://review.opendev.org/72826114:47
ftarasenkoarne_wiebalck: sorry for inconvinience, yes, I do clean before server boot14:47
ftarasenkoI have only one partion, so it should be first. In Train nothing is required before instance boot? I take disk-image-builder and run it with additional mdamd package14:49
arne_wiebalckftarasenko: one single partition should be fine14:49
arne_wiebalckftarasenko: the image needs to able to assemble the RAID, let me check what we do ...14:50
*** kaifeng has joined #openstack-ironic14:51
*** ociuhandu has quit IRC14:51
ftarasenkoarne_wiebalck: thank you! Maybe, you reconfigure bootloader? But I've checked and found raid drivers in mine14:52
arne_wiebalckftarasenko: we have "rd.auto" as a kernel boot param14:52
arne_wiebalckftarasenko: to automatically start the md devices14:52
*** beekneemech is now known as bnemec14:53
ftarasenkoarne_wiebalck: Good point! I'll provision my image and and this option manually on first boot. Thank you!14:54
arne_wiebalckftarasenko: Good luck :)14:55
openstackgerritRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Reduce timeout for tempest job  https://review.opendev.org/72863514:56
*** ociuhandu has joined #openstack-ironic14:59
TheJulia#startmeeting ironic15:00
TheJuliao/15:00
openstackMeeting started Mon May 18 15:00:10 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
iurygregoryo/15:00
openstackThe meeting name has been set to 'ironic'15:00
bdoddo/15:00
TheJuliaGood morning everyone!15:00
kaifengo/15:00
rpittauo/15:00
rpiosoo/15:00
TheJuliaOur agenda can be found on the wiki, as always!15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
arne_wiebalcko/15:00
erbarro/15:00
TheJuliaOne note, I'm also in another call, so hopefully I won't become distracted.15:00
rajiniro/15:00
ajyao/15:01
TheJulia#topic Announcements / Reminders15:01
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:01
TheJuliaA few things to note!15:01
*** stendulker has joined #openstack-ironic15:01
TheJulia#info Next monday is a holiday for the states, I won't be around. Dmitry will be running the meeting next week.15:02
stendulkero/15:02
TheJulia#info Virtual PTG is coming soon. June 1-515:02
*** chihhsin has joined #openstack-ironic15:02
TheJulia#link https://www.openstack.org/ptg/15:02
TheJulia#info We have a PTG topics etherpad. Add your crazy ideas!15:03
TheJulia#link https://etherpad.opendev.org/p/Ironic-VictoriaPTG-Planning15:03
TheJulia#info We have a priorites document for the Victoria cycle15:04
TheJulia#link https://review.opendev.org/#/c/720100/15:04
patchbotpatch 720100 - ironic-specs - WIP - Victoria Cycle Priorit(y|ies) - 4 patch sets15:04
TheJuliaOne last item!15:04
bfournieo/15:04
TheJulia#info Jroll is hanging up his +2 for the time being. Thanks jroll! Until it is time to give it back to you :)15:05
jroll<315:05
TheJuliaDoes anyone have anything else to raise today?15:05
iurygregoryI think we are good15:05
TheJuliaWe have no action items, so we can skip that step!15:06
TheJuliaOne last thing, I may take some more time off next week.15:07
TheJuliaMy car is calling me...15:07
TheJuliaAnd parts will be delivered tonight actually :)15:07
iurygregory\o/15:07
TheJuliaAnyway, onward!15:08
TheJulia#topic Review subteam status reports15:08
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:09
TheJulia#link Review subteam status reports15:09
TheJulia#undo15:09
openstackRemoving item from minutes: #link Review15:09
TheJulia#link https://etherpad.opendev.org/p/IronicWhiteBoard15:09
TheJuliathinks look fairly good, it would be awesome for the v6 CI job to get traction/movement15:11
TheJulias/thinks/things/15:12
* TheJulia wonders when she will begin typing tink instead of think15:12
TheJulia</ibmjoke>15:12
iurygregorylol15:12
TheJuliatink being a ?norweigan? spelling for think15:12
*** cdearborn has joined #openstack-ironic15:13
uzumakiAlso the star wars misa pronounciation of the same15:13
uzumaki"Misa tinks TheJulia and herself are good friends.." <says in the weird voice>15:15
rpittauTheJulia: I was going to approve the release model, but maybe I can +2 and leave it open in case we want to wait a little bit more ?15:17
iurygregoryI would give more time for people who had questions to give a +115:18
iurygregoryjust to make sure they understand our plan15:18
openstackgerritVerification of a change to openstack/ironic failed: Mark more configuration options as reloadable  https://review.opendev.org/72586715:21
TheJuliasorry, other meeting grabbing my attention, one minute15:21
TheJuliarpittau: I think that is reasonable15:21
TheJuliauzumaki: yes, you fit in here15:22
TheJuliaAre we good to proceed?15:22
rpittaulet's15:22
iurygregory++15:22
uzumaki+115:22
TheJuliaokay!15:22
TheJulia#topic Deciding on priorities for the coming week15:23
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:23
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:23
TheJuliaStarting at line 10815:23
rpittauprobably IPV6 job needs some love (and a rebase) ?15:23
TheJuliaLooks like someone removed the merged links?15:23
TheJulia(thanks!)15:24
rpittauoh yeah, I did, I was updating the section15:24
TheJuliaawesome15:24
TheJuliaIs everyone good with the items marked for the next meeting (line 163)15:24
rpittauI'd like to add the lower-constraints change, they shoud not take long to review15:25
*** ociuhandu has quit IRC15:25
TheJuliarpittau: ++15:25
TheJuliaplease!15:25
rpittauthanks15:25
rpittauadd on L17515:26
rpittauerr15:26
rpittau17715:26
TheJuliaadded the link to 16415:26
rpittauok! :D15:26
TheJuliaAnything else?15:26
kaifengi wonder what's in the linux-firmware package that need to be in the dib?15:26
rpittaukaifeng: it's not for ci, it's for baremetal15:27
TheJuliakaifeng: I have long wondered this15:27
TheJuliabottom line, yes it is for actual baremetal15:28
kaifengOh, I thought it contains stuff like firmware, so it's building images that may provide such service?15:28
rpittauI don't remember the details, but since we're publishing the image and that image is used in baremetal, we need the firmware there15:28
rpittaukaifeng: https://storyboard.openstack.org/#!/story/200738515:29
openstackgerritMerged openstack/ironic-inspector master: Treat endpoints with trailing slashes the same way as without them  https://review.opendev.org/72718715:29
kaifengrpittau: thanks, okay so some drivers may be included there..15:30
rpittauyeah15:30
TheJuliaYeah, I think the point is microcode15:30
TheJuliaAnyway, I guess we're good with priorities for the week?15:31
kaifeng+115:31
TheJuliaMoving on to the baremetal sig15:32
openstackgerritMerged openstack/networking-baremetal master: Bump hacking version to 3.0.0 and fix pep8 test  https://review.opendev.org/72715215:32
TheJulia#topic Baremetal SIG15:32
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:32
TheJuliaOne item worth noting, we're anticipating initial feedback from the OSF on the whitepaper this week.15:32
TheJuliaI think that is it, is there anything else arne_wiebalck ?15:32
arne_wiebalckI did some more changes.15:32
* rpittau needs to finish stuff there :/15:32
arne_wiebalckNo, not really.15:32
TheJuliaI meant to on Friday, then tweeked my back... again.15:32
* TheJulia needs to work on physical activity big time15:33
arne_wiebalckThere is no real feedback from the OSF yet.15:33
TheJuliak15:33
arne_wiebalckOtherwise have a look at comments/reviews and chime in.15:33
TheJuliaI can reach out to them if we don't really have feedback by wednesdays15:34
TheJuliawednesday15:34
arne_wiebalckThere is no real rush.15:34
arne_wiebalckOTOH, we should not let things slip.15:34
TheJulia++15:34
TheJuliaPutting a reminder on my calendar15:34
arne_wiebalckThis week is short here, Thu is a hoiday15:34
arne_wiebalckholiday15:35
TheJuliaawesome!15:35
rpittauyep15:35
arne_wiebalckMost people take Friday off15:35
TheJuliaOkay, are we good to move on to open discussion15:35
* rpittau included15:35
* TheJulia should take Friday off now....15:35
rpittaudo it!15:35
rpittauboth :)15:35
TheJuliayeah, the car demands work15:35
* TheJulia laughs slightly evilly15:35
rpittauthose talking cars, so needy15:35
TheJuliaheh15:35
TheJuliano, it is the cars that are needy15:35
* arne_wiebalck hopes the car is not laughing as well15:36
TheJuliaThat would be moderately disturbing15:36
TheJuliaMoving on!15:36
*** ociuhandu has joined #openstack-ironic15:36
TheJulia#topic Open Discussion15:36
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:36
TheJuliaAnything else this week?15:36
rpittauwell I'm moving forward with testing on ubuntu focal and filed the first bug \o/ https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/187932515:37
openstackLaunchpad bug 1879325 in libvirt (Ubuntu) "Unable to remove disk metadata on vm" [Undecided,New]15:37
rpittauactually that's second, the first one was upstream and should be fixed, but the package needs to be updated in ubuntu15:38
arne_wiebalckFor the PTG (and for the SIG), should we plan/offer some type of explicit operator feedback ... or is this not the right forum/event?15:39
arne_wiebalckIt says operators should attend.15:40
* arne_wiebalck just checked15:40
arne_wiebalckNot sure if there is demand.15:40
arne_wiebalckMaybe better for the summit.15:41
QianbiaoHello, I do not know whether this question is suitable in this meeting. I am wondering when this patch will be reviewed: https://review.opendev.org/#/c/728123/315:41
patchbotpatch 728123 - ironic - Feature: Add raid configuration support for ibmc d... - 3 patch sets15:41
Qianbiao has ussuri release work finished now?15:41
kaifengspeaking of firmware, does anyone know what's BOOTX64.EFI doing? It seems we can boot with grubx64.efi directly, this confusing me for a long time.15:42
rpittauQianbiao: yes, we're in victoria now15:42
*** gyee has joined #openstack-ironic15:42
Qianbiao<rpittau> got15:42
rpittauQianbiao: I'll get to that as soon as possible, not sure about this week though15:43
Qianbiaothanks, we are waiting for that patch, then we can intergated ironic with metal315:43
TheJuliaarne_wiebalck: that is a good idea, perhaps add an item to the list?15:44
arne_wiebalckTheJulia: on the etherpad ?15:44
TheJuliakaifeng: BOOTX64.EFI is actually shim15:44
TheJuliaso for secure boot15:45
TheJuliaarne_wiebalck: in the victoria ptg planning etherpad15:45
arne_wiebalckTheJulia: +115:45
kaifengTheJulia: that means if I am not doing a secure uefi, then we can just jump to grubx64 right?15:45
TheJuliakaifeng: yes15:46
kaifengTheJulia: thanks for explaining :)15:46
TheJuliaDo we have anything else to discuss this morning?15:47
rpittauwe don't have shimx64.efi for the secure boot ?15:47
TheJuliarpittau: depends on case :)15:47
rpittauwe can continue the discussion in a different moment outside of the meeting15:47
rpittauoh yeah, ofc15:48
TheJuliaThanks everyone!15:49
TheJuliaHave a wonderful week!15:49
rpittauthanks TheJulia, you too :)15:49
rpiosoTheJulia: Thank you!15:49
TheJulia#endmeeting15:49
*** 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:49
openstackMeeting ended Mon May 18 15:49:23 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:49
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-05-18-15.00.html15:49
arne_wiebalckTheJulia: thanks15:49
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-05-18-15.00.txt15:49
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-05-18-15.00.log.html15:49
kaifengo/15:50
QianbiaoMeeting is over :). I have a question for kaifeng in my mind for a while, are u chinese?15:50
kaifengQianbiao yes I am15:51
TheJulia:)15:51
Qianbiao:) you name looks like a chinese.15:51
Qianbiaos/you/your/15:51
kaifengSo we are in the same timezone :)15:51
Qianbiaoyes15:51
QianbiaoI am a freelancer, currently work for ironic ibmc driver for HUAWEI.15:52
kaifengI know that, I remember your proposed the ibmc driver at the end of ?Stein?15:53
Qianbiaoyes, good memory~15:53
kaifengalways in a hurry hehe15:54
Qianbiao:) plan from HUAWEI.15:54
Qianbiaothey are customer driven.15:54
kaifengI would say we are customer driven :(15:54
kaifengtoo15:55
Qianbiao:)15:55
Qianbiaoi thought you work for OpenStack ironic.15:55
QianbiaoYou got a wechat? we may talk later.15:56
Qianbiaonot disturb this channel15:56
kaifengyeah, i was involving ironic since 201715:56
kaifengQianbiao: sure, you can find me by the name 'direcat'15:57
Qianbiaogot15:57
* kaifeng is a cat pretends to a dire wolf15:58
* TheJulia is intrigued by this wechat name :)15:58
TheJuliaexcellent!15:58
Qianbiao:）15:58
kaifengaddicted to novel :)15:59
Qianbiaoneed to go, folks. See u later.16:00
*** Qianbiao is now known as Qianbiao|afk16:01
kaifenggood night o/16:01
Qianbiao|afknight ~16:01
*** Qianbiao|afk has quit IRC16:01
rpittaugood night! o/16:05
*** rpittau is now known as rpittau|afk16:06
*** stendulker has quit IRC16:06
*** ociuhandu_ has joined #openstack-ironic16:18
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Migration from oslo.rootwrap to oslo.privsep  https://review.opendev.org/72894216:21
*** ociuhandu has quit IRC16:22
*** ociuhandu_ has quit IRC16:23
openstackgerritMerged openstack/ironic stable/train: Revert "Add timeout and retries to JSON RPC client"  https://review.opendev.org/72878416:31
openstackgerritMerged openstack/ironic-tempest-plugin master: Add standalone redfish jobs  https://review.opendev.org/72067516:31
iurygregory\o/16:34
openstackgerritMerged openstack/ironic-inspector stable/ussuri: Treat endpoints with trailing slashes the same way as without them  https://review.opendev.org/72888516:34
*** lucasagomes has quit IRC16:35
*** scott__c has joined #openstack-ironic16:36
scott__cHey everyone! I'm completely new to Ironic, and I'm doing a research project to determine if/how more "exotic" hardware can be integrated with OpenStack. So I have a number of questions! First, I understand that if the machine supports some kind of out-of-band management, then there's a chance Ironic can support - even if that entails custom hardware types and drivers?16:43
openstackgerritMerged openstack/ironic stable/ussuri: redfish: handle hardware that is unable to set persistent boot  https://review.opendev.org/72790316:44
scott__cFor example, I have a machine that supports IPMI, but uses different firmware (U-Boot instead of BIOS)16:45
*** k_mouza has quit IRC16:48
*** k_mouza has joined #openstack-ironic16:49
*** ricolin_ has joined #openstack-ironic16:53
*** k_mouza has quit IRC16:54
*** ricolin has quit IRC16:56
*** ricolin_ has quit IRC17:00
*** ricolin has joined #openstack-ironic17:00
iurygregoryscott__c, depending on how exotic your hardware is you may need to create a driver for it, or maybe use the generic ones =)17:05
openstackgerritMerged openstack/ironic-python-agent-builder master: Install linux-firmware in DIB images  https://review.opendev.org/72676917:05
scott__c@iurygregory As an example. I have a machine running a PowerPC CPU. It relies on U-Boot firmware (not BIOS), but it does support IPMI. So I guess my question there is if BIOS is a requirement to integrate with Ironic?17:06
openstackgerritMerged openstack/ironic master: Do all serialization in the expose decorator  https://review.opendev.org/70448617:10
*** uzumaki has quit IRC17:12
*** uzumaki has joined #openstack-ironic17:12
*** dking has joined #openstack-ironic17:14
ftarasenkoarne_wiebalck: thank you, rd.auto helps to start boot in 3-5 seconds. However, it's still a bit strange, cause I use latest IPA and have rd.md.uuid set in my GRUB.17:16
openstackgerritVerification of a change to openstack/ironic failed: Add `network_data` field to ironic node object  https://review.opendev.org/68791017:18
*** k_mouza has joined #openstack-ironic17:24
openstackgerritAndreas Jaeger proposed openstack/ironic master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72895717:27
*** k_mouza has quit IRC17:28
openstackgerritAndreas Jaeger proposed openstack/ironic master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72895717:32
openstackgerritAndreas Jaeger proposed openstack/ironic master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72895717:34
*** kaifeng has quit IRC18:10
TheJuliascott__c: It would likely help if you wanted to make an etherpad with the cases, we migh tbe able to crowd source answers18:21
TheJuliascott__c: and high level, no bios is not a requirement, like power9 gear has an ipmi path through openbmc, and the bootloader groks the pxelinux file format18:22
TheJuliawhile it doesn't actually execute pxelinux becasue it is from a different architecture18:22
* TheJulia is finally done with calls and meetings for the day and has started making lunch18:23
*** ociuhandu has joined #openstack-ironic18:28
*** ociuhandu has quit IRC18:32
*** scott_c has joined #openstack-ironic18:33
*** scott__c has quit IRC18:36
arne_wiebalckftarasenko: so, with rd.auto your instance boots?18:53
*** rmcall has joined #openstack-ironic19:03
arne_wiebalckftarasenko: also: do the rd.md.uuid values make it to grub.cfg and the kernel cmdline?19:09
JayFTheJulia: iurygregory: dtantsur|afk: Any plans for an SPUC 3? I'd enjoy being able to participate again19:09
TheJuliano plans off hand, but I love the idea!19:09
iurygregoryJayF, ++ for it if we can do before the PTG to get crazy ideas =)19:09
JayFWhat about this Friday?19:09
JayFJFDI is always the best way to get something going :D19:10
TheJulia++19:10
iurygregoryThis friday is ok fr me =)19:10
JayFCan someone actually subbed to the list email openstack-discuss about it?19:11
* iurygregory googles JFDI...19:11
JayFJust [] do it19:11
JayFfill in the blank ;)19:11
iurygregorywith a word that starts with F =)19:11
iurygregoryup to your imagination19:11
JayFjust like in RTFM, for example19:11
iurygregoryI can send the email, so we are keeping the same time and also using TheJulia bluejeans? XD19:12
*** zer0c00l has joined #openstack-ironic19:12
TheJuliaI'm officially taking friday off at this point, but I can join with a beverage!19:12
TheJuliaiurygregory: sounds good to me19:12
JayFzer0c00l should be coming, he's providing the crazy pre-PTG ideas ;)19:12
zer0c00llol19:12
TheJuliaIdeas are always welcome19:13
* TheJulia goes and puts in for time off19:13
zer0c00lideas and bodies to work on ideas :)19:13
JayFjust as long as you're not putting my ass on the line ;)19:13
iurygregoryLOL19:14
TheJuliaBodies are also welcome, as long as they are functional19:14
TheJuliaand have operating control units19:14
zer0c00lhahaha ofcourse.19:14
iurygregoryshould we use the time from the 1st SPUC or the 2nd?19:15
TheJuliawhat were the times?19:15
TheJuliait is all a blur to me19:15
iurygregorythe 2nd was at 2pm UTC19:15
iurygregorythe 1st was at 5PM19:16
iurygregoryUTC19:16
* iurygregory has meeting at 3pm UTC19:16
TheJuliaI can do either time... as long as I'm awake19:18
JayFI'd prefer a little later than 2pm UTC if possible19:18
JayF5pm UTC would be A+++19:18
zer0c00l2 pm UTC is 7 am PST :(19:19
iurygregoryouch19:19
zer0c00l5 pm UTC would be great!19:19
iurygregoryok let's go with 5pm UTC19:19
JayFthanks iurygregory, it's nice to get to see everyone's faces and talk over stuff19:19
JayFand I have four whole days to find a funny hat IRL instead of having to cheat with webcam overlays :P19:20
iurygregorylol19:20
ftarasenkoarne_wiebalck: yes, with rd.auto it boots. what do you mean asking about value? it matches with mdraid id19:22
iurygregoryemail sent19:29
TheJuliascott_c: by chance did you see my earlier reply. I just noticed your client quit and rejoined after I replied19:39
TheJuliaiurygregory: thanks19:39
iurygregoryTheJulia, yw =)19:40
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Migration from oslo.rootwrap to oslo.privsep  https://review.opendev.org/72894219:44
openstackgerritAndreas Jaeger proposed openstack/python-ironicclient master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72901419:50
*** k_mouza has joined #openstack-ironic19:50
*** k_mouza has quit IRC19:50
openstackgerritAndreas Jaeger proposed openstack/python-ironic-inspector-client master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72901519:51
TheJuliavdrok: any particular patch you need our eyes on first for the client-id changes you've been making?19:59
*** uzumaki has quit IRC20:15
*** Lucas_Gray has joined #openstack-ironic20:38
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant  https://review.opendev.org/70503020:44
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant  https://review.opendev.org/70503020:46
*** hogepodge has joined #openstack-ironic21:01
*** jdandrea has quit IRC21:19
stevebakerTheJulia: yes, basic auth in inspector and jsonrpc would be good I think. It could be worth putting the middleware into ironic-lib now rather than landing it in ironic first21:20
TheJuliaYeah, that does kind of make sense21:22
stevebakerTheJulia: also for jsonrpc should it have a separate auth file? Mixing system and real users seems wrong, on the other hand: Unix21:24
TheJuliaseparate file seems separate file seems reasonable because front-end/back-end different connections/operational scope21:25
stevebaker+121:26
*** dking has quit IRC21:31
*** scott_c has quit IRC21:33
*** sshnaidm is now known as sshnaidm|afk22:00
*** threestrands has joined #openstack-ironic22:06
*** eandersson has joined #openstack-ironic22:09
eanderssonAnyone know why I would be getting an is_image_available error for AttributeError: properties when trying to write my own ipa hardware handler after making reboot a required step?22:12
eanderssonhttp://paste.openstack.org/show/s5omQdJ9FTsZVKxB3Oem/22:14
eanderssonLooks like it might be fixed in a newer version22:16
TheJuliao/ eandersson22:31
eanderssono/22:31
TheJuliaeandersson: no idea, my best guess is maybe the properties field is missing22:32
TheJuliathe fact that is coming out of the glance image hnadling code is kind of surprising22:32
eanderssonI am hoping it's just a cache issue, because the image for sure has properties.22:33
*** dking has joined #openstack-ironic22:33
eanderssonThis is Rocky so might be worth upgrading to something newer as well :D22:33
TheJuliamaybe, although I don't think we've touched that code in a long time22:34
TheJuliawell22:34
TheJuliaclients may have been swapped22:34
eanderssonIt is really weird thou that it only happens if I set reboot to required during cleaning22:34
TheJuliaSo if your re-hitting the cache, I wonder22:34
TheJuliareboot required, anything else?22:35
eanderssonhttp://paste.openstack.org/show/V3N0wyuLBSE1NKRleDf0/22:36
eanderssonAnd all the script does it execute a bash script (dummy hello world one that returns exit code 0)22:36
TheJuliaeandersson: that is happening remotely on ipa?22:37
TheJuliaand getting sent back?22:37
*** dking has quit IRC22:37
eanderssonThe logs are from ironic22:38
TheJuliaok22:38
eanderssonbut the hardware manager is on the ipa22:38
eanderssonhttps://github.com/openstack/ironic-python-agent/tree/master/ironic_python_agent/hardware_managers22:38
eanderssonA really basic one that juist inherits hardware.GenericHardwareManager and adds a exec call to /usr/local/test/test.sh22:39
TheJuliaso I suspect that what is occuring is rocky doesn't grok reboots in deploy22:40
TheJuliaclean step flag of getting reboot_requetsed causes _cleaning_reboot to be called22:41
eanderssonI could try to upgrade, but would take some time to prep that22:42
TheJuliawhich triggers ramdisk preparation for cleanintgg22:42
TheJuliacleaning22:42
TheJuliaand then reboots the machine22:42
TheJuliaso... err yeah. that would be why, its likely trying to pull cleaning ramdisk back down22:42
TheJuliaany reason your trying to do firmware upgrades in deployment and not in cleaning?22:43
TheJuliaaside form a node can sit idle for a while possibly22:43
eanderssonThis is actually during cleaning22:44
eanderssonbut might be a mistake if you are asking that question :D22:45
TheJuliahmm, looking22:45
TheJuliaI seem to be confused22:46
eanderssonGonna rebase the IPA as well, this might be some old version of the IPA.22:46
TheJuliaoh22:47
TheJuliawait22:47
TheJuliaipa is likely fine22:47
TheJuliaI was confused about the interface fo rsome reason22:47
TheJulialooking at the stack trace it is in the cache cleanup22:47
openstackgerritSteve Baker proposed openstack/ironic-lib master: Implement Basic HTTP authentication middleware  https://review.opendev.org/72907022:49
TheJuliaeandersson: how is your conductor on disk space?22:51
eanderssonlow :D22:52
eanderssonbut still 20GB free22:52
openstackgerritSteve Baker proposed openstack/ironic master: Enable Basic HTTP authentication middleware  https://review.opendev.org/72746722:52
*** tkajinam has joined #openstack-ironic22:53
eanderssonunless it's using /tmp and there is a lot less :D22:53
TheJuliaso, it looks like it is using whatever is configured as the cache directory22:54
TheJuliaIt is trying to clean up cached data22:54
TheJuliaand my hunch is that folder has uuids that are no longer in glance22:55
TheJuliaat least, that is my hunch22:55
eanderssonYea that is for sure possible22:55
eanderssonThis is our lab so... things... magically disappear22:56
TheJuliaqueue magic gif22:56
TheJuliaeandersson: what do you see in the folder defined by [conf]tftp_master_path ?23:01
TheJuliaeandersson: also, check your cleaning ramdisk parameters, to ensure kernel/ramdisk uuid match valid glance images23:03
TheJuliain theory, what shoudl be happening is that it is trying to lookup the size of the files to figure out how much is needed space wise in the cache, not before, but the cache code is a little hard to follow23:04
*** Lucas_Gray has quit IRC23:24
eanderssonSorry in a meeting! ~10 minutes23:25
TheJuliano worries, just winding my day trying to figure something out by not looking at the ipxe source23:31
* TheJulia is doomed to have to look in the morning23:31
*** yolanda has quit IRC23:36
TheJuliahmmmm23:37
* TheJulia ponders ipxe23:37
*** scott_c has joined #openstack-ironic23:43
*** scott__c has joined #openstack-ironic23:48
*** scott_c has quit IRC23:51
eanderssonTheJulia: It's /var/lib/tftboot (technically it's the default /tftboot, but it's in a container that maps to /var/lib/tftboot on the host)23:57
eanderssonI have actually uploaded multiple images and I keep updating the uuid for these so they do exist.23:58
eanderssonPlus it does boot into the image, and fails just on the reboot during cleaning.23:58
« Sunday, 2020-05-17 Index Tuesday, 2020-05-19 »

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