| *** JasonF is now known as JayF | 00:54 | |
| janders | good morning Ironic o/ | 02:46 |
|---|---|---|
| arne_wiebalck | Good morning, janders and Ironic! | 07:32 |
| opendevreview | Denys Mishchenko proposed openstack/sushy-tools master: Save previous hd boot device when setting Pxe https://review.opendev.org/c/openstack/sushy-tools/+/869379 | 07:46 |
| rpittau | good morning ironic! o/ | 08:00 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix deprecated module ansible lint error https://review.opendev.org/c/openstack/bifrost/+/866040 | 08:58 |
| *** priteau_ is now known as priteau | 08:58 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/zed: Fix CI https://review.opendev.org/c/openstack/ironic/+/868789 | 09:04 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/yoga: Fix CI https://review.opendev.org/c/openstack/ironic/+/869045 | 09:05 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Revert setuptools disable autodiscovery https://review.opendev.org/c/openstack/ironic/+/869539 | 09:07 |
| rpittau | JayF, TheJulia, dtantsur ^ I'm checking other places where we disabled autodiscovery to revert the change now that the pbr fix has landed | 09:08 |
| rpittau | wondering if we should bump the min required version as well | 09:09 |
| rpittau | mmm probably will have to wait for the new release anyway | 09:10 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix CI https://review.opendev.org/c/openstack/bifrost/+/868303 | 09:14 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Fix CI https://review.opendev.org/c/openstack/bifrost/+/868303 | 09:14 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/zed: Fix CI https://review.opendev.org/c/openstack/ironic/+/868789 | 09:15 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/yoga: Fix CI https://review.opendev.org/c/openstack/ironic/+/869045 | 09:15 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-ui master: Fix tox4 and setuptools errors https://review.opendev.org/c/openstack/ironic-ui/+/868755 | 09:16 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-ui master: Fix tox4 and setuptools errors https://review.opendev.org/c/openstack/ironic-ui/+/868755 | 09:17 |
| kubajj | Good morning janders, arne_wiebalck, rpittau and everybody! | 09:20 |
| rpittau | hey kubajj :) | 09:22 |
| arddennis | Hi. I've created a patch like it was suggested before: https://review.opendev.org/c/openstack/sushy-tools/+/869379 | 09:26 |
| arddennis | The problem it solves is the BootSourceOverrideEnabled Once" case with the libvirt. If Pxe boot is expected once, then after server deployment it will not try the other boot methods. So I've just added the hd boot devices to be always included if they were previously defined. I can probably go further and pass BootSourceOverrideEnabled into set_boot_device, but I am not sure if it is required or if the proposed cha | 09:26 |
| arddennis | nge breaks any CI. | 09:26 |
| dtantsur | good morning folks | 09:39 |
| rpittau | welcome back dtantsur :) | 11:03 |
| rpittau | dtantsur: thanks for the comment on https://review.opendev.org/c/openstack/ironic/+/868521, I guess a discussion is in order with JayF :) | 11:04 |
| kubajj | When I try to pull in order to rebase, I get this error. This is a problem on my side, right? fatal: unable to access 'https://opendev.org/openstack/ironic/': server certificate verification failed. CAfile: none CRLfile: none | 11:23 |
| dtantsur | kubajj: could be on you, try curling the same URL? | 11:24 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-ui master: Fix tox4 and setuptools errors https://review.opendev.org/c/openstack/ironic-ui/+/868755 | 11:28 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-ui master: Fix tox4 errors https://review.opendev.org/c/openstack/ironic-ui/+/868755 | 11:29 |
| iurygregory | good morning Ironic | 11:33 |
| opendevreview | Merged openstack/ironic master: Docs: Add considerations to anaconda docs https://review.opendev.org/c/openstack/ironic/+/854045 | 14:19 |
| * TheJulia wonders if anyone would find the SRBAC model interesting | 14:46 | |
| iurygregory | for a Summit talk? | 14:48 |
| TheJulia | yeah | 14:51 |
| JayF | TheJulia: can you run meeting this morning? I'm running late 😐 | 14:55 |
| rpittau | dtantsur, JayF, we probably need to reach a deal for https://review.opendev.org/c/openstack/ironic/+/868521 :) | 14:55 |
| TheJulia | JayF sure | 14:56 |
| iurygregory | TheJulia, i would be interested in the topic =) +1 | 14:56 |
| dtantsur | rpittau: I'm not convinced we're in disagreement :) see my clarification | 14:58 |
| TheJulia | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Jan 9 15:00:15 2023 UTC and is due to finish in 60 minutes. The chair is TheJulia. 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 |
| TheJulia | #chair JayF | 15:00 |
| opendevmeet | Current chairs: JayF TheJulia | 15:00 |
| TheJulia | Good morning #Ironic! | 15:00 |
| rpittau | o/ | 15:00 |
| dtantsur | o/ | 15:00 |
| TheJulia | It is that time again, time for our weekly meeting! | 15:00 |
| rloo | o/ | 15:00 |
| hjensas | o/ | 15:00 |
| TheJulia | Our agenda this week is posted to the wiki. | 15:01 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| TheJulia | #topic Announcements / Reminders | 15:01 |
| TheJulia | Happy new year! | 15:01 |
| erbarr | o/ | 15:01 |
| TheJulia | There is a single standing reminder to review the items tagged ironic-week-prio. | 15:02 |
| TheJulia | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:02 |
| TheJulia | Right now, a LOT of that is tox related, so we should likely just discuss that and what needs to stay/go/be abanodoned, etc. | 15:02 |
| rpittau | besides tox 4, CI will be broken until new version of PBR lands | 15:03 |
| TheJulia | Nice | 15:03 |
| TheJulia | so... hack code week?! | 15:03 |
| rpittau | and bifrost CI also depends on https://review.opendev.org/c/openstack/ironic/+/868521 that hopefully will merge soon(TM) :) | 15:04 |
| TheJulia | In other news, we have less than *two days* left to submit summit CFP items for the upcoming OpenInfra summit. I highly encourage brainstorming ideas or raising questions on what people would find interesting/like to hear during Open Discussion or after the meeting | 15:04 |
| TheJulia | #link https://cfp.openinfra.dev/app/start | 15:04 |
| TheJulia | Does anyone have anything else to announce or remind us of this week? | 15:04 |
| rpittau | virtual PTG in march? | 15:05 |
| TheJulia | I guess we do need to get an etherpad started for that and so on and so forth | 15:05 |
| TheJulia | Do I have a volunteer to start that? | 15:06 |
| TheJulia | (Since I can't seem to find one) | 15:06 |
| TheJulia | #crickets | 15:07 |
| iurygregory | o/ | 15:07 |
| TheJulia | iurygregory: volunteering? | 15:07 |
| iurygregory | yeah | 15:08 |
| TheJulia | awesome | 15:08 |
| TheJulia | #action Iury to create virtual ptg etherpad for us to get started on planning that. | 15:08 |
| TheJulia | #topic Review action items from previous meetings | 15:08 |
| TheJulia | JayF: It appears you had an action item from our prior meeting regarding moving a discussion back to the list? | 15:08 |
| JayF | I'll follow up on whatever it was when I'm at a PC... | 15:09 |
| TheJulia | ack | 15:09 |
| TheJulia | Moving on! | 15:09 |
| TheJulia | #topic Review Ironic CI status & update whiteboard if needed | 15:09 |
| TheJulia | We already briefly discussed CI. *kaboom* | 15:09 |
| TheJulia | rpittau: could you update https://etherpad.opendev.org/p/IronicWhiteBoard with something to indicate what we should expect and what to watch out for ? | 15:10 |
| rpittau | will do | 15:11 |
| TheJulia | Much appreciated | 15:11 |
| TheJulia | Moving on! | 15:11 |
| TheJulia | #topic 2023.1 Work in progress | 15:11 |
| TheJulia | #link https://etherpad.opendev.org/p/IronicWorkstreams2023.1 | 15:11 |
| TheJulia | It looks like we need to try and get traction moving on sharding this week | 15:12 |
| TheJulia | Regarding SQLAlchemy, I guess we're still pending the metal3 job before we revert the last bit from ironic? | 15:13 |
| iurygregory | I think you are right | 15:13 |
| TheJulia | Looks like we need to update the status for that job. | 15:13 |
| TheJulia | I'm going to skip over the tox 4.0 topic which is listed next, since we already briefly touched upon it and the most current state. | 15:14 |
| dtantsur | I'm just back, need to pick up where I left | 15:14 |
| TheJulia | ack | 15:14 |
| TheJulia | It appears we have a topic of Future of Bugfix releases. | 15:14 |
| TheJulia | #topic Future of Bugfix releases | 15:15 |
| TheJulia | I'm not prepared to discuss this one this week, do we want to punt this one for this week? | 15:15 |
| TheJulia | #crickets | 15:16 |
| TheJulia | :) | 15:16 |
| TheJulia | I'm taking this as a punt for this week topic. | 15:16 |
| dtantsur | it's JayF's topic | 15:16 |
| TheJulia | Yeah, and he is not at a computer at the moment... | 15:17 |
| dtantsur | we cannot discuss without him | 15:17 |
| TheJulia | indeed | 15:17 |
| TheJulia | Onward, we can always come back to it | 15:17 |
| TheJulia | #topic Human availability update | 15:17 |
| TheJulia | I'm around most of this and next week aside from next Monday which is a Holiday in the US. | 15:17 |
| * dtantsur is not a human, beep-beep | 15:17 | |
| TheJulia | I'll be out for a conference the first few days of the first full week in feburary | 15:18 |
| TheJulia | Anyone else have schedule updates to denote? | 15:18 |
| * TheJulia gets out a stack of punch cards :) | 15:18 | |
| dtantsur | I should be online until late February | 15:18 |
| iurygregory | same for me | 15:18 |
| TheJulia | It appears our next topic is RFE Review, we have none listed, so I guess we should just move on to Open Discussion | 15:19 |
| TheJulia | #topic Open Discussion | 15:19 |
| dtantsur | we don't have a SIG section any more? | 15:19 |
| dtantsur | I was going to bring up https://github.com/OpenStackweb/ironic-website/pull/52 - text needs review | 15:19 |
| TheJulia | not in the agenda, but easy enough | 15:20 |
| dtantsur | merging it means pulling the trigger on the announcement :) | 15:20 |
| TheJulia | the bots are merging it | 15:21 |
| TheJulia | oh... well. did we want to wait? | 15:21 |
| dtantsur | I did not, assuming you actually reviewed the text ;) | 15:22 |
| TheJulia | I skimmed it, didn't see anything which seemed problematic or mispelled | 15:22 |
| dtantsur | #link https://ironicbaremetal.org/blog/baremetal-sig-2023q1/ Information about the upcoming bare metal SIG meetup | 15:23 |
| rloo | i see a typo but i guess it is too late (sorry) | 15:23 |
| TheJulia | \o/ | 15:23 |
| dtantsur | rloo: we can fix things after the fact, I assume | 15:24 |
| TheJulia | Always | 15:24 |
| rloo | ok, i'll comment/look... | 15:24 |
| TheJulia | Anything else for us to discuss today? | 15:24 |
| kubajj | o/ | 15:25 |
| TheJulia | No deep burning questions that must be answered regarding Ironic or related components/areas? | 15:26 |
| TheJulia | Well, in that case, Thanks everyone! | 15:26 |
| TheJulia | Have a wonderful week! | 15:26 |
| TheJulia | #endmeeting | 15:27 |
| opendevmeet | Meeting ended Mon Jan 9 15:27:38 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:27 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-01-09-15.00.html | 15:27 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-01-09-15.00.txt | 15:27 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-01-09-15.00.log.html | 15:27 |
| rpittau | TheJulia: I've updated the CI status section in the etherpad | 15:27 |
| JayF | Thanks TheJulia... Woke up not feeling well and might end up out today | 15:33 |
| TheJulia | JayF: I almost did the same, feel better! | 15:33 |
| TheJulia | iurygregory: did you have a chance to discuss metrics names with folks? | 15:34 |
| iurygregory | still waiting for a response | 15:34 |
| TheJulia | ack, thanks | 15:34 |
| TheJulia | I'm assuming most people returning today are still sifting through emails and piles of messages | 15:35 |
| iurygregory | yeah | 15:35 |
| Kirill_ | HI! working with ironic_prometheus_exporter and cought issue: if bm node is swithced off it returns None instead of sensor info. and here: https://github.com/openstack/ironic-prometheus-exporter/blob/master/ironic_prometheus_exporter/parsers/ipmi.py#L174 we caught an exception - keyerror. as a result every 5min in logs we can see a large amount of such errors. | 15:50 |
| Kirill_ | so i'd like to add checker before before accessing the dict or maybe its better not to send requests to such bm. Who can help with this?) | 15:52 |
| dtantsur | iurygregory: ^^ | 15:52 |
| Kirill_ | example of error: "ERROR ironic_prometheus_exporter.parsers.ipmi KeyError: 'Sensor Reading'" | 15:53 |
| TheJulia | hmmm | 15:54 |
| TheJulia | Kirill_: I think your going to need to collect and supply the raw ipmi sensor dump output | 15:54 |
| iurygregory | hummmm | 15:55 |
| iurygregory | how does your ironic.conf looks like? | 15:55 |
| iurygregory | ok nvm I thought we had something for nodes that were off.. send_sensor_data_for_undeployed_nodes | 15:56 |
| * TheJulia backs away from the unit test run after changing common/policy.py and a successful test run | 15:56 | |
| iurygregory | but it's for undeployed nodes | 15:57 |
| * iurygregory brb lunch time | 15:58 | |
| Kirill_ | oh, from config /etc/kolla/ironic-conductor/ironic.conf:send_sensor_data_for_undeployed_nodes = true. | 15:59 |
| Kirill_ | but another question. is it a good behavior - if bm is switched of and we set true we catch such error | 16:01 |
| TheJulia | ... I feel like the sensor *should* still be reporting | 16:01 |
| TheJulia | but... I guess maybe they don't on that make/model? | 16:01 |
| TheJulia | maybe it makes sense to just suppress? | 16:02 |
| TheJulia | or make it configurable? | 16:02 |
| iurygregory | yeah, since is ipmi it would depend on the HW vendor how the data looks like I think | 16:03 |
| iurygregory | suppress/make configurable makes sense | 16:03 |
| rpittau | good night! o/ | 16:13 |
| Kirill_ | what do you mean 'make it configurable', add list of sensors in config? | 16:13 |
| Kirill_ | what do you think: do we need adding 'check on None+debug logging(node_id)' enstead of catching such exception. | 16:16 |
| TheJulia | debug logging each node might be problematic if the same behavior is in say... a thousand nodes | 16:22 |
| TheJulia | but also just putting a catch for keyerror might not be ideal | 16:22 |
| TheJulia | but... maybe that *is* the right thing | 16:22 |
| TheJulia | If there is no value, there is no value | 16:22 |
| TheJulia | and it should all be driven what is in the ipmi sdr output | 16:23 |
| TheJulia | goodnight rpittau | 16:28 |
| TheJulia | Kirill_: can you share the entire exception please? | 16:29 |
| Kirill_ | ERROR ironic_prometheus_exporter.parsers.ipmi Traceback (most recent call last): | 16:32 |
| Kirill_ | 2022-12-28 16:31:13.625 7 ERROR ironic_prometheus_exporter.parsers.ipmi File "/var/lib/kolla/venv/lib/python3.6/site-packages/ironic_prometheus_exporter/parsers/ipmi.py", line 174, in extract_labels | 16:32 |
| Kirill_ | 2022-12-28 16:31:13.625 7 ERROR ironic_prometheus_exporter.parsers.ipmi entity_id = category_info['data'][entry]['Entity ID'] | 16:32 |
| Kirill_ | 2022-12-28 16:31:13.625 7 ERROR ironic_prometheus_exporter.parsers.ipmi KeyError: 'Entity ID' | 16:32 |
| TheJulia | Okay, that is what I thought | 16:33 |
| TheJulia | perhaps just independently catch KeyError ? | 16:34 |
| TheJulia | so yeah, I'm working on a patch to add "service" scope support | 16:36 |
| TheJulia | and... yeah. Definitely need to do a presentation on this | 16:36 |
| Kirill_ | yes, sounds good with independently catch, but what should we do in catch blok?) log, or just skip it | 16:44 |
| TheJulia | I suspect we would just skip it | 16:54 |
| TheJulia | we have no data to process, thus no data to return | 16:54 |
| Kirill_ | ++ | 17:05 |
| Kirill_ | thanks | 17:05 |
| Kirill_ | i've got another question:https://review.opendev.org/c/openstack/ironic-specs/+/866537 does it have enough approves or we need some more? | 17:06 |
| TheJulia | Kirill_: replied to your two questions and workflowed it | 17:15 |
| TheJulia | congrats! | 17:15 |
| Kirill_ | thanks!) | 17:15 |
| opendevreview | Merged openstack/ironic-specs master: new spec: support of vnc console. https://review.opendev.org/c/openstack/ironic-specs/+/866537 | 17:37 |
| TheJulia | iurygregory: by chance, did you hear back? | 19:38 |
| iurygregory | TheJulia, not yet =( if I don't hear anything by tomorrow morning I will reach out to someone else (sorry I was at the gym when you sent the message) | 21:25 |
| TheJulia | no worries, thanks! | 21:26 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add `service` role RBAC policy support https://review.opendev.org/c/openstack/ironic/+/869614 | 23:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!