| *** threestrands has joined #openstack-ironic | 00:09 | |
| openstackgerrit | inspurericzhang proposed openstack/networking-baremetal master: Remove the unused coding style modules https://review.opendev.org/730698 | 00:33 |
|---|---|---|
| *** rloo has quit IRC | 00:35 | |
| *** jdandrea has quit IRC | 00:46 | |
| *** rcernin has quit IRC | 00:58 | |
| *** rcernin has joined #openstack-ironic | 01:03 | |
| *** sshnaidm|afk is now known as sshnaidm|off | 01:34 | |
| *** k_mouza has joined #openstack-ironic | 02:21 | |
| *** jdandrea has joined #openstack-ironic | 02:24 | |
| *** k_mouza has quit IRC | 02:25 | |
| *** jdandrea has quit IRC | 02:28 | |
| *** mkrai has joined #openstack-ironic | 02:29 | |
| *** mkrai has quit IRC | 02:56 | |
| *** rcernin has quit IRC | 03:01 | |
| *** ricolin has joined #openstack-ironic | 03:13 | |
| *** rcernin has joined #openstack-ironic | 03:14 | |
| *** rcernin has quit IRC | 03:19 | |
| *** rcernin has joined #openstack-ironic | 03:19 | |
| openstackgerrit | inspurericzhang proposed openstack/ironic-tempest-plugin master: Add py38 package metadata https://review.opendev.org/731405 | 03:35 |
| *** threestrands has quit IRC | 03:49 | |
| *** rcernin has quit IRC | 03:57 | |
| *** tzumainn has quit IRC | 04:12 | |
| *** rcernin has joined #openstack-ironic | 04:16 | |
| *** ociuhandu has joined #openstack-ironic | 04:42 | |
| *** ociuhandu has quit IRC | 04:47 | |
| *** rcernin has quit IRC | 04:48 | |
| *** rcernin has joined #openstack-ironic | 04:49 | |
| *** mrda has quit IRC | 04:58 | |
| *** mrda has joined #openstack-ironic | 05:00 | |
| arne_wiebalck | Good morning, ironic! | 05:40 |
| *** ociuhandu has joined #openstack-ironic | 06:25 | |
| *** yolanda has joined #openstack-ironic | 06:43 | |
| *** rcernin has quit IRC | 06:46 | |
| *** ociuhandu has quit IRC | 06:52 | |
| *** ociuhandu has joined #openstack-ironic | 06:52 | |
| iurygregory | good morning arne_wiebalck and Ironic o/ | 07:00 |
| *** jawad_axd has joined #openstack-ironic | 07:03 | |
| arne_wiebalck | hey iurygregory o/ | 07:08 |
| arne_wiebalck | ducks as he has not reviewed iurygregory 's patch as promised | 07:08 |
| iurygregory | arne_wiebalck, hehehe rlx =) | 07:09 |
| arne_wiebalck | quite a day yesterday, but finally all seems back \o/ | 07:09 |
| * iurygregory needs more coffee to wake up | 07:09 | |
| iurygregory | you were able to fix the database problem ? | 07:10 |
| arne_wiebalck | seems so, yes | 07:10 |
| iurygregory | awesome! | 07:10 |
| iurygregory | congrats | 07:10 |
| arne_wiebalck | but before there were more issues ... | 07:11 |
| arne_wiebalck | https://twitter.com/belmiromoreira/status/1265734916953825283 | 07:11 |
| arne_wiebalck | Ironic DB issues were just the tip of the iceberg :) | 07:12 |
| iurygregory | hehe I saw that | 07:12 |
| iurygregory | =) | 07:12 |
| iurygregory | no problems with neutron? =P | 07:13 |
| arne_wiebalck | database issues | 07:14 |
| iurygregory | ouch | 07:15 |
| *** ociuhandu has quit IRC | 07:20 | |
| *** lucasagomes has joined #openstack-ironic | 07:55 | |
| *** ociuhandu has joined #openstack-ironic | 07:56 | |
| *** ociuhandu has quit IRC | 08:02 | |
| *** rcernin has joined #openstack-ironic | 08:09 | |
| *** alexmcleod has joined #openstack-ironic | 08:15 | |
| *** rcernin has quit IRC | 08:15 | |
| *** ociuhandu has joined #openstack-ironic | 08:27 | |
| *** xiaolin has joined #openstack-ironic | 08:34 | |
| *** tkajinam has quit IRC | 08:37 | |
| *** ravsingh has joined #openstack-ironic | 08:39 | |
| *** dtantsur|afk is now known as dtantsur | 08:40 | |
| dtantsur | morning ironic | 08:40 |
| *** derekh has joined #openstack-ironic | 08:43 | |
| *** rpittau|afk is now known as rpittau | 08:53 | |
| rpittau | good morning ironic! o/ | 08:53 |
| rpittau | almost 3 hours internet blackout this morning.... | 08:53 |
| *** jawad_ax_ has joined #openstack-ironic | 08:58 | |
| *** jawad_axd has quit IRC | 09:01 | |
| dtantsur | I woke up to my router losing IPv4. had to reboot it.. | 09:04 |
| dtantsur | #NOINTERNETTHURSDAY | 09:04 |
| openstackgerrit | Merged openstack/networking-baremetal master: Remove the unused coding style modules https://review.opendev.org/730698 | 09:08 |
| openstackgerrit | Merged openstack/ironic master: Cap jsonschema 3.2.0 as the minimal version https://review.opendev.org/730949 | 09:09 |
| openstackgerrit | Merged openstack/ironic stable/train: Collect ramdisk logs also during cleaning https://review.opendev.org/730771 | 09:09 |
| iurygregory | good morning dtantsur and rpittau o/ | 09:09 |
| rpittau | hey iurygregory :) | 09:09 |
| *** rcernin has joined #openstack-ironic | 09:10 | |
| *** HagunKim has joined #openstack-ironic | 09:15 | |
| iurygregory | RDO Ussuri Released \o/ | 09:18 |
| iurygregory | https://blogs.rdoproject.org/2020/05/rdo-ussuri-released/ | 09:19 |
| *** rcernin has quit IRC | 09:24 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] Test CI https://review.opendev.org/731478 | 09:25 |
| *** k_mouza has joined #openstack-ironic | 09:29 | |
| rpittau | dtantsur: it seems we might have F32 nodesets soon, or almost soon | 09:30 |
| *** ravsingh has quit IRC | 09:39 | |
| *** rcernin has joined #openstack-ironic | 09:48 | |
| *** ravsingh has joined #openstack-ironic | 09:52 | |
| *** rcernin has quit IRC | 09:54 | |
| dtantsur | neat | 09:58 |
| iurygregory | dtantsur, changed to +2 https://review.opendev.org/#/c/730759/ | 10:14 |
| patchbot | patch 730759 - ironic - Add the noop management interface to the redfish h... - 1 patch set | 10:14 |
| iurygregory | =) | 10:14 |
| dtantsur | thx! | 10:14 |
| *** rpittau is now known as rpittau|bbl | 10:18 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support Debian Buster and add CI jobs for it https://review.opendev.org/731240 | 10:21 |
| *** k_mouza has quit IRC | 10:28 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Install diskimage-builder outside of venv https://review.opendev.org/731186 | 10:32 |
| openstackgerrit | Merged openstack/bifrost master: Remove explicit support for EOL Fedora versions and Debian Jessie https://review.opendev.org/731191 | 10:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Correct handling enable_venv in pip_install.yml https://review.opendev.org/731493 | 10:40 |
| *** k_mouza has joined #openstack-ironic | 10:41 | |
| *** k_mouza has quit IRC | 10:41 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: Add IPv6 ci Job https://review.opendev.org/724087 | 10:41 |
| *** k_mouza has joined #openstack-ironic | 10:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Install diskimage-builder outside of venv https://review.opendev.org/731186 | 10:50 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support Debian Buster and add CI jobs for it https://review.opendev.org/731240 | 10:53 |
| *** lucasagomes has quit IRC | 10:54 | |
| *** sshnaidm|off has quit IRC | 10:55 | |
| *** lucasagomes has joined #openstack-ironic | 10:57 | |
| *** ociuhandu has quit IRC | 11:04 | |
| *** ociuhandu has joined #openstack-ironic | 11:05 | |
| *** Qianbiao has joined #openstack-ironic | 11:10 | |
| *** ociuhandu has quit IRC | 11:14 | |
| *** ociuhandu has joined #openstack-ironic | 11:14 | |
| *** Qianbiao has quit IRC | 11:15 | |
| dtantsur | No space left on device in bifrost jobs \o/ | 11:21 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: CI: move the libvirt storage pool to /opt https://review.opendev.org/731499 | 11:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Fix detection of mkisofs https://review.opendev.org/731502 | 11:32 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support Debian Buster and add CI jobs for it https://review.opendev.org/731240 | 11:33 |
| *** Qianbiao has joined #openstack-ironic | 11:33 | |
| Qianbiao | hi dtantsur, for the rpm spec, should i keep python2 support? | 11:35 |
| dtantsur | Qianbiao: only if want to support releases older than Ussuri | 11:36 |
| Qianbiao | Hmm, i am not sure about this, because the lib itself supports python2. | 11:36 |
| Qianbiao | but this rpm package, it is only required by metal3 ironic image. | 11:37 |
| iurygregory | dtantsur, ouch is that the case where we need to use the libvirt storage pool? | 11:38 |
| dtantsur | iurygregory: s/use/move/ yes | 11:38 |
| iurygregory | gotcha | 11:38 |
| iurygregory | =) | 11:38 |
| dtantsur | Qianbiao: then I guess you should be fine with Ussuri and on (and only Python 3) | 11:38 |
| Qianbiao | yes. | 11:39 |
| *** ociuhandu has quit IRC | 11:46 | |
| *** ociuhandu has joined #openstack-ironic | 11:48 | |
| *** jawad_ax_ has quit IRC | 11:52 | |
| *** ociuhandu has quit IRC | 11:59 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: CI: move the libvirt storage pool to /opt https://review.opendev.org/731499 | 12:01 |
| *** rh-jelabarre has joined #openstack-ironic | 12:03 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Fix detection of mkisofs https://review.opendev.org/731502 | 12:04 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support Debian Buster and add CI jobs for it https://review.opendev.org/731240 | 12:04 |
| bfournie | dtantsur: nice catch on ipmitool using 4 retries as default, when I run the command with -N 1 it only takes 2 seconds with the Cipher Suites issue versus 10 seconds if I don't provide -N | 12:04 |
| dtantsur | 2 seconds sounds good enough :) | 12:05 |
| *** rh-jelabarre has quit IRC | 12:05 | |
| *** rpittau|bbl is now known as rpittau | 12:05 | |
| bfournie | dtantsur: yep | 12:06 |
| openstackgerrit | Merged openstack/ironic master: Remove deprecated [DEFAULT]fatal_exception_format_errors https://review.opendev.org/728102 | 12:09 |
| openstackgerrit | Merged openstack/ironic-inspector master: Update lower-constraints.txt https://review.opendev.org/727237 | 12:09 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Set default tempest_test_timeout for ironic-base https://review.opendev.org/730761 | 12:09 |
| *** rh-jelabarre has joined #openstack-ironic | 12:10 | |
| dtantsur | has anyone also noticed that patches that fix or improve CI fail most of the times? | 12:10 |
| rpittau | dtantsur: that's true, probably because tweaking and improving require lot of tentatives and experiment to succeed | 12:11 |
| dtantsur | mmm, no I mean transient failures and rechecks | 12:11 |
| dtantsur | the one increasing the JSON RPC timeout is undergoing its 9th recheck now | 12:12 |
| rpittau | is zuul afraid of change? | 12:12 |
| dtantsur | I guess so | 12:12 |
| *** dtantsur is now known as dtantsur|brb | 12:14 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic master: New configuration parameter to use ipmitool retries https://review.opendev.org/728261 | 12:21 |
| openstackgerrit | Merged openstack/ironic master: Add the noop management interface to the redfish hardware type https://review.opendev.org/730759 | 12:28 |
| *** ravsingh has quit IRC | 12:38 | |
| iurygregory | zuul is mad at us | 12:42 |
| iurygregory | =( | 12:42 |
| rpittau | zuul is mad in general :D | 12:42 |
| iurygregory | also it depends on the changes we are trying | 12:43 |
| iurygregory | because we are fixing X and CI says " I will fail because of Y" | 12:43 |
| iurygregory | =D | 12:43 |
| *** Goneri has joined #openstack-ironic | 12:46 | |
| openstackgerrit | Merged openstack/ironic master: Add agent_token param to docstrings https://review.opendev.org/730747 | 12:47 |
| *** ociuhandu has joined #openstack-ironic | 12:48 | |
| *** rloo has joined #openstack-ironic | 12:49 | |
| openstackgerrit | Merged openstack/ironic master: Change IRONIC_VM_SPECS_RAM default https://review.opendev.org/730623 | 12:52 |
| *** ociuhandu has quit IRC | 12:54 | |
| *** priteau has joined #openstack-ironic | 12:59 | |
| TheJulia | good morning | 12:59 |
| rpittau | good morning TheJulia :) | 13:00 |
| TheJulia | anyone else spotted agent VMs that take 500+ seconds to get to DHCP? | 13:00 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Fix check packages command https://review.opendev.org/730829 | 13:00 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Fix pip3 path https://review.opendev.org/730573 | 13:00 |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: [WIP] Add Fedora 30 based job https://review.opendev.org/731522 | 13:00 |
| TheJulia | since my patch has 2x+2+wf, I'm guessing yes | 13:01 |
| rpittau | TheJulia: yeah :/ | 13:01 |
| iurygregory | good morning TheJulia =) | 13:01 |
| *** jdandrea has joined #openstack-ironic | 13:04 | |
| TheJulia | I think I figured out the test I felt I was missing on the boot iso patch I'd been working on. | 13:04 |
| TheJulia | which reminds me, I wonder if supermicro sales has responded yet | 13:05 |
| TheJulia | nope :( | 13:07 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: iPXE ISO Ramdisk booting https://review.opendev.org/729464 | 13:09 |
| TheJulia | Now I just need to actually get on the console of a machine and test the prior patch | 13:09 |
| *** dtantsur|brb is now known as dtantsur | 13:09 | |
| dtantsur | morning TheJulia | 13:09 |
| dtantsur | TheJulia: yes, I've seen this as well | 13:09 |
| *** dking has joined #openstack-ironic | 13:15 | |
| TheJulia | dtantsur: good morning! | 13:19 |
| *** lucasagomes has quit IRC | 13:19 | |
| *** lucasagomes has joined #openstack-ironic | 13:20 | |
| TheJulia | dtantsur: would it make sense to see if we can twist someone working closesly with the MOC stuff to review https://review.opendev.org/703778 ? | 13:28 |
| patchbot | patch 703778 - ironic - Add RPC objects for deployment API - 11 patch sets | 13:28 |
| arne_wiebalck | https://twitter.com/ArneWiebalck/status/1265998242346602499 | 13:29 |
| TheJulia | iurygregory: Left some thoughts on https://review.opendev.org/#/c/714453, lmk because I can change my vote right now, just... I'm becoming very skittish about more CI jobs | 13:31 |
| patchbot | patch 714453 - ironic - Wholedisk+Uefi job - 14 patch sets | 13:31 |
| iurygregory | TheJulia, looking | 13:33 |
| iurygregory | not add a tempest suite? O.o | 13:33 |
| * iurygregory is confused | 13:33 | |
| iurygregory | I got the feeling you want to have a tempest scenario that would validate that for us | 13:34 |
| iurygregory | and we could run that scenario in a job that runs with uefi | 13:34 |
| iurygregory | s/uefi/uefi + wholedisk | 13:35 |
| iurygregory | but we don't have a job that does that =( all jobs running with uefi are using partition images | 13:36 |
| iurygregory | our initial idea was to have a job that would be doing uefi + wholedisk | 13:37 |
| TheJulia | iurygregory: no, I'm thinking build a suite of uefi+grub specific jobs | 13:38 |
| TheJulia | I understand that, the key is being able to run the basic job with slightly different settings and a different image type | 13:38 |
| iurygregory | that makes sense | 13:38 |
| TheJulia | I'm just super skittish about new jobs right now as we've seen we're mostly red in CI right now :( | 13:39 |
| iurygregory | I will start working on a new scenario for uefi + grub jobs | 13:40 |
| TheJulia | arne_wiebalck: wow impressive improvement | 13:40 |
| *** tzumainn has joined #openstack-ironic | 13:40 | |
| arne_wiebalck | TheJulia: it is so much better to work with :-D | 13:41 |
| TheJulia | tzumainn: I was just thinking about you, but I swear I didn't say your name :) | 13:41 |
| tzumainn | TheJulia, hahaha, well, here I am! | 13:44 |
| dtantsur | tzumainn: (back from a meeting) yeah, it wouldn't hurt. I'm also not sure when I'll be able to take it again. | 13:45 |
| tzumainn | dtantsur, I feel like I may be missing part of this conversation... ? | 13:46 |
| *** ociuhandu has joined #openstack-ironic | 13:46 | |
| dtantsur | tzumainn: ugh, sorry, it was for TheJulia | 13:46 |
| tzumainn | haha, no problem! | 13:46 |
| dtantsur | saw you nick, and my hands typed it automatically :) | 13:46 |
| TheJulia | tzumainn: so I was kind of wondering if we could get you to at least take a little time and review the deployment API stuff dmitry got started on which I believe was to help support the MOC use cases | 13:47 |
| tzumainn | TheJulia, sure! | 13:47 |
| TheJulia | if dtantsur is willing, I mean, you could also take the work over! we only have a fininte number of dtantsur's | 13:48 |
| TheJulia | basically being 1 | 13:48 |
| TheJulia | :) | 13:48 |
| dtantsur | I'd be very open to someone taking it over | 13:49 |
| dtantsur | and yes len(dtantsurs) == 1 | 13:49 |
| tzumainn | hahahaha - let me see if I can free up some time from what the moc folks are asking - it may be possible though! | 13:49 |
| tzumainn | I think it's definitely something that the moc would want in the near term, so I could see it becoming a priority | 13:50 |
| TheJulia | Up to them, I'm sure they are plotting other things at the same time :) | 13:51 |
| TheJulia | Speaking of which, I've not heard much from them. I assume the current state of affairs has kind of derailed regular communication | 13:51 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Increase callback timeout https://review.opendev.org/731363 | 13:52 |
| tzumainn | yeah, I think a lot of effort going on right now just to make sure that students are settled and able to work | 13:52 |
| TheJulia | Yeah | 13:52 |
| dtantsur | w00t, the debian job passed https://review.opendev.org/#/c/731240/ | 13:58 |
| patchbot | patch 731240 - bifrost - Support Debian Buster and add CI jobs for it - 9 patch sets | 13:58 |
| TheJulia | yay | 13:58 |
| rpittau | nice | 13:58 |
| * dtantsur hates ansible variable handling | 14:00 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: CI: move the libvirt storage pool to /opt https://review.opendev.org/731499 | 14:03 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Add the noop management interface to the redfish hardware type https://review.opendev.org/731539 | 14:07 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/train: Add the noop management interface to the redfish hardware type https://review.opendev.org/731540 | 14:07 |
| dtantsur | TheJulia: something I'm pondering is whether we should backport everything that lands to bifrost to stable/ussuri and have another minor release from it | 14:09 |
| dtantsur | it seems that a lot of changes are various post-python-3 and other important repairs/readditions | 14:10 |
| * dtantsur -> tea | 14:10 | |
| TheJulia | tea sounds wonderful | 14:12 |
| * TheJulia goes and takes the puppy out who is demanding to go out :( | 14:12 | |
| rpittau | dtantsur: backporting sounds reasonable, as you said all the changes are mainly python3-related | 14:13 |
| TheJulia | dtantsur: that would be reasonable in my mind | 14:17 |
| *** rpittau is now known as rpittau|brb | 14:18 | |
| *** khansa has joined #openstack-ironic | 14:20 | |
| *** khansa has quit IRC | 14:21 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: CI: move the libvirt storage pool to /opt https://review.opendev.org/731499 | 14:26 |
| dtantsur | okay, bifrost doesn't officially follow the stable policy, so we have a bit more freedom there | 14:28 |
| *** rpittau|brb is now known as rpittau | 14:29 | |
| *** ravsingh has joined #openstack-ironic | 14:38 | |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Add Fedora 30 based job https://review.opendev.org/731522 | 14:42 |
| *** ravsingh has quit IRC | 14:47 | |
| *** HagunKim has quit IRC | 15:04 | |
| TheJulia | it is tooling, not a service | 15:27 |
| dtantsur | yeah, but the policy can be applied to anything | 15:28 |
| openstackgerrit | Merged openstack/ironic master: Raise the JSON RPC connection timeout to 2 minutes and disable retries https://review.opendev.org/728445 | 15:50 |
| openstackgerrit | Merged openstack/ironic master: Set default tempest_test_timeout for ironic-base https://review.opendev.org/730761 | 15:50 |
| dtantsur | yay! | 15:52 |
| TheJulia | sweet! | 15:53 |
| iurygregory | thank you zuul <3 | 15:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: agent: do not hard power off if sort power off actually succeeds https://review.opendev.org/731575 | 16:01 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: agent: do not hard power off if soft power off actually succeeds https://review.opendev.org/731575 | 16:02 |
| *** ociuhandu has quit IRC | 16:06 | |
| *** ociuhandu has joined #openstack-ironic | 16:07 | |
| rpittau | good night! o/ | 16:08 |
| *** rpittau is now known as rpittau|afk | 16:08 | |
| *** lucasagomes has quit IRC | 16:09 | |
| * TheJulia wants to call it a night but it is just too early | 16:12 | |
| *** priteau has quit IRC | 16:14 | |
| *** ociuhandu has quit IRC | 16:14 | |
| TheJulia | dtantsur: on that rare poweroff issue, can we get some more details for reviewing? I'm trying to understand the complete context | 16:15 |
| openstackgerrit | Merged openstack/bifrost master: Fix detection of mkisofs https://review.opendev.org/731502 | 16:21 |
| *** Qianbiao has quit IRC | 16:22 | |
| dtantsur | TheJulia: you mean, expand https://review.opendev.org/#/c/731575/2/releasenotes/notes/agent-power-off-2115fcfaac030bd0.yaml further? | 16:29 |
| patchbot | patch 731575 - ironic - agent: do not hard power off if soft power off act... - 2 patch sets | 16:29 |
| dtantsur | a guess right now is that it may happen that the node powers off so fast that IPA never sends a response | 16:29 |
| dtantsur | at least theoretically it's not impossible | 16:29 |
| *** jdandrea has quit IRC | 16:34 | |
| TheJulia | oh, yeah... that could be a thing, but the get power state through power wouldn't raise an exception which seems weird. I guess I'm wondering where this is actually being hit for context | 16:39 |
| *** dr_gogeta86 has joined #openstack-ironic | 16:41 | |
| dtantsur | TheJulia: it's a suspect for a downstream issue | 16:42 |
| dr_gogeta86 | hi | 16:42 |
| TheJulia | and that seems plausable issue wise | 16:42 |
| TheJulia | we're in a ramdisk, there is not much more to do other than sync() | 16:42 |
| dr_gogeta86 | dtantsur, how to debug bootloader installation | 16:42 |
| dr_gogeta86 | ? | 16:42 |
| dtantsur | could you provide any details? your request is too broad. | 16:43 |
| *** ociuhandu has joined #openstack-ironic | 16:46 | |
| *** ociuhandu has quit IRC | 16:52 | |
| * dtantsur needs to go, see y'all | 16:52 | |
| iurygregory | bye dtantsur | 16:52 |
| *** ociuhandu has joined #openstack-ironic | 16:54 | |
| *** derekh has quit IRC | 16:57 | |
| *** dtantsur is now known as dtantsur|afk | 17:00 | |
| TheJulia | dr_gogeta86: Upon deployment failure, by default, the agent uploads logs to the conductor | 17:01 |
| TheJulia | typically in like /var/log/ironic/deploy_logs or something like that | 17:01 |
| dtantsur|afk | s/_logs// | 17:01 |
| * dtantsur|afk goes for real now | 17:01 | |
| dr_gogeta86 | TheJulia, you say the tar file ? | 17:01 |
| TheJulia | Inside there, you should be able to track down which file the logs inside the tar file, which will hopefully give you an idea of what exactly is occuring | 17:01 |
| TheJulia | specifically your looking for logging for ironic-python-agent | 17:02 |
| TheJulia | dr_gogeta86: yes | 17:02 |
| *** jdandrea has joined #openstack-ironic | 17:07 | |
| dr_gogeta86 | TheJulia, May 28 12:00:35 host-10-30-214-46 ironic-python-agent[1172]: 2020-05-28 12:00:35.808 1172 INFO root [-] Command image.install_bootloader completed: Command name: install_bootloader, params: {u'efi_system_part_uuid': None, u'root_uuid': u'228c59ea-82f0-4ee8-9d03-5620e5f0fafb'}, status: SUCCEEDED, result: None. | 17:13 |
| dr_gogeta86 | is not an uefi machine | 17:13 |
| dr_gogeta86 | is a virtual machine | 17:13 |
| dr_gogeta86 | on uefi machines works | 17:13 |
| TheJulia | Virutal machines can be UEFI or Bios mode | 17:13 |
| dr_gogeta86 | on ovirt/RHEV? | 17:14 |
| TheJulia | looks like it is in bios mode | 17:14 |
| TheJulia | yup | 17:14 |
| dr_gogeta86 | is bios mode btw | 17:14 |
| TheJulia | Indeed, so what seems to be the issue since it seems to think that a bootloader is setup and good to go | 17:14 |
| dr_gogeta86 | maybe i find the culprit | 17:14 |
| dr_gogeta86 | g cmd (subprocess): chroot /tmp/tmpn2JSCi /bin/sh -c "grub2-install /dev/sdb" execute /usr/lib/python2.7/site-packages/oslo_concurrency/processutils.py:372 | 17:14 |
| dr_gogeta86 | why ? | 17:15 |
| dr_gogeta86 | wrong disk | 17:15 |
| dr_gogeta86 | ages ago there is a special capability to force a disk but i din't remember | 17:15 |
| TheJulia | sounds like the root device is being chosen as /dev/sdb | 17:16 |
| TheJulia | most likely, it is a smaller disk than /dev/sda | 17:16 |
| TheJulia | and rhev/ovirt sounds like it is only trying to load /dev/sda? | 17:16 |
| TheJulia | There is, root device hints | 17:17 |
| dr_gogeta86 | root_device='{"size": ">= 60", "rotational": false}' | 17:17 |
| TheJulia | and what is sda ? | 17:17 |
| dr_gogeta86 | now i try specify | 17:17 |
| dr_gogeta86 | there are 3 disks | 17:17 |
| TheJulia | rotational is often wrong inside VMs, for what it is worth | 17:17 |
| dr_gogeta86 | sda ( root 150 ) and two 50 disks | 17:18 |
| TheJulia | since they all get emulated or translated | 17:18 |
| dr_gogeta86 | i can safely remove rotational | 17:18 |
| dr_gogeta86 | is copy past from documentation | 17:18 |
| TheJulia | yeah | 17:18 |
| dr_gogeta86 | i can use the size | 17:18 |
| TheJulia | I think so, give it a try :) | 17:18 |
| *** ricolin has quit IRC | 17:22 | |
| openstackgerrit | Merged openstack/ironic master: Increase callback timeout https://review.opendev.org/731363 | 17:29 |
| *** ociuhandu has quit IRC | 17:31 | |
| *** priteau has joined #openstack-ironic | 17:31 | |
| gudrutis2 | Hello o/, anyone here :) | 17:32 |
| gudrutis2 | ? | 17:32 |
| *** ociuhandu has joined #openstack-ironic | 17:32 | |
| *** priteau has quit IRC | 17:42 | |
| *** ociuhandu has quit IRC | 17:45 | |
| *** ociuhandu has joined #openstack-ironic | 17:46 | |
| *** ociuhandu has quit IRC | 17:51 | |
| gudrutis2 | I have this issue that some Ironic-inspector APIs are missing, so when executing certain commands I get 404 error | 18:00 |
| openstackgerrit | Verification of a change to openstack/ironic failed: New configuration parameter to use ipmitool retries https://review.opendev.org/728261 | 18:06 |
| TheJulia | gudrutis2: your going to have to provide more detail. Is the service installed, is it configured, are things pointing to it, how are you using it? | 18:17 |
| *** yolanda has quit IRC | 18:22 | |
| gudrutis2 | TheJulia: looks like we figured it out, ironic-conductor and ironic-inspector use different ports and we misconfigured it. | 18:26 |
| *** k_mouza has quit IRC | 18:38 | |
| TheJulia | gudrutis2: that would do it! | 18:54 |
| *** alexmcleod has quit IRC | 18:58 | |
| arne_wiebalck | mea culpa, I misconfigured the endpoints | 19:02 |
| arne_wiebalck | bye everyone o/ | 19:02 |
| TheJulia | It would be awesome if I could get a core reviewer to take a look at https://review.opendev.org/#/c/731575/2 | 19:06 |
| patchbot | patch 731575 - ironic - agent: do not hard power off if soft power off act... - 2 patch sets | 19:06 |
| TheJulia | It makes sense now that I actually talked to people who encountered it | 19:07 |
| TheJulia | added comments | 19:07 |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/ussuri: Increase callback timeout https://review.opendev.org/731599 | 19:10 |
| TheJulia | rpioso: you around? | 19:41 |
| rpioso | TheJulia: Here I am :-) | 19:41 |
| TheJulia | so, I've been looking at https://review.opendev.org/#/c/727906 | 19:42 |
| patchbot | patch 727906 - ironic (stable/train) - redfish: handle hardware that is unable to set per... - 5 patch sets | 19:42 |
| TheJulia | and it looks like the None value can reach the method which is not possible in Sushy 2.0.3 to signal "don't try this" | 19:43 |
| TheJulia | in other words, we need to reference the mapping for true/false. I think if not enabled then we could just send once | 19:43 |
| TheJulia | so we might need to add "if enabled is None: set it to boot once, otherwise try setting continnious | 19:43 |
| TheJulia | " | 19:43 |
| TheJulia | then proceed with fallback | 19:43 |
| rpioso | TheJulia: Pulling it up to refresh my memory. | 19:44 |
| * TheJulia assumes there are sequences of seek() and read() | 19:46 | |
| rpioso | TheJulia: I understand the issue. | 20:06 |
| TheJulia | \o/ | 20:13 |
| TheJulia | okay, I gave up trying to actually leave a comment in gerrit. I think my web browser is unhappy with me | 20:13 |
| TheJulia | as long as you grok it \o/ | 20:13 |
| rpioso | TheJulia: I'm concerned about https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/redfish/management.py#L201-L202 | 20:13 |
| rpioso | It could be continuous is being requested when it's already set to that. | 20:14 |
| *** priteau has joined #openstack-ironic | 20:21 | |
| TheJulia | the conundrum is if none | 20:22 |
| TheJulia | it could already be set to it but we need to override it | 20:24 |
| TheJulia | or at least try | 20:24 |
| *** ociuhandu has joined #openstack-ironic | 20:25 | |
| *** priteau has quit IRC | 20:26 | |
| openstackgerrit | Merged openstack/ironic stable/train: Add the noop management interface to the redfish hardware type https://review.opendev.org/731540 | 20:29 |
| rpioso | TheJulia: We only want to override it if the BMC Redfish service doesn't support Continuous. | 20:30 |
| rpioso | Older firmware does, some current doesn't, and who knows what the future holds. | 20:30 |
| rpioso | If memory serves, we found that business on lines 201-202 did not help. | 20:31 |
| TheJulia | well, if it doesn't we're going to fall back anyway | 20:32 |
| *** ociuhandu has quit IRC | 20:32 | |
| TheJulia | I guess it could be as simple "if enabled: assert continuous, else: assert once | 20:33 |
| TheJulia | " | 20:33 |
| rpioso | Or eliminate lines 201-202 and make the default of _set_boot_device() be once, instead of None. | 20:35 |
| rpioso | Note that enabled is a sushy value -- once, continuous, or disabled. They're all strings. | 20:35 |
| TheJulia | I understand it is a sushy value. If we don't raise an excpetion from the attempt of continious, we never fallback | 20:37 |
| *** k_mouza has joined #openstack-ironic | 20:38 | |
| rpioso | +1 | 20:39 |
| rpioso | reset_boot_device() expects once to be used. | 20:39 |
| TheJulia | I guess why I was suggesting if it is at all not false (and thus None) is so that we otherwise try to run it continiously then fallback if that fails. If false, then it is definitely one time | 20:41 |
| *** k_mouza has quit IRC | 20:43 | |
| * TheJulia looks at clock, hears gardener running all sorts of loud equipment and thinks "maybe this afternoon was not meant to be productive" | 20:43 | |
| rpioso | TheJulia: It's a bit dizzying, because there are two fallbacks in play. The inner fallback deals with the two different APIs, while the outer deals with continuous to once. Unfortunately, the desired setting is unknown when called from reset_boot_device() and when lines 201-202 change it to None after finding the desired setting equals the present setting. | 20:55 |
| rpioso | Could this bug exist on more recent branches running with older sushy? | 20:56 |
| TheJulia | I don't think so, we change sushy in ussuri such that it would take None as no action | 20:58 |
| rpioso | As long as the operator upgrades sushy with ironic, it's good. I'm thinking stable/ussuri ironic with old sushy. Perhaps that's not a thing. | 21:00 |
| TheJulia | Well, we want stable/train to still kind of work whilst allowing stable/ussuri's sushy :) | 21:01 |
| rpioso | Yeah, that's the other combo, old ironic and new sushy. | 21:02 |
| rpioso | Got that one. | 21:02 |
| rpioso | And that combo is almost certainly good eating sushy ;-) | 21:03 |
| TheJulia | I really need sushi | 21:03 |
| * rpioso feels the same, but is avoiding raw food during the pandemic. Don't want to risk needing medical care for that. | 21:04 | |
| TheJulia | yeah | 21:05 |
| TheJulia | Speaking of medical care, I'm going to go lay down. I'm not feeling so great and the gardener is making too much noise outside | 21:05 |
| rpioso | +1 | 21:05 |
| openstackgerrit | Merged openstack/ironic master: CI: use the default devstack cirros version https://review.opendev.org/731116 | 21:06 |
| *** jdandrea has quit IRC | 21:44 | |
| *** threestrands has joined #openstack-ironic | 21:45 | |
| openstackgerrit | Merged openstack/ironic master: New configuration parameter to use ipmitool retries https://review.opendev.org/728261 | 22:08 |
| *** tkajinam has joined #openstack-ironic | 22:54 | |
| *** rcernin has joined #openstack-ironic | 23:07 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Add IPv6 ci Job https://review.opendev.org/724087 | 23:10 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Extend vmedia job timeout https://review.opendev.org/731627 | 23:40 |
| *** rloo has quit IRC | 23:56 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!