*** altlogbot_3 has joined #openstack-ironic | 00:13 | |
*** altlogbot_3 has left #openstack-ironic | 00:13 | |
*** nfedorov has quit IRC | 01:40 | |
*** zaneb has joined #openstack-ironic | 02:28 | |
*** zaneb has quit IRC | 02:46 | |
*** paras333 has joined #openstack-ironic | 03:04 | |
*** paras333 has quit IRC | 03:17 | |
*** mgoddard has quit IRC | 03:33 | |
*** mgoddard has joined #openstack-ironic | 03:33 | |
*** paras333 has joined #openstack-ironic | 03:44 | |
*** zaneb has joined #openstack-ironic | 03:47 | |
*** zaneb has quit IRC | 03:58 | |
*** paras333 has quit IRC | 03:59 | |
*** paras333 has joined #openstack-ironic | 05:56 | |
*** paras333 has quit IRC | 06:04 | |
yoctozepto | morning | 06:17 |
---|---|---|
iurygregory | good morning janders yoctozepto and Ironic o/ | 06:43 |
yoctozepto | o/ iurygregory | 06:46 |
arne_wiebalck | Good morning yoctozepto iurygregory and Ironic! | 06:55 |
yoctozepto | o/ arne_wiebalck | 06:56 |
* yoctozepto needs to learn this hello line | 06:56 | |
janders | hey yoctozepto iurygregory arne_wiebalck and Ironic o/ | 06:57 |
arne_wiebalck | Good morning janders o/ | 06:57 |
yoctozepto | o/ janders | 06:57 |
* yoctozepto going for hello-full-mesh | 06:57 | |
iurygregory | hey arne_wiebalck o/ | 07:09 |
*** rpittau|afk is now known as rpittau | 07:26 | |
rpittau | good morning ironic! o/ | 07:26 |
yoctozepto | o/ rpittau | 07:31 |
rpittau | hey yoctozepto :) | 07:31 |
iurygregory | morning rpittau o/ | 07:37 |
rpittau | hey iurygregory :) | 07:38 |
opendevreview | Radosław Piliszek proposed openstack/python-ironic-inspector-client master: [doc] Fix lldp_basic name https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/794782 | 07:40 |
*** paras333 has joined #openstack-ironic | 08:02 | |
*** paras333 has quit IRC | 08:10 | |
janders | hey rpittau o/ | 08:35 |
*** sshnaidm|afk is now known as sshnaidm | 08:51 | |
* dtantsur rubs eyes | 08:57 | |
opendevreview | Merged openstack/python-ironic-inspector-client master: Expose more interface fields https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/794843 | 08:58 |
*** sshnaidm has quit IRC | 08:58 | |
*** sshnaidm has joined #openstack-ironic | 08:59 | |
rpittau | hey janders :) | 09:08 |
rpittau | morning dtantsur :) | 09:08 |
yoctozepto | Ironic cores, do you normally accept backports of simple life-improving enhancements like the one in https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/794843 ? | 09:18 |
rpittau | yoctozepto: technically that's a feature, so I'm afraid not | 09:20 |
yoctozepto | rpittau: ok | 09:21 |
opendevreview | Aija Jauntēva proposed openstack/ironic master: Redfish: Get only RAID controller's physical disks https://review.opendev.org/c/openstack/ironic/+/795074 | 09:37 |
opendevreview | Merged openstack/ironic-python-agent master: Ignore efi grub2-install failure https://review.opendev.org/c/openstack/ironic-python-agent/+/794363 | 09:47 |
*** paras333 has joined #openstack-ironic | 10:08 | |
*** paras333 has quit IRC | 10:16 | |
arne_wiebalck | bye everyone, see you tomorrow o/ | 11:25 |
yoctozepto | bye arne_wiebalck | 11:26 |
janders | see you tomorrow Ironic o/ | 11:35 |
*** Luzi has joined #openstack-ironic | 11:42 | |
*** Luzi has quit IRC | 11:51 | |
*** tosky has joined #openstack-ironic | 11:59 | |
*** paras333 has joined #openstack-ironic | 12:14 | |
*** paras333 has quit IRC | 12:22 | |
opendevreview | Merged openstack/ironic stable/wallaby: Avoid unnecessary validation in boot interfaces https://review.opendev.org/c/openstack/ironic/+/790441 | 12:37 |
*** zaneb has joined #openstack-ironic | 12:39 | |
*** Qianbiao has joined #openstack-ironic | 12:41 | |
Qianbiao | hello ironic | 12:42 |
rpittau | hey Qianbiao :) | 12:45 |
opendevreview | Merged openstack/ironic-python-agent master: Burn-in: Add disk step https://review.opendev.org/c/openstack/ironic-python-agent/+/792644 | 12:46 |
Qianbiao | hey rpittau o/ | 12:47 |
Qianbiao | happy fresh week | 12:47 |
rpittau | thanks! | 12:48 |
*** lmcgann has joined #openstack-ironic | 12:49 | |
dtantsur | wait, it's Monday again? | 12:50 |
*** Luzi has joined #openstack-ironic | 12:52 | |
*** paras333 has joined #openstack-ironic | 12:56 | |
dtantsur | mgoddard: FYI https://review.opendev.org/c/openstack/bifrost/+/794177/ is marked as WIP and won't merge until it's changed | 13:05 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-lib stable/wallaby: utils.execute: log stdout and stderr even on failure https://review.opendev.org/c/openstack/ironic-lib/+/795045 | 13:06 |
opendevreview | Dmitry Tantsur proposed openstack/ironic-lib stable/victoria: utils.execute: log stdout and stderr even on failure https://review.opendev.org/c/openstack/ironic-lib/+/795046 | 13:06 |
*** tzumainn has joined #openstack-ironic | 13:09 | |
*** Mahnoor_Asghar has joined #openstack-ironic | 13:10 | |
*** rloo has joined #openstack-ironic | 13:14 | |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Reasonable error message if network_interface is not found https://review.opendev.org/c/openstack/bifrost/+/795106 | 13:17 |
TheJulia | dtantsur: yes :( | 13:19 |
TheJulia | iurygregory: you going to be on for review jam? | 13:19 |
dtantsur | le sigh | 13:19 |
dtantsur | good morning TheJulia | 13:19 |
TheJulia | good morning dtantsur | 13:19 |
iurygregory | TheJulia, today I probably won't but tomorrow I can join | 13:20 |
iurygregory | good morning TheJulia =) | 13:20 |
TheJulia | iurygregory: ack | 13:20 |
rpittau | good morning TheJulia :) | 13:21 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Reasonable error message if network_interface is not found https://review.opendev.org/c/openstack/bifrost/+/795106 | 13:21 |
dtantsur | okay, hopefully now people won't get confused ^^^ | 13:21 |
TheJulia | Seems quiet in here today, did everyone get coffee? | 13:25 |
mgoddard | ah, thanks dtantsur | 13:26 |
* rpittau will get the second coffee i n3 minutes | 13:27 | |
dtantsur | TheJulia: does sushi count? | 13:27 |
*** sdanni has joined #openstack-ironic | 13:29 | |
*** nfedorov has joined #openstack-ironic | 13:32 | |
TheJulia | sushi for breakfast? | 13:50 |
TheJulia | And our library name has forever ruined my brain's spelling of sushi. ;) | 13:50 |
rpittau | it is indeed good | 13:50 |
*** Luzi has quit IRC | 13:51 | |
TheJulia | dtantsur: did you see the graph I posted from Friday? | 13:52 |
TheJulia | buhman: review some of my recent changes, I need to fix the topic though | 13:52 |
*** tzumainn has quit IRC | 13:57 | |
dtantsur | TheJulia: I was out on Friday, so likely no | 14:00 |
dtantsur | and yes, sushi for breakfast! | 14:00 |
dtantsur | (because we cooked so much yesterday that we had 2 good portions left for today) | 14:00 |
TheJulia | sushi++ | 14:00 |
TheJulia | https://usercontent.irccloud-cdn.com/file/bCKTykY0/node-performance-all-patches.png | 14:00 |
TheJulia | Review jam in progress https://meetpad.opendev.org/ironic | 14:01 |
dtantsur | very nice! | 14:01 |
dtantsur | I'll need to skip both review and the meeting today, sorry | 14:01 |
dtantsur | suspecting I was the one to run it, could someone run it instead? | 14:01 |
dtantsur | (I can hopefully do it next Monday then) | 14:01 |
TheJulia | I can run it | 14:01 |
dtantsur | thx! | 14:01 |
TheJulia | no problem | 14:02 |
* dtantsur back in a few hours | 14:02 | |
*** kkillsfirst has joined #openstack-ironic | 14:02 | |
TheJulia | dtantsur: keep in mind that is with keeping a lot of the internal object work in place, *but* overhead wise it is not that much with what is left, the bulk is santization that remains as you can see with the final patch. It also shifts some of the overhead to upfront instead of constant stream of 1000+ queries to do the list. :( | 14:03 |
TheJulia | so the DBs should be *much* happier | 14:03 |
TheJulia | at least, in theory | 14:03 |
rpittau | I think it was me running the meeting today | 14:05 |
*** tzumainn has joined #openstack-ironic | 14:05 | |
opendevreview | Merged openstack/bifrost stable/victoria: Install DIB dependencies in bifrost-ironic-install role https://review.opendev.org/c/openstack/bifrost/+/794177 | 14:06 |
opendevreview | Merged openstack/ironic master: Move provision states documentation to the user guide https://review.opendev.org/c/openstack/ironic/+/793946 | 14:08 |
TheJulia | rpittau: well, if your able/willing ? I can run it | 14:25 |
rpittau | TheJulia: yeah, I will, I volunteered last time :) | 14:26 |
*** Mahnoor_Asghar has quit IRC | 14:34 | |
TheJulia | okay | 14:48 |
*** stendulker has joined #openstack-ironic | 14:59 | |
rpittau | #startmeeting ironic | 15:00 |
opendevmeet | Meeting started Mon Jun 7 15:00:13 2021 UTC and is due to finish in 60 minutes. The chair is rpittau. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
iurygregory | o/ | 15:00 |
opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
TheJulia | o/ | 15:00 |
erbarr | o/ | 15:00 |
stendulker | o/ | 15:00 |
rpittau | Hello everyone! | 15:00 |
ajya | o/ | 15:01 |
rpioso | o/ | 15:01 |
TheJulia | Greetings | 15:01 |
rpittau | Do we actually have the quorum? | 15:02 |
TheJulia | uhh... that is a good question | 15:02 |
TheJulia | 7 counting yourself | 15:02 |
rpittau | :/ | 15:02 |
TheJulia | 8 | 15:02 |
TheJulia | so technically yes :\ | 15:02 |
TheJulia | but it is borderline | 15:02 |
*** zbitter has joined #openstack-ironic | 15:02 | |
TheJulia | well, zbitter appearing might make that 9 attendees | 15:02 |
iurygregory | lol | 15:03 |
rpittau | :) | 15:03 |
rpittau | alright, let's do it! | 15:03 |
rloo | o/ | 15:03 |
TheJulia | \o rloo | 15:03 |
rpittau | rloo to the rescue | 15:03 |
rpittau | #topic Announcements / Reminders | 15:03 |
rloo | (sorry, doorbell rang...) | 15:03 |
rloo | i mean, that's why i'm a bit late :) | 15:03 |
rpittau | #info Xena-2 is on 15 July | 15:03 |
zbitter | TheJulia: nope, just flaky network ;) | 15:04 |
rpittau | #info last week we released Sprint 1 | 15:04 |
rpittau | Does anyone have anything else to announce or remind us of? | 15:04 |
iurygregory | Ironic 18.0.0 =) | 15:04 |
iurygregory | is available | 15:04 |
rpittau | yay \o/ | 15:05 |
rloo | whee | 15:05 |
TheJulia | \o/ | 15:05 |
rpittau | seems no other announcements, moving forward | 15:06 |
Qianbiao | \0/ | 15:06 |
rpittau | #topic Review action items from previous meeting | 15:06 |
rpittau | we don't have any action items from last time | 15:06 |
rpittau | moving on | 15:07 |
rpittau | #topic Review subteam status reports | 15:07 |
rpittau | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:07 |
rpittau | starting from L64 | 15:07 |
*** Mahnoor_Asghar has joined #openstack-ironic | 15:08 | |
*** JayF has joined #openstack-ironic | 15:08 | |
rpittau | I think we can mark iscsi removal as done | 15:08 |
*** zaneb has quit IRC | 15:08 | |
rpittau | kaifeng and arne_wiebalck are not here, any updates on node error history? | 15:09 |
rpittau | iurygregory: postpone privsep discussion to next time? | 15:09 |
iurygregory | rpittau, probably since Dmitry is not here | 15:09 |
rpittau | sounds good | 15:09 |
iurygregory | we can also discuss in the patch | 15:10 |
rpittau | sure | 15:10 |
rpittau | iurygregory: anything new on virtual media visibility ? | 15:11 |
iurygregory | rpittau, still writting code WIP | 15:11 |
rpittau | ok | 15:11 |
rpittau | team status looks ok, anything else to add ? | 15:13 |
iurygregory | I think we can move on =) | 15:14 |
TheJulia | Nothing status wise really | 15:14 |
rpittau | alrighty | 15:14 |
rpittau | #info Deciding on priorities for the coming week | 15:14 |
iurygregory | topic? =) | 15:14 |
TheJulia | yes, topic :) | 15:14 |
rpittau | errr | 15:14 |
rpittau | bad copy-paste | 15:14 |
TheJulia | heh | 15:15 |
rpittau | #topic Deciding on priorities for the coming week | 15:15 |
TheJulia | So I would really appreciate some of the db performance patches I've got up in review | 15:15 |
rpittau | current prio https://tinyurl.com/ironic-weekly-prio-dash | 15:15 |
TheJulia | it makes getting node list *much* faster | 15:15 |
rpittau | sure! | 15:15 |
rpittau | we definitely don't have many priorities left | 15:16 |
iurygregory | yeah | 15:16 |
TheJulia | I'll add the hash tags then | 15:16 |
JayF | TheJulia: is that whole line ready to merge? I reviewed them somewhat this weekend but didn't know you were ready to land 'em | 15:16 |
rpittau | ok, thanks | 15:16 |
ajya | can another core take a look at https://review.opendev.org/c/openstack/ironic/+/792088 ? | 15:16 |
TheJulia | JayF: close, I see some failures on the api patch but I've not had a chance to inspect them, I think it was +1'ed by zuul on the next to last change so it is likely stupidly minor | 15:17 |
rpittau | ajya: I added the tag in case we don't get to it today | 15:17 |
ajya | rpittau: thanks | 15:17 |
TheJulia | oh, the actual api layer change, it was the sanitizer change that didn't :\ | 15:17 |
TheJulia | pep8/bandit figures | 15:18 |
* TheJulia blinks | 15:18 | |
rpittau | pep8 is unforgiving | 15:18 |
TheJulia | it says it is complexity too | 15:18 |
TheJulia | figures | 15:18 |
TheJulia | I should have that sorted today | 15:19 |
rpittau | ok, we have I think a bit more space for priorities this week, anyone else has patches to add ? | 15:22 |
iurygregory | I would appreciate reviews in the event spec hehe | 15:23 |
iurygregory | it's already in the priority | 15:23 |
TheJulia | looks like we may want to get our eyes on https://review.opendev.org/c/openstack/ironic/+/768009 | 15:23 |
rpittau | yup, ok | 15:23 |
iurygregory | TheJulia, ++ | 15:23 |
rpittau | I added that too | 15:23 |
TheJulia | I need to re-review it | 15:24 |
TheJulia | We also need the upper layers | 15:24 |
TheJulia | We have someone pushing forward node tags | 15:24 |
TheJulia | which was added to the db ages ago | 15:24 |
TheJulia | kaifeng posted a question noting we may need to re-evaluate need | 15:25 |
TheJulia | which is avlid | 15:25 |
TheJulia | valid | 15:25 |
TheJulia | I think we can proceed | 15:27 |
rpittau | yeah, we look good on priorities | 15:27 |
rpittau | #topic Discussion | 15:27 |
rpittau | no discussions this week, let's move on | 15:27 |
rpittau | #topic Baremetal SIG | 15:28 |
iurygregory | Bare Metal SIG will meet tomorrow Tue June 8, 2021, at 2pm UTC on zoom. | 15:28 |
rpittau | reminder! Bare Metal SIG will meet tomorrow Tue June 8, 2021, at 2pm UTC on zoom | 15:28 |
iurygregory | "Topic-of-the-day" presentation by Dmitry Tantsur (dtantsur) with an "Introduction to the Ironic Python Agent Builder" | 15:28 |
iurygregory | arne_wiebalck asked me to give the update =) | 15:28 |
rpittau | ok :) | 15:28 |
rpittau | that's all for the Baremetal SIG | 15:29 |
rpittau | #topic RFE review | 15:29 |
rpittau | we don't have anything scheduled, but if anyone has RFEs to propose feel free to do it now | 15:30 |
rpittau | alright, let's move on :) | 15:30 |
rpittau | #topic Open Discussion | 15:30 |
rpittau | arne_wiebalck had a topic for open discussion but he asked to postpone to next week | 15:31 |
rpittau | anything else we want to discuss today ? | 15:31 |
TheJulia | nothing on the top of my head | 15:32 |
rpittau | ok! Last but not least.... | 15:32 |
rpittau | #topic Who is going to run the next meeting? | 15:32 |
TheJulia | I'll run it | 15:32 |
rpittau | thanks TheJulia | 15:32 |
iurygregory | tks TheJulia =) | 15:32 |
rpittau | and thanks everyone! | 15:32 |
rpittau | #endmeeting | 15:32 |
opendevmeet | Meeting ended Mon Jun 7 15:32:46 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:32 |
opendevmeet | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-06-07-15.00.html | 15:32 |
opendevmeet | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-06-07-15.00.txt | 15:32 |
opendevmeet | Log: http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-06-07-15.00.log.html | 15:32 |
Qianbiao | thanks rpittau | 15:33 |
rpittau | have a wonderful week everyone :) | 15:33 |
sdanni | Hey! hoping it's not too late... We would like to get more feedback on our security interface spec: https://review.opendev.org/c/openstack/ironic-specs/+/576718. | 15:34 |
sdanni | comments and suggestions are welcomed! | 15:35 |
TheJulia | o/ sdanni | 15:37 |
rpittau | sdanni: that's in the priorities, I'll tryu to get there during the week | 15:37 |
TheJulia | sdanni: I think I commented on the let me know item you had | 15:37 |
TheJulia | driver_internal_info most likely | 15:37 |
TheJulia | :) | 15:37 |
sdanni | lmcgann: anything to add? | 15:38 |
lmcgann | nothing else at the moment | 15:41 |
*** Qianbiao has quit IRC | 15:53 | |
rpittau | bye! o/ | 16:00 |
*** rpittau is now known as rpittau|afk | 16:01 | |
lmcgann | TheJulia: in regards to your comment on driver_internal_info, I think I answered your question, but let me know if there's anything I missed or am not understanding! | 16:02 |
TheJulia | lmcgann: okay, I'll try to cycle back to it after I finish my current administrative email | 16:07 |
*** stendulker_ has joined #openstack-ironic | 16:08 | |
*** zbitter is now known as zaneb | 16:12 | |
*** stendulker has quit IRC | 16:12 | |
*** Mahnoor_Asghar has quit IRC | 16:16 | |
*** jelabarre-rh has joined #openstack-ironic | 16:17 | |
*** stendulker_ has quit IRC | 16:18 | |
opendevreview | Merged openstack/bifrost master: Reasonable error message if network_interface is not found https://review.opendev.org/c/openstack/bifrost/+/795106 | 16:26 |
opendevreview | Merged openstack/ironic master: Improve agent_client logging https://review.opendev.org/c/openstack/ironic/+/793995 | 17:06 |
*** mjturek has joined #openstack-ironic | 17:12 | |
*** bburns has joined #openstack-ironic | 17:20 | |
*** bburns has quit IRC | 17:24 | |
*** bburns has joined #openstack-ironic | 17:25 | |
opendevreview | Dmitry Tantsur proposed openstack/ironic stable/wallaby: Improve agent_client logging https://review.opendev.org/c/openstack/ironic/+/795135 | 17:53 |
*** kkillsfirst has quit IRC | 18:06 | |
opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/wallaby: Reasonable error message if network_interface is not found https://review.opendev.org/c/openstack/bifrost/+/795136 | 18:31 |
*** kkillsfirst has joined #openstack-ironic | 18:38 | |
TheJulia | good morning kkillsfirst | 18:43 |
* dtantsur likes the nick | 18:45 | |
kkillsfirst | Hello TheJulia | 18:46 |
TheJulia | I wasn't going to say anything, but I think it is awesome too. | 18:48 |
*** qwebirc7749 has joined #openstack-ironic | 18:48 | |
*** qwebirc7749 has quit IRC | 18:48 | |
dtantsur | have a good night folks! | 18:49 |
TheJulia | o/ | 18:51 |
TheJulia | stevebaker: paras333: my laptop crashed :( | 19:50 |
paras333 | TheJulia: that's ok we are anyways done with the call | 19:56 |
* TheJulia decides it is best to just get lunch | 20:02 | |
*** rloo has quit IRC | 20:05 | |
*** rloo has joined #openstack-ironic | 20:06 | |
buhman | TheJulia: (you asked for reviews on db-perf-1) is moderate code golf welcome? | 20:09 |
TheJulia | code golf? | 20:15 |
TheJulia | keep in mind, I'm trying to make a series which is backportable | 20:15 |
*** zbitter has joined #openstack-ironic | 21:01 | |
*** jlvillal has joined #openstack-ironic | 21:04 | |
*** rloo has quit IRC | 21:07 | |
*** rloo has joined #openstack-ironic | 21:07 | |
janders | good morning Ironic o/ | 21:07 |
*** zaneb has quit IRC | 21:07 | |
buhman | TheJulia: yeah; I was going to suggest changing the "if x in <list>" operations to set operations. | 21:50 |
TheJulia | buhman: eh, It won't make much of a performance difference in the grand scheme of things | 21:52 |
buhman | https://gist.github.com/buhman/c3d32950acb29a6a899f923838e7e7d0 those two are equivalent | 21:52 |
buhman | ok | 21:52 |
*** tosky has quit IRC | 22:04 | |
TheJulia | buhman: oh, yeah, even that far up in the stack... *shrugs* | 22:29 |
*** lmcgann has quit IRC | 22:40 | |
stevebaker | TheJulia: I can replicate the ipa python2.7 test failures, so I'll take a look at that | 22:57 |
TheJulia | stevebaker: okay, I can also look at it tomorrow if my work laptop decides I. An be let in tomorrow :( | 22:58 |
stevebaker | cool | 22:58 |
* TheJulia goes back to exercising because the computers hate her | 22:58 | |
*** kkillsfirst has quit IRC | 23:20 | |
*** kkillsfirst has joined #openstack-ironic | 23:32 | |
*** jelabarre-rh has quit IRC | 23:53 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!