Monday, 2021-11-15

*** ricolin_ is now known as ricolin04:18
hgyGood afternoon07:26
arne_wiebalckGood morning, Ironic!07:39
iurygregorygood morning arne_wiebalck and Ironic o/07:43
arne_wiebalckhey iurygregory o/07:56
iurygregoryo/07:58
rpittaugood morning ironic! o/08:12
iurygregorymorning rpittau o/08:15
rpittauhey iurygregory :)08:17
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent master: Use json for lsblk output  https://review.opendev.org/c/openstack/ironic-python-agent/+/77539108:21
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Build and publish arm64 debian based ipa ramdisk  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/81581508:21
arne_wiebalckhey rpittau, good morning! o/08:22
rpittauhey arne_wiebalck :)08:24
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent master: Call execute from ironic-lib in hardware.py  https://review.opendev.org/c/openstack/ironic-python-agent/+/81679108:27
opendevreviewVerification of a change to openstack/ironic bugfix/18.1 failed: Fix idrac-wsman deploy with existing non-BIOS jobs  https://review.opendev.org/c/openstack/ironic/+/81710708:42
opendevreviewRiccardo Pittau proposed openstack/ironic-lib master: Restore blkid compatibility with Centos 7  https://review.opendev.org/c/openstack/ironic-lib/+/81792210:18
opendevreviewMerged openstack/ironic master: Fix RedfishManagement.get_mac_addresses and related functions  https://review.opendev.org/c/openstack/ironic/+/81672610:21
opendevreviewMerged openstack/ironic-python-agent bugfix/8.1: Output verbose info from efibootmgr  https://review.opendev.org/c/openstack/ironic-python-agent/+/81746810:23
jandershey hgy_ arne_wiebalck iurygregory rpittau and Ironic o/10:24
iurygregoryjanders, o/10:24
jandersFedora upgrade question:10:24
jandersis anyone else observing weirdness where detaching a chromium browser tab makes Gnome switch to a different desktop?10:25
janders(as if the assumption was that all the disconnected tabs go to the newest chromium window)10:25
jandersso far that is the only thing that I don't like past-upgrade (and it feels soo weird)10:25
iurygregoryI don't use chromium .-.  and I never tested this XD10:25
janders:) my user satisfaction from chrom* series of products is lesser every release, so may be time to start looking at alternatives 10:26
iurygregorynormally I only use firefox, sometimes google chrome itself10:27
opendevreviewMerged openstack/ironic-python-agent bugfix/8.1: Delete EFI boot entry duplicate labels first  https://review.opendev.org/c/openstack/ironic-python-agent/+/81746910:29
dtantsurgood morning ironic10:40
* dtantsur uses neither Gnome nor Chromium10:40
rpittauhey janders, I use chrome and I never had issues with that, but I don't use gnome :/10:54
iurygregorymorning dtantsur 10:56
* iurygregory is wondering what rpittau and dtantsur are using instead of gnome ... kde? i3?10:57
dtantsurMATE10:58
dtantsurkinda Gnome 210:58
iurygregoryinteresting =)10:58
rpittaucinnamon :)10:59
iurygregorynow I'm puzzled11:00
iurygregoryXD11:00
janderscinnamon sounds tasty11:02
dtantsurso we both decided to stick with Gnome 2. I'm using its fork, rpittau - Gnome 2 rebuilt on technologies of Gnome 3.11:02
rpittauyeah 11:02
janderssounds like a good way to minimise having to cope with "change for the sake of change" issues11:03
dtantsuryeah11:03
dtantsurI'm thinking of giving Gnome 3 or recent KDE a try, but it's very low on my priority list11:03
jandersI never liked Gnome3, was just too lazy to do something about it11:03
iurygregorysince I'm planning to do a fresh install in December I might not use gnome this time =)11:04
rpittauTheJulia: about https://review.opendev.org/c/openstack/ironic-python-agent/+/817561, we didn't move tinyipa for ussuri to tinycore 11.x because of the 'kernel too old' issue as we're using bionic there for the nodeset11:10
opendevreviewMerged openstack/ironic stable/victoria: Fix idrac-wsman deploy with existing non-BIOS jobs  https://review.opendev.org/c/openstack/ironic/+/81696511:20
* arne_wiebalck missed the start of window manager discussions :-D11:20
opendevreviewMerged openstack/ironic stable/ussuri: Fix idrac-wsman deploy with existing non-BIOS jobs  https://review.opendev.org/c/openstack/ironic/+/81696611:20
opendevreviewMerged openstack/ironic stable/train: Fix idrac-wsman deploy with existing non-BIOS jobs  https://review.opendev.org/c/openstack/ironic/+/81696711:20
dtantsurarne_wiebalck: you can still share your preferences :)11:21
arne_wiebalckI moved to i3 some years ago and never looked back :)11:21
arne_wiebalckjust tired of moving windows around11:22
dtantsuryeah, fair enough11:22
dtantsurI mostly tolerate traditional managers because I basically never move windows anyway11:23
arne_wiebalckit is the initial placement, not wasting any space11:23
arne_wiebalckin the end, I guess I am just lazy ;)11:24
dtantsur:D11:24
dtantsurI used to use https://awesomewm.org/11:24
arne_wiebalcki3 and tmux are probably the tools which had the most impact on the way I work in the past years11:24
dtantsurI can imagine11:25
dtantsurI would add learning vim properly as a big performance booster11:25
arne_wiebalckI have to admit I use i3 with mouse-mode11:25
* arne_wiebalck ducks11:25
arne_wiebalckdtantsur: true11:25
arne_wiebalckdtantsur: changing topics, I'd appreciate your thoughts on https://storyboard.openstack.org/#!/story/200968011:26
dtantsurarne_wiebalck: I'd still just cargo-cult what is already done for virtual media11:36
dtantsurah, overlooked the part about weak vs strong, interesting11:36
arne_wiebalckyeah, I tried cargo-culting first :)11:37
dtantsurI suspect stripping W/ may be a bug11:37
arne_wiebalckI guess timeu put it there for a reason?11:37
arne_wiebalckI am not sure if the qualifier should be part of the If-* precondition or not.11:39
arne_wiebalckDropping the stripping would solve the issue with my hardware of course, but I guess iurygregory tested this as well with virtual media.11:40
arne_wiebalckon real hardware11:40
janderssee you tomorrow Ironic o/11:40
arne_wiebalckbye janders o/11:43
dtantsurhonestly from reading https://www.ietf.org/rfc/rfc7232.txt I cannot reliably understand if stripping W/ is the right thing11:45
dtantsurit does sound like sending weak tags in If-Match makes no sense?11:46
iurygregoryI know the etag part we have in sushy was added to handle the case of vmedia in Lenovo HW, to me it makes sense that we send the Etag we have available (not sure if we need to strip the W/)11:46
dtantsur"If a listed ETag has the W/ prefix indicating a weak entity tag, this comparison algorithm will never match it." - from https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/If-Match11:46
dtantsurarne_wiebalck: it's probably good to ask your vendor what they meant by sending a weak etag but requiring a (strong) etag back.11:48
arne_wiebalckdtantsur: they do not require a strong one back11:49
arne_wiebalckfrom what I see, they send 'W/"123"' and expect 'W/"123"' back11:49
arne_wiebalckIronic sends only the value back11:50
arne_wiebalcknot the whole etag11:50
iurygregorymaybe we can fix this and see how Lenovo HW will react?11:51
arne_wiebalckiurygregory: did you test virtual media on Lenovo HW ?11:51
iurygregoryarne_wiebalck, I don't have Lenovo HW to test11:52
arne_wiebalckhow did you test virtual media?11:52
iurygregorywasn't me who tested, but timeu =)11:52
arne_wiebalckaha :)11:52
arne_wiebalckso I guess that hardware only expects the value, maybe ?11:53
iurygregorymaybe =)11:53
iurygregorythis is why we love HW https://xkcd.com/927/ :D11:54
arne_wiebalckI think we need to a) understand if the behavior I see is standard compliant and if it is not b) if we should absorb this in sushy or not11:56
arne_wiebalck(if it is compliant, sushy needs to be patched)11:56
dtantsurarne_wiebalck: If-Match: W/"..." must NEVER match per RFC11:57
dtantsurIt's literally a "please return me HTTP 412" request :)11:57
dtantsur(see the quote I pasted above)11:58
arne_wiebalckdtantsur: oh, ok, I see what you mean:12:03
arne_wiebalckthey an etag which has a qualifier, but regard the whole thing as the etag when the requester comes back12:04
arne_wiebalck*they send12:04
dtantsuryep12:04
arne_wiebalckso, you think it is an implementation bug on their side?12:04
arne_wiebalckis it legit/expected to turn the weak one into a strong one (as Ironic does)?12:08
* arne_wiebalck grabs a quick lunch12:08
iurygregory" If a resource supports an ETag, it shall use the RFC7232-defined ETag strong validator." based on this I think they should only have strong ETag no?12:09
iurygregoryhttp://redfish.dmtf.org/schemas/DSP0266_1.11.0.html#etags12:09
opendevreviewMerged openstack/ironic bugfix/18.1: Fix idrac-wsman deploy with existing non-BIOS jobs  https://review.opendev.org/c/openstack/ironic/+/81710712:11
TheJuliaGood morning12:40
TheJuliarpittau: yeah, I'm aware of that, but I thought I'd explore a few things before I call for nuking support for tinyipa on older branches12:41
* TheJulia wipes sleep from eyes12:41
arne_wiebalckiurygregory: right12:44
arne_wiebalckiurygregory: but does that mean the resource should only have a strong etag, or requests should only send strong etags?12:44
iurygregoryarne_wiebalck, tricky question.... because they also mention that the Etag mechanism supports strong and weak validation12:45
iurygregorymraineri, can you help us understand a bit more about ETags in redfish? =)12:46
arne_wiebalckiurygregory: maybe they refer to etags in general have this12:46
arne_wiebalckmraineri: context in https://storyboard.openstack.org/#!/story/200968012:47
rpittauTheJulia: good morning! alright, let me know if I can help, we did give a try in the past to switch to focal as nodeset but there were too many things to change/backport and some were just not applicable12:50
TheJuliarpittau: I ave that a try too, and devstack config seems to detonate in some cases12:51
TheJulias/ave/gave/12:51
arne_wiebalckiurygregory: I would interpret the Redfish standard as you do: no weak Etags. This would mean: my vendor implementation is not compliant and Sushy should not strip W/, but rather raise an error when it finds W/ :)14:02
iurygregory++ =)14:02
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic-specs master: Yoga Themes  https://review.opendev.org/c/openstack/ironic-specs/+/81530814:26
opendevreviewRiccardo Pittau proposed openstack/ironic-lib master: Restore blkid compatibility with Centos 7  https://review.opendev.org/c/openstack/ironic-lib/+/81792214:53
iurygregory#startmeeting ironic14:59
opendevmeetMeeting started Mon Nov 15 14:59:59 2021 UTC and is due to finish in 60 minutes.  The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot.14:59
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:59
opendevmeetThe meeting name has been set to 'ironic'14:59
iurygregoryo/15:00
arne_wiebalcko/15:00
* iurygregory wondering if we have quorum ....15:00
iurygregorylet's give a couple of minutes to see =)15:01
rlooo/15:01
rpioso\o15:01
stendulkero/15:01
iurygregorydo we feel like we should have the meeting? (we have one topic to discuss related to CI...) 15:02
rpittauo/15:03
iurygregoryok, we can do a quick meeting if people agree =)15:04
arne_wiebalcklet's do it :)15:04
dtantsuro/15:04
iurygregoryWelcome to our weekly meeting everyone o/15:04
iurygregoryOur agenda can be found on the wiki.15:05
iurygregory#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:05
iurygregory#topic Announcements / Reminders15:05
iurygregorydoes anyone have anything to announce/remind us of?15:05
iurygregorycrickets \o/ (let's move on)15:07
iurygregoryskipping action items/subteam status15:07
dtantsurwhen do we have the next bugfix release?15:07
iurygregoryFirst week of Dec dtantsur =)15:07
iurygregoryhttps://review.opendev.org/c/openstack/ironic-specs/+/815308 15:08
iurygregory=)15:08
dtantsurcool, thanks!15:08
iurygregory#topic Deciding on priorities for the coming week15:08
iurygregoryanyone has patches that needs review?15:08
dtantsurI guess I do as always :)15:10
iurygregoryfeel free to drop them :D15:10
dtantsurhttps://review.opendev.org/c/openstack/sushy/+/815479 https://review.opendev.org/c/openstack/sushy/+/817792 https://review.opendev.org/c/openstack/ironic/+/81738615:11
dtantsurall fishy business15:11
dtantsuralso https://review.opendev.org/c/openstack/ironic/+/81672115:11
iurygregorysushi fixes :D15:11
dtantsurand https://review.opendev.org/c/openstack/sushy/+/816717 :)15:11
dtantsurI've been doing a lot of sushy recently, I admit15:11
rloois this ready for review? https://review.opendev.org/c/openstack/ironic-specs/+/815308? We should just get that done?15:11
iurygregoryI've updated with details on some topics, if anyone has feedback please provide them =) 15:13
rpittauI already added https://review.opendev.org/c/openstack/ironic-lib/+/817922 but just wanted to point the attention to that, seems like we broke ironic-lib/ipa compatibility with centos7 15:13
dtantsuriurygregory: I've just suggested where to put my name15:13
iurygregoryrpittau, oh wow!15:13
iurygregorydtantsur, cool I will update =)15:13
iurygregoryI think we can move on to discussion?15:16
iurygregory#topic Discussion15:17
iurygregory#info TinyIPA 10.x build can't connect to git.qemu.org due to ssl cert update. Do we excise tinyipa on Ussuri/Train? Do we drop Ussuri? A bit of both?15:18
opendevreviewVerification of a change to openstack/ironic master failed: [iRMC] Convert the type of irmc_port to int  https://review.opendev.org/c/openstack/ironic/+/81715415:18
dtantsurI think the build itself uses plain HTTP in some places15:18
dtantsurmaybe using --insecure won't be too terrible?</devil>15:19
iurygregoryhummmm not a bad idea if we can have CI running with tinyipa I would say15:19
iurygregoryespecially in train for downstream reasons =)15:20
iurygregoryanyone against dtantsur idea? 15:21
iurygregoryrpittau, do you have any thoughts on this? =)15:22
rpittauiurygregory: sorry, I'm on 2 meeting at the same time :/15:22
rpittauI'd rather keep ussuri or older for the time being, so I'm ok with dtantsur proposal15:22
iurygregoryrpittau, yeah (sorry I forgot the downstream meeting >.<)15:23
iurygregorycool =)15:23
iurygregory#topic Baremetal SIG15:23
iurygregoryarne_wiebalck, anything for the Baremetal SIG?15:23
arne_wiebalcknothing beyond: llet me know if you have a topic for the next session15:24
dtantsurwhen is that?15:24
arne_wiebalckotherwise I will come and look for topics :)15:24
arne_wiebalcksecond Tue in Dec15:24
iurygregory07 Dec ?15:24
dtantsur14th?15:25
arne_wiebalck7th or 14th15:25
iurygregoryoh in that case 14th15:25
arne_wiebalckbut we can adapt as well15:25
dtantsurhave we talked about metal3 already? I can prepare something if there is interest.15:25
iurygregorydtantsur, we haven't afaik =)15:25
arne_wiebalckI don't think we have, and that would be great.15:25
arne_wiebalckdtantsur: pick your preferred date.15:25
dtantsurI don't think I care. 14th just to be strictly correct re 2nd Tue?15:26
arne_wiebalcksounds good to me, thanks!15:26
iurygregorymakes sense to me 15:26
arne_wiebalckspecific topic or intro/overview?15:26
arne_wiebalckfor metal315:26
dtantsuroverview, maybe some demo?15:27
arne_wiebalckuh, nice, I like how we have demos now :)15:27
arne_wiebalcksounds great, thanks dtantsur !15:27
iurygregorytks dtantsur 15:27
iurygregorywe don't have any topics in RFE review/Open discussion so let's go to the last topic :D15:28
iurygregory#topic Who is going to run the next meeting?15:29
iurygregorydo we have a volunteer?15:29
rpittauif we move the meeting one hour later, I can do it :P15:29
iurygregorynot sure if we can :D 15:30
iurygregoryI don't mind running the next meeting15:30
iurygregoryit will be complicated in December for me :D 15:30
iurygregorytks everyone!15:30
iurygregory#endmeeting15:30
opendevmeetMeeting ended Mon Nov 15 15:30:52 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:30
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-15-14.59.html15:30
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-15-14.59.txt15:30
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-15-14.59.log.html15:30
rpittauI'm actually seriously proposing to move the meeting one hour later15:31
arne_wiebalckrpittau: this would be fine by me.15:33
rpittauafter the time change we have a conflict downstream and if I'm lucky I have 2 meetings at the same time, otherwise 3 :/15:36
muellerbehello @all15:37
iurygregoryrpittau, 3?! woot?!15:38
rpittauiurygregory: ARM :)15:38
iurygregoryoh I wasn't aware about this one =)15:38
iurygregoryhello muellerbe 15:39
muellerbe#startmeeting ironic16:01
opendevmeetMeeting started Mon Nov 15 16:01:32 2021 UTC and is due to finish in 60 minutes.  The chair is muellerbe. Information about MeetBot at http://wiki.debian.org/MeetBot.16:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:01
opendevmeetThe meeting name has been set to 'ironic'16:01
muellerbe#endmeeting 16:02
opendevmeetMeeting ended Mon Nov 15 16:02:35 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:02
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-15-16.01.html16:02
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-15-16.01.txt16:02
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-11-15-16.01.log.html16:02
muellerbesorry for starting the meeting, I cannot handle my local calendar and the timezone, #lostintimezone ;) I try to be part at next week16:12
arne_wiebalckmuellerbe: yoiu hold the record now for the shortest Ironic meeting ever :)16:14
arne_wiebalck*you16:14
dtantsur:D16:18
dtantsuryeah, this meeting is in UTC, so it shifts for everyone at some point16:18
timeuarne_wiebalck: sorry was AFK until now and saw your message only now. I vaguely remember that there were some issues with ETags and Lenovo virtual media and if I remember correctly I had to strip the \W to make it work.  16:18
arne_wiebalcktimeu: thanks, this is what I expected reading your code16:19
timeubut I would have to play around to see what exactly failed 16:19
timeuI can patch it out and see what error Lenovo HW returns16:19
arne_wiebalcktimeu: I have some other hardware now which gives me a weak etag as well, but wants the whole etag back :-S16:20
arne_wiebalcktimeu: I am trying to clarify what the standard actually allows:  https://redfishforum.com/thread/565/weak-etags16:21
muellerbearne_wiebalck, is this a good record or ... ;)16:21
arne_wiebalckmuellerbe: the best :-D16:22
timeuah great, let's see what they say o_016:24
NobodyCamGood Morning Ironic Folks16:55
arne_wiebalckhey NobodyCam o/16:58
NobodyCamhey hey Good Morning Arne!16:58
rpittauhey NobodyCam :)17:08
NobodyCamMorning rpittau :) happy monday17:09
opendevreviewMerged openstack/ironic stable/wallaby: Fix idrac-wsman deploy with existing non-BIOS jobs  https://review.opendev.org/c/openstack/ironic/+/81696417:21
opendevreviewMerged openstack/ironic stable/xena: Fix idrac-wsman deploy with existing non-BIOS jobs  https://review.opendev.org/c/openstack/ironic/+/81696317:21
opendevreviewDmitry Tantsur proposed openstack/ironic master: Reduce the number of small functions in pxe_utils  https://review.opendev.org/c/openstack/ironic/+/81799117:24
*** jgwentworth is now known as melwitt17:34
arne_wiebalckbye everyone o/18:15
rpittaugood night! o/18:16
opendevreviewDmitry Tantsur proposed openstack/ironic master: Add an option to create inspector-compatible boot.ipxe  https://review.opendev.org/c/openstack/ironic/+/81799818:42
opendevreviewDmitry Tantsur proposed openstack/bifrost master: [WIP] iPXE fallback script  https://review.opendev.org/c/openstack/bifrost/+/81800018:45
dtantsurgood night o/18:51
opendevreviewMerged openstack/ironic master: [Trivial] Clarify conditions under which power recovery is attempted  https://review.opendev.org/c/openstack/ironic/+/81673318:52
opendevreviewDmitry Tantsur proposed openstack/ironic master: Add an option to create inspector-compatible boot.ipxe  https://review.opendev.org/c/openstack/ironic/+/81799819:09
jandersgood morning Ironic o/23:18
stevebaker[m]good afternoon \o23:47

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