Monday, 2019-04-15

*** hwoarang has quit IRC00:07
*** hwoarang has joined #openstack-ironic00:10
*** ijw has quit IRC00:15
*** ijw has joined #openstack-ironic00:15
*** ijw has quit IRC00:16
*** ijw has joined #openstack-ironic00:16
*** Goneri has joined #openstack-ironic00:35
*** Goneri has quit IRC01:03
*** ijw has quit IRC01:35
*** hwoarang has quit IRC02:00
*** ijw has joined #openstack-ironic02:01
*** hwoarang has joined #openstack-ironic02:02
openstackgerritGhanshyam Mann proposed openstack/python-ironic-inspector-client master: Dropping the py35 testing  https://review.openstack.org/65238602:31
*** hwoarang has quit IRC02:39
*** hwoarang has joined #openstack-ironic02:43
*** hwoarang has quit IRC03:16
*** hwoarang has joined #openstack-ironic03:23
*** ijw has quit IRC04:01
*** ijw_ has joined #openstack-ironic04:01
openstackgerritDigambar proposed openstack/ironic master: Modify the iDRAC driver to use realtime RAID creation  https://review.openstack.org/63490304:20
*** ijw_ has quit IRC04:29
*** ijw has joined #openstack-ironic04:30
*** ijw has quit IRC04:36
*** ijw has joined #openstack-ironic04:37
*** whoami-rajat has joined #openstack-ironic04:37
*** rcernin has quit IRC05:08
*** rcernin has joined #openstack-ironic05:10
*** Lucas_Gray has joined #openstack-ironic05:10
*** jtomasek has joined #openstack-ironic05:27
*** ijw has quit IRC05:46
*** ijw has joined #openstack-ironic05:47
*** ijw has quit IRC05:53
*** Lucas_Gray has quit IRC06:03
*** cjloader has quit IRC06:11
*** yaawang has quit IRC06:13
*** pcaruana has joined #openstack-ironic06:19
openstackgerritAnkit Kumar proposed openstack/ironic stable/rocky: Fix for error in validating checksum  https://review.openstack.org/65257006:22
*** hwoarang has quit IRC06:28
*** hwoarang has joined #openstack-ironic06:29
*** yaawang has joined #openstack-ironic06:30
*** toabctl has joined #openstack-ironic06:32
*** yaawang has quit IRC06:40
*** e0ne has joined #openstack-ironic06:48
*** e0ne has quit IRC06:49
*** e0ne has joined #openstack-ironic06:52
*** hwoarang has quit IRC06:52
*** hwoarang has joined #openstack-ironic06:53
*** ijw has joined #openstack-ironic06:55
*** yaawang has joined #openstack-ironic06:58
*** sungn has joined #openstack-ironic06:58
sungnhello06:59
*** rcernin has quit IRC07:05
*** e0ne has quit IRC07:12
*** iurygregory has joined #openstack-ironic07:17
iurygregorygood morning07:17
*** spotz has joined #openstack-ironic07:22
*** andrein has joined #openstack-ironic07:28
*** e0ne has joined #openstack-ironic07:31
*** rpittau|afk is now known as rpittau07:34
rpittaugood morning ironic! o/07:34
*** andrein has quit IRC07:40
iurygregoryrpittau, you alive? \o/07:42
rpittauiurygregory: kind of xD07:42
iurygregoryI can imagine hehehe07:42
*** e0ne has quit IRC07:52
rpittauTheJulia: that is what happen when people try to get rid of rabbits :)07:53
*** e0ne has joined #openstack-ironic07:57
*** xXraphXx has joined #openstack-ironic07:59
*** lucasagomes has joined #openstack-ironic07:59
*** gkadam has joined #openstack-ironic08:04
etingofrpittau, rocks! o/08:05
*** gkadam is now known as gkadam-afk08:06
rpittauetingof: thanks! :)08:06
*** tssurya has joined #openstack-ironic08:06
*** dougsz has joined #openstack-ironic08:07
*** dtantsur|afk is now known as dtantsur08:30
dtantsurmorning ironic08:30
rpittauhey dtantsur :)08:31
*** andrein has joined #openstack-ironic08:33
*** gkadam-afk is now known as gkadam08:36
openstackgerritMerged openstack/bifrost master: Replace hardcoded yum command by package manager variable  https://review.openstack.org/65224908:37
openstackgerritMerged openstack/sushy master: Update model to support ApplyTime annotations  https://review.openstack.org/64365608:39
iurygregorymorning dtantsur o/08:51
openstackgerritRiccardo Pittau proposed openstack/networking-generic-switch master: Removing obsolete oslosphinx  https://review.openstack.org/65259508:59
openstackgerritDmitry Tantsur proposed openstack/ironic master: [POC][WIP] Publish baremetal endpoint via mdns  https://review.openstack.org/65181009:08
*** e0ne has quit IRC09:09
*** sungn has quit IRC09:14
*** e0ne has joined #openstack-ironic09:18
*** mkrai has joined #openstack-ironic09:33
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client master: Dropping the py35 testing  https://review.openstack.org/65238609:39
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client master: Dropping the py35 testing  https://review.openstack.org/65238609:43
openstackgerritMerged openstack/ironic-specs master: A fresh way of looking at step retrieval  https://review.openstack.org/60619909:45
openstackgerritMerged openstack/ironic-specs master: Add details on target_raid_config for Software RAID support  https://review.openstack.org/64310009:45
*** diga has joined #openstack-ironic09:46
digaGood Morning Ironic!09:46
openstackgerritMerged openstack/ironic-ui master: Dropping the py35 testing  https://review.openstack.org/65238309:48
dtantsurmorning diga09:49
digadtantsur: Hey09:50
digadtantsur: my patch is passing all zuul checks, can you take a look https://review.openstack.org/#/c/634903/09:51
patchbotpatch 634903 - ironic - Modify the iDRAC driver to use realtime RAID creation - 16 patch sets09:51
dtantsurI'll put it on my queue, no hard promises though09:52
digadtantsur: already got +1 last time but since there is problem in jsonschema in requirement which you fixed it09:52
digadtantsur: :) sure09:52
digarpittau: arne_wiebalck: I also request you guys, please take a look09:52
*** e0ne has quit IRC09:55
openstackgerritMerged openstack/ironic-inspector-specs master: Fix docs job  https://review.openstack.org/65199109:56
openstackgerritAnkit Kumar proposed openstack/ironic stable/rocky: Fix for error in validating checksum  https://review.openstack.org/65257010:01
*** Lucas_Gray has joined #openstack-ironic10:02
*** e0ne has joined #openstack-ironic10:06
openstackgerritMerged openstack/ironic-python-agent master: Dropping the py35 testing  https://review.openstack.org/65238210:25
openstackgerritMerged openstack/python-ironicclient master: Dropping the py35 testing  https://review.openstack.org/65238710:25
*** e0ne has quit IRC10:26
*** andrein has quit IRC10:27
openstackgerritMerged openstack/sushy master: Dropping the py35 testing  https://review.openstack.org/65238810:28
dtantsurfolks, does anyone have a clue what is wrong about this command? https://review.openstack.org/#/c/651810/4/devstack/lib/ironic10:33
patchbotpatch 651810 - ironic - [POC][WIP] Publish baremetal endpoint via mdns - 4 patch sets10:33
*** e0ne has joined #openstack-ironic10:35
rpittaudtantsur: I don't see anything wrong at a glance, is it giving you error ?10:35
dtantsurrpittau: nope, but the packages are dropped10:36
dtantsuroh, I think it is blocking the outgoing packages10:45
rpittaudtantsur: I think you're missing the rule on output10:45
dtantsurhah :)10:45
rpittauoh yeah ok :)10:45
openstackgerritDmitry Tantsur proposed openstack/ironic master: [POC][WIP] Publish baremetal endpoint via mdns  https://review.openstack.org/65181010:46
*** andrein has joined #openstack-ironic10:47
*** bfournie has quit IRC10:52
*** diga has quit IRC10:56
openstackgerritUros Orozel proposed openstack/ironic-python-agent master: Ironic python agent does not extract correct available memory  https://review.openstack.org/64809611:00
*** e0ne has quit IRC11:10
*** Wryhder has joined #openstack-ironic11:23
*** Lucas_Gray has quit IRC11:24
*** Wryhder is now known as Lucas_Gray11:24
openstackgerritDmitry Tantsur proposed openstack/ironic stable/stein: Ansible module fix: stream_url  https://review.openstack.org/65262511:52
openstackgerritDmitry Tantsur proposed openstack/ironic stable/rocky: Ansible module fix: stream_url  https://review.openstack.org/65262611:53
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: Ansible module fix: stream_url  https://review.openstack.org/65262711:53
openstackgerritDmitry Tantsur proposed openstack/ironic stable/stein: Ansible module: fix partition_configdrive.sh file  https://review.openstack.org/65262811:53
openstackgerritDmitry Tantsur proposed openstack/ironic stable/rocky: Ansible module: fix partition_configdrive.sh file  https://review.openstack.org/65263011:53
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: Ansible module: fix partition_configdrive.sh file  https://review.openstack.org/65263111:54
openstackgerritDmitry Tantsur proposed openstack/ironic stable/stein: Ansible module: fix clean error handling  https://review.openstack.org/65263211:55
openstackgerritDmitry Tantsur proposed openstack/ironic stable/rocky: Ansible module: fix clean error handling  https://review.openstack.org/65263311:55
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: Ansible module: fix clean error handling  https://review.openstack.org/65263411:55
openstackgerritMerged openstack/ironic-lib master: Dropping the py35 testing  https://review.openstack.org/65238111:58
openstackgerritDmitry Tantsur proposed openstack/ironic master: [POC][WIP] Publish baremetal endpoint via mdns  https://review.openstack.org/65181012:02
*** livelace has joined #openstack-ironic12:05
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: [POC][WIP] Fetch baremetal endpoint from mdns  https://review.openstack.org/65201712:05
*** jcoufal has joined #openstack-ironic12:10
*** e0ne has joined #openstack-ironic12:16
*** sungn has joined #openstack-ironic12:19
*** andrein has quit IRC12:26
openstackgerritDmitry Tantsur proposed openstack/ironic master: Do not try to create temporary URLs with zero lifetime  https://review.openstack.org/64905212:30
*** Lucas_Gray has quit IRC12:34
*** andrein has joined #openstack-ironic12:34
openstackgerritMerged openstack/python-ironic-inspector-client master: Dropping the py35 testing  https://review.openstack.org/65238612:35
*** e0ne has quit IRC12:40
*** Lucas_Gray has joined #openstack-ironic12:41
*** e0ne has joined #openstack-ironic12:41
*** bfournie has joined #openstack-ironic12:41
*** mkrai has quit IRC12:55
*** e0ne has quit IRC12:58
jrollmorning everyone13:00
dtantsurmorning jroll13:01
TheJuliagood morning13:05
dtantsurmorning TheJulia13:08
*** Lucas_Gray has quit IRC13:11
*** e0ne has joined #openstack-ironic13:17
*** jroll has quit IRC13:25
*** jroll has joined #openstack-ironic13:26
*** e0ne has quit IRC13:28
*** Goneri has joined #openstack-ironic13:30
*** rloo has joined #openstack-ironic13:31
*** e0ne has joined #openstack-ironic13:34
*** mjturek has joined #openstack-ironic13:42
*** rh-jelabarre has joined #openstack-ironic13:44
*** bnemec has joined #openstack-ironic13:46
rpittauhi TheJulia :)13:46
dtantsurrpittau: no more iptables errors, but the packages still do not go through :(13:47
TheJuliadtantsur: ???13:48
TheJuliaLooks like our general multinode job is break happy now :(13:48
dtantsurTheJulia: I cannot make mdns work in the CI environment. The ramdisk and ironic do not see each other.13:48
mgoddardTheJulia: mind if I propose some bifrost stable releases?13:48
TheJuliamgoddard: anytime!13:49
mgoddardTheJulia: :)13:49
TheJuliadtantsur: do not see each other?!?13:49
TheJulialike packets dropping in the night?13:49
* dtantsur suspects we may need something with routing (?)13:49
dtantsurTheJulia: yeah, like that13:49
TheJuliadtantsur: on a flat network?13:49
dtantsurTheJulia: yep13:49
TheJuliaand the announcer is attached directly to the flat network?13:50
TheJulialike same layer213:50
TheJuliaand a layer3 address13:50
iurygregorymorning TheJulia o/13:50
*** priteau has joined #openstack-ironic13:51
dtantsurTheJulia: this is something I'm trying to remember about our devstack setup13:51
dtantsurit may not be the case13:51
TheJuliadtantsur: got some patches to look at? I'll gladly take a look later today13:52
dtantsurTheJulia: https://review.openstack.org/#/c/652017/ and https://review.openstack.org/#/c/651810/13:52
patchbotpatch 652017 - ironic-python-agent - [POC][WIP] Fetch baremetal endpoint from mdns - 3 patch sets13:52
patchbotpatch 651810 - ironic - [POC][WIP] Publish baremetal endpoint via mdns - 6 patch sets13:52
dtantsurI really wonder if we have the same problem as we had with TFTP13:53
rpittauI was thinking the same13:53
dtantsuralthough DHCP works..13:53
jrolldtantsur: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L220813:54
dtantsurjroll: right, I *think* I'm doing the same with https://review.openstack.org/#/c/651810/6/devstack/lib/ironic13:54
patchbotpatch 651810 - ironic - [POC][WIP] Publish baremetal endpoint via mdns - 6 patch sets13:54
TheJuliadhcp is magically bound13:55
jrolldtantsur: but also, I think the dhcp runs in a network namespace which is connected to the virtual network - you might need to run mdns the same way13:55
dtantsurjroll: well, ironic-inspector DHCP is pretty regular, I think, and it still works..13:55
TheJuliaThat doesn't need to be done for flat networks13:55
jrollah13:56
jrollthis may help: https://serverfault.com/questions/121032/forward-mdns-from-one-subnet-to-another13:57
dtantsuralthough there is some weird stuff here https://github.com/openstack/ironic-inspector/blob/master/devstack/plugin.sh#L361-L37413:57
jrollfirst answer tells you why it isn't working, second gives some suggestions13:57
dtantsurmmm, yeah, this is my suspect right now13:58
dtantsurwhich means that we may not be able to test this on devstack13:58
jrollunless we could use something like avahi-daemon13:59
dtantsurmaybe, I haven't looked into it13:59
jrollyeah, same13:59
dtantsurI guess this problem is specific to devstack, because ironic is not listening on the L2 network where bare metals are14:00
dtantsurand we don't even have an easy to access interface for that14:00
rpittauthat would mean setting up avahi in the same subnet as the ramdisk14:00
dtantsuryeah, which is inside a namespace14:00
TheJuliadtantsur: I think I see your issue14:00
jrollthat isn't necessarily specific to devstack, but I get what you mean :)14:00
TheJuliacomments on the patch14:00
*** cdearborn has joined #openstack-ironic14:01
dtantsurto be clear, my goal is not to make it Work On DevStack :)14:01
dtantsurTheJulia: mm, this may be a problem, but not here. I can imagine receiving packages with "127.0.0.1", but we're getting none..14:01
TheJuliano, think of it as raw socket code14:02
dtantsurto elaborate, the address we advertise does not have to be on the same subnet. it's just a bit of information, mdns doesn't care what is there14:02
TheJulia127.0.0.1 is purely on the lo interface14:02
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management to redfish hw type  https://review.openstack.org/65178514:02
dtantsurright, but we're not getting to using it14:03
TheJuliasure you can do aliased loopbacks, but that is a super un-common configuration14:03
dtantsurthe ramdisk doesn't receive any packages14:03
dtantsur(and devstack endpoints resolve correctly, unless the CI is messed up)14:03
TheJuliabut devstack endpoint is the public IP of the node yes?14:03
TheJuliaAnd we don't want to try and route and forward multicast packets in CI I take it?14:04
dtantsurright, I see what you mean. it IS a problem, just not a problem I'm facing yet :)14:05
TheJuliaI guess the ultimate question.... where is mdns actually listening14:05
dtantsurall interfaces (which is unlikely to include the one inside of neutron namespace..)14:06
*** e0ne has quit IRC14:06
TheJuliaoh, yeah... maybe try and test on a bifrost job since there is no neutron14:07
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicators REST API endpoints  https://review.openstack.org/65267614:07
dtantsurTheJulia: it would be cool to test it on bifrost eventually, but I don't think Depends-On IPA patches will actually work. or maybe it will?14:08
* dtantsur needs more coffee, preferably with brandy14:08
*** etingof is now known as etingof|brb14:10
*** livelace has quit IRC14:10
TheJuliadtantsur: bifrost uses dib14:11
TheJuliaI guess I'd be cool merging code if I can test it locally and it wouldn't be too hard to do14:11
dtantsurTheJulia: okay, as to resolving names, we can change the SRV record to point at a name14:14
*** livelace has joined #openstack-ironic14:16
NobodyCamGood Morning Ironic'ers :)14:16
dtantsurmorning NobodyCam14:17
iurygregorymorning NobodyCam14:17
NobodyCam:) Happy Monday dtantsur :)14:17
NobodyCamhey hey good Morning iurygregory :)14:18
*** sthussey has joined #openstack-ironic14:18
openstackgerritMerged openstack/networking-generic-switch master: Removing obsolete oslosphinx  https://review.openstack.org/65259514:18
*** Lucas_Gray has joined #openstack-ironic14:21
*** e0ne has joined #openstack-ironic14:21
openstackgerritMichael Turek proposed openstack/ironic master: devstack: Remove syslinux dependency  https://review.openstack.org/65128514:29
openstackgerritDmitry Tantsur proposed openstack/ironic master: [POC][WIP] Publish baremetal endpoint via mdns  https://review.openstack.org/65181014:34
*** cjloader has joined #openstack-ironic14:34
*** ijw has quit IRC14:44
*** ijw has joined #openstack-ironic14:44
* TheJulia begins powering up the coffee making device for round 214:45
*** stendulker has joined #openstack-ironic14:50
*** baha has joined #openstack-ironic14:52
*** dnuka has joined #openstack-ironic14:56
*** kaifeng has joined #openstack-ironic14:57
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Apr 15 15:00:11 2019 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
TheJuliao/15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
dtantsuro/15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
jrollohai15:00
iurygregoryo/15:00
kaifengo/15:00
TheJuliaGood morning everyone!15:00
jiapeio/15:00
rpittauo/15:00
cdearborno/15:00
rlooo/15:00
dnukao/15:00
stendulkero/15:00
TheJuliaOur agenda for this week can be found on the wiki.15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
cdearbornTheJulia: good morning!15:00
TheJulia#topic Announcements / Reminders15:01
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:01
rpiosoo/15:01
mgoddard\o15:01
TheJulia#info A tentative PTG schedule has been proposed for ironic. Please let TheJulia know if there are any questions, concerns, or items that need to be added/changed.15:02
TheJulia#link https://etherpad.openstack.org/p/DEN-train-ironic-ptg15:02
TheJulia#info The Denver PTG ironic team evening gathering doodle has been posted.15:02
TheJulia#link https://doodle.com/poll/e6q5e6pm72wbiwtz15:02
*** ijw_ has joined #openstack-ironic15:03
TheJulia#info Git repositories will be migrating to OpenDev on Friday of this week week. tl;dr gerrit will be down.15:03
TheJulia#link http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html15:04
TheJuliaand finally from the list for this week!15:04
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Support reapply with supplied introspection data  https://review.openstack.org/63903915:04
TheJulia#info Python 3.5 "support" is going to be dropped from the master branches.15:04
TheJulia#link http://lists.openstack.org/pipermail/openstack-discuss/2019-April/005097.html15:04
dtantsurand Stein was released ;)15:04
TheJulia#info And Stein was released. Great job everyone!15:05
TheJuliaDoes anyone have anything they would like to announce or remind us of this week?15:05
*** ijw has quit IRC15:07
TheJuliaI guess not!15:07
TheJuliaProceeding onward!15:07
dtantsur++15:07
TheJulia#topic Reviewing action items from previous meeting15:07
*** openstack changes topic to "Reviewing action items from previous meeting (Meeting topic: ironic)"15:07
TheJulia#link http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-04-08-15.00.html15:07
TheJuliaWe had three action items. All of them mine \o/15:08
TheJuliaI followed up with neutron folk regarding smartnics. They anticipate early in the cycle for merging their side of the code.15:08
TheJuliaI added the fast-track tempest test scenario to the proposed list for review.15:09
TheJuliaAnd I created the doodle.15:09
TheJuliaSo that was it!15:09
TheJuliaOnward!15:09
*** w14161_1 has joined #openstack-ironic15:09
rloothank you TheJulia. * 3 :)15:09
TheJulia#topic Review subteam status reports15:09
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:09
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:10
TheJuliaStarting around line 26815:10
TheJuliamgoddard: Regarding https://review.openstack.org/#/c/641731/15:11
patchbotpatch 641731 - ironic - WIP: Add iDRAC RAID deploy steps - 5 patch sets15:11
mgoddardTheJulia: I should take that out :)15:11
TheJuliaWIP?15:11
mgoddardIt's still WIP, I would still like feedback, but it hasn't moved along15:12
TheJuliaShould we add it to the list to review?15:12
mgoddardI think the WIP scares people off, but sure15:12
TheJuliaIt definitely does15:13
mgoddardrealistically I won't have time to update until after the summit, I'm on vacation for a week from tomorrow15:13
TheJuliaack15:13
cdearbornmgoddard:  I will try to find some time to review today15:13
TheJuliaif there are minor things would you like people to revise it?15:13
mgoddardthanks cdearborn that would be great15:14
*** e0ne has quit IRC15:14
TheJulialocking would <3 reviews, just saying. I'll gladly write an etcd driver if we land the framework15:15
TheJuliaLooks like we might want to go back to considering graphical console. Looks like they have been working on updating the patches.15:16
iurygregorywould be possible to add to review https://review.openstack.org/#/c/647774/  https://review.openstack.org/#/c/651466/ ? both have +2  Both are cherry-pick to stable/stein (Zuulv3+Python3 jobs)15:16
patchbotpatch 647774 - python-ironicclient (stable/stein) - Move to zuulv3 - 2 patch sets15:16
patchbotpatch 651466 - python-ironicclient (stable/stein) - Run jobs under python2 and python3 - 1 patch set15:16
*** Lucas_Gray has quit IRC15:16
TheJuliaAnyway, that is the only questions I have from the list. Is everyone else good to proceed?15:17
TheJuliaiurygregory: I'll add them to the list15:17
*** Lucas_Gray has joined #openstack-ironic15:17
iurygregoryTheJulia, ty =)15:17
dtantsur++ proceed15:18
TheJulia#topic Deciding on priorities for the coming week15:18
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:18
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:18
TheJuliastarting around line 12515:18
* TheJulia removes merged things15:18
TheJuliaSo I've got a long list of patches, about half of them are stable reviews15:19
TheJuliaStarting at 185.15:20
TheJuliaI think they are all worthy to add and we should be able to nuke the stable patches from the list quickly15:20
* TheJulia makes more coffee for everyone15:21
rloo++15:22
* TheJulia adds15:22
*** e0ne has joined #openstack-ironic15:23
rpittauthe exception refactoring is kind of tricky to test in CI because of the cross-project reference and I think it breaks grenade anyway if not backported ?15:23
dtantsurlooks about right, although probably too much for a week15:24
dtantsurrpittau: grenade update all packages, so it will pull the newer ironic-lib15:24
dtantsur* updates15:24
TheJuliaYeah, a lot of sushy patches for good things like sensor data eventually :)15:25
mgoddardare we standardising this sensor data?15:25
TheJuliaI'm good with the list, I'll likely turn around and approve most of the stable stuff anyway15:25
mgoddardthe ipmitool data is a mess15:26
mgoddardsorry, derailing15:26
mgoddardlater15:26
TheJuliamgoddard: oh yeah, very much so :(15:26
rpittaudtantsur: yeah, I'm just puzzled by the output of the upgrade check, but we can talk later15:27
TheJuliaI guess we should proceed to RFE review?15:27
* dtantsur hides15:27
rpittaulet's15:28
TheJuliaProceeding onward!15:28
TheJulia#topic RFE Review15:28
*** openstack changes topic to "RFE Review (Meeting topic: ironic)"15:28
*** etingof|brb is now known as etingof15:28
TheJuliaAnd now that dtantsur is hiding, he brings us a fun neat addition which could be extremely useful for things like usb drives :)15:28
dtantsur#link https://storyboard.openstack.org/#!/story/2005393 Use mDNS for discovering ironic and ironic-inspector services by the ramdisk15:29
dtantsurso, there is a lot of text there, but the idea is to allow IPA that got started anyhow to find ironic locally15:29
dtantsuressentially, enabling Bring-Your-Own-IPA :) be it a flash drive, a virtual media implementation that we do not support, or anything15:29
dtantsurthe technical side is to use DNS service discovery + multicast DNS (details in the RE)15:30
dtantsur* RFE15:30
TheJuliaI <3 it, personally15:30
TheJuliathe v6 limitation is concerning, but I feel like we would be driving innovation15:31
TheJuliaI really like the txt record, is there a length limit to the txt field?15:31
dtantsurthe length of a UDP package, largely15:31
TheJuliaso tail end of a packet15:31
*** mkrai has joined #openstack-ironic15:32
TheJuliaMakes sense I guess15:32
dtantsurto put it simply, yes15:32
dtantsurso we shouldn't get overly creative with using it, but we can add e.g. inspector collectors list15:32
TheJuliaMakes tons of sense to me15:32
TheJuliaor maybe even a trusted cert url :\15:33
dtantsuryep15:33
TheJuliaI'm all for this feature, are there any questions?15:33
TheJuliaTotally as an aside, great job detailing everything dtantsur!15:34
*** e0ne has quit IRC15:34
dtantsurthx :)15:34
TheJuliaIt reads as very well thought out15:34
* TheJulia begins to spin up the crickets15:34
TheJuliaAny concerns out there? Are we good with this RFE?15:35
kaifengI am good with it15:35
TheJuliaI'm also good with it15:36
rpittautotally good with that :)15:36
* rloo still reading/grok'ing, wondering why it isn't a spec but maybe we're good with it in a story.15:36
dtantsurrloo: there's really not much to say in a formal spec structure15:36
dtantsurno API changes, no RPC changes, no driver changes, etc15:37
dtantsurso if I'm asked to do a spec, I'll mostly copy-paste the whole text into the "Proposed solution" :)15:37
rloodtantsur: just config changes? (and code?)15:37
dtantsuryeah, config and code15:37
rlooand versioning?15:37
TheJuliano versioning impact15:38
dtantsurno API or RPC changes -> no versioning?15:38
rloothought we had ipa versioning and this includes ipa code changes?15:38
rlooany security issues?15:38
dtantsurwe have IPA API versioning, but this is not API15:38
*** gyee has joined #openstack-ironic15:38
TheJuliaWe do, but only to track conductor -> IPA API changes15:38
dtantsur(have we actually finished implementing it btw?)15:38
rloo(I haven't read the RFC or proposed SIG guidelines yet15:38
TheJuliadtantsur: basic guards I believe15:39
rlooi am not against it, just cannot be for it. but if we have enough cores that are good with it?15:39
TheJuliasecurity wise, local segment so attached... and I believe the idea is to fallback to this if we don't have command lines?15:39
kaifengif IPA broadcasting service, it's a way for polling instead of lookup, just imaging15:40
TheJuliaerr, command line/kernel params?15:40
mgoddardshould the mdns server be integrated with the conductor?15:40
mgoddardis it not an external service?15:40
rloofwiw, the usefulness of the spec process are the questions/sections, to make sure they are all covered :)15:40
dtantsurmgoddard: yep, why not? I mean, we could use Avahi, but that's a big thing to maintain compared to the actual code we care about15:40
dtantsurI don't mind writing a spec out of it tomorrow morning if people actually will review it :)15:41
* TheJulia thinks rfe-approved and then time for open discussion15:41
mgoddarddtantsur: it's another dependency15:41
rloodtantsur: as long as folks are good with it (and eg gone through and addressed/thought about things we typically ask in a spec)15:41
dtantsurmgoddard: python-zeroconf is quite small, one python file15:42
rloodtantsur: i mean, if no one wants a spec and you/others think you've addressed any/most issues.15:42
rloodtantsur: then don't write a spec.15:42
mgoddarddtantsur: ok, we can probably handle that15:42
TheJulia#topic Open Discussion15:42
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:42
rloowondering now about making it optional. guess overkill to make it eg some sort of plugin.15:43
TheJuliaSo what if someone wants to or needs to run avhali???15:43
dtantsurrloo: we don't have these kind of plugins, but it will be guarded by a configuration option (on the ironic side)15:43
rloodtantsur: right, that's the overkill (for now :))15:43
TheJuliaMaybe a logical iteration may be to plugin-ize it15:44
TheJuliabut not starting out15:44
dtantsurTheJulia: we got them covered: set [mdns]enabled=False and configure avahi the way they want :)15:44
TheJuliadtantsur: very true!15:44
TheJuliaso plugins then are kind of silly15:44
TheJuliaAnyway15:44
TheJuliamgoddard: yes, ipmi sensor data is crazy15:44
mgoddardTheJulia: yeah. Will there be an effort to provide some sort of schema?15:45
TheJuliaiurygregory has been having some fun converting some sensor data to something prometheus can read via a plugin and it has been all sorts of fun.15:45
iurygregoryit's \o/15:45
TheJuliamgoddard: I _think_ the ideal path is for us to create plugins that can pre-sort/transform data and then transmit or offer up data as we go on15:46
TheJuliaso if sensor data from vendor zz-alpha has super weird formatting, it can be transformed by a specific plugin15:46
TheJuliajust someone would need to create that transform15:47
mgoddardUltimately, it's a set of time series, with a name, 'dimensions' (in Influx speak) and possibly metadata15:47
TheJuliaiurygregory: do you have the link handy to your pull request?15:47
iurygregoryTheJulia, yup15:48
mgoddardI remember the  IPMI sensor data being a big JSON blob, really we need a list of measurements15:48
TheJuliamgoddard: it varies by bmc15:48
TheJuliaThere is a whole huge section in the ipmi standard doc that you have to wrap your brain around to grok it. I think for redfish we should end up with something slightly more sane... Hopefully15:49
iurygregorymgoddard, yup is a huge json15:49
iurygregoryhttps://github.com/metalkube/ironic-prometheus-exporter/pull/215:49
mgoddardwill ironic-prometheus-exporter be usable outside of metalkube?15:50
iurygregorymgoddard, yup15:50
iurygregoryis bassicaly an oslo notifier driver15:50
* etingof thinks that Redfish gives more freedom to the vendors that want to distinguish themselves15:51
iurygregorythat will convert IPMI JSON metrics to prometheus format15:51
iurygregoryI need to drop now, if you need anything tomorrow we can talk15:51
TheJuliawe may want to take something like that into ironic or into our namespace at some point, but yeah, more thought likely required15:51
TheJuliagoodnight iurygregory15:51
iurygregorygood night ppl o/15:52
mgoddardWe were actually talking about this today, but with monasca15:52
mgoddardhow do we get sensor data from nodes into monasca?15:52
TheJuliaso how does monasca collect data?15:53
mgoddardthe approach will probably be similar - consume rabbit notifications, translate, push to monasca15:53
TheJuliayeah15:53
TheJuliaor15:53
mgoddardmonasca has metrics identified by a name, with 'dimensions' - most commonly hostname15:53
TheJuliause the notifier plugin framework and grab the message before it is ever sent to rabbit15:53
mgoddardthere is that option. we'd need to do translation in ironic15:54
TheJuliaso maybe a ironic-monasca-exporter?15:54
TheJulias/a/an/15:54
mgoddardalso have some 'business' logic for adding metadata to our metrics15:54
openstackgerritIlya Etingof proposed openstack/sushy master: Deprecate System-specific `IndicatorLED` state constants  https://review.openstack.org/65270915:54
mgoddardbut we could potentially do that in monasca later15:55
TheJuliaI already added some of the metadata that we needed as well into the messages15:55
TheJuliabut there may always be more needed15:55
TheJuliaThis sounds like an awesome integration point potentially15:56
*** andrein has quit IRC15:56
TheJuliaAnyway, we have four minutes left15:56
TheJuliaDoes anyone have anything else they would like to discuss?15:56
mgoddardyou see my desire for standardisation? Standard metric names that at most would have some sensible transformation to get into prometheus/monasca/whatever format15:57
TheJuliaI think we would need to take several vendors and essentially boil down what they offer into a condensed data set to really create standard metric names15:58
w14161_1For L3 Deploy spec, ironic already approved it. BUt it was not implemented yet, did we have plan to implement L3 deploy spec?15:58
TheJulia"drives_healthy" "power_healthy" instead of "drive1 connected: healthy" and "psu1: ok" and "psu3: ok"15:59
TheJuliaw14161_1: It is not implemented. It is more about getting IP configuration information without DHCP. We had a contributor that started it but then went on to focus on other things and I think someone expressed interest in trying to pick it up this cycle16:00
TheJuliaAnyway, it is time for the end of the meeting16:00
TheJuliaThanks everyone!16:00
TheJuliaThis doesn't mean we have to stop chatting though :)16:00
TheJulia#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-dev mailing list."16:00
openstackMeeting ended Mon Apr 15 16:00:34 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
openstackgerritIlya Etingof proposed openstack/sushy master: Deprecate System-specific `IndicatorLED` state constants  https://review.openstack.org/65270916:00
w14161_1Thx Julia16:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-04-15-15.00.html16:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-04-15-15.00.txt16:00
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-04-15-15.00.log.html16:00
*** dnuka has quit IRC16:00
*** stendulker has quit IRC16:01
*** kaifeng has quit IRC16:01
openstackgerritMadhuri Kumari proposed openstack/ironic-specs master: Add support for Intel Speed Select in Ironic  https://review.openstack.org/65145916:10
mkraidtantsur: Hi, I will post the node allocation update API on wednesday. I am still working on it.16:13
dtantsurgreat!16:17
*** hwoarang has quit IRC16:21
*** hwoarang has joined #openstack-ironic16:22
rpittaugood night! o/16:28
*** rpittau is now known as rpittau|afk16:28
*** mkrai has quit IRC16:42
*** dtantsur is now known as dtantsur|afk16:44
dtantsur|afk\o16:44
TheJuliagoodnight16:44
*** priteau has quit IRC16:47
*** priteau has joined #openstack-ironic16:48
*** ijw_ has quit IRC16:49
*** e0ne has joined #openstack-ironic16:51
*** ijw has joined #openstack-ironic16:51
*** dsneddon has joined #openstack-ironic16:52
openstackgerritMerged openstack/python-ironicclient stable/stein: Move to zuulv3  https://review.openstack.org/64777416:53
openstackgerritMerged openstack/python-ironicclient stable/stein: Run jobs under python2 and python3  https://review.openstack.org/65146616:53
*** lucasagomes has quit IRC16:57
*** tssurya has quit IRC16:58
*** ijw has quit IRC17:00
*** w14161_1 has quit IRC17:00
*** ijw has joined #openstack-ironic17:00
*** dougsz has quit IRC17:02
*** Lucas_Gray has quit IRC17:13
*** mbeierl has joined #openstack-ironic17:14
*** gkadam has quit IRC17:15
*** ijw has quit IRC17:17
*** ijw has joined #openstack-ironic17:17
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management harness to ManagementInterface  https://review.openstack.org/64967517:31
*** ijw has quit IRC17:35
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management to redfish hw type  https://review.openstack.org/65274017:36
*** ijw has joined #openstack-ironic17:38
*** w14161_1 has joined #openstack-ironic17:53
*** e0ne has quit IRC18:00
*** sungn has quit IRC18:04
*** andrein has joined #openstack-ironic18:24
arne_wiebalckdiga: I'm off this week, should be back next week.18:27
*** e0ne has joined #openstack-ironic19:02
*** e0ne has quit IRC19:06
*** andrein has quit IRC19:06
*** ijw has quit IRC19:10
*** jcoufal has quit IRC19:38
*** penick has quit IRC19:46
*** andrein has joined #openstack-ironic19:54
*** mjturek has quit IRC20:07
*** baha has quit IRC20:07
*** rh-jelabarre has quit IRC20:11
*** pcaruana has quit IRC20:38
*** ijw has joined #openstack-ironic20:41
*** Goneri has quit IRC20:44
*** priteau has quit IRC20:44
*** tonyb[m] has joined #openstack-ironic21:27
tonyb[m]So who knows about kolla and ironic?21:27
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicators REST API endpoints  https://review.openstack.org/65178521:31
*** whoami-rajat has quit IRC21:54
*** iurygregory has quit IRC22:02
*** jtomasek has quit IRC22:03
*** bfournie has quit IRC22:04
eanderssonWhen in standalone mode do you need to render config-drive from scratch?22:30
eanderssone.g. --config-drive '{"meta_data": {"hostname": "server1.cluster"}}'22:30
eanderssonWould this be the only data in the config drive?22:31
eanderssonor would some data like ethernet mac address etc be pre populated?22:31
openstackgerritRichard G. Pioso proposed openstack/ironic master: Place upper bound on python-dracclient version  https://review.openstack.org/65140822:31
eanderssonmostly wondering how one would set up things like bonding in standalone mode22:31
*** livelace has quit IRC22:45
*** hwoarang has quit IRC23:22
*** ijw has quit IRC23:23
*** rcernin has joined #openstack-ironic23:24
*** hwoarang has joined #openstack-ironic23:27
* TheJulia reads23:32
TheJuliaeandersson: sorry, been working on the project update slides and related statistic collection23:32
TheJuliaeandersson: essentially.... yeah. You could pre-stage all the data in a folder and it can package everything (and even other files/scripts....) for configuring the machine.23:33
TheJuliaeandersson: yeah as in the only data in the drive.23:33
TheJuliaWe don't append anything else in at present. We would be willing to accept patches to add more data if a dictionary is transmitted!23:34
TheJuliaBonded interfaces in standalone, as in no neutron as well?23:34
eanderssonYea - we are looking at using ironic standalone as a sort of undercloud23:37
eanderssonto kickstart our computes23:37
jrolltonyb[m]: mgoddard is probably your best bet, but I'd say just ask? I know some things about each, but not together23:38
eanderssonbut would like baremetal cloud init to take care of the bonding23:39
eanderssonhostname etc23:40
*** hwoarang has quit IRC23:44
*** hwoarang has joined #openstack-ironic23:45
tonyb[m]jroll: https://review.openstack.org/#/q/I9db48a3462422749290bbb887c14816734ab0478 added a dependency on udevadm which isn't in the container so I figure I need a kolla patch to make it install udevadm/systemd-udev etc23:48
eanderssonTheJulia, you wouldn't know of any guides or examples23:50
eanderssonWe are unsure on the format of the data23:51
eanderssonAnd https://docs.openstack.org/ironic/latest/install/configdrive.html#when-used-standalone isn't helping23:52
eanderssonTo quote my colleague, if we could just see the inside of the example file23:52
eandersson> /dir/configdrive_files23:52
eanderssonI am sure it would all make sense23:53
*** sthussey has quit IRC23:57
eandersson*files23:58

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!