*** threestrands has joined #openstack-ironic | 00:04 | |
*** cdearborn has quit IRC | 00:06 | |
openstackgerrit | Merged openstack/ironic-inspector master: Port physical network CIDR map hook https://review.opendev.org/718594 | 00:18 |
---|---|---|
openstackgerrit | Merged openstack/ironic stable/train: Update iDRAC doc about vendor passthru timeout https://review.opendev.org/721666 | 00:18 |
*** gyee has quit IRC | 01:04 | |
*** Goneri has quit IRC | 01:12 | |
*** rh-jelabarre has quit IRC | 01:28 | |
*** Qianbiao has joined #openstack-ironic | 01:40 | |
openstackgerrit | Merged openstack/ironic-inspector master: Cleanup py27 support https://review.opendev.org/720551 | 01:46 |
*** yedongcan has joined #openstack-ironic | 01:56 | |
*** ociuhandu has joined #openstack-ironic | 02:08 | |
*** ociuhandu has quit IRC | 02:12 | |
*** mkrai has joined #openstack-ironic | 02:29 | |
*** ricolin has joined #openstack-ironic | 02:33 | |
*** jtomasek has quit IRC | 03:33 | |
*** jtomasek has joined #openstack-ironic | 03:37 | |
*** yedongcan has left #openstack-ironic | 03:57 | |
*** mraineri has quit IRC | 04:23 | |
*** ociuhandu has joined #openstack-ironic | 04:50 | |
*** ociuhandu has quit IRC | 04:55 | |
*** Qianbiao has quit IRC | 05:09 | |
*** bdodd has quit IRC | 05:10 | |
*** bdodd has joined #openstack-ironic | 05:13 | |
*** bnemec has quit IRC | 05:22 | |
*** rcernin has quit IRC | 05:24 | |
*** ricolin has quit IRC | 05:51 | |
*** ricolin has joined #openstack-ironic | 05:53 | |
*** ociuhandu has joined #openstack-ironic | 05:59 | |
*** rcernin has joined #openstack-ironic | 06:09 | |
*** rcernin has quit IRC | 06:10 | |
*** belmoreira has joined #openstack-ironic | 06:10 | |
*** rcernin has joined #openstack-ironic | 06:11 | |
*** belmoreira has quit IRC | 06:12 | |
*** belmoreira has joined #openstack-ironic | 06:23 | |
*** rcernin has quit IRC | 06:25 | |
*** rcernin has joined #openstack-ironic | 06:26 | |
*** Mahendra22 has joined #openstack-ironic | 06:29 | |
openstackgerrit | Dhuldev Valekar proposed openstack/ironic master: [WIP] DRAC: Added redfish management clean steps https://review.opendev.org/721593 | 06:33 |
iurygregory | good morning Ironic o/ | 06:37 |
*** Mahendra22 has quit IRC | 06:46 | |
arne_wiebalck | Good morning iurygregory and ironic! | 06:46 |
*** threestrands has quit IRC | 06:53 | |
*** threestrands has joined #openstack-ironic | 06:54 | |
*** threestrands has quit IRC | 06:55 | |
*** threestrands has joined #openstack-ironic | 06:55 | |
*** threestrands has quit IRC | 06:56 | |
*** threestrands has joined #openstack-ironic | 06:56 | |
iurygregory | arne_wiebalck, o/ | 06:57 |
*** threestrands has quit IRC | 06:58 | |
*** threestrands has joined #openstack-ironic | 06:58 | |
*** threestrands has quit IRC | 06:59 | |
*** threestrands has joined #openstack-ironic | 06:59 | |
*** threestrands has quit IRC | 07:01 | |
*** threestrands has joined #openstack-ironic | 07:01 | |
*** threestrands has quit IRC | 07:02 | |
*** threestrands has joined #openstack-ironic | 07:02 | |
*** threestrands has quit IRC | 07:04 | |
*** threestrands has joined #openstack-ironic | 07:04 | |
*** Mahendra22 has joined #openstack-ironic | 07:04 | |
*** threestrands has quit IRC | 07:05 | |
*** threestrands has joined #openstack-ironic | 07:06 | |
*** ricolin has quit IRC | 07:06 | |
*** threestrands has quit IRC | 07:07 | |
*** threestrands has joined #openstack-ironic | 07:07 | |
*** threestrands has quit IRC | 07:08 | |
*** threestrands has joined #openstack-ironic | 07:08 | |
*** threestrands has quit IRC | 07:10 | |
*** threestrands has joined #openstack-ironic | 07:10 | |
*** Qianbiao has joined #openstack-ironic | 07:10 | |
*** threestrands has quit IRC | 07:11 | |
*** threestrands has joined #openstack-ironic | 07:12 | |
*** threestrands has quit IRC | 07:13 | |
*** threestrands has joined #openstack-ironic | 07:13 | |
*** rcernin has quit IRC | 07:13 | |
*** ricolin has joined #openstack-ironic | 07:13 | |
*** threestrands has quit IRC | 07:14 | |
*** threestrands has joined #openstack-ironic | 07:14 | |
*** threestrands has quit IRC | 07:16 | |
*** Mahendra22 has quit IRC | 07:16 | |
*** threestrands has joined #openstack-ironic | 07:16 | |
*** threestrands has quit IRC | 07:17 | |
*** threestrands has joined #openstack-ironic | 07:18 | |
*** threestrands has quit IRC | 07:19 | |
*** threestrands has joined #openstack-ironic | 07:19 | |
*** threestrands has quit IRC | 07:20 | |
*** threestrands has joined #openstack-ironic | 07:21 | |
*** threestrands has quit IRC | 07:22 | |
*** threestrands has joined #openstack-ironic | 07:22 | |
*** threestrands has quit IRC | 07:23 | |
*** threestrands has joined #openstack-ironic | 07:24 | |
*** rpittau|afk is now known as rpittau | 07:24 | |
rpittau | good morning ironic! o/ | 07:24 |
iurygregory | morning rpittau o/ | 07:24 |
rpittau | hey iurygregory :) | 07:24 |
*** threestrands has quit IRC | 07:25 | |
*** threestrands has joined #openstack-ironic | 07:25 | |
*** threestrands has quit IRC | 07:26 | |
*** threestrands has joined #openstack-ironic | 07:27 | |
*** threestrands has quit IRC | 07:28 | |
*** threestrands has joined #openstack-ironic | 07:28 | |
*** threestrands has quit IRC | 07:29 | |
*** threestrands has joined #openstack-ironic | 07:30 | |
*** yolanda has joined #openstack-ironic | 07:30 | |
*** threestrands has quit IRC | 07:31 | |
*** threestrands has joined #openstack-ironic | 07:31 | |
*** threestrands has quit IRC | 07:32 | |
*** threestrands has joined #openstack-ironic | 07:33 | |
*** threestrands has quit IRC | 07:34 | |
*** threestrands has joined #openstack-ironic | 07:34 | |
*** threestrands has quit IRC | 07:35 | |
*** threestrands has joined #openstack-ironic | 07:36 | |
*** threestrands has quit IRC | 07:37 | |
*** threestrands has joined #openstack-ironic | 07:37 | |
*** threestrands has quit IRC | 07:38 | |
*** threestrands has joined #openstack-ironic | 07:39 | |
*** mkrai has quit IRC | 07:39 | |
*** threestrands has quit IRC | 07:40 | |
*** threestrands has joined #openstack-ironic | 07:40 | |
*** threestrands has quit IRC | 07:41 | |
*** threestrands has joined #openstack-ironic | 07:41 | |
*** threestrands has quit IRC | 07:43 | |
*** threestrands has joined #openstack-ironic | 07:43 | |
*** threestrands has quit IRC | 07:44 | |
*** threestrands has joined #openstack-ironic | 07:45 | |
*** threestrands has quit IRC | 07:46 | |
*** threestrands has joined #openstack-ironic | 07:46 | |
*** threestrands has quit IRC | 07:47 | |
*** threestrands has joined #openstack-ironic | 07:47 | |
*** threestrands has quit IRC | 07:49 | |
*** threestrands has joined #openstack-ironic | 07:49 | |
*** threestrands has quit IRC | 07:50 | |
*** threestrands has joined #openstack-ironic | 07:51 | |
*** threestrands has quit IRC | 07:52 | |
*** threestrands has joined #openstack-ironic | 07:52 | |
*** threestrands has quit IRC | 07:53 | |
* rpittau will spend the next 15-20 minutes closing tabs | 07:54 | |
*** threestrands has joined #openstack-ironic | 07:54 | |
rpittau | still seeing a lot of CI issues | 07:54 |
*** threestrands has quit IRC | 07:55 | |
*** threestrands has joined #openstack-ironic | 07:55 | |
*** threestrands has quit IRC | 07:56 | |
*** threestrands has joined #openstack-ironic | 07:57 | |
*** ociuhandu has quit IRC | 07:57 | |
*** threestrands has quit IRC | 07:58 | |
iurygregory | rpittau, yeah =( | 07:58 |
*** threestrands has joined #openstack-ironic | 07:58 | |
iurygregory | I was a bit worried about ironic-inspector-non-standalone-tempest but the failure rate doesn't look bad looking at zuul .-. | 07:59 |
*** threestrands has quit IRC | 07:59 | |
*** threestrands has joined #openstack-ironic | 08:00 | |
rpittau | tempest tests have indeed an unusual failure rates recently, despite the timeout values are increased way beyond failsafety | 08:00 |
iurygregory | well the one in grenade doesn't seem enough XD | 08:00 |
iurygregory | thats why I pushed https://review.opendev.org/#/c/721638/ | 08:00 |
patchbot | patch 721638 - ironic-inspector - Give more time for smoke tests - 1 patch set | 08:00 |
*** ociuhandu has joined #openstack-ironic | 08:00 | |
iurygregory | but non-standalone decided to make fun of me hehehe | 08:00 |
*** threestrands has quit IRC | 08:01 | |
rpittau | grenade is particularly intensive for us | 08:01 |
*** threestrands has joined #openstack-ironic | 08:01 | |
*** threestrands has quit IRC | 08:02 | |
*** threestrands has joined #openstack-ironic | 08:02 | |
rpittau | is opendev down or my isp is playing tricks again ? | 08:03 |
*** threestrands has quit IRC | 08:04 | |
iurygregory | rpittau, https://opendev.org/openstack/ironic ? | 08:04 |
rpittau | all opendev, but it's just me | 08:04 |
*** threestrands has joined #openstack-ironic | 08:04 | |
iurygregory | well here opendev.org is fine | 08:04 |
iurygregory | it seems | 08:04 |
iurygregory | if you want I can test a specific url | 08:05 |
rpittau | nah, it's my isp connection, too many people watching netflix | 08:05 |
iurygregory | ouch =( | 08:05 |
*** threestrands has quit IRC | 08:05 | |
*** threestrands has joined #openstack-ironic | 08:06 | |
*** threestrands has quit IRC | 08:07 | |
*** threestrands has joined #openstack-ironic | 08:07 | |
*** threestrands has quit IRC | 08:08 | |
*** mkrai has joined #openstack-ironic | 08:10 | |
*** lucasagomes has joined #openstack-ironic | 08:10 | |
*** Qianbiao has quit IRC | 08:11 | |
iurygregory | ok now the functional tests are failling WHAT? .-. | 08:12 |
rpittau | which one? cause it might be caused by requirements change | 08:13 |
iurygregory | openstack-tox-functional-py36 in ironic-inspector | 08:14 |
iurygregory | in my patch to add more timeout XD | 08:14 |
*** tkajinam has quit IRC | 08:15 | |
*** Qianbiao has joined #openstack-ironic | 08:18 | |
*** alexmcleod has joined #openstack-ironic | 08:36 | |
*** ociuhandu has quit IRC | 08:55 | |
*** ociuhandu has joined #openstack-ironic | 08:57 | |
*** HagunKim has joined #openstack-ironic | 08:59 | |
*** ociuhandu has quit IRC | 09:02 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic stable/train: Update iDRAC doc about soft power off timeout https://review.opendev.org/721670 | 09:03 |
*** k_mouza has joined #openstack-ironic | 09:03 | |
*** Mahendra22 has joined #openstack-ironic | 09:19 | |
*** ricolin has quit IRC | 09:22 | |
*** ricolin has joined #openstack-ironic | 09:23 | |
*** ricolin has quit IRC | 09:24 | |
*** ricolin has joined #openstack-ironic | 09:25 | |
*** ricolin has quit IRC | 09:25 | |
*** ricolin_ has joined #openstack-ironic | 09:25 | |
*** k_mouza has quit IRC | 09:31 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic stable/stein: Fix issue where server fails to reboot https://review.opendev.org/721797 | 09:33 |
*** Mahendra22 has quit IRC | 09:35 | |
*** ricolin_ is now known as ricolin | 09:36 | |
*** k_mouza has joined #openstack-ironic | 09:38 | |
*** dtantsur|afk is now known as dtantsur | 09:42 | |
dtantsur | morning ironic | 09:42 |
iurygregory | morning dtantsur | 09:46 |
*** ociuhandu has joined #openstack-ironic | 09:48 | |
*** ociuhandu has quit IRC | 09:51 | |
*** ociuhandu has joined #openstack-ironic | 09:52 | |
dtantsur | folks, can I get some reviews on https://review.opendev.org/#/c/718358/ please? | 10:02 |
patchbot | patch 718358 - ironic - Improve the command status checks in the agent's p... - 3 patch sets | 10:02 |
rpittau | dtantsur: sure | 10:03 |
rpittau | dtantsur: I was looking at recent failures in some tempest jobs and I just noticed one thing they have in common, failing to install the bootloader, it's a recent issue but can't track it to something specific | 10:04 |
rpittau | for example https://6ef9c7b9e14ab58a181a-20b98b64cfa6ea87451df6eaddafb782.ssl.cf2.rackcdn.com/721649/1/check/sushy-tempest-ironic-partition-redfish-src/17d6454/controller/logs/ironic-bm-logs/node-0_console_2020-04-22-08%3A58%3A12_log.txt | 10:04 |
dtantsur | rpittau: mount point doesn't exist is what iurygregory saw in his UEFI+wholedisk patch | 10:06 |
dtantsur | seems a general problem? | 10:06 |
rpittau | yeah, right | 10:06 |
iurygregory | yeah same thing i saw in wholedisk + uefi .-. | 10:06 |
rpittau | this started happening yesterday or 2 days ago | 10:06 |
rpittau | anything before the 20th looks fine | 10:07 |
rpittau | at least in the jobs I checked | 10:07 |
openstackgerrit | Aija Jaunteva proposed openstack/ironic stable/rocky: Fix issue where server fails to reboot https://review.opendev.org/721803 | 10:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM test backward compatibility https://review.opendev.org/721804 | 10:09 |
* iurygregory brb | 10:09 | |
*** Qianbiao has quit IRC | 10:17 | |
*** k_mouza has quit IRC | 10:22 | |
* iurygregory back | 10:29 | |
*** k_mouza has joined #openstack-ironic | 10:31 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic stable/queens: Fix issue where server fails to reboot https://review.opendev.org/721811 | 10:36 |
*** priteau has joined #openstack-ironic | 10:39 | |
*** rpittau is now known as rpittau|bbl | 10:42 | |
*** yedongcan has joined #openstack-ironic | 10:45 | |
dtantsur | hmm, the uefi job seems broken on stable/stein :( anyone has a clue? | 10:50 |
iurygregory | I can look after lunch | 10:56 |
iurygregory | we did some backports maybe we are missing something | 10:56 |
dtantsur | yeah | 10:57 |
*** Mahendra22 has joined #openstack-ironic | 11:00 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM test backward compatibility https://review.opendev.org/721804 | 11:03 |
openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: BasePhysnetHook - Follow up https://review.opendev.org/721817 | 11:04 |
openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Use autospec=True in mocks https://review.opendev.org/721819 | 11:06 |
*** k_mouza has quit IRC | 11:07 | |
openstackgerrit | Merged openstack/ironic stable/train: Update iDRAC doc about soft power off timeout https://review.opendev.org/721670 | 11:15 |
openstackgerrit | Merged openstack/ironic master: Prepare release notes/docs for 15.0 release https://review.opendev.org/721316 | 11:20 |
*** k_mouza has joined #openstack-ironic | 11:31 | |
*** jtomasek_ has joined #openstack-ironic | 11:37 | |
*** jtomasek has quit IRC | 11:39 | |
*** Qianbiao has joined #openstack-ironic | 11:39 | |
*** ociuhandu has quit IRC | 11:39 | |
*** mkrai has quit IRC | 11:40 | |
*** mkrai_ has joined #openstack-ironic | 11:40 | |
*** k_mouza has quit IRC | 11:41 | |
*** k_mouza has joined #openstack-ironic | 11:42 | |
*** Mahendra22 has quit IRC | 11:42 | |
*** rh-jelabarre has joined #openstack-ironic | 11:48 | |
*** rpittau|bbl is now known as rpittau | 11:50 | |
*** yedongcan has left #openstack-ironic | 11:55 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Add missing Redfish params to iDRAC doc https://review.opendev.org/721836 | 11:58 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM test backward compatibility https://review.opendev.org/721804 | 12:02 |
openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Use autospec=True in mocks https://review.opendev.org/721819 | 12:11 |
*** hjensas is now known as hjensas|afk | 12:15 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Give more time for grenade job https://review.opendev.org/721638 | 12:17 |
*** ociuhandu has joined #openstack-ironic | 12:23 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Give more time for grenade job https://review.opendev.org/721638 | 12:24 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector master: Give more time for grenade job https://review.opendev.org/721638 | 12:26 |
*** ociuhandu has quit IRC | 12:29 | |
*** ociuhandu has joined #openstack-ironic | 12:29 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM test backward compatibility https://review.opendev.org/721804 | 12:30 |
*** dougsz has joined #openstack-ironic | 12:30 | |
dtantsur | arne_wiebalck: another approach to test the UEFI issue ^^ | 12:30 |
openstackgerrit | Mahendra Kamble proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port https://review.opendev.org/693328 | 12:38 |
*** Goneri has joined #openstack-ironic | 12:38 | |
*** hjensas|afk is now known as hjensas | 12:41 | |
*** Qianbiao has quit IRC | 12:43 | |
*** Qianbiao has joined #openstack-ironic | 12:43 | |
*** Qianbiao has quit IRC | 12:45 | |
*** Qianbiao has joined #openstack-ironic | 12:58 | |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-ui stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/721896 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-ui stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/721897 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/ussuri https://review.opendev.org/721898 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/ironic-ui master: Add Python3 victoria unit tests https://review.opendev.org/721900 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/bifrost stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/721901 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/bifrost stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/721903 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/bifrost master: Update master for stable/ussuri https://review.opendev.org/721904 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/bifrost master: Add Python3 victoria unit tests https://review.opendev.org/721906 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/721910 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/721913 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/ussuri https://review.opendev.org/721916 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-generic-switch master: Add Python3 victoria unit tests https://review.opendev.org/721918 | 13:09 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/721928 | 13:10 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/721930 | 13:10 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/ussuri https://review.opendev.org/721933 | 13:10 |
openstackgerrit | OpenStack Release Bot proposed openstack/networking-baremetal master: Add Python3 victoria unit tests https://review.opendev.org/721935 | 13:10 |
dtantsur | oh wow | 13:10 |
iurygregory | dtantsur, you can go slow =) | 13:11 |
rpittau | openstack release bot is ddosing us | 13:11 |
dtantsur | I tend to just approve the automated tox.ini changes if they pass the CI | 13:11 |
iurygregory | rpittau, don't say this | 13:12 |
rpittau | it's the raise of the bots and it starts from here, save yourselves | 13:12 |
iurygregory | hehehehe | 13:13 |
TheJulia | good morning | 13:16 |
iurygregory | good morning TheJulia =) | 13:16 |
dtantsur | TheJulia: morning | 13:17 |
TheJulia | The bots... are just bots... doing a job... that human overlords programmed. | 13:17 |
* iurygregory agrees with TheJulia | 13:17 | |
* Qianbiao stupid human beings... You are programming for us not you. | 13:20 | |
Qianbiao | :) | 13:20 |
dtantsur | :D | 13:25 |
TheJulia | So here is the question, since they believe all of this is a simulation, are we actually the bots?!? | 13:30 |
* dtantsur watched enough of WestWorld to not feel easy about this | 13:31 | |
TheJulia | eggdrop, Timewarp edition, version 11920193 | 13:31 |
TheJulia | I've yet to watch westworld | 13:31 |
dtantsur | TheJulia: you haven't watched any seasons? | 13:34 |
TheJulia | no | 13:34 |
dtantsur | TheJulia: the 1st one is particularly worth it (but huge trigger warnings all over the place) | 13:37 |
arne_wiebalck | dtantsur: thanks for your comment and pointers for the UEFI compatibility issue, I will have a look as soon as I can | 13:39 |
* arne_wiebalck juggles too many things today | 13:40 | |
openstackgerrit | Merged openstack/ironic stable/rocky: Fix SpanLength calculation for DRAC RAID configuration https://review.opendev.org/721709 | 13:44 |
TheJulia | dtantsur: eh, triggery would be bad for me. | 13:44 |
dtantsur | TheJulia: a lot of explicit content (violence of different sorts). you're warned. | 13:45 |
*** tzumainn has joined #openstack-ironic | 13:46 | |
* iurygregory never watched WestWorld also | 13:48 | |
rpittau | I suggest to watch the original movie too, maybe before the series | 13:48 |
* dtantsur hasn't watched the original one | 13:49 | |
iurygregory | I had no idea that there was a movie LOL | 13:49 |
rpittau | I think it can be considered vintage nowadays | 13:49 |
rpittau | it's from 1973 :) | 13:49 |
*** rloo has joined #openstack-ironic | 13:51 | |
openstackgerrit | Merged openstack/networking-generic-switch stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/721910 | 13:52 |
iurygregory | wow | 13:53 |
TheJulia | I think I had actually seen part of the 70s movie. I have a habit of playing really really bad scifi movies in the background | 13:55 |
TheJulia | like... if has 1.5 stars, it is for dinner watching! | 13:55 |
rpittau | Westworld (the movie) is awesome! high score rate on rotten tomatoes too | 13:55 |
openstackgerrit | Merged openstack/networking-baremetal stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/721928 | 13:57 |
openstackgerrit | Merged openstack/networking-baremetal stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/721930 | 14:01 |
*** bnemec has joined #openstack-ironic | 14:01 | |
openstackgerrit | Merged openstack/networking-generic-switch stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/721913 | 14:08 |
TheJulia | erbarr: rajinir Is there anything specifically that is causing the dell CI to be failing? | 14:10 |
erbarr | TheJulia, install_bootloader | 14:12 |
arne_wiebalck | TheJulia: https://review.opendev.org/#/c/720781 was supposed to fix this, but does not, it fixes something else, though :) | 14:17 |
patchbot | patch 720781 - ironic - Don't break UEFI install with older IPAs - 1 patch set | 14:17 |
arne_wiebalck | dtantsur is on this as well | 14:17 |
erbarr | i tried updating a job to use x-overcloud image and maybe i missed something, i'm getting this after that | 14:17 |
erbarr | ERROR ironic.drivers.modules.agent_base ironic.common.exception.InstanceDeployFailure: Failed to install a bootloader when deploying node 849ad7ad-cf78-4bc8-9e21-cced8e0574cb. Error: {'type': 'CommandExecutionError', 'code': 500, 'message': 'Command execution failed', 'details': "Installing GRUB2 boot loader to device /dev/sda failed with Unexpected error while running command.\nCommand: mount -o bind /dev | 14:17 |
erbarr | /tmp/tmp6tc520bg/dev\nExit code: 32\nStdout: ''\nStderr: 'mount: /tmp/tmp6tc520bg/dev: mount point does not exist.\\n'."} | 14:17 |
TheJulia | erbarr: what image are you deploying to the hardware? | 14:18 |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Actually log state transition timeouts in debug log https://review.opendev.org/720620 | 14:20 |
TheJulia | an overcloud image is a partition image, it should have /dev... in theory | 14:20 |
erbarr | yea, so i tried to switch to that by changing IMAGE_URLS to a local package of overcloud-full but i'm thinking cirros is still being used somehow | 14:24 |
openstackgerrit | Merged openstack/bifrost master: Update master for stable/ussuri https://review.opendev.org/721904 | 14:27 |
openstackgerrit | Merged openstack/networking-generic-switch master: Update master for stable/ussuri https://review.opendev.org/721916 | 14:29 |
TheJulia | well that is just bizzar | 14:29 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Log when IPA fallback occurs on bootloader install https://review.opendev.org/721988 | 14:32 |
dtantsur | TheJulia: the "mount point doesn't exist" thing can be seen in the CI as well, rpittau mentioned it this morning | 14:35 |
openstackgerrit | Merged openstack/networking-baremetal master: Update master for stable/ussuri https://review.opendev.org/721933 | 14:35 |
openstackgerrit | Merged openstack/ironic-ui stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/721896 | 14:35 |
rpittau | oh yeah | 14:35 |
rpittau | TheJulia: https://6fe7a4c45f2f7bdcea66-eeccdc6968a6b16fa4ca2e3ee7c1080d.ssl.cf5.rackcdn.com/721668/1/check/ironicclient-tempest/4af52fd/controller/logs/ironic-bm-logs/node-0_console_2020-04-22-09%3A27%3A35_log.txt | 14:36 |
TheJulia | yeah, because we deploy an image that lacks the ability for it to be set | 14:36 |
openstackgerrit | Merged openstack/ironic-ui stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/721897 | 14:39 |
openstackgerrit | Merged openstack/ironic-ui master: Update master for stable/ussuri https://review.opendev.org/721898 | 14:39 |
TheJulia | erbarr: it would be awesome if IPA's debug logs were also saved in CI jobs. | 14:39 |
openstackgerrit | Merged openstack/ironic-ui master: Add Python3 victoria unit tests https://review.opendev.org/721900 | 14:39 |
dtantsur | arne_wiebalck: I don't seem to be able to make https://review.opendev.org/#/c/721804/ test what I want :( | 14:40 |
patchbot | patch 721804 - ironic - DNM test backward compatibility - 4 patch sets | 14:40 |
dtantsur | ah, right. because we don't call install_bootloader in the CI.. | 14:40 |
* dtantsur is amazingly slow today | 14:40 | |
erbarr | TheJulia: i can do that, just need to know how lol | 14:41 |
TheJulia | https://stash.dellemc-community.org/logs/dellemc-poweredge-R740-R640-UEFI-tempest-dsvm-ironic-redfish/2020-04-22:05:00/721219/2/logs/screen-ir-cond.txt <-- looks like it an access issue to the bmc on this job | 14:43 |
TheJulia | erbarr: one minute | 14:43 |
TheJulia | erbarr: so https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_1ce/721803/1/check/ironic-dsvm-standalone/1ce339a/controller/logs/ironic-bm-logs/deploy_logs/ and I suspect ironic-bm-logs is not preserved/saved since it started as baremeatl console logs, but the agent logs get put into a subdirectory | 14:45 |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/721901 | 14:49 |
openstackgerrit | Merged openstack/bifrost stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/721903 | 14:49 |
NobodyCam | Good morning ironic’ers | 14:49 |
rpittau | hey NobodyCam :) | 14:50 |
dtantsur | oh hi NobodyCam | 14:50 |
NobodyCam | Hey hey rpittau and dtantsur :) | 14:51 |
TheJulia | o/ NobodyCam | 14:51 |
NobodyCam | Good morning TheJulia | 14:52 |
dtantsur | NobodyCam: long time no see, how's life treating you? | 14:52 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: DNM test backward compatibility https://review.opendev.org/721804 | 14:54 |
dtantsur | TheJulia: I've been trying to test how IPA reacts, this is another attempt ^^^ | 14:54 |
dtantsur | * IPA reacts on unknown parameters | 14:54 |
arne_wiebalck | dtantsur: for the return code vs exception, is that true for all commands sent to the IPA, or only install_bootloader specifically? | 14:55 |
NobodyCam | Doing good. Going a little stir crazy with the shelter in place | 14:55 |
arne_wiebalck | dtantsur: I am asking as finalize_rescue further does sth very similar | 14:55 |
dtantsur | arne_wiebalck: no idea. trying in the patch above | 14:55 |
arne_wiebalck | further down | 14:55 |
dtantsur | NobodyCam: we're all in this boat, yeah. well, we can (and do) go outside. | 14:55 |
NobodyCam | Hehehe yep | 14:56 |
NobodyCam | We’ve got our masks and gloves | 14:56 |
dtantsur | actually, they've started relaxing restrictions in Germany. but introducing mandatory masks in transport and shops. | 14:56 |
NobodyCam | Very nice 👍 | 14:57 |
openstackgerrit | Merged openstack/bifrost master: Add Python3 victoria unit tests https://review.opendev.org/721906 | 14:58 |
TheJulia | oh, were getting a 200 from the agent... | 15:01 |
TheJulia | patch inbound | 15:04 |
TheJulia | at least, without unit tests, but should give us an idea | 15:04 |
TheJulia | we're likley going to have to fix ip and have code in the _command method in the short term | 15:05 |
dtantsur | yeah | 15:07 |
*** gyee has joined #openstack-ironic | 15:11 | |
TheJulia | wrote a quick test to make sure thigns are doing what I expect | 15:12 |
TheJulia | and also lost networking for a litlte bit :( | 15:12 |
TheJulia | yeah, a quick unit test was a good idea | 15:25 |
TheJulia | and finally pep8'ing | 15:37 |
*** belmoreira has quit IRC | 15:40 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix agent_client handling of embedded error codes https://review.opendev.org/722027 | 15:45 |
TheJulia | dtantsur: arne_wiebalck: erbarr ^^^ | 15:45 |
iurygregory | I like the magic in grenade, you set a new var, you think the job is green because of that, you look if the var has the value you set and it doesn't | 15:48 |
arne_wiebalck | TheJulia: Oh, nice! | 15:50 |
* arne_wiebalck was not aware how the API error was generated | 15:50 | |
openstackgerrit | Merged openstack/tenks master: Add pep8 import order check https://review.opendev.org/721660 | 15:51 |
arne_wiebalck | This should fix erbarr's issue FWIU. None the less, the patch I have put up is OK in addition as it fixes a separate issue, no? | 15:52 |
arne_wiebalck | This one https://review.opendev.org/#/c/720781 | 15:52 |
patchbot | patch 720781 - ironic - Don't break UEFI install with older IPAs - 1 patch set | 15:52 |
arne_wiebalck | has a wrong title, though | 15:53 |
*** Qianbiao has quit IRC | 15:58 | |
*** lucasagomes has quit IRC | 16:03 | |
*** igordc has joined #openstack-ironic | 16:12 | |
TheJulia | arne_wiebalck: I _think_ so | 16:13 |
TheJulia | NobodyCam: we're going super stir crazy, fwiw | 16:14 |
arne_wiebalck | TheJulia: no, both patches are needed if the node boots in UEFI mode | 16:18 |
dtantsur | TheJulia: I'm not sure we can merge it as it is. Some code in ironic may rely on getting command status rather than an exception. | 16:18 |
dtantsur | including install_bootloader in question | 16:18 |
dtantsur | and a check in agent.py as well | 16:19 |
* TheJulia runs the entire unit test suite instead of just command tests | 16:20 | |
TheJulia | I suspect when we refactored IPA, that we may have chagned how errors are returned, at least that is my perception. Then again I approached this from the standpoint that the code returned should be representitive and should be what status_code is | 16:21 |
dtantsur | it's worse, sometimes we do return the correct HTTP code | 16:22 |
dtantsur | for example on command not found | 16:22 |
* dtantsur dunno | 16:22 | |
TheJulia | yes, which is why I wrote that as I did | 16:22 |
TheJulia | so we fall back if not embedded | 16:22 |
dtantsur | right | 16:23 |
TheJulia | it doesn't look like we use code anywhere else | 16:23 |
TheJulia | as a field | 16:23 |
*** k_mouza has quit IRC | 16:24 | |
TheJulia | we don't even get a command_status back it looks like | 16:24 |
dtantsur | TheJulia: yeah, but we use command_status | 16:24 |
TheJulia | I guess CI is going to be the only real test | 16:24 |
dtantsur | CI doesn't test negative scenarios, I'm afraid | 16:25 |
*** k_mouza has joined #openstack-ironic | 16:25 | |
TheJulia | true | 16:25 |
* TheJulia ponders | 16:25 | |
*** k_mouza has quit IRC | 16:26 | |
openstackgerrit | Merged openstack/ironic master: Revert "Generalize ISO building for virtual media driver" https://review.opendev.org/721309 | 16:27 |
*** k_mouza has joined #openstack-ironic | 16:27 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Fix RAID configuration with idrac-wsman interface https://review.opendev.org/720702 | 16:27 |
*** ociuhandu has quit IRC | 16:27 | |
TheJulia | hmmm | 16:27 |
*** ociuhandu has joined #openstack-ironic | 16:28 | |
rpittau | bye all! o/ | 16:29 |
TheJulia | I think we're goign to need to look for TypeError in the string | 16:29 |
*** rpittau is now known as rpittau|afk | 16:29 | |
TheJulia | error only seems to come from the exception formatting, fwiw | 16:30 |
TheJulia | err | 16:30 |
TheJulia | code in the error field | 16:30 |
*** mkrai_ has quit IRC | 16:32 | |
*** mkrai__ has joined #openstack-ironic | 16:32 | |
TheJulia | yup, code will always be 500 when the status has failed | 16:32 |
*** ociuhandu has quit IRC | 16:32 | |
*** yaawang has joined #openstack-ironic | 16:32 | |
TheJulia | https://github.com/openstack/ironic-python-agent/blob/b9e320e76fbed100502d29d3b1d6f0586f1b873f/ironic_python_agent/errors.py#L23 | 16:32 |
*** k_mouza has quit IRC | 16:33 | |
*** yaawang_ has quit IRC | 16:33 | |
*** mkrai__ has quit IRC | 16:37 | |
* TheJulia crosses her fingers | 16:38 | |
openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Add link to other Redfish parms to iDRAC doc https://review.opendev.org/721836 | 16:43 |
*** riuzen has joined #openstack-ironic | 16:45 | |
arne_wiebalck | bye everyone o/ | 16:46 |
*** ociuhandu has joined #openstack-ironic | 16:49 | |
TheJulia | new version inbound | 16:50 |
*** mbeierl has quit IRC | 16:50 | |
*** riuzen has quit IRC | 16:51 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix agent_client handling of embedded errors https://review.opendev.org/722027 | 16:53 |
TheJulia | dtantsur: arne_wiebalck ^^^ | 16:54 |
dtantsur | I'll think about it tomorrow, if you don't mind. EINSUFFICIENTRESOURCES | 16:54 |
*** yolanda has quit IRC | 16:54 | |
*** dtantsur is now known as dtantsur|afk | 16:55 | |
dtantsur|afk | o/ | 16:55 |
TheJulia | no worries, goodnight | 16:55 |
TheJulia | erbarr: also ^^ | 16:55 |
erbarr | thanks | 16:56 |
arne_wiebalck | erbarr: if you test with UEFI, I think you would still need both patches: TheJulia's to make it an exception and the first to not re-raise except for s/w RAID | 16:57 |
erbarr | there are both cases in the CI, bios and uefi | 17:00 |
arne_wiebalck | and which combination of patches? | 17:02 |
TheJulia | so they aren't running software raid, but they do hit errors and we don't know how to fallback. We realistically always need to support falling back | 17:06 |
TheJulia | so saying we need to fallback only when there is software raid is actually wrong | 17:06 |
TheJulia | the client will get typeerror from the api regardless because of the extra argument | 17:07 |
arne_wiebalck | the fallback if there is not software RAID | 17:10 |
TheJulia | except, if older ramdisk, no matter what the exception will be raised and the first command will fail | 17:11 |
arne_wiebalck | in case of s/w RAID, there is no point in retrying | 17:11 |
arne_wiebalck | all other cases should be retried, | 17:11 |
arne_wiebalck | is that correct? | 17:11 |
arne_wiebalck | software RAID requires the parameter in the UEFI case | 17:12 |
* arne_wiebalck is maybe too confused at the end of the day | 17:12 | |
*** dougsz has quit IRC | 17:13 | |
TheJulia | yes | 17:13 |
TheJulia | you are correct | 17:13 |
TheJulia | so log/return, otherwise pass-through and allow retry | 17:13 |
TheJulia | arne_wiebalck: I'm likely confused | 17:14 |
openstackgerrit | Merged openstack/networking-baremetal master: Add Python3 victoria unit tests https://review.opendev.org/721935 | 17:15 |
* arne_wiebalck calls it a day for real now | 17:16 | |
arne_wiebalck | bye everyone o/ | 17:16 |
*** priteau has quit IRC | 17:18 | |
openstackgerrit | Merged openstack/networking-generic-switch master: Add Python3 victoria unit tests https://review.opendev.org/721918 | 17:22 |
*** ociuhandu has quit IRC | 17:24 | |
NobodyCam | TheJulia: oh ya same here | 17:40 |
TheJulia | NobodyCam: a paved canyon road may be fun this weekend. #justsaying | 17:42 |
TheJulia | I think box canyon is gravel | 17:42 |
NobodyCam | 😉 hehe | 17:43 |
TheJulia | ohhhh it is paved | 17:43 |
TheJulia | If only http://www.internationalbananamuseum.com/ was open | 17:45 |
*** cdearborn has joined #openstack-ironic | 17:55 | |
*** ociuhandu has joined #openstack-ironic | 18:02 | |
*** sshnaidm is now known as sshnaidm|afk | 18:04 | |
*** k_mouza has joined #openstack-ironic | 18:33 | |
*** k_mouza has quit IRC | 18:38 | |
*** ociuhandu has quit IRC | 18:38 | |
*** ociuhandu has joined #openstack-ironic | 18:44 | |
*** ociuhandu has quit IRC | 18:48 | |
*** ociuhandu has joined #openstack-ironic | 18:48 | |
*** dougsz has joined #openstack-ironic | 19:06 | |
*** ociuhandu has quit IRC | 19:22 | |
*** ociuhandu has joined #openstack-ironic | 19:33 | |
*** jdandrea has joined #openstack-ironic | 19:38 | |
*** ociuhandu has quit IRC | 19:38 | |
*** Lucas_Gray has joined #openstack-ironic | 20:09 | |
openstackgerrit | Verification of a change to openstack/ironic-inspector failed: Convert jobs to dib https://review.opendev.org/712516 | 20:10 |
*** Lucas_Gray has quit IRC | 20:12 | |
*** ociuhandu has joined #openstack-ironic | 20:17 | |
*** ociuhandu has quit IRC | 20:28 | |
*** dougsz has quit IRC | 20:47 | |
*** ociuhandu has joined #openstack-ironic | 21:00 | |
openstackgerrit | Merged openstack/ironic-inspector master: BasePhysnetHook - Follow up https://review.opendev.org/721817 | 21:02 |
*** ociuhandu has quit IRC | 21:10 | |
openstackgerrit | Verification of a change to openstack/ironic failed: Improve the command status checks in the agent's process_next_step https://review.opendev.org/718358 | 21:22 |
*** ociuhandu has joined #openstack-ironic | 21:41 | |
*** ociuhandu has quit IRC | 21:52 | |
*** ociuhandu has joined #openstack-ironic | 21:52 | |
*** ociuhandu has quit IRC | 21:57 | |
openstackgerrit | Merged openstack/ironic master: Fix RAID configuration with idrac-wsman interface https://review.opendev.org/720702 | 22:21 |
*** rcernin has joined #openstack-ironic | 22:31 | |
*** ociuhandu has joined #openstack-ironic | 22:44 | |
*** tkajinam has joined #openstack-ironic | 22:54 | |
*** ociuhandu has quit IRC | 23:01 | |
*** tzumainn has quit IRC | 23:09 | |
*** ociuhandu has joined #openstack-ironic | 23:33 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!