Monday, 2021-05-03

*** zzzeek has quit IRC00:15
*** paras333_ has quit IRC00:15
*** zzzeek has joined #openstack-ironic01:11
*** zzzeek has quit IRC01:17
*** ociuhandu has joined #openstack-ironic02:25
*** ociuhandu has quit IRC02:29
*** mkrai has joined #openstack-ironic03:10
iurygregorygood morning Ironic o/06:28
*** k_mouza has joined #openstack-ironic06:37
arne_wiebalckGood morning, iurygregory and Ironic!06:37
*** k_mouza has quit IRC06:41
iurygregoryarne_wiebalck, o/06:41
*** dtantsur|afk is now known as dtantsur07:10
dtantsurmorning ironic07:10
*** mkrai has quit IRC07:10
*** rpittau|afk is now known as rpittau07:19
rpittaugood morning ironic! o/07:19
cennemorning!07:19
cenne:)07:19
iurygregorymorning dtantsur rpittau cenne o/07:21
rpittauhey iurygregory :)07:21
cenneso many good mornings together ^^. Hopefully it'll be a very good morning (and week)07:24
arne_wiebalckGood morning dtantsur cenne rpittau o/07:35
rpittauhey arne_wiebalck :)07:35
* arne_wiebalck makes sure this will be a good morning by adding more good mornings07:35
rpittauheh looks better than a standard monday so far07:36
rpittaubut give it time07:36
arne_wiebalckwe have a rule in our standup to avoid the word "calm"07:36
rpittaulol07:37
arne_wiebalcklast time we used it, it was a very bad idea07:37
cenneI am going to be careful not to mention it.07:46
openstackgerritMerged openstack/ironic master: Stop testing the iscsi deploy interface  https://review.opendev.org/c/openstack/ironic/+/78895507:46
cenneSo.. Is anyone else getting build errors extraneous to the code? (Zuul)07:47
cenneTox tests seem to have passed successfully, but zuulhas some sort of SSH scare.07:48
*** tosky has joined #openstack-ironic07:50
openstackgerritRiccardo Pittau proposed openstack/ironic master: Provide an option to not cache bootable iso ramdisks  https://review.opendev.org/c/openstack/ironic/+/78873407:51
rpittaucenne: which tests ?07:52
cennehttps://zuul.opendev.org/t/openstack/build/9020dc23cb0b42f8a741013d4578683807:56
cennehttps://zuul.opendev.org/t/openstack/build/82d4419aaf5044419a065e13628c2d8907:57
cenneI'm keen for a green signal from Zuul but something or the other keeps popping up.08:00
rpittaummm openstacksdk still uses bionic there, interesting08:00
rpittauI issued a recheck, if that happen again I think we need to ask infra08:01
cenneoh. thank you rpittau.08:04
rpittaunp08:04
*** sshnaidm|off is now known as sshnaidm08:12
rpittaudtantsur: I think ipa-tempest-wholedisk-bios-ipmi-direct-dib-src is used by diskimage-builder, do we want to change that there too to ipa-tempest-bios-ipmi-direct-src so we can remove the job from ipa-builder?08:39
dtantsurrpittau: https://review.opendev.org/c/openstack/diskimage-builder/+/78897208:40
rpittauwell that answer my question :)08:40
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic stable/victoria: Automaticaly set cipher suite  https://review.opendev.org/c/openstack/ironic/+/78650708:42
dtantsurarne_wiebalck, TheJulia and whoever is interested: https://owlet.today/posts/ironic-wallaby-meetup-russian-edition/08:42
*** ricolin has joined #openstack-ironic08:54
arne_wiebalckdtantsur: Thanks, that's a great write-up!08:56
arne_wiebalckdtantsur: Graphical console via Redfish: we started having a look, but are blocked with KVMIP.08:57
arne_wiebalckdtantsur: We raised this with our vendor (how to actually use the endp oint) and will follow up. If someone else is interested in this, great!08:58
arne_wiebalckdtantsur: Otherwise, there is quite some useful feedback!08:58
dtantsurcool :)08:59
arne_wiebalckdtantsur: It seems there is a lot of "how to get started/how to contribute/how to interact" questions.08:59
iurygregoryrpittau, now you can say you got answers from ironic and release in the ipa-builder thread \o/09:00
arne_wiebalckdtantsur: Maybe the anwers to these should become short (SIG) videos or posts on the website.09:01
rpittauiurygregory: yeah09:01
arne_wiebalckrpittau: iurygregory: who would "someone else" be, outside of the Ironic community/users?09:02
rpittauarne_wiebalck: someone who's interested in using ironic in some way? so not a user yet :)09:03
*** mkrai has joined #openstack-ironic09:03
*** mkrai has quit IRC09:32
*** mkrai has joined #openstack-ironic09:37
dtantsurcould I maybe have another +2 on https://review.opendev.org/c/openstack/ironic/+/786033 please?09:40
openstackgerritMerged openstack/ironic-python-agent stable/wallaby: Fix getting memory size in some lshw output  https://review.opendev.org/c/openstack/ironic-python-agent/+/78892109:43
rpittaudtantsur: I'm reading through it09:44
cenneyey! green signal from zuul.10:06
*** mkrai_ has joined #openstack-ironic10:10
*** mkrai has quit IRC10:12
*** dtantsur is now known as dtantsur|brb10:13
openstackgerritMerged openstack/ironic-lib master: Stop testing the iscsi deploy interface  https://review.opendev.org/c/openstack/ironic-lib/+/78896810:23
openstackgerritMerged openstack/ironic-inspector master: Trivial: remove an ancient option from the CI  https://review.opendev.org/c/openstack/ironic-inspector/+/78897410:23
*** mkrai_ has quit IRC11:24
openstackgerritMerged openstack/ironic master: Deploy interface that fully relies on custom deploy steps  https://review.opendev.org/c/openstack/ironic/+/78603311:27
*** mkrai has joined #openstack-ironic11:27
dtantsur|brbw00t!11:29
*** dtantsur|brb is now known as dtantsur11:29
dtantsurfolks, please take a look at the rest of https://review.opendev.org/q/topic:%22iscsi-gate%22+status:open they're needed to proceed with the iSCSI removal11:34
*** mkrai has quit IRC11:46
*** nam-est has joined #openstack-ironic11:55
nam-estHi Ironic,11:55
nam-estDoes anyone see this error before?11:56
nam-esthttps://pastebin.com/NW01TT7s11:56
iurygregoryI never saw this error before .-.11:57
nam-estIt seems like a problem related to the language.11:58
dtantsurnam-est: looks like python-hardware returned something awkwarsd12:02
dtantsurawkward12:02
dtantsurwhich, I suspect, may be a regression in it12:02
dtantsurcould you fetch the raw extra data? it must be somewhere in the logs?12:02
nam-estdtantsur: Do you mean the log of the ironic-inspector?12:05
dtantsuryep12:05
nam-estSure12:05
dtantsurit logs all the data it receives12:05
nam-estGive me a moment, I'm waiting for my colleague to pass me the log12:09
*** tzumainn has joined #openstack-ironic12:14
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Import deployment logic from ironic-lib  https://review.opendev.org/c/openstack/ironic-python-agent/+/78896412:19
dtantsurJayF: done ^^12:19
rpittaunam-est: what's the version of python-hardware being used?12:28
rpittauI'm curious about that as we iddn't do any changes in a while to python-hardware12:30
cenneon it dtantsur:12:31
nam-estdtantsur: this is the full log of ironic-inspector: https://pastebin.com/NCWkNrQ012:37
nam-estI think the problem starts from line 90312:37
nam-estrpittau: give me a moment12:38
dtantsuryeah, disk information is malformed12:41
dtantsurrpittau: it's a regression in https://github.com/redhat-cip/hardware/commit/a0223b68f11ddfe29e92858fda17c93664aa05f4 (append vs extend)12:42
rpittaudtantsur: yeah, but that should've been fixed already12:43
rpittauthat's why I asked about the version12:43
dtantsurrpittau: do we have a release out with the fix?12:43
rpittauyeah12:43
dtantsurRDO may have delayed it12:43
rpittau0.26.0 already has the fix12:43
rpittauhttps://github.com/redhat-cip/hardware/commit/dc40c57d2983900a041b45a14532d3a7b0ad964412:44
rpittauI think there was a bug in launchpad tracking that12:44
rpittaunam-est, dtantsur: https://bugs.launchpad.net/tripleo/+bug/190191712:46
openstackLaunchpad bug 1901917 in tripleo "ci.centos jobs in master failing introspection with ERROR ironic_inspector.process TypeError: unhashable type: 'list'" [Critical,Triaged]12:46
dtantsur0.26.0 is not exactly new12:47
dtantsurnam-est: could you try updating your IPA image?12:47
rpittauyeah12:47
nam-estIPA? Not ironic-inspector?12:47
rpittau0.27.0 is the latest, and it's from 2 months ago12:47
dtantsurnam-est: IPA. the problem is on the agent side.12:48
openstackgerritDmitry Tantsur proposed openstack/ironic master: Clean up deprecated features of the agent deploy  https://review.opendev.org/c/openstack/ironic/+/78898312:48
rpittaunam-est: the issue is on ipa side, on the ramdisk12:48
nam-estI see, I will update IPA to the latest12:48
rpittaunam-est: ok, let us know how it goes12:48
nam-estSure. Let's see if the problem will go away. Thanks very much dtantsur, rpittau for your help12:49
cennedtantsur: sent a patch. testing result directly now for list_vendor_passthru12:56
cennealso I think I replicated the (strange) bug I had while `git review`. (which we discussed also on #opendev)12:58
cenneIt happened just now again when I changed the commit message without changing files.12:59
cenne*tried changing commit message12:59
dtantsurinteresting indeed13:13
*** lmcgann has joined #openstack-ironic13:15
*** rloo has joined #openstack-ironic13:17
nam-estrpittau: dtantsur: Just to confirm, I need to install the hardware package version 0.27.0 on the IPA ramdisk, right?13:23
openstackgerritMerged openstack/ironic-python-agent-builder master: Stop testing the iscsi deploy interface  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/78897113:24
rpittaunam-est: correct13:24
nam-estrpittau: thanks13:24
TheJuliagood morning13:25
rpittaugood morning TheJulia :)13:25
cennemore info regarding ~bug:  using --no-thin option in the git push gerrit command, pushes it through.13:26
TheJuliahopefully everyone had a wonderful weekend13:26
cenneGood morning TheJulia13:26
dtantsurTheJulia: morning13:27
openstackgerritDmitry Tantsur proposed openstack/virtualbmc master: Remove "iscsi" from job names  https://review.opendev.org/c/openstack/virtualbmc/+/78928213:30
*** MahnoorAsghar has joined #openstack-ironic13:35
openstackgerritDmitry Tantsur proposed openstack/virtualbmc master: Remove "iscsi" from job names  https://review.opendev.org/c/openstack/virtualbmc/+/78928213:36
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Trivial: better guard against malformed data in extra_hardware  https://review.opendev.org/c/openstack/ironic-inspector/+/78928513:43
dtantsurrpittau: ^^13:43
rpittaudone13:49
nam-estrpittau: dtantsur: Unfortunately, upgrade option is not available for us.13:50
TheJuliado we have anything we want to review jam on today?13:51
nam-estIs that possible if downlift the IPA help to solve the problem?13:51
rpittaunam-est: oh, what about cherry=pick ?13:51
dtantsurnam-est: we cannot fix the bug without changing the code :-/13:51
dtantsurdowngrade at your own risk13:51
rpittauTheJulia: I have just https://review.opendev.org/c/openstack/ironic/+/788734 :)13:52
rpittaummm just saw another comment to address13:52
TheJuliaI guess we can still jump on the call and review/discuss any specific changes people have13:53
*** tosin has joined #openstack-ironic13:53
rpittauI'm also found another thing to add there, base_iso is missing in the param list13:53
dtantsurI have https://review.opendev.org/c/openstack/ironic/+/78898313:53
nam-estrpittau: we cannot do that since we currently have no infra to build IPA13:54
rpittaunam-est: I see, so where do you get ipa ramdisk from ?13:54
nam-estI just wonder if a lower version can help. If yes, which one?13:54
rpittaunam-est: 0.26.013:54
nam-estI mean the IPA api version13:55
dtantsurnam-est: I'm not quite sure why you can downgrade IPA, but not upgrade it...13:55
rpittauand also not sure why you need to downgrade ipa13:55
rpittauthe problem is in py-hardware13:55
*** zaneb has joined #openstack-ironic13:56
* TheJulia glares at ironicbaremetal.org's build log13:57
openstackgerritMerged openstack/ironic-ui master: Add horizon-nodejs14-job template.  https://review.opendev.org/c/openstack/ironic-ui/+/78587713:57
nam-estI guess the problem had been there after a certain commit was push13:57
dtantsurnam-est: yes, and that commit was last year13:57
rpittauoh ok13:57
rpittauprobably 0.24.0 would work13:57
*** zaneb has quit IRC13:58
*** zaneb has joined #openstack-ironic13:58
dtantsurrpittau, nam-est, the latest image from https://images.rdoproject.org/centos8/master/rdo_trunk/current-tripleo/ has 0.27.114:00
dtantsurI recommend you update to it and try again. otherwise we cannot help much, unfortunately14:01
nam-estOk, I will try it.14:01
TheJuliareview jam if anyone is interested14:02
TheJuliahttps://meetpad.opendev.org/ironic14:02
nam-estdtantsur: Oh, so the latest image from current-tripleos has 0.27.0. That is cool. I thought that it was not there and we need to build ourselves.14:05
nam-estWe will try that image and see if the problem goes away. Thanks again rpittau, dtantsur14:06
openstackgerritBob Fournier proposed openstack/ironic-specs master: Include Redfish BIOS Attribute Registry in bios API  https://review.opendev.org/c/openstack/ironic-specs/+/77468114:15
*** paras333 has joined #openstack-ironic14:18
TheJuliaiurygregory: still good to run the meeting this week?14:40
rpittauTheJulia: I think it was me :)14:41
TheJuliadoh'oh14:41
TheJuliaokay, rpittau still good to run the meeting this week?14:41
rpittaueheh yeah :)14:41
TheJuliaawesome14:41
TheJuliaI added some announcements to the agenda last last week, so I think we're good :)14:41
rpittauok!14:42
iurygregoryhehe yeah it was rpittau =)14:43
iurygregoryI can run the meeting next week =)14:43
dtantsurrun, the meeting, run14:45
iurygregoryI may not participate in the meeting today, trying to debug a setup with a friend14:46
TheJuliaiurygregory: much appreciated14:51
TheJuliait *feels* more and more unlikely I'll just be MIA next week14:52
TheJuliabut one never knows14:52
openstackgerritDmitry Tantsur proposed openstack/ironic master: Clean up deprecated features of the agent deploy  https://review.opendev.org/c/openstack/ironic/+/78898314:59
rpittau#startmeeting ironic15:00
openstackMeeting started Mon May  3 15:00:21 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
dtantsuro/15:00
*** stendulker has joined #openstack-ironic15:00
rlooo/15:00
stendulkero/15:00
arne_wiebalcko/15:00
TheJuliao/15:00
rpittauHello everybody and welcome to yet another ironic weekly meeting!15:01
rpittauI'm honored to be your host today :)15:01
cenneo/15:01
rpittauyou can find our agenda at the usual place15:01
rpittau#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
rpittauLet's start with some announcements15:02
rpittau#topic Announcements / Reminders15:02
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:02
rpittau#info ironicbaremetal.org now has blogs!15:02
rpittau#link https://ironicbaremetal.org/15:02
rpittauthanks TheJulia for working on that15:03
rpittauand also...15:03
rpioso\o15:03
TheJuliaAlways happy to help :)15:03
rpittau#info To aid the effort to add more content to the website TheJulia created and etherpad to collect links and content15:03
rpittau#link https://etherpad.opendev.org/p/ironic-media15:03
rlooyay! thx TheJulia!!15:03
rpittauany idea, link, thought, drop that there :)15:03
TheJulia++15:04
rpittauon a more sad note15:04
rpittau#info mgoddard is stepping down as release liaison. Any volunteers should inquire during Open Discussion.15:04
rpittauthanks mgoddard for your contribution :)15:04
dtantsurIf iurygregory would like to do this, I'd propose him. He's been doing really good job recently.15:05
dtantsurand thank you mgoddard :)15:05
TheJulia++15:05
iurygregoryyup =) I would be happy to do15:05
rpittauany volunteers let's defer to Open Discussion15:06
rpittauand ofc I have some issues with irccloud :/15:07
rpittauanyhow15:07
TheJuliame too actually. Backing DB appears to be down15:07
rpittauyeah15:07
TheJulia:(15:07
rpittauDoes anyone have anything else to announce or remind us of?15:07
TheJuliaI'm not expecting to be available next week and I'll be away on Friday15:07
dtantsurI'll likely be away next week on Thu (holiday) and Fri15:08
dtantsurI'm also very limitedly available this week because of KubeCon EU15:08
rpittausame for Kubecon and I'll be out Thu-Fri next week15:08
arne_wiebalckI will also be off Thu/Fri next week15:08
rpittauseems no more announcements, we can move on15:10
rpittau#info Review action items from previous meeting15:10
MahnoorAsgharI wanted to mention an RFE for review. Is now a good time?15:10
MahnoorAsgharOh there's a section for that, bit early. Sorry!15:10
rpittauMahnoorAsghar: later we have an RFE review section, thanks :)15:11
MahnoorAsgharPerfect!15:11
rpittauit seems we didn't have any action items from last meeting15:11
rpittauso let's pass to the next part15:11
rpittau#topic Review Subteam status reports15:12
*** openstack changes topic to "Review Subteam status reports (Meeting topic: ironic)"15:12
rpittauI think we're still defining priorities for Xena, so no status yet15:12
TheJuliaI was about to say something similar15:12
rpittauwe do have a patch to review with the Xena themes15:13
rpittauplease have a look at it when you have a minute :) https://review.opendev.org/c/openstack/ironic-specs/+/78414315:14
rpittaualright, moving forward15:14
rpittau#topic Deciding on priorities for the coming week15:14
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:14
rpittau#link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio15:14
rpittauwe don't have a lot of priorities in the list15:15
TheJuliaI can't really think of anything else that really needs to be added right now15:15
dtantsurhttps://review.opendev.org/q/topic:%22iscsi-gate%22+(status:open%20OR%20status:merged)15:15
rpittauI'm going to add one as well https://review.opendev.org/c/openstack/ironic/+/78873415:16
TheJuliaoh yes, that is likely a good set of things to add15:16
dtantsurand the growing list of https://review.opendev.org/q/topic:%22no-iscsi%22+(status:open%20OR%20status:merged)15:16
TheJuliatagging15:16
rpittaufeel free to add ironic-week-prio tag to all of the patches there not mergeed15:16
rpittauthanks TheJulia :)15:16
arne_wiebalckburn-in is also ready to get more reviews I think: https://review.opendev.org/q/topic:%22story_2007523%2215:16
dtantsurI'd also appreciate https://review.opendev.org/c/openstack/ironic/+/78871815:17
rpittauright, thanks arne_wiebalck15:17
rpittauit looks like we have a good maount of priorities for the week now! :)15:18
arne_wiebalckheh15:18
TheJulia\o/15:18
rpittauonward!15:19
rpittau#topic Discussion15:19
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:19
rpittauI have a topic for discussion, more a yes/no question15:19
rpittaugot a reply on ipa-builder stable/branches from the ML http://lists.openstack.org/pipermail/openstack-discuss/2021-April/022032.html, should we move on then?15:19
*** dhellmann has joined #openstack-ironic15:21
arne_wiebalckhasn't Herve also replied ?15:21
iurygregoryhe did15:21
arne_wiebalckdon't see this on the page15:21
rpittauyeah, the reply does not appear there ?15:21
rpittauoh15:21
iurygregoryclick next message =)15:21
rpittauit cut by month15:21
rpittauhttp://lists.openstack.org/pipermail/openstack-discuss/2021-May/022170.html15:21
TheJuliaI think we should just go ahead and do it15:22
TheJulialazy consensus through a lack of objections15:22
rpittausounds good to me15:22
rpittauany more opinions ?15:22
arne_wiebalckWe should probably say it is unlikely we break anyone's setup.15:23
TheJulia++15:23
arne_wiebalckI think this is Herve's question.15:23
iurygregoryI think arne_wiebalck should say that :D15:23
rpittauarne_wiebalck: I agree, I had in mind to reply to Herve anyway15:23
TheJuliaIt is more about, being able to fix issues15:23
arne_wiebalckhe probably thinks we split the repos since someone wanted to use only IPAB without the IPA15:25
arne_wiebalckanyway, I am all for moving forward as suggested by rpittau15:25
rpittauok, seems we're good15:26
rpittau#topic Baremetal SIG15:26
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:26
rpittauwe're going to revisit some proposals from last meeting15:26
arne_wiebalckYeah, I would like to settle on the agenda for the next meetings.15:27
TheJuliaI kept the list since last week because I think we failed to record owners.15:27
rpittaulooks like we have topics for the next 2 meetings15:27
TheJuliaI've been thinking a scaling ironic blog post might be best, to at least start out and then maybe the sig session could be more as a Q&A15:27
arne_wiebalckWe have dtantsur for IPAB and TheJulia for Bifrost.15:27
arne_wiebalckdtantsur: June and TheJulia: July if that is ok?15:28
dtantsurshould be good15:28
arne_wiebalckwe still have a slot for next week15:28
arne_wiebalckany volunteers, suggestions?15:28
arne_wiebalck(otherwise I can do a "Why Ironic at CERN?" filler)15:29
rpittauseems like a nice filler :)15:29
arne_wiebalckTheJulia: we have done a scaling post some time ago: https://techblog.web.cern.ch/techblog/post/conductor-groups/15:30
*** dking has joined #openstack-ironic15:30
arne_wiebalckanyone else who would like the slot?15:31
TheJuliaI was thinking more a high level concepts/modeling15:31
arne_wiebalcksure, I was thinking as further input15:31
arne_wiebalckOk, I think that is it then for the SIG this week :)15:32
rpittaucool!15:32
rpittauthen let's move on15:32
rpittau#topic RFE review15:32
*** openstack changes topic to "RFE review (Meeting topic: ironic)"15:32
rpittau@MahnoorAsghar I think you had something ?15:32
MahnoorAsgharYes15:32
TheJuliaarne_wiebalck: hmm, so maybe we have an opportunity for a scaling 100 series "class" and then like s scaling 400 series class15:32
arne_wiebalckTheJulia: :-D15:33
MahnoorAsgharI wanted to ask for reviews on this RFE: https://storyboard.openstack.org/#!/story/200886615:33
arne_wiebalckTheJulia: I think this is a good idea, actually!15:33
JayFMahnoorAsghar: I wonder if that'd be a better fit as a node vendor passthru... we don't typically put hardware information at the top level of nodes15:34
TheJuliaI'm also kind of thinking vendor-passthrough15:35
rpittauMahnoorAsghar: so that's specific for Dell ?15:35
MahnoorAsgharAlright15:35
MahnoorAsgharWe were thinking of making it extendable for other vendors15:35
JayFBonus: vendor passthru will /probably/ be  easier to implement than a real API endpoint15:35
rpiosoIt always puzzled me how our consumers determine the bare metal hardware RAID controller and physical disk IDs.15:35
TheJuliaI guess my couple concerns are 1) this data only seems to be the oob data. In-band data is more detailed and may have additional things based upon inspection, and the inspection dat should be visible.15:35
JayFMahnoorAsghar: sometimes we implement a feature for the first vendor as passthru, and  then as more vendors implement it an API emerges and they converge :)15:36
TheJulia2) other vendors adopting this is a whole question since some just don't care or really *need* to care about some of the fields.15:36
JayFwe don't always have to start at the point of a vendor-agnostic convergent api15:36
TheJuliaI'm really hesitant to say "lets do this as a full api" without another vendor being behind it and committing to the same15:37
rpittauyeah, that's why I asked15:37
dtantsurTheJulia: same15:37
dtantsuralthough the idea is good15:37
dtantsuron a related note, someone on the meetup last week mentioned remote erasure via iDRAC for some RAID controllers. would be cool to support it.15:38
rpiosoJayF, TheJulia: I expect it would be a non-starter to Metal3 if the solution is not vendor agnostic.15:38
dtantsurrpioso: I don't think this claim is uniformly true15:38
dtantsuralthough it will surely be harder to provide API for15:38
dtantsuranyway, I agree with TheJulia. if no other vendors back that, it will be de facto vendor passthru15:39
rpiosodtantur: vendor-passthru makes our software harder to use and I thought we had been trying to get away from that.15:39
dtantsurwhen possible15:40
rpiosodtantsur: ^^^15:40
dtantsurthis is not proved possible yet15:40
JayFI mean, we need a viable path to having this implemented for a lot of hardware15:40
dtantsurcan we at least fetch RAID devices via pure Redfish?15:40
JayFonly a dell implementation of an API is not really good :(15:40
TheJuliaAnd some things are always going to be vendor passthrough or vendor specific, and that is okay15:40
rpiosodtantsur: Yep, agree when possible. Let's see if it is :-)15:40
openstackgerritDmitry Tantsur proposed openstack/ironic master: Update refarch with information about image_download_source  https://review.opendev.org/c/openstack/ironic/+/78937715:41
rpittaulet's maybe leave some notes on the RFE itself ?15:42
dtantsura possible approach is to fetch it during inspection and put in properties (?)15:42
MahnoorAsgharrpittau: sounds good :D15:42
rpiosodtantsur: Yes, the IDs are available via Redfish. The Redfish raid interface works :-)15:42
TheJuliadtantsur: I was thinking the same15:42
MahnoorAsghardtantsur: That could work as well15:42
JayFdtantsur: I think that's an OK approach, but if we were going to do that, we should probably think about it for more than jsut disks? IDK :/15:42
JayFHell, you could even exclude inspector -- just have IPA list disks on boot and store that15:42
JayFala clean / deploy steps15:42
JayF(We never did implement an API for "boot a ramdisk  and tell me possible clean steaps", did we?)15:43
dtantsurI have no idea how easy is to list controllers/disks in the iDRAC notion from inside the machine15:43
MahnoorAsgharJayF: that could also work15:43
dtantsurthis "Disk.Bay.0:Enclosure.Internal.0-1:RAID.Intergrated.1-1" stuff is not something we receive now IIRC15:43
rpiosoIn addition to those IPA suggestions, a purely out-of-band approach would be useful. No ramdisk boot would be required.15:44
rpiosodtantsur: How about local host Redfish or WS-Man?15:44
MahnoorAsghardtantsur: It is not available in the inspection, but is available in the ironic conductor logs15:45
dtantsurrpioso: Redfish host interface is something we should eventually research. but credentials are a huge issue.15:45
rpiosodtantsur: Understood. IPMI? I'm unfamiliar with its prospective knowledge/reporting of those IDs.15:46
JayFIPMI would be a hard no, I think15:47
JayFeven if it's possible, it'd be very vendor/config dependant15:47
dtantsuryeah15:47
rpiosoAgain, a fully OOB approach would be useful.15:47
rpiosoSounds like Redfish at some point in the future.15:47
dtantsurI feel like we should soon seriously rethink our notion of inspection.15:47
JayFdtantsur++++15:47
JayFThis is basically going down the path of "make node inspection info available over API"15:48
JayFwhich is not a bad idea15:48
dtantsuryep15:48
JayFbut I'm loathe to include all that scope in MahnoorAsghar's idea, because disks, and how they interact with root hints, is an actual UX problem now15:48
dtantsurand probably get rid of in-band and OOB inspection separation (which we've already started fwiw)15:48
rpittauI'm going to add a triaged tag to the RFE, but please remember to add your notes on the discussion there :)15:48
dtantsurany non-trivial API addition requires a spec, I think15:49
rpiosoJayF: This is not directly related to root device hints. Rather, just RAID configuration.15:49
JayFhmm15:49
rpittauwe have 10 minutes left, maybe continue the discussion after the meeting?15:50
MahnoorAsgharrpittau: Shall I add the notes from this discussion there?15:50
rpittauMahnoorAsghar: sure!15:50
MahnoorAsgharrpittau: lets continue it after the meeting15:50
rpittauok15:51
rpittaulet's move on then15:51
rpittau#topic Open Discussion15:51
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:51
*** lbragstad_ is now known as lbragstad15:51
rpittaugoing back to the "release liaison" topic, do we have other volunteers?15:52
rpittauit doesn't look so15:53
TheJuliaIt is not really that much work and if you do any release work at all, it is really just being an alternate point of conteact15:53
TheJuliaand you get the authority to ship $stuff ;)15:53
* dtantsur assumed rpittau was a liaison, but apparently no15:53
iurygregoryoh the +1 PTL =O15:53
TheJuliaOn another note, we have even more blogs now on https://ironicbaremetal.org/blog/15:54
TheJuliaedits can be made through github :)15:54
iurygregoryWOOT15:54
iurygregory:D15:54
rpittaummm I'm not, and I think I will volunteer15:54
rpittau#link15:54
rpittauwhoops15:55
rpittau#link https://ironicbaremetal.org/blog/15:55
iurygregoryanything we need to do to volunteer? like we need to send a patch to a repo or something?15:55
rpittaudtantsur, TheJulia, please consider me also for release liaison15:55
* dtantsur +115:55
TheJulia+115:56
dtantsurrpittau, iurygregory, another responsibility is to react to notifications from the release team15:56
dtantsurboth emails and gerrit (you'll be added to any reviews touching ironic deliverables)15:56
iurygregoryack =)15:56
TheJuliaiurygregory: who the current liason is for projects is tracked on the release wiki15:56
rpittausounds good :)15:56
dtantsurmm, no, not wiki15:56
TheJuliaI thought it was in the wiki, is it now a repo?15:56
dtantsurhttps://opendev.org/openstack/releases/src/branch/master/data/release_liaisons.yaml15:57
TheJulia(It has been ages since I've updated it)15:57
TheJuliaokay, then I guess it moved \o/15:57
iurygregorynice!15:57
rpittaualright 3 minutes left, do we have other topics to discuss ?15:58
rpittau#topic Who is going to run the next meeting?15:58
*** openstack changes topic to "Who is going to run the next meeting? (Meeting topic: ironic)"15:58
TheJulia\o/ and now blog links work correctly15:58
iurygregoryso the PTL needs to send a patch or should just me and/or rpittau send ? =)15:58
TheJuliaAny volunteers?15:58
iurygregoryI can run =)15:58
rpittau#info iurygregory will run next meeting! Thanks! :)15:59
*** MahnoorAsghar has quit IRC15:59
TheJuliaiurygregory: rpittau: either of you can submit a patch and the a PTL can +1 it15:59
dtantsurTheJulia: you can pick other links from my blog, especially tutorials alike15:59
TheJuliadtantsur: ack15:59
iurygregoryTheJulia, ack15:59
rpittaualright, we're at the top of the hour, let's continue the discussions out of the meeting :)15:59
rpittauthanks everyone!16:00
rpittau#endmeeting16:00
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list."16:00
openstackMeeting ended Mon May  3 16:00:08 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-05-03-15.00.html16:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-05-03-15.00.txt16:00
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2021/ironic.2021-05-03-15.00.log.html16:00
TheJulia:)16:00
arne_wiebalckthanks rpittau !16:00
TheJuliaThanks rpittau!16:00
iurygregorytks rpittau !16:00
rpittaumy pleasure :)16:00
TheJuliaSo w/r/t the blog on the website, Pixie Boots is listed as the author16:00
TheJuliaor the Bare Metal SIG16:01
TheJuliaIf anyone submits actual content beyond pointers to content, I guess we'll just have it be listed as themself16:01
*** paras333 has quit IRC16:08
*** paras333 has joined #openstack-ironic16:08
iurygregorywow, the OpenStack Summit in Austin was 5 years ago .-.16:10
dtantsurquite possibly16:10
rpittauParis was 7 years ago....16:10
TheJuliaYeah, That was a long time ago16:10
*** nam-est has quit IRC16:10
*** MahnoorAsghar has joined #openstack-ironic16:11
MahnoorAsgharShall we resume the discussion on the RFE?16:15
openstackgerritDmitry Tantsur proposed openstack/ironic master: Bye-bye iSCSI deploy, you served us well  https://review.opendev.org/c/openstack/ironic/+/78938216:15
dtantsurI wonder how many things will fail on ^^ :)16:16
dtantsur53 files changed, 115 insertions(+), 4392 deletions(-)16:16
rpittauoO16:16
openstackgerritDmitry Tantsur proposed openstack/ironic master: Bye-bye iSCSI deploy, you served us well  https://review.opendev.org/c/openstack/ironic/+/78938216:17
iurygregoryWOW16:17
*** MahnoorAsghar has quit IRC16:18
dtantsurdamn, I keep finding more and more places >_<16:19
*** MahnoorAsghar has joined #openstack-ironic16:20
JayFjroll: https://review.opendev.org/c/openstack/ironic/+/789382 you should vote on this for old time sake16:21
jrollomg16:21
JayFdtantsur: jroll: https://review.opendev.org/c/openstack/ironic/+/789382 is AKA "OnMetal team wins near-decade-long deploy driver war"16:21
jrollyep16:21
JayFI will put that patch in my review queue, it's too long for a meme vote :D16:21
arne_wiebalckdtantsur: what will happen when the controller meets nodes which have the deploy interface set to iscsi in the DB?16:22
dtantsur:)16:22
openstackgerritDmitry Tantsur proposed openstack/ironic master: Bye-bye iSCSI deploy, you served us well  https://review.opendev.org/c/openstack/ironic/+/78938216:22
dtantsurarne_wiebalck: they'll be unusable until fixed16:22
iurygregoryTheJulia, dtantsur https://review.opendev.org/c/openstack/releases/+/789381 =)16:23
iurygregoryrpittau, your +1 would be good also hehe16:23
openstackgerritMerged openstack/ironic-inspector master: Trivial: better guard against malformed data in extra_hardware  https://review.opendev.org/c/openstack/ironic-inspector/+/78928516:23
TheJuliadtantsur: wouldn't it be reasonable to include an online migration?16:23
dtantsurTheJulia: we used to have it16:23
dtantsurso hopefully people with a reasonable configuration did get migrated16:23
dtantsurbut if they disabled the "direct" deploy, we couldn't do much16:24
TheJuliaahh16:24
TheJuliaokay16:24
dtantsurToday I really appreciated the stestr's --failing flag16:24
*** stendulker has quit IRC16:24
dtantsurvery handy16:24
* TheJulia happens onto the Eurobeat disney dance mix16:26
tosinHi I keep getting an error for this change https://review.opendev.org/c/openstack/ansible-collections-openstack/+/77339916:26
tosinThis is the error - InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised.16:26
tosinNot sure what to do about this?16:26
dtantsurthis is not an error, you need to find a real error16:27
tosinhttps://www.irccloud.com/pastebin/xtclqHdN/16:28
tosinthis an error?16:28
arne_wiebalckI guess for many deployments, the hardware gets retired before the iscsi deprecation happens ;)16:28
arne_wiebalckAnd new hardware is hopefully not enrolled with iscsi.16:29
dtantsurtosin: seems so. I don't know enough ansible to understand it though. you probably need to add this failure to the ignore list.16:29
tosindtantsur: ok16:31
TheJuliatosin: Yeah, I am fairly sure the field name matches the list of fields which can potentially have sensitive data in them16:32
TheJuliaas such it needs the no_log parameter also set16:32
*** dtantsur is now known as dtantsur|afk16:32
dtantsur|afko/16:32
TheJuliagoodnight dtantsur16:32
rpittaubye everyone! o/16:32
*** rpittau is now known as rpittau|afk16:32
MahnoorAsgharBye guys!16:33
openstackgerritBob Fournier proposed openstack/ironic master: Retrieve BIOS registry and include fields in bios API  https://review.opendev.org/c/openstack/ironic/+/78670716:35
tosinTheJulia: that makes sense. Thank you16:35
*** gyee has joined #openstack-ironic16:46
*** tkajinam has quit IRC16:48
openstackgerritMerged openstack/metalsmith master: Stop testing the iscsi deploy interface  https://review.opendev.org/c/openstack/metalsmith/+/78897516:58
*** k_mouza has joined #openstack-ironic17:07
*** dhellmann_ has joined #openstack-ironic17:13
*** dhellmann has quit IRC17:15
*** dhellmann_ is now known as dhellmann17:16
arne_wiebalckbye everyone o/17:20
cennebye17:27
TheJuliao/17:27
cenneenjoy your days off everyone17:29
TheJuliaThanks cenne17:29
cenne^u^17:29
cenneand thanks dtantsur!17:30
*** cp- has joined #openstack-ironic17:40
*** paras333 has quit IRC18:33
*** dsneddon has joined #openstack-ironic18:42
TheJuliaAny thoughts on a published integration list? https://deploy-preview-29--ironicbaremetal.netlify.app/integrations/18:56
*** trandles has quit IRC19:05
dkingWhen using openstack baremetal node deploy $node_identifier --config-drive '{"user_data": ...}}', is that data still sent as JSON?19:05
dkingI noticed that what little documentation I could find for cloud-init, it looks like it's supposed to be yaml, and unlike "meta_data" and "network_data", the created filename in the configdrive mount does not name "user_data" with the .json extension. I'm trying to figure out why my "user_data" didn't seem to work.19:09
openstackgerritMerged openstack/ironic-python-agent master: Stop testing the iscsi deploy interface  https://review.opendev.org/c/openstack/ironic-python-agent/+/78897319:21
*** k_mouza has quit IRC19:36
*** ayoung has joined #openstack-ironic19:36
*** cp- has quit IRC19:53
*** gyee has quit IRC19:53
*** SpamapS has quit IRC19:53
*** lbragstad has quit IRC19:54
*** fdegir has quit IRC19:54
*** priteau has quit IRC19:54
*** hjensas has quit IRC19:54
*** akahat|ruck has quit IRC19:54
*** cp- has joined #openstack-ironic19:55
*** gyee has joined #openstack-ironic19:55
*** SpamapS has joined #openstack-ironic19:55
*** lbragstad has joined #openstack-ironic19:55
*** fdegir has joined #openstack-ironic19:55
*** priteau has joined #openstack-ironic19:55
*** hjensas has joined #openstack-ironic19:55
*** akahat|ruck has joined #openstack-ironic19:55
*** cp- has quit IRC19:55
*** cp- has joined #openstack-ironic19:56
*** dhellmann_ has joined #openstack-ironic20:11
TheJuliadsneddon: https://specs.openstack.org/openstack/ironic-specs/specs/12.1/support-smart-nic.html#security-impact20:12
*** dhellmann has quit IRC20:12
*** dhellmann_ is now known as dhellmann20:13
*** k_mouza has joined #openstack-ironic20:36
*** k_mouza has quit IRC20:41
*** tosin has quit IRC20:43
*** rloo has quit IRC22:10
*** ayoung has quit IRC22:28
*** paras333_ has joined #openstack-ironic22:35
*** paras333 has joined #openstack-ironic22:39
*** ayoung has joined #openstack-ironic22:40
*** paras333 has quit IRC22:44
*** tosky has quit IRC22:50
stevebakerTheJulia: an update on my centos lvm issue, I can remount / and mount the missing /tmp, /home and have a usable system (until reboot). I might try comparing systemd unit differences between centos and rhel22:56
*** rcernin has joined #openstack-ironic23:07

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