| TheJulia | cid: sorry for the delay in responding, juggling a few things today. Generally just achieving the needful is the most important aspect, the exact how is less important as long as the overall goal is achieved | 01:03 |
|---|---|---|
| cid | TheJulia: Understood! | 04:39 |
| cid | Of course, I understand everyone is busy 😂 | 04:47 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent bugfix/9.12: Update .gitreview for bugfix/9.12 https://review.opendev.org/c/openstack/ironic-python-agent/+/921431 | 09:54 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector bugfix/12.2: Update .gitreview for bugfix/12.2 https://review.opendev.org/c/openstack/ironic-inspector/+/921432 | 10:10 |
| iurygregory | good morning | 11:11 |
| JayF | dtantsur: iurygregory: did you want a chance to look at https://review.opendev.org/c/openstack/ironic-specs/+/890164 before it workflows (runbooks) | 11:59 |
| iurygregory | JayF, I did, totally forgot to add my vote on it, my bad | 12:00 |
| JayF | not a problem, it's obvious you all are busy downstream so I wanted to make sure nothing "snuck by" so to speak | 12:02 |
| JayF | especially Dmitry tends to have strong opinions on API things (which is a good thing!!) | 12:02 |
| dtantsur | I'll take a very brief look once the current meeting ends | 12:03 |
| iurygregory | JayF, yeah.. customer cases, bugs, testing.. =( | 12:03 |
| dtantsur | MEETINGS | 12:05 |
| * dtantsur has had one more "please don't rewrite ironic in go, you're going to hate it" sort of a conversation | 12:06 | |
| iurygregory | omg | 12:06 |
| iurygregory | I'm happy I'm not in this type of meetings | 12:07 |
| dtantsur | JayF: more of a food for thought.. do you envision that less privileged users will be able to see the list of steps? | 12:14 |
| dtantsur | like, is it possible that the list of steps is something you don't want the lessees to see even if they can use the runbook | 12:15 |
| dtantsur | (W+1 in the meantime) | 12:15 |
| opendevreview | cid proposed openstack/networking-generic-switch master: Add VLAN and Port whitelist https://review.opendev.org/c/openstack/networking-generic-switch/+/888047 | 12:26 |
| iurygregory | we still have mentions for whitelist? O.o | 12:27 |
| opendevreview | Merged openstack/ironic-specs master: Self-Service via Runbooks https://review.opendev.org/c/openstack/ironic-specs/+/890164 | 12:30 |
| TheJulia | yeah, the term usage needs to be fixed | 12:38 |
| JayF | dtantsur: I think it'll be a separate policy knob (naturally, because it's a separate CRUD endpoint from set_provision_state), but I would find that weird for a default configuration | 12:58 |
| JayF | iurygregory: the draft of the docs review identified a lot of non-inclusive language | 12:59 |
| JayF | iurygregory: I suggest you provide that review feedback on that change (it's actually mine brought up from downstream so I'll take the blame) | 12:59 |
| iurygregory | JayF, no need to take the blame =) | 13:00 |
| iurygregory | it's no ones fault =) | 13:01 |
| opendevreview | Merged openstack/ironic master: Excise use of non-timezone aware utcnow() https://review.opendev.org/c/openstack/ironic/+/920965 | 13:14 |
| opendevreview | Merged openstack/ironic master: Assert URL consistency for agent_url https://review.opendev.org/c/openstack/ironic/+/921294 | 13:14 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP/DNM: initial mercury work https://review.opendev.org/c/openstack/ironic/+/921458 | 15:46 |
| TheJulia | dtantsur: ^^^ was totally just initial hacking on stuff, nowhere near anything review ready | 16:04 |
| dtantsur | yeah, I stopped my curiosity after realizing this :) | 16:04 |
| TheJulia | I just wanted it in gerrit since It was on my desktop | 16:04 |
| TheJulia | and I'm going up into the mountains :) | 16:04 |
| dtantsur | mountains \o/ | 16:05 |
| dtantsur | I hope you enjoy it | 16:09 |
| TheJulia | I am sure I will, it should be cooler up there | 16:10 |
| dtantsur | meanwhile, we don't have any docs on hardware metrics or IPE, do we? | 16:10 |
| dtantsur | ah, they are called notifications | 16:14 |
| dtantsur | mm, no, notifications is a different beast | 16:19 |
| dtantsur | iurygregory: as the father of IPE, do you have any idea ^^? | 16:21 |
| TheJulia | Well, IPE was mainly to match what the conductor gathered which is driver specific | 16:22 |
| TheJulia | and the redfish driver does *selective* collection | 16:22 |
| TheJulia | becuase each resource and all need separate GET requests | 16:22 |
| dtantsur | yeah, I've seen this code while working on the interop profile | 16:23 |
| dtantsur | I guess now I also need to write a redfish documentation for it | 16:23 |
| iurygregory | dtantsur, reading | 16:23 |
| iurygregory | I think I added in some place the metrics collected... | 16:24 |
| dtantsur | TheJulia: we need to look into newer Redfish, I think it now features to $include subresources | 16:28 |
| dtantsur | "With $levels , a single GET request can include the subordinate resource collections, such as Processors and | 16:30 |
| dtantsur | Memory . | 16:30 |
| dtantsur | exciting stuff, we need to use it | 16:30 |
| TheJulia | oh cool, so that would be much lower overhead to collect data | 16:30 |
| TheJulia | ++ | 16:30 |
| TheJulia | okay, really really really going up the moutnain now | 16:30 |
| TheJulia | bbiab | 16:30 |
| iurygregory | ok, I was sure we had a file in the bifrost jobs with the IPE metrics ... | 16:30 |
| iurygregory | and I can't find <O> | 16:30 |
| dtantsur | Could someone remind me, why did we decide for IPE to be a separate project rather than a part of Ironic? Some dependencies? | 16:38 |
| dtantsur | iurygregory: IPE has some useful docstrings but nothing public.. | 16:39 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Testing https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/921463 | 16:44 |
| iurygregory | dtantsur, so the IPE is only a oslo_ messaging_notifications driver that can parse the data we have from sensor data in ironic | 16:46 |
| iurygregory | I think this was the reason | 16:47 |
| dtantsur | We definitely lack a "How it works" page as well | 16:48 |
| dtantsur | Since now I'm wondering how easier everything would be if Ironic just had a Prometheus endpoint | 16:48 |
| iurygregory | How it works https://www.youtube.com/watch?v=fnDxxHa9nW8 :D | 16:49 |
| iurygregory | 5min:30s | 16:50 |
| dtantsur | that millennials with their tiktoks!! | 16:50 |
| * dtantsur old man yelling at cloud | 16:50 | |
| iurygregory | gen Z and X, millennials don't use tiktoks! | 16:51 |
| * dtantsur cries in millennial | 16:51 | |
| dtantsur | seriously though, we need this written, even if as a few paragraphs and a diagram | 16:51 |
| iurygregory | I'm also a millennial... | 16:52 |
| iurygregory | dtantsur, sure I will work on it | 16:52 |
| iurygregory | I have doc appointment in my afternoon, so tomorrow I will try to work on this | 16:52 |
| dtantsur | Thanks! | 16:52 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-prometheus-exporter master: Trivial: log traceback on an unexpected error https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/921465 | 16:53 |
| dtantsur | iurygregory: so, we write many-many files and then merge them as a /metrics endpoint, right? | 16:53 |
| dtantsur | i wonder if we ever purge any files or this stuff just grows indefinitely | 16:53 |
| iurygregory | dtantsur, we write 1 file per baremetal node | 16:54 |
| iurygregory | humm if the node is deleted, maybe..., I never tested this! | 16:54 |
| dtantsur | ah, right | 16:54 |
| iurygregory | <facepalm> | 16:54 |
| TheJulia | and back | 18:19 |
| cid | o/ | 20:31 |
| iurygregory | oh ok, seems like only in IPE job i can see the metrics XD https://zuul.opendev.org/t/openstack/build/299d1263290f4f78af57a2f18285f67b/log/controller/logs/ipe/node-0-hardware.redfish.metrics | 21:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!