| openstackgerrit | Steve Baker proposed openstack/ironic-python-agent stable/train: Prevent broken partition image UEFI deploys https://review.opendev.org/c/openstack/ironic-python-agent/+/782885 | 00:03 |
|---|---|---|
| openstackgerrit | Steve Baker proposed openstack/ironic-python-agent stable/train: Add fstab pointer to EFI partition https://review.opendev.org/c/openstack/ironic-python-agent/+/782886 | 00:03 |
| *** gyee has quit IRC | 00:05 | |
| *** jamesdenton has quit IRC | 00:08 | |
| *** jamesden_ has joined #openstack-ironic | 00:08 | |
| *** tosky has quit IRC | 00:10 | |
| *** gyee has joined #openstack-ironic | 00:19 | |
| *** derekh has quit IRC | 00:20 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Follow-up Automaticaly set cipher suite https://review.opendev.org/c/openstack/ironic/+/783759 | 00:21 |
| openstackgerrit | Merged openstack/bifrost master: Allow disabling integrated dhcp from bifrost cli https://review.opendev.org/c/openstack/bifrost/+/782700 | 00:32 |
| *** k_mouza has joined #openstack-ironic | 00:35 | |
| *** k_mouza has quit IRC | 00:39 | |
| openstackgerrit | Merged openstack/bifrost master: Remove explicit default from store_true https://review.opendev.org/c/openstack/bifrost/+/782775 | 00:44 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Fix Bandit check https://review.opendev.org/c/openstack/ironic/+/783749 | 01:15 |
| *** jamesden_ has quit IRC | 01:17 | |
| *** jamesdenton has joined #openstack-ironic | 01:17 | |
| *** jamesdenton has quit IRC | 01:26 | |
| *** jamesden_ has joined #openstack-ironic | 01:26 | |
| *** rcernin has quit IRC | 01:46 | |
| *** rcernin has joined #openstack-ironic | 01:51 | |
| *** ricolin_ has joined #openstack-ironic | 01:56 | |
| *** jamesden_ has quit IRC | 02:00 | |
| *** jamesdenton has joined #openstack-ironic | 02:01 | |
| *** mkrai has joined #openstack-ironic | 02:03 | |
| *** uzumaki has quit IRC | 02:11 | |
| *** zzzeek has quit IRC | 02:18 | |
| *** zzzeek has joined #openstack-ironic | 02:22 | |
| *** pmannidi has joined #openstack-ironic | 02:29 | |
| *** jamesdenton has quit IRC | 03:01 | |
| *** jamesdenton has joined #openstack-ironic | 03:01 | |
| *** uzumaki has joined #openstack-ironic | 03:10 | |
| *** mkrai has quit IRC | 03:18 | |
| *** mkrai has joined #openstack-ironic | 03:18 | |
| *** pmannidi has quit IRC | 03:43 | |
| *** ricolin_ has quit IRC | 03:47 | |
| *** uzumaki has quit IRC | 04:10 | |
| *** gyee has quit IRC | 04:12 | |
| *** rh-jlabarre has quit IRC | 04:15 | |
| *** jamesdenton has quit IRC | 04:21 | |
| *** jamesdenton has joined #openstack-ironic | 04:21 | |
| *** tzumainn has quit IRC | 04:38 | |
| *** ricolin has quit IRC | 04:47 | |
| *** sdanni has quit IRC | 05:14 | |
| *** abhishekyadav has joined #openstack-ironic | 05:21 | |
| *** pmannidi has joined #openstack-ironic | 05:26 | |
| *** paras333 has quit IRC | 05:42 | |
| *** paras333_ has joined #openstack-ironic | 05:52 | |
| *** uzumaki has joined #openstack-ironic | 05:52 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites https://review.opendev.org/c/openstack/ironic/+/744117 | 05:57 |
| *** paras333_ has quit IRC | 05:57 | |
| *** uzumaki has quit IRC | 05:58 | |
| openstackgerrit | Arun S A G proposed openstack/ironic master: Add agent_state and agent_status params to heartbeat https://review.opendev.org/c/openstack/ironic/+/778058 | 06:08 |
| openstackgerrit | Arun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface https://review.opendev.org/c/openstack/ironic/+/780398 | 06:08 |
| openstackgerrit | Dhuldev Valekar proposed openstack/ironic master: Add iDRAC management via Redfish to idrac HW type https://review.opendev.org/c/openstack/ironic/+/721593 | 06:21 |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites https://review.opendev.org/c/openstack/ironic/+/744117 | 06:24 |
| openstackgerrit | vinay50muddu proposed openstack/ironic master: Support clean/deploy step to add certificates to iLO https://review.opendev.org/c/openstack/ironic/+/783133 | 06:33 |
| arne_wiebalck | Good morning, ironic! | 06:39 |
| *** k_mouza has joined #openstack-ironic | 06:40 | |
| janders | good morning arne_wiebalck o/ | 06:41 |
| *** k_mouza has quit IRC | 06:45 | |
| arne_wiebalck | hey janders o/ | 06:45 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ussuri: Refactor part of image module https://review.opendev.org/c/openstack/ironic-python-agent/+/782348 | 06:48 |
| *** pmannidi has quit IRC | 06:57 | |
| *** mkrai has quit IRC | 07:04 | |
| *** paras333_ has joined #openstack-ironic | 07:07 | |
| *** frigo has joined #openstack-ironic | 07:10 | |
| frigo | hello Ironic! | 07:10 |
| *** k_mouza has joined #openstack-ironic | 07:17 | |
| *** k_mouza has quit IRC | 07:22 | |
| *** jamesdenton has quit IRC | 07:25 | |
| *** jamesdenton has joined #openstack-ironic | 07:25 | |
| *** paras333_ has quit IRC | 07:27 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Always add 'boot_method' vmedia in redfish/ilo vmedia boot https://review.opendev.org/c/openstack/ironic/+/783722 | 07:27 |
| *** rpittau|afk is now known as rpittau | 07:29 | |
| rpittau | good morning ironic! o/ | 07:29 |
| *** rcernin has quit IRC | 07:30 | |
| *** ociuhandu has joined #openstack-ironic | 07:52 | |
| *** ricolin has joined #openstack-ironic | 07:52 | |
| *** k_mouza has joined #openstack-ironic | 07:56 | |
| rpittau | something seems wrong in ironic-tempest-partition-uefi-redfish-vmedia | 08:01 |
| *** lucasagomes has joined #openstack-ironic | 08:01 | |
| *** ociuhandu has quit IRC | 08:03 | |
| *** tosky has joined #openstack-ironic | 08:12 | |
| *** rcernin has joined #openstack-ironic | 08:12 | |
| *** frigo has quit IRC | 08:16 | |
| *** frigo has joined #openstack-ironic | 08:17 | |
| *** ociuhandu has joined #openstack-ironic | 08:19 | |
| zer0c00l | rpittau: same here | 08:20 |
| zer0c00l | keeps failing https://review.opendev.org/c/openstack/ironic/+/778058 | 08:20 |
| zer0c00l | The controller has failed. | 08:21 |
| zer0c00l | i am going to hit the bed, need to be up for 10 am meeting. | 08:22 |
| zer0c00l | SSL certs? | 08:23 |
| zer0c00l | https://a455f3929f61c821afa0-d5237dd035b9f3c7532839ca52b23c88.ssl.cf2.rackcdn.com/783722/1/gate/ironic-tempest-partition-uefi-redfish-vmedia/2a99a2b/controller/logs/ironic-bm-logs/node-1_console_2021-03-30-07%3A12%3A31_log.txt | 08:23 |
| rpittau | yeah | 08:23 |
| zer0c00l | i guess perhaps ask in #infra? | 08:24 |
| janders | good morning rpittau o/ | 08:24 |
| rpittau | hey janders :) | 08:24 |
| rpittau | zer0c00l: maybe, it's just weird that is failing on one job only | 08:24 |
| *** zer0c00l is now known as zer0c00l|afk | 08:25 | |
| janders | rpittau would you have time to have a look at a weird problem with https://review.opendev.org/c/openstack/ironic/+/744117? | 08:25 |
| zer0c00l|afk | espeically when the release deadline in two days :S | 08:25 |
| zer0c00l|afk | *especially | 08:25 |
| janders | I think it's almost done (logic is fixed and it passes linter/unit tests) but it's sucking in some weird characters from somewhere and breaking in lab testing | 08:26 |
| rpittau | janders: I can take some more weird, yeah :) | 08:26 |
| janders | ok! getting the pastes ready! thanks rpittau :) | 08:26 |
| janders | hopefully it's something dumb/easy caused by staring at the same lines for way too long | 08:26 |
| janders | rpittau issue is happening somewhere around here I think: https://review.opendev.org/c/openstack/ironic/+/744117/23/ironic/conductor/steps.py#127 | 08:29 |
| rpittau | janders: ok, I'm reading the patch | 08:29 |
| janders | error mesage and some debug info: http://paste.openstack.org/show/804026/ | 08:29 |
| janders | and this explains where I put the extra logging (might be useful for interpreting ^): http://paste.openstack.org/show/804028/ | 08:29 |
| janders | TL; DR I think it's sucking in #033[00m from somewhere, making integers strings and it goes downhill from there | 08:30 |
| janders | I suspect it has to do with newlines and multiopt format of CONF.conductor.clean_step_priority_override but that's just a guess, I might be off | 08:30 |
| rpittau | janders: #033[00m is a color shell code.... mmmm | 08:32 |
| janders | whatever rubbish it's sucking it, at least it's a pretty colour :) | 08:32 |
| rpittau | lol yeah | 08:32 |
| janders | s/it/in | 08:32 |
| *** rcernin has quit IRC | 08:33 | |
| rpittau | to be more precise, it's the code to reset the colors in the shell | 08:34 |
| janders | I'm re-running the test with s/"if prio_overrides is not None:"/"if False" to see if it's just my code in this conditional picking up the rubbish or something else | 08:38 |
| rpittau | I wonder if that's because that is a console stream and includes ANSI escape sequences | 08:39 |
| janders | interestingly I think if there is only one step for it to iterate on, it doesn't break | 08:39 |
| janders | but I need to confirm that (might be caching) | 08:39 |
| janders | regarding the escape sequences - where does it happen exactly? | 08:40 |
| *** abhishekyadav has quit IRC | 08:42 | |
| rpittau | janders: that's whta I'm trying to understand, maybe L159, as a workaround you could strip the ansi codes from the steps but there's probably something weird going on there | 08:42 |
| rpittau | sorry L154 | 08:42 |
| janders | rpittau ACK will try that | 08:43 |
| rpittau | janders: check what happen in L154 as result, you're using types.Dict() for the new conf option and there may be something odd there | 08:44 |
| rpittau | mmm deja vu | 08:45 |
| rpittau | janders: yep, it's there, you need to separate the value with commas | 08:45 |
| rpittau | https://opendev.org/openstack/oslo.config/src/branch/master/oslo_config/types.py#L599 | 08:46 |
| janders | so - if there is just one override in ironic.conf, does it have to have comma at the end? | 08:47 |
| *** dtantsur|afk is now known as dtantsur | 08:48 | |
| rpittau | janders: good question, I would say yes as in python you can end a single entry dict with a comma | 08:48 |
| rpittau | looking at the oslo.config code it seems it does look for a comma no matter what | 08:48 |
| janders | freaking multiopt config entries | 08:48 |
| janders | extra high maintenance it seems | 08:49 |
| dtantsur | morning ironic | 08:49 |
| janders | hey dtantsur o/ | 08:49 |
| rpittau | I really do have a weird deja vu, like I had this talk before :D | 08:49 |
| rpittau | good morning dtantsur :) | 08:49 |
| janders | rpittau entirely possible (I grepped my IRC logs and found one reference to something a little similar, but wasn't exactly that) | 08:50 |
| janders | unless I hit it before I set up IRC logging, that is a possibility | 08:50 |
| dtantsur | a trailing comma sounds super ugly :( | 08:50 |
| dtantsur | I wonder if we should just parse it ourselves, especially since we're looking for one entry | 08:50 |
| rpittau | heh yeah | 08:50 |
| janders | Ironic seems to feel the same | 08:52 |
| rpittau | what about a list of values for the parameter instead of multiple entries with the same parameter name? something like clean_step_priority_override=step:prio,step:prio,step:prio ? or something like that | 08:52 |
| janders | won't start with [conductor]/clean_step_priority_override=deploy.erase_devices_metadata:123, | 08:52 |
| *** eandersson has quit IRC | 08:53 | |
| dtantsur | rpittau: it will get really ugly really soon | 08:53 |
| rpittau | true :/ | 08:53 |
| dtantsur | will become even worse if one day we throw the driver in the picture | 08:53 |
| dtantsur | I was thinking if today we have priority_override=step:prio | 08:54 |
| dtantsur | then one day we may add more params like priority_override=step:prio,management_interface:ilo | 08:54 |
| *** dtantsur is now known as dtantsur|brb | 08:55 | |
| *** mkrai has joined #openstack-ironic | 08:57 | |
| rpittau | janders: if we plan to expland that as dtantsur|brb said, I'm not sure using types.Dict() is the best solution | 08:58 |
| rpittau | janders: I would still suggest to try stripping the ANSI codes as workaround, that should work | 09:07 |
| janders | rpittau ACK, giving it a go | 09:07 |
| rpittau | janders: another thing I've noticed is that the overridden prio is considered a str, while it should be int | 09:08 |
| *** frigo has quit IRC | 09:09 | |
| janders | I think the ANSI codes do that | 09:09 |
| *** frigo has joined #openstack-ironic | 09:09 | |
| rpittau | janders: iy could also be types.Dict() https://opendev.org/openstack/oslo.config/src/branch/master/oslo_config/types.py#L620 | 09:10 |
| janders | hmm it doesn't like Nones | 09:11 |
| rpittau | janders: I think you can try with something even simpler and NOT escape ANSI codes, since maybe we don't want to change the value_type, just try to convert override_value to int in L132 | 09:22 |
| ajya | Hi, what's going on with ironic-tempest-partition-uefi-redfish-vmedia and ironic-inspector-tempest-partition-bios-redfish-vmedia? They both started to fail around the same time | 09:25 |
| janders | rpittau ACK, testing now, thanks! :) | 09:27 |
| *** rcernin has joined #openstack-ironic | 09:29 | |
| *** derekh has joined #openstack-ironic | 09:32 | |
| janders | rpittau just to make sure I understood your hint correctly: http://paste.openstack.org/show/804035/ does this look about right? | 09:37 |
| janders | (line 20) | 09:37 |
| janders | (I'm still getting "current override_value: 123#033[00m" :/) | 09:38 |
| *** rcernin has quit IRC | 09:43 | |
| *** k_mouza has quit IRC | 09:49 | |
| *** k_mouza has joined #openstack-ironic | 09:49 | |
| *** dtantsur|brb is now known as dtantsur | 09:59 | |
| dtantsur | ajya: mmm.. it is possible that the recent IPA changes have broken something | 09:59 |
| rpittau | ajya: there's an issue with certs | 10:00 |
| dtantsur | which means, TheJulia's patch did break it.. damn | 10:00 |
| *** dking has joined #openstack-ironic | 10:00 | |
| rpittau | janders: add the conversion after the 'if override_value:' | 10:01 |
| rpittau | janders: it could be that the ANSI codes are just an artifact of your console, if you convert that to int the code should work | 10:02 |
| janders | rpittau ACK, testing now | 10:08 |
| rpittau | janders: I also left a comment in the patch to leave a trace of the discussion there too | 10:10 |
| janders | rpittau ACK thank you good idea | 10:11 |
| * dtantsur fixes the IPA problem | 10:15 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Fix incorrect lsblk tag and add a virtual media job https://review.opendev.org/c/openstack/ironic-python-agent/+/783832 | 10:22 |
| dtantsur | TheJulia: ^^ | 10:23 |
| dtantsur | actually, it won't pass the CI this way, i need to split it | 10:24 |
| janders | rpittau: magic | 10:25 |
| janders | late conversion to integer worked | 10:25 |
| janders | will upload a revised patch shortly | 10:25 |
| janders | I hate oslo sometimes | 10:25 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Fix incorrect lsblk tag and add a virtual media job https://review.opendev.org/c/openstack/ironic-python-agent/+/783832 | 10:26 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Make the virtual media job voting https://review.opendev.org/c/openstack/ironic-python-agent/+/783834 | 10:27 |
| rpittau | janders: good :) | 10:27 |
| janders | rpittau hell yeah. Wrangling this cost me whole day today :/ thanks heaps for helping me out!!! :) | 10:27 |
| rpittau | janders: anytime! glad I could help :) | 10:28 |
| dtantsur | HEADS UP: the virtual media jobs are broken, please don't recheck until they're fixed | 10:28 |
| dtantsur | please also avoid approving any changes | 10:28 |
| janders | dtantsur thanks! | 10:29 |
| * rpittau goes dealing with some squids | 10:30 | |
| janders | rpittau the marine, not the web-proxy ones? | 10:30 |
| rpittau | janders: actual squids, yeah :) | 10:30 |
| janders | yum! | 10:30 |
| janders | what's for lunch? | 10:30 |
| rpittau | janders: fregola sarda with fresh tomato sauce and squids | 10:31 |
| rpittau | janders: https://en.wikipedia.org/wiki/Fregula | 10:32 |
| janders | nice! | 10:32 |
| janders | the cous cous like thing you mentioned the other day | 10:32 |
| *** rcernin has joined #openstack-ironic | 10:32 | |
| janders | genius! never had squid like that | 10:32 |
| ajya | thanks, dtantsur, rpittau | 10:32 |
| *** rcernin has quit IRC | 10:32 | |
| arne_wiebalck | bye everyone, see you tomorrow o/ | 10:32 |
| *** rcernin has joined #openstack-ironic | 10:33 | |
| janders | arne_wiebalck o/ | 10:33 |
| janders | rpittau do you use the squid ink for this dish? | 10:36 |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Generic way to configure clean step priorites https://review.opendev.org/c/openstack/ironic/+/744117 | 10:38 |
| janders | rpittau dtantsur *if* CI resurrects later today, it would be great to get your feedback on ^^^ - maybe it can make it for Wallaby after all | 10:39 |
| *** tosin has joined #openstack-ironic | 10:40 | |
| janders | if there's need for any further fixes I will try to smash these out first thing tomorow morning | 10:40 |
| dtantsur | will try to | 10:40 |
| janders | thanks dtantsur! :) | 10:40 |
| janders | see you tomorrow Ironic o/ | 10:43 |
| dtantsur | o/ | 10:44 |
| *** ricolin_ has joined #openstack-ironic | 10:52 | |
| *** hjensas has quit IRC | 10:53 | |
| *** ricolin_ has quit IRC | 10:54 | |
| iurygregory | good morning Ironic o/ | 10:55 |
| *** ricolin_ has joined #openstack-ironic | 11:16 | |
| iurygregory | yay for crazy CI, but it seems we have patches with the fix | 11:18 |
| iurygregory | thx dtantsur =) | 11:18 |
| *** hjensas has joined #openstack-ironic | 11:19 | |
| *** bfournie has quit IRC | 11:20 | |
| *** bfournie has joined #openstack-ironic | 11:20 | |
| *** openstackgerrit has quit IRC | 11:21 | |
| *** rcernin has quit IRC | 11:22 | |
| *** ricolin_ has quit IRC | 11:24 | |
| *** mkrai has quit IRC | 11:25 | |
| *** jamesdenton has quit IRC | 11:29 | |
| *** jamesdenton has joined #openstack-ironic | 11:30 | |
| *** jamesdenton has quit IRC | 11:42 | |
| *** jamesdenton has joined #openstack-ironic | 11:43 | |
| *** rh-jlabarre has joined #openstack-ironic | 11:55 | |
| *** rh-jlabarre has quit IRC | 11:55 | |
| *** rh-jlabarre has joined #openstack-ironic | 11:56 | |
| dtantsur | folks, please review https://review.opendev.org/c/openstack/ironic-python-agent/+/783832/ | 12:00 |
| dtantsur | it won't pass the new vmedia job, but it will unblock the ironic fix | 12:00 |
| dtantsur | the vmedia job has passed on https://review.opendev.org/c/openstack/ironic-python-agent/+/783834/ which depends on both fixes | 12:00 |
| *** anuradha1904 has joined #openstack-ironic | 12:10 | |
| *** seba has joined #openstack-ironic | 12:16 | |
| iurygregory | dtantsur, I was just waiting for CI result =) I will give the +2 | 12:16 |
| dtantsur | ack | 12:17 |
| *** derekh has quit IRC | 12:18 | |
| *** derekh has joined #openstack-ironic | 12:20 | |
| *** derekh has quit IRC | 12:28 | |
| dtantsur | TheJulia: I think we're ready for inspector and bifrost releases. I'll proceed with them. | 12:41 |
| *** k_mouza has quit IRC | 12:54 | |
| *** k_mouza_ has joined #openstack-ironic | 12:54 | |
| *** jamesdenton has quit IRC | 13:00 | |
| *** derekh has joined #openstack-ironic | 13:00 | |
| *** jamesdenton has joined #openstack-ironic | 13:00 | |
| *** rloo has joined #openstack-ironic | 13:17 | |
| *** tosin has quit IRC | 13:20 | |
| TheJulia | dtantsur: ack | 13:22 |
| TheJulia | Also, good morning | 13:22 |
| * TheJulia blinks | 13:26 | |
| *** spotz has joined #openstack-ironic | 13:30 | |
| TheJulia | dtantsur: I could have sworn I was watching the console stream on the vmedia job last night and watched it pass | 13:37 |
| TheJulia | Like, I'm befuddled | 13:37 |
| *** tzumainn has joined #openstack-ironic | 13:45 | |
| *** mkrai has joined #openstack-ironic | 13:52 | |
| dtantsur | yeah, I've also realized I was testing something else.. | 13:54 |
| dtantsur | anyway, the real problem is that we did not have virtual media testing on IPA | 13:54 |
| dtantsur | also, good morning | 13:54 |
| *** mkrai has quit IRC | 14:02 | |
| *** mkrai has joined #openstack-ironic | 14:02 | |
| *** jamesdenton has quit IRC | 14:09 | |
| *** jamesdenton has joined #openstack-ironic | 14:09 | |
| iurygregory | good morning TheJulia | 14:34 |
| *** tosky has quit IRC | 14:39 | |
| *** tosky has joined #openstack-ironic | 14:39 | |
| TheJulia | dtantsur: yeah | 14:42 |
| TheJulia | It is all resolvable :( | 14:42 |
| TheJulia | just awful timing :( | 14:42 |
| TheJulia | looks like the lsblk change is almost through the gate | 14:43 |
| *** jamesdenton has quit IRC | 14:47 | |
| *** jamesden_ has joined #openstack-ironic | 14:47 | |
| iurygregory | "awful timing" I'm used to this during release time XD | 14:51 |
| iurygregory | every cycle a different thing (so we don't get bored) | 14:52 |
| rpittau | even if the patch has mergeed, we need to wait for the new tinyipa to be published before recheck | 14:55 |
| dtantsur | * before rechecking the ironic fix | 14:55 |
| dtantsur | then when it merges, we're good again | 14:56 |
| rpittau | ^ that :) | 14:56 |
| TheJulia | is https://review.opendev.org/c/openstack/ironic-python-agent/+/783888 too crazy? | 15:01 |
| *** k_mouza_ has quit IRC | 15:02 | |
| *** k_mouza has joined #openstack-ironic | 15:02 | |
| rpittau | I wouldn't say crazy, it actually makes sense IMHO | 15:05 |
| dtantsur | mmm, interesting. yeah, it makes sense. | 15:06 |
| dtantsur | JayF: please hold on rechecking stuff | 15:07 |
| dtantsur | it's not enough for IPA patch to merge, we need the build job to finish (successfully) | 15:08 |
| JayF | oooh, post jobs hadn't completed yet? | 15:08 |
| JayF | sorry about that | 15:08 |
| dtantsur | tinyIPA: https://zuul.openstack.org/stream/100da658bae84130b8d8fdefb0d1fc98?logfile=console.log | 15:08 |
| TheJulia | still running :) | 15:08 |
| *** ociuhandu has quit IRC | 15:09 | |
| JayF | TheJulia: review jam @ 10a today? | 15:13 |
| TheJulia | yeah | 15:13 |
| JayF | aight I'll move the downstream calendar invite | 15:13 |
| JayF | btw, I'm happy to add anyone to it if you want calendar invites :) | 15:14 |
| TheJulia | errr | 15:14 |
| TheJulia | I've got 11 am on my work calendar | 15:14 |
| TheJulia | oh, right | 15:14 |
| TheJulia | moving it early | 15:14 |
| JayF | yeah that was a 24h delayed response to the ping lol | 15:15 |
| dtantsur | the image build must have succeeded by now | 15:20 |
| dtantsur | but I don't see new timestamps on https://tarballs.opendev.org/openstack/ironic-python-agent/tinyipa/files/ | 15:20 |
| rpittau | dtantsur: it's there | 15:20 |
| dtantsur | ah, I see it now. maybe caching somewhere. | 15:21 |
| dtantsur | the redfish jobs have just started on the ironic change, so JayF's recheck was actually appropriate :) | 15:22 |
| rpittau | perfect timing! | 15:22 |
| JayF | Just follow my patented "Don't Sign-On To IRC until you respond to all emails" method which permits obliviously doing the right thing | 15:23 |
| JayF | lol | 15:23 |
| TheJulia | dtantsur: zuul reporting *seems* be be trailing by about 15 minutes from reality | 15:24 |
| dtantsur | ENOTENOUGHCOOKIES :) | 15:24 |
| TheJulia | Yeah, I've noticed a few things that made me go "wut" in job behavior and reporting | 15:26 |
| TheJulia | but, in the end it seems to get sorted | 15:26 |
| dtantsur | qemu-img: /dev/sda: error while converting host_device: Failed to clear the new image's first sector: Device or resource busy | 15:26 |
| dtantsur | rings any bells? | 15:26 |
| dtantsur | derekh: can this ^^ be due to the 1G vs 2G limit? | 15:27 |
| TheJulia | no, infact we kind of need to be able to kill the limit because qemu_img tries to use only fully free memory | 15:27 |
| TheJulia | as in uses the same calls as it would for a virtual machine running | 15:27 |
| TheJulia | think of that makes me want to pour a glass of Bulleit rye | 15:27 |
| derekh | dtantsur: I didn't see that error come up because of the limit | 15:28 |
| TheJulia | dtantsur: only in conversion? or actual write-out? | 15:28 |
| derekh | dtantsur: where did you get that error from? /me wonders as I was just looking at the same error message on a bugzilla | 15:29 |
| dtantsur | TheJulia: that's the same for direct deploy | 15:31 |
| dtantsur | derekh: that's from bugzilla, so likely yes | 15:31 |
| derekh | dtantsur: ack | 15:31 |
| dtantsur | if you're already looking at it, I'll leave it in your hands | 15:31 |
| derekh | dtantsur: ack | 15:32 |
| rpittau | bye everyone! See you next week! o/ | 15:33 |
| *** rpittau is now known as rpittau|afk | 15:33 | |
| * TheJulia needs days off | 15:35 | |
| dtantsur | ++ | 15:36 |
| *** tosky has quit IRC | 15:36 | |
| *** frigo has joined #openstack-ironic | 15:39 | |
| TheJulia | The joy of double booked, injecting your own agenda into someone's meeting for them to discuss and bring back $answers | 15:39 |
| *** ociuhandu has joined #openstack-ironic | 15:40 | |
| *** ociuhandu has quit IRC | 15:45 | |
| frigo | dtantsur not sure that can help, I got similar issue once because of multipathing, building an IPA ramdisk with multipathing activated, even if you choose /dev/mapper/mpatha as root disk, IPA ends up selecting /dev/sda, but it fails since it's part of a multipathed device... my 2cents | 15:47 |
| dtantsur | frigo: wow, that's interesting. but I'm quite sure multipathing is not a part of it. | 15:48 |
| frigo | (I removed the multipath support and was happy ever after^H^H so far :) | 15:48 |
| dtantsur | I'm really curious why IPA did not recognize your root device hint | 15:49 |
| dtantsur | or even: if root device hints fail, the deploy should fail immediately | 15:49 |
| dtantsur | frigo: are you sure you set root device hints correctly? if yes, I'd be really curious to see ramdisk logs. | 15:50 |
| frigo | I think it boils down to that lsblk command you are fixing | 15:50 |
| *** gyee has joined #openstack-ironic | 15:50 | |
| frigo | it detects the root disk fine, but when it's time to actually copy the image, it does not use the hint | 15:50 |
| dtantsur | Oo | 15:50 |
| frigo | it uses come kind of.. resolved.. device | 15:50 |
| dtantsur | I wonder if the md code can somehow interfere.. | 15:51 |
| frigo | anyway. I will open some story next time I reproduce | 15:51 |
| dtantsur | yes please. It's really interesting. | 15:51 |
| *** dtruong has quit IRC | 15:54 | |
| frigo | (there is no real point in having multipathing activated in the ramdisk though. It was just an experiment because I have too much free time) | 15:54 |
| *** dtruong has joined #openstack-ironic | 15:55 | |
| TheJulia | so there kind of may be a point, but it depends on the SAN controllers and archetecture | 15:56 |
| TheJulia | architecture? | 15:56 |
| * TheJulia can't spell | 15:56 | |
| * TheJulia also woke up with a migraine | 15:56 | |
| rpioso | TheJulia: Will the review jam start in 1 hour? | 15:58 |
| TheJulia | Yeah, I think so if that is the time folks are collalescing around | 15:59 |
| TheJulia | a little more friendly, in the grand scheme of the universe with time zone shifts | 15:59 |
| * TheJulia wonders... if we'll ever get rid of daylight savings time | 15:59 | |
| dtantsur | EU has decided that already. We haven't agreed on which time each country will use though :) | 16:00 |
| TheJulia | CA decided to, but requires Federal approval :( | 16:01 |
| *** lucasagomes has quit IRC | 16:01 | |
| rpioso | TheJulia: May I update the Tuesday time on https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting to: Tuesdays: 5:00 PM UTC to 5:50 PM UTC? | 16:01 |
| TheJulia | rpioso: that would be much appreciated | 16:02 |
| * dtantsur is exhausted and wishes everyone a good evening | 16:02 | |
| *** dtantsur is now known as dtantsur|afk | 16:02 | |
| *** sdanni has joined #openstack-ironic | 16:03 | |
| TheJulia | goodnight dtantsur|afk | 16:05 |
| *** iurygregory_ has joined #openstack-ironic | 16:06 | |
| *** iurygregory has quit IRC | 16:06 | |
| ajya | Is CI back to normal? | 16:09 |
| *** ociuhandu has joined #openstack-ironic | 16:11 | |
| *** ociuhandu has quit IRC | 16:11 | |
| *** ociuhandu has joined #openstack-ironic | 16:13 | |
| *** ociuhandu has quit IRC | 16:14 | |
| *** ociuhandu has joined #openstack-ironic | 16:15 | |
| *** anuradha1904 has quit IRC | 16:15 | |
| *** iurygregory_ is now known as iurygregory | 16:15 | |
| *** ddurst- is now known as ddurst | 16:16 | |
| TheJulia | almost | 16:37 |
| *** k_mouza has quit IRC | 16:38 | |
| TheJulia | if you go to zuul.opendev.org and search for change 783722 you an see the status | 16:38 |
| *** frigo has quit IRC | 16:48 | |
| *** derekh has quit IRC | 17:00 | |
| *** ociuhandu_ has joined #openstack-ironic | 17:00 | |
| *** tosky has joined #openstack-ironic | 17:03 | |
| *** frigo has joined #openstack-ironic | 17:04 | |
| *** ociuhandu has quit IRC | 17:05 | |
| *** ociuhandu_ has quit IRC | 17:05 | |
| *** eandersson has joined #openstack-ironic | 17:23 | |
| *** ociuhandu has joined #openstack-ironic | 17:24 | |
| *** mkrai has quit IRC | 17:28 | |
| *** ociuhandu has quit IRC | 17:29 | |
| *** frigo has quit IRC | 17:35 | |
| *** frigo has joined #openstack-ironic | 17:38 | |
| iurygregory | many jobs for 783722 in the second attempt =( | 17:43 |
| *** jamesden_ has quit IRC | 17:59 | |
| *** jamesdenton has joined #openstack-ironic | 18:00 | |
| *** uzumaki has joined #openstack-ironic | 18:00 | |
| *** frigo has quit IRC | 18:02 | |
| *** frigo has joined #openstack-ironic | 18:03 | |
| *** spy is now known as antonym | 18:10 | |
| TheJulia | iurygregory: yeah, Looks like clark has observed some connection losses to CI nodes | 18:13 |
| TheJulia | so it is totally unrelated, but the sheer pressure we're putting CI infrastucture under at the moment as a larger community | 18:13 |
| iurygregory | yeah =( | 18:13 |
| TheJulia | rpioso: w/r/t idrac ipmi explicit support, I've left a note on the change reflecting our conversation. One thing to be considered conversely is enabling easy migration to a hardware type at some point, I think that is definitely a good topic, but I think the horizon needs to be clear in terms of any future breaking changes which may require automatic hardware type migration, should they occur... if they are | 18:15 |
| TheJulia | invasive such that the base driver may no longer work. | 18:15 |
| TheJulia | oh jeeze | 18:19 |
| TheJulia | 3rd now | 18:19 |
| TheJulia | w/r/t the vmedia fix | 18:19 |
| TheJulia | rpioso: also added item to the list of topics, line 227 | 18:38 |
| *** k_mouza has joined #openstack-ironic | 18:39 | |
| TheJulia | https://etherpad.opendev.org/p/ironic-xena-ptg | 18:39 |
| *** k_mouza has quit IRC | 18:43 | |
| rpioso | TheJulia: Thank you so much :) | 18:48 |
| *** frigo has quit IRC | 18:59 | |
| *** ociuhandu has joined #openstack-ironic | 19:03 | |
| TheJulia | requeued 783722... our own releases being approved tanked CI \o/ | 19:08 |
| * TheJulia suspects we can't issue any reward cookies as a result of this | 19:12 | |
| *** frigo has joined #openstack-ironic | 19:14 | |
| *** ociuhandu has quit IRC | 19:17 | |
| iurygregory | \o/ | 19:17 |
| * TheJulia tries to recall... what was she working on before meetings started this morning | 19:18 | |
| *** ociuhandu has joined #openstack-ironic | 19:18 | |
| iurygregory | TheJulia, like we did the backports for https://review.opendev.org/q/I04a3391f85412dcabc6105bd91beb1da25bdfc19 it's fine to backport https://review.opendev.org/c/openstack/ironic/+/770526 to Victoria? (and maybe ussuri) | 19:19 |
| TheJulia | backport to victoria, s/features/fixes the release note, and then cherry pick on down... preferably to Train | 19:20 |
| TheJulia | if you don't take it to train, that is not a big deal, but just keep a list on the etherpad of what is where | 19:20 |
| iurygregory | I will wait our CI to be normal (because there is a follow-up https://review.opendev.org/c/openstack/ironic/+/783759) so after it merges I can cherry-pick locally + squash before sending to the other branches hehe | 19:22 |
| iurygregory | thanks for the info \o/ I will try to backport till train =) | 19:22 |
| * iurygregory brb time for more coffee | 19:22 | |
| TheJulia | iurygregory: makes sense, just note it in the initial commit on victoria | 19:23 |
| TheJulia | and maybe lets let that one get merged before the other cherry-picks | 19:23 |
| iurygregory | sure | 19:32 |
| *** ociuhandu has quit IRC | 19:41 | |
| *** ociuhandu has joined #openstack-ironic | 20:00 | |
| janders | good morning Ironic o/ | 20:06 |
| iurygregory | morning janders o/ | 20:07 |
| janders | CI still isn't happy right? | 20:07 |
| iurygregory | janders, yeah till 783722 merges | 20:09 |
| janders | thanks iurygregory will keep an eye out on this one | 20:10 |
| janders | heading out for early morning bike ride, back around my usual time :) | 20:10 |
| iurygregory | enjoy! | 20:11 |
| iurygregory | I love that https://zuul.openstack.org/status#783722 doesn't show the change '-' | 20:12 |
| *** frigo has quit IRC | 20:14 | |
| iurygregory | now it shows lol | 20:25 |
| TheJulia | yeah scheduler restart | 20:33 |
| iurygregory | oh | 20:36 |
| *** ociuhandu has quit IRC | 20:42 | |
| *** uzumaki has quit IRC | 20:42 | |
| *** ociuhandu has joined #openstack-ironic | 20:46 | |
| *** k_mouza has joined #openstack-ironic | 20:59 | |
| *** k_mouza has quit IRC | 21:03 | |
| *** ayoung has quit IRC | 21:05 | |
| *** ociuhandu has quit IRC | 21:07 | |
| *** rcernin has joined #openstack-ironic | 21:10 | |
| *** zzzeek has quit IRC | 21:13 | |
| *** uzumaki has joined #openstack-ironic | 21:13 | |
| *** zzzeek has joined #openstack-ironic | 21:15 | |
| *** sdanni has quit IRC | 21:25 | |
| *** k_mouza has joined #openstack-ironic | 21:27 | |
| *** jamesdenton has quit IRC | 21:28 | |
| *** jamesdenton has joined #openstack-ironic | 21:28 | |
| *** k_mouza has quit IRC | 21:32 | |
| *** ociuhandu has joined #openstack-ironic | 21:38 | |
| *** rcernin has quit IRC | 21:38 | |
| *** ociuhandu has quit IRC | 21:58 | |
| *** rcernin has joined #openstack-ironic | 22:18 | |
| *** rcernin has quit IRC | 22:19 | |
| *** rcernin has joined #openstack-ironic | 22:19 | |
| *** zbitter has joined #openstack-ironic | 22:20 | |
| *** zaneb has quit IRC | 22:21 | |
| *** gudrutis has quit IRC | 22:42 | |
| *** openstackstatus has quit IRC | 22:42 | |
| *** gudrutis9 has joined #openstack-ironic | 22:42 | |
| *** early` has joined #openstack-ironic | 22:42 | |
| *** janders1 has joined #openstack-ironic | 22:42 | |
| *** gudrutis9 is now known as gudrutis | 22:42 | |
| *** openstack has joined #openstack-ironic | 22:44 | |
| *** ChanServ sets mode: +o openstack | 22:44 | |
| *** rloo has quit IRC | 22:49 | |
| *** antonym has joined #openstack-ironic | 23:09 | |
| *** openstackgerrit has joined #openstack-ironic | 23:12 | |
| openstackgerrit | Arun S A G proposed openstack/ironic master: Add agent_status and agent_status_message params to heartbeat https://review.opendev.org/c/openstack/ironic/+/778058 | 23:12 |
| openstackgerrit | Arun S A G proposed openstack/ironic master: Add support for configdrive in anaconda interface https://review.opendev.org/c/openstack/ironic/+/780398 | 23:12 |
| *** tosky has quit IRC | 23:14 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!