| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Add support for verify steps https://review.opendev.org/c/openstack/ironic/+/800001 | 04:21 |
|---|---|---|
| iurygregory | good morning janders and Ironic o/ | 06:30 |
| *** rpittau|afk is now known as rpittau | 07:15 | |
| rpittau | good morning ironic! o/ | 07:15 |
| iurygregory | morning rpittau o/ | 07:17 |
| rpittau | hey iurygregory :) | 07:17 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/wallaby: [DNM] Test lower-constraints should not run on stable branch https://review.opendev.org/c/openstack/ironic/+/803142 | 07:23 |
| dtantsur | morning ironic | 07:37 |
| dtantsur | NobodyCam: group operations is something we should eventually get to | 07:38 |
| iurygregory | good morning dtantsur | 07:39 |
| janders | good morning iurygregory rpittau dtantsur and Ironic o/ | 07:46 |
| rpittau | hey janders :) | 07:46 |
| janders | heading off to drop off the car for oil service, back bit later | 07:47 |
| janders | (I should prob learn how to do these myself, but there's only that many hours in a day :/) | 07:57 |
| opendevreview | Riccardo Pittau proposed openstack/networking-baremetal master: Increase version of hacking and pycodestyle https://review.opendev.org/c/openstack/networking-baremetal/+/803148 | 08:02 |
| dtantsur | folks, looking for some reviews on https://review.opendev.org/c/openstack/bifrost/+/782531 please. it's required for Fedora (as well as https://review.opendev.org/c/openstack/bifrost/+/803104/) | 08:12 |
| cenne | Good morning ironic! | 08:14 |
| iurygregory | morning cenne o/ | 08:19 |
| cenne | hey o/ | 08:19 |
| *** sshnaidm_ is now known as sshnaidm | 08:20 | |
| dtantsur | cenne: hi! what were your europython highlights? :) | 08:47 |
| cenne | dtantsur: I liked one of the fun talks by Antonis Christofides. It was satire, I think you'd love it | 09:00 |
| dtantsur | what topic was it on? | 09:01 |
| cenne | Why you always had trouble understanding metaclasses by Antonis Christofides | 09:02 |
| cenne | It's not what it seems though :D | 09:02 |
| cenne | It goes into all the misnomers the speaker thinks exists in the tech world. And how it affects our thinking. | 09:04 |
| dtantsur | ah, I see. yeah, cool :) | 09:06 |
| cenne | 'George's function' introduction was cool imo, in the talk. If you see the talk you'll get it :p | 09:07 |
| cenne | Heard a few stories. It was nice to get a feel for the different backgrounds people come from , | 09:07 |
| cenne | And their journeys. I love stories! | 09:07 |
| cenne | - Claudia Comito [Connecting Communities: Interdisciplinary Science project] | 09:07 |
| cenne | - Joost Lek [helping fly a helicopter on Mars] | 09:07 |
| dtantsur | I guess I should check it out | 09:07 |
| cenne | yes! I think you'll enjoy it. | 09:12 |
| cenne | The funding talk seemed was cool. Towards a sustainable solution to open source sustainability by Tobie Langel | 09:13 |
| cenne | I liked the illustration (of amount). *mind-blowing* | 09:13 |
| cenne | *seemed. | 09:13 |
| cenne | They had this wonder.me thing. It's.. like a virtual conference floorspace. you can move across rooms and walk the hallway, bump into people :P | 09:15 |
| cenne | Wanted to try that and hang out for a while. Unfortunately couldn't find the time. :( | 09:16 |
| dtantsur | yeah, time is always an issue | 09:18 |
| janders | dtantsur iurygregory thank you for reviews / +2s on https://review.opendev.org/c/openstack/sushy/+/802690 | 09:22 |
| janders | now that it's merged I will start going through instructions from rpittau to get this into ironic-image | 09:22 |
| dtantsur | janders: you probably need to start with a backport | 09:24 |
| iurygregory | I was about to say this ^ | 09:25 |
| opendevreview | Merged openstack/bifrost master: Drop external tftp service in favor to use dnsmasq's one https://review.opendev.org/c/openstack/bifrost/+/782531 | 09:29 |
| opendevreview | Merged openstack/bifrost master: CI: use legacy crypto on Fedora with Cirros https://review.opendev.org/c/openstack/bifrost/+/803104 | 09:29 |
| janders | dtantsur iurygregory that's right | 09:38 |
| opendevreview | Merged openstack/networking-baremetal master: Increase version of hacking and pycodestyle https://review.opendev.org/c/openstack/networking-baremetal/+/803148 | 09:40 |
| opendevreview | Merged openstack/sushy master: Removing optional fields from insert_media payload https://review.opendev.org/c/openstack/sushy/+/802690 | 10:03 |
| opendevreview | Merged openstack/sushy master: Fix OEM required attribute parsing https://review.opendev.org/c/openstack/sushy/+/800267 | 10:03 |
| opendevreview | Merged openstack/sushy master: Add resource root property https://review.opendev.org/c/openstack/sushy/+/799504 | 10:03 |
| dtantsur | probably the time for a sushy release ^^ | 10:03 |
| rpittau | dtantsur: I added the request https://review.opendev.org/c/openstack/releases/+/803165 | 10:11 |
| dtantsur | thanks! | 10:13 |
| opendevreview | cenne proposed openstack/ironic master: Add api endpoint for setting boot_mode and secure_boot state https://review.opendev.org/c/openstack/ironic/+/800084 | 10:47 |
| cenne | Can I get a review on this please ^ | 10:49 |
| dtantsur | I'll add it to my list | 11:00 |
| cenne | thanks. | 11:14 |
| opendevreview | Jacob Anders proposed openstack/sushy master: Change defaults - optional insert_media attributes https://review.opendev.org/c/openstack/sushy/+/802452 | 11:39 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Move Nginx code to a new role bifrost-nginx-install https://review.opendev.org/c/openstack/bifrost/+/802532 | 11:40 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: DNM test the upgrade job https://review.opendev.org/c/openstack/bifrost/+/800673 | 11:45 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: DNM test the upgrade job https://review.opendev.org/c/openstack/bifrost/+/800673 | 11:46 |
| dtantsur | would appreciate some reviews on https://review.opendev.org/c/openstack/bifrost/+/802754 it has given me pain since a while | 11:49 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Trivial: fix a warning in bifrost-keystone-client-config https://review.opendev.org/c/openstack/bifrost/+/803196 | 11:51 |
| opendevreview | Jacob Anders proposed openstack/sushy stable/wallaby: Removing optional fields from insert_media payload https://review.opendev.org/c/openstack/sushy/+/803197 | 11:52 |
| iurygregory | dtantsur, lgtm, I would ask for a releasenote, but you can push in other patch to avoid running all CI jobs again =) | 11:58 |
| dtantsur | iurygregory: my reasoning behind no release notes was: 1) it's only in testing code, 2) it shouldn't be user-visible for most people except for hardcore devs like me who keep their environments for months :) | 12:01 |
| dtantsur | I can create one if you think it's not convincing | 12:01 |
| iurygregory | dtantsur, it's fine =) | 12:02 |
| janders | see you tomorrow Ironic o/ | 12:08 |
| iurygregory | bye janders o/ | 12:10 |
| rpittau | the limited l-c job looks ok, no running on wallaby https://review.opendev.org/c/openstack/ironic/+/803142 | 12:19 |
| rpittau | if we're still ok with that we can move forward with https://review.opendev.org/c/openstack/ironic/+/801350 | 12:19 |
| iurygregory | I've added the hashtag ^ | 12:20 |
| opendevreview | Merged openstack/bifrost master: Keep sushy-emulator state directory in /var/lib https://review.opendev.org/c/openstack/bifrost/+/802754 | 13:23 |
| *** pmannidi is now known as pmannidi|AFK | 14:08 | |
| opendevreview | Riccardo Pittau proposed openstack/networking-generic-switch master: Increase version of hacking and pycodestyle https://review.opendev.org/c/openstack/networking-generic-switch/+/803213 | 14:15 |
| TheJulia | NobodyCam: That is a really good idea. I like it! There would be technical details as in mechanics to work out since I *suspect* the right way to do it and avoid impinging upon the API too heavily is to have cross-conductor RPC interaction where the webserver files a "do cluster deploy" to a single conductor, and that single conductor then issues the tasks for all of the nodes which would require the conductor to file a | 14:32 |
| TheJulia | message on the topic or be able to reach the jsonrpc ports on the various conductors. | 14:32 |
| dtantsur | good morning TheJulia | 14:33 |
| TheJulia | good morning dtantsur | 14:33 |
| dtantsur | there is also a question if whether it should be done per chassis or per some other grouping (tags) | 14:33 |
| iurygregory | good morning TheJulia | 14:33 |
| rpittau | good morning TheJulia :) | 14:33 |
| TheJulia | dtantsur: this is also super valid, I would suspect yes is the anser | 14:33 |
| TheJulia | answer | 14:33 |
| TheJulia | free form but then also hyper consistent chassis needs are totally a thing | 14:34 |
| dtantsur | and maybe we should account for a potential group operation in the new deployment API | 14:34 |
| TheJulia | "my blades int his chassis always do the same thing" | 14:34 |
| dtantsur | so many cool ideas, so little time.. | 14:34 |
| TheJulia | Dunno, the idea, at least as what NobodyCam mentioned to me when we were waiting on dinner Saturday was the cluster is always consistent and contained, and always deployed together | 14:35 |
| dtantsur | yep, but if we do only ^^^, people will start abusing chassis for grouping arbitrary nodes | 14:35 |
| TheJulia | agreed | 14:35 |
| TheJulia | that was a concern I started to type out | 14:35 |
| dtantsur | bloody upgrade CI, Y U NO WRK? | 14:36 |
| TheJulia | and then I thought about it for about five minutes and thought "maybe it would be okay, since they may know their "chassis" better than we do | 14:36 |
| iurygregory | I think dtantsur will start to flip tables in his flat... | 14:37 |
| TheJulia | so what if we ensure all of the tables are padded? | 14:38 |
| dtantsur | my back hurts, that's the only reason I don't | 14:38 |
| TheJulia | my back also hurts | 14:38 |
| TheJulia | I don't know *what* I've done to it, but it hates me | 14:40 |
| iurygregory | I know this feeling also | 14:41 |
| dtantsur | I raised my hand on Friday. A terrible mistake. | 14:41 |
| * TheJulia wonders if the ironic community needs a yoga instrutor | 14:41 | |
| dtantsur | well, at least we'll have yoga :) | 14:42 |
| dtantsur | NO SERIOUSLY | 14:44 |
| rpittau | should we found a back-hurting club? | 14:44 |
| dtantsur | about the right time | 14:44 |
| dtantsur | I have a testing patch that removes the ironic packages from /opt/bifrost.. it still starts the old code. HOW? | 14:44 |
| TheJulia | dtantsur: ls -la /opt/stack/bifrost/bin/ironic-conductor ? | 14:45 |
| iurygregory | back-hurting club OMG | 14:45 |
| dtantsur | I've removed the old code. The restart happens after it. Pbr reports the correct version is installed. | 14:46 |
| dtantsur | I'm slowly going insane (even more than I usually am) | 14:46 |
| dtantsur | okay, I'll try removing /opt/stack/ironic as well | 14:46 |
| TheJulia | dtantsur: I'm asking because it sounds like it is making a copy of the source and squerreling it into the venv | 14:47 |
| TheJulia | oh wait, its not going to be a link | 14:47 |
| dtantsur | yep | 14:47 |
| dtantsur | and we re-run installation completely | 14:47 |
| TheJulia | gah | 14:47 |
| dtantsur | which has worked in my environment for dozens times already | 14:47 |
| TheJulia | hmmm | 14:47 |
| dtantsur | there must be some insight that I'm lacking | 14:47 |
| dtantsur | the only difference between the CI and my local bifrost is that I use a direct git clone (the default), while the CI uses copying from zuul | 14:48 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: DNM test the upgrade job https://review.opendev.org/c/openstack/bifrost/+/800673 | 14:48 |
| * TheJulia suspects more coffee required | 14:50 | |
| dtantsur | no change in bugs stats... I wonder if the script is broken or we've achieved a perfect balance (as it should be) | 14:50 |
| dtantsur | TheJulia: JFYI xinetd is gone from bifrost | 14:52 |
| TheJulia | dtantsur: I saw :) | 14:52 |
| dtantsur | and we have Fedora support again | 14:52 |
| iurygregory | <insert happy dance gif> | 14:53 |
| TheJulia | \o/ | 14:53 |
| TheJulia | o/ Qianbiao | 14:55 |
| Qianbiao | Hi TheJulia | 14:55 |
| Qianbiao | Good morning | 14:55 |
| TheJulia | Good morning! Hope your doing well :) | 14:55 |
| Qianbiao | :) being buzy those days. | 14:56 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Update the supported Fedora versions https://review.opendev.org/c/openstack/bifrost/+/803222 | 14:56 |
| dtantsur | speaking of fedora ^^^ | 14:56 |
| dtantsur | hi Qianbiao, long time no chat :) | 14:56 |
| Qianbiao | morning dtantsur | 14:56 |
| Qianbiao | lol yeah, quit a long time | 14:56 |
| iurygregory | hey Qianbiao o/ | 14:56 |
| Qianbiao | Good morning iurygregory. | 14:56 |
| Qianbiao | Good morning Ironic and all friends. :) | 14:57 |
| rpittau | hey Qianbiao :) | 14:57 |
| Qianbiao | Hey rpittau | 14:57 |
| Qianbiao | Is it the time for meeting. | 15:00 |
| rpittau | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Aug 2 15:00:11 2021 UTC and is due to finish in 60 minutes. The chair is rpittau. 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 |
| TheJulia | o/ | 15:00 |
| JayF | o/ | 15:00 |
| dtantsur | o/ | 15:00 |
| erbarr | o/ | 15:00 |
| Qianbiao | o/ | 15:00 |
| stendulker_ | o/ | 15:00 |
| ajya | o/ | 15:00 |
| rpioso | \o | 15:00 |
| iurygregory | o/ | 15:00 |
| rpittau | welcome everyone! | 15:01 |
| rpittau | our agenda for the meeting can be found here: | 15:01 |
| rpittau | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| rpittau | #topic Announcements / Reminder | 15:01 |
| rpittau | #info Xena Midcycle | 15:02 |
| rpittau | according to the doodle (thanks iurygregory!) the midcycle will take place on August 23 (3-4PM UTC ), 24 (2-3PM UTC) and 25 (2-3PM) | 15:03 |
| iurygregory | we may not use all 3 days since we don't have a lot of topics | 15:03 |
| iurygregory | =) | 15:03 |
| rpittau | we're still in time to add more topics! | 15:03 |
| iurygregory | yeah! | 15:03 |
| rpittau | #info Please add topics for the midcycle slot: https://etherpad.opendev.org/p/ironic-xena-midcycle | 15:04 |
| JayF | Can someone update the whiteboard with the links to planning docs for... yeah, I'll do it :D | 15:04 |
| rpittau | thanks JayF :) | 15:04 |
| iurygregory | I would say we can wait till 16 with the topic list | 15:04 |
| iurygregory | 16th August (2 weeks from today) | 15:04 |
| rpittau | 1 week in advance sounds good | 15:04 |
| TheJulia | that sounds fair | 15:05 |
| rpittau | anything more to add for the midcycle today ? | 15:05 |
| iurygregory | I think we are good | 15:06 |
| rpittau | #info Yoga PTG (October 18-22, 2021) | 15:06 |
| rpittau | we still have time before the PTG | 15:06 |
| rpittau | friendly reminder to add ideas and topics here: | 15:06 |
| rpittau | #link https://etherpad.opendev.org/p/ironic-yoga-ptg | 15:06 |
| rpittau | alright, anything else to announce or remind of? | 15:07 |
| rpittau | ok, moving forward | 15:08 |
| rpittau | #topic Review action items from previous meeting | 15:08 |
| TheJulia | Uhh, I *think* RH folks are all off next friday? | 15:08 |
| rpittau | oh yeah, right! | 15:09 |
| JayF | What sort of RH-specific holiday is it? lol | 15:09 |
| rpittau | Friday the 13th all Red Hat people will be off | 15:09 |
| dtantsur | we have a quarterly holiday, at least while covid is ongoing | 15:09 |
| JayF | International Hat day, and they are featuing the Fedora this year? /s | 15:09 |
| dtantsur | also FYI: I'll be out the whole next week, then Mon and Tue | 15:09 |
| JayF | dtantsur: ah, that's nice :D | 15:09 |
| dtantsur | hopefully reduced reachability :) | 15:10 |
| * TheJulia needs to take a little PTO | 15:11 | |
| iurygregory | yay day off again \o/ | 15:11 |
| rpittau | I don't see action items from last time | 15:12 |
| rpittau | TheJulia: you should :) | 15:12 |
| * rpittau having some network issues | 15:12 | |
| rpittau | anyway | 15:12 |
| rpittau | #topic Review subteam status reports | 15:12 |
| rpittau | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:12 |
| rpittau | around line 60 | 15:13 |
| opendevreview | Merged openstack/bifrost master: Update the supported Fedora versions https://review.opendev.org/c/openstack/bifrost/+/803222 | 15:15 |
| rpittau | I have ther feeling the node history work will come up at the midcycle | 15:16 |
| TheJulia | iurygregory: I'll try and punt the error history patch along this week | 15:16 |
| iurygregory | tks TheJulia =) | 15:17 |
| rpittau | TheJulia: RBAC work is completed? :) | 15:17 |
| TheJulia | do we want to declare the rbac work done, short of ripping out the old policies which can't happen now until ?next? cycle? | 15:17 |
| iurygregory | and there is also the tests we want in tempest | 15:18 |
| TheJulia | more tests? | 15:18 |
| iurygregory | we already add everything? :D | 15:18 |
| TheJulia | the patch merged with enforce scope enabled on several jobs :) | 15:18 |
| iurygregory | more is always better (kidding we will probably kill our CI if we start adding more things :D ) | 15:18 |
| TheJulia | we just don't have tests in keystone that do default policy checking int erms of "we expect a default deployment to have this policy behavior" | 15:19 |
| TheJulia | All of that testing was done as unit tests | 15:19 |
| rpittau | TheJulia: if all the testing is in place, maybe add a note on removing the old policies to the xena PTG doc so that can be re-taken during the next cycle? | 15:20 |
| TheJulia | works for me | 15:20 |
| rpittau | thanks! | 15:20 |
| TheJulia | done | 15:22 |
| rpittau | good :) | 15:22 |
| rpittau | I think we can move on | 15:22 |
| TheJulia | yup | 15:22 |
| rpittau | #topic Deciding on priorities for the coming week | 15:22 |
| rpittau | so much lag..... | 15:23 |
| rpittau | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:23 |
| TheJulia | I noticed https://review.opendev.org/c/openstack/ironic-inspector/+/786571, and wonder if we should get some eyes on it | 15:24 |
| rpittau | yep | 15:24 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/803128 would be good | 15:25 |
| dtantsur | I have a few Bifrost things https://review.opendev.org/c/openstack/bifrost/+/802532 https://review.opendev.org/c/openstack/bifrost/+/803196 | 15:25 |
| rpittau | ok, feel free to add the ironic-week-prio tag | 15:26 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/801870 and https://review.opendev.org/c/openstack/ironic/+/801871 (which needs the prior refactoring patch) | 15:26 |
| TheJulia | both are backports | 15:26 |
| rpittau | yes, they're already in the list | 15:27 |
| TheJulia | doh! | 15:27 |
| rpittau | a reminder helps, I missed them :) | 15:27 |
| TheJulia | iurygregory: I've approved the spec move to backlog | 15:28 |
| rpittau | any more prios for the week ? | 15:28 |
| iurygregory | TheJulia, tks! I just saw the notification :D | 15:29 |
| cenne | previous prios stay props right? | 15:29 |
| TheJulia | cenne: yes | 15:29 |
| iurygregory | cenne, correct | 15:29 |
| TheJulia | So onward? | 15:29 |
| rpittau | let's | 15:29 |
| cenne | thanks. | 15:29 |
| rpittau | #topic Discussion | 15:29 |
| rpittau | we have one request for discussion from JayF | 15:30 |
| JayF | So I wanted to let folks know, in two weeks time I'll be changing jobs, and will no longer be working on Ironic as part of my full time day job. | 15:30 |
| rpittau | oh :/ | 15:30 |
| iurygregory | JayF, noooooooo =( | 15:30 |
| JayF | I'll spend a small amount of personal time continuing to perform reviews until I lose context on the project. | 15:30 |
| JayF | and as always I'll idle here if you need to scratch my brain about something :) | 15:31 |
| iurygregory | JayF, good luck on your next job :D | 15:31 |
| TheJulia | JayF: Congratulations on the new job! | 15:31 |
| rpittau | JayF: congrats on the new job! :) | 15:31 |
| cenne | bests on your new job JayF! | 15:31 |
| TheJulia | JayF: Should we expect a flury of patches to be upstreamed? | 15:31 |
| dtantsur | JayF: le sigh.. but congrats! | 15:32 |
| JayF | TheJulia: I don't think I've ever pushed a flurry of anything up :P | 15:32 |
| JayF | I don't have any outstanding work to upstream at this time | 15:32 |
| TheJulia | JayF: True, Okay | 15:32 |
| JayF | Thanks for all the congrats; I will miss working with you all eventually, but I can't miss you if I don't go away entirely :D | 15:34 |
| TheJulia | :) | 15:34 |
| JayF | That was essentially all I had for my discussion item; if we wanted to move on? | 15:35 |
| stendulker_ | All the best JayF :) | 15:35 |
| rpittau | ok, moving forward, thanks JayF :) | 15:35 |
| rpittau | #topic Baremetal SIG | 15:36 |
| rpittau | #info Next Baremetal SIG meeting in September, skipping August due to vacations. | 15:36 |
| rpittau | #info Tentative topic as an operator roundtable. | 15:36 |
| rpittau | and I think that's all on the baremetal sig | 15:36 |
| TheJulia | Yup | 15:36 |
| rpittau | we don't have any RFE to review, so I'll skip that | 15:36 |
| rpittau | #topic Open Discussion | 15:37 |
| TheJulia | I may have lost my mind even more -> We got two kittens | 15:37 |
| JayF | Do we have any DIB core crossover with Ironic? I've been trying to get a DIB patch merged with very little success. | 15:37 |
| rpittau | \o/ kittens | 15:37 |
| dtantsur | TheJulia: (kittens++)++ | 15:37 |
| * rpittau saw a corgi in Versailles yesterday for the first time | 15:38 | |
| TheJulia | JayF: we don't, although I've been wondering if we need someone or something | 15:38 |
| dtantsur | JayF: no.. I think they only have 2-3 cores active | 15:38 |
| dtantsur | TheJulia: we demand proof-pics of kittens! | 15:38 |
| rpittau | JayF: dib merges take some time unfortunately :/ | 15:38 |
| iurygregory | easy path is to ping in their channel hehe | 15:38 |
| JayF | https://review.opendev.org/c/openstack/diskimage-builder/+/802592 is the patch, if we get more +1s on it maybe someone will land it? | 15:38 |
| dtantsur | meanwhile, I have a small RFE https://storyboard.openstack.org/#!/story/2009093 | 15:38 |
| JayF | this is exclusively to enable something downstream I'm doing, so not really upstream-ironic-related, but just thought I'd ask :D | 15:38 |
| JayF | dtantsur: +1 to that RFE, I'm going to comment that we should explictly test that the redaction of those fields happens | 15:39 |
| rpittau | JayF: that's ok, I added a +1 :) | 15:39 |
| dtantsur | JayF: if you mean driver_info, I think we redact everything that has "password" in it | 15:40 |
| JayF | dtantsur: Yeah; I do; and I know but I like being explicit and validating vs having a security issue :) | 15:40 |
| rpittau | dtantsur: approved also from me | 15:40 |
| dtantsur | totally :) | 15:40 |
| * TheJulia has kittens conquering her office chair | 15:41 | |
| dtantsur | surrender, resistance is futile | 15:41 |
| TheJulia | so, I am fairly sure our api sanitization code checks that fields are sanitized, but anything with "username", "password", "secret" in their name get redacted | 15:42 |
| dtantsur | not username, but yeah | 15:42 |
| TheJulia | oh, yeah | 15:42 |
| TheJulia | not username your right | 15:42 |
| opendevreview | Merged openstack/ironic-specs master: Event Subscription Idea https://review.opendev.org/c/openstack/ironic-specs/+/785742 | 15:43 |
| JayF | Yeah, I'm almost 100% certain it'll just be free to have it redacted | 15:43 |
| TheJulia | *but* tests should go in for new known values | 15:43 |
| TheJulia | to ensure it | 15:43 |
| cenne | rise of the kittens yes! ^^ | 15:44 |
| * TheJulia needs an excuse to put them into a slide deck or blog post | 15:44 | |
| dtantsur | you don't really need an excuse, just do it! :) | 15:45 |
| iurygregory | agree ^] | 15:45 |
| TheJulia | hehe | 15:45 |
| TheJulia | Anyway, if anyone has a little time this week, I'd love to discuss the upcoming openinfra live and brainstorm things to be talked about | 15:46 |
| TheJulia | Seems like I'll do the presentation | 15:46 |
| rpittau | absolutely | 15:46 |
| TheJulia | So if anyone has things they feel are particularly important to highlight, lmk | 15:46 |
| rpittau | is it the 12th? | 15:46 |
| TheJulia | yes | 15:47 |
| iurygregory | sounds good to me | 15:47 |
| TheJulia | https://docs.google.com/presentation/d/1VDOH_-6H0ZFHDwE7vSj4GypFBPUy8_7LJ2wl419hlu0/edit?usp=sharing <-- anyone should be able to comment. | 15:47 |
| TheJulia | And we already have an Anonymous Squirrel and Anonymous Sheep :) | 15:48 |
| rpittau | I'm probably the squirrel :D | 15:48 |
| * iurygregory wondering if google has Anonymous Corgi :D | 15:48 | |
| JayF | just gotta F5 that google doc until you are an Anonymous Kitty | 15:48 |
| rpittau | lol | 15:49 |
| TheJulia | lol | 15:49 |
| * TheJulia wonders if there is Anonymous Loki | 15:49 | |
| TheJulia | s/is/is an/ | 15:49 |
| rpittau | probably in a different timeline :P | 15:49 |
| iurygregory | rpittau, nice answer! | 15:49 |
| rpittau | alright, anything else to discuss today? :) | 15:50 |
| iurygregory | I think we can go to the last topic =) | 15:50 |
| TheJulia | I don't think so, as long as we have our glorious purpose | 15:50 |
| rpittau | the most important one | 15:50 |
| rpittau | #topic Who is going to run the next meeting? | 15:50 |
| iurygregory | I can if there is no objections =) | 15:51 |
| TheJulia | None from me | 15:51 |
| rpittau | I'll allow it :P | 15:51 |
| iurygregory | hehehe | 15:51 |
| rpittau | thanks iurygregory :) | 15:51 |
| iurygregory | np | 15:51 |
| TheJulia | Thanks! | 15:51 |
| rpittau | thanks everyone! | 15:51 |
| rpittau | #endmeeting | 15:51 |
| opendevmeet | Meeting ended Mon Aug 2 15:51:56 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:51 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-02-15.00.html | 15:51 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-02-15.00.txt | 15:51 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-08-02-15.00.log.html | 15:51 |
| Qianbiao | Thanks | 15:53 |
| * TheJulia fires up music and proceeds to investigate a bug | 15:56 | |
| dtantsur | ooookay, removing /opt/stack/ironic fixes the upgrade | 16:14 |
| rpittau | \o/ | 16:15 |
| dtantsur | that's finally a clue. now I just need to interpret it. | 16:15 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: DNM test the upgrade job https://review.opendev.org/c/openstack/bifrost/+/800673 | 16:19 |
| rpittau | good night! o/ | 16:24 |
| *** rpittau is now known as rpittau|afk | 16:24 | |
| NobodyCam | Good Morning Ironic'ers | 16:24 |
| NobodyCam | Happy Monday! | 16:24 |
| dtantsur | morning NobodyCam | 16:25 |
| *** sshnaidm is now known as sshnaidm|afk | 16:25 | |
| NobodyCam | Morning dtantsur :) | 16:26 |
| NobodyCam | and you know I would be the first to abuse chassis deploy | 16:26 |
| NobodyCam | ;p | 16:26 |
| dtantsur | I never doubted that ;) | 16:27 |
| NobodyCam | hehehehe | 16:27 |
| TheJulia | s/abuse/extensively leverage/ | 16:29 |
| NobodyCam | well yes! | 16:30 |
| TheJulia | I do think that it is a fair use case from an operations point of view | 16:38 |
| TheJulia | I also think we can't get too hung up on designing it | 16:39 |
| TheJulia | "what makes sense" is a strong indicator for us | 16:39 |
| opendevreview | Merged openstack/ironic stable/wallaby: Suppress policy deprecation and default change warnings https://review.opendev.org/c/openstack/ironic/+/799253 | 16:39 |
| * dtantsur is stares at ansible in disbelieve | 16:43 | |
| dtantsur | $ git diff | 16:45 |
| dtantsur | Segmentation fault (core dumped) | 16:45 |
| dtantsur | bloody computers | 16:45 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Remove destination when doing copy_from_local_path https://review.opendev.org/c/openstack/bifrost/+/803234 | 16:50 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Avoid a double restart of ironic components https://review.opendev.org/c/openstack/bifrost/+/803235 | 16:53 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: DNM test the upgrade job https://review.opendev.org/c/openstack/bifrost/+/800673 | 16:54 |
| dtantsur | maybe I have a fix. maybe. we'll know soon. | 16:55 |
| dtantsur | meanwhile, have a nice evening | 16:55 |
| dtantsur | o/ | 16:55 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Avoid a double restart of ironic components https://review.opendev.org/c/openstack/bifrost/+/803235 | 16:58 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Remove destination when doing copy_from_local_path https://review.opendev.org/c/openstack/bifrost/+/803234 | 16:58 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: DNM test the upgrade job https://review.opendev.org/c/openstack/bifrost/+/800673 | 16:58 |
| * TheJulia slides a Guinness across the intertubes to dmitry for the segmentation fault. | 17:28 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Implements node history: database https://review.opendev.org/c/openstack/ironic/+/768009 | 23:08 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Record Errors in history https://review.opendev.org/c/openstack/ironic/+/803292 | 23:08 |
| TheJulia | ^^ is just like two hours of "what would this kind of look like hacking" | 23:09 |
| TheJulia | its going to be invasive to capture, but really just lots of "grunt" work updating the code | 23:09 |
| janders | good morning Ironic o/ | 23:15 |
| *** pmannidi|AFK is now known as pmannidi | 23:50 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!