| opendevreview | Merged openstack/ironic master: [trivial] remove note on non-voting snmp job https://review.opendev.org/c/openstack/ironic/+/902358 | 00:09 |
|---|---|---|
| iurygregory | JayF, I'm around if you still need | 00:48 |
| opendevreview | paresh proposed openstack/ironic master: [WIP] Fixes Raid creation in iLO6 https://review.opendev.org/c/openstack/ironic/+/902744 | 06:18 |
| *** MikeCTZA_ is now known as MikeCTZA | 07:04 | |
| rpittau | good morning ironic! o/ | 07:43 |
| n0ob | perhaps not the perfect channel (feel free to recommend a better one): are redfish collections supposed to have an "Id" entry? Chap 3.1 of the spec says "Id | 09:12 |
| n0ob | " is required but "Id" is missing in the sushy-emulator collections. | 09:13 |
| dtantsur | n0ob: probably an oversight on our side | 09:16 |
| n0ob | dtantsur: unclear, dell seems to be ommitting them too | 09:20 |
| dtantsur | "c" in redfish stands for "consistency" :) | 09:21 |
| n0ob | dtantsur: lol this made my day | 09:21 |
| iurygregory | good morning Ironic | 10:01 |
| iurygregory | dtantsur, tks for the review in the retry patch o/ going to update | 10:37 |
| iurygregory | now I probably have enough caffeine to start coding | 10:37 |
| dtantsur | JayF: I agree (re ironic-unmaintained-core), feel free to add me. | 10:41 |
| iurygregory | dtantsur, do you think it makes sense to handle the 500 errors as part of requests.exceptions.HTTPError ? https://paste.opendev.org/show/biCW67GObdN17iMgNtgI/ or should we specify each type of exception in the tenacity decorator and check status_code to match the http code and raise the right exception? | 10:58 |
| dtantsur | iurygregory: the paste looks good to me, I'd not try to enumerate all errors. | 10:59 |
| iurygregory | dtantsur, I was trying that locally and after sometime I was like waitttt =) | 10:59 |
| iurygregory | time to start my 3rd coffee =) | 11:00 |
| rpittau | JayF: feel free to add me to the ironic-unmaintained-core group | 11:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2023.2: Handle internal server errors while configuring secure boot https://review.opendev.org/c/openstack/ironic/+/902674 | 12:23 |
| opendevreview | Damien RANNOU proposed openstack/ironic-python-agent master: [WIP] USB device discovery https://review.opendev.org/c/openstack/ironic-python-agent/+/902769 | 12:54 |
| opendevreview | Merged openstack/ironic master: Fix log message var reference https://review.opendev.org/c/openstack/ironic/+/902492 | 13:00 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2023.2: Fix log message var reference https://review.opendev.org/c/openstack/ironic/+/902675 | 13:21 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2023.1: Fix log message var reference https://review.opendev.org/c/openstack/ironic/+/902676 | 13:22 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Retry in ProxyError during post inspector data https://review.opendev.org/c/openstack/ironic-python-agent/+/902448 | 13:24 |
| iurygregory | done =) | 13:24 |
| iurygregory | dtantsur, I have some good news regarding https://bugs.launchpad.net/sushy-tools/+bug/2039458 I will be mentoring a person that will work on this ^^ | 13:25 |
| dtantsur | w000t! | 14:19 |
| opendevreview | Merged openstack/sushy-tools master: Use existing nvram path in libvirt driver to support session daemon mode https://review.opendev.org/c/openstack/sushy-tools/+/902418 | 14:31 |
| *** dking is now known as Guest9420 | 14:33 | |
| *** Guest9420 is now known as dking | 14:33 | |
| opendevreview | Damien RANNOU proposed openstack/ironic-python-agent master: [WIP] USB device discovery https://review.opendev.org/c/openstack/ironic-python-agent/+/902769 | 14:42 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Retry in ProxyError during post inspector data https://review.opendev.org/c/openstack/ironic-python-agent/+/902448 | 15:02 |
| iurygregory | dtantsur, done ^ tks for the review o/ | 15:02 |
| dking | Good morning. For the metal3 guys, I'm back to having the problem with BMO repopulating Ironic. The logs that I get seem to only show that the non existent nodes cannot be found. It's been like that overnight currently. Does anybody know what process should be repopulating Ironic so that I can begin troubleshooting? | 15:12 |
| dtantsur | TheJulia: I'm not fully aware of the context behind https://opendev.org/openstack/ironic-lib/src/branch/master/ironic_lib/json_rpc/client.py#L93-L95 but this statement seems quite false to me | 15:20 |
| dtantsur | dking: https://github.com/metal3-io/baremetal-operator/blob/63abe2db90306baa805680ec391b055d9fe5be31/pkg/provisioner/ironic/ironic.go#L351-L438 | 15:21 |
| TheJulia | Look at the default topic/name code in topic creation. Anyway, I’ll be back Monday. | 15:22 |
| dtantsur | TheJulia: I think it's <topic>.<host> where <host> can have many dots | 15:22 |
| dtantsur | anyway, talk to you then. | 15:22 |
| TheJulia | Host name not fqdn | 15:23 |
| dtantsur | Does not really matter. Whatever gethostname gives us. | 15:23 |
| dtantsur | Or an IP address (back to this comment). | 15:24 |
| TheJulia | Yeah, local host or host name as the first of a fqdn | 15:24 |
| TheJulia | Anyway, family stuffs | 15:24 |
| dtantsur | Good luck! I'll play with it and email you my findings so that you don't see my pings here :) | 15:25 |
| dking | dtantsur: Thank you. That should give me a place to start. | 15:25 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Documentation: tuning worker threads https://review.opendev.org/c/openstack/ironic/+/902783 | 15:34 |
| opendevreview | Merged openstack/ironic stable/2023.2: Handle internal server errors while configuring secure boot https://review.opendev.org/c/openstack/ironic/+/902674 | 15:36 |
| JayF | dtantsur: rpittau: iurygregory: For purposes of good commnication, I'm going to email the list about that once the TC email hits, and have you all assent there. Since we hit a quick consensus on "Ironic wants a separate group", I won't stress about it too much ahead of time. | 16:00 |
| iurygregory | JayF, awesome tks! | 16:10 |
| JayF | Rosmaita is taking care of the logistics of making the group | 16:11 |
| JayF | he was looking for an example change to point to, we'll be the example | 16:12 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix GET for conductors with a port or IPv6 address https://review.opendev.org/c/openstack/ironic/+/902790 | 16:13 |
| dtantsur | This is a funny one ^^^ | 16:13 |
| dtantsur | and speaking of funny patches, could I get another +2 on https://review.opendev.org/c/openstack/ironic-lib/+/901863 please | 16:14 |
| iurygregory | will look in a few, going for lunch | 16:15 |
| JayF | https://review.opendev.org/c/openstack/sushy/+/902028 this needs 1 more +2A | 16:17 |
| rpittau | that's a nice oslo_utils function | 16:18 |
| * dtantsur nods | 16:21 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-lib master: Trivial: remove an incorrect comment https://review.opendev.org/c/openstack/ironic-lib/+/902793 | 16:26 |
| rpittau | good night! o/ | 17:08 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Add a reserved workers pool https://review.opendev.org/c/openstack/ironic/+/902801 | 17:18 |
| opendevreview | Merged openstack/sushy stable/zed: Remove version field from iLO error https://review.opendev.org/c/openstack/sushy/+/902028 | 18:01 |
| opendevreview | Merged openstack/ironic-lib master: JSON RPC: add support for unix sockets and extract common code https://review.opendev.org/c/openstack/ironic-lib/+/901863 | 18:18 |
| opendevreview | Jay Faulkner proposed openstack/sushy stable/yoga: Remove version field from iLO error https://review.opendev.org/c/openstack/sushy/+/902827 | 18:59 |
| JayF | ^ was abandoned, irrelevent for yoga | 19:02 |
| opendevreview | Merged openstack/ironic-lib master: Trivial: remove an incorrect comment https://review.opendev.org/c/openstack/ironic-lib/+/902793 | 19:27 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily to restart it onto an updated replication key | 21:36 | |
| JayF | Anyone interested in doing some a bug triage party sometime tomorrow? | 21:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!