opendevreview | Julia Kreger proposed openstack/ironic master: httpboot enabled CI stuffs https://review.opendev.org/c/openstack/ironic/+/908955 | 00:01 |
---|---|---|
opendevreview | Julia Kreger proposed openstack/ironic master: ci: fix ipmi partition job name https://review.opendev.org/c/openstack/ironic/+/908956 | 00:07 |
NobodyCam | good afternoon Ironic folks. | 00:10 |
NobodyCam | did the poweroff_in_cleanfail config option get removed? | 00:11 |
TheJulia | ummm | 00:33 |
TheJulia | wasn't that a patch you proposed | 00:33 |
opendevreview | Julia Kreger proposed openstack/ironic master: ci: phase out ironic-standalone CI job https://review.opendev.org/c/openstack/ironic/+/908959 | 00:33 |
TheJulia | hmm | 00:34 |
TheJulia | no, we have tests for it | 00:34 |
TheJulia | dated 2023-04-12 | 00:35 |
TheJulia | If your looking for it in antelope, that was released March 2023 | 00:36 |
opendevreview | Merged openstack/python-ironicclient master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/python-ironicclient/+/906077 | 01:36 |
janders | Hi Folks. I am working on a simple sushy-tools patch. I ran the tests on master to make sure I am set up to work on this repo and hit this cosmetic linter issue: https://paste.openstack.org/show/bjNhflaizOnb6HNjFAML/ | 04:23 |
janders | It seems the line in question was introduced by this change which is 3 years old: https://opendev.org/openstack/sushy-tools/commit/0051bd4be05ee11f8033a744a04c106a24045518 | 04:24 |
janders | I can submit a trivial one liner fix to this but I wonder if it's an actual problem, or is my config too picky? | 04:24 |
janders | it seems unlikely no one else noticed this for an extended period and more likely that it's specific to my environment hence asking before proposing a mini "patch" | 04:46 |
TheJulia | janders: patches are always welcome | 04:52 |
janders | TheJulia ACK | 04:53 |
janders | :) | 04:53 |
rpittau | good morning ironic! o/ | 07:46 |
rpittau | cid: thank you, I left some comments in your PRs | 10:12 |
cid | rpittau: nice, seen and updated accordingly. | 10:21 |
iurygregory | good morning Ironic | 11:04 |
cid | thanks to rpittau and masghar for the PR reviews :), really appreciate it. Can we merge these PRs or further reviews? I don't really know how that works, but excited to build on them for the next ones. | 11:06 |
iurygregory | janders, I can reproduce the issue you mentioned locally (with also a lot of warnings) | 11:16 |
masghar | cid: I think it would be wise to get dtantsur's review and approval as well (he is on vacation this week though) | 11:40 |
cid | masghar, ohkay, surely. | 11:44 |
janders | iurygregory ACK, TY | 11:58 |
janders | will put a trivial patch up tomo | 11:58 |
iurygregory | tks! | 12:07 |
arne_wiebalck | Good morning, Ironic! | 13:44 |
arne_wiebalck | We just crossed 10'000 nodes in Ironic at CERN! | 13:45 |
iurygregory | WOW! | 13:45 |
iurygregory | awesome arne_wiebalck \o/ | 13:46 |
masghar | That is amazing :O \o/ | 13:50 |
rpittau | arne_wiebalck: fantastic! :) | 14:08 |
JayF | \o/ | 14:48 |
JayF | What a nice first thing to read as I open IRC! | 14:49 |
JayF | right after reading all the nice reviews cid is getting; thanks for looking over those especially since I am not around to do it this week o/ | 14:49 |
iurygregory | TheJulia, rpittau bug meeting? | 15:01 |
iurygregory | masghar, also ^ | 15:02 |
masghar | I'm free for it | 15:02 |
TheJulia | I can join in a few minutes | 15:04 |
TheJulia | I had a distraction come up and need to recover from that before 8 AM local | 15:04 |
iurygregory | ack | 15:05 |
iurygregory | when you are free let us know and we can join | 15:05 |
masghar | yep ^ | 15:05 |
iurygregory | I won't be able to stay 2hr, only 1hr (today we have a half holiday in Brazil, so I'm only working till 16 UTC, and it will be lunch time also lol) | 15:06 |
TheJulia | 5 minutes, making coffee | 15:07 |
TheJulia | I have a hard stop at 5 till 8 am local | 15:07 |
TheJulia | which is in like... 40 minutes | 15:07 |
TheJulia | iurygregory: got a meeting invite? | 15:11 |
rpittau | iurygregory: I can now | 15:11 |
iurygregory | TheJulia, I don't, would be better if you or rpittau create one (since I will leave, not sure if google would keep open or not) | 15:12 |
TheJulia | uhhhh | 15:12 |
TheJulia | give me a moment | 15:13 |
TheJulia | firing up my zoom acct | 15:13 |
iurygregory | ack | 15:13 |
TheJulia | https://us02web.zoom.us/j/5342999208 | 15:13 |
iurygregory | really zoom, you ask me to update the version now? lol | 15:14 |
* iurygregory joins via browser | 15:14 | |
TheJulia | heh | 15:15 |
TheJulia | happens a lot | 15:15 |
TheJulia | I fired up zoom because I could hand over host and leave it :) | 15:16 |
rpittau | "the host has another meeting in progress" | 15:16 |
TheJulia | when the time comes if people kept wanting to look since that was the gogole issue | 15:16 |
TheJulia | one moment | 15:16 |
TheJulia | try now | 15:16 |
iurygregory | Waiting for the host to start the meeting. | 15:16 |
iurygregory | let me try to refresh | 15:16 |
masghar | host has joined, we've let them know you're here | 15:18 |
iurygregory | yeah, same for me | 15:18 |
TheJulia | riccardo and I are here, I'm not seeing anything :( | 15:18 |
masghar | interesting | 15:19 |
iurygregory | weird .-. | 15:19 |
iurygregory | maybe because I'm trying from browser? | 15:20 |
masghar | (same) | 15:20 |
TheJulia | https://us02web.zoom.us/j/5342999208?pwd=Sm1tZk5HOFVSSnRIalNnbmdiRU1BUT09 | 15:20 |
TheJulia | try that | 15:20 |
masghar | works | 15:21 |
iurygregory | I think this will count as the bug deputy for this week lol =P | 15:57 |
rpittau | that was fun :D | 15:59 |
JayF | https://bugs.launchpad.net/ironic/+bug/1526561 I'm very proud of 8-year-ago me. | 16:20 |
JayF | Usually I see something that old and cringe, but that was a good bug! | 16:20 |
iurygregory | Bye everyone! see you on Feb 29 =) | 16:23 |
rpittau | bye iurygregory o/ | 16:26 |
JayF | o/ | 16:28 |
rpittau | good night! o/ | 17:00 |
JayF | Just a reminder: nomination window will be opening soonish for TC/PTL nominations. I do not intend to run for another year as Ironic PTL. I am happy to help anyone who wants to nominate themselves to be the next PTL. | 17:09 |
*** jph4 is now known as jph | 18:32 | |
opendevreview | Merged openstack/ironic-lib master: Remove compatibility with Python < 3.8 https://review.opendev.org/c/openstack/ironic-lib/+/905811 | 20:31 |
opendevreview | Julia Kreger proposed openstack/ironic master: httpboot enabled CI stuffs https://review.opendev.org/c/openstack/ironic/+/908955 | 22:46 |
opendevreview | Julia Kreger proposed openstack/ironic master: ci: fix ipmi partition job name https://review.opendev.org/c/openstack/ironic/+/908956 | 22:47 |
opendevreview | Julia Kreger proposed openstack/ironic master: ci: phase out ironic-standalone CI job https://review.opendev.org/c/openstack/ironic/+/908959 | 23:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!