*** diurnalist has joined #openstack-ironic | 00:00 | |
*** diurnalist has quit IRC | 00:09 | |
*** yaawang has quit IRC | 00:14 | |
*** yaawang has joined #openstack-ironic | 00:14 | |
*** diurnalist has joined #openstack-ironic | 00:36 | |
*** diurnalist has quit IRC | 00:59 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert root controller to plain controller https://review.opendev.org/741064 | 01:02 |
---|---|---|
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert v1 controller to plain, return JSON https://review.opendev.org/741065 | 01:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Break out collection functions for json usage https://review.opendev.org/743676 | 01:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: New argument validate decorator https://review.opendev.org/742795 | 01:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert drivers endpoint to plain JSON https://review.opendev.org/741865 | 01:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Add status_code argument to @method.expose https://review.opendev.org/743677 | 01:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Add @method.body decorator https://review.opendev.org/743678 | 01:02 |
openstackgerrit | Steve Baker proposed openstack/ironic master: Convert allocation endpoint to plain JSON https://review.opendev.org/743679 | 01:02 |
*** diurnalist has joined #openstack-ironic | 01:11 | |
*** ricolin has joined #openstack-ironic | 01:12 | |
*** diurnalist has quit IRC | 01:16 | |
*** k_mouza has joined #openstack-ironic | 01:19 | |
*** janders has quit IRC | 01:20 | |
*** k_mouza has quit IRC | 01:24 | |
*** yaawang has quit IRC | 01:31 | |
*** yaawang has joined #openstack-ironic | 01:32 | |
*** diurnalist has joined #openstack-ironic | 01:43 | |
*** k_mouza has joined #openstack-ironic | 01:49 | |
*** diurnalist has quit IRC | 01:50 | |
*** diurnalist has joined #openstack-ironic | 01:51 | |
*** k_mouza has quit IRC | 01:53 | |
*** rh-jlabarre has quit IRC | 01:55 | |
*** janders has joined #openstack-ironic | 02:02 | |
*** diurnalist has quit IRC | 02:09 | |
*** diurnalist has joined #openstack-ironic | 02:17 | |
*** diurnalist has quit IRC | 02:25 | |
*** rloo has quit IRC | 02:25 | |
*** diurnalist has joined #openstack-ironic | 02:27 | |
*** gyee has quit IRC | 02:34 | |
*** diurnalist has quit IRC | 02:37 | |
*** yaawang has quit IRC | 02:41 | |
*** yaawang has joined #openstack-ironic | 02:42 | |
*** k_mouza has joined #openstack-ironic | 02:42 | |
*** tzumainn has quit IRC | 02:42 | |
*** Goneri has quit IRC | 02:46 | |
*** k_mouza has quit IRC | 02:47 | |
*** SuiongNg|ITRI_TW has joined #openstack-ironic | 03:00 | |
*** yaawang has quit IRC | 03:14 | |
*** yaawang has joined #openstack-ironic | 03:15 | |
*** k_mouza has joined #openstack-ironic | 03:20 | |
*** diurnalist has joined #openstack-ironic | 03:23 | |
*** k_mouza has quit IRC | 03:25 | |
*** diurnalist has quit IRC | 03:29 | |
*** yaawang has quit IRC | 03:47 | |
*** yaawang has joined #openstack-ironic | 03:48 | |
*** diurnalist has joined #openstack-ironic | 03:59 | |
*** sziviani has quit IRC | 04:04 | |
*** ociuhandu has joined #openstack-ironic | 04:22 | |
*** ociuhandu has quit IRC | 04:26 | |
*** diurnalist has quit IRC | 04:32 | |
openstackgerrit | paresh sao proposed openstack/ironic master: [WIP] OOB one button secure erase for iLO5 based HPE Proliant servers. https://review.opendev.org/743868 | 05:12 |
*** diurnalist has joined #openstack-ironic | 05:19 | |
*** yaawang has quit IRC | 05:23 | |
*** yaawang has joined #openstack-ironic | 05:24 | |
*** diurnalist has quit IRC | 05:24 | |
*** diurnalist has joined #openstack-ironic | 06:10 | |
openstackgerrit | Felix Maurer proposed openstack/ironic master: Use TLS for json_rpc when configured https://review.opendev.org/743912 | 06:11 |
*** diurnalist has quit IRC | 06:15 | |
*** akahat is now known as akahat|PTO | 06:30 | |
*** ociuhandu has joined #openstack-ironic | 06:30 | |
*** yaawang has quit IRC | 06:32 | |
*** yaawang has joined #openstack-ironic | 06:34 | |
*** ociuhandu has quit IRC | 06:35 | |
*** ricolin has quit IRC | 06:43 | |
*** ricolin has joined #openstack-ironic | 06:43 | |
*** ricolin has quit IRC | 06:44 | |
*** uzumaki has joined #openstack-ironic | 06:45 | |
uzumaki | morning ironic o/ | 06:45 |
janders | good morning uzumaki | 06:46 |
uzumaki | you's it going janders ? o/ | 06:46 |
uzumaki | how's* | 06:46 |
janders | doing well, thank you :) | 06:46 |
janders | how are you? | 06:46 |
*** ricolin has joined #openstack-ironic | 06:46 | |
uzumaki | I'm fine, thanks :) what's up? | 06:46 |
janders | tackling some interesting MultiOpt + list-of-dicts issues | 06:47 |
janders | I've been fighting this for the last couple days | 06:47 |
uzumaki | Ohh my | 06:47 |
janders | I got some pointers yesterday from the guys here as well as oslo guys | 06:47 |
janders | re-testing now | 06:47 |
janders | was working on something else earlier in the day | 06:47 |
uzumaki | I like how you say "interesting issues" meaning you perception of the problem is not frustration, you're focused ;) | 06:47 |
janders | there is an element of frustration but as long as there are knowledgeable people happy to give a hand, it's more of a positive challenge | 06:48 |
uzumaki | that's very true indeed! | 06:48 |
janders | that's the great thing about the community we're in | 06:48 |
uzumaki | ikr? It's only been a while for me here, but I love coming here, just to help out, talk to people and stuff, I love this community | 06:49 |
*** diurnalist has joined #openstack-ironic | 07:01 | |
iurygregory | good morning uzumaki janders and Ironic! | 07:01 |
uzumaki | morning iurygregory ! how's it going? | 07:02 |
janders | good morning iurygregory | 07:02 |
iurygregory | uzumaki, doing good =) | 07:02 |
uzumaki | ahan? CI no longer giving you a hard time? | 07:02 |
iurygregory | well I try to not include CI when people ask how's it going =) | 07:03 |
uzumaki | haha! =D | 07:04 |
uzumaki | I was naive enough to think that Zuul had started behaving again x( | 07:04 |
uzumaki | well, all things considered, you must be doing pretty well in anticipation of the long weekend ;) | 07:05 |
iurygregory | well zuul has being a good boy and we gave a cookie | 07:05 |
uzumaki | iurygregory, i thought I was the cookie ambassador =O how can he get a cookie without it getting through me? | 07:05 |
openstackgerrit | Merged openstack/ironic master: Trivial: fix a minor issue in standalone docs and improve formatting https://review.opendev.org/743692 | 07:05 |
iurygregory | uzumaki, everybody can give cookies to zuul | 07:06 |
iurygregory | how can we get +1 from it?! | 07:06 |
uzumaki | Oh, I see! | 07:06 |
*** diurnalist has quit IRC | 07:06 | |
iurygregory | hehehehe | 07:06 |
uzumaki | xD | 07:07 |
uzumaki | any other special holidays coming up in cz? iurygregory | 07:07 |
iurygregory | uzumaki, only on Sep | 07:07 |
uzumaki | independence day? | 07:08 |
iurygregory | St. Wenceslas Day | 07:08 |
iurygregory | Independence is on Oct | 07:08 |
uzumaki | Interesting.. I'd been meaning to ask, if you sat on the porch with a basket of cookies/candies on the children's day handing it out to kids? :D | 07:09 |
uzumaki | uncle iurygregory | 07:09 |
iurygregory | LMAO | 07:09 |
rpittau | good morning ironic! o/ | 07:09 |
iurygregory | good morning rpittau o/ | 07:10 |
uzumaki | morning rpittau o/ | 07:10 |
rpittau | hey iurygregory :) | 07:10 |
rpittau | hey uzumaki :) | 07:10 |
uzumaki | Uncle iurygregory was telling us something, go on iurygregory xD | 07:10 |
iurygregory | I have the feeling today is Friday lol | 07:10 |
uzumaki | iurygregory, c'mon now, stop rubbing our noses into your long weekend xD | 07:10 |
iurygregory | lol | 07:11 |
iurygregory | I still need to work tomorrow =) | 07:11 |
uzumaki | wooot? :O | 07:11 |
rpittau | isn't it Friday?!?!? | 07:11 |
rpittau | :/ | 07:11 |
iurygregory | =( | 07:12 |
* iurygregory blames uzumaki | 07:12 | |
* uzumaki is wondering what he did.. | 07:12 | |
iurygregory | only because it's not friday yet =) | 07:12 |
* iurygregory grabs more coffee | 07:12 | |
* uzumaki thinks iurygregory needs a LOT more coffee xD | 07:13 | |
* rpittau looks around and doesn't know where to start | 07:13 | |
* uzumaki holds a coffee to rpittau here you go, start here | 07:14 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Fix idrac-wsman RAID apply_configuration https://review.opendev.org/743796 | 07:14 |
rpittau | uzumaki: that sounds like a great start :) | 07:14 |
uzumaki | it sure does! | 07:14 |
*** yaawang has quit IRC | 07:15 | |
*** yaawang has joined #openstack-ironic | 07:18 | |
*** yaawang has quit IRC | 07:26 | |
iurygregory | lol | 07:26 |
*** yaawang has joined #openstack-ironic | 07:26 | |
* janders noticed that coffee time in Europe is beer time in Australia | 07:27 | |
janders | hmm | 07:27 |
*** diurnalist has joined #openstack-ironic | 07:27 | |
janders | doesn't seem to work the other way round though | 07:27 |
janders | oh well | 07:28 |
janders | iurygregory rpittau I've done more testing on the oslo/multiopt issue | 07:28 |
janders | http://paste.openstack.org/show/796449/ | 07:28 |
iurygregory | Well in Brno Beer time is 24h I would say lol | 07:29 |
janders | item_type=types.Dict(value_type=types.Dict()) seems to turn the old problem into another problem but the nested dicts are still unparsable AFAIK | 07:29 |
iurygregory | damm =( | 07:30 |
janders | iurygregory haha then it does work the other way around, too, at least when it comes to (aus)coffee => (cz)beer | 07:30 |
iurygregory | hehehe | 07:30 |
janders | before I call it quits and start re-thinking the data structure for the config, can you have a look at my testing and cross-check if I come to the right conclusions? | 07:31 |
iurygregory | so basically oslo is crazy and it add the double quotes to make things worst for us right? | 07:31 |
janders | yeah looks like it | 07:31 |
* janders is digging out an example | 07:31 | |
janders | '{"shred_random_overwrite_iterations"' | 07:31 |
janders | that's a sample key | 07:32 |
*** rcernin has quit IRC | 07:32 | |
iurygregory | OGW | 07:32 |
janders | '{"shred_random_overwrite_iterations"': '"1"' | 07:32 |
janders | and key:value pair | 07:32 |
* iurygregory blames oslo | 07:32 | |
*** diurnalist has quit IRC | 07:32 | |
janders | I don't think we can work with this | 07:33 |
uzumaki | iurygregory, what's OGW? | 07:33 |
janders | +1 | 07:33 |
iurygregory | Oh God Why | 07:33 |
janders | :) | 07:33 |
uzumaki | iurygregory, LOL xD | 07:33 |
iurygregory | =) | 07:33 |
janders | haha! :) noted | 07:34 |
iurygregory | ʕノ•ᴥ•ʔノ ︵ ┻━┻ | 07:34 |
iurygregory | damm the copy didn't went perfect | 07:34 |
janders | a friend of mine used to say, while asked this question in regards to systemd issues | 07:34 |
iurygregory | ʕノ•ᴥ•ʔノ ︵ ┻━┻ | 07:34 |
janders | "but why?" | 07:34 |
uzumaki | well, it does show a bear throwing a table over | 07:35 |
janders | "because f*** you that's why" | 07:35 |
rpittau | looks like someone with giant ears | 07:35 |
uzumaki | janders, LOL xD | 07:35 |
iurygregory | janders, that totally makes sense | 07:35 |
janders | haha that's pretty good ASCII art iurygregory | 07:35 |
iurygregory | http://asciimoji.com/ ++ | 07:35 |
uzumaki | I'd say it's unicode | 07:35 |
uzumaki | what appear to be the arms, are a japanese letter, if i'm not mistaken, which would make this unicode | 07:36 |
*** dougsz has joined #openstack-ironic | 07:38 | |
rpittau | janders: just out of curiosity, have you tried passing the values as comma-separated instead of an actual dictionary ? | 07:38 |
janders | rpittau I was thinking exactly this | 07:38 |
janders | I think if there's only [A-Z][a-z][0-9] and [,] in the config, oslo won't freak | 07:39 |
rpittau | yeah | 07:39 |
rpittau | my thought exactly looking at the parser | 07:39 |
janders | I think braces and quotes make it freak out | 07:39 |
*** ociuhandu has joined #openstack-ironic | 07:39 | |
rpittau | should also make users life a bit easier :) | 07:39 |
janders | +1 | 07:40 |
janders | I will write something up and test it | 07:40 |
rpittau | orrr..... crazy thought, maybe you can pass a json file for each option ? | 07:40 |
janders | include steps.d/*.json style? | 07:41 |
rpittau | something like that, yeah | 07:41 |
iurygregory | json file would make things easier I would say | 07:41 |
rpittau | well, let's try with the simple one first :D | 07:41 |
janders | yeah that wouldn't go through oslo right? | 07:41 |
janders | file name goes through oslo | 07:41 |
janders | content does not | 07:41 |
iurygregory | since oslo wouldn't touch the file lol | 07:41 |
rpittau | yeah | 07:41 |
iurygregory | exactly | 07:42 |
rpittau | you pass teh file, we read the file and extract the values | 07:42 |
iurygregory | and we take over the world | 07:43 |
* rpittau just compared itself to a json parser | 07:43 | |
janders | that's right! :) | 07:43 |
janders | OSI summit lightning talk | 07:43 |
rpittau | :) | 07:43 |
janders | I fought oslo... and I won | 07:43 |
iurygregory | ++ | 07:44 |
SuiongNg|ITRI_TW | Hi, I have a question about rescue mode and agent token. | 07:44 |
SuiongNg|ITRI_TW | It appears that I cannot rescue the node more than on time. | 07:44 |
SuiongNg|ITRI_TW | If I rescue a node for the second times, the conductor would refuse to accept the heartbeat because | 07:44 |
SuiongNg|ITRI_TW | the agent token has already been generated in the first rescue attempt. | 07:44 |
SuiongNg|ITRI_TW | The workflow would look like this : rescue (success) > unrescue (success) > rescue (fail) | 07:44 |
SuiongNg|ITRI_TW | Is this normal ? | 07:44 |
rpittau | SuiongNg|ITRI_TW: interesting, do you have maybe the logs of the ipa ramdisk and ironic conductor? | 07:45 |
iurygregory | can you share some logs ? | 07:45 |
rpittau | SuiongNg|ITRI_TW: also which version are you on? | 07:45 |
SuiongNg|ITRI_TW | ironic-python-agent 6.2.0, the conductor is 15.0.0 (ubuntu | 07:48 |
SuiongNg|ITRI_TW | "message": "The agent token has already been retrieved. IPA may not operate as intended and the deployment may fail depending on settings in the ironic deployment." | 07:50 |
*** diurnalist has joined #openstack-ironic | 07:53 | |
SuiongNg|ITRI_TW | 2020-07-30 07:50:29.096 2007618 INFO eventlet.wsgi.server [req-3527099a-a551-42fa-91c4-18667a69a917 - - - - -] ::ffff:127.0.0.1 "POST / HTTP/1.1" status: 200 len: 337 time: 0.0229883 | 07:53 |
SuiongNg|ITRI_TW | 2020-07-30 07:53:03.908 2007618 WARNING ironic.conductor.manager [req-993a6201-a569-4a70-a820-f4afa26bddbc - - - - -] An agent token generation request is being refused as one is already present for node 6a838a47-6683-477b-b036-699e34f6eedd | 07:53 |
SuiongNg|ITRI_TW | 2020-07-30 07:53:03.911 2007618 INFO eventlet.wsgi.server [req-993a6201-a569-4a70-a820-f4afa26bddbc - - - - -] ::ffff:127.0.0.1 "POST / HTTP/1.1" status: 200 len: 2911 time: 0.0291045 | 07:53 |
SuiongNg|ITRI_TW | 2020-07-30 07:53:04.006 2007618 ERROR ironic.conductor.manager [req-3c7e2a47-af88-4f7e-936c-a1f541731dff - - - - -] Suspicious activity detected for node 6a838a47-6683-477b-b036-699e34f6eedd when attempting to heartbeat. Heartbeat request has been rejected as the version of ironic-python-agent indicated in the heartbeat operation should support agent token functionality. | 07:53 |
rpittau | SuiongNg|ITRI_TW: please don't post long logs here, use an external service | 07:54 |
*** diurnalist has quit IRC | 07:58 | |
SuiongNg|ITRI_TW | http://paste.openstack.org/show/796451/ | 07:58 |
janders | iurygregory rpittau victory http://paste.openstack.org/show/796452/ | 08:01 |
janders | it's very interesttting how it works | 08:01 |
rpittau | janders: nice :) | 08:02 |
janders | this: clean_step_add_params=erase_devices.shred_random_overwrite_iterations:1,erase_devices.shred_final_overwrite_with_zeros:True,erase_devices.agent_continue_if_ata_erase_failed:False | 08:02 |
janders | automagically turns into this with oslo: {'erase_devices.shred_random_overwrite_iterations': '1', 'erase_devices.shred_final_overwrite_with_zeros': 'True', 'erase_devices.agent_continue_if_ata_erase_failed': 'False'} | 08:02 |
rpittau | yeah, I was expecting that to work with : instead of . | 08:02 |
iurygregory | janders, \o/ | 08:03 |
janders | I suppose because MultiOpt item_type is dict, it handles the dicts itself and doesn't like being fed dicts | 08:03 |
janders | in a way what I was doing was feeding chicken with chicken | 08:03 |
janders | people do it but please dont | 08:04 |
rpittau | lol | 08:04 |
janders | I wonder if I specified item_type as string or list and fed it dicts whether that would work too | 08:04 |
janders | I guess there is one way to find out... | 08:04 |
iurygregory | yeah | 08:08 |
*** lucasagomes has joined #openstack-ironic | 08:08 | |
*** Lucas_Gray has joined #openstack-ironic | 08:11 | |
janders | it also doesn't inject any quotes | 08:12 |
janders | but the resulting structure is harder to work with | 08:12 |
janders | as it is a list of strings not dicts | 08:12 |
janders | dict-like strings I should say but they are missing outer braces, so would need pre-processing | 08:12 |
janders | the comma separated key:value pairs which are then used by oslo to build the dict seems to be the winning combination | 08:13 |
janders | I suppose it's worth letting openstack-oslo guys know we solved this right? | 08:13 |
janders | this way next time someone asks they have a solution ready | 08:14 |
rpittau | SuiongNg|ITRI_TW: I believe you will have to try and update your ironic version to 15.1.0 | 08:14 |
rpittau | SuiongNg|ITRI_TW: looks like a bug that was fixed in that version | 08:14 |
rpittau | janders: yep, that would be nice :) | 08:14 |
SuiongNg|ITRI_TW | ok, thank you very much. I will update my ironic installation. | 08:15 |
*** Lucas_Gray has quit IRC | 08:21 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Fix idrac-wsman RAID apply_configuration https://review.opendev.org/743796 | 08:22 |
*** Lucas_Gray has joined #openstack-ironic | 08:27 | |
*** rcernin has joined #openstack-ironic | 08:28 | |
*** rcernin has quit IRC | 08:34 | |
*** derekh has joined #openstack-ironic | 08:39 | |
*** diurnalist has joined #openstack-ironic | 08:44 | |
*** diurnalist has quit IRC | 08:48 | |
*** diurnalist has joined #openstack-ironic | 08:56 | |
*** yaawang has quit IRC | 08:58 | |
*** yaawang has joined #openstack-ironic | 08:59 | |
*** diurnalist has quit IRC | 09:01 | |
*** dtantsur|afk is now known as dtantsur | 09:03 | |
dtantsur | morning ironic | 09:03 |
dtantsur | janders: clean_step_add_params is a cool idea, but I'd suggest delaying it until you sort out the priorities | 09:06 |
dtantsur | janders: also, dicts in oslo.config don't look like python dicts | 09:06 |
dtantsur | their format is key1:value1,key2:value2 | 09:06 |
dtantsur | quotes are not supported by default, so to be able to do key1:"value1,still value1" you need to opt-in | 09:07 |
dtantsur | mmm, I see you've discovered it later in the scrollback | 09:08 |
dtantsur | arne_wiebalck: thinking about the apply_configuration deploy step.. what will happen if we call create_configuration on the already existing configuration? | 09:09 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: POC deploy-time RAID https://review.opendev.org/741227 | 09:11 |
*** diurnalist has joined #openstack-ironic | 09:12 | |
dtantsur | arne_wiebalck: the background of my question: should we default delete_existing to True for software RAID? | 09:14 |
*** diurnalist has quit IRC | 09:17 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add cirros-specific FAQ item for troubleshooting https://review.opendev.org/742703 | 09:17 |
*** sshnaidm|afk is now known as sshnaidm | 09:20 | |
janders | dtantsur thank you for your insights | 09:25 |
janders | so - I "split" this work and get priorities working first, correct? | 09:25 |
dtantsur | yep | 09:25 |
janders | cool! | 09:25 |
janders | and just confirming what we figured out earlier - the original change is abandoned so I submit the work for review as a new change, correct? | 09:26 |
dtantsur | janders: correct | 09:26 |
dtantsur | if you needed just a few fixes, we could restore the old change. but the current RFE is quite different | 09:27 |
janders | ACK | 09:27 |
dtantsur | iurygregory: what's the state https://review.opendev.org/#/c/741886/ and the tempest plugin patch? do they work as intended? | 09:27 |
patchbot | patch 741886 - ironic - Add tempest default_boot_option and altflavor - 5 patch sets | 09:27 |
dtantsur | rpioso, ajya, hi! we have a person using metal3 and hitting http://paste.openstack.org/raw/796454/ any ideas? | 09:30 |
iurygregory | dtantsur, hey | 09:31 |
iurygregory | looking at the logs they seem to be testing with both images | 09:32 |
iurygregory | so I think it's working | 09:32 |
dtantsur | okay, so ready for review? | 09:32 |
iurygregory | yup | 09:32 |
rpittau | iurygregory: I was looking at that, I think there's something missing on L297 in devstack/lib/ironic :) | 09:34 |
rpittau | L2997 sorry | 09:34 |
iurygregory | rpittau, ack will look in a few min | 09:35 |
iurygregory | meeting | 09:35 |
*** k_mouza has joined #openstack-ironic | 09:36 | |
ajya | dtantsur: haven't seen this. Is there anywhere response logged? Smth that starts with "Received response from". Otherwise it looks that there was no response and can't decode what's not received (hence 0-1 position) | 09:39 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Log the traceback of unexpected errors when verifying power credentials https://review.opendev.org/743941 | 09:39 |
dtantsur | a trivial improvement ^^^ | 09:39 |
dtantsur | ajya: I don't see a response. and we see the same problem with the redfish driver. | 09:39 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-lib master: Update version of hacking https://review.opendev.org/743942 | 09:44 |
*** ntt__ has joined #openstack-ironic | 09:47 | |
ajya | dtantsur: what state is the server? That GetRemoteServicesAPIStatus is checking if idrac is ready and it looks like it can't get event that. If server looks OK, could try to reset idrac | 09:48 |
dtantsur | ajya: I can only relay your request to the person | 09:48 |
dtantsur | but I suspect they've somehow put wrong credentials.. | 09:48 |
*** dougsz has quit IRC | 09:49 | |
ajya | dtantsur: I think then there is error saying that, but could be. There are cases when server is in e.g. BIOS setup, then iDRAC is down, though in those cases it times out. | 09:52 |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Allow HttpImageService to accept custom certificate https://review.opendev.org/742936 | 09:53 |
*** diurnalist has joined #openstack-ironic | 10:03 | |
*** mkrai has joined #openstack-ironic | 10:03 | |
*** dougsz has joined #openstack-ironic | 10:04 | |
openstackgerrit | Merged openstack/ironic master: AgentRAID: Account for empty results in post-configuration checks https://review.opendev.org/743690 | 10:06 |
*** diurnalist has quit IRC | 10:08 | |
*** k_mouza has quit IRC | 10:13 | |
janders | wrapping up for the night, thank you for all your help, see you tomorrow | 10:16 |
janders | have a good one everyone | 10:16 |
*** ociuhandu_ has joined #openstack-ironic | 10:17 | |
*** ociuhandu has quit IRC | 10:19 | |
*** ociuhandu_ has quit IRC | 10:21 | |
*** ociuhandu has joined #openstack-ironic | 10:22 | |
*** dtantsur is now known as dtantsur|brb | 10:23 | |
*** ociuhandu has quit IRC | 10:27 | |
*** k_mouza has joined #openstack-ironic | 10:35 | |
*** rcernin has joined #openstack-ironic | 10:37 | |
openstackgerrit | Merged openstack/ironic stable/train: Remove locks before RPC bus is started https://review.opendev.org/743809 | 10:37 |
*** uzumaki has quit IRC | 10:50 | |
openstackgerrit | Felix Maurer proposed openstack/ironic master: Use TLS for json_rpc when configured https://review.opendev.org/743912 | 10:53 |
*** rcernin has quit IRC | 10:54 | |
*** diurnalist has joined #openstack-ironic | 10:55 | |
*** diurnalist has quit IRC | 11:00 | |
*** ociuhandu has joined #openstack-ironic | 11:00 | |
openstackgerrit | Merged openstack/ironic master: Reset power state upon adoption failure https://review.opendev.org/743321 | 11:21 |
openstackgerrit | Merged openstack/ironic master: Add cirros-specific FAQ item for troubleshooting https://review.opendev.org/742703 | 11:21 |
*** sshnaidm_ has joined #openstack-ironic | 11:27 | |
*** sshnaidm has quit IRC | 11:27 | |
*** sshnaidm_ is now known as sshnaidm | 11:30 | |
*** mkrai has quit IRC | 11:32 | |
*** mkrai_ has joined #openstack-ironic | 11:32 | |
*** mkrai_ has quit IRC | 11:38 | |
*** uzumaki has joined #openstack-ironic | 11:40 | |
*** ociuhandu has quit IRC | 11:44 | |
*** ociuhandu has joined #openstack-ironic | 11:57 | |
*** rh-jelabarre has joined #openstack-ironic | 12:01 | |
*** ociuhandu_ has joined #openstack-ironic | 12:15 | |
*** ociuhandu has quit IRC | 12:18 | |
*** derekh has quit IRC | 12:23 | |
*** k_mouza has quit IRC | 12:24 | |
*** k_mouza has joined #openstack-ironic | 12:39 | |
*** k_mouza has quit IRC | 12:43 | |
*** Wryhder has joined #openstack-ironic | 12:44 | |
*** Lucas_Gray has quit IRC | 12:44 | |
*** Wryhder is now known as Lucas_Gray | 12:44 | |
*** tkajinam has quit IRC | 12:47 | |
openstackgerrit | Riccardo Pittau proposed openstack/sushy-tools master: Update version of hacking https://review.opendev.org/743962 | 12:49 |
openstackgerrit | Riccardo Pittau proposed openstack/sushy-cli master: Update version of hacking https://review.opendev.org/743963 | 12:50 |
*** yaawang has quit IRC | 12:51 | |
*** rloo has joined #openstack-ironic | 12:52 | |
*** yaawang has joined #openstack-ironic | 12:53 | |
openstackgerrit | Felix Maurer proposed openstack/ironic master: Use TLS for json_rpc when configured https://review.opendev.org/743912 | 12:53 |
*** dtantsur|brb is now known as dtantsur | 12:53 | |
dtantsur | deploy-time RAID working \o/ https://zuul.opendev.org/t/openstack/build/96f015c91b9b4d929a0f6ab996309486/log/controller/logs/screen-ir-cond.txt?severity=2#6882 | 12:54 |
dtantsur | I only need to understand if it's a good idea to default delete_existing to True | 12:55 |
*** diurnalist has joined #openstack-ironic | 12:57 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option and altflavor https://review.opendev.org/741886 | 12:59 |
*** diurnalist has quit IRC | 13:06 | |
*** sshnaidm is now known as sshnaidm|mtg | 13:08 | |
*** derekh has joined #openstack-ironic | 13:11 | |
*** k_mouza has joined #openstack-ironic | 13:16 | |
*** k_mouza has quit IRC | 13:20 | |
TheJulia | good morning | 13:20 |
rpittau | good morning TheJulia :) | 13:21 |
TheJulia | dtantsur: a pre-coffee response to 743606, please take a quick look because if I do what you want we'll have to likely double catch typeerror to try and just log the original exception. | 13:26 |
* TheJulia goes and gets coffee | 13:26 | |
*** k_mouza has joined #openstack-ironic | 13:26 | |
dtantsur | morning TheJulia | 13:27 |
TheJulia | actually maybe we wouldn't need to double catch | 13:28 |
TheJulia | the original exception is what generates the typeerror | 13:28 |
iurygregory | good morning TheJulia =) | 13:28 |
TheJulia | when it tries to go through the logging code, which is why I did the explicit wrap of str() | 13:29 |
TheJulia | because otherwise we would risk the exact same thing again most likely | 13:29 |
* TheJulia gets coffee | 13:29 | |
iurygregory | ping reminder TheJulia should have breakfast and do some slides | 13:29 |
*** k_mouza has quit IRC | 13:30 | |
TheJulia | gaaaah | 13:32 |
iurygregory | sorry for the reminder | 13:34 |
TheJulia | its okay | 13:34 |
iurygregory | maybe wasn't the best time =X | 13:35 |
dtantsur | TheJulia: %s is strictly equivalent to calling str() | 13:37 |
dtantsur | there is never a case for using them together | 13:37 |
dtantsur | (it's a nit though) | 13:37 |
TheJulia | dtantsur: I believe it is up to the actual driver writing the logs, and in the case possible to hit it explodes | 13:38 |
*** diurnalist has joined #openstack-ironic | 13:38 | |
dtantsur | nonetheless, it's about moving the str() call around within the same try..catch | 13:38 |
* dtantsur does not understand this patch in the end | 13:40 | |
TheJulia | Yeah, I'm not sure what is actually causing the original exception | 13:40 |
TheJulia | I'm just 95% sure I know where it is occuring at | 13:40 |
*** k_mouza has joined #openstack-ironic | 13:41 | |
TheJulia | The whole thing really makes no sense, but maybe I should just checkout some supermicro gear out of one of the labs and see how many times I can get it to crash on inspection | 13:41 |
TheJulia | I bet it is something in or about the underlying config thoug | 13:41 |
TheJulia | tough | 13:42 |
*** k_mouza has quit IRC | 13:45 | |
*** tzumainn has joined #openstack-ironic | 13:47 | |
*** sdanni has joined #openstack-ironic | 13:49 | |
*** k_mouza has joined #openstack-ironic | 13:50 | |
*** Goneri has joined #openstack-ironic | 13:52 | |
*** k_mouza has quit IRC | 13:54 | |
*** k_mouza has joined #openstack-ironic | 13:59 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: [WIP] Test software RAID in deploy-time on Victoria and newer https://review.opendev.org/741227 | 14:03 |
*** k_mouza has quit IRC | 14:04 | |
*** diurnalist has quit IRC | 14:05 | |
*** k_mouza has joined #openstack-ironic | 14:10 | |
*** k_mouza has quit IRC | 14:14 | |
*** k_mouza has joined #openstack-ironic | 14:15 | |
*** k_mouza has quit IRC | 14:19 | |
openstackgerrit | Felix Maurer proposed openstack/ironic master: Use TLS for json_rpc when configured https://review.opendev.org/743912 | 14:20 |
*** k_mouza has joined #openstack-ironic | 14:22 | |
*** k_mouza has quit IRC | 14:27 | |
*** diurnalist has joined #openstack-ironic | 14:27 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Install openstacksdk from pypi by default https://review.opendev.org/743989 | 14:29 |
sdanni | Hi TheJulia, I downloaded the ussuri ramdisk from https://tarballs.opendev.org/openstack/ironic-python-agent/dib/, directly uploaded the *.kernel and *.initramfs files as images, and after pxe boot, it failed to start all dhcp interfaces and Network Manager Online. Have you faced the similar issue? | 14:30 |
openstackgerrit | Merged openstack/ironic master: Fix idrac-wsman RAID apply_configuration https://review.opendev.org/743796 | 14:31 |
*** diurnalist has quit IRC | 14:31 | |
JayF | sdanni: I sorta renew what was said yesterday; it sounds like you're missing a driver in the ramdisk. Do you know if any OS of that version (CentOS 7 or 8) boots and works on that machine? I know RHEL/CentOS 8 removed support for some pretty common hardware (older dells). | 14:35 |
JayF | sdanni: getting that working at this point is basically just troubleshooting the OS on the ramdisk; there are ways to get access .. one is to build with the DIB dynamic-login element and providing login credentials, then using them on the console (https://docs.openstack.org/diskimage-builder/latest/elements/dynamic-login/README.html)p | 14:36 |
TheJulia | sdanni: exactly what JayF said. Basically we see these sorts of issues when the base OS has no idea what to do about networking because of a lack of drivers or understanding how to bring the interfaces up. By default they do attempt to bring up all interfaces found, fwiw. | 14:40 |
sdanni | JayF, TheJulia: CentOS 8 worked on that machine several days before. And now it doesn't work anymore. After updating the ramdisk, I don't see yesterday's issue anymore. | 14:41 |
sdanni | Yeah, a lack of driver makes sense for me. | 14:42 |
JayF | The nice thing is, you should be able to be pretty confident the images you downloaded worked because they're used in CI/integration tests -- I'm sorry there's not much more to give you, but it's gotta be something specific to how the image is booting on the server. Checking logs/etc on the server is the path to go. | 14:42 |
JayF | Another helpful element to add in for troubleshooting is https://docs.openstack.org/diskimage-builder/latest/elements/journal-to-console/README.html | 14:42 |
TheJulia | Oh, that is a good one | 14:43 |
TheJulia | I always forget because in CI the console is ttyS0 | 14:43 |
JayF | TheJulia: we have logged consoles here for troubleshooting. journal-to-console is just good, good stuff | 14:43 |
sdanni | Thanks JayF, a lot of helpful information! | 14:43 |
TheJulia | ++ | 14:43 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: requirements: remove paramiko and simplify setuptools version https://review.opendev.org/743991 | 14:44 |
dtantsur | JayF: TIL! does it make sense to use it by default? | 14:47 |
TheJulia | dtantsur: I sure think so! | 14:47 |
JayF | dtantsur: I don't have any idea who the users of our curated images are, so I don't have a strong opinion. | 14:47 |
JayF | There can be some downsides of putting journal to console, such as on machines with very slow consoles, printing messages to console can actually slow down what happens. | 14:48 |
TheJulia | numerous operators without special driver needs | 14:48 |
TheJulia | it ca | 14:48 |
TheJulia | can | 14:48 |
TheJulia | But we already do it in CI | 14:48 |
rpioso | Good morning, ironic :-) | 14:48 |
TheJulia | good morning rpioso | 14:48 |
JayF | Sounds like it's a pretty easy choice then :) | 14:48 |
rpioso | TheJulia: o/ | 14:48 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Fix idrac-wsman RAID apply_configuration https://review.opendev.org/743992 | 14:49 |
dtantsur | TheJulia: have I mentioned that I've got deploy-time software RAID working in the CI? | 14:50 |
rpioso | dtantsur: ty for the backport. | 14:50 |
dtantsur | np | 14:50 |
*** diurnalist has joined #openstack-ironic | 14:50 | |
rpioso | dtantsur: re: ^^^^^^^, that's an interesting error -- "Error: 'latin-1' codec can't encode characters in position 0-1: ordinal not in range(256)". More context would be helpful. Also, which version of the iDRAC FW is installed? | 14:54 |
dtantsur | rpioso: asking. if you're on kubernetes slack, it's in #cluster-api-baremetal | 14:55 |
TheJulia | dtantsur: I saw the comment | 14:57 |
* rpioso is effectively not on slack, but appreciates the pointer | 14:59 | |
dtantsur | good for you :) | 14:59 |
openstackgerrit | Merged openstack/ironic-lib master: Update version of hacking https://review.opendev.org/743942 | 15:01 |
openstackgerrit | Merged openstack/ironic master: Enforce autospec in test_iscsi_deploy https://review.opendev.org/743820 | 15:01 |
rpioso | :) | 15:01 |
*** k_mouza has joined #openstack-ironic | 15:04 | |
*** k_mouza has quit IRC | 15:08 | |
*** k_mouza has joined #openstack-ironic | 15:12 | |
*** k_mouza has quit IRC | 15:16 | |
*** k_mouza has joined #openstack-ironic | 15:21 | |
iurygregory | https://review.opendev.org/741886 https://review.opendev.org/741445 | 15:23 |
patchbot | patch 741886 - ironic - Add tempest default_boot_option and altflavor - 6 patch sets | 15:23 |
patchbot | patch 741445 - ironic-tempest-plugin - Update basic_ops - 13 patch sets | 15:23 |
*** gyee has joined #openstack-ironic | 15:25 | |
*** k_mouza has quit IRC | 15:25 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: set EVENTLET_NO_GREENDNS to 'yes' https://review.opendev.org/742011 | 15:26 |
TheJulia | dtantsur: https://review.opendev.org/#/c/742011/ <-- should at least make one of the inspector jobs happier. Just changed it to noqa the os import so hopefully it excludes everything else. Fun thing it passed pep8 on my desktop. :( | 15:27 |
patchbot | patch 742011 - ironic-python-agent - set EVENTLET_NO_GREENDNS to 'yes' - 3 patch sets | 15:27 |
*** k_mouza has joined #openstack-ironic | 15:28 | |
*** k_mouza has quit IRC | 15:32 | |
*** k_mouza has joined #openstack-ironic | 15:33 | |
*** k_mouza has quit IRC | 15:37 | |
*** sshnaidm|mtg is now known as sshnaidm | 15:39 | |
openstackgerrit | Verification of a change to openstack/bifrost failed: Add non-voting jobs with the openstack ansible collection from source https://review.opendev.org/743508 | 15:42 |
*** mkrai has joined #openstack-ironic | 15:42 | |
dtantsur | rpioso: iDRAC Firmware Version: 4.00.00.00 | 15:45 |
*** livelace has joined #openstack-ironic | 15:52 | |
*** lucasagomes has quit IRC | 16:09 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Test software RAID in deploy-time on Victoria and newer https://review.opendev.org/741227 | 16:12 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Enable deploy-time software RAID in standalone jobs https://review.opendev.org/744009 | 16:14 |
dtantsur | here it goes ^^^ | 16:14 |
*** ntt__ has quit IRC | 16:17 | |
*** trandles has joined #openstack-ironic | 16:19 | |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Clean up requirements https://review.opendev.org/743991 | 16:20 |
dtantsur | folks, would really appreciate reviews on https://review.opendev.org/#/c/741894/ to close this topic for now | 16:24 |
patchbot | patch 741894 - bifrost - Support configuring credential-less deploy - 5 patch sets | 16:24 |
*** dtantsur is now known as dtantsur|afk | 16:25 | |
dtantsur|afk | o/ | 16:25 |
*** dougsz has quit IRC | 16:30 | |
dking | I have a probably unusual request. I'm interested in trying to run containerized Ironic/Bifrost. Essentially, I want to be able to launch a Docker service which gives me just whatever I need to run Ironic, such as ironic-conductor, ironic-inspector, ironic-api, and maybe the other services such as rabbitmq, and maybe dnsmasq. I don't necessarily need keystone or DIB. Would anybody be able to recommend the best route to take for | 16:42 |
dking | that? | 16:42 |
*** derekh has quit IRC | 16:50 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Log the traceback of unexpected errors when verifying power credentials https://review.opendev.org/743941 | 16:57 |
*** ociuhandu_ has quit IRC | 17:02 | |
*** ociuhandu has joined #openstack-ironic | 17:09 | |
*** uzumaki has quit IRC | 17:12 | |
*** mkrai has quit IRC | 17:14 | |
*** ociuhandu has quit IRC | 17:14 | |
sdanni | Hi JayF, TheJulia, I tried dynamic-login when building ramdisk image. The doc says 'append rootpwd=”$ENCRYPTED_PASSWORD” to the kernel command line on boot', but I'm not sure where to append the password argument. | 17:21 |
JayF | from memory, I think it's pxe_append_params in ironic.conf? | 17:21 |
JayF | check ironic.conf.sample to confirm | 17:21 |
TheJulia | dking: so, as a single container or multiple containers, because I think all services and components... feels like we should already have a docker file for that | 17:21 |
JayF | sdanni: https://docs.openstack.org/ironic/ussuri/configuration/sample-config.html appears to agree | 17:22 |
TheJulia | yeah, it is pxe_append_params | 17:22 |
TheJulia | in the [pxe] section | 17:22 |
sdanni | Cool! Thanks! | 17:23 |
dking | TheJulia: I could go either way. The ones I've seen so far seem like they need configuration after they start up, but I can't see why they don't just have everything they need and just mount the config directories. | 17:23 |
TheJulia | iurygregory: you around? | 17:24 |
openstackgerrit | Merged openstack/bifrost master: Add non-voting jobs with the openstack ansible collection from source https://review.opendev.org/743508 | 17:26 |
iurygregory | TheJulia, finishing some exercises, in about 30 min is ok? | 17:29 |
TheJulia | iurygregory: sure, I'm just wondering if there is two metal3 changes you can look at | 17:29 |
TheJulia | And exercise actually sounds like a REALLY good idea | 17:29 |
iurygregory | the PRs on the images? | 17:30 |
*** sdanni has quit IRC | 17:30 | |
TheJulia | yeah | 17:31 |
TheJulia | ironic-image and ironic-inspector-image | 17:31 |
iurygregory | ack I will do, I think i looked at the ironic-image yesterday | 17:31 |
TheJulia | iurygregory: looks like zaneb is going to also propose the openshift namespace cherry-picks | 17:32 |
*** diurnalist has quit IRC | 17:33 | |
*** Lucas_Gray has quit IRC | 17:44 | |
*** diurnalist has joined #openstack-ironic | 17:44 | |
TheJulia | dking: looks like we killed the dockerfile at some point | 17:46 |
TheJulia | dking: you can just run install in a container, that should work just fine. | 17:46 |
*** sdanni has joined #openstack-ironic | 17:47 | |
*** diurnalist has quit IRC | 17:49 | |
dking | TheJulia: Well, I'd like to, as the DevOps folks say, treat them like cattle, not pets. So, I'd like to kill and spawn a new one easily if that's possible, rather than having to bring it up and then run external scripts. If that's possible, of course. | 17:52 |
TheJulia | dking: well, installer to generate the base container and then just start the services on container start | 17:53 |
clarkb | mnaser was working on simple container images for openstack services. I'm not sure if he got to ironic, but if he did those may be useable | 17:53 |
mnaser | clarkb: the infrastructure is all there, we've gotten through a few, ironic should happen when we can get to it.. | 17:54 |
mnaser | once we integrate it'll be part of our operator so it'll actually be an image that's tested using tempest | 17:54 |
clarkb | mnaser: you've moved away from adding the dockerfiles in project through right? where is the work happening now? | 17:57 |
mnaser | clarkb: well, not moved away as much as i haven't gotten traction so the goal once we have all (major) services inside containers and tested, we'd propose those into projects. https://opendev.org/vexxhost/openstack-operator/src/branch/master/images | 17:58 |
*** livelace has quit IRC | 18:11 | |
*** uzumaki has joined #openstack-ironic | 18:12 | |
TheJulia | fwiw, we afaik have no objection to putting dockerfile in ironic projets if it makes sense. I honestly think the resistance was really short signted | 18:20 |
TheJulia | sighted | 18:20 |
dking | I think that would be nice. Ironic is different from a lot of the other services in that it has a lot of use cases where a full stack isn't being run. | 18:23 |
*** diurnalist has joined #openstack-ironic | 18:35 | |
*** livelace has joined #openstack-ironic | 18:36 | |
*** diurnalist has quit IRC | 18:40 | |
openstackgerrit | Amy Marrich (spotz) proposed openstack/virtualbmc master: Correct vmbc add command https://review.opendev.org/744025 | 18:42 |
clarkb | https://review.opendev.org/#/c/713975/4/Dockerfile is the in repo POC for keystone | 18:42 |
patchbot | patch 713975 - keystone - DNM: container images poc - 4 patch sets | 18:42 |
clarkb | ironic would likely be slightly more complicated as you'll probably want different images for each entrypoint but otherwise pretty similar | 18:43 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Capture TypeError when the logger fails https://review.opendev.org/743606 | 18:51 |
*** diurnalist has joined #openstack-ironic | 19:26 | |
sdanni | JayF, TheJulia: after we login with dynamic-login element, we add the ip address (192.168.1.2) that the machine previously cannot connect with, and we can successfully curl this ip. After running ironic-python-agent.py, we again lose the connectivity with 192.168.1.2. The issue may related to ironic-python-agent I guess? | 19:41 |
mnaser | TheJulia: i can submit a general patch but i cant guarantee that it actually works... until we implement it inside our operator and consume that image | 19:48 |
*** uzumaki has quit IRC | 20:14 | |
*** ianychoi has joined #openstack-ironic | 20:41 | |
dking | clarkb: Thanks for the information. I tried that with keystone and ironic, and I get "Package mysql-client is not available, but is referred to by another package." | 20:43 |
*** ociuhandu has joined #openstack-ironic | 20:46 | |
clarkb | dking: I'm guessing that means debian has switched to mariadb | 20:47 |
dking | Probably. I think everything else has. | 20:49 |
dking | Yeah. It looks like if I change it to mariadb-*, it can get started. | 20:50 |
*** uzumaki has joined #openstack-ironic | 20:52 | |
dking | Well, for ironic, anyway, and the it fails for gcc not being installed. For keystone, it complains about python3-all-dev. | 20:55 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/stein: improve error messages during node lookup failures https://review.opendev.org/744052 | 20:57 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/rocky: improve error messages during node lookup failures https://review.opendev.org/744053 | 20:57 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/queens: improve error messages during node lookup failures https://review.opendev.org/744054 | 20:57 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option and altflavor https://review.opendev.org/741886 | 20:58 |
iurygregory | small update to fix the postgress for wholedisk+partition testing (I didn't noticed before since the job was non-voting) | 20:58 |
*** ociuhandu_ has joined #openstack-ironic | 21:04 | |
*** ociuhandu has quit IRC | 21:06 | |
*** uzumaki has quit IRC | 21:07 | |
*** ociuhandu_ has quit IRC | 21:10 | |
*** ociuhandu has joined #openstack-ironic | 21:11 | |
*** livelace has quit IRC | 21:14 | |
*** sshnaidm is now known as sshnaidm|afk | 21:42 | |
rpioso | dtantsur: We support 4.10.10.10 in the most recently released version of our downstream ironic idrac driver solution. Don't know if it would help, but they may want to consider upgrading to that version. Again, more Conductor log context would be helpful. | 22:03 |
rpioso | dtantsur|afk: ^^^ | 22:03 |
*** tkajinam has joined #openstack-ironic | 22:05 | |
rpioso | TheJulia: Is there a story for making DeployInterface.prepare should probably become a step? | 22:13 |
* rpioso takes a do over | 22:14 | |
rpioso | TheJulia: Is there a story for creating a deploy step from DeployInterface.prepare? | 22:16 |
TheJulia | rpioso: I'm not entirely sure it totally CAN | 22:20 |
TheJulia | it might be that it needs to in part, but discussion is needed | 22:20 |
TheJulia | so no, afaik there is no story | 22:20 |
rpioso | TheJulia: How do you suggest we move forward? | 22:24 |
rpioso | TheJulia: I believe you and dtantsur understand the issue we're facing with executing an OOB deploy step which needs to perform a reboot to apply configuration. That causes the one-time PXE boot to be consumed and deployment to fail. | 22:27 |
*** sdanni has quit IRC | 22:37 | |
*** rloo has quit IRC | 22:58 | |
JayF | rpioso: that's a situation that's handled (I hope/presume) for in-band IPA clean steps with reboot_requested=true... perhaps that could be a model to follow? | 22:59 |
rpioso | JayF: Do you have any insights why that model hasn't been applied to out-of-band? And please confirm for me the location of reboot_requested. | 23:01 |
*** rcernin has joined #openstack-ironic | 23:13 | |
*** rcernin has quit IRC | 23:14 | |
*** rcernin has joined #openstack-ironic | 23:14 | |
*** diurnalist has quit IRC | 23:19 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Fix TypeError on agent lookup failure https://review.opendev.org/744077 | 23:19 |
TheJulia | dtantsur|afk: ^^^^ You'll want a beverage before reviewing that | 23:20 |
TheJulia | or at least before looking at the story and then looking at it | 23:20 |
TheJulia | rpioso: The consensus was to make that part of the decorator so it could be declared and then acted upon appropriately | 23:21 |
TheJulia | yeah,, we discussed doing it that way because otherwise we expect to be able to at least ask the agent what IS possible | 23:21 |
TheJulia | which I believe we wrote up in the shanghai notes and the story | 23:22 |
TheJulia | dtantsur|afk: wf-2'ed my original change | 23:26 |
*** bdodd_ has quit IRC | 23:38 | |
*** bdodd has joined #openstack-ironic | 23:42 | |
*** diurnalist has joined #openstack-ironic | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!