Wednesday, 2021-03-24

« Tuesday, 2021-03-23 Index Thursday, 2021-03-25 »
*** tosky has quit IRC00:09
*** alexmcleod has quit IRC00:42
*** alexmcleod has joined #openstack-ironic00:42
*** paras333_ has joined #openstack-ironic01:24
*** paras333_ has quit IRC01:28
*** rcernin has quit IRC01:34
*** rcernin has joined #openstack-ironic01:45
*** tzumainn has quit IRC01:45
*** rcernin has quit IRC01:46
*** rcernin has joined #openstack-ironic01:46
*** rh-jelabarre has quit IRC02:05
*** rcernin has quit IRC02:45
*** rcernin has joined #openstack-ironic02:57
*** rcernin has quit IRC02:57
*** rcernin has joined #openstack-ironic02:58
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Automaticaly set cipher suite  https://review.opendev.org/c/openstack/ironic/+/77052603:27
*** rcernin has quit IRC03:31
*** rcernin has joined #openstack-ironic03:38
*** rcernin has quit IRC03:52
*** rcernin has joined #openstack-ironic03:52
*** rcernin has quit IRC03:53
*** rcernin has joined #openstack-ironic03:54
*** paras333_ has joined #openstack-ironic04:16
*** paras333_ has quit IRC04:21
openstackgerritArun S A G proposed openstack/ironic master: Add anaconda support in the pxe boot driver  https://review.opendev.org/c/openstack/ironic/+/77589104:29
openstackgerritArun S A G proposed openstack/ironic master: Validate the kickstart template and file before use  https://review.opendev.org/c/openstack/ironic/+/77744804:29
openstackgerritArun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat  https://review.opendev.org/c/openstack/ironic/+/77805804:29
openstackgerritArun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface  https://review.opendev.org/c/openstack/ironic/+/78039804:29
*** jamesdenton has quit IRC04:30
*** jamesden_ has joined #openstack-ironic04:31
openstackgerritMerged openstack/sushy-tools master: Delete temporary file after media ejection  https://review.opendev.org/c/openstack/sushy-tools/+/78251404:50
*** jamesden_ has quit IRC04:55
openstackgerritArun S A G proposed openstack/ironic master: Add anaconda support in the pxe boot driver  https://review.opendev.org/c/openstack/ironic/+/77589104:55
openstackgerritArun S A G proposed openstack/ironic master: Validate the kickstart template and file before use  https://review.opendev.org/c/openstack/ironic/+/77744804:55
openstackgerritArun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat  https://review.opendev.org/c/openstack/ironic/+/77805804:55
openstackgerritArun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface  https://review.opendev.org/c/openstack/ironic/+/78039804:55
*** jamesdenton has joined #openstack-ironic04:56
openstackgerritJacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites  https://review.opendev.org/c/openstack/ironic/+/74411705:02
*** paras333_ has joined #openstack-ironic05:21
*** jamesdenton has quit IRC05:32
*** abhishekyadav has joined #openstack-ironic05:32
*** paras333_ has quit IRC05:34
*** jamesdenton has joined #openstack-ironic05:40
*** jamesdenton has quit IRC05:52
*** jamesdenton has joined #openstack-ironic05:53
*** paras33__ has joined #openstack-ironic06:07
arne_wiebalckGood morning, ironic!06:48
openstackgerritliuzhuangzhuang proposed openstack/ironic master: update grub2 file name  https://review.opendev.org/c/openstack/ironic/+/78048106:54
arne_wiebalckTheJulia: https://storyboard.openstack.org/#!/story/200874506:59
*** moshiur has joined #openstack-ironic07:08
openstackgerritAlexander Evseev proposed openstack/bifrost master: Drop external tftp service in favor to use dnsmasq's one  https://review.opendev.org/c/openstack/bifrost/+/78253107:11
jandersgood morning arne_wiebalck o/07:20
*** paras33__ has quit IRC07:21
arne_wiebalckhey janders o/07:22
*** parallax has quit IRC07:29
*** mkrai has joined #openstack-ironic07:33
*** mkrai has quit IRC07:55
*** mkrai_ has joined #openstack-ironic07:55
*** paras333 has joined #openstack-ironic08:09
*** rcernin has quit IRC08:09
*** mkrai_ has quit IRC08:14
*** rpittau|afk is now known as rpittau08:19
*** mkrai has joined #openstack-ironic08:19
rpittaugood morning ironic! o/08:19
*** ricolin has quit IRC08:27
*** ricolin has joined #openstack-ironic08:32
*** dtantsur|afk is now known as dtantsur08:32
dtantsurmorning ironic08:32
dtantsurarne_wiebalck: morning! you mentioned you tried jitter, how exactly did you implement it?08:39
arne_wiebalckdtantsur: this jitter is how you suggested to do it initially: randomized starts of the conductors08:40
arne_wiebalckdtantsur: good morning :)08:40
*** tosky has joined #openstack-ironic08:41
dtantsurah08:42
arne_wiebalckdtantsur: it does the job, but a jitter in the periodics as you have in mind now would be much more convenient08:43
dtantsurI agree, but I haven't found a good way to implement it08:45
*** abhishekyadav has quit IRC08:51
*** rcernin has joined #openstack-ironic08:52
*** lucasagomes has joined #openstack-ironic08:55
*** derekh has joined #openstack-ironic08:56
*** rcernin has quit IRC08:57
openstackgerritMerged openstack/ironic master: Update idrac-redfish RAID release note  https://review.opendev.org/c/openstack/ironic/+/78239108:57
dtantsurcould someone review https://review.opendev.org/c/openstack/ironic/+/782266 please?09:03
dtantsurI hope we can fit it into Wallaby09:03
*** ociuhandu has joined #openstack-ironic09:06
*** rcernin has joined #openstack-ironic09:08
*** ociuhandu has quit IRC09:10
*** ociuhandu has joined #openstack-ironic09:17
rpittaudtantsur: I was just having a look at that, +w09:18
*** dtantsur has quit IRC09:22
*** ociuhandu has quit IRC09:22
*** dtantsur has joined #openstack-ironic09:26
*** mkrai has quit IRC09:26
*** ociuhandu has joined #openstack-ironic09:27
*** dtantsur has quit IRC09:28
*** mkrai has joined #openstack-ironic09:28
*** dtantsur has joined #openstack-ironic09:29
*** rcernin has quit IRC09:32
*** dtantsur has quit IRC09:36
*** ociuhandu has quit IRC09:36
*** dtantsur has joined #openstack-ironic09:38
*** ociuhandu has joined #openstack-ironic09:42
*** ociuhandu has quit IRC09:48
*** parallax has joined #openstack-ironic09:54
*** rcernin has joined #openstack-ironic09:57
*** strigazi has joined #openstack-ironic09:58
*** ociuhandu has joined #openstack-ironic10:01
*** rcernin has quit IRC10:02
*** ociuhandu has quit IRC10:02
*** ociuhandu has joined #openstack-ironic10:03
openstackgerritDhuldev Valekar proposed x/sushy-oem-idrac master: Add reset iDRAC method to manager  https://review.opendev.org/c/x/sushy-oem-idrac/+/78225410:03
openstackgerritJacob Anders proposed openstack/ironic master: Follow up of Use OOB inspection to fetch MACs for IB inspection  https://review.opendev.org/c/openstack/ironic/+/78240610:06
janders^ dtantsur rpittau that stuff up in the unit test was funny :)10:10
*** paras333 has quit IRC10:11
*** rcernin has joined #openstack-ironic10:12
openstackgerritJacob Anders proposed openstack/ironic master: Follow up of Use OOB inspection to fetch MACs for IB inspection  https://review.opendev.org/c/openstack/ironic/+/78240610:17
janders^ fixed the same problem in a better way now10:17
*** rcernin has quit IRC10:22
dtantsurcool10:22
openstackgerritMerged openstack/ironic stable/victoria: Allow unsupported redfish set_boot_mode  https://review.opendev.org/c/openstack/ironic/+/78200510:23
openstackgerritVerification of a change to openstack/tenks failed: Test idempotence of deploy and teardown in CI  https://review.opendev.org/c/openstack/tenks/+/66096410:23
openstackgerritMark Goddard proposed openstack/tenks master: Fix ansible lint  https://review.opendev.org/c/openstack/tenks/+/78268210:35
*** mkrai has quit IRC10:45
*** ociuhandu has quit IRC10:46
*** ociuhandu has joined #openstack-ironic10:47
openstackgerritMerged openstack/ironic master: Support pre-built deploy/rescue ISO in Redfish  https://review.opendev.org/c/openstack/ironic/+/78226610:51
*** ociuhandu has quit IRC10:54
*** rcernin has joined #openstack-ironic11:01
*** ociuhandu has joined #openstack-ironic11:01
*** rcernin has quit IRC11:01
*** rcernin has joined #openstack-ironic11:04
*** ociuhandu has quit IRC11:07
*** ociuhandu has joined #openstack-ironic11:18
*** ociuhandu has quit IRC11:33
iurygregorygood morning janders arne_wiebalck rpittau dtantsur and Ironic o/11:49
rpittauhey iurygregory :)11:49
arne_wiebalckhey iurygregory o/11:49
jandershey iurygregory o/11:54
openstackgerritRiccardo Pittau proposed openstack/bifrost master: [WIP] Allow disabling integrated dhcp from bifrost cli  https://review.opendev.org/c/openstack/bifrost/+/78270011:57
openstackgerritFrancois Rigault proposed openstack/sushy-tools master: Return an integer value for Cores and Threads  https://review.opendev.org/c/openstack/sushy-tools/+/78270111:57
*** rcernin has quit IRC11:58
openstackgerritAija Jauntēva proposed openstack/ironic master: Fix idrac-wsman BIOS step async error handling  https://review.opendev.org/c/openstack/ironic/+/75582111:59
*** ociuhandu has joined #openstack-ironic12:03
*** ociuhandu has quit IRC12:04
*** ociuhandu has joined #openstack-ironic12:05
*** ociuhandu has quit IRC12:05
*** rcernin has joined #openstack-ironic12:06
dtantsuriurygregory: hi, why isn't there backport-candidate for inspector?12:08
*** ociuhandu has joined #openstack-ironic12:11
*** sshnaidm|off is now known as sshnaidm12:13
*** frigo has joined #openstack-ironic12:13
iurygregorydtantsur, this is expected since we are testing in ironic first to see if it would make sense and if it's useful. since my initial idea was to identify the label and add a hashtag for backport to make things more easier (just looking at the hashtag - without having to add this one)12:14
dtantsurah12:14
iurygregorysorry for the long text :D12:14
*** rh-jelabarre has joined #openstack-ironic12:20
openstackgerritFrancois Rigault proposed openstack/sushy-tools master: Return an integer value for Cores and Threads  https://review.opendev.org/c/openstack/sushy-tools/+/78257712:22
*** rcernin has quit IRC12:22
frigoaaah finally! The previous "git review" ended up creating a new change for some reason...12:22
frigoHello Ironic! how are you today12:22
iurygregorymorning frigo =)12:23
dtantsurhi frigo, surviving so far :)12:23
*** ociuhandu has quit IRC12:23
openstackgerritMerged openstack/ironic-inspector master: Use port.id instead of port.uuid  https://review.opendev.org/c/openstack/ironic-inspector/+/78258012:32
openstackgerritMerged openstack/ironic-inspector master: Add ibmc_address to ipmi_address_fields  https://review.opendev.org/c/openstack/ironic-inspector/+/77070812:32
*** ociuhandu has joined #openstack-ironic12:32
*** ociuhandu has quit IRC12:39
*** ociuhandu has joined #openstack-ironic12:40
*** paras333 has joined #openstack-ironic12:41
janderssee you tomorrow Ironic o/12:41
openstackgerritBernd Mueller proposed openstack/ironic-python-agent master: typo chanages -> changes  https://review.opendev.org/c/openstack/ironic-python-agent/+/78270912:53
*** ociuhandu has quit IRC12:55
iurygregorybye janders o/12:57
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector stable/victoria: Use port.id instead of port.uuid  https://review.opendev.org/c/openstack/ironic-inspector/+/78271712:58
*** ociuhandu has joined #openstack-ironic13:01
openstackgerritDhuldev Valekar proposed x/sushy-oem-idrac master: Add reset iDRAC method to manager  https://review.opendev.org/c/x/sushy-oem-idrac/+/78225413:01
*** ociuhandu has quit IRC13:02
*** ociuhandu has joined #openstack-ironic13:02
TheJuliagood morning everyone13:02
dtantsurmorning TheJulia13:03
dtantsurhow is your migraine today?13:03
TheJuliagone, just starting to wake up13:05
iurygregorybrains first13:09
iurygregoryops, breakfast first13:09
dtantsurbrains'n'breakfast13:09
iurygregoryhehe13:10
*** rloo has joined #openstack-ironic13:11
*** jamesdenton has quit IRC13:13
*** jamesdenton has joined #openstack-ironic13:13
*** ociuhandu has quit IRC13:16
TheJuliammm brains and breakfast13:19
TheJuliasounds perfect!13:19
openstackgerritsonali bhausaheb borkar proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port  https://review.opendev.org/c/openstack/ironic/+/69332813:19
*** ociuhandu has joined #openstack-ironic13:25
*** ociuhandu has quit IRC13:30
*** dansmith has quit IRC13:31
*** ociuhandu has joined #openstack-ironic13:35
*** dansmith has joined #openstack-ironic13:37
*** ociuhandu has quit IRC13:41
*** tzumainn has joined #openstack-ironic13:43
*** ociuhandu has joined #openstack-ironic13:53
mnaserhttps://review.opendev.org/c/openstack/ironic/+/782552 is ready for reviews once i get the right amount of rechecks for it to pass :P13:58
iurygregorymnaser, did you give cookies to zuul? :D14:00
openstackgerritDhuldev Valekar proposed openstack/ironic master: DRAC: Added redfish management clean steps  https://review.opendev.org/c/openstack/ironic/+/72159314:01
*** ociuhandu has quit IRC14:04
*** ociuhandu has joined #openstack-ironic14:04
TheJuliazuul cookies are the best cookies of recheck14:10
*** juanoterocas has joined #openstack-ironic14:14
TheJuliaHere brain... where are you?!14:14
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Allow overriding public URL for virtual media  https://review.opendev.org/c/openstack/ironic/+/78276014:25
dtantsurthis is roughly what I had in mind with remote virtual media deployments ^^14:25
iurygregoryfor me it makes sense the idea14:27
TheJuliaI'm ont a fan of the name public, since I don't think any of that should be public like direct attached to the internet ever14:32
TheJuliabut... conceptually I'm okay with it14:32
dtantsurpurple!14:32
dtantsur:)14:32
TheJulias/ont/not/14:32
TheJulia++14:32
dtantsurseriously though, I agree, but I'm out of ideas14:32
dtantsurroutable? will it be clear?14:32
TheJuliawith yellow highlights!14:32
dtantsur\o/14:32
*** ociuhandu has quit IRC14:33
TheJuliapublic might work, and maybe just put in the release note "this does not mean attach diretly to the open internet"14:33
TheJuliaIronic developers will respond with giant mallots14:33
TheJuliaand then cry.... lots14:33
rpittauI have a bad feeling about that :)14:34
dtantsuris external better?14:35
rpittauI was thinking the same14:35
TheJuliaYeah, I kind of like that14:35
TheJuliaso yellow flower boxes now?14:35
dtantsuryep. problem solved!14:39
dtantsurspeaking of which, flower boxes are pending transfer to the balcony finally14:39
openstackgerritJulia Kreger proposed openstack/ironic stable/ussuri: IPMI: Handle vendor set boot device differences  https://review.opendev.org/c/openstack/ironic/+/78276414:40
TheJulia\o/14:40
rpittauthat reminds me we have a couple of flower vases in our balcony that are probably frozen at this point14:40
TheJulia:(14:41
dtantsur:(14:42
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Allow overriding public URL for virtual media  https://review.opendev.org/c/openstack/ironic/+/78276014:42
dtantsurokay, got rid of public now14:42
TheJuliais review loading for folks?14:42
dtantsurehmm, except that in the commit message :D14:42
rpittauit is for me14:42
dtantsurloaded for me14:42
TheJuliaokay, now it loaded14:42
openstackgerritJulia Kreger proposed openstack/ironic stable/ussuri: IPMI: Handle vendor set boot device differences  https://review.opendev.org/c/openstack/ironic/+/78276414:44
openstackgerritJulia Kreger proposed openstack/ironic stable/ussuri: IPMI: Handle vendor set boot device differences  https://review.opendev.org/c/openstack/ironic/+/78276414:45
*** mkrai has joined #openstack-ironic14:51
*** ociuhandu has joined #openstack-ironic14:51
openstackgerritJulia Kreger proposed openstack/ironic stable/train: IPMI: Handle vendor set boot device differences  https://review.opendev.org/c/openstack/ironic/+/78277114:53
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Allow disabling integrated dhcp from bifrost cli  https://review.opendev.org/c/openstack/bifrost/+/78270014:55
TheJuliaa backporting we will go, a backporting we will go....14:55
TheJuliabfournie: I'm guessing https://review.opendev.org/c/openstack/ironic/+/772239 should wait on the ipmi chnage to each branch before backporting? I think that is where things went semi-sideways for me on the pure ipmi change14:56
*** ociuhandu has quit IRC14:58
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Remove explicit default from store_true  https://review.opendev.org/c/openstack/bifrost/+/78277515:02
*** ociuhandu has joined #openstack-ironic15:04
openstackgerritDhuldev Valekar proposed openstack/ironic master: DRAC: Added redfish management clean steps  https://review.opendev.org/c/openstack/ironic/+/72159315:08
*** ociuhandu has quit IRC15:08
*** moshiur has quit IRC15:13
dtantsurTheJulia: yep, I think in the correct order your changes comes first15:17
dtantsurthen my detect_vendor, then Bob's supermicro15:17
openstackgerritJulia Kreger proposed openstack/sushy stable/queens: CI: Restrict tempest run on sushy  https://review.opendev.org/c/openstack/sushy/+/78278115:18
TheJuliaoh, do you have a link to your detect_vendor change?15:18
* TheJulia looks15:19
TheJuliahttps://review.opendev.org/c/openstack/ironic/+/77159515:19
TheJuliaon the list15:20
*** ociuhandu has joined #openstack-ironic15:22
*** frigo has quit IRC15:26
*** ociuhandu has quit IRC15:26
dtantsurTheJulia: my external_blah_blah change, would you qualify it as a feature or more a fix? I'm undecided.15:36
openstackgerritMerged openstack/ironic-inspector stable/victoria: Use port.id instead of port.uuid  https://review.opendev.org/c/openstack/ironic-inspector/+/78271715:37
openstackgerritMerged openstack/ironic stable/stein: [Trivial] Fix testing of volume connector exception  https://review.opendev.org/c/openstack/ironic/+/77946915:38
TheJuliadtantsur: eh, more towards fix as it makes things better for operator configuration15:39
TheJuliaor maybe... more explicit15:39
dtantsuragreed15:46
* TheJulia looks at the loot of zuul cookies and wonders if review takes them15:56
*** ociuhandu has joined #openstack-ironic16:03
bfournieTheJulia: sorry missed the question earlier, yes the redfish change definitely depends on dtantsur detect_vendor change although I *think* it could be backported before the ipmi change, best to backport in consisistent order16:03
TheJuliaI'd prefer not to do before since it creates merge conflicts16:16
*** gyee has joined #openstack-ironic16:23
iurygregoryless merge conflicts == more happiness16:24
TheJulia\o/ https://review.opendev.org/c/openstack/ironic/+/767432 passed CI16:25
TheJuliawell, GrumpyCI as a Service16:25
*** ociuhandu has quit IRC16:26
TheJuliaYes, happiness ++16:26
iurygregoryI will look after lunch o/16:28
TheJuliamuch appreciated16:28
TheJuliait is the last patch on my must merge list for secure rbac for this cycle16:28
openstackgerritDmitry Tantsur proposed openstack/ironic master: Allow overriding an external URL for virtual media  https://review.opendev.org/c/openstack/ironic/+/78276016:31
dtantsurhelp with the release note/docs wording is appreciated ^^16:31
TheJuliareviwing one of arun's patches real quick and will look16:32
*** uzumaki has joined #openstack-ironic16:37
rpittauyou left public in the commit message! :D16:37
dtantsur>_<16:39
dtantsurokay, it's a commit message, it's not operator facing :)16:39
TheJuliaheh16:41
TheJuliadtantsur: commented16:41
*** uzumaki has quit IRC16:41
*** uzumaki has joined #openstack-ironic16:42
openstackgerritDmitry Tantsur proposed openstack/ironic master: Allow overriding an external URL for virtual media  https://review.opendev.org/c/openstack/ironic/+/78276016:43
dtantsurTheJulia: many thanks, what about now ^^?16:43
*** dtantsur is now known as dtantsur|brb16:43
openstackgerritBernd Mueller proposed openstack/ironic master: add hint for getting qcow2 images work  https://review.opendev.org/c/openstack/ironic/+/78282416:46
TheJuliawut16:46
TheJuliadtantsur|brb: maybe requirements, but I'm good with it and riccardo is good with it16:47
TheJuliaany objection if I pluralize it and push it through?16:47
dtantsur|brbnone at all (exercising time, back later)16:50
*** mkrai has quit IRC16:51
*** ociuhandu has joined #openstack-ironic16:52
TheJuliak16:52
openstackgerritJulia Kreger proposed openstack/ironic master: Allow overriding an external URL for virtual media  https://review.opendev.org/c/openstack/ironic/+/78276016:54
*** ociuhandu_ has joined #openstack-ironic16:57
*** ociuhandu has quit IRC17:00
*** ociuhandu_ has quit IRC17:01
openstackgerritMerged openstack/ironic-python-agent master: typo chanages -> changes  https://review.opendev.org/c/openstack/ironic-python-agent/+/78270917:01
*** ociuhandu has joined #openstack-ironic17:07
*** lucasagomes has quit IRC17:08
*** ociuhandu has quit IRC17:11
TheJuliawhere is my mind....17:20
arne_wiebalckTheJulia: In order to fix the problem of the out of sync ESPs on existing machines, I started to RAID them now with a script (umount/create/mount seasoned with some grub2-mkconfig and grubby). Seems to work ok'ish. The real fix should be in the IPA, I guess. Sooner or later someone else will run into this ...17:22
* TheJulia arne_wiebalck Yeah,, I think we've have similar issues we've been chasing down17:23
rpittaugood night! o/17:24
*** rpittau is now known as rpittau|afk17:24
TheJuliagoodnight rpittau|afk17:24
arne_wiebalckTheJulia: we have nodes running around with their rescue kernel :-S17:24
arne_wiebalckbye rpittau|afk o/17:24
TheJuliaugh17:25
arne_wiebalckbye everyone o/17:29
TheJuliastevebaker: rpioso: iurygregory: fyi https://storyboard.openstack.org/#!/story/200871217:56
iurygregoryinteresting...17:58
iurygregoryI'm a bit puzzled by the " cross boot mode scenario "18:01
iurygregoryO.o18:01
iurygregorylike idrac needs to be configured for uefi also no and so the node..18:01
*** jamesdenton has quit IRC18:04
TheJuliawell, I think it is an example of a change being asserted and things are going sideways as a change is queued18:05
*** jamesdenton has joined #openstack-ironic18:05
iurygregoryyeah18:06
iurygregorywe would probably need some condition before trying to call boot_mode_utils.sync_boot_mode(task)18:07
iurygregorynot just go ahead and sync...18:07
TheJuliayeah, likely18:07
TheJuliabut the *pending application* nature complicates that18:08
iurygregorybfournie, fyi also ^18:08
* TheJulia hunts for a bug from like 6 years ago18:08
iurygregoryyeah =(18:09
*** juanoterocas has quit IRC18:10
bfournieso the problem is multiple sets of boot mode resulting in " Pending configuration values are already committed" ?18:18
bfournieugh18:18
*** rcernin has joined #openstack-ironic18:19
bfournieTheJulia: we can make that sync_boot_mode calls vendor specific for Supermicro only like we did here - https://review.opendev.org/c/openstack/ironic/+/773656/1/ironic/drivers/modules/redfish/management.py.  That sync_boot_mode was added just for Supermicro since setting the device w/o the boot_mode resets the boot_mode18:21
*** frigo has joined #openstack-ironic18:22
*** rcernin has quit IRC18:23
frigoarne_wiebalck out of curiosity how do you end up with 2 ESP? and do you know why the wrong ESP would be selected? (normally you give a disk ID when setting boot entries through efibootmgr, the firmware should not have to guess...)18:28
TheJuliafrigo: software raid18:29
frigoalso you are copying a whole disk inside a software raid device ?^H^H18:29
TheJuliaohhh, I do think they do that18:29
frigoso the ESP is like, within the software RAID ? at least from the IPA point of view18:30
TheJuliado I think it is creating a new esp18:31
TheJulias/do/so/18:31
TheJuliaand unique ones on multiple drives comprising the mirror sets for the OS18:31
frigook so a lot of ESP18:32
*** dtantsur|brb is now known as dtantsur18:33
dtantsurfrigo: UEFI does not understand ESP partitions inside software RAID, so we have to copy it from inside of the image to the top level.18:37
bfournieTheJulia: I've added a comment to https://storyboard.openstack.org/#!/story/200871218:37
*** paras333 has quit IRC18:38
frigook, but then the firmware only see that new partition?18:39
frigoso the problem is caused by for example, kernel upgrades, where a different efi partition is picked up?18:40
dtantsuryep. the OS may use the initial partition, but UEFI only recognizes the copied one18:42
dtantsurI guess we may need to change /etc/fstab18:42
frigookok got it. So it's a quite specific problem18:43
dtantsursoftware RAID is a quite specific problem overall :D18:43
dtantsuryou don't want to see the code to build it in ironic :)18:44
dtantsur(not picking on Arne and others, it has to be like that)18:44
frigowhole disk with EFI on software RAID with the grub.cfg inside the EFI partition, and you do some grub update after18:44
frigo(stop ranting about the RAID code:D  )18:44
dtantsurheh18:45
dtantsurhaving grub.cfg on the EFI partition is probably a normal case18:45
dtantsurotherwise how does it know where to find its configuration?18:46
frigo"on my machine"  it redirects to the boot partition18:46
frigo sudo cat efi/EFI/ubuntu/grub.cfg18:46
frigosearch.fs_uuid 80b3c734-07e8-4a49-93e7-d3f7dd828100 root18:46
frigoset prefix=($root)'/grub'18:46
frigoconfigfile $prefix/grub.cfg18:46
dtantsuraha, ubuntu does a cool trick18:47
dtantsurmy fedora really puts the whole file in EFI/fedora18:47
iurygregorybfournie, I think it would be ok to change to only call for Supermicro18:49
bfournieiurygregory: I think so too, we started down that path in https://review.opendev.org/c/openstack/ironic/+/773656 when the ability to detect vendors was added, when https://review.opendev.org/c/openstack/ironic/+/758856 was added we couldn't detect vendors18:52
iurygregoryyeah18:53
iurygregory=)18:53
*** frigo has quit IRC18:55
zer0c00lTheJulia: I updated the release_mappings here  https://review.opendev.org/c/openstack/ironic/+/778058 . Surprisingly it did not raise any conflicts!19:05
TheJuliazer0c00l: okay, thanks!19:06
*** dking has joined #openstack-ironic19:07
dtantsurdinner time, see you tomorrow19:08
*** dtantsur is now known as dtantsur|afk19:08
dkingI feel like this is a silly question, but how do I control boot order for EFI scripts?19:09
dkingI finally got a baremetal server to boot using UEFI and that works fine, but it looks like it's not defaulting to boot on the new ironic efi. I can select it from the BIOS, though.19:09
zer0c00lit is not booting uefi on first boot after deployment?19:11
*** sshnaidm is now known as sshnaidm|afk19:12
dkingzer0c00l: Actually, now that you mention it, yes, I believe that it does. Which makes it more odd. I know that at the end of the firstboot, I typically see a shutdown, and I did with this one, too. So, I suppose the issue occurred from second boot on. That makes me even more puzzled.19:13
dkingI may have just got my head around it. So, when Ironic directly reboots the system, it probably sends the direct command to boot the hard disk. But when it does not, such as when the system is rebooted from the command line or perhaps through a physical power cycle, it uses the BIOS default. And so, I probably should just disable the option to allow boot into the default UEFI shell.19:19
*** uzumaki has quit IRC19:20
stevebakermorning19:29
*** uzumaki has joined #openstack-ironic19:35
*** paras333_ has joined #openstack-ironic19:37
*** livelace has joined #openstack-ironic19:43
TheJuliao/ stevebaker19:45
TheJuliadking: efibootmgr19:45
TheJuliadking: is it just old records that are enitrely invalid and not falling back or...?19:46
TheJuliadking: some do fallback to efishell, fwiw19:47
dkingTheJulia: It's the default UEFI shell that gets booted into.19:48
dkingWhich just seems to boot into an efi shell and sit there.19:48
*** juanoterocas has joined #openstack-ironic19:48
TheJuliadking: so yeah, it *could* be the command to boot to disk being misinterpretted19:49
TheJuliawhat hardware is this?19:49
dkingI was thinking that I would have to do it when I setup UEFI mode, probably through the vendor's command line during inspection. I need something that won't require human intervention.19:50
dkingTheJulia: Supermicro19:50
TheJuliaOH!19:50
TheJuliawhat version of ironic are you running?19:50
dkingIt's Victoria, but it was from a stable branch a few months ago.19:51
dkingAnd standalone.19:51
TheJuliaheh19:51
TheJuliaokay19:51
TheJuliaso... let me get you a link19:51
TheJuliaOkay, apparently this was backpored a while back19:53
TheJuliahttps://opendev.org/openstack/ironic/commit/ab08c200209aa852897b3a936bcaed9928b8b3fe19:53
TheJuliaipmi or redfish?19:54
dkingRedfish at the moment.19:57
TheJuliadking: https://review.opendev.org/c/openstack/ironic/+/77365619:57
dkingOh, neat! So, tht was backported to Victoria, too?19:58
TheJuliayes, that is on victoria19:58
TheJuliaThat may be why things are not quite as expected19:58
TheJuliait going into UEFI shell sounds like that is just the running default of the host19:59
TheJuliaIronic *should* be injecting a new entry and be setting it to default19:59
TheJuliaif the EFI firmware doesn't honor the new entry, I don't know19:59
dkingAwesome! So, that would be a great help! I'll try to update my local images.20:09
dkingTheJulia: Would that just work for IPMI, though, as it mentions ipmitool? Both are available, but we were hoping to move over to Redfish if possible.20:41
*** ociuhandu has joined #openstack-ironic20:43
*** ociuhandu has quit IRC20:48
TheJuliadking: both ipmi and redfish, although slightly different items20:50
TheJuliaone the override is dropped by redfish20:50
TheJuliaipmi, turned out supermicro needed specific raw values to be sent to set it to uefi mode20:51
*** jamesdenton has quit IRC20:52
*** jamesden_ has joined #openstack-ironic20:52
*** rcernin has joined #openstack-ironic21:01
*** rcernin has quit IRC21:01
*** rcernin has joined #openstack-ironic21:02
*** livelace has quit IRC21:03
*** jamesden_ has quit IRC21:25
*** jamesdenton has joined #openstack-ironic21:25
openstackgerritJay Faulkner proposed openstack/ironic-python-agent master: Do not permit IPA standalone to be enabled by conf  https://review.opendev.org/c/openstack/ironic-python-agent/+/78286321:30
openstackgerritMerged openstack/sushy stable/ussuri: Fix ExtendedInfo error handling for non-list item  https://review.opendev.org/c/openstack/sushy/+/78202921:36
openstackgerritJay Faulkner proposed openstack/ironic-python-agent master: Do not permit IPA standalone to be enabled by conf  https://review.opendev.org/c/openstack/ironic-python-agent/+/78286321:52
*** rloo has quit IRC22:09
*** rcernin has quit IRC22:11
*** juanoterocas has quit IRC22:12
*** rcernin has joined #openstack-ironic22:26
*** rcernin has quit IRC22:31
*** rcernin has joined #openstack-ironic22:32
jandersgood morning Ironic o/22:33
iurygregorygood morning janders o/23:03
jandershey iurygregory o/23:03
*** jamesdenton has quit IRC23:32
*** jamesdenton has joined #openstack-ironic23:33
*** gyee has quit IRC23:35
jandersis there any known problem with CI? I had a simple patch fail CI three times, on a different test each time :/23:38
*** uzumaki has quit IRC23:49
*** uzumaki has joined #openstack-ironic23:49
« Tuesday, 2021-03-23 Index Thursday, 2021-03-25 »

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