Monday, 2019-04-01

*** dhellmann has quit IRC00:05
*** dhellmann has joined #openstack-ironic00:08
*** tiendc has joined #openstack-ironic01:54
*** dsneddon has quit IRC02:14
*** irclogbot_1 has quit IRC02:19
*** edleafe has quit IRC02:22
*** dsneddon has joined #openstack-ironic03:17
*** dsneddon has quit IRC03:21
*** dsneddon has joined #openstack-ironic03:36
*** dsneddon has quit IRC03:41
*** dsneddon has joined #openstack-ironic03:51
*** dsneddon has quit IRC03:59
*** dsneddon has joined #openstack-ironic03:59
*** dsneddon has quit IRC04:05
*** Lucas_Gray has joined #openstack-ironic04:39
*** ianychoi has quit IRC05:23
*** ianychoi has joined #openstack-ironic05:24
*** whoami-rajat has joined #openstack-ironic05:31
*** Lucas_Gray has quit IRC05:49
*** dsneddon has joined #openstack-ironic05:56
*** dsneddon has quit IRC06:01
*** dsneddon has joined #openstack-ironic06:06
*** S4ren has joined #openstack-ironic06:30
etingofstackoverflow is awesome today o/06:39
arne_wiebalckgood morning, ironic!06:41
*** rachit7 has joined #openstack-ironic06:44
*** S4ren has quit IRC06:57
*** rcernin has quit IRC06:58
*** pcaruana has joined #openstack-ironic06:58
*** pcaruana has quit IRC07:02
*** pcaruana has joined #openstack-ironic07:02
rpittau|afkgood morning ironic! o/07:21
*** iurygregory has joined #openstack-ironic07:22
iurygregorygood morning all07:22
*** rpittau|afk is now known as rpittau07:24
*** dsneddon has quit IRC07:28
openstackgerritDigambar proposed openstack/ironic stable/rocky: [WIP] Remove nic method dependencies from dracclient==1.5.0 from stable releases  https://review.openstack.org/64836007:37
*** e0ne has joined #openstack-ironic07:40
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP|DNM] testing uefi job with ubuntu bionic  https://review.openstack.org/64550707:48
*** dsneddon has joined #openstack-ironic07:52
*** lucasagomes has joined #openstack-ironic07:52
*** S4ren has joined #openstack-ironic08:01
*** gkadam has joined #openstack-ironic08:02
*** S4ren has quit IRC08:03
*** tssurya has joined #openstack-ironic08:08
*** andrein has joined #openstack-ironic08:10
*** e0ne has quit IRC08:11
openstackgerritMerged openstack/ironic master: Make 'noop' the explicit default of default_storage_interface  https://review.openstack.org/64867908:14
ijpascualTheJulia: I am trying to run VM creation playbooks from Bifrost manually, but even the command itself results in an error, so it is not Ansible related.08:15
*** e0ne has joined #openstack-ironic08:16
*** verma-varsha has joined #openstack-ironic08:22
verma-varshaGood morning! Could someone please find time to review this (https://review.openstack.org/#/c/642748/) patch?08:30
patchbotpatch 642748 - sushy - Add `FabricCollection` and `Fabric` classes - 2 patch sets08:30
*** iurygregory has quit IRC08:32
*** gkadam has quit IRC08:32
*** iurygregory has joined #openstack-ironic08:33
*** dtantsur|afk is now known as dtantsur08:35
dtantsureandersson: doesn't look right indeed. fortunately, we don't seem to use this option anywhere :)08:36
dtantsurmorning ironic08:36
*** derekh has joined #openstack-ironic08:37
*** stendulker has joined #openstack-ironic08:39
rpittauhey dtantsur :)08:41
*** e0ne has quit IRC08:50
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP|DNM] testing uefi job with ubuntu bionic  https://review.openstack.org/64550708:57
*** e0ne has joined #openstack-ironic08:59
*** Lucas_Gray has joined #openstack-ironic09:03
iurygregorymorning dtantsur rpittau o/09:12
openstackgerritMerged openstack/ironic-ui stable/stein: i18n: Extract strings from template files  https://review.openstack.org/64873609:13
openstackgerritMerged openstack/ironic-ui stable/stein: Fix misused html attribute 'translation' in base-node.html  https://review.openstack.org/64880109:13
openstackgerritMerged openstack/ironic-ui stable/queens: Fix misused html attribute 'translation' in base-node.html  https://review.openstack.org/64880409:13
*** jrist has quit IRC09:15
*** jrist has joined #openstack-ironic09:16
*** e0ne has quit IRC09:16
openstackgerritMerged openstack/ironic-ui stable/rocky: Fix misused html attribute 'translation' in base-node.html  https://review.openstack.org/64880309:23
*** gkadam has joined #openstack-ironic09:28
*** e0ne has joined #openstack-ironic09:35
rpittauhi iurygregory :)09:38
*** lucasagomes has quit IRC10:01
*** lucasagomes has joined #openstack-ironic10:04
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP|DNM] testing uefi job with ubuntu bionic  https://review.openstack.org/64893810:11
*** verma-varsha has quit IRC10:12
*** Lucas_Gray has quit IRC10:23
*** Lucas_Gray has joined #openstack-ironic10:31
*** andrein has quit IRC10:33
*** e0ne has quit IRC10:45
*** e0ne has joined #openstack-ironic11:03
*** stendulker has quit IRC11:07
*** e0ne has quit IRC11:10
*** S4ren has joined #openstack-ironic11:12
*** tiendc has quit IRC11:15
*** EvilienM is now known as EmilienM11:27
*** bfournie has quit IRC11:28
*** rh-jelabarre has joined #openstack-ironic11:41
*** andrein has joined #openstack-ironic11:50
*** bfournie has joined #openstack-ironic12:04
*** leakypipes is now known as jaypipes12:07
*** e0ne has joined #openstack-ironic12:11
*** priteau has joined #openstack-ironic12:11
*** trown|outtypewww is now known as trown12:12
*** rcernin has joined #openstack-ironic12:19
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP|DNM] testing uefi job with ubuntu bionic  https://review.openstack.org/64893812:36
*** e0ne has quit IRC12:38
*** dougsz has joined #openstack-ironic12:44
*** dougsz has joined #openstack-ironic12:45
*** priteau has quit IRC12:51
*** rloo has joined #openstack-ironic12:52
*** priteau has joined #openstack-ironic12:53
*** whoami-rajat has quit IRC13:00
*** whoami-rajat has joined #openstack-ironic13:05
*** munimeha1 has joined #openstack-ironic13:08
*** dougsz has quit IRC13:14
*** edleafe has joined #openstack-ironic13:14
dtantsurjroll, TheJulia, a new version of the RFE: https://storyboard.openstack.org/#!/story/200532813:15
jrolldtantsur: how do we calculate the default for something where ANY is supported? just use the order of enabled interfaces?13:18
dtantsurjroll: yeah, in this proposal I did not risk changing the defaults calculation13:18
dtantsursee note #2, however, we could go a bit crazier :)13:19
jrolldtantsur: it would change somehow, because ANY won't be in the enabled list13:20
jrollI guess I need to see code to fully understand13:21
dtantsurjroll: it will hit this https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L152-L153 and move on13:21
jrolldtantsur: ah, ANY is an additional value, supported_*_interfaces can't only return [ANY] I guess13:22
dtantsurjroll: yeah, it's not possible in this proposal13:23
jrollok13:23
dtantsurFakeHardware will be like [FakePower, ANY], with FakePower still the default.13:23
jrollI'll need to think it through more later, but it seems reasonable13:23
dtantsursure, it's not pressing13:25
*** Lucas_Gray has quit IRC13:26
*** irclogbot_1 has joined #openstack-ironic13:26
TheJuliadtantsur: before I look, is crazier simpler?13:28
dtantsurTheJulia: not really. it's more or less a matter of picking one.13:28
*** rcernin has quit IRC13:29
TheJuliaaww :(13:29
* TheJulia sips coffee and tries to wake up13:29
dtantsurTheJulia: waking up is futile13:31
TheJuliaThe thought just makes me want to go back to bed13:31
dtantsur++ I tried waking up today, it was pretty bad. 3/10, would not recommend.13:32
iurygregoryits monday =(13:32
TheJuliaheh13:32
TheJuliaI feel like I should just go get a warm blanket, wrap myself in it, and go back to sleep13:33
iurygregorymaybe 3  mugs of coffee  will help13:33
*** Lucas_Gray has joined #openstack-ironic13:33
rpittauiurygregory: tried that, didn't work :/13:33
rpittaumaybe a fourth...13:33
iurygregoryi need more coffee to deal with the metrics im going crazy hehehe13:33
jrollI've been working on waking up for 4 hours now13:33
jrollvery little progress13:33
TheJuliaiurygregory: I encourage more craziness with metrics collection13:34
etingofiurygregory should beware of the coffee substance abuse13:35
iurygregoryTheJulia, i will need a light after the upstream meeting hehe XD i have different approaches13:35
dtantsuriurygregory, rpittau, enjoying the daylight switch? :)13:36
iurygregoryetingof, that's why i always drink coffee with milk =) and normally is more milk than coffee13:36
rpittaudtantsur: not really......13:36
iurygregorydtantsur, nope13:36
TheJuliaoh my13:36
TheJuliaSo yeah, you guys are really tired13:36
iurygregoryits not fun have sun after 18h '-'13:36
* etingof started his career on the milk shake13:37
dtantsuretingof: this one? https://www.instagram.com/milkshake.thecat/13:38
* TheJulia has a blank look on her fase13:38
TheJuliadtantsur: that is better than milkshake the song13:38
*** jroll has quit IRC13:39
*** jroll has joined #openstack-ironic13:39
* dtantsur has no clue13:40
* TheJulia just does her best zombie needs coffee impression13:41
etingofbaked coffee beans -> white zombie coffee13:47
openstackgerritJulia Kreger proposed openstack/ironic master: Remove excess stop_redfish_emulator call  https://review.openstack.org/64903013:49
dtantsurrloo: hi, do you have any context on https://storyboard.openstack.org/#!/story/1703434 ?13:50
*** mbeierl has quit IRC13:51
rloodtantsur: good morning. looking...13:51
*** diga has joined #openstack-ironic13:52
*** jcoufal has joined #openstack-ironic13:53
rloodtantsur: argh. totally forgot about that :-(13:54
rloodtantsur: should still be done.13:54
dtantsurrloo: okay, would you like to still be assigned?13:55
rloodtantsur: yes, i'll do it. unless someone else wants to do it. for trains. well, i guess the good thing is that no one asked for it.13:55
rloodtantsur: thx for noticing.13:56
dtantsurnp, doing re-triage of everything :)13:56
iurygregorymeeting? \o/14:00
jrollnewp14:00
jrolluione more hour :P14:00
iurygregory=O14:00
jroller, iurygregory: one more hour14:00
etingofiurygregory needs to ntpsync ;)14:00
*** baha has joined #openstack-ironic14:01
iurygregoryetingof, yep XD14:01
*** hjensas has joined #openstack-ironic14:01
*** mjturek has joined #openstack-ironic14:03
*** priteau has quit IRC14:03
*** e0ne has joined #openstack-ironic14:06
*** Lucas_Gray has quit IRC14:07
dtantsurTheJulia, rloo: I wonder if we can abandon https://storyboard.openstack.org/#!/story/1633756 now..14:10
rloodtantsur: yes. it is 'backburner' or whatever stuff, but i don't think we want to update our specs to include it. i think if the community decides to go that way, we'll do it but otherwise, very low priority.14:12
*** Lucas_Gray has joined #openstack-ironic14:30
*** rachit7 has quit IRC14:31
*** derekh has quit IRC14:40
*** derekh has joined #openstack-ironic14:40
* TheJulia can look in a little bit14:42
TheJuliaNobodyCam: https://review.openstack.org/#/c/648193/1 if you feel like a quick review14:42
patchbotpatch 648193 - bifrost - Pin proliantutils version - 1 patch set14:42
TheJuliaNobodyCam: Another quickie https://review.openstack.org/#/c/645518/214:43
patchbotpatch 645518 - bifrost - Add git_branch variable - 2 patch sets14:43
NobodyCamWill do. But need a few minutes for coffee to take effect14:45
NobodyCamAnd good morning Ironic’ers :)14:45
*** cdearborn has joined #openstack-ironic14:47
dtantsurTheJulia, NobodyCam, there's a discussion on the first patch, please check it14:47
TheJuliaHmm. changed my vote to +1. Hmmm....]14:50
openstackgerritDmitry Tantsur proposed openstack/ironic master: Do not try to create temporary URLs with zero lifetime  https://review.openstack.org/64905214:50
TheJuliaoh my14:50
*** stendulker has joined #openstack-ironic14:51
*** mjturek has quit IRC14:51
TheJuliadtantsur: I'm kind of wondering if we should even really track that. It was a contributor code cleanup idea, so unless someone really has the urge to make it happen, I fear it will languish14:54
*** mjturek has joined #openstack-ironic14:54
dtantsurthat's my thought as well14:54
TheJuliamarked it low hanging fruit, in case someone wishes to pick it up at some point. We can re-evaluate at some point as well.14:56
*** w14161_1 has joined #openstack-ironic14:58
dtantsur-25 untriaged RFE, a busy Monday morning :D14:58
TheJuliaI've been doing code reviews this morning. Crazy how fast last week went by14:58
jroll20 wriiten by dtantsur ?14:58
jroll:P14:58
dtantsur;)14:59
*** dnuka has joined #openstack-ironic14:59
dtantsurTheJulia: time flies in enjoyable meetings :-P14:59
TheJuliaFor those that might have not picked up on it, I spent the bulk of last week in meetings14:59
iurygregorymeetings everywhere15:00
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Apr  1 15:00:16 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
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
TheJuliaSpeaking of meetings! It is time for our weekly meeting!15:00
etingof\o/15:00
TheJuliaOur agenda, as always, can be found on the wiki.15:00
dtantsur\o15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
iurygregoryo/15:00
rpittauo/15:00
arne_wiebalcko/15:00
bdoddo/15:00
dnukao/15:00
TheJuliaThis week, it looks fairly lightweight, Hopefully this will be a quick meeting and then we can go back to taking over the world.15:01
stendulkero/15:01
TheJulia#topic Announcements / Reminders15:01
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:01
*** kaifeng has joined #openstack-ironic15:01
TheJulia#info The PTG topic etherpad is up for review. Please add any additional topics and discussion by Friday of this week.15:01
rpiosoo/15:02
kaifengo/15:02
jroll\o15:02
rlooo/15:02
TheJulia#info Next week, we shall review the PTG topic etherpad and add +1s to it in order to help form the basis of an agenda.15:02
TheJulia#link https://etherpad.openstack.org/p/DEN-train-ironic-brainstorming15:02
TheJuliaDoes anyone have anything else to add?15:02
dtantsuroh, it's in 4 weeks. time flies indeed15:02
dtantsurTheJulia: we should check if we need another round of stein releases, I guess?15:02
TheJuliaWe should, although I think we have a couple minor patches still in review first15:03
cdearborno/15:03
TheJuliaWe may also need to other stable branch releases as well15:03
TheJuliadtantsur: Thursday I think seems like a good day to check on all of that.15:03
dtantsurindeed15:04
dtantsurother stable branches need sorting out the iDRAC pxe_enabled issue15:04
TheJulia++15:04
TheJuliaI talked to rpioso friday afternoon ?or was it evening? about it15:04
TheJuliaIt seems like we have a forward path there, just waiting for new patches to get posted.15:05
rpiosodtantsur, TheJulia: We're working on it.15:05
TheJuliarpioso: Awesome, thanks!15:05
TheJuliaWell if there is no other reminders or announcements, We can skip forward to reviewing subteam status reports15:06
TheJuliaSince there were no action items last week.15:06
* TheJulia hears crickets and proceeds forward15:07
TheJulia#topic Review subteam status reports15:07
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:07
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:07
TheJuliaStarting at line 24015:08
*** priteau has joined #openstack-ironic15:08
TheJuliaPython3 wise, it seems largely out of date and seems to have creeped scope to include discussion about dropping 3.5 support15:09
TheJuliaI feel it is time to maybe transform that entry? thoughts?15:09
rloo++ transform15:09
dtantsuryeah15:09
rpittausounds good15:09
iurygregory++15:09
TheJuliaOkay! Note added!15:10
rlooTheJulia: do you or who knows the status of all those things listed under Python3 first?15:11
* jroll strikes through his items on there15:11
rloothx jroll :)15:11
rpittaurloo: everything shoule be good on Python 315:11
rpittaus/shoule/should15:11
TheJuliarloo: I think we're done with python3 first except maybe bifrost, I dont' remember there. Zuul job wise which got wrapped up in all of that we still have some stuff to migrate15:11
TheJuliabut minimal at this point15:11
rloorpittau: please strike/delete stuff there that is out of date then.15:11
TheJuliaUEFI wise, we should continue to push forward there. We do need the CI job coverage at some point, hopefully I'll have time one year.15:12
rpittauTheJulia: the one uefi job on bionic is working15:12
TheJulia\o/15:12
rlooTheJulia: and docs still needed for uefi?15:12
TheJuliaSo beyond that it is just additional scenarios15:13
TheJuliarloo: I believe we merged them15:13
TheJuliayup, doc updates merged15:14
*** w14161_1 has quit IRC15:14
TheJuliaStruck through out of date items there15:14
rpittauwe should move zuulv3 migration from python3 first15:14
TheJuliarpittau: Please15:14
rpittaudone :P15:14
TheJuliaRole splitting I think is a continuing topic15:15
TheJuliaDeploy templates, I believe mgoddard's current patches are all WIPs?15:15
TheJuliaarne_wiebalck: Ohh, nice idea w/r/t automatic cleaning :)15:16
TheJuliaThank you for updating software raid :)15:16
arne_wiebalckTheJulia: thx for the reviews15:16
TheJuliaFast track, I need to find time to work on testing15:16
TheJulia#action TheJulia to add links to fast track section15:16
TheJuliaSince you know... then I can actually say we had action items :)15:16
* rloo loves action items15:16
* arne_wiebalck loves action items on someone else15:17
TheJuliaFrom my reviewing this morning, etingof's virutal media patches are still in review15:17
TheJuliaAnd phase one of locking still requires reviews15:17
rlooarne_wiebalck: wrt software RAID (L299). hasn't the spec landed?15:18
TheJuliarloo: he revised the spec15:18
arne_wiebalckrloo: the main one yes15:18
TheJuliafor additional clarity and minor behavior change from code review discussions15:18
rlooarne_wiebalck: maybe add the link to the revision then.15:18
arne_wiebalckrloo: ok!15:18
TheJuliaI suspect smartnics should be able to move again on the neutron side soon-ish. I'll inquire with the Neutron PTL.15:19
rloowrt locking (L318). are there links to any docs or story or ??15:20
TheJulia#action TheJulia to follow-up with neutron folks regarding smartnic15:20
TheJuliarloo: locking discussion has been a bit less formal and kind of a prerequisite of other items so we've been trying to hash it out on code reviews. There is a item we copied over that jroll created ages ago in storyboard... I'd have to look but I believe it is linked on the patch.15:21
TheJuliahjensas: any update on Neutron event processing?15:21
rlooTheJulia: thx15:21
TheJuliaRegarding HTTPClient booting, I've had a few people express interest in this area over the past week, so I think people may be interested in helping with this item.15:22
hjensasTheJulia: I've put a hold on working that effort until after PTG.15:22
TheJuliahjensas: ack15:22
TheJuliaThe same along with the DHCP-less/L3 Vmedia boot15:22
TheJuliaI noticed some work has continued on graphical console support, so that entry is up to date.15:23
*** zhangfei has joined #openstack-ironic15:23
TheJuliajroll: any word on conductor->ipa comm flow changes?15:24
*** e0ne has quit IRC15:24
jrollTheJulia: no updates15:24
TheJuliajroll: And you will not be at the summit/ptg?15:24
jrollif anything, it's dropped lower on my priority list :(15:24
jrollTheJulia: correct15:24
TheJulia:(15:25
TheJuliaokay15:25
jrollI'll be working, can ping me if needed15:25
TheJuliaOkay, I might add a topic of larger scope that could be tied in or impact that... I need to think of the wording15:26
TheJuliaand the actual core idea, so I don't come off as the craziest of the crazy people15:26
TheJuliaIf so, I'll try and co-ordinate15:26
jrollsure, sounds good15:26
TheJuliaIs everyone good to proceed?15:27
rloo++15:28
rpittaulet's15:28
TheJulia#topic Deciding on priorities for the coming week15:28
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:28
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:28
TheJuliaStarting at line 13315:28
* TheJulia starts by removing merged15:28
TheJuliadtantsur: I'm good with adding the allocation backfill15:29
dtantsurcool, thx15:30
TheJuliaI think the list works15:30
TheJuliaIs there anything anyone else would like to see or have some people look at this week?15:30
kaifengI may help with the ipa-polling mode if that's in concern?15:31
TheJuliakaifeng: Would you find it beneficial to your use cases?15:32
kaifengyes, it's avoid direct access to api during deployment15:33
jrollkaifeng: you're more than welcome to take that on15:34
TheJuliaOkay, I believe you will also not be at the PTG, so perhaps we should possibly consider trying to schedule some time with jroll to discuss this a little fuirther15:34
jrollI'm always here if it needs to be discussed15:34
* TheJulia still needs to think of the crazy words and how to put them in the right order15:35
jrollthe spec should have everything, I hope :)15:35
kaifengTheJulia sigh, I am not there15:35
TheJulia:(15:35
dtantsurI think some team is having a pre-PTG. maybe it's a good idea?15:35
TheJuliaIt is life sadly15:35
TheJuliadtantsur: I think it is, lets see how discussion shapes up in the etherpad and go from there15:35
dtantsurI mean, a virtual pre-PTG15:36
dtantsurright15:36
kaifengjroll: thanks :)15:36
TheJuliaExcellent!15:36
TheJuliaOkay, I guess time for RFE review if there is nothing else on priorities for the coming week.15:36
TheJulia#topic RFE Review15:36
*** openstack changes topic to "RFE Review (Meeting topic: ironic)"15:36
* TheJulia hands the micrphone to dtantsur 15:36
TheJuliamicrophone15:36
dtantsuroh, me?15:38
TheJuliayes :)15:38
* TheJulia makes more coffee for dtantsur 15:38
dtantsur#link  https://storyboard.openstack.org/#!/story/2005328 Relax restrictions on network and storage interfaces15:38
dtantsuryes, more coffee15:38
dtantsurso, this was a request by IIRC eandersson15:38
dtantsurtl;dr make all network interfaces work with all hardware types (something we explicitly avoided back in the driver composition reform times)15:39
TheJuliaI think that is reasonable, the is_compatible_with hw type check might need some fun logic for the first cycle or so15:39
TheJuliaI'm +2 on the new proposal15:40
* rloo confused. remind me, what is the current restriction on network & storage interfaces?15:40
dtantsurTheJulia: we can leave 'return True' there, as this is currently the case15:41
jrollrloo: a hardware type hardcodes the supported interfaces15:41
dtantsurrloo: they have to be enabled by the hardware type, the same way as power, management, etc15:41
rloowhat does (in the rfe) 'which so far have been practically universal' mean?15:41
dtantsurrloo: meaning, I'm not aware of a mismatch between any hardware type and any network interface15:41
rloooh, so we don't want to hardcode for only network/storage interfaces?15:41
jrollcorrect15:42
dtantsurwell, at least for now. we can talk about everything else later.15:42
rloobut still hardcode for the other interfaces?15:42
dtantsuryes15:42
TheJuliaeandersson's case is he is maintaining a downstream driver interface for his networking, and prsently he has to create new hardware types if he wants to use it, but he wants to use stock hardware types and just load up his new driver with the configuration15:42
rloowhich could be a similar thing for any interface driver? if eg someone has a downstream driver for console? (i can't remember if we have a console interface)15:43
rlooanyway, i am +1 on the idea.15:43
rlooneed to read it all to be +2 :)15:44
*** e0ne has joined #openstack-ironic15:44
TheJuliathat might be reasonable, maybe do this and then see how it goes because operators are more likely to do such things if they are empowered15:44
TheJuliaotherwise we're kind of handcuffing them15:44
rpiosoHow might this affect in-tree vendor h/w types?15:44
TheJuliait really won't affect in-tree h/w types15:44
jrollrpioso: someone may use the in-tree h/w types with out of tree storage/network interfaces15:45
jrollotherwise, no effect15:45
dtantsurI'd defer discussion other interfaces until the PTG, but the storage/network seem easy15:45
*** tssurya has quit IRC15:45
jroll++15:45
rlooi wonder if 'ANY' is the right word but maybe i'm shed biking.15:45
*** Wryhder has joined #openstack-ironic15:46
TheJuliadtantsur: ++15:46
dtantsurI'm open to suggestions :) this was based on mock.ANY15:46
TheJuliasounds like we have RFE-approved15:46
rloowe don't allow any, we allow compatible right? (any compatible)15:46
TheJuliathat was my interpretation15:46
* rpioso is unclear on what supported_xxx_interfaces means?15:46
TheJuliaIf there are no objections, we can proceed to Open Discussion15:47
*** e0ne has quit IRC15:47
*** Lucas_Gray has quit IRC15:47
dtantsurrpioso: the properties on a HardwareType that regulate which interfaces are compatible with it15:47
*** Wryhder is now known as Lucas_Gray15:47
dtantsurrpioso: like here https://github.com/openstack/ironic/blob/master/ironic/drivers/drac.py#L38-L4615:47
rpiosodtantsur: Wouldn't the RFE alter that concept?15:47
TheJuliaheh, dtantsur beat me with the link15:47
* rpioso is familiar with the drac h/w type15:48
TheJuliaChanging to Open Discussion15:48
*** priteau has quit IRC15:48
TheJulia#topic Open Discussion15:48
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:48
TheJuliaWhile the other discussion continues, does anyone have anything specific they would like to raise today?15:48
jrollrpioso: yes, it alters that concept, in that someone may now use the in-tree h/w types with out of tree storage/network interfaces15:48
rpiosojroll: Thank you for answering my question :-)15:49
* dtantsur wonders when we should start thinking about a get-together at the PTG15:49
jrollrpioso: :)15:49
TheJuliadtantsur: Hmm, yes. And we have more options not being at the old PTG venue in Stapleton, CO15:50
rloodtantsur: so we'd leave the current value of eg supported_network_interface as is?15:50
dtantsurrloo: "Finally, update GenericHardware to add ANY for supported_storage_interfaces and supported_network_interfaces, thus making them universal."15:50
dtantsurso no, add ANY (or whatever name we use) to them15:51
* arne_wiebalck now gets what TheJulia was asking him the other day about network/storage interfaces15:51
rloodtantsur: thx, didn't read that far. i don't have enough time/thoughts to grok all the details of this now, so i will rely on the rest of you that are faster than me, to decide.15:52
rpittaua couple of tests in ocata are failing for different reasons and wondering if we should fix them, they're in the patches related to the migration to gitea15:52
openstackgerritMerged openstack/bifrost master: Add git_branch variable  https://review.openstack.org/64551815:52
dtantsurrpittau: it's a good question, ocata should have gone into extended maintenance already15:52
rpiosodtanstur: I had the impression that the supported_xxx_interfaces declares, well, what's supported. Declaring ANY seems inconsistent with that concept.15:53
rpittaudtantsur: yep, that's why I'm asking :/15:53
rpittauexample: https://review.openstack.org/64639215:53
patchbotpatch 646392 - ironic (stable/ocata) - Replace openstack.org git:// URLs with https:// - 1 patch set15:53
TheJuliarpittau: we likely should fix them15:54
dtantsurrpioso: well, that's the whole point here. what if a hardware type supports any network interfaces (which is the case of all our hardware types currently)?15:54
* rloo wonders whether it makes sense for eg supported_network_interfaces to be eg 'flat_net.FlatNetwork, neutron.NeutronNetwork, ANY', so default can be explicitly specified.15:54
rloo^^ would be up to a specific hardware_type, in case they want to be explicit/suggest a default.15:55
dtantsurTheJulia, rpittau, fixing the coverage job requires bumping test-requirements/upper-constraints or whatever15:55
rpittauTheJulia: ok, I'll have a look then this week, shouldn't take too much time15:55
rpittaudtantsur: yeah, it's the psycopg version15:55
dtantsurrloo: that's exactly my plan15:55
dtantsur(unless I misunderstood you)15:55
jrolldtantsur: rpittau: coverage just runs tox-py27 and generates coverage. since py27 passed, it's probably just a problem with the pip cache15:55
jroll(afaik)15:56
rpittaujroll: it can be reproduced locally though15:56
jrollugh15:56
dtantsurrpittau: oh, it may be that we don't use upper-constraints correctly15:56
rloodtantsur: guess it isn't clear to me, skimming the rfe. as long as that's your plan.15:56
dtantsurfor some long time we had a not-quite-valid approach15:56
TheJuliaYeah, we had to fix it once before for one of the jobs15:57
rloorpioso: does that address your concerns (if i understand your concerns, if you have any concerns)15:57
rloorpioso, dtantsur: can i assume that if rpioso's HW type doesn't want to support ANY, that he can choose to have his hwtype not support ANY?15:57
rpiosorloo: Yes, it does. Thank you!15:57
dtantsurrloo: yes15:57
rpittaubtw there's one for rocky that has already +2 if anyone has 32.7 seconds https://review.openstack.org/64769315:58
patchbotpatch 647693 - networking-generic-switch (stable/rocky) - Fix pep8 test - 1 patch set15:58
TheJulia3 minute warning15:58
rloogood, then everyone is happy this Monday :)15:58
* dtantsur wonders how rpittau calculated that15:58
rajinirdevstack install fails on install_ironic15:59
rajinirE: Package 'ovmf' has no installation candidate15:59
TheJuliarajinir: where did you observe this?15:59
rpittauoO15:59
TheJuliaSome packagers might be using different names15:59
rajinirDell Thirdparty CI is failing on UEFI boot mode15:59
TheJuliarajinir: Example link?16:00
TheJuliaI guess that can be skipped in your CI since your using real hardware16:00
TheJuliaovmf is for virtual machines only16:00
rajinirlogs:  https://stash.dellemc-community.org/logs/38/648938/2/check/dell-hw14G-UEFI-tempest-dsvm-ironic-idrac/911daec//16:00
rpittauovmf is loading from xenial mutliverse, might be the link is not accessible16:00
TheJuliaThanks everyone!16:00
NobodyCam:)16:00
rpiosoTheJulia: Thank you16:01
rajinirrpittau: may be16:01
TheJuliarajinir: That is also a Xenial build, you should likely switch to bionic16:01
rajinirrpittau: TheJulia: This is new16:01
rpittaurajinir: that looks outdated16:02
TheJulia#endmeeting16:02
*** 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:02
openstackMeeting ended Mon Apr  1 16:02:39 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:02
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-04-01-15.00.html16:02
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-04-01-15.00.txt16:02
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-04-01-15.00.log.html16:02
*** dnuka has quit IRC16:02
rajinirrpittau>  I meant this is new failure, may be someone checked in something in devstack install_ironic that broke the builds for hardwares16:03
*** diga has quit IRC16:03
*** lucasagomes has quit IRC16:04
TheJuliarajinir: We switched to Bionic which I think has different package namings for Xenial. CI should be switching to Bionic16:04
rpittaurajinir: TheJulia: that log comes from a WIP16:05
rpittauI was testing the new package16:05
rpittauof course it doesn't work16:05
rpittauhttps://review.openstack.org/64893816:05
patchbotpatch 648938 - ironic - [WIP|DNM] testing uefi job with ubuntu bionic - 2 patch sets16:05
*** dtantsur is now known as dtantsur|afk16:07
dtantsur|afk\o16:07
rajinir<rpittau,TheJulia what should we do? I'm a bit lost16:09
arne_wiebalckTheJulia: Thx for your reviews on the s/w RAID patches, will go through the details tomorrow.16:09
rpittaurajinir: the error you're seeing is totally normal, it comes from a test job that is not merged and it will not be merged16:09
rpittaurajinir: what you should do in general is migrating the CI to ubuntu bionic16:10
arne_wiebalckTheJulia: Concerning the spec: this was mostly to be clear on what was considered when I wrote the spec.16:10
arne_wiebalckTheJulia: And I did not consider/test partition images.16:10
arne_wiebalckTheJulia: And I simply doubt they would work out of the box :)16:11
rajinir<rpittau,TheJulia ok, thanks. Will try and migrate to ubuntu bionic.  Will have to figure out how16:12
TheJuliaarne_wiebalck: oh! I thought it was the other way around16:13
arne_wiebalckTheJulia: ?16:13
rpittaugood night! o/16:13
TheJuliarajinir: you may want to ask in #openstack-infra to see if they can shed light on their configuration changes16:13
*** rpittau is now known as rpittau|afk16:13
TheJuliaso you can replicate them in your CI16:13
*** Lucas_Gray has quit IRC16:13
rajinirTheJulia: yes, planning to do that. Thank you16:13
TheJuliaarne_wiebalck: that it was just partition images not whole disk iamges16:13
* TheJulia goes and cooks breakfast16:13
arne_wiebalckTheJulia: Whole disk images would have grub in the wrong place.16:14
TheJuliaIneed16:14
TheJuliaindeed16:14
* TheJulia senses deeper confusion and adds coffee to the todo list16:14
arne_wiebalckTheJulia: The proposed code hence explicitly installs the bootloader if whole_disk_image AND sw_RAID.16:14
TheJuliaahh16:15
TheJuliaWell the bootable device change is a little more worrisome, considering it can vary with GPT. I guess that is the next logical iteration16:15
* TheJulia really really really locks screen and goes to the kitchen16:16
* arne_wiebalck does not want to get in between TheJulia and her breaksfast16:16
arne_wiebalckTheJulia: Yes, that's why I mention GPT as well: not tested yet, so almost certainly broken.16:17
arne_wiebalckTheJulia: Sth to look at. But we only started last week into GPT images for our cloud.16:17
arne_wiebalck TheJulia: Wasn't required so far.16:17
arne_wiebalckTheJulia: With GPT in the image, the RAID  patches may need some adaptation.16:18
*** kaifeng has quit IRC16:18
arne_wiebalckTheJulia: Let me know if you have a suggestion how to make all this more clear in the spec.16:19
*** zhangfei has quit IRC16:20
arne_wiebalckbye all o/16:21
*** dsneddon has quit IRC16:24
TheJuliammmmmmm coffeeeeee16:24
*** stendulker has quit IRC16:24
TheJuliaarne_wiebalck: I'd have to re-read the spec to come up with suggestions,a nd it might jsut be the way Im reading things today16:25
arne_wiebalckTheJulia: Well, it was obviously confusing if it confused you.16:25
*** baha has quit IRC16:25
arne_wiebalckTheJulia: Anyway, have another look (if you have a moment), and let me know.16:26
* arne_wiebalck really needs to go now o/16:26
TheJuliaarne_wiebalck: goodnight :)16:26
*** hwoarang has quit IRC16:31
*** hwoarang has joined #openstack-ironic16:36
*** e0ne has joined #openstack-ironic16:45
*** dsneddon has joined #openstack-ironic16:49
*** trown is now known as trown|lunch16:49
*** andrein has quit IRC16:49
*** baha has joined #openstack-ironic16:58
eanderssondtantsur|afk, yep found it in a few projects and no one seems to be using it17:00
eanderssonprobably something that just carried over17:01
*** derekh has quit IRC17:02
*** gkadam has quit IRC17:05
*** dsneddon has quit IRC17:06
openstackgerritIlya Etingof proposed openstack/ironic master: Run IPMI, SNMP and Redfish BMC emulators as stack  https://review.openstack.org/64849617:13
*** e0ne has quit IRC17:35
*** zigo has quit IRC17:37
*** S4ren has quit IRC17:38
*** dsneddon has joined #openstack-ironic17:47
*** SASAA has joined #openstack-ironic17:57
*** e0ne has joined #openstack-ironic17:57
*** trown|lunch is now known as trown18:04
*** MattMan_1 has quit IRC18:16
*** MattMan_1 has joined #openstack-ironic18:16
*** andrein has joined #openstack-ironic18:20
*** e0ne has quit IRC18:28
openstackgerritIlya Etingof proposed openstack/ironic master: Improve VirtualBMC use in Devstack  https://review.openstack.org/64851018:49
*** e0ne has joined #openstack-ironic19:27
*** jcoufal has quit IRC19:45
openstackgerritMark Goddard proposed openstack/bifrost master: Pin proliantutils version  https://review.openstack.org/64819319:53
*** dsneddon has quit IRC20:03
*** ijw has joined #openstack-ironic20:07
openstackgerritMerged openstack/bifrost master: Add defaults for Ubuntu 18.04  https://review.openstack.org/64581320:14
*** dsneddon has joined #openstack-ironic20:21
*** rh-jelabarre has quit IRC20:22
*** rh-jelabarre has joined #openstack-ironic20:26
*** dsneddon has quit IRC20:27
*** dsneddon has joined #openstack-ironic20:29
*** whoami-rajat has quit IRC20:30
*** dsneddon has quit IRC20:34
*** trown is now known as trown|outtypewww20:42
*** mjturek has quit IRC20:45
*** dsneddon has joined #openstack-ironic20:48
*** e0ne has quit IRC21:03
*** pcaruana has quit IRC21:17
*** bfournie has quit IRC21:27
*** baha has quit IRC21:29
*** samueldmq has joined #openstack-ironic21:36
*** SASAA has quit IRC21:56
*** munimeha1 has quit IRC22:01
*** iurygregory has quit IRC22:09
openstackgerritJulia Kreger proposed openstack/ironic master: Make it possible to send sensor data for all nodes  https://review.openstack.org/64872822:27
*** betherly has joined #openstack-ironic22:34
*** betherly has quit IRC22:39
*** rcernin has joined #openstack-ironic22:40
*** rh-jelabarre has quit IRC22:55
*** hwoarang has quit IRC23:35
*** hwoarang has joined #openstack-ironic23:37
openstackgerritUros Orozel proposed openstack/ironic-python-agent master: Ironic python agent does not extract correct available memory  https://review.openstack.org/64809623:49
*** rloo has quit IRC23:50
openstackgerritUros Orozel proposed openstack/ironic-python-agent master: Ironic python agent does not extract correct available memory  https://review.openstack.org/64809623:52
*** andrein has quit IRC23:55
*** openstackgerrit has quit IRC23:56
*** antonym has quit IRC23:58

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