| *** opendevtest <opendevtest!~limnoria@104.239.144.232> has joined #openstack-ironic | 01:14 | |
| *** opendevmeet <opendevmeet!~limnoria@104.239.144.232> has joined #openstack-ironic | 01:23 | |
| *** opendevmeet is now known as Guest1653 | 01:23 | |
| *** opendevmeet` <opendevmeet`!~limnoria@104.239.144.232> has joined #openstack-ironic | 01:32 | |
| *** opendevmeet` <opendevmeet`!~limnoria@104.239.144.232> has joined #openstack-ironic | 01:39 | |
| *** opendevmeet` <opendevmeet`!~limnoria@104.239.144.232> has joined #openstack-ironic | 01:43 | |
| *** opendevmeet <opendevmeet!~limnoria@104.239.144.232> has joined #openstack-ironic | 02:01 | |
| *** sdanni <sdanni!~sdanni@pool-98-118-100-55.bstnma.fios.verizon.net> has quit IRC (Ping timeout: 480 seconds) | 02:26 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Remote host closed the connection) | 02:47 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 02:51 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Ping timeout: 480 seconds) | 03:00 | |
| *** opendevmeet <opendevmeet!~limnoria@104.239.144.232> has joined #openstack-ironic | 03:06 | |
| *** jelabarre-rh <jelabarre-rh!~jlabarre@2601:184:c300:2231:fc5a:69de:cf7f:7208> has quit IRC (Ping timeout: 480 seconds) | 03:11 | |
| *** eandersson <eandersson!~eandersso@ec2-34-212-68-203.us-west-2.compute.amazonaws.com> has quit IRC (Quit: bye) | 03:12 | |
| *** eandersson <eandersson!~eandersso@ec2-34-212-68-203.us-west-2.compute.amazonaws.com> has joined #openstack-ironic | 03:12 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 03:18 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Ping timeout: 480 seconds) | 03:26 | |
| *** redrobot <redrobot!~redrobot@108-84-79-198.lightspeed.snantx.sbcglobal.net> has quit IRC (Remote host closed the connection) | 03:40 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 04:05 | |
| *** tzumainn_ <tzumainn_!~tzumainn@2601:19b:881:8630:fa34:41ff:feb1:df6e> has quit IRC (Ping timeout: 480 seconds) | 04:10 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Ping timeout: 480 seconds) | 04:13 | |
| *** ricolin_ <ricolin_!~ricolin@118.150.144.205> has joined #openstack-ironic | 04:31 | |
| *** ricolin <ricolin!~ricolin@118.150.144.205> has quit IRC (Ping timeout: 480 seconds) | 04:35 | |
| iurygregory | good morning Ironic! Happy Friday! | 06:05 |
|---|---|---|
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 06:06 | |
| *** stendulker <stendulker!~stendulke@122.171.207.150> has joined #openstack-ironic | 06:17 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Ping timeout: 480 seconds) | 06:19 | |
| *** Zainub_Wahid <Zainub_Wahid!~Zainub_Wa@202.47.34.153> has joined #openstack-ironic | 06:27 | |
| arne_wiebalck | Good morning, iurygregory janders and Ironic! | 06:59 |
| iurygregory | morning arne_wiebalck o/ | 07:00 |
| *** tosky <tosky!~luigi@dynamic-adsl-78-13-253-141.clienti.tiscali.it> has joined #openstack-ironic | 07:14 | |
| *** rpittau|afk is now known as rpittau | 07:17 | |
| rpittau | good morning ironic! o/ | 07:17 |
| iurygregory | hey rpittau o/ | 07:17 |
| rpittau | woohoo I got banned from freenode! \o/ | 07:17 |
| iurygregory | WHAT?! | 07:17 |
| iurygregory | what did you say to get banned? | 07:18 |
| rpittau | nothing | 07:18 |
| iurygregory | <O> | 07:18 |
| opendevreview | likui proposed openstack/ironic-prometheus-exporter master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/795933 | 07:18 |
| rpittau | I'm tempted to write to their support, just for fun | 07:19 |
| iurygregory | rpittau, lol | 07:19 |
| opendevreview | likui proposed openstack/ironic-prometheus-exporter master: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/795934 | 07:20 |
| opendevreview | likui proposed openstack/ironic-prometheus-exporter master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/795935 | 07:25 |
| opendevreview | likui proposed openstack/ironic-ui master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/ironic-ui/+/795937 | 07:30 |
| opendevreview | likui proposed openstack/ironic-ui master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/ironic-ui/+/795939 | 07:31 |
| opendevreview | likui proposed openstack/ironic-ui master: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-ui/+/795940 | 07:32 |
| opendevreview | likui proposed openstack/ironic-python-agent master: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-python-agent/+/795942 | 07:37 |
| opendevreview | Riccardo Pittau proposed openstack/sushy stable/wallaby: Fix incorrect formatting and a Python 3.10 failure https://review.opendev.org/c/openstack/sushy/+/795877 | 07:40 |
| *** stendulker <stendulker!~stendulke@122.171.207.150> has quit IRC (Ping timeout: 480 seconds) | 07:53 | |
| *** lucasagomes <lucasagomes!~lucasagom@89.100.20.18> has joined #openstack-ironic | 07:56 | |
| *** opendevstatus is now known as Guest1684 | 08:09 | |
| iurygregory | wait... the bot can change his nick O.o | 08:10 |
| *** derekh <derekh!~derekh@2a01:b340:80:6d6e:6423:15e0:3703:2> has joined #openstack-ironic | 08:10 | |
| iurygregory | I probably need more coffee | 08:10 |
| rpittau | the bot is alive! | 08:11 |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 08:12 | |
| iurygregory | run for your lives :D | 08:13 |
| *** tosky <tosky!~luigi@dynamic-adsl-78-13-253-141.clienti.tiscali.it> has quit IRC (Ping timeout: 480 seconds) | 08:13 | |
| *** mgoddard- <mgoddard-!~mgoddard@238.240.125.91.dyn.plus.net> has joined #openstack-ironic | 08:14 | |
| *** mgoddard <mgoddard!~mgoddard@187.240.125.91.dyn.plus.net> has quit IRC (Ping timeout: 480 seconds) | 08:18 | |
| *** mgoddard- is now known as mgoddard | 08:18 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Ping timeout: 480 seconds) | 08:20 | |
| *** timeu <timeu!~uemit.ser@mordor.imp.ac.at> has quit IRC (Quit: WeeChat 2.8) | 08:23 | |
| *** sshnaidm is now known as sshnaidm|afk | 08:24 | |
| *** tosky <tosky!~luigi@dynamic-adsl-78-13-253-141.clienti.tiscali.it> has joined #openstack-ironic | 08:24 | |
| *** opendevmeet <opendevmeet!~limnoria@104.239.144.232> has joined #openstack-ironic | 08:39 | |
| *** opendevmeet is now known as Guest1685 | 08:40 | |
| *** opendevstatus_ <opendevstatus_!~opendevst@104.130.70.91> has joined #openstack-ironic | 08:46 | |
| *** opendevstatus_ is now known as opendevstatus__ | 08:47 | |
| *** opendevstatus__ is now known as opendevstatus___ | 08:47 | |
| *** opendevstatus___ is now known as opendevstatus____ | 08:47 | |
| *** opendevstatus____ is now known as opendevstatus_____ | 08:48 | |
| *** opendevstatus_____ is now known as opendevstatus______ | 08:48 | |
| iurygregory | seems like the bots are alive for real :D | 08:48 |
| arne_wiebalck | iurygregory: your door bell rang? | 08:52 |
| dtantsur | TheJulia: great! could you follow-up with using the standard csv module if possible? | 08:52 |
| iurygregory | arne_wiebalck, nope ( I hope it won't) | 08:53 |
| iurygregory | morning dtantsur o/ | 08:53 |
| dtantsur | good morning | 08:55 |
| *** opendevstatus______ <opendevstatus______!~opendevst@104.130.70.91> has quit IRC (Ping timeout: 480 seconds) | 08:58 | |
| *** Guest1684 <Guest1684!~opendevst@eavesdrop01.openstack.org> has quit IRC (Remote host closed the connection) | 08:58 | |
| *** Qianbiao <Qianbiao!~Qianbiao@94.74.82.60> has joined #openstack-ironic | 09:05 | |
| opendevreview | Merged openstack/ironic-python-agent master: Utilize CSV file for EFI loader selection https://review.opendev.org/c/openstack/ironic-python-agent/+/795862 | 09:12 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/795934 | 09:12 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/795935 | 09:12 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/795933 | 09:13 |
| *** opendevstatus_ <opendevstatus_!~opendevst@158.69.72.85> has joined #openstack-ironic | 09:14 | |
| *** opendevstatus_ is now known as opendevstatus__ | 09:16 | |
| *** opendevstatus__ is now known as opendevstatus___ | 09:16 | |
| *** opendevstatus___ is now known as opendevstatus____ | 09:16 | |
| *** opendevstatus____ is now known as opendevstatus_____ | 09:16 | |
| *** opendevstatus_____ is now known as opendevstatus______ | 09:16 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.0: Utilize CSV file for EFI loader selection https://review.opendev.org/c/openstack/ironic-python-agent/+/795882 | 09:25 |
| *** opendevstatus______ <opendevstatus______!~opendevst@158.69.72.85> has quit IRC (Ping timeout: 480 seconds) | 09:25 | |
| *** timeu <timeu!~uemit.ser@mordor.imp.ac.at> has joined #openstack-ironic | 09:29 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent stable/wallaby: Make _get_efi_bootloaders return relative paths https://review.opendev.org/c/openstack/ironic-python-agent/+/795883 | 09:30 |
| rpittau | we need this ^ before backporting the CSV change | 09:30 |
| *** opendevstatus <opendevstatus!~opendevst@104.239.144.232> has joined #openstack-ironic | 09:32 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent stable/wallaby: Utilize CSV file for EFI loader selection https://review.opendev.org/c/openstack/ironic-python-agent/+/795965 | 09:34 |
| rpittau | there you go ^ | 09:34 |
| opendevreview | Merged openstack/sushy stable/wallaby: Fix incorrect formatting and a Python 3.10 failure https://review.opendev.org/c/openstack/sushy/+/795877 | 09:51 |
| *** opendevmeet <opendevmeet!~limnoria@104.239.144.232> has joined #openstack-ironic | 09:57 | |
| *** opendevmeet is now known as Guest1689 | 09:57 | |
| *** Guest1690 <Guest1690!~limnoria@104.239.144.232> has joined #openstack-ironic | 10:02 | |
| *** opendevmeet <opendevmeet!~limnoria@104.239.144.232> has joined #openstack-ironic | 10:13 | |
| iurygregory | ty =) | 10:13 |
| *** opendevmeet is now known as Guest1692 | 10:14 | |
| *** opendevmeet <opendevmeet!~limnoria@104.239.144.232> has joined #openstack-ironic | 11:15 | |
| *** Guest1712 <Guest1712!~opendevst@104.239.144.232> has quit IRC (Remote host closed the connection) | 11:17 | |
| *** Guest1713 <Guest1713!~opendevst@149.202.169.13> has quit IRC (Ping timeout: 480 seconds) | 11:18 | |
| *** opendevstatus <opendevstatus!~opendevst@104.239.144.232> has joined #openstack-ironic | 11:19 | |
| *** opendevstatus_ <opendevstatus_!~opendevst@104.130.219.52> has joined #openstack-ironic | 11:41 | |
| *** opendevstatus_ is now known as opendevstatus__ | 11:43 | |
| *** opendevstatus__ is now known as opendevstatus___ | 11:43 | |
| *** opendevstatus___ is now known as opendevstatus____ | 11:43 | |
| *** opendevstatus____ is now known as opendevstatus_____ | 11:43 | |
| *** opendevstatus_____ is now known as opendevstatus______ | 11:43 | |
| *** jelabarre-rh <jelabarre-rh!~jlabarre@2601:184:c300:2231:46fc:5644:1527:c695> has joined #openstack-ironic | 11:43 | |
| *** jelabarre-rh <jelabarre-rh!~jlabarre@2601:184:c300:2231:46fc:5644:1527:c695> has quit IRC (Remote host closed the connection) | 11:44 | |
| *** jelabarre-rh <jelabarre-rh!~jlabarre@2601:184:c300:2231:46fc:5644:1527:c695> has joined #openstack-ironic | 11:44 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 11:49 | |
| *** opendevstatus______ <opendevstatus______!~opendevst@104.130.219.52> has quit IRC (Ping timeout: 480 seconds) | 11:51 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Ping timeout: 480 seconds) | 11:57 | |
| rpittau | if anyone has a minute it would be great to have those 2 backports approved https://review.opendev.org/c/openstack/ironic-python-agent/+/795883 https://review.opendev.org/c/openstack/ironic-python-agent/+/795965 | 12:10 |
| bburns | kogito | 12:11 |
| iurygregory | dtantsur, I've pushed https://review.opendev.org/c/openstack/releases/+/795991 with multiple sushy releases =) ussuri to xena | 12:12 |
| iurygregory | rpittau, looking now | 12:13 |
| *** opendevstatus_ <opendevstatus_!~opendevst@104.130.219.164> has joined #openstack-ironic | 12:13 | |
| rpittau | thanks iurygregory :) | 12:14 |
| *** opendevstatus_ is now known as opendevstatus__ | 12:14 | |
| *** opendevstatus__ is now known as opendevstatus___ | 12:15 | |
| *** opendevstatus___ is now known as opendevstatus____ | 12:15 | |
| *** opendevstatus____ is now known as opendevstatus_____ | 12:15 | |
| *** opendevstatus_____ is now known as opendevstatus______ | 12:15 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 12:16 | |
| *** opendevstatus <opendevstatus!~opendevst@104.239.144.232> has quit IRC (Remote host closed the connection) | 12:18 | |
| *** opendevstatus <opendevstatus!~opendevst@104.239.144.232> has joined #openstack-ironic | 12:19 | |
| janders | see you next week Ironic o/ | 12:21 |
| iurygregory | bye janders enjoy the weekend | 12:22 |
| janders | you too iurygregory o/ | 12:22 |
| *** opendevstatus <opendevstatus!~opendevst@104.239.144.232> has quit IRC (Remote host closed the connection) | 12:22 | |
| *** opendevstatus______ <opendevstatus______!~opendevst@104.130.219.164> has quit IRC (Ping timeout: 480 seconds) | 12:24 | |
| *** opendevstatus <opendevstatus!~opendevst@104.239.144.232> has joined #openstack-ironic | 12:27 | |
| *** opendevstatus_ <opendevstatus_!~opendevst@213.32.72.249> has joined #openstack-ironic | 12:42 | |
| *** lmcgann <lmcgann!~lmcgann@2601:19b:b00:f350:f854:f5f2:22de:6296> has joined #openstack-ironic | 12:42 | |
| *** opendevstatus_ is now known as opendevstatus__ | 12:44 | |
| *** opendevstatus__ is now known as opendevstatus___ | 12:44 | |
| *** opendevstatus___ is now known as opendevstatus____ | 12:44 | |
| *** opendevstatus____ is now known as opendevstatus_____ | 12:44 | |
| *** opendevstatus_____ is now known as opendevstatus______ | 12:44 | |
| *** opendevstatus______ <opendevstatus______!~opendevst@213.32.72.249> has quit IRC (Ping timeout: 480 seconds) | 12:53 | |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: DNM experiment https://review.opendev.org/c/openstack/bifrost/+/796002 | 12:55 |
| *** redrobot <redrobot!~redrobot@108-84-79-198.lightspeed.snantx.sbcglobal.net> has joined #openstack-ironic | 12:59 | |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Clean up kernel params https://review.opendev.org/c/openstack/bifrost/+/796002 | 13:00 |
| TheJulia | Good Morning everyone | 13:03 |
| TheJulia | dtantsur: couldn't that be entirely overkill when we only need the two fields? | 13:03 |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Remote host closed the connection) | 13:04 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 13:04 | |
| TheJulia | and then we also need to ensure multibyte is handled properly | 13:04 |
| *** rloo <rloo!~rloo@2001:4998:ef83:17::105d> has joined #openstack-ironic | 13:04 | |
| dtantsur | TheJulia: I think using built-in facilities is always better than using string manipulations | 13:05 |
| dtantsur | dunno re multibyte, but I think you can use the module on an already open file? | 13:05 |
| TheJulia | if you can then it makes sense | 13:05 |
| dtantsur | TheJulia: https://docs.python.org/3/library/csv.html#csv.reader | 13:05 |
| TheJulia | all to read a single line *sighs* | 13:07 |
| *** tzumainn_ <tzumainn_!~tzumainn@2601:19b:881:8630:fa34:41ff:feb1:df6e> has joined #openstack-ironic | 13:07 | |
| TheJulia | I take it testing downstream with the fix is good? | 13:07 |
| dtantsur | I assume so. bfournie, derekh? | 13:07 |
| TheJulia | I mean, we know it seems to do the needful, of course, I'll need to backport down to train as well but that is easy enough to do | 13:09 |
| *** tzumainn_ is now known as tzumainn | 13:11 | |
| *** tzumainn <tzumainn!~tzumainn@2601:19b:881:8630:fa34:41ff:feb1:df6e> has quit IRC () | 13:12 | |
| *** tzumainn <tzumainn!~tzumainn@2601:19b:881:8630:fa34:41ff:feb1:df6e> has joined #openstack-ironic | 13:12 | |
| *** opendevstatus_ <opendevstatus_!~opendevst@104.130.26.53> has joined #openstack-ironic | 13:13 | |
| *** opendevstatus_ is now known as opendevstatus__ | 13:15 | |
| *** opendevstatus__ is now known as opendevstatus___ | 13:15 | |
| *** opendevstatus___ is now known as opendevstatus____ | 13:15 | |
| *** opendevstatus____ is now known as opendevstatus_____ | 13:15 | |
| *** opendevstatus_____ is now known as opendevstatus______ | 13:15 | |
| *** jnamdar <jnamdar!~oftc-webi@static-176-139-14-39.ftth.abo.bbox.fr> has joined #openstack-ironic | 13:17 | |
| *** opendevstatus______ <opendevstatus______!~opendevst@104.130.26.53> has quit IRC (Ping timeout: 480 seconds) | 13:24 | |
| opendevreview | Verification of a change to openstack/ironic failed: Refactor iDRAC OEM extension manager calls https://review.opendev.org/c/openstack/ironic/+/795275 | 13:30 |
| *** sdanni <sdanni!~sdanni@pool-98-118-100-55.bstnma.fios.verizon.net> has joined #openstack-ironic | 13:31 | |
| *** stendulker <stendulker!~stendulke@122.171.207.150> has quit IRC (Ping timeout: 480 seconds) | 13:34 | |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Refactor bootloader csv file read to use csv module https://review.opendev.org/c/openstack/ironic-python-agent/+/796012 | 13:44 |
| TheJulia | dtantsur: ^^ | 13:44 |
| TheJulia | not exactly pretty | 13:44 |
| TheJulia | .__next__() to get the first and only line :\ | 13:45 |
| iurygregory | wow | 13:45 |
| TheJulia | it returns an iterator instead of a list | 13:45 |
| iurygregory | still better than Go lol | 13:46 |
| dtantsur | I guess using next() would be a bit nicer.. and should we catch StopIteration in the unlikely event the file is empty? | 13:46 |
| dtantsur | ++ to better than Go, it only has 'for' loops for everything | 13:46 |
| TheJulia | I suspect the odds would be extremely remote | 13:46 |
| TheJulia | but perhaps? | 13:47 |
| TheJulia | oh wait | 13:47 |
| TheJulia | if it is empty, we're done. | 13:47 |
| TheJulia | we can't proceed. | 13:47 |
| iurygregory | and *boom* | 13:47 |
| dtantsur | we can still try other bootloader. or at least fail with something better than StopIteration | 13:48 |
| dtantsur | maybe we should cover the whole thing in try.. except.. and report a better error? | 13:48 |
| TheJulia | then we have to handle full lists of everything | 13:48 |
| TheJulia | not just targetted authortative files | 13:48 |
| TheJulia | and then we also have to guess | 13:48 |
| TheJulia | the way it is written, we treat a CSV file as the single source of truth | 13:49 |
| TheJulia | it is authortative, we don't care about other loaders | 13:49 |
| TheJulia | Granted, where this gets complex and we may want to consider multiple *is* if someone tries to hand something some sort of multi-arch image | 13:51 |
| TheJulia | but we need to likely sort out what should be authortative by architecture well in advance | 13:52 |
| TheJulia | (and things like fedora will cry because they do something similar for multi-os booting | 13:52 |
| opendevreview | Verification of a change to openstack/ironic failed: Refactor iDRAC OEM extension manager calls https://review.opendev.org/c/openstack/ironic/+/795275 | 13:52 |
| *** bfournie <bfournie!~bfournie@pool-108-49-98-118.bstnma.fios.verizon.net> has quit IRC (Quit: Leaving.) | 13:56 | |
| opendevreview | Merged openstack/ironic-python-agent stable/wallaby: Make _get_efi_bootloaders return relative paths https://review.opendev.org/c/openstack/ironic-python-agent/+/795883 | 13:58 |
| TheJulia | I *feel* like someone trying to do multi-os images is like a 0.25% of the problem space at this time, it will likely grow with time, which is why I headed in the keep it simple path | 13:58 |
| rpittau | one down, one to go | 13:58 |
| TheJulia | rpittau: how bad was the backport to stable/wallaby for the first patch, or was it clean? | 14:01 |
| rpittau | clean after the first patch | 14:01 |
| TheJulia | rpittau: just the list of bootloaders? | 14:02 |
| TheJulia | err, nope | 14:03 |
| rpittau | no, well, the entire patch | 14:03 |
| rpittau | but it went through nicely | 14:03 |
| rpittau | no change | 14:03 |
| *** kkillsfirst <kkillsfirst!~kkillsfir@24-119-124-196.cpe.sparklight.net> has joined #openstack-ironic | 14:03 | |
| opendevreview | Merged openstack/ironic-python-agent stable/wallaby: Utilize CSV file for EFI loader selection https://review.opendev.org/c/openstack/ironic-python-agent/+/795965 | 14:04 |
| rpittau | aaaand two | 14:04 |
| TheJulia | \o/ | 14:04 |
| iurygregory | YAY | 14:04 |
| *** bfournie <bfournie!~bfournie@pool-108-49-98-118.bstnma.fios.verizon.net> has joined #openstack-ironic | 14:05 | |
| TheJulia | hopefully these are clean going all the way back to train after this first hop | 14:05 |
| TheJulia | well, second | 14:05 |
| rpittau | oh sorry I idn't think about backporting deeper :/ | 14:06 |
| TheJulia | no worries | 14:06 |
| TheJulia | I've got to take it all the way to train | 14:06 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Reduce logging verbosity when collecting logs https://review.opendev.org/c/openstack/ironic-python-agent/+/796015 | 14:06 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Make _get_efi_bootloaders return relative paths https://review.opendev.org/c/openstack/ironic-python-agent/+/796016 | 14:06 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/victoria: Utilize CSV file for EFI loader selection https://review.opendev.org/c/openstack/ironic-python-agent/+/796017 | 14:07 |
| *** Zainub_Wahid <Zainub_Wahid!~Zainub_Wa@202.47.34.153> has quit IRC (Read error: Connection reset by peer) | 14:07 | |
| TheJulia | luckilly, we've done lots of backporting in this section of the code | 14:08 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Make _get_efi_bootloaders return relative paths https://review.opendev.org/c/openstack/ironic-python-agent/+/796021 | 14:14 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/ussuri: Utilize CSV file for EFI loader selection https://review.opendev.org/c/openstack/ironic-python-agent/+/796022 | 14:14 |
| TheJulia | arne_wiebalck: you around? | 14:17 |
| arne_wiebalck | TheJulia: yes :) | 14:18 |
| TheJulia | have you had any luck with software raid and centos8-stream? | 14:18 |
| arne_wiebalck | Yes, it works for us. | 14:18 |
| arne_wiebalck | It was never broken. | 14:18 |
| arne_wiebalck | Well. | 14:19 |
| arne_wiebalck | It was never broken because of cs8. | 14:19 |
| arne_wiebalck | It did not work b/c of our image which did not include mdadm in the initrd. | 14:19 |
| TheJulia | okay | 14:20 |
| TheJulia | weird | 14:20 |
| arne_wiebalck | Once we stopped ignoring dracut errors, it worked. | 14:20 |
| TheJulia | we're seeing grub2-probe failures when attempting to generate configuration | 14:20 |
| arne_wiebalck | I may see them as well, but I am not sure they are fatal. | 14:21 |
| arne_wiebalck | That is, I have seen them during debugging, but then I would not trust my debugging. | 14:22 |
| arne_wiebalck | I am creating an instance right now, will check what the logs say ... | 14:22 |
| TheJulia | arne_wiebalck: much appreciated | 14:34 |
| * TheJulia tries to remember the trick to mocking file open on py2 | 14:34 | |
| arne_wiebalck | TheJulia: it seems to work, i.e. it boots (this is on Victoria (IPA on Ussuri), Ironic s/w RAID, RAIDed ESPs and a cs8 image) | 14:36 |
| arne_wiebalck | TheJulia: there are some errors in the logs (boot loader installation fails), but I will move to a Victoria IPA before looking closer | 14:37 |
| TheJulia | hmm | 14:38 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/train: Make _get_efi_bootloaders return relative paths https://review.opendev.org/c/openstack/ironic-python-agent/+/796030 | 14:54 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent stable/train: Utilize CSV file for EFI loader selection https://review.opendev.org/c/openstack/ironic-python-agent/+/796031 | 14:55 |
| TheJulia | arne_wiebalck: stock centos stream images? | 14:55 |
| TheJulia | or modified | 14:55 |
| TheJulia | ? | 14:55 |
| arne_wiebalck | TheJulia: these are cs8 images we build ourselves | 14:56 |
| derekh | TheJulia: sorry missed the ping, Yup, seems ok on the env I checked it on, Re: <TheJulia> I take it testing downstream with the fix is good? | 14:57 |
| TheJulia | arne_wiebalck: ahh! | 14:57 |
| TheJulia | derekh: awesome news | 14:58 |
| arne_wiebalck | TheJulia: I get the same errors with a Victoria IPA. Installing the bootloader fails (!) since we try to mount the already mounted EFI partition. The node boots none the less :) | 15:08 |
| arne_wiebalck | TheJulia: The mount error *may* be related to the mirrored ESPs. | 15:08 |
| arne_wiebalck | TheJulia: I will need to add a "if not mounted" to the image to see if the bootloader installation would then work. | 15:09 |
| JayF | If someone wants to land a thing this morning, https://review.opendev.org/c/openstack/ironic/+/787512 is getting a little dusty and already has a +2 from me | 15:18 |
| opendevreview | Merged openstack/ironic-python-agent master: Remove lower-constraints job https://review.opendev.org/c/openstack/ironic-python-agent/+/795942 | 15:20 |
| NobodyCam | Good Morning Ironic folks... | 15:28 |
| NobodyCam | and OFC.... | 15:28 |
| NobodyCam | TGIF! | 15:28 |
| dtantsur | \o/ | 15:28 |
| NobodyCam | good morning dtantsur | 15:28 |
| dtantsur | morning :) | 15:28 |
| iurygregory | good morning NobodyCam | 15:28 |
| NobodyCam | morning iurygregory | 15:29 |
| opendevreview | Merged openstack/ironic master: Update Redfish RAID disk_type unit test https://review.opendev.org/c/openstack/ironic/+/795504 | 15:38 |
| opendevreview | Merged openstack/ironic master: Update CI jobs list and description in docs https://review.opendev.org/c/openstack/ironic/+/787512 | 15:44 |
| *** Qianbiao <Qianbiao!~Qianbiao@94.74.82.60> has quit IRC (Remote host closed the connection) | 15:46 | |
| rpittau | have a great weekend! o/ | 15:46 |
| *** rpittau is now known as rpittau|afk | 15:47 | |
| iurygregory | bye rpittau|afk o/ | 15:47 |
| TheJulia | arne_wiebalck: if I remember correctly, not dib based right? | 15:53 |
| TheJulia | good morning NobodyCam | 15:54 |
| NobodyCam | Good Morning TheJulia :) Happy Friday | 15:54 |
| *** lucasagomes <lucasagomes!~lucasagom@89.100.20.18> has quit IRC (Quit: Leaving) | 15:56 | |
| *** stendulker <stendulker!~stendulke@122.171.34.148> has joined #openstack-ironic | 16:02 | |
| arne_wiebalck | TheJulia: correct | 16:04 |
| arne_wiebalck | TheJulia: it is a snapshot of a kickstart based virtual machine | 16:04 |
| arne_wiebalck | TheJulia: I added the 'if', and now it reports the expected "'grub2-install: error: this utility cannot be used for EFI platforms because it does not support UEFI Secure Boot." | 16:09 |
| arne_wiebalck | TheJulia: The node boots, though. | 16:11 |
| arne_wiebalck | TheJulia: Seems like our images do not require grub2-install at all :) | 16:13 |
| TheJulia | oh goodie :) | 16:16 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Only mount the ESP if not yet mounted https://review.opendev.org/c/openstack/ironic-python-agent/+/796045 | 16:23 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Restrict publishing jobs to master https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/790416 | 16:25 |
| arne_wiebalck | TheJulia: ^^ makes our image run grub2-install (and fail with the expected secure boot failure). The additional check should not harm anyone else. I think the ESP is mounted in our case as there is an ESP line in fstab in our image alread (which may not be the case in general). | 16:32 |
| opendevreview | Merged openstack/python-ironicclient master: Add missing unit tests for provision state commands https://review.opendev.org/c/openstack/python-ironicclient/+/377607 | 16:39 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent-builder stable/wallaby: Restrict publishing jobs to master https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/795891 | 16:43 |
| dtantsur | have a great weekend folks! | 16:59 |
| *** derekh <derekh!~derekh@2a01:b340:80:6d6e:6423:15e0:3703:2> has quit IRC (Quit: Leaving) | 17:00 | |
| JayF | spuc? | 17:01 |
| dtantsur | JayF: I can join for 15 mins | 17:02 |
| dtantsur | if folks are up for it. TheJulia? | 17:02 |
| JayF | your choice :) I'm the only one in right now lol | 17:02 |
| TheJulia | oh sure! | 17:02 |
| TheJulia | link? | 17:02 |
| JayF | https://redhat.bluejeans.com/250125662 | 17:03 |
| *** stendulker <stendulker!~stendulke@122.171.34.148> has quit IRC (Ping timeout: 480 seconds) | 17:17 | |
| TheJulia | https://review.opendev.org/c/openstack/ironic-python-agent/+/796031 and https://review.opendev.org/c/openstack/ironic-python-agent/+/796032 | 17:30 |
| TheJulia | Both passed tests \o/ | 17:31 |
| TheJulia | Those are train specifically | 17:31 |
| TheJulia | kkillsfirst: is it https://docs.openstack.org/ironic/latest/user/deploy.html#image-information? | 17:46 |
| -opendevstatus- NOTICE: Zuul is being restarted for server reboots | 17:47 | |
| arne_wiebalck | bye everyone, see you next week o/ | 17:53 |
| *** kkillsfirst <kkillsfirst!~kkillsfir@24-119-124-196.cpe.sparklight.net> has quit IRC (Ping timeout: 480 seconds) | 18:37 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Remote host closed the connection) | 19:17 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 19:17 | |
| *** kkillsfirst <kkillsfirst!~kkillsfir@24-119-124-196.cpe.sparklight.net> has joined #openstack-ironic | 19:25 | |
| *** zaneb <zaneb!~zaneb@0001a018.user.oftc.net> has quit IRC (Remote host closed the connection) | 20:00 | |
| *** zaneb <zaneb!~zaneb@2600:1005:b021:6d28:4d4e:e8b4:3301:a4fb> has joined #openstack-ironic | 20:01 | |
| *** donnyd <donnyd!sid368272@id-368272.tooting.irccloud.com> has joined #openstack-ironic | 20:54 | |
| *** donnyd <donnyd!sid368272@id-368272.tooting.irccloud.com> has quit IRC () | 21:06 | |
| *** donnyd <donnyd!sid368272@id-368272.tooting.irccloud.com> has joined #openstack-ironic | 21:06 | |
| *** sdanni <sdanni!~sdanni@pool-98-118-100-55.bstnma.fios.verizon.net> has quit IRC (Ping timeout: 480 seconds) | 21:17 | |
| *** zaneb <zaneb!~zaneb@0001a018.user.oftc.net> has quit IRC (Ping timeout: 480 seconds) | 21:33 | |
| *** lmcgann <lmcgann!~lmcgann@2601:19b:b00:f350:f854:f5f2:22de:6296> has quit IRC (Ping timeout: 480 seconds) | 21:35 | |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP/DNM: Remove md check on EFI asset preservation https://review.opendev.org/c/openstack/ironic-python-agent/+/796068 | 21:42 |
| TheJulia | stevebaker: ^^ w/r/t raid5 | 21:46 |
| TheJulia | err | 21:46 |
| TheJulia | raid1 | 21:46 |
| *** rloo <rloo!~rloo@2001:4998:ef83:17::105d> has quit IRC (Quit: Leaving...) | 21:56 | |
| *** tosky <tosky!~luigi@dynamic-adsl-78-13-253-141.clienti.tiscali.it> has quit IRC () | 22:13 | |
| *** Guest1612 <Guest1612!~danielk@2600:1700:ff20:48c0::47> has quit IRC (Quit: Leaving.) | 22:39 | |
| *** kkillsfirst <kkillsfirst!~kkillsfir@24-119-124-196.cpe.sparklight.net> has quit IRC (Quit: Leaving) | 23:08 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Remote host closed the connection) | 23:34 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has joined #openstack-ironic | 23:38 | |
| *** paras333 <paras333!~paras333@209.6.68.100> has quit IRC (Ping timeout: 480 seconds) | 23:46 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!