| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: service steps https://review.opendev.org/c/openstack/ironic/+/884783 | 01:02 |
|---|---|---|
| opendevreview | yatin proposed openstack/ironic master: Source install dnsmasq-2.87 https://review.opendev.org/c/openstack/ironic/+/888121 | 05:31 |
| rpittau | good morning ironic! o/ | 07:07 |
| rpittau | jfargen: you can modify that using the baremetal command, something like baremetal node set <your-node> --boot-interface redfish-virtual-media should work | 07:18 |
| *** jroll9 is now known as jroll | 09:27 | |
| *** jroll4 is now known as jroll | 09:41 | |
| iurygregory | good morning Ironic | 11:48 |
| opendevreview | Merged openstack/sushy stable/zed: Requests must always have a read/connect timeout https://review.opendev.org/c/openstack/sushy/+/889112 | 13:05 |
| opendevreview | Merged openstack/sushy stable/xena: Requests must always have a read/connect timeout https://review.opendev.org/c/openstack/sushy/+/889115 | 13:05 |
| opendevreview | Merged openstack/sushy stable/yoga: Requests must always have a read/connect timeout https://review.opendev.org/c/openstack/sushy/+/889114 | 13:32 |
| jfargen | rpittau: I saw that, but was guessing that it could all be done when the node is imported. | 13:48 |
| rpittau | jfargen: sorry, I saw now that you said in the inventory | 13:56 |
| rpittau | you can specify that in the json where you define the node https://docs.openstack.org/bifrost/wallaby/user/howto.html#json-file-format | 13:56 |
| jfargen | yes, adding it to the json where the node is defined would be best, cannot find the right key value pair. Added boot_interface: 'redfish-virtual-media', but it makes bifrost/ironic unhappy. | 14:01 |
| jfargen | Is there a way to look up all the key value pairs that are suppored in the inventory file? | 14:08 |
| jfargen | https://pastebin.com/HjLPkZSx | 14:25 |
| jfargen | Actually, the import of the node worked with "boot_interface": "redfish-virtual-media", but the node was created with boot_interface: ipxe. | 14:26 |
| jfargen | The pastebin contains the inventory.json. | 14:26 |
| * TheJulia wonders if ther eis a list of writable fields someplace | 14:30 | |
| rpittau | TheJulia: I highly doubt it :/ | 14:39 |
| TheJulia | Originally, the thoguht process and model was to try to write whatever we were given | 14:46 |
| TheJulia | that seems like that changed at some point | 14:46 |
| rpittau | in theory adding boot_interface to the first level should work | 14:47 |
| TheJulia | under a node like jfargen has done right? | 14:47 |
| rpittau | yeah | 14:47 |
| TheJulia | That is what I thoguht as well | 14:47 |
| TheJulia | thought | 14:48 |
| rpittau | it's literally a node definition | 14:48 |
| TheJulia | I know, I still somehow remember the three scotch night at the downtown seattle marriot where I wrote that | 14:48 |
| rpittau | lol | 14:48 |
| JayF | For interfaces, don't we allow you to set it on the bifrost-cli? | 14:50 |
| JayF | That's the only thing I keep thinking of, that --network-interface flag | 14:50 |
| JayF | but that's actually for a network interface, not for a NetworkInterface, isn't it | 14:50 |
| JayF | it always screws with my head | 14:50 |
| TheJulia | sorry! | 14:52 |
| rpittau | it's jsut for network-interface | 14:52 |
| rpittau | other variables can be passed using extra-vars combined to enroll, but I'm not sure you can do it node by node | 14:52 |
| TheJulia | challenge is this code was written before we delineated the interfaces | 14:58 |
| TheJulia | https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L219-L235 | 15:02 |
| TheJulia | Seems fairly specific.... | 15:02 |
| TheJulia | the inbound validator lists it as a string field | 15:03 |
| rpittau | yeah | 15:03 |
| rpittau | I'm doing a test | 15:06 |
| TheJulia | rpittau: k, much appreciated | 15:06 |
| TheJulia | I'm polishing presentation for in about 2 hours | 15:06 |
| rpittau | TheJulia: good luck :) | 15:06 |
| jfargen | Okay, I figured it out. | 15:27 |
| jfargen | [cloud-user@bifrost-0 tasks]$ pwd | 15:29 |
| jfargen | (bifrost) [cloud-user@bifrost-0 tasks]$ diff main.yml main.yml.orig | 15:29 |
| jfargen | 27d26 | 15:29 |
| jfargen | < boot_interface: "{{ boot_interface | default(omit) }}" | 15:29 |
| jfargen | I added boot_interface to the ironic-enroll-dynamic role. | 15:30 |
| rpittau | jfargen: yeah I was going to tell you that it's not supported by default in enroll-dynamic :) | 15:31 |
| rpittau | we should probably add the options for the various interfaces, otherwise they get the default based on the driver | 15:33 |
| jfargen | Is anyone interested in working directly with me to get bifrost working end to end with redfish-virtual-media and documented? | 15:35 |
| rpittau | jfargen: my suggestion would be to start proposing a patch, I can definitely provide guidance and reviews and I'm pretty sure more people will be interested | 15:38 |
| jfargen | Okay, thanks! | 15:39 |
| rpittau | good night! o/ | 15:49 |
| * TheJulia looks forward to approving such a patch | 16:33 | |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP: preserve/handle config drives on 4k block devices https://review.opendev.org/c/openstack/ironic-python-agent/+/888794 | 19:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!