rpittau | good morning ironic! Happy Friday! o/ | 06:57 |
---|---|---|
rpittau | when any core has a moment please review https://review.opendev.org/c/openstack/virtualpdu/+/922372, planning to release virtualpdu after that, thanks! | 08:01 |
dtantsur | rpittau: +2 | 10:33 |
sylvr | Hello! I'm having an issue with my IPA, it can't contact the Seed/Bifrost after/during introspection | 10:56 |
rpittau | thanks dtantsur | 11:12 |
sylvr | sylvr: it seems that the callback isn't working properly, as the node is looking up itself in Bifrost with what seems to me like wrong infos | 11:27 |
sylvr | `Looking up node with adresses "<MAC1>,<MAC2>" and UUID None at https://192.168.1.2:6385` this seems wrong has it uses both interfaces MAC adress and "UUID None" | 11:29 |
sylvr | I don't see port 6385 as open on my Seed-Bifrost either | 11:43 |
iurygregory | good morning ironic | 11:47 |
iurygregory | rpittau, looking now | 11:47 |
sylvr | How can I fully format Bifrost database? so that deleted nodes can be enrolled/discovered again ? | 12:10 |
opendevreview | Merged openstack/virtualpdu master: Update to match latest development cycle https://review.opendev.org/c/openstack/virtualpdu/+/922372 | 12:23 |
cid | Hey sylvr, I wish I knew how to go about what you were asking. | 12:33 |
cid | Everyone seems to not be on seat yet or occupied at the moment; you'll get help. | 12:34 |
cid | And since you know exactly what you are trying to achieve, it's becomes easier to Google those. | 12:36 |
iurygregory | just double checking but when doing VIF attach the option params to send are port_uuid / portgroup_uuid instead of port_id / portgroup_id righ ? https://docs.openstack.org/api-ref/baremetal/#attach-a-vif-to-a-node | 12:38 |
JayF | fwiw, the behavior sylvr was seeing was mostly expected in terms of the log | 14:07 |
JayF | we send the macs from the node to the API to do a lookup | 14:07 |
JayF | so the IPA doesn't know what node it's on until *after* it gets a response | 14:07 |
TheJulia | Yeah, unless it was booted via virtual media | 14:21 |
dtantsur | Even then - although that's something we should fix | 14:22 |
TheJulia | the fact we end up normally logging "None" in common cases? | 14:22 |
dtantsur | I think so | 14:22 |
TheJulia | yeah, it is a super simple change to make | 14:23 |
JayF | that's a pretty good perspective on it I hadn't considered | 14:29 |
dtantsur | We really need to pass node UUID through virtual media. It's not hard to do, just someone to sit down and do both sides of the equation. | 14:31 |
TheJulia | I thought we already did | 14:31 |
TheJulia | not doing so I think would be a bug at this point | 14:31 |
dtantsur | I don't disagree :) | 14:31 |
rpittau | bye everyone have a great weekend! o/ | 14:51 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Fix rendering of Redfish properties in the documentation https://review.opendev.org/c/openstack/ironic/+/922482 | 15:01 |
opendevreview | Jacob Anders proposed openstack/sushy-tools master: [WIP] Add support for BIOS update emulation https://review.opendev.org/c/openstack/sushy-tools/+/909500 | 15:05 |
opendevreview | cid proposed openstack/ironic master: Allow disabling bios deployments https://review.opendev.org/c/openstack/ironic/+/922243 | 15:22 |
opendevreview | cid proposed openstack/ironic master: Allow disabling bios deployments https://review.opendev.org/c/openstack/ironic/+/922243 | 15:23 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Reorganize the documentation front page https://review.opendev.org/c/openstack/ironic/+/922484 | 15:40 |
dtantsur | This ^^^ is pretty bold, and I wonder how much disagreement I'm going to see | 15:40 |
dtantsur | it's largely inspired by the feedback that Reverbverbverb gave us, although it only makes a tiny step towards the end goal | 15:41 |
dtantsur | I do think it's a step *towards* it at least | 15:41 |
dtantsur | cc JayF | 15:41 |
TheJulia | I feel super bad I've not had a chance to look at any of it | 15:42 |
TheJulia | but getting pulled in numerous directions right now | 15:42 |
dtantsur | I can imagine | 15:42 |
dtantsur | It's not a long read actually, so may qualify for the Friday evening read | 15:42 |
opendevreview | cid proposed openstack/ironic master: Allow disabling bios deployments and/or enrollments https://review.opendev.org/c/openstack/ironic/+/922243 | 16:50 |
opendevreview | cid proposed openstack/ironic master: Flexible IPMI credential persistence method configuration https://review.opendev.org/c/openstack/ironic/+/917229 | 17:12 |
JayF | dtantsur: I pointed Reverbverbverb to that change downstream in slack | 17:25 |
* TheJulia goes back to therapy change | 17:48 | |
opendevreview | Julia Kreger proposed openstack/ironic master: Remove deprecated idrac wsman driver interfaces https://review.opendev.org/c/openstack/ironic/+/922340 | 18:20 |
TheJulia | Ahh, Therapy | 18:20 |
opendevreview | cid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype] https://review.opendev.org/c/openstack/ironic/+/922142 | 19:39 |
opendevreview | cid proposed openstack/ironic master: WIP: Self-Service via Runbooks [Prototype] https://review.opendev.org/c/openstack/ironic/+/922142 | 19:41 |
cid | o// | 19:42 |
JayF | o/ | 19:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!