rpittau | good morning ironic! o/ | 06:28 |
---|---|---|
opendevreview | Dmitry Tantsur proposed openstack/metalsmith master: Deprecate instance network boot https://review.opendev.org/c/openstack/metalsmith/+/854239 | 07:41 |
kubajj | good morning everybody! | 08:03 |
opendevreview | Dmitry Tantsur proposed openstack/metalsmith master: Deprecate instance network boot https://review.opendev.org/c/openstack/metalsmith/+/854239 | 08:53 |
iurygregory | good morning Ironic | 10:54 |
*** janders2 is now known as janders | 11:06 | |
*** kubajj_ is now known as kubajj | 11:06 | |
*** buhman_ is now known as buhman | 11:06 | |
*** MahnoorAsghar_ is now known as MahnoorAsghar | 11:06 | |
*** jungleboyj_ is now known as jungleboyj | 11:06 | |
*** seongsoocho_ is now known as seongsoocho | 11:06 | |
*** rpittau_ is now known as rpittau | 11:06 | |
*** vkmc_ is now known as vkmc | 11:06 | |
*** TheJulia_ is now known as TheJulia | 11:06 | |
opendevreview | Dmitry Tantsur proposed openstack/metalsmith master: Deprecate instance network boot https://review.opendev.org/c/openstack/metalsmith/+/854239 | 11:19 |
*** arne_wiebalck_ is now known as arne_wiebalck | 11:24 | |
opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: WIP: SoftwareRAID: Enable skipping RAIDS https://review.opendev.org/c/openstack/ironic-python-agent/+/852999 | 11:59 |
kubajj | The above change is the first prototype of the skipping RAIDs. I am not including tests yet as reviews might change the code a lot. In the first version, we decided not to check that the logical disks on the skip list are actually present. They are just not created. Therefore, skip list should not be defined the first time one creates the RAIDs. | 12:05 |
kubajj | TheJulia: dtantsur: JayF: If any of you have some minute to check that this is heading in the right direction, feedback would be very appreciated. | 12:14 |
opendevreview | wangjiaqi proposed openstack/virtualbmc master: remove unicode from code https://review.opendev.org/c/openstack/virtualbmc/+/854328 | 12:29 |
TheJulia | good morning | 13:19 |
rpittau | good morning TheJulia :) | 13:27 |
iurygregory | green \o/ https://review.opendev.org/c/openstack/metalsmith/+/854239 | 13:37 |
TheJulia | \o/ | 13:38 |
iurygregory | if someone can look at https://review.opendev.org/c/openstack/ironic-lib/+/844666 | 13:38 |
TheJulia | I just approved it as a non-code change | 13:44 |
TheJulia | kubajj: have you added me as a reviewer to the patch(es)? | 13:44 |
kubajj | TheJulia: I didn't know that I should. | 13:45 |
TheJulia | kubajj: I have an absurd number of patches in my gerrit UI, so it helps me, and the case is similar for many of us | 13:46 |
kubajj | Ok, will add you. Should I add Jay and Dmitry too? | 13:46 |
dtantsur | I don't use gerrit notifications, relying on gertty instead | 13:48 |
dtantsur | this week I'm a bit overloaded with metal3 work, so only limited review throughput | 13:48 |
kubajj | dtantsur: Ok, I won't add you then | 13:49 |
kubajj | TheJulia: I saw that you were solving some issues with tempest. Could you please help me with the failing test on this change? https://review.opendev.org/c/openstack/ironic-python-agent/+/853182 I am a bit lost in what it is actually testing. | 13:52 |
*** mnasiadka_ is now known as mnasiadka | 14:25 | |
*** dansmith_ is now known as dansmith | 14:29 | |
opendevreview | Merged openstack/ironic-lib master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/ironic-lib/+/844666 | 15:15 |
opendevreview | Merged openstack/ironic-lib master: remove unicode prefix from code https://review.opendev.org/c/openstack/ironic-lib/+/854151 | 15:15 |
TheJulia | kubajj: so it timed out | 15:39 |
TheJulia | at least, the deploy timed out and tempest detected it and failed the test | 15:39 |
TheJulia | kubajj: look at https://32e1d254b0f6aec7db24-0dc364dd82ce235e57ab66b60f625d93.ssl.cf1.rackcdn.com/853182/3/check/ironic-standalone-ipa-src/f3dde69/controller/logs/screen-ir-cond.txt | 15:40 |
TheJulia | search for "Aug 17 16:10:34.592478" | 15:40 |
TheJulia | hmmmm | 15:42 |
TheJulia | https://32e1d254b0f6aec7db24-0dc364dd82ce235e57ab66b60f625d93.ssl.cf1.rackcdn.com/853182/3/check/ironic-standalone-ipa-src/f3dde69/controller/logs/ironic-bm-logs/node-2_no_ansi_2022-08-17-16%3A10%3A40_log.txt | 15:44 |
TheJulia | "2022-08-17 16:10:04.386 695" and the second after | 15:45 |
rpittau | bye everyone, see you tomorrow o/ | 15:48 |
opendevreview | Merged openstack/metalsmith master: Deprecate instance network boot https://review.opendev.org/c/openstack/metalsmith/+/854239 | 16:00 |
TheJulia | o/ | 16:05 |
TheJulia | umm... hmmm | 16:23 |
TheJulia | kubajj: I think it might be unrelated... | 16:32 |
TheJulia | to your patch that is. | 16:32 |
TheJulia | working on trying to pin it down | 16:32 |
TheJulia | yeah, this looks unrelated | 16:34 |
TheJulia | tl;dr the output of mdadm has changed | 16:35 |
TheJulia | https://www.irccloud.com/pastebin/IAMVXkbh/ | 16:39 |
TheJulia | JayF: You mentioned /dev/md stuffs being broken being mentioned in gentoo recently right? | 16:52 |
JayF | something about RAID names being unsupported | 16:52 |
JayF | Everyone in the channel I'm asking about this in thinks I'm crazy | 17:02 |
JayF | so I'm going to just assume I'm having daydreams about linux software now | 17:02 |
TheJulia | eh, it is life? ;) | 17:03 |
opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Fix software raid output poisoning https://review.opendev.org/c/openstack/ironic-python-agent/+/854437 | 17:15 |
TheJulia | kubajj: I think ^ should fix things for you | 17:15 |
JayF | looking | 17:18 |
JayF | +2 | 17:19 |
TheJulia | that was a bit of a pain to trace down, but it broke my brain looking at the actual output from mdadm | 17:21 |
TheJulia | I suspect we should backport it, but maybe just as a general thing to wallaby. | 17:39 |
JayF | I'll put it on my list | 17:42 |
JayF | do we cut bugfix releases for IPA? | 17:42 |
JayF | yes we do, don't we? it's on the whiteboard I think | 17:42 |
TheJulia | yeah we do | 17:43 |
TheJulia | I looked at the nearby areas of code and most of it was last touched a long time ago so I wouldn't expect any merge conflicts | 17:44 |
JayF | yeah | 17:44 |
JayF | generally I'm taking things back to train | 17:44 |
JayF | unless there's a good reason otherwise | 17:44 |
JayF | btw, lots of progress on those nova patches too, I think I have everything backported at least to victoria now | 17:44 |
TheJulia | I'm really not supporting much software raid wise back on train, fwiw | 17:44 |
TheJulia | I *really* hope the user survey has version details which line up with what the operators indicated in Berlin this year | 17:46 |
stevebaker[m] | good morning | 21:12 |
opendevreview | Jay Faulkner proposed openstack/sushy stable/wallaby: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/854454 | 21:42 |
opendevreview | Jay Faulkner proposed openstack/sushy stable/yoga: Capture requests errors https://review.opendev.org/c/openstack/sushy/+/854456 | 21:43 |
TheJulia | o/ stevebaker[m] | 23:41 |
stevebaker[m] | o hey | 23:42 |
TheJulia | did you revise the second dib patch? | 23:42 |
stevebaker[m] | TheJulia: this one? yes its ready for review I think https://review.opendev.org/c/openstack/diskimage-builder/+/840144 | 23:43 |
TheJulia | stevebaker[m]: ugh: https://review.opendev.org/c/openstack/diskimage-builder/+/854427 | 23:50 |
TheJulia | looks like it is in merge conflict with the one I just approved | 23:52 |
TheJulia | but, it LGTM | 23:52 |
stevebaker[m] | oh, nifty fix mgoddard | 23:53 |
TheJulia | ++ | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!