| dtantsur | JayF: pip install --user is unfortunately less well known, so some users will definitely miss that. For me it means more complex ansible playbooks, but I can live with that :) | 07:51 |
|---|---|---|
| rpittau | dtantsur: but --user is the default now, unless you run as root or sudo | 08:05 |
| rpittau | JayF: thanks for the bugs, the ironic-lib one seems quite important | 08:30 |
| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: Adding support to view indiv. cpu-core info https://review.opendev.org/c/openstack/ironic-python-agent/+/909346 | 08:38 |
| rpittau | JayF: this https://bugs.launchpad.net/ironic/+bug/2071741 looks reasonable to me, we can add it for discussion for the next meeting if we don't have approval during the week | 08:38 |
| opendevreview | Merged openstack/ironic master: Fixup: Move release note to proper directory https://review.opendev.org/c/openstack/ironic/+/923327 | 09:02 |
| rpittau | oh I know how this ^ happened, I ran reno under the ironic dir :D | 09:04 |
| cid | Sounds like something I could do :) | 09:14 |
| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: Trying awaitlet in IPA https://review.opendev.org/c/openstack/ironic-python-agent/+/923380 | 10:04 |
| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: Trying awaitlet in IPA https://review.opendev.org/c/openstack/ironic-python-agent/+/923380 | 10:09 |
| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: Trying awaitlet in IPA (Experimental/Not Working) https://review.opendev.org/c/openstack/ironic-python-agent/+/923380 | 10:09 |
| opendevreview | Merged openstack/ironic master: Add tox env for local dev; update documentation https://review.opendev.org/c/openstack/ironic/+/922348 | 10:18 |
| iurygregory | Schedule for the OIS Asia 24 is out https://2024.openinfraasia.org/a/schedule | 11:04 |
| iurygregory | good morning Ironic o/ | 11:04 |
| opendevreview | Peter Rudenko proposed openstack/ironic master: Nvidia Bluefield DPU redfish customization. https://review.opendev.org/c/openstack/ironic/+/923396 | 13:26 |
| iurygregory | <eyes> | 13:34 |
| dtantsur | TheJulia: ^^^ | 14:54 |
| opendevreview | Mark Goddard proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 15:26 |
| TheJulia | dtantsur: given how minimal that is, I'd almost rather just do a vendor match inside of the redfish hardware type | 16:00 |
| dtantsur | TheJulia: yeah, I'm thinking along the same lines. But also: using a forced reboot has implications. For instance, we don't know if the operation succeeds at all. | 16:00 |
| TheJulia | dtantsur: do we *really* know anyway | 16:01 |
| TheJulia | ... so, there is a counter we could check actually | 16:01 |
| TheJulia | but it *might* not be viable on that specific hardware | 16:01 |
| dtantsur | Right now we check that the node gets powered off, then powered on | 16:01 |
| TheJulia | yeah | 16:01 |
| dtantsur | At least we know it went through the cycle | 16:01 |
| TheJulia | yeah | 16:01 |
| TheJulia | ... ummmmm | 16:01 |
| dtantsur | I'm also curious about the motivation of the change: we also have explicit power off in the deploy interfaces | 16:02 |
| TheJulia | https://github.com/openstack/sushy/blob/master/sushy/resources/system/system.py#L67 should increment | 16:02 |
| dtantsur | if it's available at all | 16:03 |
| TheJulia | dtantsur: explicit hardware limitation, aiui | 16:03 |
| TheJulia | exactly | 16:03 |
| dtantsur | Right, hence my point: this is not the only place where we have explicit power off | 16:03 |
| dtantsur | In fact, I'm not sure how often Ironic itself uses reboot vs power on/off | 16:03 |
| TheJulia | agree | 16:03 |
| TheJulia | it *really* doesn't, but we need more details. That pesky "why" question | 16:04 |
| dtantsur | yep | 16:04 |
| TheJulia | NobodyCam: do you know a Peter Rudenko? | 16:05 |
| TheJulia | I'm wondering which part of nvidia this is coming from, the background context can likely help | 16:07 |
| TheJulia | since it would explain why | 16:07 |
| iurygregory | ++ | 16:18 |
| rpittau | good night! o/ | 16:25 |
| opendevreview | cid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype] https://review.opendev.org/c/openstack/ironic/+/922142 | 18:09 |
| opendevreview | Cyril Roelandt proposed openstack/sushy-tools master: Python 3.12: do not use ssl.wrap_socket https://review.opendev.org/c/openstack/sushy-tools/+/923442 | 19:44 |
| cid | o/ | 20:50 |
| opendevreview | Jacob Anders proposed openstack/sushy-tools master: Replace hardcoded BiosVersion with an updatable field https://review.opendev.org/c/openstack/sushy-tools/+/909487 | 21:17 |
| JayF | clearly I need to find the config for file paths and update the doc :D https://www.irccloud.com/pastebin/7YdN42kb/cache_cleanup_err | 21:50 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Require hashed passwords for rescue by default https://review.opendev.org/c/openstack/ironic/+/923452 | 22:54 |
| JayF | ^ trivial, just saw the note/todo while looking into the cache cleanup thing | 22:55 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Localdev: disable caching and related periodic https://review.opendev.org/c/openstack/ironic/+/923453 | 22:56 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Localdev: disable caching and related periodic https://review.opendev.org/c/openstack/ironic/+/923453 | 22:57 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP Direct HTTPBoot iso interface? https://review.opendev.org/c/openstack/ironic/+/921417 | 23:16 |
| TheJulia | That took me too long to realize I had left a stray comma on the end of a line | 23:17 |
| JayF | comma comma comma comma comma chameleon | 23:19 |
| JayF | I synced with cid today on Runbooks, even though that patch is labelled prototype it's getting pretty close | 23:19 |
| JayF | I think by the next revision it'll be helpful to have someone elses' eyes on it | 23:19 |
| JayF | my main question is that right now, it's end-to-end feature in one commit/PR | 23:19 |
| JayF | not sure if we want to break it up, but I'm leaning towards no, even though it's a beast to review | 23:19 |
| JayF | just something to think about | 23:20 |
| TheJulia | I should have time friday unless someone screams or downstream is on fire | 23:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!