| janders | TheJulia dtantsur|afk regarding https://review.opendev.org/#/c/762998/ for now I 1) disabled autodiscovery in my lab and 2) updated my lab setup doco to do this every time. I ran some tests and I am happy with how things work now. I am still happy to reach out to ML/OPS guys to see what the general sentiment is about this change, but those changes | 00:00 |
|---|---|---|
| janders | solve my immediate problem. Thanks for good advice! :) | 00:00 |
| patchbot | patch 762998 - bifrost - Disable inspector discovery by default - 2 patch sets | 00:00 |
| janders | s/changes/tweaks I should say :) | 00:01 |
| openstackgerrit | Merged openstack/ironic-inspector stable/victoria: Power off before inspection https://review.opendev.org/762651 | 00:20 |
| openstackgerrit | Merged openstack/ironic master: Update .rst files https://review.opendev.org/759236 | 00:20 |
| openstackgerrit | Merged openstack/ironic master: Minor follow-up doc change https://review.opendev.org/763045 | 00:21 |
| *** Goneri has quit IRC | 00:36 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Guard conductor from consuming all of the ram https://review.opendev.org/726483 | 00:40 |
| *** rloo has quit IRC | 00:47 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Set default to prevent out of memory conditions https://review.opendev.org/763107 | 00:48 |
| *** k_mouza has joined #openstack-ironic | 01:08 | |
| *** jawad_axd has quit IRC | 01:19 | |
| *** k_mouza has quit IRC | 01:20 | |
| *** k_mouza has joined #openstack-ironic | 01:20 | |
| *** k_mouza has quit IRC | 01:25 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-inspector stable/victoria: Revert "Power off before inspection" https://review.opendev.org/763111 | 01:30 |
| *** rcernin has quit IRC | 01:43 | |
| *** stevebaker has quit IRC | 01:59 | |
| openstackgerrit | likui proposed openstack/ironic-inspector master: Reuse the docs deps to benefit from constraints https://review.opendev.org/763116 | 02:00 |
| *** rcernin has joined #openstack-ironic | 02:20 | |
| openstackgerrit | likui proposed openstack/ironic-python-agent master: Reuse the docs deps to benefit from constraints https://review.opendev.org/763117 | 02:21 |
| *** rcernin has quit IRC | 02:24 | |
| *** rcernin has joined #openstack-ironic | 02:28 | |
| *** rcernin has quit IRC | 02:31 | |
| *** rcernin has joined #openstack-ironic | 02:32 | |
| *** stevebaker has joined #openstack-ironic | 02:37 | |
| *** Qianbiao has joined #openstack-ironic | 02:38 | |
| openstackgerrit | Merged openstack/ironic master: Limit the default value of [api]api_workers to 4 https://review.opendev.org/761185 | 03:06 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix disk label to account for UEFI https://review.opendev.org/762540 | 03:10 |
| *** mkrai has joined #openstack-ironic | 03:21 | |
| openstackgerrit | Merged openstack/ironic master: New argument validate decorator https://review.opendev.org/742795 | 03:23 |
| stevebaker | \o/ | 03:26 |
| *** stevebaker has quit IRC | 03:50 | |
| Qianbiao | Morning stevebaker | 03:50 |
| *** k_mouza has joined #openstack-ironic | 04:21 | |
| *** zzzeek has quit IRC | 04:24 | |
| *** zzzeek has joined #openstack-ironic | 04:25 | |
| *** k_mouza has quit IRC | 04:26 | |
| *** Qianbiao has quit IRC | 04:37 | |
| *** paras333 has quit IRC | 04:37 | |
| *** Qianbiao has joined #openstack-ironic | 04:38 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Use OOB inspection to fetch MACs for IB inspection https://review.opendev.org/750943 | 04:38 |
| *** zzzeek has quit IRC | 04:46 | |
| *** zzzeek has joined #openstack-ironic | 04:49 | |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Use OOB inspection to fetch MACs for IB inspection https://review.opendev.org/750943 | 05:14 |
| *** anuradha1904 has joined #openstack-ironic | 05:32 | |
| *** tzumainn has quit IRC | 05:37 | |
| *** ociuhandu has joined #openstack-ironic | 05:44 | |
| *** zzzeek has quit IRC | 05:47 | |
| *** zzzeek has joined #openstack-ironic | 05:48 | |
| *** ociuhandu has quit IRC | 05:49 | |
| *** gyee has quit IRC | 06:08 | |
| *** Qianbiao has quit IRC | 06:20 | |
| arne_wiebalck | Good morning, ironic! | 06:41 |
| *** Qianbiao has joined #openstack-ironic | 06:46 | |
| *** rpittau|afk is now known as rpittau | 06:49 | |
| rpittau | good morning ironic! o/ | 06:49 |
| openstackgerrit | Merged openstack/ironic master: Add expose body decorator, status_code argument https://review.opendev.org/743677 | 06:52 |
| openstackgerrit | Merged openstack/ironic master: Duplicate trait validation with jsonschema https://review.opendev.org/745778 | 07:10 |
| openstackgerrit | Merged openstack/ironic master: Utility functions for REST API JSON handling https://review.opendev.org/751160 | 07:10 |
| openstackgerrit | Merged openstack/ironic master: Convert allocations endpoint to plain JSON https://review.opendev.org/743679 | 07:10 |
| openstackgerrit | Merged openstack/ironic master: Convert bios endpoint to plain JSON https://review.opendev.org/745475 | 07:10 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Convert chassis endpoint to plain JSON https://review.opendev.org/745476 | 07:10 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Convert conductors endpoint to plain JSON https://review.opendev.org/745779 | 07:10 |
| *** dtantsur|afk is now known as dtantsur | 07:51 | |
| dtantsur | morning ironic | 07:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/victoria: Limit the default value of [api]api_workers to 4 https://review.opendev.org/763133 | 07:52 |
| iurygregory | good morning arne_wiebalck rpittau dtantsur and Ironic o/ | 07:55 |
| rpittau | hey iurygregory :) | 07:58 |
| rpittau | good morning dtantsur :) | 07:58 |
| dtantsur | who's up for the wsme review jam? | 07:58 |
| rpittau | I'm in! | 07:58 |
| rpittau | just need to find my glasses.... | 07:59 |
| iurygregory | I will join in a few | 07:59 |
| dtantsur | get acquainted with https://review.opendev.org/#/c/751160/ since a lot of functions are used from it | 07:59 |
| patchbot | patch 751160 - ironic - Utility functions for REST API JSON handling (MERGED) - 16 patch sets | 07:59 |
| iurygregory | need one more coffee before looking at code hehe | 07:59 |
| dtantsur | for voice: https://meetpad.opendev.org/ironic | 08:00 |
| *** stevebaker has joined #openstack-ironic | 08:01 | |
| stevebaker | dtantsur: hey | 08:02 |
| dtantsur | stevebaker: o/ | 08:02 |
| rpittau | hey stevebaker :) | 08:02 |
| dtantsur | same link as yesterday | 08:02 |
| stevebaker | rpittau: ohai | 08:02 |
| stevebaker | https://meetpad.opendev.org/ironic | 08:03 |
| janders | hi all | 08:03 |
| janders | sorry for being a couple minutes late | 08:03 |
| dtantsur | no problem at all | 08:03 |
| janders | no webcam for me, straight off the bike :) | 08:04 |
| stevebaker | its a no camera session | 08:04 |
| janders | sounds good | 08:05 |
| *** abdysn has joined #openstack-ironic | 08:05 | |
| *** fmuyassarov has joined #openstack-ironic | 08:06 | |
| *** k_mouza has joined #openstack-ironic | 08:22 | |
| dtantsur | stevebaker: can you actually hear me? | 08:25 |
| dtantsur | I have a feeling that only janders hears me.. | 08:25 |
| rpittau | dtantsur: I can't :/ | 08:25 |
| stevebaker | dtantsur: nope | 08:26 |
| *** k_mouza has quit IRC | 08:26 | |
| dtantsur | well, thank you jitsi.. | 08:26 |
| dtantsur | with chromium broken I don't know what my options are... | 08:27 |
| janders | maybe it would be worthwhile jumping to Bluejeans? | 08:27 |
| janders | stevebaker rpittau iurygregory what do you reckon? | 08:27 |
| iurygregory | I'm using firefox | 08:28 |
| iurygregory | but I think the correct thing is to use google chrome for jitsi | 08:28 |
| janders | I'm on chromium | 08:28 |
| dtantsur | iurygregory: sure (I mean, it's lame, but whatever), but it's completely broken atm | 08:28 |
| janders | maybe it only works between matching browsers? :D | 08:28 |
| iurygregory | yeah no worries | 08:28 |
| iurygregory | =) | 08:28 |
| dtantsur | okay, for everyone except for janders :) | 08:28 |
| rpittau | I'm also using firefox | 08:29 |
| stevebaker | i'm on firefox too | 08:29 |
| janders | maybe chromium only works when your laptop is upside down? | 08:29 |
| stevebaker | https://redhat.bluejeans.com/5541716156 | 08:29 |
| iurygregory | janders, for chromium there is a bug open hehe | 08:30 |
| janders | maybe I need to install the latest updates to get the bug, LOL | 08:30 |
| iurygregory | janders, https://bugzilla.redhat.com/show_bug.cgi?id=1895920 | 08:31 |
| openstack | bugzilla.redhat.com bug 1895920 in chromium "Chromium 86 crashes on WebRTC videos when switching window" [Urgent,New] - Assigned to spotrh | 08:31 |
| iurygregory | blujeans decided to kick me D: | 08:44 |
| *** iurygregory has quit IRC | 08:44 | |
| *** tosky has joined #openstack-ironic | 08:45 | |
| *** ociuhandu has joined #openstack-ironic | 08:47 | |
| *** iurygregory has joined #openstack-ironic | 08:50 | |
| *** rcernin has quit IRC | 08:52 | |
| *** lucasagomes has joined #openstack-ironic | 09:02 | |
| janders | wil go grab some dinner now | 09:07 |
| *** mgoddard has joined #openstack-ironic | 09:07 | |
| janders | thanks for including me - sorry I 'm not in position to contribute a whole lot to the discussion yet, but it was super informative to me | 09:07 |
| janders | it's good to know where all these things live in the code | 09:08 |
| dtantsur | janders: this is great :) | 09:09 |
| *** derekh has joined #openstack-ironic | 09:10 | |
| *** rcernin has joined #openstack-ironic | 09:15 | |
| *** Abdallahyas has joined #openstack-ironic | 09:21 | |
| *** abdysn has quit IRC | 09:23 | |
| *** abdysn has joined #openstack-ironic | 09:31 | |
| *** ociuhandu has quit IRC | 09:32 | |
| *** Abdallahyas has quit IRC | 09:33 | |
| *** rcernin has quit IRC | 09:45 | |
| *** ociuhandu has joined #openstack-ironic | 09:49 | |
| *** ociuhandu has quit IRC | 09:49 | |
| *** ociuhandu has joined #openstack-ironic | 09:49 | |
| *** yolanda__ has joined #openstack-ironic | 09:50 | |
| *** yoctozepto has quit IRC | 09:50 | |
| *** yoctozepto has joined #openstack-ironic | 09:51 | |
| *** rcernin has joined #openstack-ironic | 09:55 | |
| *** zzzeek has quit IRC | 09:56 | |
| *** zzzeek has joined #openstack-ironic | 09:59 | |
| *** k_mouza has joined #openstack-ironic | 10:14 | |
| iurygregory | dtantsur, not sure if I sent the link for the openstacksdk change before https://review.opendev.org/#/c/753092/ (in case I forgot...) | 10:19 |
| patchbot | patch 753092 - openstacksdk - Volume Target support for Ironic on OpenStack SDK - 13 patch sets | 10:19 |
| dtantsur | I have it bookmarked, just never got enough time | 10:19 |
| iurygregory | no worries =) | 10:23 |
| *** Abdallahyas has joined #openstack-ironic | 10:29 | |
| *** abdysn has quit IRC | 10:33 | |
| *** Abdallahyas has quit IRC | 10:35 | |
| dtantsur | iurygregory: so, I've prototyped a proposal for BMC events, wdyt? https://storyboard.openstack.org/#!/story/2008366 | 10:44 |
| dtantsur | ditto rpittau and everyone who cares | 10:44 |
| iurygregory | dtantsur, looking | 10:44 |
| dtantsur | it's probably worth a spec, but I need to get feedback first | 10:44 |
| *** dougsz has joined #openstack-ironic | 10:46 | |
| *** belmoreira has joined #openstack-ironic | 10:46 | |
| iurygregory | in my mind we can just make sure it works for redfish.. | 10:48 |
| dtantsur | we cannot introduce API just for redfish though | 10:48 |
| rpittau | api needs to be generic | 10:48 |
| iurygregory | yeah but we can give error in case not using redfish? | 10:48 |
| iurygregory | like not sure if ipmi has support for events subscription | 10:49 |
| dtantsur | UnsupportedDriverExtension or how it's called | 10:49 |
| dtantsur | IPMI probably not, I'd expect all vendors drivers to actually have something like that | 10:49 |
| dtantsur | but dunno | 10:49 |
| iurygregory | yeah | 10:49 |
| dtantsur | we could go down the vendor passthru path | 10:49 |
| dtantsur | note that we *may* implement a poll model internally | 10:50 |
| dtantsur | i.e. a periodic task that collects events than sends them | 10:50 |
| rpittau | mmm so the event won't be sent immediately when it's generated | 10:50 |
| dtantsur | in a poll model - no | 10:51 |
| dtantsur | in the redfish model - pretty immediately | 10:51 |
| iurygregory | oh ok | 10:51 |
| dtantsur | the next thing I'm thinking about is how to squash identical subscriptions together | 10:52 |
| dtantsur | i.e. if two users request the same subscription to the same node, end up with only one subscription | 10:53 |
| iurygregory | yeah its important to see how this would work | 10:55 |
| rpittau | I guess we'll have to define how subscriptions can be considered identical | 10:56 |
| dtantsur | yep, writing it right now | 10:57 |
| rpittau | in which case I mean | 10:57 |
| rpittau | ok | 10:57 |
| * rpittau will check later, cooking time! | 10:58 | |
| *** yolanda__ is now known as yolanda | 11:00 | |
| rpittau | . | 11:00 |
| dtantsur | rpittau, iurygregory, updated | 11:00 |
| iurygregory | dtantsur, ack | 11:00 |
| *** dougsz has quit IRC | 11:03 | |
| *** abdysn has joined #openstack-ironic | 11:09 | |
| fmuyassarov | Hello Everyone! What is the recommended way of testing my local changes in the python-ironicclient? | 11:11 |
| dtantsur | fmuyassarov: wouldn't just testing against ironic work? | 11:12 |
| fmuyassarov | dtantsur, I have added a new field in the node object (in the ironicclient) and now I wanted to test it against my ironic containers. | 11:15 |
| fmuyassarov | it is related to disk cleaning what we discussed earlier last week, so I added no_automated_clean in the ironicclient but not sure how to test it | 11:17 |
| *** Qianbiao has quit IRC | 11:17 | |
| *** Qianbiao has joined #openstack-ironic | 11:18 | |
| dtantsur | fmuyassarov: if you can access ironic, you can create a new node (may be invalid) and play with it | 11:18 |
| *** rcernin has quit IRC | 11:18 | |
| *** rcernin has joined #openstack-ironic | 11:19 | |
| *** parallax has joined #openstack-ironic | 11:22 | |
| *** mkrai has quit IRC | 11:24 | |
| *** rcernin has quit IRC | 11:27 | |
| fmuyassarov | dtantsur, Sorry, but I'm a bit confused. Let's say I want to run "baremetal node set --no-automated-clean node_UUID" which should set node.automated_clean to False. So, in Metal3-dev-env we download quay.io/metal3-io/ironic-client image for the Ironic python client. Can I, for example, create a docker image out of my python-ironicclient branch and reference that docker image in the Metal3-dev-env ? | 11:30 |
| janders | dtantsur TheJulia trandles a very cool story from StackHPC: https://www.stackhpc.com/sc20-top500.html - an Ironic machine in the top 100 of the Top500 :) | 11:31 |
| *** rcernin has joined #openstack-ironic | 11:33 | |
| *** Qianbiao has quit IRC | 11:40 | |
| openstackgerrit | Merged openstack/ironic master: Convert chassis endpoint to plain JSON https://review.opendev.org/745476 | 11:45 |
| *** rcernin has quit IRC | 11:46 | |
| *** rcernin has joined #openstack-ironic | 11:52 | |
| dtantsur | fmuyassarov: I'm not strong in containers, I'd recommend just installing your checkout of ironicclient in a virtual env | 11:52 |
| *** ociuhandu has quit IRC | 11:52 | |
| dtantsur | janders: Oooooooo! | 11:52 |
| fmuyassarov | dtantsur, sure. Thank you. | 11:53 |
| *** rcernin has quit IRC | 11:55 | |
| *** ociuhandu has joined #openstack-ironic | 11:58 | |
| *** ociuhandu has quit IRC | 11:58 | |
| *** Qianbiao has joined #openstack-ironic | 12:08 | |
| Qianbiao | <fmuyassarov> yes, ofc you can update your ironicclient in docker image. | 12:11 |
| openstackgerrit | Merged openstack/ironic master: Convert conductors endpoint to plain JSON https://review.opendev.org/745779 | 12:12 |
| openstackgerrit | Merged openstack/ironic master: Convert deploy_templates endpoint to plain JSON https://review.opendev.org/745780 | 12:12 |
| openstackgerrit | Merged openstack/ironic master: Convert drivers endpoint to plain JSON https://review.opendev.org/741865 | 12:13 |
| dtantsur | go-go-go! | 12:13 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Convert event endpoint to plain JSON https://review.opendev.org/746623 | 12:13 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Convert nodes endpoint to plain JSON https://review.opendev.org/747820 | 12:13 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Convert ports endpoint to plain JSON https://review.opendev.org/750120 | 12:13 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Convert portgroups endpoint to plain JSON https://review.opendev.org/750864 | 12:13 |
| dtantsur | boooooo | 12:13 |
| *** zzzeek has quit IRC | 12:13 | |
| rpittau | :/ | 12:13 |
| *** zzzeek has joined #openstack-ironic | 12:16 | |
| *** zzzeek has quit IRC | 12:27 | |
| *** zzzeek has joined #openstack-ironic | 12:28 | |
| *** ociuhandu has joined #openstack-ironic | 12:30 | |
| *** Abdallahyas has joined #openstack-ironic | 12:37 | |
| *** thiagop has joined #openstack-ironic | 12:38 | |
| *** abdysn has quit IRC | 12:41 | |
| *** abdysn has joined #openstack-ironic | 12:41 | |
| *** Abdallahyas has quit IRC | 12:42 | |
| iurygregory | time to offer cookies to zuul | 12:43 |
| janders | I suggest cookies soaked in booze.... | 12:46 |
| janders | biscotti & vin santo? :) | 12:46 |
| *** Qianbiao has quit IRC | 12:51 | |
| *** Qianbiao has joined #openstack-ironic | 12:51 | |
| *** anuradha1904 has quit IRC | 12:52 | |
| iurygregory | probably =) | 12:56 |
| *** paras333 has joined #openstack-ironic | 12:56 | |
| *** Abdallahyas has joined #openstack-ironic | 12:59 | |
| *** abdysn has quit IRC | 13:00 | |
| *** abdysn has joined #openstack-ironic | 13:00 | |
| *** Abdallahyas has quit IRC | 13:04 | |
| janders | see you tomorrow Ironic o/ | 13:04 |
| *** dtantsur is now known as dtantsur|brb | 13:06 | |
| *** abdysn has quit IRC | 13:10 | |
| *** Goneri has joined #openstack-ironic | 13:29 | |
| *** ociuhandu has quit IRC | 13:32 | |
| fmuyassarov | Qianbiao, thanks for the reply. Will try that. | 14:05 |
| Qianbiao | fmuyassarov yw | 14:06 |
| openstackgerrit | Verification of a change to openstack/ironic-tempest-plugin failed: Run redfish victoria job on focal https://review.opendev.org/760102 | 14:08 |
| *** rloo has joined #openstack-ironic | 14:08 | |
| *** mbuil has quit IRC | 14:12 | |
| *** dtantsur|brb is now known as dtantsur | 14:22 | |
| dtantsur | morning TheJulia | 14:22 |
| dtantsur | TheJulia: I'd like some early feedback on https://storyboard.openstack.org/#!/story/2008366 please | 14:22 |
| TheJulia | dtantsur: should I be well caffinated first? | 14:23 |
| TheJulia | or only semi-caffinated? | 14:23 |
| dtantsur | TheJulia: semi will work | 14:23 |
| *** ociuhandu has joined #openstack-ironic | 14:23 | |
| dtantsur | it's not rocket science there, rather a question of whether we even want to go that direction | 14:23 |
| TheJulia | but rocket science videos are the best videos on youtube | 14:24 |
| dtantsur | ++ | 14:24 |
| *** ociuhandu has quit IRC | 14:28 | |
| TheJulia | so we build an api to manage subscriptions, and we build an unauthenticated target endpoint under nodes (wasn't that a pita amount of code in vendor passthru for agent ?) to be able to receive and then ironic just turns around and retransmits. Does it do this retransmission from the API or is it the conductor? I could see the conductor, although this may be something, that we may need to craft explicit | 14:28 |
| TheJulia | scaling guidence around | 14:28 |
| *** ociuhandu has joined #openstack-ironic | 14:28 | |
| dtantsur | likely conductor. yes. | 14:28 |
| TheJulia | (it would also seem like retransmitting won't need an exclusive lock.... | 14:29 |
| TheJulia | ) | 14:29 |
| dtantsur | I think a shared lock should be fine | 14:29 |
| TheJulia | that locking is extra database activity we don't need under load if we can help it | 14:29 |
| TheJulia | Why is there the whole thing about rabbit? | 14:30 |
| dtantsur | because people are going to ask :) | 14:30 |
| dtantsur | an argument could be made that this work is taking it too far and we only need a vendor pass-thru to add/remove subscriptions directly in the BMC | 14:30 |
| *** fmuyassarov has quit IRC | 14:31 | |
| TheJulia | eh, ironic serving as an intermediary is good in larger scale more prescriptive environments anyway | 14:32 |
| TheJulia | may I slightly revise it? | 14:34 |
| dtantsur | of course, just let me know the tl;dr | 14:34 |
| TheJulia | tl;dr I'm going to revise the rabbitmq bit | 14:34 |
| dtantsur | ah, do whatever you want with it :) | 14:36 |
| TheJulia | Added one more bullet point | 14:38 |
| TheJulia | so it could also be a separate API surface, fwiw | 14:38 |
| TheJulia | or specific listener | 14:39 |
| TheJulia | that installs with ironic and uses the rpc code to send $things to the conductor | 14:39 |
| TheJulia | That would help avoid a lot of the code in the node object, fwiw | 14:39 |
| dtantsur | hmm, not sure I fully understand the idea | 14:40 |
| dtantsur | I'm afraid the complexity of maintaining another API service may overweight the benefits | 14:40 |
| TheJulia | That other API can be extremely stupid | 14:41 |
| dtantsur | sure, but it doesn't cancel out the fact that it has to be installed, configured and managed | 14:41 |
| TheJulia | and only handle the events | 14:41 |
| dtantsur | i.e. I'm talking about operation complexity, not code | 14:41 |
| TheJulia | Indeed, but at the same time using ironic's api adds additional networking traffic and connectivity requirements for BMC networks. | 14:41 |
| TheJulia | IF the listener is sitting on the BMC network, it is far more secure. | 14:42 |
| dtantsur | hmmm | 14:42 |
| dtantsur | it could be solved by dedicated instances of ironic API fwiw | 14:42 |
| TheJulia | keeping all the networking pieces working and flowing and changing how existing bmc networks work to allow a router and outbound traffic... may be too much for operators | 14:42 |
| TheJulia | this is also true | 14:43 |
| TheJulia | not something everyone groks with how ironic works until they understand the under the hood mechanics/design | 14:43 |
| TheJulia | We should likely have at least a diagram in our documentation on that. | 14:43 |
| *** tzumainn has joined #openstack-ironic | 14:45 | |
| dtantsur | one of the reason to try switching to flask: we could split our API into blueprints, with each of them potentially easily exposed via wsgi | 14:46 |
| TheJulia | yeah | 14:47 |
| TheJulia | One day hopefully | 14:52 |
| *** ociuhandu has quit IRC | 15:11 | |
| *** mkrai has joined #openstack-ironic | 15:19 | |
| *** ociuhandu has joined #openstack-ironic | 15:40 | |
| openstackgerrit | Merged openstack/ironic master: Convert event endpoint to plain JSON https://review.opendev.org/746623 | 15:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: [WIP] Copy any configuration from the virtual media https://review.opendev.org/763207 | 15:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] inject TLS certificate when using virtual media https://review.opendev.org/758427 | 15:50 |
| *** ociuhandu_ has joined #openstack-ironic | 15:59 | |
| *** ociuhandu has quit IRC | 16:02 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Fix default disk label with partition images https://review.opendev.org/762453 | 16:04 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Fix boot mode detection for partition images https://review.opendev.org/762531 | 16:08 |
| TheJulia | ^^ just commit message updates arne_wiebalck | 16:08 |
| openstackgerrit | Merged openstack/ironic master: Convert nodes endpoint to plain JSON https://review.opendev.org/747820 | 16:19 |
| openstackgerrit | Merged openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/762781 | 16:19 |
| *** mkrai has quit IRC | 16:19 | |
| *** mkrai has joined #openstack-ironic | 16:20 | |
| *** ociuhandu_ has quit IRC | 16:22 | |
| *** ociuhandu has joined #openstack-ironic | 16:22 | |
| *** Qianbiao has quit IRC | 16:27 | |
| *** belmoreira has quit IRC | 16:27 | |
| TheJulia | fyi, added the second community goal to our meeting agenda. It merged after I did the most recent revision of the priorities for the cycle. :( | 16:28 |
| *** ociuhandu has quit IRC | 16:37 | |
| *** mkrai has quit IRC | 16:38 | |
| *** ociuhandu has joined #openstack-ironic | 16:45 | |
| *** lucasagomes has quit IRC | 17:02 | |
| *** ociuhandu_ has joined #openstack-ironic | 17:03 | |
| *** ociuhandu has quit IRC | 17:06 | |
| *** ociuhandu_ has quit IRC | 17:07 | |
| arne_wiebalck | for direct deploy and image_download_source=http: are the images removed from the conductor immediately after deployment? | 17:07 |
| * dtantsur doesn't remember | 17:08 | |
| * arne_wiebalck is looking into moving from iscsi to direct and deployed a first node \o/ | 17:10 | |
| TheJulia | arne_wiebalck: \o/ | 17:12 |
| *** trandles has quit IRC | 17:25 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Fix vendor info retrieval for some versions of lshw https://review.opendev.org/762850 | 17:27 |
| openstackgerrit | Arne Wiebalck proposed openstack/ironic master: [doc][trivial] Fix spelling error https://review.opendev.org/763230 | 17:27 |
| *** mgoddard has quit IRC | 17:33 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Add fstab pointer to EFI partition https://review.opendev.org/748260 | 17:33 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Add clean step 'erase_pstore' https://review.opendev.org/762537 | 17:38 |
| openstackgerrit | Merged openstack/python-ironicclient master: Use TOX_CONSTRAINTS_FILE https://review.opendev.org/762451 | 17:38 |
| arne_wiebalck | bye everyone o/ | 17:47 |
| *** k_mouza has quit IRC | 18:05 | |
| *** k_mouza has joined #openstack-ironic | 18:06 | |
| *** k_mouza has quit IRC | 18:10 | |
| *** derekh has quit IRC | 18:14 | |
| *** dtantsur is now known as dtantsur|afk | 18:15 | |
| dtantsur|afk | o/ | 18:15 |
| *** hjensas__ is now known as hjensas | 18:22 | |
| -openstackstatus- NOTICE: The Gerrit service at review.opendev.org is being restarted quickly as a pre-upgrade sanity check, estimated downtime is less than 5 minutes. | 18:36 | |
| rpittau | good night! o/ | 18:39 |
| *** rpittau is now known as rpittau|afk | 18:39 | |
| *** k_mouza has joined #openstack-ironic | 19:00 | |
| *** openstackgerrit has quit IRC | 19:02 | |
| *** k_mouza has quit IRC | 19:04 | |
| *** openstackgerrit has joined #openstack-ironic | 19:24 | |
| openstackgerrit | Merged openstack/ironic master: Convert portgroups endpoint to plain JSON https://review.opendev.org/750864 | 19:24 |
| openstackgerrit | Merged openstack/ironic master: Convert ramdisk endpoint to plain JSON https://review.opendev.org/750857 | 19:24 |
| openstackgerrit | Merged openstack/ironic master: Convert volume endpoint to plain JSON https://review.opendev.org/750858 | 19:24 |
| openstackgerrit | Merged openstack/ironic-inspector stable/victoria: Revert "Power off before inspection" https://review.opendev.org/763111 | 19:24 |
| openstackgerrit | Merged openstack/ironic master: [doc][trivial] Fix spelling error https://review.opendev.org/763230 | 19:24 |
| stevebaker | morning | 19:40 |
| stevebaker | wow much merging | 19:41 |
| TheJulia | o/ | 19:43 |
| TheJulia | merging good :) | 19:44 |
| TheJulia | gives me less heartburn towards the end of cycle | 19:44 |
| TheJulia | gmann: o/ looking at https://bugs.launchpad.net/nova/+bug/1875418 does the same basic bug impact all projects regarding generated policies? | 19:58 |
| openstack | Launchpad bug 1875418 in OpenStack Compute (nova) "Generated policy.json in Ussuri is broken by default" [High,Fix released] - Assigned to Ghanshyam Mann (ghanshyammann) | 19:58 |
| openstackgerrit | James Denton proposed openstack/networking-generic-switch master: Support multiple links in link_local_information dict of portgroups https://review.opendev.org/557051 | 19:59 |
| gmann | TheJulia: not all, only who are moving towards the new policy (scope_type). | 20:00 |
| TheJulia | okay | 20:00 |
| gmann | TheJulia: and to avoid this in advance before service start migrating to new RBAC , this community goal is one effort to deprecate the JSON formatted file - https://governance.openstack.org/tc/goals/selected/wallaby/migrate-policy-format-from-json-to-yaml.html | 20:01 |
| TheJulia | yeah, I just managed to get the code working and passing in ironic and I'm doing the reno for that | 20:01 |
| TheJulia | which was why I'm asking for clarification :) | 20:02 |
| gmann | ok | 20:03 |
| TheJulia | I'm making ironic's reno more vague in accordance with the prophecy | 20:04 |
| *** gyee has joined #openstack-ironic | 20:10 | |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Introduce common personas for secure RBAC https://review.opendev.org/763255 | 20:18 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Bump oslo.log requirement to 4.3.0 https://review.opendev.org/763256 | 20:18 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for baremetal nodes https://review.opendev.org/763257 | 20:18 |
| iurygregory | speaking of RBAC new patches appeared O.o | 20:20 |
| iurygregory | everyday I'm starting to believe more and more in magic... | 20:21 |
| TheJulia | heh | 20:24 |
| TheJulia | lol | 20:24 |
| TheJulia | gmann: btw, I didn't see a storyboard story referenced, is the effort being tracked in storyboard? | 20:25 |
| gmann | TheJulia: i have not setup storyboard for this from my experience of previous goal tracking. storyboard are not updated by many and it slow too. I will setup etehrpad this week to track the work | 20:26 |
| gmann | TheJulia: if you use the gerrit topic mentioned in goal doc, it will be easy to track | 20:26 |
| *** lbragstad has joined #openstack-ironic | 20:27 | |
| TheJulia | gmann: I dunno, I found it kind of handy on the last few efforts Ironic had to participate in. Anyway... | 20:27 |
| TheJulia | lbragstad: you didn't need to toss up patches, I would have gone and wrapped my head around it. | 20:28 |
| TheJulia | lbragstad: but thanks! | 20:28 |
| gmann | TheJulia: did you add the fallback to json file logic also ? I am putting that in oslo_policy side (will push the patch soon) as central place and avoid doing it on each service side | 20:29 |
| TheJulia | gmann: I have... I mean if your going to do that, a lot of the example is pointless | 20:30 |
| gmann | TheJulia: ok, no issue replace it later like I will do in nova. oslo policy one will take time as it need new release etc | 20:32 |
| TheJulia | yeah | 20:32 |
| lbragstad | TheJulia no worries - they certainly aren't complete, but it at least gets the ball rolling | 20:32 |
| TheJulia | this at least makes it semi-compatible | 20:32 |
| TheJulia | btw, are the policy examples being switched to yaml format as well? | 20:32 |
| TheJulia | and already done it seems | 20:33 |
| gmann | TheJulia: yeah, also doc mentioning policy.json also | 20:33 |
| TheJulia | yup | 20:34 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for baremetal nodes https://review.opendev.org/763257 | 20:38 |
| * TheJulia blinks seeing that original policy text in ironic is from the days of newton | 20:38 | |
| *** ricolin has quit IRC | 20:40 | |
| lbragstad | i think that's when alaski was working on all this stuff | 20:42 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Policy json to yaml migration https://review.opendev.org/763262 | 20:50 |
| *** mgoddard has joined #openstack-ironic | 20:50 | |
| TheJulia | lbragstad: I was looking at some of it briefly yesterday and finding lines typed out by aeva orignally... so it was a long... long... time ago. | 20:51 |
| * lbragstad nods | 20:51 | |
| TheJulia | gmann: ^^ one down, one to go, but I'm going to hold off on ironic-inspector until I see that at least unit test wise in CI things seem good. | 20:51 |
| * iurygregory adds the patch to his review list | 21:02 | |
| *** rcernin has joined #openstack-ironic | 21:13 | |
| * TheJulia goes into town | 21:37 | |
| *** ayoung has joined #openstack-ironic | 21:40 | |
| *** mgoddard has quit IRC | 21:44 | |
| *** trandles has joined #openstack-ironic | 21:49 | |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for ports https://review.opendev.org/763267 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for port groups https://review.opendev.org/763268 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for chassis https://review.opendev.org/763269 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for baremetal drivers https://review.opendev.org/763270 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for node and driver passthru https://review.opendev.org/763271 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for utility APIs https://review.opendev.org/763272 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for volume APIs https://review.opendev.org/763273 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for conductor APIs https://review.opendev.org/763274 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for the allocation APIs https://review.opendev.org/763275 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for the event APIs https://review.opendev.org/763276 | 22:05 |
| openstackgerrit | Lance Bragstad proposed openstack/ironic master: Implement secure RBAC for the deploy templates APIs https://review.opendev.org/763277 | 22:05 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/connectors endpoint to plain JSON https://review.opendev.org/750859 | 22:06 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/targets endpoint to plain JSON https://review.opendev.org/750860 | 22:07 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Expunge the internal version of WSME https://review.opendev.org/750861 | 22:07 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Don't allow patching port internal_info https://review.opendev.org/763278 | 22:07 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: JSON conversion followup change https://review.opendev.org/763279 | 22:07 |
| *** ayoung has quit IRC | 22:18 | |
| *** ayoung has joined #openstack-ironic | 22:20 | |
| *** rh-jlabarre has joined #openstack-ironic | 22:25 | |
| *** vkmc has quit IRC | 22:26 | |
| *** rpioso has quit IRC | 22:26 | |
| *** naved001 has quit IRC | 22:26 | |
| *** vkmc has joined #openstack-ironic | 22:26 | |
| *** rpioso has joined #openstack-ironic | 22:26 | |
| *** naved001 has joined #openstack-ironic | 22:26 | |
| *** johnsom has quit IRC | 22:27 | |
| *** rh-jelabarre has quit IRC | 22:28 | |
| *** johnsom has joined #openstack-ironic | 22:29 | |
| *** thiagop has quit IRC | 22:40 | |
| *** paras333 has quit IRC | 22:44 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Policy json to yaml migration https://review.opendev.org/763262 | 22:44 |
| *** paras333 has joined #openstack-ironic | 22:55 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Reuse the docs deps to benefit from constraints https://review.opendev.org/763116 | 23:06 |
| JayF | TheJulia: am I nuts, or does JSON not parse as valid YAML? | 23:08 |
| JayF | TheJulia: (looking at 763262) | 23:08 |
| JayF | I thought JSON was valid YAML | 23:08 |
| TheJulia | it can be converted | 23:08 |
| TheJulia | but doesn't translate when you do a nested object | 23:08 |
| JayF | So some JSON is valid YAML, but not all? | 23:09 |
| TheJulia | basically yes when converted | 23:10 |
| JayF | OK, that helps. I was wondering why we were putting so much effort into folks properly migrating JSON files | 23:10 |
| * TheJulia goes and prepares prepares the dinner for the slow cookening | 23:11 | |
| * JayF gives that a real review now | 23:11 | |
| TheJulia | JayF: more than that is there is desire to move to secure rbac across projects | 23:11 |
| TheJulia | see lance's patches | 23:11 |
| TheJulia | to make that easy to digest, it will need to be yaml, at least that is why I suspect | 23:11 |
| * TheJulia prepares the cookening | 23:12 | |
| JayF | I'm on some seriously old "policy should be yaml" bugs | 23:12 |
| JayF | so I'm glad it's finally happening | 23:12 |
| JayF | but that fueled some of my confusion (because I remembered some project shipped YAML policy sample files, and shifted parsing to a yaml parser to permit comments) | 23:12 |
| *** ddurst has quit IRC | 23:14 | |
| *** ddurst has joined #openstack-ironic | 23:18 | |
| janders | good morning Ironic o/ | 23:23 |
| TheJulia | good morning janders | 23:23 |
| janders | had some really weird H/W issues these morning... Lenovo laptop won't charge off a non-Lenovo dock unless it's temporarily plugged into a Lenovo dock first | 23:23 |
| TheJulia | ugh | 23:24 |
| TheJulia | what model? | 23:24 |
| janders | before you ask why would one do this - we swap desks with my wife, depending on who needs nice background for video calls | 23:24 |
| janders | and she's Dell I'm Lenovo | 23:24 |
| janders | T490s | 23:25 |
| TheJulia | ok | 23:25 |
| janders | sounds like a real bad mix of over-engineering and bugs :/ | 23:25 |
| TheJulia | I think i leaned towards mobility on my hardware refresh so I'm curious what will appear | 23:25 |
| janders | power should be power... though I do get it why it can be more complex with USB-C docks | 23:26 |
| janders | I plugged it into the other desk suspecting the Dell dock is hung or dead but no... docks are fine, it's the T490 | 23:26 |
| janders | "nice" | 23:26 |
| janders | oh well morning dose of frustration tick, now back to work :) | 23:27 |
| janders | Thank you for reviewing the OOB/MAC+inspector patches TheJulia iurygregory and dtantsur|afk - appreciate your feedback :) will be working on these patches today | 23:37 |
| JayF | My HP Envy takes either a barrel plug or a USB-C. Both work great at full wattage. HP software (installed by windows update drivers) gives a scary UNAPPROVED CHARGER notification when you do so though. | 23:41 |
| JayF | It's silly to have a universal (USB-C) charging standard then be like "nah, use ours instead" | 23:42 |
| gmann | TheJulia: thanks, overall lgtm. you need to replace Victoria to Wallaby in few places | 23:44 |
| TheJulia | gmann: well, we can't backport this. | 23:46 |
| TheJulia | really... | 23:46 |
| TheJulia | maybe we could, but this whole thing is putting us in a not great situation | 23:46 |
| * TheJulia has strong feels that shoudl likely just enjoy a tasty beverage at this point | 23:46 | |
| gmann | yeah backporting is not good idea for deprecating things | 23:47 |
| iurygregory | janders, morning np o/ | 23:47 |
| TheJulia | iurygregory: go to bed | 23:48 |
| iurygregory | TheJulia, sorry I was watching a presentation from the Andromedev projects since I was a mentor | 23:48 |
| TheJulia | time for ZzZzZzZzZz | 23:48 |
| TheJulia | ahh, cool | 23:48 |
| iurygregory | yeah | 23:49 |
| iurygregory | now I'm going to sleep hehe | 23:49 |
| iurygregory | the final result was https://review.opendev.org/#/c/753092/ =) | 23:51 |
| patchbot | patch 753092 - openstacksdk - Volume Target support for Ironic on OpenStack SDK - 13 patch sets | 23:51 |
| *** ayoung has quit IRC | 23:55 | |
| *** ayoung has joined #openstack-ironic | 23:57 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!