| opendevreview | Jacob Anders proposed openstack/ironic master: Fix inherited periodic tasks being registered more than once https://review.opendev.org/c/openstack/ironic/+/1000406 | 03:31 |
|---|---|---|
| frickler | question mostly to native speakers: is "provision state" the same (shortened?) thing as "provisioning state"? for me, the latter sounds better and maybe should be used consistently e.g. in https://docs.openstack.org/ironic/latest/user/states.html, or would that just bloat the text with no improvement? | 09:09 |
| * dtantsur has the same confusion | 10:37 | |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix inherited periodic tasks being registered more than once https://review.opendev.org/c/openstack/ironic/+/1000406 | 10:56 |
| opendevreview | Merged openstack/python-ironicclient master: Drop unnecessary allowlist_externals https://review.opendev.org/c/openstack/python-ironicclient/+/999360 | 11:14 |
| opendevreview | Merged openstack/python-ironicclient master: ruff: Enable common checks https://review.opendev.org/c/openstack/python-ironicclient/+/999361 | 11:16 |
| opendevreview | Merged openstack/ironic bugfix/38.0: fix runbook create notifications https://review.opendev.org/c/openstack/ironic/+/1000497 | 11:18 |
| opendevreview | Merged openstack/ironic bugfix/37.0: fix runbook create notifications https://review.opendev.org/c/openstack/ironic/+/1000496 | 11:18 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix inherited periodic tasks being registered more than once https://review.opendev.org/c/openstack/ironic/+/1000406 | 11:52 |
| opendevreview | Esther Domfeh proposed openstack/python-ironicclient master: Add new fields to node history detailed view https://review.opendev.org/c/openstack/python-ironicclient/+/1000705 | 12:22 |
| TheJulia | frickler: provisioning being active tense as if something is occurring at all times, where as provision state is static so... word perception and tense | 13:08 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: Add request context user tracking to node history records https://review.opendev.org/c/openstack/ironic/+/993901 | 13:45 |
| opendevreview | Esther Domfeh proposed openstack/ironic master: Add power tracking telemetry in node history https://review.opendev.org/c/openstack/ironic/+/996347 | 13:45 |
| TheJulia | ... one thing I've been wondering recently... for entirely selfish and fun evil ideas, if we need an endpoint for quick index counters | 13:52 |
| dtantsur | for what sorry? | 14:09 |
| JayF | /v1/nodes/count (?) | 14:12 |
| JayF | or /v1/count/nodes lol | 14:12 |
| JayF | like that sorta thing? similar to /v1/shards? | 14:12 |
| dtantsur | hmm /v1/stats? | 14:12 |
| TheJulia | counters in general | 14:14 |
| JayF | adamcarthur5: ^ might have some thoughts on this as the maintainer of openstack-exporter | 14:14 |
| TheJulia | I like the higher level endpoint idea, I was just wondering "how many nodes in what states" and could that be useful *without* walking the entire nodes stucture | 14:14 |
| TheJulia | that whole "closer to the db == more performant for targetted interactions" | 14:15 |
| cardoe | So that's what we've got some grafana dashboards for | 16:57 |
| cardoe | I think we utilize data from openstack-exporter for those. | 16:57 |
| cardoe | IMHO, we'd be better served by having a metrics endpoint that conforms to something that can just consume it nicely. | 16:58 |
| JayF | cardoe: yeah, my thought is work with adamcarthur5 to see if there's some data folks often want rolled up... then see if we can roll 'em up for the project | 17:20 |
| JayF | cardoe: TheJulia: we should make enough noise about this outside of Ironic contexts so if other projects have interest we can share a format | 17:20 |
| TheJulia | Agree 100% | 17:23 |
| TheJulia | I think the key question will be the what, and worst comes to worst we could always wire our prometheus exporter with some stuff so it can collect them, but I can also see a lot of usefulness in intermedary dashboards/interactions/monitoring to be able to curl for aggregates views as well | 17:24 |
| JayF | I know vexxhost has been pulling at this string to, developing a DB-slurping alternative to the openstack-exporter (which gets data through API scraping) ... but generally there's just perf left on the table | 17:24 |
| TheJulia | today, we short of shovel that into the exporter, but thats also conductor runtime counts | 17:25 |
| TheJulia | so, number of times this thing has executed sort of thing on this conductor | 17:25 |
| TheJulia | mnaser: You might have thoughts/ideas/love/hate the last 20 or so lines ;) | 17:25 |
| TheJulia | and point the discussion to someone else of course | 17:25 |
| adamcarthur5 | I'm not 100% sure what the biggest bottlenecks in ironic metric scraping are at the moment, but I'll look into it. If it makes sense I'm happy to work on things ironic-side to make metrics better | 18:02 |
| TheJulia | Well, I guess more to the point, we already know scraping the API surface itself is super overhead intensive if your just hunting down some aggregate counts | 18:03 |
| TheJulia | so if there are super useful aggregate counts to surface, then it would be worthwhile to try and do them because doing count on indexed columns is just *WAY* faster :) | 18:04 |
| cardoe | JayF: Rackspace has had the DB-slurping since you were there I think. | 19:08 |
| cardoe | I don't know if I'd use the aggregate counts though. Cause people wanna drill down always. | 19:09 |
| cardoe | e.g. I have a dashboard in Grafana which has all my nodes and their status. But there's also all nodes and their resource_class | 19:10 |
| cardoe | I click on a given resource_class and now my view of the statuses changes to just those machines. | 19:11 |
| cardoe | https://github.com/openstack-exporter/openstack-exporter/blob/099c51a03d038758663f5a7101895dc4b46d2800/exporters/ironic.go#L22 | 19:13 |
| cardoe | https://github.com/openstack-exporter/openstack-exporter/pull/536 is a refactor that would be good for this | 19:14 |
| cardoe | but honestly it's a big chunky change... I'd really like to see that refactored into the plumbing and then per service | 19:14 |
| cardoe | cause it's just not something any one person can test. | 19:15 |
| cardoe | Like I could test the services I use and +1 those changes. And someone else could test the services they use and +1 that change. | 19:15 |
| cardoe | That'd be easier to review. | 19:15 |
| opendevreview | Merged openstack/ironic master: Document ansible deploy interface SSH host key checking https://review.opendev.org/c/openstack/ironic/+/1000023 | 20:21 |
| cardoe | alright looking for some feedback from folks | 20:40 |
| cardoe | I've got a plugin for OSC which I'm debating if it'll be upstream worthy | 20:40 |
| cardoe | I've added "openstack cloud" which operates a lot like "kubectl ctx" and "kubectl config" | 20:41 |
| cardoe | I've added a context.yaml next to clouds.yaml and you no longer need to set OS_CLOUD since it'll record it in there | 20:42 |
| cardoe | You can do "openstack cloud list" and it'll show you all the configs you've got in clouds.yaml and show you the current context | 20:42 |
| JayF | cardoe: 🎶 an openstack supernova 🎶 | 20:43 |
| rm_work | that is pretty cool TBH 😛 | 20:43 |
| cardoe | JayF: shhhhh you might trigger people | 20:43 |
| rm_work | though I think I did something similar with bashrc functions 😛 | 20:44 |
| rm_work | would be nice for it to be official | 20:44 |
| cardoe | I'll propose it. | 20:46 |
| cardoe | I've got a bunch of stuff that'll never be upstreamable but should be. | 20:46 |
| JayF | fwiw, I don't have opinions one way or the other | 20:47 |
| JayF | that's a better question for -sdks imo | 20:47 |
| cardoe | Well I'm asking regular users. | 20:48 |
| cardoe | Past UX improvements to OSC have resulted in suggestions that I use the alternate CLI written in another language. | 20:50 |
| JayF | I would suggest ignoring that feedback and appealing to the masses. I don't think the entire OpenStack SDK team would agree with that. | 20:51 |
| TheJulia | brraaaains | 21:01 |
| TheJulia | anyone seen my brains?!? | 21:01 |
| TheJulia | cardoe: sounds like an awesome plugin, feel free to quote for indication of support | 21:02 |
| TheJulia | JayF: +1 "appealing to the massses" | 21:02 |
| TheJulia | my feline supervisor of darkness (i.e. not the single braincell'ed derpahaulic or his sister the troublemaking gray tabby) has decided it is time for post-work stress relief | 21:05 |
| opendevreview | Merged openstack/ironic master: Extend the RBAC policy matrix to steps https://review.opendev.org/c/openstack/ironic/+/992542 | 21:08 |
| opendevreview | Merged openstack/ironic master: Fix BIOS firmware update failure detection on Dell iDRAC https://review.opendev.org/c/openstack/ironic/+/997366 | 21:17 |
| opendevreview | Merged openstack/ironic master: Add TLS configuration to agent lookup response https://review.opendev.org/c/openstack/ironic/+/987887 | 21:27 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!