| rpittau | good morning ironic! o/ | 05:55 |
|---|---|---|
| rpittau | jfargen: the error you're seeing means that the schema of redfish applied by the firmware is the same as it was before, which is really odd, I would double check the version currently in use and try to verify the schema with some curl calls | 06:33 |
| rpittau | jfargen: basic bifrost configuration is pxe/bios/ipmi, if you need something different you'll have to configure it before running the installation, or modify the parameters in the config files | 06:45 |
| rpittau | jfargen: how the machines are enrolled/deployed depends on what's provided in the inventory, for example for redfish virtual media you will need to specify the redfish-virtual-media driver | 06:45 |
| rpittau | jfargen: since you have dell machines though the normal redfish driver probably won't work, that may be another reason why you're seeing that error, you need to use idrac-redfish or idrac-redfish-virtual-media | 06:45 |
| rpittau | oh yeah, happy friday! :) | 06:51 |
| opendevreview | Merged openstack/ironic master: Fix retry logic logging https://review.opendev.org/c/openstack/ironic/+/889117 | 07:05 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Wait for secure boot state change if it's not immediate https://review.opendev.org/c/openstack/ironic/+/863999 | 08:55 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Wait for secure boot state change if it's not immediate https://review.opendev.org/c/openstack/ironic/+/889163 | 08:55 |
| janders | ^ trying to pick up Dmitry's change backfired a bit, gotta clean this up | 08:56 |
| samuelkunkel[m] | <rpittau> "jfargen: since you have dell..." <- We have many hundreds of DELL Machines (R640, R6525, C6420, C6525, R750). All of them work fine with the generic redfish driver | 09:40 |
| samuelkunkel[m] | jfargen: did you check that redfish is enabled in der idrac & does the user you use have enough permissions? | 09:41 |
| samuelkunkel[m] | (It should be enabled by default) | 09:41 |
| samuelkunkel[m] | I remember that we use pretty generous permissions | 09:41 |
| iurygregory | good morning Ironic | 11:35 |
| rpittau | samuelkunkel[m]: that's interesting, so you don't have idrac-redfish enabled in the ironic config? | 12:28 |
| samuelkunkel[m] | Not to my knowledge. The DELL Nodes have the exact same configuration as the HPE and Supermicro nodes | 12:30 |
| samuelkunkel[m] | And there is nothing specific enabled within the ironic configuration | 12:30 |
| rpittau | alright, and weird :) | 12:34 |
| samuelkunkel[m] | If you run the redfish ironic profile compatibility check (from the redfish dmtf repo) against the idracs its happy | 12:39 |
| samuelkunkel[m] | So from that I assume it supports everything ironic expects | 12:39 |
| samuelkunkel[m] | And I assume the ironic redfish profile is taking only the generic redfish settings / options into account | 12:40 |
| jfargen | Adding boot_interface to the inventory would be more idea. | 13:20 |
| jfargen | "boot_interface": "redfish-virtual-media", | 13:21 |
| jfargen | Something like that? | 13:21 |
| rpittau | jfargen: that's on the node side, do you also have that enabled in the ironic config ? | 13:22 |
| jfargen | This is the first time I have looked at bifrost. | 13:25 |
| jfargen | enabled_hardware_types: "ipmi,redfish,manual-management,ilo" | 13:25 |
| jfargen | Not sure if there are other parameters. | 13:26 |
| rpittau | that should be enough to have generic redfish, including redfish-virtual-media | 13:30 |
| rpittau | I would add idrac too, just to be sure | 13:32 |
| TheJulia | We should likely add bmw dis stuffs to bifrost’s defaults | 13:32 |
| rpittau | it's ckind of funny that we have ilo in the defaults but no idrac | 13:33 |
| jfargen | From /etc/ironic/ironic.conf I see this as well. enabled_boot_interfaces = ipxe,pxe,redfish-virtual-media | 13:33 |
| ashinclouds[m] | I was at hpe when we started bifrost | 13:33 |
| rpittau | jfargen: yep, that looks correct | 13:33 |
| rpittau | ashinclouds[m]: ah! | 13:33 |
| TheJulia | Ugh, I need to use that client more | 13:34 |
| TheJulia | Anyway, what my matrix client said :) | 13:34 |
| rpittau | :D | 13:34 |
| jfargen | Tried to set the node to provision state provide and it went into this cleaning state. | 13:35 |
| jfargen | Going to get some coffee. | 13:36 |
| TheJulia | jfargen: that is expected | 13:38 |
| opendevreview | Merged openstack/bifrost master: Correct JSON by adding missing comma https://review.opendev.org/c/openstack/bifrost/+/889107 | 14:01 |
| rpittau | bye everyone, have a great weekend! o/ | 14:30 |
| jfargen | provision state = cleaned failed | 14:52 |
| jfargen | what state should the node be in? | 14:53 |
| jfargen | $ baremetal node show 20000000-0000-0000-0000-000000000002 --fit-width | 14:54 |
| jfargen | +------------------------+-------------------------------------------------------------------------------------------------------------+ | 14:54 |
| jfargen | | Field | Value | | 14:54 |
| jfargen | +------------------------+-------------------------------------------------------------------------------------------------------------+ | 14:54 |
| jfargen | | allocation_uuid | None | | 14:54 |
| jfargen | | automated_clean | None | | 14:54 |
| jfargen | | bios_interface | redfish | | 14:54 |
| jfargen | | boot_interface | redfish-virtual-media | 14:54 |
| TheJulia | Ideally available, but something failed | 14:55 |
| TheJulia | Look at the last_error field | 14:55 |
| mohammed | Are there periodic runs for metal3 test on ironic builds that run on master branch ? If so how can I check on zuul only these master builds not changes builds ? | 15:46 |
| jfargen | | last_error | During sync_power_state, max retries exceeded for node 20000000-0000-0000-0000-000000000002, node state | | 15:49 |
| jfargen | | | power on does not match expected state 'power off'. Updating DB state to 'power on' Switching node to | | 15:49 |
| jfargen | | | maintenance mode. | 15:49 |
| iurygregory | mohammed, we don't have periodic runs but every patch to ironic repo will run the metal3-integration job | 15:50 |
| iurygregory | https://zuul.opendev.org/t/openstack/builds?job_name=metal3-integration&project=openstack/ironic | 15:51 |
| mohammed | iurygregory: Thanks! I have been checking those but I was a bit confused if the error are on the master or caused by the change itself | 15:52 |
| iurygregory | yeah that's trye | 15:54 |
| iurygregory | true* | 15:54 |
| masghar | Hello everyone! I have some trouble trying to test my patch migrating inspection hooks to ironic (from ironic inspector, to deprecate ironic-inspector in the future). Link: https://review.opendev.org/c/openstack/ironic/+/887554 | 16:00 |
| masghar | I am using bifrost, and testing this on VMs made by bifrost. My nodes keep getting stuck in inspect_wait, and I am not too sure if its because the nodes' interfaces are set incorrectly.. | 16:01 |
| masghar | For example: One node has driver ipmi, boot_interface=ipxe, deploy_interface=ramdisk, inspect_interface=agent, network_interface=noop | 16:03 |
| masghar | If anyone knows, is there something wrong with the interface values at the surface level? | 16:04 |
| JayF | jfargen: unset maintenance on it, baremetal node manage; baremetal node provide | 16:19 |
| JayF | jfargen: basically my assumption is that the machine started in the wrong power state vs what Ironic expects | 16:19 |
| JayF | jfargen: so you just wanna retry which means getting node.maintenance set to false (there's a special cli command just for this), and getting it to reclean which is manage (move from clean failed -> managable) then provide (move from managable -> available) | 16:19 |
| * JayF is not actually here today but thought he's try to help | 16:20 | |
| opendevreview | Victor Morales proposed openstack/bifrost master: Fix ipaddr deprecation warning https://review.opendev.org/c/openstack/bifrost/+/889195 | 16:40 |
| TheJulia | jfargen: is that machine enrolled in another ironic? | 16:44 |
| TheJulia | mohammed: looks like the latest metal3 job failure was due to the change set, whatever Jacob was doing with his change | 16:51 |
| mohammed | TheJulia : I thought the same ! I have built image after yesterday fix on metal3CI and I am following that so far it looks green thanks :) | 16:53 |
| iurygregory | masghar, when you have a node you can try to validate their interfaces using /v1/nodes/{node_ident}/validate this would give an idea if you are using wrong values (there is also the option via the client) | 17:28 |
| *** melwitt is now known as jgwentworth | 18:26 | |
| jfargen | TheJulia: I was thinking the same thing, going to check on it. | 19:45 |
| NobodyCam | Good afternoon Ironic folks | 23:41 |
| NobodyCam | Happy late Friday | 23:41 |
| NobodyCam | dumb question will iso-image-create package IPA in a UEFI image? | 23:42 |
| TheJulia | Dunno | 23:48 |
| TheJulia | Honestly, never tried | 23:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!