| opendevreview | Yu Zou proposed openstack/ironic master: feat: add verify ca conf support for drivers https://review.opendev.org/c/openstack/ironic/+/947544 | 02:43 |
|---|---|---|
| opendevreview | Yu Zou proposed openstack/ironic master: feat: add verify ca conf support for drivers https://review.opendev.org/c/openstack/ironic/+/947544 | 02:44 |
| arne_wiebalck | Good morning, Ironic! | 06:47 |
| rpittau | good morning ironic! o/ | 07:01 |
| queensly[m] | Good morning | 07:03 |
| freemanboss[m] | Good morning | 07:17 |
| Ayo[m] | Good morning | 07:18 |
| abongale | Good morning Ironic! | 08:39 |
| opendevreview | 雒海艇 proposed openstack/networking-generic-switch master: Remove H3C unit test temporarily due to netmiko dev branch dependency https://review.opendev.org/c/openstack/networking-generic-switch/+/948049 | 09:11 |
| opendevreview | 雒海艇 proposed openstack/networking-generic-switch master: Add H3C switch support in supported-devices.rst https://review.opendev.org/c/openstack/networking-generic-switch/+/948050 | 09:11 |
| Sandzwerg[m] | Hello wonderful people of ironic. As always when I have time to look into this I have an issue. Can someone tell me if it's possible to have more than one capability on a node? For example boot_mode:uef and secure_boot: true. I tried to do it as json but that fails during deployment openstack baremetal node set --property capability={"boot_mode": "uefi", "secure_boot": "true"} that a dict object has no attribute split. Before I | 14:26 |
| Sandzwerg[m] | tried to add just the secure_boot but that overwrote the boot_mode and just adding a "," or ";" also looks wrong. I think I'm holding it wrong but I don't know how to hold it right | 14:26 |
| JayF | I think it'd be something like | 14:28 |
| JayF | openstack baremetal node set --capability boot_mode=uefi --capability secure_boot=true | 14:28 |
| Sandzwerg[m] | ohhhh I didn't think of that, let me try it. | 14:29 |
| dtantsur | Hmm, I don't think we actually have a reasonable command to do that.. | 14:33 |
| Sandzwerg[m] | hmm no when I state multiple capabilities only the last one "wins" | 14:35 |
| Sandzwerg[m] | https://docs.openstack.org/ironic/queens/install/advanced.html mentions that I should quote the string and doing --property capabilities="boot_mode:uefi,secure_boot:true" shows both as "capabilities": "bootmode:uefi,secure_boot:true" but that looks a bit odd to me (especially in the json output compared to e.g. the root device hint | 14:42 |
| Sandzwerg[m] | * https://docs.openstack.org/ironic/queens/install/advanced.html mentions that I should quote the string and doing --property capabilities="boot_mode:uefi,secure_boot:true" shows both as "capabilities": "bootmode:uefi,secure_boot:true"´ but that looks a bit odd to me (especially in the json output compared to e.g. the root device hint | 14:42 |
| Sandzwerg[m] | * https://docs.openstack.org/ironic/queens/install/advanced.html mentions that I should quote the string and doing --property capabilities="boot_mode:uefi,secure_boot:true" shows both as "capabilities": "bootmode:uefi,secure_boot:true" but that looks a bit odd to me (especially in the json output compared to e.g. the root device hint` | 14:42 |
| Sandzwerg[m] | Let's see if it works | 14:43 |
| Sandzwerg[m] | OK, that indeed works and I no longer have a dict splitting error even when it's not intuitve to me. Thanks for rubber ducking that with me :) Is there any chance to note this in the documentation so that I can look it up next time instead of remebering it? | 14:47 |
| dtantsur | Sandzwerg[m]: this, or we'll be extremely grateful if you could actually implement that --capability argument that JayF just made up :) | 14:54 |
| dtantsur | (and probably --instance-capability? a similar thing exists in instance_info, but it's a normal dict rather than... that) | 14:54 |
| JayF | "made up" is such a nasty term for my brainstorming ideation :P | 14:59 |
| Sandzwerg[m] | I'm not sure my python is good enough to implement that yet (or have the time currently 🫠) but I could at least try to work on the documentation. Were would be a good place to add it? I think there is no generic "capabilities" section but it's only mentioned when it's used for something like local boot in the link above | 15:09 |
| dtantsur | JayF: I wanted to say hallucinated, but was afraid it wound sound just rude instead of a funny reference to AI :) | 15:13 |
| dtantsur | Sandzwerg[m]: a capabilities section in the user guide would be a useful addition | 15:14 |
| Sandzwerg[m] | As it's own page or on that advanced page (or somewhere else)? | 16:14 |
| TheJulia | I think I need to go find a beach and chill on it for a week or two | 16:15 |
| Sandzwerg[m] | Sounds like a good plan. May I suggest the addition of ice cream and drinks with little umbrellas? | 16:16 |
| *** awb_ is now known as awb | 16:42 | |
| TheJulia | Sandzwerg[m]: ++ | 19:37 |
| * JayF just got approval and will almost certainly be in Paris \o/ | 21:58 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!