| *** k_mouza has quit IRC | 00:01 | |
| *** hoonetorg has quit IRC | 00:04 | |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Removing libselinux-python package from Fedora dependencies https://review.opendev.org/740678 | 00:05 |
|---|---|---|
| *** dsneddon has quit IRC | 00:12 | |
| *** k_mouza has joined #openstack-ironic | 00:12 | |
| *** dsneddon has joined #openstack-ironic | 00:14 | |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Removing libselinux-python package from Fedora dependencies https://review.opendev.org/740678 | 00:16 |
| *** k_mouza has quit IRC | 00:17 | |
| *** hoonetorg has joined #openstack-ironic | 00:17 | |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Removing libselinux-python package from Fedora dependencies https://review.opendev.org/740678 | 00:18 |
| janders | apologies for the noise - was fiddling with the release notes | 00:18 |
| janders | as far as I am concerned, the minipatch is ready for review | 00:19 |
| *** hoonetorg has quit IRC | 00:36 | |
| *** gyee has quit IRC | 00:52 | |
| *** hoonetorg has joined #openstack-ironic | 00:58 | |
| *** rh-jelabarre has quit IRC | 01:23 | |
| *** mkrai has joined #openstack-ironic | 01:48 | |
| *** janders has quit IRC | 02:07 | |
| *** Goneri has quit IRC | 02:29 | |
| *** janders has joined #openstack-ironic | 02:29 | |
| *** Lucas_Gray has quit IRC | 02:44 | |
| *** ricolin_ has joined #openstack-ironic | 03:00 | |
| *** dsneddon has quit IRC | 03:08 | |
| *** dsneddon has joined #openstack-ironic | 03:11 | |
| *** dsneddon has quit IRC | 03:17 | |
| *** tzumainn has quit IRC | 03:18 | |
| *** dsneddon has joined #openstack-ironic | 03:22 | |
| *** mkrai has quit IRC | 03:26 | |
| *** dsneddon has quit IRC | 03:27 | |
| *** dsneddon has joined #openstack-ironic | 03:28 | |
| *** mkrai has joined #openstack-ironic | 03:33 | |
| *** dsneddon has quit IRC | 03:34 | |
| *** dsneddon has joined #openstack-ironic | 03:44 | |
| *** iurygregory has quit IRC | 03:54 | |
| *** gudrutis has quit IRC | 03:54 | |
| *** TheJulia has quit IRC | 03:54 | |
| *** gudrutis0 has joined #openstack-ironic | 03:54 | |
| *** mbuil has quit IRC | 03:54 | |
| *** mbuil has joined #openstack-ironic | 03:55 | |
| *** TheJulia has joined #openstack-ironic | 03:55 | |
| *** dsneddon has quit IRC | 04:22 | |
| *** uzumaki has joined #openstack-ironic | 04:25 | |
| *** mkrai has quit IRC | 04:40 | |
| *** dsneddon has joined #openstack-ironic | 04:52 | |
| *** abdysn has joined #openstack-ironic | 04:54 | |
| *** ociuhandu has joined #openstack-ironic | 05:01 | |
| *** ociuhandu has quit IRC | 05:05 | |
| *** janders has quit IRC | 05:31 | |
| *** janders has joined #openstack-ironic | 05:40 | |
| *** dsneddon has quit IRC | 06:07 | |
| *** dsneddon has joined #openstack-ironic | 06:08 | |
| *** dsneddon has quit IRC | 06:18 | |
| *** diurnalist has quit IRC | 06:22 | |
| *** dsneddon has joined #openstack-ironic | 06:25 | |
| *** diurnalist has joined #openstack-ironic | 06:29 | |
| uzumaki | bonjour ironic! o | 06:37 |
| uzumaki | o/ | 06:37 |
| *** dsneddon has quit IRC | 06:37 | |
| janders | uzumaki: g'day o/ | 06:38 |
| uzumaki | how you doing janders ? \o/ | 06:38 |
| janders | I'm doing well, thank you | 06:39 |
| janders | and you? | 06:39 |
| uzumaki | a bit tired, so early in the day.. but I'm well.. it's going to be a rough day, got meetings straight for many many hours.. | 06:39 |
| janders | oh dear.. good luck with the meetings! :) | 06:40 |
| uzumaki | thank you! I'll probably need a lot of that | 06:41 |
| janders | if there was a way to send coffee over IP links I could offer some of that too | 06:41 |
| uzumaki | well, we do have a coffee script, won't be long before we have a coffee transport protocol | 06:42 |
| janders | haha! :) I look forward to that | 06:42 |
| uzumaki | we all do! how's your day looking? | 06:42 |
| janders | I think somewhere in Belgium there was a block of apartments with a beer pipeline | 06:43 |
| uzumaki | oh boy.. if it's beer, I'd wager it must be in Ireland xD | 06:43 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Adds support SUM based firmware update as deploy step https://review.opendev.org/740893 | 06:44 |
| janders | quite productive actually... fixed one little bug, gotta track down a couple related ones now | 06:45 |
| janders | thank you for asking | 06:45 |
| uzumaki | that's really good! I believe no bug is little, they all matter, we've all seen these 'little' ones making the hell break loose on us all the tim | 06:45 |
| janders | very true :) | 06:46 |
| arne_wiebalck | Good morning, ironic! | 06:49 |
| janders | g'day arne_wiebalck | 06:52 |
| arne_wiebalck | janders: o/ | 06:53 |
| *** dsneddon has joined #openstack-ironic | 06:57 | |
| *** janders has quit IRC | 07:00 | |
| *** rcernin has quit IRC | 07:05 | |
| *** iurygregory_ has joined #openstack-ironic | 07:10 | |
| *** janders has joined #openstack-ironic | 07:11 | |
| iurygregory_ | good morning Ironic! | 07:11 |
| janders | g'day iurygregory_ | 07:16 |
| janders | regarding our yesterday chat about reno - here's updated gerrit: https://review.opendev.org/#/c/740678/4/releasenotes/notes/releasenote-94bcb2b0da207f94.yaml | 07:17 |
| patchbot | patch 740678 - bifrost - Removing libselinux-python package from Fedora dep... - 4 patch sets | 07:17 |
| janders | let me know if this looks better | 07:17 |
| uzumaki | morning iurygregory_ o/ | 07:18 |
| uzumaki | comment ca va arne_wiebalck ? \o/ | 07:18 |
| iurygregory_ | hey janders and uzumaki | 07:30 |
| *** iurygregory_ is now known as iurygregory | 07:31 | |
| iurygregory | janders, looking now | 07:31 |
| uzumaki | how's it going iurygregory ? | 07:31 |
| iurygregory | uzumaki, doing good and you? | 07:32 |
| iurygregory | janders, I've left a comment on the releasenote | 07:33 |
| uzumaki | I've been well.. Into the meeting marathon today.. | 07:36 |
| *** diurnalist has quit IRC | 07:44 | |
| *** dtantsur|afk is now known as dtantsur | 07:56 | |
| dtantsur | morning ironic | 07:56 |
| iurygregory | morning dtantsur | 07:56 |
| arne_wiebalck | uzumaki: tout va bien, merci! | 08:00 |
| iurygregory | morning arne_wiebalck o/ | 08:02 |
| arne_wiebalck | hey iurygregory o/ | 08:02 |
| *** lucasagomes has joined #openstack-ironic | 08:08 | |
| *** rcernin has joined #openstack-ironic | 08:09 | |
| *** Lucas_Gray has joined #openstack-ironic | 08:10 | |
| janders | iurygregory: thank you | 08:13 |
| janders | just fixing it now | 08:13 |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Removing libselinux-python package from Fedora dependencies https://review.opendev.org/740678 | 08:13 |
| iurygregory | janders, np o/ | 08:13 |
| iurygregory | +2 | 08:14 |
| *** rcernin has quit IRC | 08:14 | |
| *** rcernin has joined #openstack-ironic | 08:15 | |
| openstackgerrit | Jacob Anders proposed openstack/bifrost master: Removing libselinux-python package from Fedora dependencies https://review.opendev.org/740678 | 08:15 |
| janders | thank you iurygregory | 08:15 |
| janders | I made one more little tweak... there was one word that didn't need to be there can't type today apparently | 08:16 |
| *** janders has quit IRC | 08:21 | |
| *** mkrai has joined #openstack-ironic | 08:21 | |
| *** janders has joined #openstack-ironic | 08:22 | |
| *** rcernin has quit IRC | 08:42 | |
| *** derekh has joined #openstack-ironic | 08:48 | |
| *** ociuhandu has joined #openstack-ironic | 08:50 | |
| *** Qianbiao has joined #openstack-ironic | 08:50 | |
| *** donnyd has quit IRC | 08:53 | |
| *** donnyd has joined #openstack-ironic | 08:53 | |
| *** Qianbiao has quit IRC | 08:59 | |
| *** alexmcleod has quit IRC | 09:05 | |
| *** alexmcleod has joined #openstack-ironic | 09:05 | |
| dtantsur | folks, do I get it right that our CI is still broken? | 09:17 |
| dtantsur | is anyone looking at it today? I can look only in a few hours | 09:17 |
| dtantsur | ah, https://review.opendev.org/#/c/740730/ ? | 09:17 |
| patchbot | patch 740730 - python-openstackclient - Fix compatibility issue in 5.3 - 2 patch sets | 09:17 |
| *** k_mouza has joined #openstack-ironic | 09:35 | |
| *** diurnalist has joined #openstack-ironic | 09:43 | |
| iurygregory | dtantsur, not sure if you saw TheJulia chat on the channel | 09:47 |
| dtantsur | yep, I saw it after asking :) | 09:48 |
| iurygregory | dtantsur, gotcha =) | 09:48 |
| *** dtantsur is now known as dtantsur|bbl | 09:49 | |
| dtantsur|bbl | back in the afternoon | 09:49 |
| *** tkajinam has quit IRC | 10:12 | |
| *** dsneddon has quit IRC | 10:19 | |
| *** dsneddon has joined #openstack-ironic | 10:23 | |
| *** dsneddon has quit IRC | 10:35 | |
| *** dsneddon has joined #openstack-ironic | 10:36 | |
| janders | are you guys satisfied with my FC32 libselinux patch, can I consider this mini-task complete? | 10:45 |
| *** mkrai has quit IRC | 10:49 | |
| *** dsneddon has quit IRC | 10:53 | |
| *** dsneddon has joined #openstack-ironic | 10:56 | |
| janders | if so, what would be the next thing I should be looking at? | 10:57 |
| *** dsneddon has quit IRC | 11:06 | |
| *** dsneddon has joined #openstack-ironic | 11:07 | |
| *** rcernin has joined #openstack-ironic | 11:13 | |
| *** mkrai has joined #openstack-ironic | 11:13 | |
| *** dsneddon has quit IRC | 11:13 | |
| janders | wrapping up for tonight, will check the messages tomorrow morning | 11:27 |
| janders | have a good one guys | 11:27 |
| janders | o/ | 11:27 |
| *** Lucas_Gray has quit IRC | 11:28 | |
| *** mkrai has quit IRC | 11:31 | |
| *** mkrai has joined #openstack-ironic | 11:32 | |
| *** dsneddon has joined #openstack-ironic | 11:33 | |
| *** Lucas_Gray has joined #openstack-ironic | 11:33 | |
| *** dsneddon has quit IRC | 11:45 | |
| *** iurygregory has quit IRC | 11:46 | |
| *** dsneddon has joined #openstack-ironic | 11:46 | |
| *** dsneddon has quit IRC | 11:53 | |
| *** dsneddon has joined #openstack-ironic | 11:54 | |
| *** iurygregory has joined #openstack-ironic | 12:00 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:09 | |
| *** dsneddon has quit IRC | 12:11 | |
| *** dsneddon has joined #openstack-ironic | 12:11 | |
| *** derekh has quit IRC | 12:23 | |
| *** mkrai has quit IRC | 12:25 | |
| iurygregory | shadower, hey | 12:46 |
| *** dsneddon has quit IRC | 12:47 | |
| *** rcernin has quit IRC | 12:48 | |
| *** dsneddon has joined #openstack-ironic | 12:58 | |
| *** tzumainn has joined #openstack-ironic | 12:59 | |
| *** derekh has joined #openstack-ironic | 13:03 | |
| *** dsneddon has quit IRC | 13:08 | |
| *** dsneddon has joined #openstack-ironic | 13:10 | |
| *** Goneri has joined #openstack-ironic | 13:14 | |
| *** dsneddon has quit IRC | 13:16 | |
| *** dsneddon has joined #openstack-ironic | 13:23 | |
| openstackgerrit | Merged openstack/bifrost master: Removing libselinux-python package from Fedora dependencies https://review.opendev.org/740678 | 13:24 |
| *** dsneddon has quit IRC | 13:28 | |
| *** dsneddon has joined #openstack-ironic | 13:33 | |
| *** dsneddon has quit IRC | 13:38 | |
| *** benj_ has quit IRC | 13:39 | |
| *** rloo has joined #openstack-ironic | 13:39 | |
| *** dsneddon has joined #openstack-ironic | 13:39 | |
| *** benj_ has joined #openstack-ironic | 13:41 | |
| TheJulia | looks like we're going to be waiting on https://review.opendev.org/#/c/740730 for python 3.8 compatibility, on a plus side it should only be merged in ~2 hours | 13:44 |
| patchbot | patch 740730 - python-openstackclient - Fix compatibility issue in 5.3 - 3 patch sets | 13:44 |
| *** dtantsur|bbl is now known as dtantsur | 13:45 | |
| dtantsur | morning TheJulia | 13:45 |
| iurygregory | good morning TheJulia =) | 13:47 |
| *** dsneddon has quit IRC | 14:00 | |
| *** dsneddon has joined #openstack-ironic | 14:02 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Wipe agent token during reboot or power off https://review.opendev.org/739964 | 14:02 |
| dtantsur | TheJulia: ^^ | 14:02 |
| dtantsur | janders: you can use this query to look for relatively easy tasks: https://storyboard.openstack.org/#!/story/list?status=active&tags=low-hanging-fruit&project_group_id=75 | 14:04 |
| TheJulia | dtantsur: will take a look this morning, thanks | 14:11 |
| *** lucasagomes has quit IRC | 14:13 | |
| *** lucasagomes has joined #openstack-ironic | 14:14 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add missing agent RAID compatibility for ilo5 and idrac https://review.opendev.org/740975 | 14:26 |
| *** uzumaki has quit IRC | 14:29 | |
| *** abdysn has quit IRC | 14:29 | |
| *** johnthetubaguy has quit IRC | 14:50 | |
| *** johnthetubaguy has joined #openstack-ironic | 14:52 | |
| *** dsneddon has quit IRC | 14:56 | |
| *** dsneddon has joined #openstack-ironic | 14:57 | |
| *** dsneddon has quit IRC | 15:14 | |
| *** dsneddon has joined #openstack-ironic | 15:15 | |
| *** mkrai has joined #openstack-ironic | 15:24 | |
| *** dsneddon has quit IRC | 15:31 | |
| *** mkrai has quit IRC | 15:39 | |
| *** mkrai has joined #openstack-ironic | 15:40 | |
| *** gyee has joined #openstack-ironic | 15:50 | |
| *** dsneddon has joined #openstack-ironic | 15:50 | |
| *** dsneddon has quit IRC | 15:55 | |
| *** mkrai has quit IRC | 15:57 | |
| *** dsneddon has joined #openstack-ironic | 15:58 | |
| *** rloo has quit IRC | 15:59 | |
| *** lucasagomes has quit IRC | 15:59 | |
| *** rloo has joined #openstack-ironic | 16:00 | |
| gudrutis0 | Hi ironic | 16:01 |
| openstackgerrit | Richard G. Pioso proposed openstack/ironic master: WIP: Fix idrac HW type DevStack plug-in support https://review.opendev.org/738973 | 16:01 |
| *** Lucas_Gray has quit IRC | 16:01 | |
| *** dsneddon has quit IRC | 16:05 | |
| *** sshnaidm is now known as sshnaidm|afk | 16:08 | |
| dtantsur | hi gudrutis0 | 16:08 |
| *** dsneddon has joined #openstack-ironic | 16:17 | |
| *** aedc_ has joined #openstack-ironic | 16:24 | |
| *** aedc_ has quit IRC | 16:24 | |
| * TheJulia is almost done reviewing the whitepaper | 16:26 | |
| *** aedc has quit IRC | 16:27 | |
| *** rloo has quit IRC | 16:33 | |
| *** rloo has joined #openstack-ironic | 16:33 | |
| TheJulia | arne_wiebalck: there are a number of small edits to the cern section, have you had a chance to look at them? | 16:34 |
| *** k_mouza has quit IRC | 16:46 | |
| arne_wiebalck | TheJulia: will check ... | 16:47 |
| *** sshnaidm|afk is now known as sshnaidm | 16:48 | |
| TheJulia | I've approved some that seem super non-trivial | 16:48 |
| TheJulia | err | 16:48 |
| TheJulia | super trivial | 16:48 |
| arne_wiebalck | What? Oxford comma discussion are never trivial! | 16:48 |
| arne_wiebalck | ;) | 16:49 |
| arne_wiebalck | And German sentence lengths :) | 16:49 |
| TheJulia | I know! | 16:53 |
| TheJulia | Some of it made sense given meaning, some I've gone "ehh.. no" | 16:53 |
| *** ociuhandu_ has joined #openstack-ironic | 16:58 | |
| *** derekh has quit IRC | 16:58 | |
| *** ociuhandu has quit IRC | 17:01 | |
| *** ociuhandu_ has quit IRC | 17:02 | |
| openstackgerrit | Aija Jaunteva proposed openstack/ironic-specs master: [WIP] Operationally efficient and fast-track bare metal hardware provisioning https://review.opendev.org/740721 | 17:15 |
| arne_wiebalck | bye everyone o/ | 17:18 |
| TheJulia | new osc release in process of being cut to fix ci issuez | 17:23 |
| TheJulia | issues | 17:23 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Isolate the ISO creation logic from redfish https://review.opendev.org/739174 | 17:24 |
| *** k_mouza has joined #openstack-ironic | 17:25 | |
| *** rloo has quit IRC | 17:27 | |
| *** rloo has joined #openstack-ironic | 17:27 | |
| *** k_mouza has quit IRC | 17:30 | |
| dtantsur | TheJulia: great! | 17:31 |
| *** dtantsur is now known as dtantsur|afk | 17:31 | |
| TheJulia | Yeah :( | 17:31 |
| TheJulia | should be approved and through requirements by tomorrow | 17:31 |
| dtantsur|afk | at least we're past the releases.... | 17:31 |
| dtantsur|afk | anyway, g'night folks | 17:31 |
| openstackgerrit | Aija Jaunteva proposed openstack/ironic-specs master: [WIP] Operationally efficient and fast-track bare metal hardware provisioning https://review.opendev.org/740721 | 17:32 |
| *** dsneddon has quit IRC | 17:58 | |
| *** diurnalist has quit IRC | 17:59 | |
| *** diurnalist has joined #openstack-ironic | 18:26 | |
| *** lbotell has joined #openstack-ironic | 18:30 | |
| *** rmcall has joined #openstack-ironic | 18:30 | |
| *** k_mouza has joined #openstack-ironic | 18:32 | |
| lbotell | Hi: | 18:34 |
| *** k_mouza has quit IRC | 18:37 | |
| *** ricolin_ has quit IRC | 18:41 | |
| TheJulia | lbotell: o/ | 18:43 |
| TheJulia | so will anyone believe we don't have anything detailing how to delete a port? | 18:43 |
| *** lbotell14 has joined #openstack-ironic | 18:43 | |
| *** alexmcleod has quit IRC | 18:46 | |
| mjturek | oh my! | 18:59 |
| TheJulia | o/ mjturek | 19:06 |
| mjturek | o/ TheJulia :) | 19:06 |
| TheJulia | heh, also only two references to "node delete" | 19:13 |
| *** ociuhandu has joined #openstack-ironic | 19:14 | |
| *** ociuhandu has quit IRC | 19:23 | |
| lbotell | o/ TheJulia | 19:31 |
| dhellmann | o/ I need some API help. I would like to change the boot_mode capability of an existing node without erasing any of the other capabilities. Do I need to fetch the capabilities, munge the string myself, and post it back? Or is there some path value I can use to set just the boot mode? | 19:44 |
| dhellmann | maybe /properties/capabilities/boot_mode ? | 19:44 |
| *** lbotell has quit IRC | 19:48 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Follow-up on blocking port deletions https://review.opendev.org/741039 | 19:52 |
| TheJulia | dhellmann: you _should_ be able just to directly patch it | 19:53 |
| dhellmann | well, no, not /properties/capabilities/boot_mode that gives a "boot_mode' is not a valid list index" error | 19:53 |
| TheJulia | hmm | 19:53 |
| dhellmann | yeah, I'm having trouble figuring out the right structure | 19:53 |
| * TheJulia wonders if you can directly patch capabilities | 19:53 | |
| TheJulia | it is nested dictionaries which may be why | 19:53 |
| dhellmann | adding it when the host is registered seems to work, but I'm concerned with keeping it updated | 19:54 |
| dhellmann | gophercloud gives me this as the capabilities: "cpu_vt:true,cpu_aes:true,cpu_hugepages:true,cpu_hugepages_1g:true" | 19:54 |
| TheJulia | I think your trying to update the wrong field | 19:55 |
| dhellmann | maybe? how do I set the boot mode for a host? | 19:55 |
| TheJulia | and it is confusing because we have two places once for scheduling and informing things like nova and one for declaring what you want. Conundrum is the first is also supported so operators can explicitly say "i will only offer x" | 19:56 |
| TheJulia | hmm | 19:58 |
| dhellmann | https://docs.openstack.org/ironic/ussuri/install/advanced.html#boot-mode-support only talks about the property/capabilities stuff | 19:58 |
| TheJulia | https://docs.openstack.org/ironic/latest/install/standalone.html | 19:58 |
| TheJulia | checking because I don't think I've ever personally used deploy_boot_mode directly | 19:59 |
| TheJulia | yeah, looksl ike that is what you want | 20:00 |
| *** lbotell has joined #openstack-ironic | 20:00 | |
| TheJulia | instance_info/deploy_boot_mode=uefi | 20:00 |
| dhellmann | oh, so it goes in the instance_info? | 20:01 |
| TheJulia | yup, that is the other place it is supported | 20:01 |
| dhellmann | ok | 20:02 |
| TheJulia | again, thanks to flavors and scheduling, the properties/capabilities field gets viewed as usable too... and it is if your willing to handle the field value | 20:02 |
| TheJulia | oddly enough, I thought you could directly patch each object, but it may be the jsonpatch document is not super thrilled about the nesting | 20:02 |
| dhellmann | and there it's called "deploy_boot_mode" not "boot_mode" | 20:02 |
| iurygregory | you can patch the boot_mode | 20:03 |
| TheJulia | dhellmann: deploy_boot_mode is what is in the code for the instance_info field | 20:04 |
| iurygregory | patch = [{'path': '/instance_info/capabilities', 'op': 'add', 'value': {'boot_mode': 'uefi'}}] | 20:04 |
| iurygregory | update_node(node.uuid, patch) | 20:05 |
| dhellmann | iurygregory : how do I change a value that is already there? | 20:05 |
| iurygregory | oh you want to override? | 20:05 |
| dhellmann | I need, no matter what state the node is in when I start, to set the mode to the value I want for the next provisioning operation. | 20:05 |
| dhellmann | So, if there is no value I need to add it and if there is a value I need to ensure it is correct. | 20:06 |
| dhellmann | I know how to do those 2 operations generally, but my attempts so far with the capabilities has replaced all of the capabilities instead of just the one value | 20:06 |
| dhellmann | it's very likely I've constructed my API call wrong | 20:06 |
| dhellmann | I don't care which field I use, since I'm going to set the value for every provisioning operation | 20:07 |
| TheJulia | dhellmann: I think iury is right, I seem to remember with the nested dict that one can directly generate a patch object but it is doing to come down to how it gets rendered. | 20:07 |
| dhellmann | so if the instance_info deploy_boot_mode is the right "user facing" way to set the value, then I can do that | 20:07 |
| TheJulia | yeah, do it that way since you should be ideally declaring that all at once | 20:09 |
| dhellmann | cool, that's what I'll do then | 20:09 |
| TheJulia | if you can get the dictionary that is being sent for your attempts now, it might be good for us to check the other path because it _should_ work by just treating it as a separate entry, but yeah | 20:10 |
| TheJulia | whatever is easiest for you doug | 20:10 |
| iurygregory | I know that for node we have 'op' 'replace' | 20:12 |
| iurygregory | if you have a value you will need to probably use that? | 20:12 |
| iurygregory | so your patch would be | 20:13 |
| iurygregory | patch = [{'path': '/instance_info/capabilities', 'op': 'replace', 'value': {'boot_mode': 'uefi'}}] | 20:13 |
| iurygregory | I don't have an env to test if it would work .-. | 20:14 |
| iurygregory | but I can give a try tomorrow dhellmann (if you are not in a hurry ofc) | 20:14 |
| TheJulia | I ran into some weirdness when I wrote the ansible modules mk1 but it could have just been me | 20:14 |
| TheJulia | or jsonpatches are evil | 20:14 |
| dhellmann | I think I tried /properties/capabilities with a string "boot_mode:uefi" and /properties/capabilities/boot_mode with "uefi" but not /property/capabilities with a dict value | 20:15 |
| dhellmann | the fact that I get back a string instead of a dict did not lead me to realize the API was expecting a dict there | 20:15 |
| *** lbotell has quit IRC | 20:15 | |
| iurygregory | the PATCH will take a list of dicts | 20:15 |
| dhellmann | that's not how the gophercloud library sets things up | 20:19 |
| TheJulia | Maybe this is why the last time I discussed this area I was sitting at the bar in the Westin Seattle... enjoying several tasty beverages | 20:19 |
| iurygregory | ++ | 20:19 |
| TheJulia | (When I was hacking on the first os_ironic_ | 20:20 |
| TheJulia | ) | 20:20 |
| dhellmann | setting deploy_boot_mode in instance_info didn't fail. I wonder how I can verify that the value was actually used... | 20:20 |
| iurygregory | I will try to get a metal3-dev-env with a virtual BM and see if I can use the gophercloud to update the info | 20:20 |
| iurygregory | well you can check the node to see if it's there | 20:21 |
| TheJulia | dhellmann: it becomes an explict A or B path if memory serves | 20:21 |
| TheJulia | iurygregory: I think he was meaning "like actually set to the hardware" | 20:21 |
| iurygregory | oh =) | 20:21 |
| dhellmann | the odd thing is the old code registered the node by setting capabilities to the string "boot_mode:uefi" and then later something else came along and added more values and it seemed to do the right thing | 20:21 |
| iurygregory | the magic of Ironic ^ | 20:22 |
| dhellmann | iurygregory : what I want to do is set it to a value that I know should not work with the host and then verify that provisioning does not work | 20:22 |
| TheJulia | dhellmann: inspection | 20:22 |
| dhellmann | inspection sets the other values? | 20:22 |
| TheJulia | it does | 20:22 |
| TheJulia | traits that can be used for nova to schedule against | 20:22 |
| dhellmann | that makes sense. what was confusing is that even though I gave the wrong type for capabilities to begin with, inspection "fixed" it | 20:22 |
| dhellmann | without my registration call, I see this: 'capabilities': 'cpu_vt:true,cpu_aes:true,cpu_hugepages:true,cpu_hugepages_1g:true' | 20:23 |
| dhellmann | with it, I saw 'capabilities': 'boot_mode:uefi,cpu_vt:true,cpu_aes:true,cpu_hugepages:true,cpu_hugepages_1g:true' | 20:23 |
| iurygregory | if you used PATCH and the deployment failed that would be the sign... | 20:23 |
| dhellmann | the boot_mode was the first item in the string | 20:23 |
| dhellmann | here's the change I'm making: https://github.com/metal3-io/baremetal-operator/pull/586/commits/d5c740b5c29601da46cd989989bc2020f0f0381b | 20:24 |
| iurygregory | legacy would be "bios" | 20:25 |
| iurygregory | https://github.com/openstack/ironic/blob/master/ironic/common/boot_modes.py | 20:26 |
| TheJulia | yeah, looks like the mapping needs to be changed if I'm groking what I'm looking at correctly | 20:28 |
| dhellmann | fixed, thanks | 20:28 |
| dhellmann | https://github.com/metal3-io/baremetal-operator/pull/586/commits/a851f784d236dec27bc975ff27424765c2c837e6 | 20:29 |
| dhellmann | https://github.com/metal3-io/baremetal-operator/pull/586/commits | 20:29 |
| TheJulia | now where did my braingo | 20:29 |
| TheJulia | ugh, about another hour before the upper requirements change should merge | 20:31 |
| TheJulia | Then the recheck party shall begin | 20:31 |
| *** k_mouza has joined #openstack-ironic | 20:32 | |
| iurygregory | my brain is not in the best state to review Go code, I will do as the first thing in the morning after some coffee =) | 20:32 |
| janders | good morning Ironic o/ | 20:35 |
| TheJulia | good morning janders | 20:35 |
| janders | (or good evening or good afternoon) | 20:35 |
| janders | :) | 20:35 |
| TheJulia | Early afternoon here | 20:37 |
| iurygregory | morning janders | 20:38 |
| janders | hey iurygregory :) | 20:38 |
| *** lbotell has joined #openstack-ironic | 20:42 | |
| *** lbotell94 has joined #openstack-ironic | 20:43 | |
| *** lbotell94 has left #openstack-ironic | 20:48 | |
| *** lbotell94 has joined #openstack-ironic | 20:49 | |
| *** lbotell94 has left #openstack-ironic | 20:54 | |
| *** lbotell94 has joined #openstack-ironic | 20:54 | |
| *** k_mouza has quit IRC | 21:06 | |
| *** lbotell94 has quit IRC | 21:08 | |
| janders | I had a look at the low-hanging-fruits on the storyboard and would like to look deeper into https://storyboard.openstack.org/#!/story/2007646 | 21:09 |
| *** dsneddon has joined #openstack-ironic | 21:11 | |
| janders | what's the best setup to work on this? devstack? bifrost? | 21:12 |
| TheJulia | looking | 21:48 |
| TheJulia | janders: bifrost most likely since you don't need anything besides ironic | 21:50 |
| janders | ok! thank you TheJulia | 21:55 |
| TheJulia | no problem :) | 22:16 |
| TheJulia | well, only 2 out of 9 rechecked items are already failing. I "guess" that is a good sign | 22:17 |
| TheJulia | arne_wiebalck: it occurs to me that the best place to check the oom patch is the multinode job... just turn the value up to like 4GB... | 22:22 |
| TheJulia | so half way in the conductors stop taking more work on | 22:23 |
| TheJulia | well.. heavy memory work | 22:23 |
| *** rcernin has joined #openstack-ironic | 22:44 | |
| *** rcernin has quit IRC | 22:51 | |
| *** rcernin has joined #openstack-ironic | 22:51 | |
| *** tkajinam has joined #openstack-ironic | 22:58 | |
| *** lbotell14 has quit IRC | 23:09 | |
| *** lbotell1 has joined #openstack-ironic | 23:10 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Creating a new state transition "recover" https://review.opendev.org/741063 | 23:15 |
| janders | just getting started on this one... workflow-1 | 23:17 |
| *** sw3 has quit IRC | 23:31 | |
| TheJulia | oh, no worries :) | 23:36 |
| TheJulia | ugh, 2 in gate... I'm sensing our failure rate is still bad | 23:36 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Add json and param parsing to args https://review.opendev.org/704489 | 23:37 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Add wsme core types, remove WSME https://review.opendev.org/704490 | 23:37 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert root path / to use plain JSON https://review.opendev.org/740142 | 23:37 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert root controller to plain controller https://review.opendev.org/741064 | 23:37 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert v1 controller to plain, return JSON https://review.opendev.org/741065 | 23:37 |
| *** janders has quit IRC | 23:39 | |
| *** rloo has quit IRC | 23:46 | |
| TheJulia | hmm | 23:47 |
| TheJulia | stevebaker: my that is a number of changes :) | 23:47 |
| stevebaker | TheJulia: that series might get frighteningly long once I dig in. I think I'll just have to keep some change unproposed to be nice to CI | 23:50 |
| TheJulia | stevebaker: don't worry about CI | 23:52 |
| TheJulia | I think if I can get a couple reviews on the the fixes around ironic-inspector it would help thins | 23:52 |
| TheJulia | things | 23:52 |
| stevebaker | I'll take a look | 23:54 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!