Monday, 2022-06-27

opendevreviewMerged openstack/metalsmith stable/xena: metalsmith_instances module, unprovision by hostname  https://review.opendev.org/c/openstack/metalsmith/+/84710200:22
opendevreviewMerged openstack/metalsmith stable/yoga: CI: Extend timeouts for RAX  https://review.opendev.org/c/openstack/metalsmith/+/84744600:46
opendevreviewMerged openstack/ironic master: Drop python2 from bindep.txt  https://review.opendev.org/c/openstack/ironic/+/84537401:39
opendevreviewMerged openstack/ironic master: No deploy_kernel/ramdisk with the ramdisk deploy and no cleaning  https://review.opendev.org/c/openstack/ironic/+/84738801:48
opendevreviewMerged openstack/ironic bugfix/19.0: deploy_utils: only check glance for image properties kernel/ramdisk  https://review.opendev.org/c/openstack/ironic/+/84447301:53
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic bugfix/20.2: No deploy_kernel/ramdisk with the ramdisk deploy and no cleaning  https://review.opendev.org/c/openstack/ironic/+/84765701:56
opendevreviewMerged openstack/sushy-tools master: aarch64/arm64 libvirt VMs should default to SCSI controller  https://review.opendev.org/c/openstack/sushy-tools/+/84238503:14
arne_wiebalckGood morning, Ironic!06:13
jandersgood morning arne_wiebalck and Ironic o/07:20
arne_wiebalckhey janders o/07:50
rpittaugood morning ironic! o/07:53
dmelladomorning ;)08:05
*** vkmc_ is now known as vkmc09:10
opendevreviewMark Goddard proposed openstack/ironic-python-agent-builder master: Add smartmontools package to DIB image  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/84775409:27
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent bugfix/8.3: Remove swift stable/yoga pin  https://review.opendev.org/c/openstack/ironic-python-agent/+/84755210:41
iurygregorygood morning Ironic!10:46
jandershey iurygregory o/10:51
opendevreviewHarald Jensås proposed openstack/ironic-lib master: Remove python version 2 pkgs from bindep  https://review.opendev.org/c/openstack/ironic-lib/+/83031911:04
janderssee you tomorrow Ironic o/12:08
rpittaubye janders o/12:09
opendevreviewMerged openstack/ironic-python-agent master: Warn when smartctl not found  https://review.opendev.org/c/openstack/ironic-python-agent/+/84759112:10
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic bugfix/20.2: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766112:11
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic bugfix/20.2: Configure CI for bugfix/20.2  https://review.opendev.org/c/openstack/ironic/+/84777712:40
TheJuliaI think I might be a zombie this morning13:09
TheJuliaGood morning everyone!13:09
iurygregorygood morning TheJulia =)13:22
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: Discover what is going on with dhcp  https://review.opendev.org/c/openstack/ironic/+/84760113:23
TheJuliaI wonder if we just need to set network manager to configure and quit...13:29
opendevreviewMerged openstack/ironic master: CI: Default to TinyIPA when nested virt is not possible  https://review.opendev.org/c/openstack/ironic/+/84741213:44
rpittauiurygregory, TheJulia, when you have a moment can you please check the fix for ipa bugfix/8.3 CI https://review.opendev.org/c/openstack/ironic-python-agent/+/847552/1 ? thanks!14:20
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent bugfix/8.3: Fix discovering WWN/serial for devicemapper devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/84715814:21
rpittaualso this please ^ :)14:22
rpittauoh yeah and we probably need this first https://review.opendev.org/c/openstack/ironic/+/847550 :D14:22
TheJuliaso what is the exact order since they are not dependency chained?14:27
rpittaummm they should be, there's a Depends-On in the first for ipa, anyway the first one is https://review.opendev.org/c/openstack/ironic/+/847550 but I jsut realized it failed an arm64 unit test run.....14:28
iurygregoryrpittau, looking now14:28
opendevreviewJulia Kreger proposed openstack/ironic stable/yoga: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766314:28
opendevreviewJulia Kreger proposed openstack/ironic stable/xena: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766414:29
opendevreviewJulia Kreger proposed openstack/ironic stable/wallaby: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766514:29
iurygregoryrpittau, I will keep an eye on all patches since you need https://review.opendev.org/c/openstack/ironic/+/847550 first because of the Depends-On14:30
rpittauyep14:30
opendevreviewJulia Kreger proposed openstack/ironic stable/victoria: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766614:34
opendevreviewJulia Kreger proposed openstack/ironic stable/ussuri: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766714:34
TheJuliaWell, can't take it to train, but that is better than nothing14:34
iurygregorywe probably want that in some bugfix :D branches 14:37
rpittauTheJulia: we probably need that backported to at least 3 bugfix branches, I can take care of that14:38
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/20.2: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766114:38
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/19.0: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766914:39
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/18.1: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84781014:39
rpittauthat should be all14:39
iurygregoryrpittau, https://review.opendev.org/c/openstack/ironic/+/847777 green \o/14:40
rpittaummm it failed one job :/14:41
iurygregorywoot14:41
iurygregoryironic-tempest-partition-bios-ipmi-pxe https://zuul.opendev.org/t/openstack/build/f2bbb458c601440c84b76a51746743d3 : FAILURE in 1h 57m 30s14:42
iurygregoryofc...14:42
iurygregorymaybe we should merge https://review.opendev.org/c/openstack/ironic/+/84766114:42
rpittauyeah let's wait for that too14:43
TheJuliaI *highly* advise so14:43
rpittaulet's merge all of them14:43
iurygregoryrpittau, lol why there is a patch set #2? :D14:44
rpittauwhere ?14:44
iurygregoryhttps://review.opendev.org/c/openstack/ironic/+/84766114:45
iurygregory"Uploaded patch set 2."14:45
rpittauerrrr cause I thought it was not present in that branch14:45
iurygregoryops14:46
rpittaumelius est abundare quam deficere14:46
TheJuliaheh14:47
TheJuliais it bad that I could basically read that?14:47
rpittauit's actually very good :)14:47
* TheJulia runs it through a translator to make sure14:47
rpittau:D14:47
TheJuliaso... no, I did not get it right14:47
TheJuliawell, vocabulary is my issue basically14:48
rpittauwe may need to pin neutron and maybe other projects in other bugfix branches14:51
TheJuliaè meglio esaurirsi che svanire14:52
* TheJulia feels a need to go watch Highlander14:52
rpittau:D14:52
TheJuliahttps://i.imgflip.com/11lsar.jpg14:54
rpittaulol14:55
iurygregory#startmeeting ironic15:00
opendevmeetMeeting started Mon Jun 27 15:00:16 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'ironic'15:00
TheJuliao/15:00
iurygregoryHello everyone o/15:00
rpittauo/15:00
rpioso\o15:00
iurygregorywelcome to our weekly meeting :D15:00
TheJuliao/15:01
ajyao/15:01
iurygregorywe don't have a big agenda, so it will probably be a quick meeting (and we don't have quorum it seems =) )15:01
iurygregoryyou can find the agenda in the wiki15:01
iurygregory#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
iurygregory#topic Announcements / Reminder 15:01
iurygregory#info ML thread - Thoughts on closing some branches15:02
iurygregory#link http://lists.openstack.org/pipermail/openstack-discuss/2022-June/029274.html15:02
iurygregorytoday I've sent this email to the ML, feel free to provide feedback =)15:02
TheJuliasounds good to me15:02
iurygregoryI15:02
iurygregoryI'm planing in open the request next monday15:03
TheJuliak15:03
iurygregoryif no objections on the ML15:03
TheJuliaI may have patches that would need to be abandoned, that is fine15:03
iurygregorywe can try to get them merged I would say I will make a list of open patches in all branches 15:03
iurygregorynot sure if CI will help (it will depend on the branch)15:04
iurygregoryDoes anyone have anything that would like to add to Announcements / Reminder ?15:04
TheJuliauhh15:04
TheJuliaLikely worth mentioning we're having a lot of issues with CI. In particular emulated VM performance which we have some "fixes" for, as well as issues with NetworkManager on centos9 not honoring dhcp properly.15:05
stendulkero/15:06
iurygregoryCI at least for master was in a better shape after your patches TheJulia (so I didn't mention)15:06
iurygregoryre cs9 I wasn't aware of that, I will add to the etherpad in case is not there yet15:06
TheJuliaiurygregory: go ahead and add it please15:07
iurygregorywill do =)15:07
TheJuliait is the conenctivity issues from what I can tell15:07
TheJuliaI'm trying to get more info out of it but all the tricks dont' seem to work, so I might have to reproduce locally or something15:07
TheJuliaactually, I should just ask infra to hold a job15:08
iurygregoryack15:08
* TheJulia will do that after the meeting15:08
iurygregoryok, moving on15:09
iurygregorywe don't have action items, skipping15:10
iurygregory#topic Review subteam status reports15:10
iurygregory#link https://etherpad.opendev.org/p/IronicWhiteBoard15:10
iurygregorystarting on L7515:11
opendevreviewMerged openstack/ironic bugfix/20.2: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766115:12
TheJulia\o/15:12
iurygregory*magic*15:13
iurygregory:D15:13
iurygregorynot much to add in the status, I think we can move on15:15
iurygregory#topic Deciding on priorities for the coming week15:16
iurygregory#link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio15:16
TheJuliaI'd kind of like https://review.opendev.org/c/openstack/ironic-python-agent/+/839084 to get some eyes15:17
TheJuliabut it is not high on my priority list (in other words, I just want to get it off my plate)15:17
iurygregorynp from me15:18
iurygregoryI've added some patches to fix CI in our bugfix/20.2 for ironic15:18
TheJuliaiurygregory: ack15:19
iurygregorysince we need to get jobs running on it15:19
iurygregoryand we also need https://review.opendev.org/c/openstack/ironic/+/84765715:19
iurygregoryhopefully CI will be happy :D15:20
iurygregoryit's already a backport so I added to the list =)15:20
iurygregoryanyone have more patches that needs review? I know we have some patches that needs attention already (but since CI was in a bad shape we had to fix it first)15:22
rpittauI think I added a couple already15:22
rpittaumainly Ci fixes15:22
iurygregoryrpittau, tks!15:23
rpittauoh yeah I will add one more as it's just missing approval https://review.opendev.org/c/openstack/ironic-python-agent/+/77539115:23
rpittaudone, thanks15:24
TheJuliafixed ;)15:24
iurygregoryok, moving on o/15:25
rpittauah thanks! :)15:25
iurygregory#topic Baremetal SIG 15:25
iurygregoryarne is not around today15:25
iurygregoryTheJulia, do you have something for the SIG?15:25
TheJuliaI don't *really*, at least that I've not had a chance to talk with arne_wiebalck about15:25
TheJuliaI migh thave a lead on a talk or three or this fall, maybe starting in August15:26
iurygregorynice =)15:26
iurygregoryskipping RFE review since we don't have any in the agenda15:26
iurygregory#topic Open discussion15:26
iurygregorydo we have something we would like to discuss?15:27
TheJuliaI guess I have two items15:27
iurygregorygo ahead =)15:27
TheJuliaThe first, with the RBAC work, there was the direction change from the TC on that admin is actually project scoped but system admin sort of level access15:27
TheJuliabut we don't have that encoded in our policies15:27
TheJuliaand I'm curious if anyone would object if we did that, *and* captured the project ID so all resources they create are inherently "owned" outright15:28
TheJuliaits kind of another slice of access/usage, but could save us some headache15:28
TheJuliaAt least in the OpenStack context15:28
TheJuliaany concerns/objections?15:29
iurygregoryafter reading "could save us some headache" I'm +1 to it15:30
TheJuliak15:30
rpittauno objections, that does make sense IMHO15:30
iurygregorydid they update the docs or something about this change in RBAC?15:30
TheJuliaiurygregory: passed an updating resolution15:30
TheJuliaafter we did our work15:30
iurygregoryTheJulia, got it15:30
iurygregoryand since is a TC decision (only if we have very strong objections we wouldn't do that I would say...)15:31
TheJuliaSo, other thought which comes to mind. Have we considered kind of re-visioning our position in the OpenStack ecosystem? I'm starting to wonder if being at such a lower layer means we might be better suited to kind of "pull a zuul"...15:32
iurygregoryofc it will be good to hear the thoughts from operators etc (maybe arne_wiebalck since they are moving with RBAC if I recall)15:32
TheJuliaiurygregory: well, tc decisions are basically non-binding, fwiw15:32
TheJuliathey are recommendations with potential threat of non-compliance of being booted from openstack15:32
TheJuliaGoing back to vision, I guess it is time to revisit overall15:33
iurygregoryre-vision ++ from, I was planning in write an email about this (since we will probably need to reach out our contributors )15:34
TheJuliaRe, the change, that is a good point, I wonder how it would impact arne_wiebalck’s environment15:34
TheJuliaI think we should hold several discussions, say over the next two months to try and see where we have consensus as to where we’re at and where we’re going15:35
iurygregory++15:36
iurygregoryprobably bring up in the next meeting (see if we have more folks that would engage in the discussion)15:37
TheJulia++15:37
iurygregoryand after that maybe start a thread in the openstack-discuss15:37
TheJulia++15:38
TheJuliasounds good to me15:38
iurygregorycool15:39
iurygregoryI think that's all for open discussion right TheJulia ?15:40
TheJuliayup, all from me15:40
iurygregoryok o/15:40
iurygregory#topic Who is going to run the next meeting?15:41
iurygregoryDo we have any volunteer?15:41
iurygregoryI will run the next meeting o/15:42
iurygregoryThanks everyone!15:42
iurygregory#endmeeting15:42
opendevmeetMeeting ended Mon Jun 27 15:42:51 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:42
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-06-27-15.00.html15:42
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-06-27-15.00.txt15:42
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-06-27-15.00.log.html15:42
rpittaubye everyone, see you alter this week or next week o/16:03
iurygregorybye rpittau o/16:17
opendevreviewMerged openstack/ironic-inspector master: Drop lower-constraints.txt and its testing  https://review.opendev.org/c/openstack/ironic-inspector/+/84007516:21
opendevreviewJulia Kreger proposed openstack/ironic master: DNM: Discover what is going on with dhcp  https://review.opendev.org/c/openstack/ironic/+/84760116:30
opendevreviewVerification of a change to openstack/ironic-python-agent master failed: Use json for lsblk output  https://review.opendev.org/c/openstack/ironic-python-agent/+/77539117:10
*** Guest3279 is now known as rcastillo17:20
opendevreviewMerged openstack/ironic stable/yoga: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766319:02
opendevreviewMerged openstack/ironic stable/wallaby: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766519:02
opendevreviewMerged openstack/ironic stable/xena: CI: Add iweb to the use tinyipa on list  https://review.opendev.org/c/openstack/ironic/+/84766419:02
iurygregoryIf any core has sometime https://review.opendev.org/c/openstack/ironic/+/847777 this is to enable voting jobs in our bugfix branch 20.2 for ironic21:34

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!