*** pmannidi is now known as pmannidi|AFK | 03:19 | |
opendevreview | Jacob Anders proposed openstack/ironic master: Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/800001 | 05:53 |
---|---|---|
arne_wiebalck | Good morning janders and Ironic! | 06:03 |
janders | good morning arne_wiebalck o/ | 06:03 |
opendevreview | Jacob Anders proposed openstack/ironic master: Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/800001 | 06:06 |
iurygregory | good morning arne_wiebalck and Ironic o/ | 06:28 |
janders | hey iurygregory o/ | 06:28 |
iurygregory | hey janders o/ | 06:29 |
arne_wiebalck | Good morning iurygregory o/ | 06:32 |
opendevreview | Jacob Anders proposed openstack/ironic master: Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/800001 | 06:49 |
janders | ^ let's see what CI thinks of my tests :) | 06:50 |
janders | off for an evening walk, back in a bit | 06:50 |
janders | how long did you guys stay off exercise after your covid vaccinations? I don't have much side effects any more (except for still feeling a little tired) | 06:52 |
iurygregory | prometheanfire, I haven't seen any issue with tox py38 for Ironic jobs related to the SQLAlchemy error that is reported in the patch you mentioned =( | 06:52 |
janders | just being careful (it was pfizer, which is said to have super-rare cardio complications, that is the reason behind being careful) | 06:53 |
iurygregory | janders, fyi astrazeneca has more cardio complications than pfizer =) | 06:53 |
arne_wiebalck | janders: I used this as an excuse for a week. | 06:54 |
iurygregory | arne_wiebalck, LOL | 06:54 |
arne_wiebalck | iurygregory: :-D | 06:55 |
iurygregory | arne_wiebalck, I hope it was after the 2nd shot not after the 1st :D | 06:55 |
iurygregory | the nice thing I've discovered during the weekend is that the system here in CZ says I didn't get the vaccine :D | 06:56 |
iurygregory | so I don't have any certificate \o/ | 06:56 |
arne_wiebalck | iurygregory: any shot ofc | 07:35 |
iurygregory | arne_wiebalck, hahaha best idea XD | 07:35 |
arne_wiebalck | iurygregory: ai ... moving around I used my pass already a couple of times | 07:35 |
arne_wiebalck | iurygregory: even to buy a sandwich and to sit down in the bakery I needed it | 07:36 |
iurygregory | yeah same here in CZ | 07:37 |
iurygregory | I need to figure out how to get the cert now hehe | 07:37 |
iurygregory | or I will need to keep doing antigen-test | 07:37 |
janders | thanks iurygregory arne_wiebalck | 07:41 |
janders | I may get back on my bike tomorrow | 07:41 |
opendevreview | Tadeas Kot proposed openstack/python-ironicclient master: Add support for fields in drivers CLI https://review.opendev.org/c/openstack/python-ironicclient/+/805978 | 07:46 |
jeeva | network question, if my openstack controllers are on the untagged MAAS PXE network, what is their place in ironic? To stay as is, or must they also be the same as the ironic PXE untagged VLAN ? | 08:20 |
opendevreview | Aija Jauntēva proposed openstack/ironic master: Remove manager param for iDRAC OEM calls https://review.opendev.org/c/openstack/ironic/+/792200 | 08:46 |
dtantsur | good morning ironic, happy Monday | 09:27 |
dtantsur | janders: re exercising: I had my vaccine on Friday, went climbing on Monday | 09:28 |
iurygregory | good morning dtantsur | 09:30 |
opendevreview | Aija Jauntēva proposed openstack/ironic master: Add RAID mode conversion for idrac-redfish https://review.opendev.org/c/openstack/ironic/+/804112 | 09:36 |
opendevreview | Aija Jauntēva proposed openstack/ironic master: Add clear foreign config to idrac-redfish RAID https://review.opendev.org/c/openstack/ironic/+/804848 | 10:10 |
dtantsur | folks, we've collected a few outstanding bifrost patches, could you take a look? https://review.opendev.org/q/project:openstack%252Fbifrost+status:open | 10:38 |
dtantsur | iurygregory: are you looking into the sqlalchemy issue? | 10:42 |
dtantsur | (I can do it as a warm-up for the week) | 10:42 |
iurygregory | dtantsur, I was trying to reproduce the issue but no lucky yet | 10:43 |
dtantsur | yeah, upgrading oslo.db to 11.0.0 doesn't seem enough | 10:44 |
iurygregory | yup | 10:44 |
*** sshnaidm is now known as sshnaidm|off | 10:46 | |
dtantsur | prometheanfire: I'm starting to suspect a race or regression in oslo.db | 10:46 |
dtantsur | potentially in https://opendev.org/openstack/oslo.db/commit/22b44ee18b1585bc2943a29ddaa18051cb1344ed | 10:47 |
iurygregory | dtantsur, maybe we should also raise this in #openstack-oslo to see if they have some ideas? | 11:02 |
dtantsur | iurygregory: likely. could you please do it? I'm not feeling well. | 11:02 |
iurygregory | yup! | 11:03 |
iurygregory | take care =) | 11:03 |
opendevreview | Jacob Anders proposed openstack/ironic master: Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/800001 | 11:19 |
janders | thank you dtantsur - sounds like I may try getting on the bike tomorrow :) | 11:43 |
janders | see you tomrorow Ironic o/ | 11:43 |
iurygregory | bye janders o/ | 11:46 |
opendevreview | Merged openstack/ironic master: Add better error messages for invalid conf molds https://review.opendev.org/c/openstack/ironic/+/805222 | 12:07 |
opendevreview | Verification of a change to openstack/ironic master failed: Minor formatting and doc changes to change boot mode feature commit. https://review.opendev.org/c/openstack/ironic/+/805841 | 12:15 |
opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix iDRAC import configuration job with errors https://review.opendev.org/c/openstack/ironic/+/805431 | 12:17 |
TheJulia | good morning | 13:07 |
arne_wiebalck | Hey TheJulia o/ | 13:10 |
dtantsur | morning TheJulia | 13:11 |
iurygregory | good morning TheJulia =) | 13:14 |
opendevreview | Julia Kreger proposed openstack/ironic master: Revert "Allow reboot to hard disk following iso ramdisk deploy." https://review.opendev.org/c/openstack/ironic/+/805284 | 13:21 |
opendevreview | Verification of a change to openstack/ironic master failed: Minor formatting and doc changes to change boot mode feature commit. https://review.opendev.org/c/openstack/ironic/+/805841 | 13:33 |
prometheanfire | dtantsur: fun | 13:36 |
dtantsur | yeah.. I don't quite get what is going on | 13:37 |
dtantsur | iurygregory wanted to ask on #openstack-oslo | 13:37 |
iurygregory | I did =) | 13:38 |
iurygregory | but no answer yet | 13:39 |
iurygregory | I'm not sure who are the oslo.db experts to ping, so I just asked in the channel =) | 13:40 |
opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: Move virtual media endpoint handlers to a separate blueprint https://review.opendev.org/c/openstack/sushy-tools/+/806603 | 14:35 |
iurygregory | lmcgann, I just noticed that some of the patches for the keylime are in merge conflict, it would be good to fix before the Review Jam tomorrow =) | 14:36 |
lmcgann | will do! | 14:36 |
opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: [WIP] Support uploading certificates for virtual media https://review.opendev.org/c/openstack/sushy-tools/+/806253 | 14:40 |
iurygregory | #startmeeting ironic | 15:00 |
opendevmeet | Meeting started Mon Aug 30 15:00:00 2021 UTC and is due to finish in 60 minutes. The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
iurygregory | o/ | 15:00 |
erbarr | o/ | 15:00 |
iurygregory | Hello everyone, welcome to our weekly meeting! | 15:00 |
arne_wiebalck | o/ | 15:00 |
ajya | o/ | 15:00 |
dtantsur | o/ | 15:00 |
rloo | o/ | 15:00 |
iurygregory | You can find our agenda in the wiki | 15:00 |
iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
rpioso | \o | 15:00 |
TheJulia | o/ | 15:00 |
iurygregory | #topic Announcements / Reminder | 15:01 |
iurygregory | #info We have topics for the Review Jam tomorrow =) | 15:01 |
iurygregory | Anyone has something to announce or remind us? | 15:01 |
TheJulia | Iury will be our glorius PTL for Yoga! | 15:02 |
dtantsur | \o/ | 15:02 |
iurygregory | \o/ | 15:02 |
rloo | yay, many thanks iurygregory!!! | 15:02 |
iurygregory | np, I will do my best \o/ | 15:03 |
TheJulia | :) | 15:03 |
iurygregory | #topic Review action items from previous meeting | 15:03 |
iurygregory | since we don't have any action items from the previous meeting, moving on | 15:04 |
iurygregory | #topic Review subteam status reports | 15:04 |
iurygregory | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:04 |
iurygregory | starting on L60 | 15:04 |
TheJulia | Looks like everyone is getting pulled downstream pretty hardcore | 15:06 |
iurygregory | yup, it happens from time to time =( | 15:07 |
dtantsur | hey, I've produced a ton of bifrost and sushy-tools patches :D | 15:07 |
iurygregory | I think dtantsur will add a lot patches for the Deciding on priorities for the coming week :D | 15:07 |
dtantsur | with great pleasure! | 15:07 |
TheJulia | Sounds most excellent | 15:08 |
iurygregory | oh nice, we have a section for the Keylime work in the subteam status :D | 15:09 |
iurygregory | dtantsur, TheJulia Driver structure and model doesn't have any updates for like 1month, can I just add "no-updates"? | 15:10 |
TheJulia | Yeah, I've been pulled downstream so much I haven't really had time to think about any of it | 15:10 |
dtantsur | yes please | 15:11 |
iurygregory | ack =) | 15:11 |
dtantsur | I'll likely spend the rest of the cycle redfishing | 15:11 |
TheJulia | dtantsur: is that like going fishing on a boat?! | 15:11 |
dtantsur | I would love to try :) | 15:11 |
dtantsur | but my curse is figuring out TLS apparently | 15:12 |
iurygregory | I think dtantsur will become a chef :D | 15:12 |
dtantsur | Principal Fugu Engineer | 15:12 |
TheJulia | so next time we're all in Germany, dtantsur can cook us fish :) | 15:12 |
iurygregory | Fugu LOL | 15:12 |
TheJulia | lol | 15:12 |
dtantsur | TheJulia: we have very nice sushi fish here thanks to a few authentic japanese shops | 15:13 |
TheJulia | One day :) | 15:13 |
dtantsur | sigh | 15:13 |
TheJulia | sigh++ | 15:13 |
iurygregory | almost dinner time and we are talking about food \o/ | 15:13 |
iurygregory | sounds about right :D | 15:13 |
iurygregory | I think we are good on the subteam status, moving on | 15:14 |
iurygregory | #topic Deciding on priorities for the coming week | 15:14 |
ajya | can these be added - https://review.opendev.org/c/openstack/ironic/+/804112 https://review.opendev.org/c/openstack/ironic/+/804848 ? | 15:14 |
TheJulia | ajya: is there a story for background on those? | 15:15 |
iurygregory | makes sense to me | 15:15 |
iurygregory | yeah having a story RFE would be good =) | 15:16 |
dtantsur | can I just add everything non-WIP from https://review.opendev.org/q/owner:dtantsur%2540protonmail.com+status:open+(project:openstack/bifrost+OR+project:openstack/sushy-tools) ? | 15:16 |
ajya | TheJulia: no, it's to get it closer to idrac-wsman behavior | 15:16 |
dtantsur | I don't want to paste every patch here | 15:16 |
TheJulia | iurygregory: I'm not sure it is an rfe, but can't be sure without some more words somplace | 15:16 |
TheJulia | dtantsur: sounds reasonable | 15:16 |
iurygregory | TheJulia, well they are with feature in the releasenotes thats why I said rfe hehe | 15:17 |
TheJulia | iurygregory: ahh, yeah | 15:17 |
arne_wiebalck | I have added https://review.opendev.org/c/openstack/ironic/+/801425 (this is the fast track retry on failed BMC connections, seems to prevent fast track nodes from being switched off) | 15:17 |
iurygregory | dtantsur, I think you can =) | 15:17 |
TheJulia | arne_wiebalck: random bmc unhappiness? | 15:18 |
arne_wiebalck | TheJulia: I think UDP packet loss | 15:18 |
ajya | I can create story with tasks for these patches with some background info, otherwise patches should be self-contained. | 15:19 |
TheJulia | I guess that is bound to happen on conductors with lots of nodes | 15:19 |
arne_wiebalck | TheJulia: that is what I am thinking | 15:19 |
TheJulia | ajya: sounds good, and also lets explicitly state if there is a backport intent | 15:19 |
arne_wiebalck | TheJulia: and we do retries for power sync, but not on fast track | 15:19 |
TheJulia | arne_wiebalck: makes sense | 15:19 |
* arne_wiebalck should try moving to redfish to fix these instead of retrying ... | 15:20 | |
TheJulia | eh, we've still seem bmc's go out for lunch for a moment | 15:20 |
TheJulia | so makes sense to retry anwyay | 15:20 |
iurygregory | ajya, I think a story would be good | 15:20 |
arne_wiebalck | TheJulia: ++ | 15:20 |
iurygregory | ajya, feel free to add the hashtag in the patches you mentioned o/ | 15:22 |
ajya | ok, thanks | 15:22 |
iurygregory | anyone has more patches that needs review? | 15:23 |
TheJulia | Not on my end | 15:23 |
iurygregory | I think we are good with the amount of patches we added | 15:24 |
iurygregory | moving on | 15:24 |
iurygregory | #topic Discussion | 15:25 |
iurygregory | We don't have any topics for Discussion in the agenda, but I will give a few minutes to see if anyone has something | 15:25 |
TheJulia | Could/should we just move to Open Discussion then? | 15:25 |
iurygregory | maybe, arne_wiebalck do you have something for the SIG? | 15:26 |
arne_wiebalck | yes | 15:26 |
iurygregory | #topic Baremetal SIG | 15:26 |
iurygregory | go ahead =) | 15:26 |
arne_wiebalck | we said we would do an operator feedback session next time | 15:26 |
arne_wiebalck | this is what we have on the etherpad atm | 15:26 |
arne_wiebalck | IIRC, we said we would decide nearer the time if that would make sense | 15:27 |
arne_wiebalck | given that there was also the openinfra live event | 15:27 |
TheJulia | Yeah, we kind of didn't push that forward with holidays and the chance of getting something larger organized, yet that fell through | 15:27 |
arne_wiebalck | right | 15:27 |
arne_wiebalck | while I would like to have such a discussion I am not sure operators would show up by themselves | 15:28 |
arne_wiebalck | given the attendance we had over the last months | 15:28 |
TheJulia | No getting them to appear in a community setting seems to be one of the hardest things to do | 15:28 |
arne_wiebalck | so, we could either reach out directly or skip and have a topic of the day | 15:29 |
arne_wiebalck | thoughts? | 15:29 |
iurygregory | well, maybe we can try to reach out the osopsmeetup ? | 15:29 |
iurygregory | and if we know some operators we can try to ask if they are interested? | 15:30 |
arne_wiebalck | I think they face similar issues with participation | 15:30 |
arne_wiebalck | osops I mean | 15:30 |
jssfr | what's an operator feedback session? | 15:30 |
arne_wiebalck | jssfr: Ironic users come with their feedback, pain points, ideas, ... | 15:31 |
arne_wiebalck | jssfr: Ironic users/operators/admins | 15:31 |
jssfr | oh | 15:31 |
jssfr | never heard of it :) | 15:31 |
TheJulia | iurygregory: I think it might make sense to engage them | 15:31 |
arne_wiebalck | heh, that is the problem | 15:31 |
TheJulia | iurygregory: reach out and say "hey, we're willing to listen/chat" if interested | 15:31 |
TheJulia | but I don't know if that makes a baremetal SIG session | 15:31 |
iurygregory | right | 15:32 |
TheJulia | I'd <3 soemthing to helpf or planning with the next development cycle though | 15:32 |
arne_wiebalck | TheJulia: well, we can try and offer to listen | 15:32 |
iurygregory | we can probably come up with something for the PTG? | 15:32 |
arne_wiebalck | TheJulia: if noone comes, fine, at least we gave the chance | 15:32 |
iurygregory | or even a separate event | 15:32 |
iurygregory | arne_wiebalck, ++ | 15:32 |
TheJulia | yeah, true | 15:33 |
* TheJulia wonders if anyone has posted questions/comments on our existing videos | 15:33 | |
arne_wiebalck | ok, so how about everyone reaches out and we see what happens? | 15:33 |
TheJulia | sounds reasonable | 15:34 |
iurygregory | yeah, I agree | 15:34 |
arne_wiebalck | like, we know some operators/users :) | 15:34 |
arne_wiebalck | ok, let's try this then | 15:34 |
iurygregory | #action everyone to reach out to know operators/users | 15:35 |
arne_wiebalck | this is for tomorrow in a week | 15:35 |
arne_wiebalck | Sep 7th | 15:35 |
arne_wiebalck | I will send out a mail s well. | 15:35 |
iurygregory | arne_wiebalck, ++ | 15:35 |
arne_wiebalck | that is it from me for the SIG | 15:36 |
iurygregory | tweet also :D | 15:36 |
arne_wiebalck | yes, will do | 15:36 |
iurygregory | tks arne_wiebalck ! | 15:36 |
iurygregory | skipping RFE review since we don't have any | 15:36 |
iurygregory | #topic Open Discussion | 15:36 |
TheJulia | I'm suspecting we should likely start discussing/planning the PTG | 15:37 |
arne_wiebalck | I have something :) | 15:37 |
iurygregory | go ahead arne_wiebalck | 15:37 |
iurygregory | TheJulia,++ | 15:37 |
arne_wiebalck | For the burn-in we are interested in the log output in more or less real time. | 15:37 |
arne_wiebalck | We were thinking about having sth like fluentd on the image. | 15:38 |
arne_wiebalck | The other option would be to ship logs to the conductor and use the conductor logging. | 15:38 |
arne_wiebalck | The conductor gets the IPA logs eventually. | 15:38 |
arne_wiebalck | Now, for fast track this is more tricky, since it will only do this at the end. | 15:39 |
TheJulia | I'm not opposed to it, but how would it work when the conductor is just a process in a container | 15:39 |
arne_wiebalck | How does this work now for the IPA logs when the conductor is in a container? | 15:40 |
arne_wiebalck | My thinking is to leverage the logging that should be in place for the conductor anyway. | 15:40 |
TheJulia | arne_wiebalck: just lives on a filesystem, likely a bind mounted filesystem | 15:40 |
arne_wiebalck | Using fluentd on the IPA pushes the issue downstream and to image building. | 15:41 |
TheJulia | indeed | 15:41 |
TheJulia | so realistically, it could be anywhere configured by an operator | 15:41 |
arne_wiebalck | Having logs flow into the conductor more regularly offers operators to use them even in fast track. | 15:41 |
TheJulia | yeah | 15:42 |
arne_wiebalck | Imagine 100s of nodes in fast track, logs sent into the conductor, and onwards to ES, for instance. | 15:43 |
arne_wiebalck | We have a mechanism to get the logs already. | 15:43 |
arne_wiebalck | It is just run only once and collects a lot of additional logsd. | 15:43 |
arne_wiebalck | logs | 15:43 |
TheJulia | I guess it is more an agent/image thing than anything else? | 15:43 |
arne_wiebalck | Yeah, you could see it that way. But configuring the IPA to push logs into a monitoring infra may be more difficult than the controllers which may do this anyway. | 15:44 |
arne_wiebalck | I am trying to leverage the existing pipeline. | 15:45 |
TheJulia | sounds good | 15:45 |
arne_wiebalck | I would imagine fast track users want the IPA logs, no? | 15:46 |
arne_wiebalck | And they may not want to wait until the IPA is fully done (which I think is when the logs are collected). | 15:46 |
arne_wiebalck | Anyway, if anyone has thoughts on this, let me know :) | 15:47 |
TheJulia | It makes sense to get them and if they are doing something like burn in it does also make sense | 15:47 |
TheJulia | And is really just more operator capable at that point | 15:47 |
arne_wiebalck | Yes, burn-in is the use case at hand. | 15:47 |
arne_wiebalck | The h/w colleagues would like to get the network speed results for their records for instance. | 15:48 |
arne_wiebalck | There is a logical difference between burn-in and benchmark, but it does not make sense to run a benchmark when the burn-in provides these values. | 15:48 |
TheJulia | indeed | 15:49 |
arne_wiebalck | No need to decide anything now, I was just thinking that sending logs more regularly may be sensible for fast track. | 15:50 |
arne_wiebalck | In general, not only burn-in. | 15:50 |
* arne_wiebalck stops now :) | 15:50 | |
iurygregory | it may be sensible I would say, just wondering the overhead that can cause | 15:51 |
arne_wiebalck | iurygregory: yes, atm, we create an archive for a clean "cycle" | 15:52 |
arne_wiebalck | iurygregory: we will need to avoid sending the same logs over and over again | 15:53 |
iurygregory | interesting, I do like the idea | 15:53 |
iurygregory | next week we can talk about the PTG topics I would say since we only have 5minutes now | 15:54 |
TheJulia | Sounds good | 15:54 |
iurygregory | we already have the etherpad https://etherpad.opendev.org/p/ironic-yoga-ptg | 15:55 |
iurygregory | feel free to add topics =) | 15:55 |
iurygregory | #topic Who is going to run the next meeting? | 15:55 |
iurygregory | do we have any volunteers? | 15:55 |
rloo | is next mon a holiday for folks? it is in canada and i think US. | 15:56 |
iurygregory | *-* 06 Sep? | 15:56 |
iurygregory | maybe we can skip if people agree | 15:56 |
rloo | labor day in US | 15:56 |
iurygregory | it's my bday so I give +1 to skip LOL | 15:57 |
rloo | that's a really good reason to celebrate! | 15:57 |
iurygregory | yeah =) | 15:58 |
dtantsur | ++ | 15:59 |
iurygregory | so we good to skip next weekly meeting? | 15:59 |
rloo | crickets means... yes :D | 16:00 |
iurygregory | yeah :D | 16:00 |
iurygregory | I will send an email | 16:00 |
iurygregory | tks everyone! | 16:00 |
iurygregory | #endmeeting | 16:00 |
opendevmeet | Meeting ended Mon Aug 30 16:00:26 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-30-15.00.html | 16:00 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-30-15.00.txt | 16:00 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-30-15.00.log.html | 16:00 |
arne_wiebalck | thanks iurygregory ! | 16:00 |
TheJulia | Thanks iurygregory! | 16:00 |
ajya | TheJulia: when you have time, can you revisit this patch https://review.opendev.org/c/openstack/ironic/+/792200 that got unblocked by dependency release? Thanks. | 16:03 |
TheJulia | ajya: done | 16:10 |
opendevreview | Leo McGann proposed openstack/ironic master: Adds Attestation Interface to Node https://review.opendev.org/c/openstack/ironic/+/755836 | 16:23 |
opendevreview | Merged openstack/ironic master: Minor formatting and doc changes to change boot mode feature commit. https://review.opendev.org/c/openstack/ironic/+/805841 | 16:51 |
arne_wiebalck | bye everyone, see you tmrw o/ | 16:56 |
opendevreview | Verification of a change to openstack/ironic master failed: Remove manager param for iDRAC OEM calls https://review.opendev.org/c/openstack/ironic/+/792200 | 17:39 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!