opendevreview | Merged openstack/metalsmith stable/xena: metalsmith_instances module, unprovision by hostname https://review.opendev.org/c/openstack/metalsmith/+/847102 | 00:22 |
---|---|---|
opendevreview | Merged openstack/metalsmith stable/yoga: CI: Extend timeouts for RAX https://review.opendev.org/c/openstack/metalsmith/+/847446 | 00:46 |
opendevreview | Merged openstack/ironic master: Drop python2 from bindep.txt https://review.opendev.org/c/openstack/ironic/+/845374 | 01:39 |
opendevreview | Merged openstack/ironic master: No deploy_kernel/ramdisk with the ramdisk deploy and no cleaning https://review.opendev.org/c/openstack/ironic/+/847388 | 01:48 |
opendevreview | Merged openstack/ironic bugfix/19.0: deploy_utils: only check glance for image properties kernel/ramdisk https://review.opendev.org/c/openstack/ironic/+/844473 | 01:53 |
opendevreview | Iury 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/+/847657 | 01:56 |
opendevreview | Merged openstack/sushy-tools master: aarch64/arm64 libvirt VMs should default to SCSI controller https://review.opendev.org/c/openstack/sushy-tools/+/842385 | 03:14 |
arne_wiebalck | Good morning, Ironic! | 06:13 |
janders | good morning arne_wiebalck and Ironic o/ | 07:20 |
arne_wiebalck | hey janders o/ | 07:50 |
rpittau | good morning ironic! o/ | 07:53 |
dmellado | morning ;) | 08:05 |
*** vkmc_ is now known as vkmc | 09:10 | |
opendevreview | Mark Goddard proposed openstack/ironic-python-agent-builder master: Add smartmontools package to DIB image https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/847754 | 09:27 |
opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.3: Remove swift stable/yoga pin https://review.opendev.org/c/openstack/ironic-python-agent/+/847552 | 10:41 |
iurygregory | good morning Ironic! | 10:46 |
janders | hey iurygregory o/ | 10:51 |
opendevreview | Harald Jensås proposed openstack/ironic-lib master: Remove python version 2 pkgs from bindep https://review.opendev.org/c/openstack/ironic-lib/+/830319 | 11:04 |
janders | see you tomorrow Ironic o/ | 12:08 |
rpittau | bye janders o/ | 12:09 |
opendevreview | Merged openstack/ironic-python-agent master: Warn when smartctl not found https://review.opendev.org/c/openstack/ironic-python-agent/+/847591 | 12:10 |
opendevreview | Iury 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/+/847661 | 12:11 |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic bugfix/20.2: Configure CI for bugfix/20.2 https://review.opendev.org/c/openstack/ironic/+/847777 | 12:40 |
TheJulia | I think I might be a zombie this morning | 13:09 |
TheJulia | Good morning everyone! | 13:09 |
iurygregory | good morning TheJulia =) | 13:22 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Discover what is going on with dhcp https://review.opendev.org/c/openstack/ironic/+/847601 | 13:23 |
TheJulia | I wonder if we just need to set network manager to configure and quit... | 13:29 |
opendevreview | Merged openstack/ironic master: CI: Default to TinyIPA when nested virt is not possible https://review.opendev.org/c/openstack/ironic/+/847412 | 13:44 |
rpittau | iurygregory, 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 |
opendevreview | Riccardo 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/+/847158 | 14:21 |
rpittau | also this please ^ :) | 14:22 |
rpittau | oh yeah and we probably need this first https://review.opendev.org/c/openstack/ironic/+/847550 :D | 14:22 |
TheJulia | so what is the exact order since they are not dependency chained? | 14:27 |
rpittau | mmm 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 |
iurygregory | rpittau, looking now | 14:28 |
opendevreview | Julia Kreger proposed openstack/ironic stable/yoga: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847663 | 14:28 |
opendevreview | Julia Kreger proposed openstack/ironic stable/xena: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847664 | 14:29 |
opendevreview | Julia Kreger proposed openstack/ironic stable/wallaby: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847665 | 14:29 |
iurygregory | rpittau, I will keep an eye on all patches since you need https://review.opendev.org/c/openstack/ironic/+/847550 first because of the Depends-On | 14:30 |
rpittau | yep | 14:30 |
opendevreview | Julia Kreger proposed openstack/ironic stable/victoria: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847666 | 14:34 |
opendevreview | Julia Kreger proposed openstack/ironic stable/ussuri: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847667 | 14:34 |
TheJulia | Well, can't take it to train, but that is better than nothing | 14:34 |
iurygregory | we probably want that in some bugfix :D branches | 14:37 |
rpittau | TheJulia: we probably need that backported to at least 3 bugfix branches, I can take care of that | 14:38 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/20.2: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847661 | 14:38 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847669 | 14:39 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847810 | 14:39 |
rpittau | that should be all | 14:39 |
iurygregory | rpittau, https://review.opendev.org/c/openstack/ironic/+/847777 green \o/ | 14:40 |
rpittau | mmm it failed one job :/ | 14:41 |
iurygregory | woot | 14:41 |
iurygregory | ironic-tempest-partition-bios-ipmi-pxe https://zuul.opendev.org/t/openstack/build/f2bbb458c601440c84b76a51746743d3 : FAILURE in 1h 57m 30s | 14:42 |
iurygregory | ofc... | 14:42 |
iurygregory | maybe we should merge https://review.opendev.org/c/openstack/ironic/+/847661 | 14:42 |
rpittau | yeah let's wait for that too | 14:43 |
TheJulia | I *highly* advise so | 14:43 |
rpittau | let's merge all of them | 14:43 |
iurygregory | rpittau, lol why there is a patch set #2? :D | 14:44 |
rpittau | where ? | 14:44 |
iurygregory | https://review.opendev.org/c/openstack/ironic/+/847661 | 14:45 |
iurygregory | "Uploaded patch set 2." | 14:45 |
rpittau | errrr cause I thought it was not present in that branch | 14:45 |
iurygregory | ops | 14:46 |
rpittau | melius est abundare quam deficere | 14:46 |
TheJulia | heh | 14:47 |
TheJulia | is it bad that I could basically read that? | 14:47 |
rpittau | it's actually very good :) | 14:47 |
* TheJulia runs it through a translator to make sure | 14:47 | |
rpittau | :D | 14:47 |
TheJulia | so... no, I did not get it right | 14:47 |
TheJulia | well, vocabulary is my issue basically | 14:48 |
rpittau | we may need to pin neutron and maybe other projects in other bugfix branches | 14:51 |
TheJulia | è meglio esaurirsi che svanire | 14:52 |
* TheJulia feels a need to go watch Highlander | 14:52 | |
rpittau | :D | 14:52 |
TheJulia | https://i.imgflip.com/11lsar.jpg | 14:54 |
rpittau | lol | 14:55 |
iurygregory | #startmeeting ironic | 15:00 |
opendevmeet | Meeting 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 |
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 |
iurygregory | Hello everyone o/ | 15:00 |
rpittau | o/ | 15:00 |
rpioso | \o | 15:00 |
iurygregory | welcome to our weekly meeting :D | 15:00 |
TheJulia | o/ | 15:01 |
ajya | o/ | 15:01 |
iurygregory | we don't have a big agenda, so it will probably be a quick meeting (and we don't have quorum it seems =) ) | 15:01 |
iurygregory | you can find the agenda in the wiki | 15:01 |
iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
iurygregory | #topic Announcements / Reminder | 15:01 |
iurygregory | #info ML thread - Thoughts on closing some branches | 15:02 |
iurygregory | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-June/029274.html | 15:02 |
iurygregory | today I've sent this email to the ML, feel free to provide feedback =) | 15:02 |
TheJulia | sounds good to me | 15:02 |
iurygregory | I | 15:02 |
iurygregory | I'm planing in open the request next monday | 15:03 |
TheJulia | k | 15:03 |
iurygregory | if no objections on the ML | 15:03 |
TheJulia | I may have patches that would need to be abandoned, that is fine | 15:03 |
iurygregory | we can try to get them merged I would say I will make a list of open patches in all branches | 15:03 |
iurygregory | not sure if CI will help (it will depend on the branch) | 15:04 |
iurygregory | Does anyone have anything that would like to add to Announcements / Reminder ? | 15:04 |
TheJulia | uhh | 15:04 |
TheJulia | Likely 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 |
stendulker | o/ | 15:06 |
iurygregory | CI at least for master was in a better shape after your patches TheJulia (so I didn't mention) | 15:06 |
iurygregory | re cs9 I wasn't aware of that, I will add to the etherpad in case is not there yet | 15:06 |
TheJulia | iurygregory: go ahead and add it please | 15:07 |
iurygregory | will do =) | 15:07 |
TheJulia | it is the conenctivity issues from what I can tell | 15:07 |
TheJulia | I'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 something | 15:07 |
TheJulia | actually, I should just ask infra to hold a job | 15:08 |
iurygregory | ack | 15:08 |
* TheJulia will do that after the meeting | 15:08 | |
iurygregory | ok, moving on | 15:09 |
iurygregory | we don't have action items, skipping | 15:10 |
iurygregory | #topic Review subteam status reports | 15:10 |
iurygregory | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:10 |
iurygregory | starting on L75 | 15:11 |
opendevreview | Merged openstack/ironic bugfix/20.2: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847661 | 15:12 |
TheJulia | \o/ | 15:12 |
iurygregory | *magic* | 15:13 |
iurygregory | :D | 15:13 |
iurygregory | not much to add in the status, I think we can move on | 15:15 |
iurygregory | #topic Deciding on priorities for the coming week | 15:16 |
iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:16 |
TheJulia | I'd kind of like https://review.opendev.org/c/openstack/ironic-python-agent/+/839084 to get some eyes | 15:17 |
TheJulia | but it is not high on my priority list (in other words, I just want to get it off my plate) | 15:17 |
iurygregory | np from me | 15:18 |
iurygregory | I've added some patches to fix CI in our bugfix/20.2 for ironic | 15:18 |
TheJulia | iurygregory: ack | 15:19 |
iurygregory | since we need to get jobs running on it | 15:19 |
iurygregory | and we also need https://review.opendev.org/c/openstack/ironic/+/847657 | 15:19 |
iurygregory | hopefully CI will be happy :D | 15:20 |
iurygregory | it's already a backport so I added to the list =) | 15:20 |
iurygregory | anyone 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 |
rpittau | I think I added a couple already | 15:22 |
rpittau | mainly Ci fixes | 15:22 |
iurygregory | rpittau, tks! | 15:23 |
rpittau | oh yeah I will add one more as it's just missing approval https://review.opendev.org/c/openstack/ironic-python-agent/+/775391 | 15:23 |
rpittau | done, thanks | 15:24 |
TheJulia | fixed ;) | 15:24 |
iurygregory | ok, moving on o/ | 15:25 |
rpittau | ah thanks! :) | 15:25 |
iurygregory | #topic Baremetal SIG | 15:25 |
iurygregory | arne is not around today | 15:25 |
iurygregory | TheJulia, do you have something for the SIG? | 15:25 |
TheJulia | I don't *really*, at least that I've not had a chance to talk with arne_wiebalck about | 15:25 |
TheJulia | I migh thave a lead on a talk or three or this fall, maybe starting in August | 15:26 |
iurygregory | nice =) | 15:26 |
iurygregory | skipping RFE review since we don't have any in the agenda | 15:26 |
iurygregory | #topic Open discussion | 15:26 |
iurygregory | do we have something we would like to discuss? | 15:27 |
TheJulia | I guess I have two items | 15:27 |
iurygregory | go ahead =) | 15:27 |
TheJulia | The 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 access | 15:27 |
TheJulia | but we don't have that encoded in our policies | 15:27 |
TheJulia | and I'm curious if anyone would object if we did that, *and* captured the project ID so all resources they create are inherently "owned" outright | 15:28 |
TheJulia | its kind of another slice of access/usage, but could save us some headache | 15:28 |
TheJulia | At least in the OpenStack context | 15:28 |
TheJulia | any concerns/objections? | 15:29 |
iurygregory | after reading "could save us some headache" I'm +1 to it | 15:30 |
TheJulia | k | 15:30 |
rpittau | no objections, that does make sense IMHO | 15:30 |
iurygregory | did they update the docs or something about this change in RBAC? | 15:30 |
TheJulia | iurygregory: passed an updating resolution | 15:30 |
TheJulia | after we did our work | 15:30 |
iurygregory | TheJulia, got it | 15:30 |
iurygregory | and since is a TC decision (only if we have very strong objections we wouldn't do that I would say...) | 15:31 |
TheJulia | So, 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 |
iurygregory | ofc 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 |
TheJulia | iurygregory: well, tc decisions are basically non-binding, fwiw | 15:32 |
TheJulia | they are recommendations with potential threat of non-compliance of being booted from openstack | 15:32 |
TheJulia | Going back to vision, I guess it is time to revisit overall | 15:33 |
iurygregory | re-vision ++ from, I was planning in write an email about this (since we will probably need to reach out our contributors ) | 15:34 |
TheJulia | Re, the change, that is a good point, I wonder how it would impact arne_wiebalck’s environment | 15:34 |
TheJulia | I 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 going | 15:35 |
iurygregory | ++ | 15:36 |
iurygregory | probably bring up in the next meeting (see if we have more folks that would engage in the discussion) | 15:37 |
TheJulia | ++ | 15:37 |
iurygregory | and after that maybe start a thread in the openstack-discuss | 15:37 |
TheJulia | ++ | 15:38 |
TheJulia | sounds good to me | 15:38 |
iurygregory | cool | 15:39 |
iurygregory | I think that's all for open discussion right TheJulia ? | 15:40 |
TheJulia | yup, all from me | 15:40 |
iurygregory | ok o/ | 15:40 |
iurygregory | #topic Who is going to run the next meeting? | 15:41 |
iurygregory | Do we have any volunteer? | 15:41 |
iurygregory | I will run the next meeting o/ | 15:42 |
iurygregory | Thanks everyone! | 15:42 |
iurygregory | #endmeeting | 15:42 |
opendevmeet | Meeting ended Mon Jun 27 15:42:51 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:42 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-06-27-15.00.html | 15:42 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-06-27-15.00.txt | 15:42 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-06-27-15.00.log.html | 15:42 |
rpittau | bye everyone, see you alter this week or next week o/ | 16:03 |
iurygregory | bye rpittau o/ | 16:17 |
opendevreview | Merged openstack/ironic-inspector master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/ironic-inspector/+/840075 | 16:21 |
opendevreview | Julia Kreger proposed openstack/ironic master: DNM: Discover what is going on with dhcp https://review.opendev.org/c/openstack/ironic/+/847601 | 16:30 |
opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Use json for lsblk output https://review.opendev.org/c/openstack/ironic-python-agent/+/775391 | 17:10 |
*** Guest3279 is now known as rcastillo | 17:20 | |
opendevreview | Merged openstack/ironic stable/yoga: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847663 | 19:02 |
opendevreview | Merged openstack/ironic stable/wallaby: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847665 | 19:02 |
opendevreview | Merged openstack/ironic stable/xena: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847664 | 19:02 |
iurygregory | If 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 ironic | 21:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!