| janders | good morning Ironic o/ | 00:01 |
|---|---|---|
| openstackgerrit | Merged x/sushy-oem-idrac master: Add real-time status check to Lifecycle service https://review.opendev.org/c/x/sushy-oem-idrac/+/776224 | 00:28 |
| *** paras333 has joined #openstack-ironic | 00:31 | |
| *** tosky has quit IRC | 00:35 | |
| *** paras333 has quit IRC | 00:36 | |
| *** ociuhandu has joined #openstack-ironic | 00:47 | |
| *** ociuhandu has quit IRC | 01:00 | |
| *** ianychoi_ has joined #openstack-ironic | 01:03 | |
| iurygregory | morning janders o/ | 01:05 |
| *** mushrushu_ has joined #openstack-ironic | 01:06 | |
| *** early` has joined #openstack-ironic | 01:06 | |
| *** mushrushu has quit IRC | 01:12 | |
| *** ianychoi has quit IRC | 01:12 | |
| *** early has quit IRC | 01:13 | |
| *** jlvillal has quit IRC | 01:13 | |
| *** akahat has quit IRC | 01:13 | |
| *** lourot has quit IRC | 01:13 | |
| *** mushrushu_ is now known as mushrushu | 01:13 | |
| *** lourot has joined #openstack-ironic | 01:19 | |
| *** jamesdenton has quit IRC | 01:22 | |
| *** rloo has quit IRC | 01:23 | |
| *** jamesdenton has joined #openstack-ironic | 01:23 | |
| *** Rotan has joined #openstack-ironic | 01:52 | |
| *** mkrai has joined #openstack-ironic | 01:59 | |
| *** rcernin has quit IRC | 02:37 | |
| *** uzumaki has joined #openstack-ironic | 02:38 | |
| *** paras333 has joined #openstack-ironic | 02:48 | |
| *** paras333 has quit IRC | 02:53 | |
| *** Rotan has quit IRC | 03:01 | |
| *** rcernin has joined #openstack-ironic | 03:19 | |
| *** rcernin has quit IRC | 03:26 | |
| *** rcernin has joined #openstack-ironic | 03:31 | |
| *** mkrai has quit IRC | 03:43 | |
| *** mkrai has joined #openstack-ironic | 03:55 | |
| *** zzzeek has quit IRC | 04:18 | |
| *** zzzeek has joined #openstack-ironic | 04:22 | |
| *** ociuhandu has joined #openstack-ironic | 04:24 | |
| *** ociuhandu has quit IRC | 04:30 | |
| *** uzumaki has quit IRC | 04:50 | |
| *** paras333 has joined #openstack-ironic | 05:05 | |
| *** paras333 has quit IRC | 05:10 | |
| openstackgerrit | sonali bhausaheb borkar proposed x/sushy-oem-idrac master: Update export system config action constants https://review.opendev.org/c/x/sushy-oem-idrac/+/780703 | 05:19 |
| *** jamesdenton has quit IRC | 05:21 | |
| *** jamesden_ has joined #openstack-ironic | 05:22 | |
| *** k_mouza has joined #openstack-ironic | 06:06 | |
| *** anuradha1904 has joined #openstack-ironic | 06:09 | |
| *** k_mouza has quit IRC | 06:10 | |
| *** ociuhandu has joined #openstack-ironic | 06:13 | |
| *** ociuhandu has quit IRC | 06:17 | |
| *** ociuhandu has joined #openstack-ironic | 06:18 | |
| *** ociuhandu has quit IRC | 06:28 | |
| *** ociuhandu has joined #openstack-ironic | 06:29 | |
| *** ociuhandu has quit IRC | 06:35 | |
| *** ociuhandu has joined #openstack-ironic | 07:00 | |
| *** ociuhandu has quit IRC | 07:00 | |
| *** ociuhandu has joined #openstack-ironic | 07:00 | |
| *** paras333 has joined #openstack-ironic | 07:21 | |
| *** rcernin has quit IRC | 07:24 | |
| *** paras333 has quit IRC | 07:26 | |
| arne_wiebalck | Good morning, ironic! | 07:49 |
| *** jamesden_ has quit IRC | 07:49 | |
| *** jamesdenton has joined #openstack-ironic | 07:49 | |
| *** rcernin has joined #openstack-ironic | 07:54 | |
| *** rpittau|afk is now known as rpittau | 07:58 | |
| rpittau | good morning ironic! o/ | 07:58 |
| janders | good morning arne_wiebalck rpittau and Ironic o/ | 08:05 |
| arne_wiebalck | hey janders o/ | 08:05 |
| rpittau | hey janders :) | 08:05 |
| *** rcernin has quit IRC | 08:06 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/ussuri: [DNM] test ussuri CI https://review.opendev.org/c/openstack/ironic/+/780723 | 08:09 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Add tinyipa ussuri jobs https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/780556 | 08:12 |
| openstackgerrit | Moshiur Rahman proposed openstack/ironic-python-agent-builder master: Fix: IPA image building with OpenSuse. https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778726 | 08:17 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Centralize variables https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778942 | 08:17 |
| *** ociuhandu has quit IRC | 08:19 | |
| rpittau | quick doc update if anyone has a minute https://review.opendev.org/c/openstack/ironic/+/779206 | 08:23 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use git hash instead of release to build ipmitool https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/780554 | 08:29 |
| *** zigo has joined #openstack-ironic | 08:29 | |
| *** paras333 has joined #openstack-ironic | 08:48 | |
| *** paras333 has quit IRC | 08:53 | |
| *** tosky has joined #openstack-ironic | 09:00 | |
| *** derekh has joined #openstack-ironic | 09:06 | |
| *** lucasagomes has joined #openstack-ironic | 09:08 | |
| openstackgerrit | Merged openstack/ironic master: Update dev quickstart docs https://review.opendev.org/c/openstack/ironic/+/779206 | 09:08 |
| openstackgerrit | liuzhuangzhuang proposed openstack/ironic master: update grub file name for uefi boot mode https://review.opendev.org/c/openstack/ironic/+/780481 | 09:11 |
| *** akahat has joined #openstack-ironic | 09:48 | |
| *** sshnaidm has quit IRC | 09:50 | |
| openstackgerrit | Aija Jauntēva proposed x/sushy-oem-idrac master: Add import system configuration https://review.opendev.org/c/x/sushy-oem-idrac/+/770614 | 09:54 |
| *** dtantsur|afk is now known as dtantsur | 09:56 | |
| dtantsur | morning ironic | 09:56 |
| dtantsur | TheJulia: just a nit, you have 3 if clauses in a row, could be one joined with "and" | 09:57 |
| *** k_mouza has joined #openstack-ironic | 09:59 | |
| rpittau | good morning dtantsur :) | 10:01 |
| janders | good morning dtantsur o/ | 10:02 |
| *** Qianbiao has joined #openstack-ironic | 10:04 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Allow ansible deploys to be fast-tracked https://review.opendev.org/c/openstack/ironic/+/778005 | 10:20 |
| *** sshnaidm has joined #openstack-ironic | 10:27 | |
| *** ociuhandu has joined #openstack-ironic | 10:44 | |
| *** paras333 has joined #openstack-ironic | 11:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/victoria: Fix port id vs uuid in ValidateInterfacesHook https://review.opendev.org/c/openstack/ironic-inspector/+/780474 | 11:09 |
| *** paras333 has quit IRC | 11:10 | |
| iurygregory | good morning arne_wiebalck dtantsur rpittau janders and Ironic o/ | 11:35 |
| janders | hey iurygregory o/ | 11:44 |
| *** bdodd has quit IRC | 11:52 | |
| *** bdodd_ has joined #openstack-ironic | 11:52 | |
| arne_wiebalck | hey iurygregory o/ | 11:55 |
| *** ociuhandu has quit IRC | 12:18 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:25 | |
| *** ericlei has joined #openstack-ironic | 12:27 | |
| janders | see you tomorrow Ironic o/ | 12:34 |
| *** ociuhandu has joined #openstack-ironic | 12:38 | |
| *** ociuhandu has quit IRC | 12:39 | |
| *** ociuhandu has joined #openstack-ironic | 12:40 | |
| *** ericlei has quit IRC | 12:40 | |
| *** rcernin has joined #openstack-ironic | 12:47 | |
| *** ociuhandu has quit IRC | 12:51 | |
| *** rcernin has quit IRC | 12:56 | |
| *** ociuhandu has joined #openstack-ironic | 13:02 | |
| TheJulia | dtantsur: ahh, okay, I was thinking that was it, but needed to be sure | 13:03 |
| dtantsur | good morning TheJulia | 13:04 |
| TheJulia | Always wonderful when a follow-up horribly breaks things | 13:06 |
| *** ericlei has joined #openstack-ironic | 13:10 | |
| dtantsur | heh | 13:10 |
| TheJulia | did you see my comment on stevebaker's boot mode change? | 13:10 |
| dtantsur | I think I did, and I think I removed my -2 pending further reviews | 13:11 |
| TheJulia | okay | 13:11 |
| TheJulia | I was just thinking when talking to him, if it is not there, we can't set it, and it should be none, which would give us more compatability with those that we can't set using the override | 13:11 |
| TheJulia | *sigh* | 13:11 |
| * TheJulia tries to decide what to start with this morning | 13:13 | |
| *** rloo has joined #openstack-ironic | 13:14 | |
| *** ericlei has quit IRC | 13:19 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: Follow-up to RBAC allocation changes https://review.opendev.org/c/openstack/ironic/+/780677 | 13:23 |
| openstackgerrit | Merged openstack/ironic-inspector stable/victoria: Fix port id vs uuid in ValidateInterfacesHook https://review.opendev.org/c/openstack/ironic-inspector/+/780474 | 13:30 |
| TheJulia | do we have any thoughts on going ahead and releasing inspector? | 13:42 |
| dtantsur | there may be more changes incoming still? | 13:48 |
| TheJulia | Doesn't look terribly likely | 13:48 |
| TheJulia | tbh | 13:49 |
| dtantsur | yep, but who knows? | 13:49 |
| TheJulia | true | 13:49 |
| dtantsur | we said that the end of sprint 3 is Apr 5th | 13:49 |
| TheJulia | true | 13:49 |
| dtantsur | we could, of course, prepare everything for the release: review release notes, etc | 13:49 |
| TheJulia | Yeah, we need to start doing that anyway | 13:50 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/778767 wants a review from a core :) | 13:51 |
| *** yoctozepto has quit IRC | 13:53 | |
| TheJulia | dtantsur: commented on https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/780556... or at least | 13:54 |
| TheJulia | I clicked post | 13:54 |
| TheJulia | and wait | 13:54 |
| dtantsur | TheJulia: I don't think it's the explanation. these jobs don't run any tempest tests (or tempest at all). | 13:55 |
| TheJulia | At least, that was my thought because he diretly responded to me that it was to fix the ussuri jobs and tempest on master | 13:56 |
| TheJulia | since we're dead in the water there with trying to patch ussuri ipa | 13:56 |
| dtantsur | I think we had a back-and-forth that ended up in an idea to run these on IPA-builder | 13:56 |
| dtantsur | hence my comment | 13:56 |
| dtantsur | running them anywhere else is useless: pretty much only IPA-builder changes can break these jobs | 13:57 |
| TheJulia | I don't really know. It is so b roken I really juts want to rip the jobs out at this point. | 13:57 |
| TheJulia | but that is my pure frustration at the moment | 13:57 |
| TheJulia | err, frustration speaking | 13:57 |
| TheJulia | eh, CI VM's getting a new kernel can break them | 13:57 |
| TheJulia | which did happen | 13:57 |
| TheJulia | killing chroot() functionality | 13:57 |
| openstackgerrit | Merged openstack/sushy-cli master: Update master for stable/wallaby https://review.opendev.org/c/openstack/sushy-cli/+/780291 | 13:58 |
| rpittau | TheJulia: ussuri ipa will be fixed once https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/780556 and https://review.opendev.org/c/openstack/ironic-python-agent/+/780558 merge | 13:58 |
| rpittau | dtantsur: ^ | 13:59 |
| rpittau | the jobs run on ipa ussuri | 13:59 |
| rpittau | or WILL run | 13:59 |
| dtantsur | they should not run on IPA | 14:00 |
| rpittau | how do we publish ipa in ussuri then ? | 14:00 |
| dtantsur | ah, wait, this one is probably correct | 14:00 |
| dtantsur | (I was thinking about our previous conversation) | 14:00 |
| rpittau | ah ok :) | 14:00 |
| dtantsur | you need depends-on though | 14:01 |
| openstackgerrit | Merged openstack/sushy-cli master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/sushy-cli/+/780292 | 14:01 |
| dtantsur | and probably also run them on IPA-builder | 14:01 |
| openstackgerrit | Merged openstack/sushy-cli stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/sushy-cli/+/780289 | 14:01 |
| openstackgerrit | Merged openstack/sushy-cli stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/sushy-cli/+/780290 | 14:01 |
| TheJulia | ipa builder uploads separately, but the job is broken on chroot trying to patch for ansible, not the build itself | 14:01 |
| dtantsur | since IPA-builder has chances of breaking them | 14:01 |
| rpittau | not sure depends-on will work in this case as the zuul jobs are not taken from cloned repos | 14:01 |
| dtantsur | mmm? | 14:02 |
| dtantsur | IPA-builder is branchless, depends-on it should work | 14:02 |
| TheJulia | yeah, you can't really depends-on job configs unless it is for the same repository | 14:02 |
| dtantsur | yes you can | 14:02 |
| TheJulia | at least, that is how I understand it | 14:02 |
| rpittau | mmmm | 14:02 |
| dtantsur | I did it many times :) | 14:02 |
| dtantsur | I'd like them to run at least non-voting on IPA-builder since IPA-builder changes may break it | 14:02 |
| rpittau | sounds good | 14:03 |
| rpittau | ok let me change the patches then | 14:03 |
| dtantsur | (maybe only the check job on IPA-builder, both jobs on IPA) | 14:03 |
| rpittau | yeah | 14:03 |
| TheJulia | In the mean time, any objections if we just non-voting all the broken tempest jobs, otherwise we're never going to get any changes/fixes in there | 14:03 |
| rpittau | heh ok :/ | 14:03 |
| dtantsur | which exactly jobs are we talking about? | 14:04 |
| dtantsur | on which branch? | 14:04 |
| TheJulia | ussuri jobs | 14:04 |
| TheJulia | ironic-standalone-ussuri to be precise | 14:04 |
| dtantsur | how many do we have broken? disabling multinode is fine, all of them - not so much | 14:04 |
| dtantsur | you mean, on the tempest plugin? | 14:04 |
| TheJulia | yeah, multinodes are alrady non-voting | 14:05 |
| TheJulia | yes | 14:05 |
| TheJulia | already | 14:05 |
| * dtantsur needs to pay attention to a meeting sorry | 14:05 | |
| TheJulia | well, already in this patch | 14:05 |
| rpittau | standalone will be fixed | 14:05 |
| TheJulia | like? when? | 14:05 |
| rpittau | when we merge the 2 patches I wrote before | 14:05 |
| TheJulia | Keep in mind, streses is talking | 14:05 |
| rpittau | we'll have a new ipa ramdisk | 14:05 |
| * TheJulia sighs | 14:05 | |
| rpittau | with the correct kernel | 14:05 |
| rpittau | TheJulia: no worries, I understand the frustration, there was a moment where a wrong tinyipa was published, unfortunately we didn't pin ipa-builder, it will be from now on | 14:06 |
| TheJulia | I guess, regardless, I should just go ahead and prepare for us to keep extra['vif_port_id'] | 14:08 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Add tinyipa ussuri jobs https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/780556 | 14:08 |
| rpittau | addded the non voting job ^ | 14:08 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/ussuri: Re-enable the ussuri tinyipa jobs https://review.opendev.org/c/openstack/ironic-python-agent/+/780558 | 14:11 |
| rpittau | added the depends-on (let's see if it works) ^ | 14:11 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/ussuri: Re-enable the ussuri tinyipa jobs https://review.opendev.org/c/openstack/ironic-python-agent/+/780558 | 14:12 |
| rpittau | rmrrr wrong rebase | 14:12 |
| TheJulia | that last one the re-rebased one? | 14:14 |
| rpittau | the last one is the correct one | 14:14 |
| TheJulia | okay, that is what I thought, but I saw your message afterwards and was a little confused | 14:15 |
| rpittau | :) | 14:15 |
| openstackgerrit | Aija Jauntēva proposed x/sushy-oem-idrac master: Add import system configuration https://review.opendev.org/c/x/sushy-oem-idrac/+/770614 | 14:17 |
| TheJulia | dtantsur: w/r/t https://review.opendev.org/c/openstack/ironic/+/777147, would you mind terribly adding a quick release note indicating the ironic-lib version that is required? | 14:19 |
| dtantsur | TheJulia: I don't think we duplicate requirements.txt (or even should) | 14:20 |
| TheJulia | but that requires operators have knowledge of or evaluate it. Our release notes should indicate something in the case of moving a huge chunk of code out. At least I think | 14:20 |
| dtantsur | I can do it, but I think it's wrong. | 14:21 |
| TheJulia | Just keep it vauge | 14:21 |
| dtantsur | I don't think you can expect anyone not to check dependencies on upgrade | 14:21 |
| TheJulia | heh | 14:21 |
| TheJulia | I'm trying to think how to respond "that was the last thing we ever did" in operations | 14:21 |
| TheJulia | "oh, we upgraded... rutro things are broken... what the dependencies changed! UGH!" | 14:22 |
| TheJulia | (rinse+repeat)*5000 | 14:22 |
| TheJulia | :) | 14:22 |
| dtantsur | :D | 14:22 |
| TheJulia | additionally: release note on https://review.opendev.org/c/openstack/ironic/+/769793 | 14:32 |
| dtantsur | I was sure I had one.. will add in a few minutes | 14:46 |
| openstackgerrit | Mohammed Naser proposed openstack/ironic stable/victoria: Allow users to configure priority for {create,delete}_configuration https://review.opendev.org/c/openstack/ironic/+/780830 | 14:48 |
| openstackgerrit | Mohammed Naser proposed openstack/ironic stable/ussuri: Allow users to configure priority for {create,delete}_configuration https://review.opendev.org/c/openstack/ironic/+/780831 | 14:50 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-ui master: Add a release note for Wallaby https://review.opendev.org/c/openstack/ironic-ui/+/780870 | 14:52 |
| TheJulia | dtantsur: no worries | 14:52 |
| TheJulia | mnaser: feature backports? | 14:53 |
| mnaser | TheJulia: /me shrugs -- to me it feels more of something that is necessary for operation of hw raid | 14:53 |
| mnaser | esp with the fact if you don't clean disks you end up with multiple partitions, and if you clean disks it will take forever (because it wont run delete/create config) | 14:54 |
| TheJulia | ironic-cores, I posted a reno for ironic-ui since it had none | 14:54 |
| TheJulia | mnaser: could they be slightly reframed as a bug then ;) | 14:54 |
| mnaser | TheJulia: most definetly ;) and i think arne_wiebalck mentioned they already do something similar in their hardware manager (assuming that they couldnt do it natively via ironic bc it was not possible) | 14:55 |
| TheJulia | mnaser: from the release team standpoint, we get lots of flack and push back on versioning whenever they see anything featurish, so framing it helps when they review our release notes prior to approving our releases | 14:55 |
| mnaser | as an operator, without that patch, hardware raid is unusable -- i could totally run a backport in our local stuff but that would not be doing our users a service | 14:55 |
| TheJulia | put that in the release note as a fix ;) | 14:56 |
| arne_wiebalck | we are having these steps as part of our downstream h/w manager | 14:56 |
| TheJulia | I'm not the one you have to really justify it to in the end. | 14:56 |
| arne_wiebalck | mnaser: s/hardware/software/, no? | 14:57 |
| mnaser | oh sorry, yes, software raid, correct | 14:57 |
| arne_wiebalck | mnaser: I think it is a usable/unusable as hardware RAID, it follows the very same flow ... but, yeah, we discussed to have run in auto-cleaning at the time and I +2ed the change I think, so I am with you :) | 14:58 |
| mnaser | gotcha, so update the release note? commit msg? leave a review explaining? | 14:58 |
| mnaser | i prefer #3 to make the commits similar | 14:59 |
| TheJulia | mnaser: amend the release note on victoria | 14:59 |
| TheJulia | and we can just reback port that to ussuri | 14:59 |
| mnaser | okey | 14:59 |
| TheJulia | thanks! | 14:59 |
| * TheJulia clicks publish edit in gerrit and waits() | 14:59 | |
| mnaser | TheJulia: select is more suitable for this morning slowness | 15:00 |
| mnaser | :) | 15:00 |
| openstackgerrit | sonali bhausaheb borkar proposed x/sushy-oem-idrac master: Update export system config action constants https://review.opendev.org/c/x/sushy-oem-idrac/+/780703 | 15:00 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Allow ansible deploys to be fast-tracked https://review.opendev.org/c/openstack/ironic/+/778005 | 15:00 |
| TheJulia | mnaser: heh | 15:00 |
| dtantsur | let me float a crazy idea here. what if we have an agent-based driver that fully relies on in-band deploy steps? | 15:00 |
| dtantsur | so, as an operator I have a ramdisk that can do magic with deploy steps. I don't want ironic to do anything other than starting IPA and executing deploy steps as appropriate. | 15:01 |
| dtantsur | essentially, direct deploy without built-in deploy steps | 15:01 |
| TheJulia | +2+A ? | 15:01 |
| dtantsur | how does it sound? | 15:01 |
| TheJulia | I mean, there are gremlins and possibly some corgis there | 15:01 |
| TheJulia | but it seems very reasonable and similar to Anaconda deploy, just a slightly different mechanism. | 15:02 |
| dtantsur | gremlins and corgis are up to a deployer to handle | 15:02 |
| dtantsur | e.g. in openshift cases there is a magical command to run. pretty much one deploy step that runs a command. | 15:02 |
| * dtantsur is trying to prevent people from NOT using IPA | 15:02 | |
| TheJulia | understood | 15:03 |
| TheJulia | and I think it makes sense | 15:03 |
| TheJulia | do a thing, thanks | 15:03 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: API to force manual cleaning without booting IPA https://review.opendev.org/c/openstack/ironic/+/769793 | 15:09 |
| dtantsur | TheJulia: actually with a release note this time ^^^ | 15:09 |
| TheJulia | dtantsur: thanks | 15:12 |
| TheJulia | sigh, looks like grub changed their network boot pattern 14 months ago | 15:12 |
| TheJulia | https://github.com/rhboot/grub2/commit/cb2f15c544895e1f3d540dd39d36c4611bdf5b7b | 15:13 |
| dtantsur | le sigh | 15:14 |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Use git hash instead of release to build ipmitool https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/780554 | 15:15 |
| dtantsur | TheJulia: https://storyboard.openstack.org/#!/story/2008719 a short RFE. I think in terms of code it's really not much code. | 15:19 |
| *** bburns has quit IRC | 15:20 | |
| *** mkrai has quit IRC | 15:22 | |
| *** mkrai_ has joined #openstack-ironic | 15:22 | |
| TheJulia | I added the grub change to the priority list | 15:23 |
| *** rh-jelabarre has quit IRC | 15:24 | |
| TheJulia | it is new as of yesterday, but we'll need to get that merged at some point since it looks like this will break with grub 2.06 | 15:24 |
| TheJulia | which released rc-1 | 15:24 |
| TheJulia | err, rc1 | 15:24 |
| dtantsur | oh | 15:24 |
| TheJulia | the patch author needs to add compatibility :( | 15:24 |
| *** bburns has joined #openstack-ironic | 15:28 | |
| openstackgerrit | sonali bhausaheb borkar proposed x/sushy-oem-idrac master: Update export system config class and constants https://review.opendev.org/c/x/sushy-oem-idrac/+/780703 | 15:32 |
| *** ha1f3mpty has joined #openstack-ironic | 15:35 | |
| ha1f3mpty | hello, can anyone help me with introspection of nodes with PERC H700 raid controller? When I do introspection it shows nothing for disk. I have made sure that the raid has a virtual disk setup and if I do an install manually of an OS it sees the disk. | 15:35 |
| *** hjensas has quit IRC | 15:35 | |
| TheJulia | ha1f3mpty: you need a ramdisk that has the appropriate drivers. is the driver included in the kernel or does it need to be installed like an RPM? | 15:40 |
| *** gyee has joined #openstack-ironic | 15:43 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: dhcp.sh predictible ifnames https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/779476 | 15:46 |
| *** uzumaki has joined #openstack-ironic | 15:51 | |
| *** anuradha1904 has quit IRC | 15:52 | |
| *** rh-jelabarre has joined #openstack-ironic | 15:52 | |
| *** rh-jelabarre has quit IRC | 15:53 | |
| *** rh-jelabarre has joined #openstack-ironic | 15:53 | |
| *** anuradha1904 has joined #openstack-ironic | 16:04 | |
| *** adrianc has quit IRC | 16:07 | |
| *** adrianc has joined #openstack-ironic | 16:07 | |
| rpittau | dtantsur: I stand corrected, the depends-on worked, which is good as the job is successful :) | 16:15 |
| *** Qianbiao has quit IRC | 16:18 | |
| rpittau | the non-voting one in ipa-builder although is failing, we would need to pin ipa to ussuri for that to work | 16:20 |
| rpittau | I wonder if we could use override-checkout | 16:25 |
| *** ociuhandu has quit IRC | 16:27 | |
| *** ociuhandu has joined #openstack-ironic | 16:27 | |
| dtantsur | I think we can | 16:34 |
| rpittau | yeah, I'll add that to ipa-builder in another patch, we really need those 2 to merge | 16:35 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Use ussuri branch for tinyipa check job for ussuri https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/780927 | 16:39 |
| TheJulia | Are we up for a review jam today? | 16:39 |
| * dtantsur is not, sorry | 16:40 | |
| TheJulia | No worries | 16:40 |
| rpittau | I also can't :/ | 16:45 |
| TheJulia | Okay, well, it is also an hour later for me since yay daylight savings time | 16:46 |
| TheJulia | \o/ no more conflict | 16:46 |
| TheJulia | . o O (until later in the year...) | 16:46 |
| *** rloo has quit IRC | 16:53 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Follow-up to RBAC allocation changes https://review.opendev.org/c/openstack/ironic/+/780677 | 16:53 |
| TheJulia | \o/ passes tempest | 16:54 |
| dtantsur | w00t! | 16:54 |
| *** rloo has joined #openstack-ironic | 16:54 | |
| *** rloo has quit IRC | 16:55 | |
| *** rloo has joined #openstack-ironic | 16:55 | |
| *** rloo has quit IRC | 16:56 | |
| *** rloo has joined #openstack-ironic | 16:56 | |
| TheJulia | now if we can just move forward with the old vif_port_id stuff | 16:58 |
| *** ociuhandu_ has joined #openstack-ironic | 16:59 | |
| *** lucasagomes has quit IRC | 17:02 | |
| *** ociuhandu has quit IRC | 17:02 | |
| *** ociuhandu_ has quit IRC | 17:03 | |
| *** dking has joined #openstack-ironic | 17:04 | |
| dking | Is there a specific channel for sushy? | 17:06 |
| dtantsur | no, only this one | 17:07 |
| *** dking has quit IRC | 17:13 | |
| *** dking has joined #openstack-ironic | 17:14 | |
| dking | So, I'm having an issue PXE booting my Supermicro servers with redfish. I get "Could not figure out the allowed values for the reset system action for System 1" I'm supposing that there's something missing in the redfish output. | 17:15 |
| dking | Here's what I get from redfish/v1/Systems/1 "Actions" section: | 17:16 |
| dtantsur | something seems fishy with your redfish | 17:16 |
| dking | { "Oem": {}, "#ComputerSystem.Reset": { "target": "/redfish/v1/Systems/1/Actions/ComputerSystem.Reset", "@Redfish.ActionInfo": "/redfish/v1/Systems/1/ResetActionInfo" } } | 17:16 |
| dtantsur | (sorry, couldn't resist) | 17:16 |
| dking | lol. That joke stinks. :) | 17:16 |
| dtantsur | :D | 17:16 |
| *** rloo has quit IRC | 17:16 | |
| dtantsur | mm, okay, this is strange (thank you supermicro). They define a call but don't define any supported reset values? | 17:17 |
| *** rloo has joined #openstack-ironic | 17:17 | |
| dking | But yeah, that's my thought. | 17:17 |
| dking | They do, but under the "Boot" section. | 17:17 |
| dtantsur | WUT | 17:17 |
| dking | From the same URL, just further up: "Boot": { "BootSourceOverrideEnabled": "Disabled", "BootSourceOverrideMode": "Legacy", "BootSourceOverrideTarget": "None", "BootSourceOverrideTarget@Redfish.AllowableValues": [ "None", "Pxe", "Floppy", "Cd", "Usb", "Hdd", "BiosSetup" ] } | 17:18 |
| mraineri | The "ActionInfo" location shows more detailed info about supported parameters and values | 17:18 |
| mraineri | So, if you go to /redfish/v1/Systems/1/ResetActionInfo, you'll see what types of resets are supported | 17:18 |
| dtantsur | omg, yet another place to support? | 17:18 |
| mraineri | We primarily did ActionInfo for more complex actions with lots of parameters | 17:19 |
| mraineri | And we recommend people support "ResetType@Redfish.AllowableValues" for simple things like this | 17:19 |
| dking | Oh, okay. yes, you can get that from redfish/v1/Systems/1/ResetActionInfo | 17:19 |
| dtantsur | yeah, ironic has been happily relying on this | 17:19 |
| dtantsur | dking: could you paste what you see there? | 17:19 |
| mraineri | But it looks like SuperMicro just dropped one thing in favor of the other without considering backwards compatability | 17:19 |
| dtantsur | le sigh | 17:20 |
| rpittau | backwards compatibility is overrated | 17:20 |
| dtantsur | ++ | 17:20 |
| dking | ActionInfo: http://paste.openstack.org/show/803625/ | 17:20 |
| mraineri | If you take a look at this sample code, there's some example for how to manage both cases simultaneously | 17:20 |
| mraineri | https://github.com/DMTF/Redfish-Tacklebox/blob/master/redfish_utilities/systems.py | 17:21 |
| mraineri | def get_system_reset_info | 17:21 |
| rpittau | mmm interesting | 17:21 |
| dking | lol. Yeah, Supermicro isn't really the most reliable when it comes to Redfish support. But if it's something that we really need, I think we can push them a bit to implement it. Who knows how long it would take, though. | 17:21 |
| dtantsur | dking: okay, so this is missing from sushy. could you file a bug with all our findings? | 17:21 |
| dtantsur | code is even more welcome ;) | 17:21 |
| dking | Okay, so sushy needs the get_system_reset_info routine? | 17:22 |
| mraineri | Or something that follows similar logic | 17:23 |
| dtantsur | dking: all the cases where we handle actions need something like that | 17:23 |
| mraineri | That code I referenced is from a different project | 17:23 |
| dking | Yeah, I see that. Okay. It's my first time looking at sushy code, so I may not be able to write any code up, but I can put in a bug. Thanks for the help. | 17:25 |
| dtantsur | yep, a bug is very appreciated | 17:25 |
| dking | Also, I'm kinda glad to know that it's not totally on Supermicro as being able to patch things here could be much wquicker. | 17:25 |
| dtantsur | yeah, software is much easier to fix :) | 17:26 |
| rpittau | dking: you probably need to check in the system module and go from there, for example: https://opendev.org/openstack/sushy/src/branch/master/sushy/resources/system/system.py#L44 | 17:27 |
| rpittau | bye everyone! o/ | 17:29 |
| *** rpittau is now known as rpittau|afk | 17:29 | |
| dking | rpittau: Yes, that's where I was looking. I was actually starting with the error on line 169 and working my way back. So, I'm thinking that it has something further back than _get_reset_action_element, or perhaps ActionsField, but there's a lot of class inheritance in there that's slowing me down moving backwards. | 17:30 |
| *** ociuhandu has joined #openstack-ironic | 17:30 | |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Add tinyipa ussuri jobs https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/780556 | 17:34 |
| *** ociuhandu has quit IRC | 17:34 | |
| *** dtantsur is now known as dtantsur|afk | 17:47 | |
| dtantsur|afk | o/ | 17:47 |
| *** derekh has quit IRC | 18:03 | |
| *** lbragstad has quit IRC | 18:07 | |
| *** k_mouza has quit IRC | 18:09 | |
| arne_wiebalck | bye everyone o/ | 18:18 |
| *** lbragstad has joined #openstack-ironic | 18:27 | |
| *** rloo has quit IRC | 18:30 | |
| dking | o/ | 18:30 |
| *** rloo has joined #openstack-ironic | 18:33 | |
| *** rloo has quit IRC | 18:34 | |
| *** rloo has joined #openstack-ironic | 18:34 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Allow ansible deploys to be fast-tracked https://review.opendev.org/c/openstack/ironic/+/778005 | 18:38 |
| * TheJulia glares at CI | 18:49 | |
| openstackgerrit | Verification of a change to openstack/ironic-python-agent-builder failed: dhcp.sh predictible ifnames https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/779476 | 19:11 |
| *** persia has quit IRC | 19:46 | |
| *** persia has joined #openstack-ironic | 19:47 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: API to force manual cleaning without booting IPA https://review.opendev.org/c/openstack/ironic/+/769793 | 19:50 |
| *** persia has quit IRC | 19:52 | |
| *** persia has joined #openstack-ironic | 19:54 | |
| *** mkrai_ has quit IRC | 19:59 | |
| *** trandles has joined #openstack-ironic | 20:00 | |
| *** persia has quit IRC | 20:01 | |
| *** persia has joined #openstack-ironic | 20:02 | |
| *** tosky has quit IRC | 20:05 | |
| *** tosky has joined #openstack-ironic | 20:05 | |
| *** paras333_ has joined #openstack-ironic | 20:11 | |
| *** yoctozepto has joined #openstack-ironic | 20:14 | |
| *** paras333_ has quit IRC | 20:15 | |
| *** juanoterocas has joined #openstack-ironic | 20:18 | |
| *** hjensas has joined #openstack-ironic | 20:32 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Revise release notes for 17.0 release https://review.opendev.org/c/openstack/ironic/+/780973 | 20:41 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add prelude for Ironic 17.0. https://review.opendev.org/c/openstack/ironic/+/780974 | 20:41 |
| *** anuradha1904 has quit IRC | 20:42 | |
| *** paras333 has joined #openstack-ironic | 21:05 | |
| *** paras333 has quit IRC | 21:09 | |
| *** juanoterocas has quit IRC | 21:17 | |
| *** juanoterocas has joined #openstack-ironic | 21:19 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: API to force manual cleaning without booting IPA https://review.opendev.org/c/openstack/ironic/+/769793 | 21:32 |
| *** rh-jelabarre has quit IRC | 21:57 | |
| *** stevebaker has quit IRC | 22:07 | |
| *** ha1f3mpty has quit IRC | 22:13 | |
| *** rcernin has joined #openstack-ironic | 22:24 | |
| *** lbragstad_ has joined #openstack-ironic | 22:29 | |
| *** hjensas has quit IRC | 22:35 | |
| *** lbragstad has quit IRC | 22:35 | |
| *** bburns has quit IRC | 22:35 | |
| *** sshnaidm has quit IRC | 22:35 | |
| *** gyee has quit IRC | 22:40 | |
| *** sshnaidm has joined #openstack-ironic | 22:41 | |
| *** hjensas has joined #openstack-ironic | 22:42 | |
| *** juanoterocas has quit IRC | 22:56 | |
| *** paras333 has joined #openstack-ironic | 23:05 | |
| *** paras333 has quit IRC | 23:10 | |
| *** stevebaker has joined #openstack-ironic | 23:14 | |
| *** rcernin has quit IRC | 23:38 | |
| *** rcernin has joined #openstack-ironic | 23:39 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!