| *** akahat|rover is now known as akahat | 05:12 | |
| opendevreview | kamlesh chauvhan proposed openstack/sushy-oem-idrac master: fix tox4 https://review.opendev.org/c/openstack/sushy-oem-idrac/+/869441 | 09:29 |
|---|---|---|
| opendevreview | Merged openstack/sushy-oem-idrac master: fix tox4 https://review.opendev.org/c/openstack/sushy-oem-idrac/+/869441 | 09:46 |
| opendevreview | Merged openstack/sushy-oem-idrac master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859266 | 09:58 |
| opendevreview | kamlesh chauvhan proposed openstack/sushy-oem-idrac stable/zed: fix tox4 https://review.opendev.org/c/openstack/sushy-oem-idrac/+/869450 | 10:01 |
| opendevreview | Merged openstack/sushy-oem-idrac stable/zed: fix tox4 https://review.opendev.org/c/openstack/sushy-oem-idrac/+/869450 | 10:27 |
| opendevreview | Merged openstack/sushy-oem-idrac stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859264 | 10:46 |
| opendevreview | Merged openstack/sushy-oem-idrac stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859265 | 10:46 |
| iurygregory | good morning Ironic | 11:41 |
| opendevreview | Merged openstack/ironic master: Enable alternative storage for inventory https://review.opendev.org/c/openstack/ironic/+/866056 | 12:00 |
| TheJulia | good morning | 14:39 |
| opendevreview | Julia Kreger proposed openstack/ironic-lib master: Provide an interface to store metrics https://review.opendev.org/c/openstack/ironic-lib/+/865311 | 16:50 |
| * TheJulia tries to grok the promethus library | 17:32 | |
| TheJulia | iurygregory: o/ | 17:48 |
| TheJulia | So I'm curious, is the reason everything in ipe is gauge because we didn't want to override counter structures, or was all of the data just gauges instead of counters? | 17:49 |
| JayF | TheJulia: kubajj: There's only like, 10 weeks until release. Where are we with inspector migration? It's kinda at a middle point, yeah? Do we need to apply more focus to that (I can maybe help?) to ensure we get to a happy place? | 19:37 |
| TheJulia | my impression is we're going to have to carry inspector for at least another cycle | 19:38 |
| JayF | Meaning, until C now, right? | 19:39 |
| TheJulia | yeah | 19:41 |
| TheJulia | until we have an api side wired in on ironic to support it, we can tell people to switch over | 19:42 |
| JayF | Have we completed any user facing features (yet) for this release? Nothing significant, right? | 19:45 |
| JayF | I wouldn't even count sharding because it's not really user-facing until nova/net-bm consumes it | 19:46 |
| TheJulia | we should detail that it exists in advance, but last I looked the API patch was still awaiting revision | 19:52 |
| TheJulia | https://docs.openstack.org/releasenotes/ironic/unreleased.html | 19:53 |
| TheJulia | :( | 19:53 |
| JayF | yeah, sharding was sorta swamped by all the other keep-the-lights-on shenanigans | 19:53 |
| JayF | (and tbh putting together conference cfp stuff downstream) | 19:54 |
| TheJulia | yeah | 19:54 |
| TheJulia | https://www.irccloud.com/pastebin/ZOxEiTKg/ | 20:31 |
| TheJulia | iurygregory: JayF ^^ thoughts? concerns? shortened names to make it less insane and more relatable given service=ironic is on everything with the prometheus modeling of data | 20:31 |
| iurygregory | TheJulia, the past lgtm | 20:33 |
| iurygregory | paste* | 20:34 |
| TheJulia | descriptions could be better I guess, but I dont want to code a ton of knowledge into this thing | 20:34 |
| iurygregory | I just need to double check the rules they have in prometheus for name formats.. I'm wondering if having ConductorManager is ok or not, I can double check on monday with some monitoring folks about it | 20:35 |
| TheJulia | ++ | 20:35 |
| TheJulia | please, that would be much appreciated | 20:36 |
| TheJulia | It is an artifact of some of our coding structure | 20:36 |
| TheJulia | and class construciton | 20:36 |
| iurygregory | just added a not here in my TODO list | 20:36 |
| TheJulia | but we can always go "oh, we know about this" | 20:36 |
| iurygregory | yeah =) | 20:37 |
| * iurygregory needs to finish the talk proposal to submit during the weekend XD | 20:38 | |
| TheJulia | enjoy | 20:39 |
| TheJulia | I'm going to work towards a stopping point on this for now, tests next week | 20:39 |
| opendevreview | Julia Kreger proposed openstack/ironic-prometheus-exporter master: WIP: Support extraction of ironic internal metrics https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/869509 | 20:44 |
| TheJulia | iurygregory: ^^ | 20:44 |
| TheJulia | oh | 20:45 |
| TheJulia | need to fix dependency info | 20:45 |
| JayF | that seems OK to me TheJulia | 20:45 |
| iurygregory | TheJulia, ack o/ | 20:45 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Get conductor metric data https://review.opendev.org/c/openstack/ironic/+/865447 | 20:47 |
| opendevreview | Julia Kreger proposed openstack/ironic-prometheus-exporter master: WIP: Support extraction of ironic internal metrics https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/869509 | 20:47 |
| TheJulia | whee service role addition is in review | 20:54 |
| TheJulia | guess I might be able to hammer that out next week too | 20:54 |
| TheJulia | anyway, I need to step away, my eyes are starting to complain | 20:54 |
| JayF | TheJulia: https://review.opendev.org/c/openstack/ironic/+/854045 mind if I take this over the line? | 21:01 |
| TheJulia | Sure | 21:03 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Docs: Add considerations to anaconda docs https://review.opendev.org/c/openstack/ironic/+/854045 | 21:06 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Docs: Add considerations to anaconda docs https://review.opendev.org/c/openstack/ironic/+/854045 | 21:07 |
| JayF | ^^ is just a rebase of a docs change; should be a trivial review | 21:07 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: API support for CRUD node.shard https://review.opendev.org/c/openstack/ironic/+/866235 | 23:13 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: DB & Object layer for node.shard https://review.opendev.org/c/openstack/ironic/+/864236 | 23:15 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: API support for CRUD node.shard https://review.opendev.org/c/openstack/ironic/+/866235 | 23:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!