Tuesday, 2019-08-27

*** rh-jelabarre has quit IRC00:14
*** kbaegis has quit IRC00:18
*** gyee has quit IRC00:18
*** threestrands has joined #openstack-ironic00:24
*** seongsoocho has joined #openstack-ironic00:35
*** ociuhandu has joined #openstack-ironic00:53
*** gregoryo has joined #openstack-ironic00:57
*** ociuhandu has quit IRC00:57
*** rcernin has quit IRC01:13
*** dsneddon has quit IRC01:46
*** dsneddon has joined #openstack-ironic01:48
*** dsneddon has quit IRC01:54
*** mbeierl has quit IRC01:54
*** dsneddon has joined #openstack-ironic01:59
*** dsneddon has quit IRC02:03
*** rloo has quit IRC02:06
*** andrein has quit IRC02:11
*** rcernin has joined #openstack-ironic02:13
*** kaifeng has joined #openstack-ironic02:15
kaifengMorning ironic o/02:15
kaifengFYI our whiteboard is modified by unknown, part of contents are updated by machine translation. Please use a new etherpad for doing this, e.g. IronicWhiteBoard.<locale> and put a link in the main whiteboard.02:20
*** whoami-rajat has joined #openstack-ironic02:27
*** dsneddon has joined #openstack-ironic02:32
openstackgerritMerged openstack/ironic-python-agent-builder master: Fix a typo that prevents AUTHORIZE_SSH from working  https://review.opendev.org/67861103:14
openstackgerritMerged openstack/ironic-python-agent-builder master: Accounts for Ubuntu paths on pip installation  https://review.opendev.org/67860603:17
*** gkadam has joined #openstack-ironic03:51
*** gkadam has quit IRC03:51
*** jtomasek has quit IRC04:00
*** cp is now known as cp-04:02
*** Nisha_Agarwal has joined #openstack-ironic04:05
*** stendulker has joined #openstack-ironic04:12
*** Nisha_Agarwal has quit IRC04:15
*** bdodd_ has joined #openstack-ironic04:17
*** bdodd has quit IRC04:17
*** mkrai has joined #openstack-ironic04:31
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow-up for deploy steps for Redfish BIOS interface  https://review.opendev.org/67769504:40
*** jtomasek has joined #openstack-ironic04:41
*** whoami-rajat has quit IRC04:42
*** dsneddon has quit IRC04:45
*** ociuhandu has joined #openstack-ironic04:53
*** ociuhandu has quit IRC04:57
*** dsneddon has joined #openstack-ironic05:07
*** dsneddon has quit IRC05:15
*** dsneddon has joined #openstack-ironic05:47
*** rachit7 has joined #openstack-ironic05:48
*** hamzy has quit IRC05:59
openstackgerritMichael Davies proposed openstack/ironic master: Adding file uri support for ipa image location  https://review.opendev.org/67872906:03
*** andrein has joined #openstack-ironic06:05
*** e0ne has joined #openstack-ironic06:06
*** e0ne has quit IRC06:11
*** dsneddon has quit IRC06:15
*** dsneddon has joined #openstack-ironic06:16
arne_wiebalckGood morning, ironic!06:26
*** whoami-rajat has joined #openstack-ironic06:30
kaifengmorning arne_wiebalck o/06:38
arne_wiebalckgm kaifeng o/06:38
*** dsneddon has quit IRC06:41
*** dsneddon has joined #openstack-ironic06:44
*** dsneddon has quit IRC06:44
mkraimorning arne_wiebalck kaifeng o/06:44
arne_wiebalckhey mkrai o/06:44
kaifengmorning mkrai06:45
*** dsneddon has joined #openstack-ironic06:48
*** trident has quit IRC07:00
*** mkrai_ has joined #openstack-ironic07:02
*** mkrai__ has joined #openstack-ironic07:05
*** mkrai has quit IRC07:05
*** mkrai_ has quit IRC07:08
*** trident has joined #openstack-ironic07:10
etingofrpioso, I am referring to the response body (which is missing, despite being recommended), not to the response header (which is present)07:19
*** threestrands has quit IRC07:32
*** dsneddon has quit IRC07:32
*** dtantsur|afk is now known as dtantsur07:37
dtantsurthanks TheJulia07:37
dtantsurmorning everyone07:37
*** stendulker has quit IRC07:38
*** rcernin has quit IRC07:40
*** andrein has quit IRC07:52
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: DIB: add missing ironic-python-agent-create-rescue-user.sh  https://review.opendev.org/67874207:52
dtantsurnext fix if you have time today morning ^^07:52
openstackgerritMerged openstack/ironic-python-agent master: Handle floppy disk controllers  https://review.opendev.org/67728107:53
*** lucasagomes has joined #openstack-ironic07:53
kaifengmorning dtantsur08:00
*** dsneddon has joined #openstack-ironic08:01
openstackgerritMerged openstack/ironic-python-agent master: Release note follow-up to floppy fix  https://review.opendev.org/67863908:01
dtantsurkaifeng: I really wonder how we revert the etherpad..08:04
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Add a diskimage-builder based image building job  https://review.opendev.org/67796008:10
openstackgerritNisha Brahmankar proposed openstack/ironic-python-agent master: Adds CI job that uses DIB build images  https://review.opendev.org/67623908:11
kaifengdtantsur: etherpad has history, but I looked and found the last save point is in Aug 27, so I guess there is no other choices other than manually.08:11
dtantsursigh08:11
kaifengI have update weekly priority section, but I barely looked the 3rd party ci so can't remember what it said before..08:12
*** andrein has joined #openstack-ironic08:13
kaifengmaybe I can leave some hint words there, who are familiar with the original text could do a restoration.08:13
*** mkrai__ has quit IRC08:15
*** dougsz has joined #openstack-ironic08:17
dtantsuryep, thanks!08:19
*** alexmcleod has joined #openstack-ironic08:22
kaifengdone, feel free to update my terrible reverse translation :P08:28
dtantsurkaifeng: many-many thanks! I'll let the CI people recover their notes. That's how we'll learn whether they're paying attention or not ;)08:35
*** gregoryo has quit IRC08:40
*** mkrai__ has joined #openstack-ironic08:44
*** priteau has joined #openstack-ironic08:50
*** derekh has joined #openstack-ironic08:50
*** mkrai__ has quit IRC08:54
*** bobmel has quit IRC08:54
*** Lucas_Gray has joined #openstack-ironic09:01
kaifengdtantsur: no big deal :) Just saw the ML it could be some magic done by the Chrome..09:07
dtantsuryeah09:07
*** Lucas_Gray has quit IRC09:08
*** mkrai__ has joined #openstack-ironic09:10
*** Lucas_Gray has joined #openstack-ironic09:13
openstackgerritmurat emir proposed openstack/bifrost master: Add bifrost-configure-software-raid role  https://review.opendev.org/67861409:23
*** xXraphXx has joined #openstack-ironic09:26
*** dtantsur is now known as dtantsur|bbl09:31
*** lucasagomes has quit IRC09:53
*** lucasagomes has joined #openstack-ironic09:58
*** bobmel has joined #openstack-ironic09:59
*** Lucas_Gray has quit IRC10:04
*** Lucas_Gray has joined #openstack-ironic10:06
*** e0ne has joined #openstack-ironic10:11
*** Nisha_ has joined #openstack-ironic10:21
*** e0ne has quit IRC10:26
*** e0ne has joined #openstack-ironic10:28
*** mkrai__ has quit IRC10:30
*** e0ne has quit IRC10:32
openstackgerritmurat emir proposed openstack/bifrost master: Add bifrost-configure-software-raid role  https://review.opendev.org/67861410:47
TheJuliawow my head hurts :(11:02
TheJuliaso what language did the etherpad get translated to?11:03
* TheJulia goes and finds coffee11:03
TheJuliadtantsur|bbl: I guess the ironic-python-agent post-job is failing? last build 8/22 :(11:05
openstackgerritMerged openstack/networking-baremetal master: Fix unit tests for networking-baremetal  https://review.opendev.org/67864611:08
*** tesseract has joined #openstack-ironic11:11
kaifengTheJulia: it's Simplified Chinese :)11:14
TheJuliakaifeng: Somehow I had a feeling that would be the case :)11:15
TheJuliacan I get another core reviewer to look at https://review.opendev.org/#/c/678742/111:17
patchbotpatch 678742 - ironic-python-agent-builder - DIB: add missing ironic-python-agent-create-rescue... - 1 patch set11:17
*** hamzy has joined #openstack-ironic11:18
* TheJulia gets coffee11:18
*** Nisha_ has quit IRC11:23
kaifengTheJulia: looks good, but I didn't look at the script that generates password11:23
kaifengI'll assume it works fine11:24
openstackgerritM V P Nitesh proposed openstack/ironic master: UEFI PXE Provisioning for wholedisk images  https://review.opendev.org/66486411:25
TheJuliakaifeng: the password is written out by ipa11:27
*** bdodd has joined #openstack-ironic11:34
*** bdodd_ has quit IRC11:35
kaifengTheJulia: hmm, there is a create_rescue_user seems to be doing the same thing?11:39
TheJuliaI think there are two cases, one the user provides it or the ramdisk generates and provides it11:40
kaifengack, thanks for the explaination, we usually use dib with fixed password for accessing bms.11:45
*** bfournie has quit IRC11:50
TheJuliaThere was something about it, it was just so long ago11:54
TheJuliaI think otherwise nova generates a random password to send to ironic11:54
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow-up for deploy steps for Redfish BIOS interface  https://review.opendev.org/67769511:55
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add iLO RAID deploy steps  https://review.opendev.org/67427111:55
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add deploy steps for iLO BIOS interface  https://review.opendev.org/67427011:55
openstackgerritShivanand Tendulker proposed openstack/ironic master: WIP: Add iDRAC RAID deploy steps  https://review.opendev.org/64173111:57
kaifengIndeed, admin_pass if memory served, but it needs things like cloudinit to apply the configuration12:01
*** rh-jelabarre has joined #openstack-ironic12:05
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow-up for deploy steps for Redfish BIOS interface  https://review.opendev.org/67769512:06
*** dtantsur|bbl is now known as dtantsur12:08
dtantsurTheJulia: oh damn, you're right http://zuul.openstack.org/builds?job_name=ironic-python-agent-buildimage-tinyipa12:08
dtantsuralso g'morning12:08
TheJulia*sigh*12:09
dtantsurmy bad indeed, sorry..12:11
*** mrtmr has joined #openstack-ironic12:12
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Fix the post job for tinyipa/coreos  https://review.opendev.org/67879512:13
dtantsurTheJulia, kaifeng ^^12:13
openstackgerritShivanand Tendulker proposed openstack/ironic master: WIP: Add iDRAC RAID deploy steps  https://review.opendev.org/64173112:14
dtantsurat some point I'll stop messing with these jobs. at some point in a distant future..12:14
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add deploy steps for iLO BIOS interface  https://review.opendev.org/67427012:15
openstackgerritAnkit Kumar proposed openstack/ironic master: Add iso less virtual media deploy  https://review.opendev.org/67799612:15
mrtmrhey everyone, I send a new review to bifrost but there is a bug i guess before to my commit it is about opensuse.in my opinion zuul not verified my review because of this12:15
TheJuliamrtmr: link please?12:16
TheJuliadtantsur: approved, since it is a post job12:17
dtantsurthx!12:17
TheJuliadtantsur: no worries12:17
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add iLO RAID deploy steps  https://review.opendev.org/67427112:17
mrtmrTheJulia https://review.opendev.org/#/c/678614/ this mine and this one is about opensuse https://review.opendev.org/#/c/677181/12:19
patchbotpatch 678614 - bifrost - Add bifrost-configure-software-raid role - 5 patch sets12:19
patchbotpatch 677181 - bifrost (stable/stein) - Remove openSUSE Leap 42.3  jobs - 1 patch set12:19
mrtmrowww okey12:19
TheJuliainteresting....12:23
TheJuliayeah, the master branch change already merged, your committing to master12:23
TheJuliadtantsur: the real downside with all the various gate issues is... having to wait :(12:25
dtantsurtrue :(12:25
dtantsurbut I'm fixing several at the same time, so it's never boring \o/12:25
*** bobmel has quit IRC12:30
*** bobmel has joined #openstack-ironic12:30
*** bobmel has quit IRC12:30
mrtmrmy checks are passed except this one. could you review my commit O:312:34
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Run sqlite in MT mode  https://review.opendev.org/67879712:35
*** bobmel has joined #openstack-ironic12:36
*** dsneddon has quit IRC12:36
*** bfournie has joined #openstack-ironic12:39
*** mbeierl has joined #openstack-ironic12:41
*** priteau has quit IRC12:44
*** mbeierl has quit IRC12:45
*** dougsz has quit IRC12:46
*** rloo has joined #openstack-ironic12:52
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Run a standalone job in the gate  https://review.opendev.org/67593712:53
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: DNM testing the new CI job  https://review.opendev.org/67880912:54
TheJuliadtantsur: heh, yeah, that is kind of where my evening was last night12:56
*** dougsz has joined #openstack-ironic12:57
openstackgerritNisha Brahmankar proposed openstack/ironic-python-agent master: Adds CI job that uses DIB build images  https://review.opendev.org/67623912:59
*** bobmel has quit IRC13:00
*** mbeierl has joined #openstack-ironic13:00
*** jcoufal has joined #openstack-ironic13:00
*** bobmel has joined #openstack-ironic13:00
*** dsneddon has joined #openstack-ironic13:03
*** Goneri has joined #openstack-ironic13:05
*** yolanda has quit IRC13:06
*** yolanda__ has joined #openstack-ironic13:06
openstackgerritMerged openstack/ironic-python-agent-builder master: DIB: add missing ironic-python-agent-create-rescue-user.sh  https://review.opendev.org/67874213:06
*** dsneddon has quit IRC13:09
*** sthussey has joined #openstack-ironic13:10
TheJuliawe really need to get an updated IPA image up :(13:21
dtantsuryeah..13:23
dtantsursorry, this IPA-builder work messes up everything :)13:23
*** Lucas_Gray has quit IRC13:24
*** whoami-rajat has quit IRC13:25
TheJuliaNo worries13:27
dtantsurwe should really rewrite the whole post job in ansible13:39
* dtantsur puts on his TODO list13:39
*** dsneddon has joined #openstack-ironic13:42
*** priteau has joined #openstack-ironic13:44
TheJuliaHeh13:46
TheJuliaThe append only list? :)13:46
dtantsurwell, sometimes I remove items from it. Sometimes because they're done :)13:48
dtantsure.g. the IPA-builder work has been on it for.. mm.. years?13:48
*** rachit7 has quit IRC13:51
*** rloo has quit IRC13:52
*** e0ne has joined #openstack-ironic13:54
*** mkrai has joined #openstack-ironic13:58
TheJuliaheh13:58
TheJuliayeah13:58
*** jcoufal_ has joined #openstack-ironic14:02
*** rachit7 has joined #openstack-ironic14:03
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Do not reuse sqlite connection  https://review.opendev.org/67879714:05
*** jcoufal has quit IRC14:05
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Do not reuse sqlite connection  https://review.opendev.org/67879714:08
*** rloo has joined #openstack-ironic14:13
openstackgerritShivanand Tendulker proposed openstack/ironic master: Follow-up for deploy steps for Redfish BIOS interface  https://review.opendev.org/67769514:16
openstackgerritJulia Kreger proposed openstack/networking-baremetal master: Fix networking-baremetal CI  https://review.opendev.org/67570114:36
*** priteau has quit IRC14:39
openstackgerritJulia Kreger proposed openstack/ironic master: Extend base runtime  https://review.opendev.org/67887014:48
*** whoami-rajat has joined #openstack-ironic14:48
*** rloo has quit IRC14:52
TheJuliadtantsur: ^^^ networking baremetal fires 2x VMs on a single test, which normally I don't think should be a big deal as we know, but it is under the single wallclock of the job, so I think extending might be a good idea sine that should cover... most other similar issues. Initial networking failures will get caught by the callback timer anyway.14:57
TheJuliaalso, silly variable performance test VMs14:57
TheJuliaThe networking-baremetal fix also now disables cleaning... for it is just silly to test14:58
*** dsneddon has quit IRC14:58
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Abort waiting early if reached a failure state  https://review.opendev.org/67887715:03
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: DNM testing the new CI job  https://review.opendev.org/67880915:03
*** dsneddon has joined #openstack-ironic15:04
*** rachit7 has quit IRC15:06
*** alexmcleod has quit IRC15:07
*** dsneddon has quit IRC15:10
*** mkrai has quit IRC15:14
*** mrtmr has quit IRC15:18
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888415:30
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888415:32
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888415:33
*** gyee has joined #openstack-ironic15:37
*** dsneddon has joined #openstack-ironic15:39
*** mkrai has joined #openstack-ironic15:42
TheJuliagahhhhh15:45
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888415:45
*** e0ne has quit IRC15:52
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888416:04
*** lucasagomes has quit IRC16:07
*** jawad_axd has joined #openstack-ironic16:10
*** alexmcleod has joined #openstack-ironic16:17
openstackgerritMerged openstack/ironic-python-agent master: Fix the post job for tinyipa/coreos  https://review.opendev.org/67879516:17
*** mkrai has quit IRC16:19
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888416:20
*** andrein has quit IRC16:29
*** dtantsur is now known as dtantsur|afk16:42
*** amotoki is now known as amotoki_16:42
dtantsur|afkg'night16:42
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888416:47
*** rloo has joined #openstack-ironic16:53
*** rloo has quit IRC16:55
*** rloo has joined #openstack-ironic16:55
*** dougsz has quit IRC16:59
*** e0ne has joined #openstack-ironic17:01
*** derekh has quit IRC17:02
*** jawad_axd has quit IRC17:03
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: Software raid: default partition table type change  https://review.opendev.org/67480317:04
*** andrein has joined #openstack-ironic17:07
*** alexmcleod has quit IRC17:21
openstackgerritJulia Kreger proposed openstack/ironic-tempest-plugin master: Enable boot_option to be specified  https://review.opendev.org/67863317:29
*** tesseract has quit IRC17:30
*** rh-jelabarre has quit IRC17:36
*** rh-jelabarre has joined #openstack-ironic17:38
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888417:38
openstackgerritJulia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local  https://review.opendev.org/49211417:40
TheJuliawoot, tinyipa master is updated17:43
TheJulia*dance*17:43
*** e0ne has quit IRC17:49
*** Nisha_Agarwal has joined #openstack-ironic17:49
*** jcoufal_ has quit IRC17:55
*** jcoufal has joined #openstack-ironic17:59
TheJuliabdodd: you around?18:17
*** jcoufal_ has joined #openstack-ironic18:29
*** jcoufal has quit IRC18:32
*** Nisha_Agarwal has quit IRC18:34
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job  https://review.opendev.org/67888418:47
*** e0ne has joined #openstack-ironic18:51
*** andrein has quit IRC19:03
NobodyCamGood Morning, well Afternoon Ironic folks!19:05
*** e0ne has quit IRC19:19
TheJuliao/ NobodyCam19:34
TheJuliaIt would be greatly appreciated if cores could take a look at https://review.opendev.org/#/c/676889/19:35
patchbotpatch 676889 - sushy - Change OEM extensions architecture - 3 patch sets19:35
openstackgerritMerged openstack/ironic master: Adjust placement query for reserved nodes  https://review.opendev.org/67867519:38
*** andrein has joined #openstack-ironic19:45
rlooTheJulia: I looked and I left some questions :)20:05
*** jcoufal has joined #openstack-ironic20:17
*** ijw has joined #openstack-ironic20:19
*** jcoufal_ has quit IRC20:20
*** andrein has quit IRC20:27
dhellmannI think I've asked this before, but how can I get a list of all of the kinds of cleaning steps that I can add to a manual cleaning API call? https://docs.openstack.org/api-ref/baremetal/?expanded=change-node-provision-state-detail#change-node-provision-state20:44
JayFI don't believe that API exists, but IMBW (more like, IMB out of date)20:45
rlooJayF: you are not wrong. it doesn't exist. we tried/struggled/couldn't figure out how to do it programmatically cuz it depends on IPA.20:46
*** jcoufal has quit IRC20:46
rloodhellmann: i think we decided to document them, but I don't think we even did that :-( Let me see...20:47
dhellmanndocs would be ok20:47
dhellmanntoday I have no idea what I can pass to that call, so I'm just looking for some basic info20:47
JayFThe cleaning steps available can vary based on hardware on the system and plugins (if added) to the ironic python agent20:48
rloodhellmann: so sorry, we didn't document them: https://docs.openstack.org/ironic/latest/admin/cleaning.html20:48
JayFso we could document the standard set20:48
dhellmannyeah, I started with that doc20:48
dhellmannwhere would I go looking in code if I was going to try to figure it out that way?20:48
dhellmannhaving a list of the standard items in the docs would be helpful, yeah20:49
rloodhellmann: ilo is pretty good with their docs: https://docs.openstack.org/ironic/latest/admin/drivers/ilo.html#node-cleaning-support20:49
dhellmannoh, so there might be more info in the driver docs?20:49
dhellmannah, well, not for idrac20:50
rloodhellmann: well. yeah. depends on the vendor and if they documented.20:50
dhellmannhow does the software know what steps there are? if I wanted to dig in the code, what would I be looking for?20:51
rloodhellmann: code that are clean steps have the clean_step decorator.20:52
dhellmannok, cool, thanks20:52
rloodhellmann: something like @base.clean_step(priority=0).20:52
dhellmannwhich repo(s) should I look at?20:53
dhellmannironic? ipa?20:53
rloodhellmann: https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/base.py#L150920:53
dhellmannso all of the code using that decorator will be in the ironic repo?20:54
rloodhellmann: that's for ironic. for ipa, i'm not sure how those are implemented.20:54
dhellmannok20:54
*** ijw has quit IRC20:57
*** ijw has joined #openstack-ironic20:59
rloodhellmann: wrt ipa, I see this documentation: https://docs.openstack.org/ironic-python-agent/latest/contributor/hardware_managers.html#custom-hardwaremanagers-and-cleaning21:00
jrollin ipa, a "hardware manager" provides clean steps. it implements a `get_clean_steps` method to define them. example: https://opendev.org/x/proliantutils/src/branch/master/proliantutils/ipa_hw_manager/hardware_manager.py#L2521:01
dhellmannso are the steps defined by ironic? or the agent? or both?21:01
jrollin-tree example: https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/hardware.py#L139321:01
dhellmannENOIDEAWHATIMDOING21:01
jrollboth, yes21:01
dhellmannok21:02
jrollironic collects them from the agent. which is why it's hard to make an API for this.21:02
*** ijw has quit IRC21:03
dhellmannI can see that "what cleaning steps can I use with my hardware?" is hard to answer. I would settle for an answer to "what are all of the possible cleaning steps available at all?"21:03
dhellmanneven if it's just based on whatever is in the stock code bases for ironic components, and doesn't take into account user-installed extensions21:04
rloodhellmann: i suppose we could do that (answer the latter question for intree stuff). in a doc... Now we just need some warm bodies to do it.21:04
JayFdhellmann: part of the answer to "what are all of the possible cleaning steps available at all?" can be dependant on the agent hardware managers loaded (Which are intentionally downstream-pluggable + can turn on/off based on hardware) ... so even that is hard to answer exactly (although we could document the set that upstream ships, for sure)21:12
dhellmannJayF : how about "what clean steps are implemented anywhere in the upstream ironic source code?21:12
JayFThat is possible. And something that should be documented statically if nothing else21:13
dhellmannI mean, help me out here. I don't want to have to internalize all of the ironic source code in order to be able to use the api! :-)21:13
*** Goneri has quit IRC21:13
JayFCan we get past the Y and go back to the X  -- what do you specifically want to do right now21:13
rloodhellmann: we discussed how-to-get-clean-steps several times, eg in queens (https://etherpad.openstack.org/p/ironic-queens-ptg-contentious-topics, L79). TheJulia took a stab at a spec for it I believe, and I think we agreed that we couldn't agree on an API and that we should document it. but I think we forgot/got too busy/etc.21:14
JayFWe can probably get you the info you need to be unblocked right now, but I doubt anyone is going to go run make the docs right now :)21:14
* TheJulia twitches21:15
dhellmannI am building a thing that sits between my users and ironic. I want to add things like BIOS updates and making sure the host is wiped before it is reused. I understand that those are "cleaning steps" but I don't now what steps they are, how to trigger them, and what else might be available.21:15
* TheJulia might be enjoying nikka Coffey grain21:15
dhellmannso I'm not blocked on trying to make a specific thing work, I am blocked on not knowing what the features even are :-)21:15
dhellmannknowing how to look in the source to find the steps is better than nothing at all, so I'll read some code for a while21:16
JayFIf you're using just an IPMI managed host, the clean steps we ship in the agent are primarily around disk-wiping21:16
* TheJulia reads21:16
JayFdhellmann: if you end up needing to write a custom hardware manager to do some of these (likely you will), you probably will want these: https://github.com/openstack/ipa-example-hardware-managers -- this + IPA hardware manager docs lay out that flow pretty well... I can't speak as much to the non-agent-related cleaning stuff21:17
dhellmannif I end up writing a custom hardware manager I think I'll just declare failure and stop21:17
dhellmannI want to use the features you all have built21:17
JayFan interface to allow plugging in to run whatever insane bios update tool your vendor came up with is one of the features ;)21:18
*** whoami-rajat has quit IRC21:18
dhellmannok, but I'm not going to be doing that or allowing my users to do that. so what do I get out of the box?21:18
JayFIf you're using ipmi+agent, disk wiping.21:18
dhellmannI'll dig around for methods with that decorator on them and see what I get21:18
JayFI think the ilo driver may do some stuff aroudn bios updates, those docs were posted above21:19
JayFThe iLo specific stuff, that does a bit more out of the box -> https://docs.openstack.org/ironic/latest/admin/drivers/ilo.html#node-cleaning-support21:19
dhellmannthis looks like a nice list for me to study: http://paste.openstack.org/show/765827/21:19
JayFThere's also the in-band cleaning bits...  https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L139321:21
JayF(we ship other hardware managers but none implement any cleaning steps)21:21
TheJuliaWe didn't even really decide to document them because of the hardware manager model21:23
TheJuliaIt was more "if we do anything we may document it"21:23
TheJuliadhellmann:  hmmm studying that list is a bit much. :( But if there is a non zero priority it should run21:25
TheJuliaBut only on that hardware type that is constructed21:25
dhellmannsure, I get that these things are hardware-specific21:26
*** jtomasek has quit IRC21:29
*** bfournie has quit IRC21:29
*** ijw has joined #openstack-ironic21:34
*** ijw has quit IRC21:37
*** ijw has joined #openstack-ironic21:37
*** trident has quit IRC22:05
*** trident has joined #openstack-ironic22:13
*** rcernin has joined #openstack-ironic22:15
*** bfournie has joined #openstack-ironic22:20
dhellmannthis code snippet and its output might be of interest as an outcome of the previous discussion: http://paste.openstack.org/show/765980/22:20
dhellmannthe list of clean steps isn't actually very interesting, or long, if you don't have access to the agent to ask it22:21
dhellmannalthough I don't see iLo in that list at all, so maybe I'm missing a dependency22:21
TheJuliadhellmann: in short, pip install proliantutils && add ilo to the enabled_hardware_types setting22:26
dhellmannI'm just loading code directly, I don't have a config file :-(22:28
dhellmannI'm trying to write you a little plugin for sphinx to build a table of this stuff22:28
TheJuliahmm...22:29
TheJulia<322:29
bdoddTheJulia: Just saw your ping from earlier today. I've been away helping my mom and dad after her recent back surgery. And will probably be doing the same for much of the next week or so.22:45
TheJuliabdodd: eek, Totally understand! I was only pinging you regarding etingof's comments just to see if you were aware, but family is more important :)22:48
TheJuliaHope she heals quickly :)22:49
bdoddTheJulia Yes, I saw them and am aware. It's next on my list to work on as soon as I can get to it. :-)22:49
bdoddThanks! :-)22:49
rpiosobdodd: My thoughts are with you and your parents.22:50
bdoddrpioso Thank you!22:50
rpiosobdodd: yw22:50
dhellmannok, here's a more complete list of the steps available through each driver: http://paste.openstack.org/show/766134/22:58
dhellmannif I was going to build a table of this information, how would I organize it? by driver? by interface?22:58
*** edleafe has left #openstack-ironic23:02
dhellmannhmm, the ilo docs list some of the cleaning steps as "automatic" and others as "manual" -- is there a way to tell those apart in the code?23:07
*** sthussey has quit IRC23:55

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