opendevreview | Steve Baker proposed openstack/bifrost master: Fix TFTP read access for enforcing SELinux https://review.opendev.org/c/openstack/bifrost/+/849246 | 00:54 |
---|---|---|
opendevreview | Steve Baker proposed openstack/bifrost master: Copy shim and grub into tftp and http directories https://review.opendev.org/c/openstack/bifrost/+/849247 | 00:54 |
opendevreview | Steve Baker proposed openstack/bifrost master: Remove enable_uefi_ipxe https://review.opendev.org/c/openstack/bifrost/+/849248 | 00:54 |
opendevreview | Steve Baker proposed openstack/bifrost master: Support PXE network boot with grub https://review.opendev.org/c/openstack/bifrost/+/807220 | 00:54 |
opendevreview | SONG SHUKUN proposed openstack/ironic stable/wallaby: [iRMC] Add SNMPv3 authentication functionality https://review.opendev.org/c/openstack/ironic/+/850707 | 01:08 |
opendevreview | SONG SHUKUN proposed openstack/ironic stable/xena: [iRMC] Add SNMPv3 authentication functionality https://review.opendev.org/c/openstack/ironic/+/850554 | 01:10 |
opendevreview | SONG SHUKUN proposed openstack/ironic stable/yoga: [iRMC] Add SNMPv3 authentication functionality https://review.opendev.org/c/openstack/ironic/+/850553 | 01:11 |
opendevreview | Steve Baker proposed openstack/ironic master: WIP Implement a DHCP driver backed by dnsmasq https://review.opendev.org/c/openstack/ironic/+/851681 | 01:16 |
opendevreview | Steve Baker proposed openstack/bifrost master: WIP Switch bifrost to use the dnsmasq DHCP driver https://review.opendev.org/c/openstack/bifrost/+/851682 | 01:26 |
opendevreview | Merged openstack/ironic master: Clarify disk_label with a warning https://review.opendev.org/c/openstack/ironic/+/850945 | 03:32 |
opendevreview | Merged openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/851336 | 03:32 |
opendevreview | Steve Baker proposed openstack/bifrost master: WIP Switch bifrost to use the dnsmasq DHCP driver https://review.opendev.org/c/openstack/bifrost/+/851682 | 03:36 |
opendevreview | OpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/851691 | 03:53 |
opendevreview | Merged openstack/networking-generic-switch master: Support 802.3ad port groups on Cumulus devices https://review.opendev.org/c/openstack/networking-generic-switch/+/844062 | 04:08 |
opendevreview | Merged openstack/networking-baremetal master: Add OpenConfig classes for network-instance https://review.opendev.org/c/openstack/networking-baremetal/+/835159 | 04:09 |
opendevreview | Merged openstack/networking-generic-switch master: Declare l2 connectivity capability to tell Neutron we can handle L2-only ports https://review.opendev.org/c/openstack/networking-generic-switch/+/845760 | 04:52 |
*** akahat_ is now known as akahat | 05:39 | |
rpittau | good morning ironic! o/ | 08:00 |
janders | good morning rpittau and Ironic o/ | 08:26 |
rpittau | hey janders :) | 08:34 |
iurygregory | good morning Ironic | 11:15 |
rpittau | hey iurygregory :) | 11:22 |
rpittau | if any core has a minute please review https://review.opendev.org/c/openstack/ironic-python-agent/+/850497 thanks! | 11:22 |
opendevreview | Merged openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/851691 | 11:39 |
*** mnasiadka_ is now known as mnasiadka | 12:26 | |
kubajj | Hello, I have a few questions regarding my change https://review.opendev.org/c/openstack/ironic-python-agent/+/850861 I was told that I should include some documentation. Which file should I put it in? Is it one of the rsts in /doc? I was also told that I might need to include a release note. How do I do that? Lastly, if anyone has any suggestions or comments on my tests (it is my first time using mock) please let me know. | 12:32 |
dtantsur | kubajj: re release note: https://docs.openstack.org/ironic/latest/contributor/faq.html#create-a-new-release-note | 12:57 |
dtantsur | docs are located in doc/source, yes. there should be a chapter for existing clean steps. | 12:58 |
TheJulia | good morning | 13:07 |
* dtantsur sees https://review.opendev.org/c/openstack/ironic/+/851681/ and dances | 13:43 | |
dtantsur | morning TheJulia | 13:43 |
opendevreview | Merged openstack/ironic-inspector master: Remove the incorrect hack for Werkzeug 2.2.0 https://review.opendev.org/c/openstack/ironic-inspector/+/851501 | 14:19 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable the ramdisk deploy by default https://review.opendev.org/c/openstack/ironic/+/851751 | 14:35 |
dtantsur | here we go ^^ | 14:35 |
iurygregory | <insert fire emoji here> | 14:36 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Stop documenting netboot and the boot_option capability https://review.opendev.org/c/openstack/ironic/+/851037 | 14:37 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: WIP finally remove netboot https://review.opendev.org/c/openstack/ironic/+/850240 | 14:37 |
* TheJulia looks at the home wifi with suspicion | 14:39 | |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add ironic-grenade-skip-level Job https://review.opendev.org/c/openstack/ironic/+/836966 | 14:47 |
iurygregory | Unknown projects: opendev.org/openstack/diskimage-builder O.o | 14:51 |
dtantsur | Oo | 14:51 |
iurygregory | it was reported by Dell CI | 14:51 |
dtantsur | iurygregory: maybe you're trying to use a branch of it.. that's my only guess | 14:51 |
dtantsur | aahhh | 14:51 |
iurygregory | I'm like WHAT?! | 14:51 |
dtantsur | they may need to somehow add it | 14:52 |
iurygregory | ajya, fyi ^ | 14:52 |
iurygregory | not sure who to ping =) | 14:52 |
iurygregory | oh probably erbarr :D | 14:52 |
erbarr | huh, let me catch up | 14:53 |
iurygregory | erbarr, I saw the error in this patch https://review.opendev.org/c/openstack/ironic/+/836966 | 14:54 |
opendevreview | Julia Kreger proposed openstack/ironic master: Concurrent Distructive/Intensive ops limits https://review.opendev.org/c/openstack/ironic/+/840006 | 14:56 |
TheJulia | there, I *finally* got a chance to rebase that patch | 14:56 |
erbarr | iurygregory: ok thanks, I'll look around | 14:56 |
iurygregory | ty! | 14:56 |
iurygregory | #startmeeting ironic | 15:00 |
opendevmeet | Meeting started Mon Aug 1 15:00:03 2022 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 | Hello ironicers, welcome to our weekly meeting! | 15:00 |
iurygregory | o/ | 15:00 |
dtantsur | o/ | 15:00 |
matfechner | o/ | 15:00 |
rpittau | o/ | 15:00 |
TheJulia | o/ | 15:00 |
erbarr | o/ | 15:00 |
ajya | o/ | 15:00 |
iurygregory | #topic Announcements / Reminder | 15:01 |
iurygregory | #topic Revise Ironic Vision #2 | 15:01 |
iurygregory | #link https://doodle.com/meeting/organize/id/eZ4L5Xgb | 15:01 |
iurygregory | I forgot to send the email earlier last week (it was thursday when I did, sorry) | 15:02 |
hjensas | o/ | 15:02 |
iurygregory | so if you would like to participate please vote till wed, so we can schedule something for next week ^^ | 15:02 |
iurygregory | #topic PTG Columbus, OH | 15:03 |
iurygregory | This is just a reminder to add topics to the etherpad :D | 15:03 |
iurygregory | #link https://etherpad.opendev.org/p/ironic-ptg-planing-Columbus-OH | 15:03 |
iurygregory | Does anyone have something that would like to Announcements / Reminder ? | 15:03 |
TheJulia | I suspect we're going to need a pre-ptg call on topics, fwiw | 15:05 |
iurygregory | TheJulia, do you think 1month before the PTG would be a good time? | 15:07 |
TheJulia | maybe 1.5 months? Maybe? | 15:07 |
TheJulia | There is also some rough plans forming up for operator attendance, but it is still in initial phases, so anything operator centric may also be good | 15:07 |
TheJulia | even if it is "lets go meet with operators" | 15:07 |
iurygregory | makes total sense to me | 15:08 |
* TheJulia goes and adds that | 15:08 | |
iurygregory | speaking about PTG | 15:08 |
iurygregory | #info Don't forget to buy your tickets | 15:09 |
iurygregory | #link https://www.eventbrite.com/e/project-teams-gathering-columbus-2022-tickets-374118026087 | 15:09 |
TheJulia | ++ | 15:09 |
TheJulia | oh, it is user survey month! | 15:09 |
TheJulia | https://www.openstack.org/user-survey/survey-2022/ | 15:09 |
TheJulia | Closes August 31st. | 15:09 |
rpittau | maybe have a (yet another) doodle on the dates for the pre-ptg meeting? | 15:09 |
iurygregory | #info It is user survey month! | 15:10 |
iurygregory | #link https://www.openstack.org/user-survey/survey-2022/ | 15:10 |
iurygregory | rpittau, yeah I will send another one for that | 15:10 |
iurygregory | for some reason I had the feeling the prices for the PTG tickets would increase in begin of August.. but is till middle of August =) | 15:11 |
TheJulia | Yeah, it is the middle of the month | 15:11 |
*** melwitt_ is now known as melwitt | 15:11 | |
iurygregory | ok, moving on o/ | 15:11 |
iurygregory | #topic Review subteam status reports | 15:12 |
iurygregory | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:12 |
iurygregory | starting around L88 | 15:12 |
TheJulia | we need to scrub our bugs | 15:15 |
dtantsur | *nod* | 15:15 |
dtantsur | just yesterday somebody complained that our bug tracker looks abandoned | 15:16 |
TheJulia | heh | 15:16 |
iurygregory | hjensas, fyi some patches from networking-baremetal could receive release notes =) | 15:16 |
TheJulia | Well, some people also haven't been replying when we ask for details | 15:16 |
dtantsur | honestly.. storyboard is not making life easier | 15:16 |
iurygregory | dtantsur, yesterday "sunday" ? .-. | 15:16 |
TheJulia | Agreed | 15:16 |
dtantsur | iurygregory: that chat I'm in is active 24/7 :) | 15:16 |
TheJulia | do we need a ptg topic? | 15:16 |
dtantsur | maybe? | 15:16 |
iurygregory | dtantsur, gotcha | 15:16 |
hjensas | iurygregory: ack, a separate reno patch makes sense. | 15:17 |
iurygregory | maybe we also need a "review jam" | 15:18 |
iurygregory | hjensas, ++ | 15:18 |
iurygregory | more like a "bugs jam" LOL =P | 15:18 |
rpittau | mmm jam.... | 15:19 |
rpittau | having one or more sessions where we try to go through some bugs would probably be useful | 15:19 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Finally remove support for netboot and the boot_option capability https://review.opendev.org/c/openstack/ironic/+/850240 | 15:20 |
dtantsur | kaboom! | 15:20 |
dtantsur | 39 files changed, 170 insertions(+), 1577 deletions(-) | 15:20 |
iurygregory | dtantsur, https://media.giphy.com/media/oe33xf3B50fsc/giphy.gif | 15:21 |
rpittau | do we want to release before that ? | 15:21 |
dtantsur | rpittau: I was thinking about making this release The Breaking One | 15:21 |
dtantsur | especially since we already merged trusted boot removal | 15:21 |
TheJulia | rpittau: even if we were to, it would be 2 major releases since we've already merged a breaking chagne afaik | 15:21 |
rpittau | sounds good to me! | 15:21 |
TheJulia | I mean, it is not unheard of us to do so... but yeah | 15:22 |
iurygregory | yeah, we did last year I think | 15:22 |
iurygregory | or it was in 2020 | 15:22 |
iurygregory | ok moving on | 15:22 |
iurygregory | #topic Deciding on priorities for the coming week | 15:23 |
iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:23 |
TheJulia | I added one already, there is a list ordering inconsistency on the ilo hw type | 15:23 |
ajya | can this patch be added https://review.opendev.org/c/openstack/sushy/+/850899 ? | 15:23 |
dtantsur | well, I've just proposed my candidate :) | 15:23 |
dtantsur | ajya++ | 15:24 |
dtantsur | also https://review.opendev.org/c/openstack/sushy/+/851392 | 15:24 |
iurygregory | ajya, sure | 15:24 |
dtantsur | mmm and https://review.opendev.org/c/openstack/ironic-lib/+/848966 if we don't want to break ironic-lib CI :) | 15:24 |
ajya | thanks, done | 15:24 |
iurygregory | dtantsur, ack | 15:25 |
iurygregory | later today I will add a patch I'm working locally | 15:26 |
iurygregory | ok I think we can move on | 15:30 |
iurygregory | #topic Baremetal SIG | 15:30 |
iurygregory | I think we should send an email that we will skip the meeting in August? | 15:31 |
iurygregory | afaik arne_wiebalck_ is on PTO | 15:31 |
rpittau | yep | 15:31 |
iurygregory | ok | 15:32 |
iurygregory | #topic Open discussion | 15:32 |
iurygregory | Does anyone have something we should discuss? | 15:32 |
TheJulia | iurygregory: yeah, lets send an email | 15:32 |
TheJulia | iurygregory: if you could that would be awesome, I'm juggling a number of things and haven't really been able to cycle back to it right now | 15:33 |
iurygregory | TheJulia, will do | 15:33 |
* iurygregory just added a post-it to not forget =) | 15:33 | |
iurygregory | ok, seems like we don't have any topics for discussion today =) | 15:36 |
iurygregory | thanks everyone! | 15:36 |
TheJulia | I added bug tracking to the ptg etherpad | 15:37 |
iurygregory | TheJulia, ack | 15:37 |
iurygregory | #endmeeting | 15:38 |
opendevmeet | Meeting ended Mon Aug 1 15:38:03 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:38 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-08-01-15.00.html | 15:38 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-08-01-15.00.txt | 15:38 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-08-01-15.00.log.html | 15:38 |
dtantsur | oh, it's the "where are your releases" time of the cycle? :) | 15:38 |
iurygregory | yay | 15:38 |
iurygregory | I will take a look at it | 15:38 |
dtantsur | I wonder if it even makes sense to release bifrost? | 15:38 |
TheJulia | I just <3 the openstack release model | 15:38 |
iurygregory | thanks for the heads up dtantsur =) | 15:38 |
* TheJulia is being sarcastic | 15:38 | |
dtantsur | yup | 15:38 |
iurygregory | TheJulia, :D | 15:38 |
iurygregory | going to grab lunch, bbl | 15:40 |
TheJulia | hjensas: I'm looking at https://review.opendev.org/c/openstack/networking-baremetal/+/835324 this morning, and wondering if it would help if there was a call to go over it | 15:41 |
TheJulia | hjensas: and you need some renos :) | 15:44 |
hjensas | TheJulia: thanks for the reviews so far. I'd be happy to have call. | 15:50 |
TheJulia | I just want the recheck to return results yet because I think that may provide additional insight | 15:53 |
TheJulia | iurygregory: so... we have a feature backport request... | 15:53 |
TheJulia | iurygregory: https://review.opendev.org/c/openstack/ironic/+/850554/ | 15:54 |
dtantsur | that better have some good justification | 15:55 |
TheJulia | yeeeaaahh... I'm not sure how that makes operator's lives better which is our feature backport loophole | 15:56 |
dtantsur | looks invasive to me. I wonder if a smaller subset can be extracted for a backport | 15:57 |
TheJulia | I think it looks mostly invasive since it is doing a big chunk of logic change for setting handling in the common.py file | 15:58 |
TheJulia | and then ultimately is a bit invasive | 15:58 |
TheJulia | look at the conf.py | 15:59 |
TheJulia | I wonder if we should get them to reframe this as an unfortunately invasive bug fix | 16:03 |
dtantsur | I wonder if a fix is possible without deprecations, option changes, etc | 16:03 |
TheJulia | well, new options are fine, the deprecation... I guess is the huge problem | 16:05 |
iurygregory | TheJulia, interesting... | 16:11 |
rpittau | good night! o/ | 16:11 |
iurygregory | the bump in drivers-requirements can be problematic | 16:11 |
iurygregory | maybe not directly request >=0.11.4 | 16:12 |
iurygregory | but limiting to < 0.12 could be ok | 16:12 |
iurygregory | "deprecated_for_removal" in the conf/irmc.py..... | 16:13 |
iurygregory | I don't see "irmc_snmp_user" conf | 16:14 |
iurygregory | yoga, xena and wallaby? WOW | 16:16 |
dtantsur | oh, driver-requirements? missed that | 16:18 |
iurygregory | dtantsur, yeah... | 16:18 |
iurygregory | I don't think we can just bump from 0.8.0 (min) to 0.11.4... | 16:19 |
dtantsur | yeah, not ideal | 16:19 |
iurygregory | specially when they want this till wallaby... | 16:20 |
dtantsur | I can easily imagine some downstream only packaging 0.8.0 | 16:20 |
dtantsur | (I wonder what RDO has) | 16:20 |
iurygregory | exactly | 16:20 |
TheJulia | 0.8.1 was *old* 3 years ago | 16:20 |
TheJulia | 0.10 is more recent, 1 year wallaby | 16:20 |
dtantsur | yeah, but 0.11.4 is very new, if I read it correctly | 16:21 |
TheJulia | yeah | 16:21 |
TheJulia | so... They tried to make it 0.12.x | 16:22 |
TheJulia | and I noticed 0.12.0 had a breaking change | 16:22 |
dtantsur | July 29th. it's 3 days old. | 16:22 |
dtantsur | We can be 100% confident that no downstreams have it for stable branches. | 16:22 |
TheJulia | yup | 16:22 |
iurygregory | I would be ok with <0.12 in the driver-requirements | 16:22 |
TheJulia | even on automated pipelines... that takes time | 16:22 |
TheJulia | hmm | 16:23 |
iurygregory | but directly bumping the min version I don't think is ideal.. | 16:23 |
dtantsur | >=0.11.4 is simply a no go | 16:23 |
TheJulia | I think that is reasonable to ask them to revise it | 16:23 |
TheJulia | since it is a gray area almost bugfix backport | 16:23 |
dtantsur | I could be convinced about >=0.11.1 or >=0.10.0, which are from last year... but not a 3 day old release | 16:23 |
TheJulia | just snmpv3 security stuffs must not have been grokked back then | 16:24 |
dtantsur | right | 16:24 |
TheJulia | Okay, i'll come upw ith some words, but It would be helpful if others commented on it as well | 16:24 |
iurygregory | I do understand that this is necessary to make it work when FIPS is enabled | 16:24 |
TheJulia | yeah | 16:24 |
iurygregory | will add to my list for review | 16:25 |
dtantsur | also it should not have been a patch version on their side | 16:25 |
dtantsur | e.g. https://opendev.org/x/python-scciclient/commit/04bb48cc814b6bdf8f9dc118722b2b8a398c5a6f seems breaking at first glance (removed the "community" argument) | 16:25 |
dtantsur | anyway, calling it a day, see you tomorrow o/ | 16:26 |
iurygregory | bye dtantsur | 16:26 |
TheJulia | so yeah | 16:26 |
TheJulia | they need to make the patch version aware | 16:26 |
TheJulia | to navigate it | 16:27 |
opendevreview | Merged openstack/sushy master: Handle mismatch between registry Identity and the way it's referred to https://review.opendev.org/c/openstack/sushy/+/851392 | 16:29 |
TheJulia | so even community is not *really* breaking in their use | 16:41 |
TheJulia | they didn't change the order and didn't kwarg it | 16:41 |
TheJulia | previously | 16:41 |
opendevreview | Julia Kreger proposed openstack/ironic master: Fix ilo boot interface order https://review.opendev.org/c/openstack/ironic/+/851039 | 18:03 |
opendevreview | Merged openstack/ironic master: Add audit middleware options to ironic.conf https://review.opendev.org/c/openstack/ironic/+/848978 | 18:30 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Anaconda CI changes https://review.opendev.org/c/openstack/ironic/+/849587 | 19:12 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Anaconda CI changes https://review.opendev.org/c/openstack/ironic/+/849587 | 19:12 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 19:15 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 19:57 |
opendevreview | Sergey Skripnick proposed openstack/sushy-tools master: List network interfaces of all types https://review.opendev.org/c/openstack/sushy-tools/+/844284 | 20:13 |
opendevreview | Sergey Skripnick proposed openstack/sushy-tools master: Add Chassis to ServiceRoot https://review.opendev.org/c/openstack/sushy-tools/+/844126 | 20:15 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 20:38 |
TheJulia | iurygregory: seems like we're not going to be able to fix ironic-ui to ship it. | 21:14 |
* iurygregory looks at open patches | 21:15 | |
TheJulia | there is just no way | 21:15 |
iurygregory | squashing some patches wouldn't help? | 21:16 |
iurygregory | just wondering... | 21:16 |
iurygregory | only horizon-nodejs16-run-test is broken ? | 21:18 |
TheJulia | We would need to start rewriting stuff to support the new version of angular | 21:18 |
TheJulia | and that would break the UI webdriver | 21:19 |
TheJulia | which means we would have to possibly re-write tests as well | 21:19 |
iurygregory | hummmm | 21:20 |
iurygregory | which version of angular we need to install? | 21:23 |
TheJulia | I don't remember, there is a thread ~2 months back on horizon plugins needing to update angualr | 21:23 |
iurygregory | ok,going to check that | 21:24 |
*** mushrushu_ is now known as mushrushu | 21:45 | |
hjensas | TheJulia: list_opt() is used only by oslo-config-generator right? | 22:12 |
opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Device management driver iface https://review.opendev.org/c/openstack/networking-baremetal/+/835170 | 23:27 |
TheJulia | hjensas: ... I *think* so | 23:32 |
opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 23:43 |
TheJulia | dtantsur: do you happen to remember the place where we can put in email addresses for rdo package maintainers? | 23:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!