| *** pmannidi|AFK is now known as pmannidi | 00:00 | |
| *** pmannidi is now known as pmannidi|AFK | 04:42 | |
| iurygregory | good morning Ironic o/ | 06:55 |
|---|---|---|
| janders | IDENTIFY test | 06:55 |
| janders | whoops :) | 06:55 |
| janders | wrong window | 06:55 |
| arne_wiebalck | Good morning janders iurygregory and Ironic! | 07:03 |
| janders | hey arne_wiebalck iurygregory and Ironic o/ | 07:03 |
| iurygregory | hehehe | 07:03 |
| iurygregory | morning janders arne_wiebalck o/ | 07:03 |
| iurygregory | welcome back arne_wiebalck :D | 07:03 |
| arne_wiebalck | iurygregory: thanks! Back on IRC and back at CERN since 6+ months :) | 07:04 |
| iurygregory | \o/ | 07:04 |
| iurygregory | nice | 07:04 |
| arne_wiebalck | 8+ months according to my stale thunderbird :) | 07:04 |
| iurygregory | hehehe I'm jealous :D (I can go back to the office but I'm still a bit afraid hehe) | 07:08 |
| arne_wiebalck | yeah, there are still rules in place, it is not the same yet | 07:11 |
| *** rpittau|afk is now known as rpittau | 07:52 | |
| rpittau | good morning ironic! o/ | 07:52 |
| Qianbiao | Morning rpittau o/ | 07:55 |
| rpittau | hey Qianbiao :) | 07:55 |
| Qianbiao | o/ happy new week | 07:55 |
| iurygregory | morning Qianbiao rpittau o/ | 07:58 |
| rpittau | hey iurygregory :) | 07:59 |
| Qianbiao | morning iurygregory | 07:59 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/wallaby: Improve main function https://review.opendev.org/c/openstack/bifrost/+/803833 | 08:06 |
| janders | good morning rpittau and Qianbiao | 08:26 |
| rpittau | hey janders :) | 08:26 |
| Qianbiao | hi janders, good morning. | 08:27 |
| opendevreview | Merged openstack/bifrost master: Improve main function https://review.opendev.org/c/openstack/bifrost/+/803799 | 08:55 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Enable priority overrides to enable/disable steps https://review.opendev.org/c/openstack/ironic/+/803887 | 09:02 |
| iurygregory | janders, I think you are adding things for verify steps in ^ O.o | 09:04 |
| iurygregory | I would be explicit in the commit that this is Fix to override the steps (since it wasn't working before) =) | 09:05 |
| janders | iurygregory: this came up in v-s work, however the bug is in the generic _get_steps() method used by clean, verify and deploy steps | 09:07 |
| janders | only clean steps use it at the moment, verify steps will use this, too | 09:07 |
| janders | does this make sense, or should I still add an explicit comment related to verify steps? | 09:07 |
| iurygregory | I would separate since we probably need to backport the fix (and we don't have v-s in stable branches =) ) | 09:08 |
| iurygregory | I would keep the fix for v-s in the patch that will introduce v-s =) | 09:08 |
| janders | oops I see what happened there | 09:08 |
| janders | some code got pulled into this that doesn't need to be there | 09:09 |
| janders | thanks for pointing this out | 09:09 |
| janders | too many local outstanding changes | 09:09 |
| iurygregory | hehehe | 09:09 |
| iurygregory | yeah | 09:09 |
| janders | only _get_steps patch was meant to go into this change | 09:09 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Enable priority overrides to enable/disable steps https://review.opendev.org/c/openstack/ironic/+/803887 | 09:11 |
| janders | and tests are ever more mixed up... | 09:14 |
| janders | :/ | 09:14 |
| janders | fixing it now | 09:14 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Enable priority overrides to enable/disable steps https://review.opendev.org/c/openstack/ironic/+/803887 | 09:20 |
| janders | iurygregory ^^ should fix things up hopefully, sorry about the mess earlier, must have mixed up directories :/ | 09:21 |
| iurygregory | ack, no worries | 09:21 |
| opendevreview | Verification of a change to openstack/sushy master failed: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/sushy/+/803601 | 09:23 |
| opendevreview | Verification of a change to openstack/ironic-lib master failed: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/ironic-lib/+/803628 | 09:37 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter master: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/803893 | 09:41 |
| cenne | Good morning Ironic! | 09:44 |
| cenne | Mornings iurygregory rpittau arne_wiebalck janders | 09:45 |
| rpittau | hey cenne :) | 09:45 |
| iurygregory | hey cenne o/ | 09:45 |
| cenne | Good morning Qianbiao | 09:45 |
| cenne | Hey o/ : ) | 09:45 |
| Qianbiao | morning cenne | 09:46 |
| arne_wiebalck | hey cenne o/ | 09:48 |
| cenne | hey : ) | 09:52 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Clean step to remove CA certificates from iLO https://review.opendev.org/c/openstack/ironic/+/799562 | 10:02 |
| opendevreview | Merged openstack/networking-generic-switch master: Increase version of hacking and pycodestyle https://review.opendev.org/c/openstack/networking-generic-switch/+/803213 | 10:44 |
| opendevreview | Verification of a change to openstack/ironic bugfix/18.1 failed: Fix subscription vendor passthru https://review.opendev.org/c/openstack/ironic/+/803678 | 10:57 |
| iurygregory | yay for Kernel Panic in CI jobs \o/ | 11:01 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/414078af24b6435fb2062f491f5eae14/log/controller/logs/ironic-bm-logs/node-0_console_2021-08-09-10:05:55_log.txt | 11:01 |
| iurygregory | ofc today is Monday <3 | 11:02 |
| opendevreview | Merged openstack/sushy master: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/sushy/+/803601 | 11:22 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Enable priority overrides to enable/disable steps https://review.opendev.org/c/openstack/ironic/+/803887 | 11:23 |
| opendevreview | Verification of a change to openstack/ironic-inspector master failed: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/ironic-inspector/+/803570 | 11:26 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/ironic-python-agent/+/803453 | 12:36 |
| rpittau | mmm I think we have a problem | 12:48 |
| rpittau | iurygregory: the kernel panic seems hitting everywhere, let's see after one or two rechecks | 12:50 |
| iurygregory | rpittau, yeah | 12:50 |
| iurygregory | if this happens again when Dmitry is on PTO we will have a pattern :D | 12:52 |
| rpittau | lol | 12:52 |
| TheJulia | good morning | 13:17 |
| rpittau | good morning TheJulia :) | 13:17 |
| iurygregory | good morning TheJulia =) | 13:19 |
| TheJulia | lovely | 13:19 |
| TheJulia | try adding root=/dev/ram0 to the kernel command line arguments | 13:19 |
| TheJulia | does it use centos stream? | 13:25 |
| *** pmannidi|AFK is now known as pmannidi | 13:27 | |
| rpittau | TheJulia: it should not be stream | 13:30 |
| TheJulia | seesms super weird | 13:49 |
| TheJulia | Who is running the meeting today? | 13:57 |
| *** pmannidi is now known as pmannidi|AFK | 14:05 | |
| TheJulia | Seems like reminders might be good | 14:08 |
| TheJulia | stevebaker: figured out why my node was failing... :\ | 14:08 |
| TheJulia | err, tripleo change | 14:09 |
| TheJulia | At least, failing periodically and randomly | 14:09 |
| iurygregory | TheJulia, I' | 14:17 |
| iurygregory | I'm* | 14:17 |
| iurygregory | sorry I was in a downstream meeting hehe | 14:18 |
| *** pmannidi|AFK is now known as pmannidi | 14:18 | |
| cenne | Good morning TheJulia | 14:20 |
| TheJulia | soooo many places in inspector don't have retry logic or basic wrappers to handle it | 14:25 |
| opendevreview | Riccardo Pittau proposed openstack/networking-baremetal master: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/networking-baremetal/+/803933 | 14:32 |
| iurygregory | sounds like some low-hanging fruit bugs we can add to a story so newcomers can work on it XD | 14:41 |
| cenne | Yay! Sounds like a good plan : ) | 14:50 |
| *** pmannidi is now known as pmannidi|AFK | 14:52 | |
| TheJulia | unfortunately, it is breaking CI | 14:53 |
| TheJulia | and likely has silently been doing so for a very long time | 14:53 |
| iurygregory | oh | 14:53 |
| iurygregory | well, that's a different case :D | 14:53 |
| iurygregory | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Aug 9 15:00:00 2021 UTC and is due to finish in 60 minutes. The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| iurygregory | o/ | 15:00 |
| stendulker | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| iurygregory | Hello ironicers! welcome to our weekly meeting! | 15:00 |
| rpioso | \o | 15:00 |
| ajya | o/ | 15:00 |
| * iurygregory wondering if we have quorum... | 15:01 | |
| * iurygregory will give a few more minutes to see | 15:01 | |
| erbarr | o/ | 15:02 |
| JayF | o/ | 15:02 |
| bfournie | o/ | 15:02 |
| iurygregory | yay! we have quorum \o/ | 15:02 |
| iurygregory | Our agenda can be found in the wiki | 15:02 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:03 |
| iurygregory | #topic Announcements / Reminder | 15:03 |
| iurygregory | #info Xena Midcycle - Please add ideas for the midcycle: https://etherpad.opendev.org/p/ironic-xena-midcycle till this Friday. | 15:03 |
| iurygregory | #info Yoga PTG (October 18-22, 2021) - Please add your ideas for the PTG: https://etherpad.opendev.org/p/ironic-yoga-ptg | 15:03 |
| iurygregory | #info All Red Hatters will be out this friday. | 15:04 |
| TheJulia | o/ | 15:04 |
| iurygregory | Does anyone have anything else to add? | 15:04 |
| TheJulia | iurygregory: when is the midcycle again? | 15:04 |
| iurygregory | TheJulia, 23 24 25 (depending on how many topics we have) | 15:05 |
| iurygregory | on 16th we will have the topics for each day- | 15:05 |
| TheJulia | ack | 15:06 |
| iurygregory | #info Midcycle August 23 (3-4PM UTC ), 24 (2-3PM UTC) and 25 (2-3PM) | 15:06 |
| cenne | o/ | 15:06 |
| iurygregory | moving to our next topic | 15:07 |
| iurygregory | #topic Review action items from previous meeting | 15:07 |
| iurygregory | No previous action items - skipping | 15:07 |
| iurygregory | #topic Review subteam status reports | 15:07 |
| iurygregory | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:07 |
| iurygregory | starting on L60 | 15:08 |
| iurygregory | TheJulia, tks for working on the node error history! | 15:09 |
| TheJulia | No problem, it is actually *way* easier since we limited scope heavily upfront | 15:12 |
| TheJulia | there is an issue with the upgrade, but I'd like to get them on the review list this week | 15:12 |
| iurygregory | ++ | 15:13 |
| iurygregory | I think we are good with the updates, moving on | 15:13 |
| TheJulia | I've also got this inspector patch I'll put on the priority review list since it is a sporatic tripleo CI failure | 15:13 |
| TheJulia | it shoudl be posted in about 3 minutes | 15:14 |
| iurygregory | ack | 15:14 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:14 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:14 |
| iurygregory | rpittau, I'm wondering if you want to add some small things you have been working on like https://review.opendev.org/q/topic:%2522increase-hacking-pycodestyle%2522+status:open | 15:15 |
| rpittau | yeah, I'll add them | 15:16 |
| rpittau | I'm abit concerned about CI at the moment | 15:16 |
| iurygregory | yeah | 15:16 |
| iurygregory | CI on monday <3 | 15:16 |
| iurygregory | if we try to approve things we are just wasting resources till we find the root cause | 15:16 |
| rpittau | yep | 15:17 |
| iurygregory | https://review.opendev.org/c/openstack/ironic-python-agent/+/803710 sounds like something we can add to the list | 15:17 |
| iurygregory | seems like* | 15:17 |
| TheJulia | is it just the virtual media jobs or is it more? | 15:18 |
| iurygregory | I saw other jobs also | 15:19 |
| rpittau | looks like ipmi is also failing somewhere, not sure if it's the same issue though | 15:19 |
| iurygregory | ^ yeah | 15:20 |
| iurygregory | maybe we can add https://review.opendev.org/c/openstack/python-ironicclient/+/803332 wdyt? | 15:20 |
| opendevreview | Julia Kreger proposed openstack/ironic-inspector master: Handle NodeLocked failures https://review.opendev.org/c/openstack/ironic-inspector/+/803935 | 15:20 |
| TheJulia | okay, I can look at CI failures after I get more coffee | 15:21 |
| TheJulia | iurygregory: sounds good to me | 15:21 |
| rpittau | I had a quick look and the ipmi seems like a different problem | 15:22 |
| iurygregory | different problem - not Kernel Panic? | 15:23 |
| rpittau | yep | 15:23 |
| TheJulia | hopefully it is nothing major but I can dig into it after the meeting | 15:24 |
| iurygregory | I saw KP in ironic-tempest-ipa-partition-pxe_ipmitool | 15:24 |
| iurygregory | but yeah we need to look at this to see =) | 15:24 |
| iurygregory | I think we are good with priorities | 15:24 |
| iurygregory | if you have patches feel free to add the hashtag later | 15:24 |
| iurygregory | #topic Discussion | 15:25 |
| iurygregory | We don't have any topics for discussion, but I will give a chance for people start one if they want =) | 15:26 |
| TheJulia | so we're really in Open Discussion? | 15:26 |
| iurygregory | not yet | 15:26 |
| iurygregory | we have Baremetal SIG / RFE review | 15:27 |
| TheJulia | One thing worth bringing up, Friday's SPUC will be JayF's final spuc most likely | 15:27 |
| TheJulia | He would <3 for people to join in | 15:27 |
| JayF | ;( | 15:27 |
| iurygregory | #info Friday's SPUC will be JayF's final spuc most likely, he would <3 for people to join in | 15:27 |
| cenne | :(- | 15:27 |
| TheJulia | So since we can't bake him a cake on video, lets at least wish him a good send off :) | 15:28 |
| iurygregory | while we are drinking :D | 15:28 |
| iurygregory | \o/ | 15:28 |
| iurygregory | moving to our next topic | 15:29 |
| iurygregory | #topic Baremetal SIG | 15:29 |
| TheJulia | I don't think there is anything SIG related, lots of folks on vacation | 15:30 |
| iurygregory | yeah | 15:30 |
| iurygregory | Arne is back :D (but he can't join today) | 15:31 |
| iurygregory | #topic RFE review | 15:31 |
| iurygregory | no items for RFE review on the agenda, does anyone have something? | 15:31 |
| TheJulia | Nothing right now | 15:32 |
| TheJulia | Speaking of which, we likely should consider reviewing sdanni and lmcgann's attestation patches | 15:32 |
| iurygregory | I saw a patch to IPA I think | 15:32 |
| iurygregory | https://review.opendev.org/c/openstack/ironic-python-agent/+/803510 | 15:33 |
| lmcgann | we actually have a demo video to share, but i was waiting until the open discussion part of the meeting | 15:33 |
| TheJulia | sdanni: lmcgann: can you guys add an entry to the whiteboard and let us know the required merge/review order? | 15:33 |
| iurygregory | #topic Open discussion | 15:34 |
| iurygregory | lmcgann, go ahead =) | 15:34 |
| sdanni | could you link us the whiteboard? | 15:34 |
| TheJulia | https://etherpad.opendev.org/p/IronicWhiteBoard | 15:34 |
| lmcgann | Perfect, so I put together a video for Keylime Ironic integration. I go over the workflow, current limitations, and a demo. https://www.youtube.com/watch?v=o7eL95BUGLA | 15:35 |
| iurygregory | #info Keylime Ironic integration demo https://www.youtube.com/watch?v=o7eL95BUGLA | 15:35 |
| sdanni | Should we add our patches in Code Review Dashboards? | 15:36 |
| iurygregory | sdanni, you can add the hashtag ironic-week-prio in the patches | 15:36 |
| iurygregory | and probably add a section to Subteam status reports | 15:37 |
| sdanni | okay! | 15:37 |
| iurygregory | for the keylime and list the order of the patches and current status | 15:37 |
| cenne | What happens in midcycle meeting? | 15:37 |
| iurygregory | cenne, we will discuss about some topics | 15:38 |
| TheJulia | cenne: we discuss current or emergent topics in a higher bandwidth medium, say video/audio calls | 15:38 |
| TheJulia | cenne: and then try to find a forward direction, often it is intended to help us wrap up work in progress | 15:39 |
| cenne | oh.. | 15:39 |
| iurygregory | TheJulia types very fast :D | 15:39 |
| iurygregory | well, I think that's all for today, should we move to the last topic? | 15:41 |
| cenne | thanks. | 15:41 |
| iurygregory | cenne, feel free to participate in the midcycle and the PTG in Oct =) they are both free :D | 15:41 |
| cenne | alright! I'll try to attend the midcycle. :) | 15:42 |
| TheJulia | I also type with lots of energy | 15:42 |
| * TheJulia has been told that my use of the enter key should break keyboards | 15:42 | |
| iurygregory | TheJulia, you should buy the big Enter :D | 15:43 |
| rpittau | big enter red button | 15:44 |
| cenne | haha. my friends would often be worried when I used their laptop's keyboards too. :p | 15:44 |
| TheJulia | lol | 15:44 |
| iurygregory | hehehe :D | 15:44 |
| iurygregory | #topic Who is going to run the next meeting? | 15:45 |
| iurygregory | any volunteers? | 15:45 |
| TheJulia | Well | 15:45 |
| TheJulia | you said we're having Midcycle session on the 16th? | 15:45 |
| rpittau | no, week after :) | 15:45 |
| iurygregory | we will have the topics for each day | 15:46 |
| iurygregory | on 16th | 15:46 |
| TheJulia | oh, okay | 15:46 |
| TheJulia | ahh, okay | 15:46 |
| TheJulia | So I could take next week off or something crazy | 15:46 |
| iurygregory | I can run the meeting without problems on 16th =) | 15:46 |
| rpittau | TheJulia: you should, it doesn't sound crazy at all | 15:46 |
| rpittau | I will be out the 16th too | 15:46 |
| * iurygregory is having the feeling everyone will be out :D | 15:47 | |
| rpittau | lots | 15:47 |
| TheJulia | iurygregory: just means the meeting will be quick | 15:47 |
| iurygregory | yeah | 15:47 |
| iurygregory | I will run the meeting on 16th =) | 15:47 |
| iurygregory | Thanks everyone! | 15:47 |
| iurygregory | #endmeeting | 15:48 |
| opendevmeet | Meeting ended Mon Aug 9 15:48:02 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:48 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-09-15.00.html | 15:48 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-09-15.00.txt | 15:48 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-09-15.00.log.html | 15:48 |
| rpittau | thanks iurygregory :) | 15:48 |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/networking-generic-switch/+/803939 | 15:48 |
| TheJulia | anyone have some good exempliarly failures? | 15:48 |
| iurygregory | https://review.opendev.org/c/openstack/ironic/+/803678 has some jobs with the failures | 15:49 |
| TheJulia | so it is centos stream 8.4 | 16:02 |
| TheJulia | or well, to be more precise, centos 8.4 with same version packages as stream | 16:03 |
| TheJulia | Yeahhhhh | 16:07 |
| TheJulia | it is stream alright | 16:07 |
| TheJulia | found it | 16:09 |
| TheJulia | "ipa-centos8-master.tar.gz2021-08-08 17:36710M " | 16:09 |
| rpittau | mmm ok weird, we're using release 8 to build the image, not 8-stream | 16:09 |
| opendevreview | Danni Shi proposed openstack/ironic-python-agent master: Add an attestation extension https://review.opendev.org/c/openstack/ironic-python-agent/+/803510 | 16:09 |
| TheJulia | well, it might not matter since they are shoving stuff in it seems | 16:09 |
| TheJulia | Anyway, I've started a download and I start taking it apart | 16:09 |
| TheJulia | it is like 2.25 times our prior image size, which is why things are failing like they are | 16:10 |
| rpittau | heh ok, ninja-updates are lethal | 16:10 |
| * TheJulia gets out the list of mythbusters one-liners https://www.youtube.com/watch?v=LxhF7iBueeE | 16:12 | |
| rpittau | TheJulia: if you look here https://tarballs.opendev.org/openstack/ironic-python-agent/dib/files/ipa-centos8-master.d/dib-manifests/dib_environment the DIB_RELEASE is correctly set to 8 | 16:13 |
| rpittau | I wonder if they switched to centos-stream repos by default | 16:13 |
| TheJulia | maybe | 16:13 |
| TheJulia | download is going slow... I miss my mesh wifi | 16:13 |
| TheJulia | rpittau: it could also be that something we clean up got moved around | 16:14 |
| TheJulia | The size also makes me think none of the cleanup works | 16:15 |
| rpittau | yep, it's way too big anyway | 16:15 |
| * TheJulia goes and checks the mesh wifi basestation replacement order status | 16:16 | |
| TheJulia | still not shipped :( | 16:16 |
| rpittau | the last commit to ipa-builder seems suspicious :) | 16:23 |
| rpittau | https://opendev.org/openstack/ironic-python-agent-builder/commit/68989576e3663cdc5a7182eea14d2ed6206d61e0 | 16:23 |
| rpittau | need to go, have a nice rest of the day everyone! o/ | 16:26 |
| *** rpittau is now known as rpittau|afk | 16:26 | |
| TheJulia | 3gb uncompressed | 16:29 |
| TheJulia | yup | 16:32 |
| TheJulia | yeah, its not pruning a thing | 16:39 |
| NobodyCam | Good Mornings Ironic folks.. Happy Monday | 16:39 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Add lower-constraints job to current development branch https://review.opendev.org/c/openstack/ironic-python-agent/+/803453 | 16:40 |
| TheJulia | good morning NobodyCam | 16:42 |
| TheJulia | Well, this is freaky | 16:42 |
| TheJulia | if I let it manually generate the command.... it doesn't work | 16:43 |
| TheJulia | err | 16:43 |
| NobodyCam | o/ TheJulia Morning | 16:43 |
| TheJulia | if I print the command, it works as expected | 16:43 |
| TheJulia | if I let the script do it, kaboom | 16:43 |
| NobodyCam | "'" + `echo "' Could it be a \"Quoting\" Issue'` + "'" | 16:44 |
| NobodyCam | awe ate the ` chr | 16:45 |
| TheJulia | I'm not sure :\ | 16:53 |
| TheJulia | hmmmmmmmm | 17:04 |
| NobodyCam | https://learningenglish.voanews.com/a/nasa-spacecraft-detects-a-hum-in-interstellar-space/5889608.html | 17:18 |
| opendevreview | cenne proposed openstack/python-ironicclient master: [WIP] Add support for changing 'boot_mode' and 'secure_boot' states https://review.opendev.org/c/openstack/python-ironicclient/+/803970 | 17:36 |
| lmcgann | Is there a reason ironic doesnt have the ability to get and object in swift, the way the ironic-inspector can? The reason I ask is because I may have to retrieve a file from swift for use in the interface Im writing and Im wondering what the recommended way to do this would be | 17:42 |
| TheJulia | lmcgann: there is no reason, we store files in swift quite a bit, just never needed to really directly get one | 17:46 |
| TheJulia | JayF: so... any ideas why your script change for ipa-b doesn't work if run in the script? | 17:46 |
| JayF | I don't have context on that statement, what's the problem? | 17:47 |
| lmcgann | TheJulia so it would be ok if I extended the swift module to add a get_object method then? | 17:48 |
| TheJulia | tl;dr, the find command is behaving differently when run from inside a script as opposed to the output you get if you just echo, copy, and paste it | 17:48 |
| TheJulia | JayF: basically, the gate is broken because it is resulting in 700+ MB images because nothing is getting pruned :\ | 17:48 |
| TheJulia | lmcgann: yup, that should be fine | 17:49 |
| lmcgann | Perfect | 17:49 |
| TheJulia | lmcgann: just be careful with who/how that information gets supplied and what account is used and whatnot | 17:50 |
| TheJulia | lmcgann: Also, if you end user context is passed in, it would be best, but if it has to run later, that may require some discussion | 17:50 |
| TheJulia | and by later, I mean as a result of a heartbeat operation or after deployment | 17:51 |
| TheJulia | https://paste.opendev.org/show/807963/ | 17:53 |
| TheJulia | JayF: ^^^ | 17:53 |
| lmcgann | Im trying to use it to pass the allowlist and excludelist files to Ironic. My thought is that it just grabs based on a url or container/object pair passed into driver_info. The files themselves would be retrieved by a method inside of the interface | 17:54 |
| JayF | TheJulia: I don't know why it's failing, it appeared to work properly when I was using it after writing it | 17:55 |
| JayF | TheJulia: I can't dig deep right now I'm in a knowledge transfer meeting downstream :| | 17:55 |
| lmcgann | Id never worked with swift up to this point so I am not sure exactly how horribly wrong this could go | 17:55 |
| TheJulia | JayF: were you always overriding the value? | 17:55 |
| JayF | TheJulia: yes | 17:55 |
| TheJulia | hmmmmm | 17:55 |
| TheJulia | JayF: okay, thanks | 17:55 |
| JayF | but I also haven't checked my initramfs for any pruned files... | 17:55 |
| JayF | one sec | 17:56 |
| JayF | explicitly checking our downstream image here to validate that | 17:57 |
| JayF | TheJulia: I can confirm my downstream images are not pruned :| I do not know what changed between the copy of code I tested and what got committed | 17:58 |
| TheJulia | JayF: ack, thanks, that at least is a datapoint for me, I'll keep hunting | 17:59 |
| JayF | I'm very sorry :( if you need to revert please give me a heads up so I can just patch out the prune downstream for now | 17:59 |
| TheJulia | JayF: k, trying to prune down possibilities now | 18:01 |
| JayF | pun intended? | 18:01 |
| TheJulia | JayF: absolutely | 18:49 |
| opendevreview | cenne proposed openstack/python-ironicclient master: Add support for changing 'boot_mode' and 'secure_boot' states https://review.opendev.org/c/openstack/python-ironicclient/+/803970 | 19:00 |
| opendevreview | cenne proposed openstack/python-ironicclient master: Add support for changing 'boot_mode' and 'secure_boot' states https://review.opendev.org/c/openstack/python-ironicclient/+/803970 | 19:06 |
| opendevreview | cenne proposed openstack/python-ironicclient master: Add support for changing 'boot_mode' and 'secure_boot' states https://review.opendev.org/c/openstack/python-ironicclient/+/803970 | 19:15 |
| TheJulia | JayF: doesn't seem to be anything with string expansion and if I force string expansion by passing through echo, same thing occurs | 19:17 |
| JayF | TheJulia: 'same thing occurs' meaning what? You see the command line created as you expect? | 20:11 |
| JayF | Hmm. I bet the single quotes are somehow messing things up | 20:39 |
| JayF | Because even though they exist in the older version, I'm not convinced they weren't interpreted by the outer bash | 20:39 |
| JayF | (vs these which exist inside a variable) | 20:39 |
| JayF | TheJulia: the problem becomes apparent if you toss a `shopt -s failglob` in there. At some point, bash is trying to glob | 21:02 |
| TheJulia | JayF: so regardless it is trying to run the expansion of the fields | 21:13 |
| TheJulia | lovely | 21:13 |
| JayF | Yeah, I'm honestly at a bit of a loss as to how to fix this. | 21:14 |
| TheJulia | The only way I think we could is to change the IFS | 21:14 |
| TheJulia | and make it something like colon delimited | 21:14 |
| JayF | It seems like fixing it may be impossible inside the limits of posix sh | 21:14 |
| TheJulia | quite possibly | 21:14 |
| JayF | you know, I bet that's why it's structured how it is now | 21:16 |
| JayF | s/now/was before my change/ | 21:16 |
| JayF | that looks almost intentionally like trying to avoid bash quoting BS | 21:16 |
| TheJulia | I was actually starting to think that as well | 21:16 |
| JayF | So I can patch downstream and revert if we need, or remove /var/log from that list of stuff to prune | 21:17 |
| JayF | hmm, how about this... how about I write a *separate* find which will delete all **files** in /var/log/ but not nuke the path | 21:17 |
| JayF | then leave /var/log out of the find command afterwards | 21:17 |
| TheJulia | JayF: ++ | 21:17 |
| JayF | so I'm going to put a pure revert up, and do that as a separate commit | 21:18 |
| TheJulia | JayF: sounds good to me | 21:18 |
| TheJulia | JayF: I was thinking that as a possible path forward earlier, but didn't know the extent in which your stuff needed to preserve $things | 21:18 |
| JayF | Well, TBH, the way it's structured now is just *asking* for this kinda problem in the future | 21:19 |
| JayF | because we just delete entire paths without configuration or consideration as to what is needed | 21:19 |
| JayF | e.g. if I write an RPM and install it that *has to read* out of /usr/share/doc/ for some reason (who knows?) it's impossible to make it work in an IPA ramdisk | 21:19 |
| TheJulia | It is, however that was also after an insane amount of time spent taking apart things and double checking at least from an upstream standpoint | 21:19 |
| TheJulia | yup | 21:20 |
| JayF | the only other option I see is making that completely opt-outable | 21:20 |
| JayF | since for most use cases, with ipxe+http, having a larger image won't be a major impact | 21:20 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent-builder master: Revert "Allow configuration of paths to prune" https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/803967 | 21:24 |
| TheJulia | Well, testing would require more memory and that is a resource allocation change like an act of congress :( | 21:28 |
| JayF | Well I'm saying, an operator could set that to skip the pruning and probably be fine | 21:30 |
| JayF | but not so much in CI :D | 21:30 |
| JayF | TheJulia: ^ is that a correct statement? Have we seen operator failures due to ramdisk-too-large or is the optimization mostly for ci-use? | 21:30 |
| TheJulia | JayF: CI + QE | 21:31 |
| TheJulia | so like humans doing basic testing needing >4GB of ram is bad | 21:31 |
| JayF | so that sounds mostly like a concern an operator could reasonably opt out of | 21:32 |
| JayF | I think I'm going to go that route, to avoid the prune. I think it's the more correct universal fix. | 21:32 |
| JayF | Or at least, we should do *both things* and not just fix var/log because it scratches my current itch | 21:33 |
| * TheJulia shrugs in the "as long as we're able to merge patches" way | 21:33 | |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent-builder master: Allow opt-out of ramdisk pruning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/803994 | 21:53 |
| JayF | going to say ^^ this + the revert is enough for now. If I can find the time this week, I'll implement the "prune all files, but not all folders, from /var/log" logic, too, so I can still get my ramdisks pruned :D | 21:54 |
| TheJulia | JayF: looks like your missing "fi" | 22:17 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent-builder master: Allow opt-out of ramdisk pruning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/803994 | 22:18 |
| JayF | TheJulia: what missing fi <.< >.> | 22:18 |
| JayF | thanks | 22:18 |
| TheJulia | I know nothing! ;) | 22:20 |
| janders | good morning Ironic o/ | 23:40 |
| TheJulia | good morning janders | 23:42 |
| janders | hey TheJulia | 23:43 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Enable priority overrides to enable/disable steps https://review.opendev.org/c/openstack/ironic/+/803887 | 23:45 |
| janders | what determines whether a clean step is treated as in-band or out-of-band? | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!