Wednesday, 2017-09-27

*** causten has joined #openstack-ironic00:00
*** penick has quit IRC00:02
*** causten has quit IRC00:05
*** rama_y has quit IRC00:05
*** bfournie has joined #openstack-ironic00:09
*** yangyapeng has quit IRC00:11
*** bfournie has quit IRC00:12
*** yangyapeng has joined #openstack-ironic00:12
*** yufei has joined #openstack-ironic00:18
*** yufei has quit IRC00:18
*** [1]rpioso has quit IRC00:25
*** hw_wutianwei1 has joined #openstack-ironic00:54
*** phuongnh has joined #openstack-ironic01:03
*** Sukhdev has quit IRC01:03
*** litao__ has joined #openstack-ironic01:06
*** stendulker has joined #openstack-ironic01:12
*** leyal has quit IRC01:31
*** lihi has quit IRC01:33
*** leyal has joined #openstack-ironic01:33
*** lihi has joined #openstack-ironic01:33
*** ijw has quit IRC01:33
*** causten has joined #openstack-ironic01:40
*** crushil has joined #openstack-ironic01:46
*** stendulker has quit IRC02:15
*** srobert has joined #openstack-ironic02:20
*** rama_y has joined #openstack-ironic02:38
*** rama_y has quit IRC02:42
*** ijw has joined #openstack-ironic03:12
*** ijw has quit IRC03:17
*** TuanLA has joined #openstack-ironic03:21
*** yangyapeng has quit IRC03:32
*** yangyapeng has joined #openstack-ironic03:32
*** yangyapeng has quit IRC03:37
*** mat128 has joined #openstack-ironic03:37
*** stendulker has joined #openstack-ironic03:39
*** mat128 has quit IRC03:41
*** Sukhdev has joined #openstack-ironic03:47
*** crushil has quit IRC03:57
*** krtaylor has quit IRC04:00
*** krtaylor has joined #openstack-ironic04:02
*** krtaylor has quit IRC04:08
*** ianychoi_ has joined #openstack-ironic04:20
*** stendulker_ has joined #openstack-ironic04:22
*** sasha21 has joined #openstack-ironic04:23
*** Sukhdev has quit IRC04:29
*** stendulker has quit IRC04:29
*** sasha2 has quit IRC04:29
*** yolanda has quit IRC04:29
*** ianychoi has quit IRC04:29
*** jrist has quit IRC04:29
*** leifmadsen has quit IRC04:29
*** jcook has quit IRC04:29
*** sambetts|afk has quit IRC04:29
*** lhinds has quit IRC04:29
*** honza has quit IRC04:29
*** honza has joined #openstack-ironic04:29
*** honza is now known as Guest8588804:30
*** sambetts_ has joined #openstack-ironic04:30
*** lhinds has joined #openstack-ironic04:30
*** leifmadsen has joined #openstack-ironic04:32
*** jcook has joined #openstack-ironic04:32
*** radez has quit IRC04:33
*** slagle has quit IRC04:34
*** trown|outtypewww has quit IRC04:34
*** crushil has joined #openstack-ironic04:34
*** trown has joined #openstack-ironic04:35
*** udesale has joined #openstack-ironic04:36
*** jrist has joined #openstack-ironic04:36
*** yolanda has joined #openstack-ironic04:37
*** racedo has quit IRC04:37
*** dpawar has joined #openstack-ironic04:39
*** radez has joined #openstack-ironic04:39
*** slagle has joined #openstack-ironic04:41
*** ankkumar has joined #openstack-ironic04:41
*** dpawar has quit IRC04:44
*** dpawar has joined #openstack-ironic04:45
*** jaianshu has joined #openstack-ironic04:54
openstackgerritMerged openstack/ironic-specs master: Move completed specs to pike 9.0 & 9.1  https://review.openstack.org/50536504:56
openstackgerritMerged openstack/ironic master: Add documentation describing each Ironic state  https://review.openstack.org/49671904:59
*** marios has joined #openstack-ironic05:12
*** causten has quit IRC05:13
*** phuongnh has quit IRC05:19
*** crushil has quit IRC05:21
*** aparnav has joined #openstack-ironic05:30
*** vsaienk0 has joined #openstack-ironic05:41
openstackgerritzenghui.shi proposed openstack/ironic-specs master: Hardware interface for BIOS configuration  https://review.openstack.org/49648105:42
*** radez has quit IRC05:52
*** ChubYann has quit IRC05:57
*** mjura has joined #openstack-ironic06:04
openstackgerritVladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode  https://review.openstack.org/35906106:09
*** jtomasek has quit IRC06:18
*** vsaienk0 has quit IRC06:18
*** vsaienk0 has joined #openstack-ironic06:19
*** fbouliane has quit IRC06:19
*** zshi__ has joined #openstack-ironic06:20
*** fbouliane has joined #openstack-ironic06:22
*** dpawar has quit IRC06:34
openstackgerritVasyl Saienko proposed openstack/ironic master: Add uWSGI support  https://review.openstack.org/50701106:38
openstackgerritVasyl Saienko proposed openstack/ironic master: [devstack] Switch ironic to uWSGI  https://review.openstack.org/50706706:38
*** rcernin has joined #openstack-ironic06:39
*** e0ne has joined #openstack-ironic06:40
*** hjensas has joined #openstack-ironic06:40
*** hjensas has quit IRC06:40
*** hjensas has joined #openstack-ironic06:40
*** hjensas has quit IRC06:41
openstackgerritVasyl Saienko proposed openstack/ironic master: Add uWSGI support  https://review.openstack.org/50701106:43
openstackgerritVasyl Saienko proposed openstack/ironic master: [devstack] Switch ironic to uWSGI  https://review.openstack.org/50706706:43
*** charcol has quit IRC06:50
*** dpawar has joined #openstack-ironic06:51
*** e0ne has quit IRC07:08
*** jaianshu has quit IRC07:11
*** causten has joined #openstack-ironic07:14
*** Dinesh_Bhor has quit IRC07:17
*** phuongnh has joined #openstack-ironic07:18
*** causten has quit IRC07:18
*** rbartal has joined #openstack-ironic07:19
*** Dinesh_Bhor has joined #openstack-ironic07:21
*** tesseract has joined #openstack-ironic07:21
pas-hamorning ironic :)07:28
*** hjensas has joined #openstack-ironic07:33
*** hjensas has quit IRC07:33
*** hjensas has joined #openstack-ironic07:33
*** mat128 has joined #openstack-ironic07:38
*** zshi has quit IRC07:39
*** mrch has joined #openstack-ironic07:42
*** mat128 has quit IRC07:44
*** ijw has joined #openstack-ironic07:47
etingofmorning pas-ha ;)07:54
*** jaganathan has joined #openstack-ironic07:57
*** ccamacho has joined #openstack-ironic07:57
openstackgerritVladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode  https://review.openstack.org/35906108:01
*** ijw has quit IRC08:02
*** ralonsoh has joined #openstack-ironic08:04
openstackgerritTuan Luong-Anh proposed openstack/ironic master: Secure boot support for irmc-virtual-media driver  https://review.openstack.org/50778008:08
*** hjensas has quit IRC08:12
*** makowals has quit IRC08:17
*** hjensas has joined #openstack-ironic08:21
*** athomas has quit IRC08:23
*** dpawar has quit IRC08:24
*** athomas has joined #openstack-ironic08:27
*** baoli has joined #openstack-ironic08:28
*** makowals has joined #openstack-ironic08:32
*** baoli has quit IRC08:32
*** derekh has joined #openstack-ironic08:47
*** chlong has quit IRC08:49
*** zshi has joined #openstack-ironic08:50
openstackgerritzenghui.shi proposed openstack/ironic master: WIP: Add BIOSInterface for BIOS configuration  https://review.openstack.org/50779308:50
*** etingof has quit IRC08:57
*** udesale has quit IRC08:57
*** udesale has joined #openstack-ironic08:57
*** amarao has joined #openstack-ironic08:59
*** jtomasek has joined #openstack-ironic09:01
*** zshi__ has quit IRC09:01
*** makowals has quit IRC09:07
*** etingof has joined #openstack-ironic09:11
*** e0ne has joined #openstack-ironic09:16
*** makowals has joined #openstack-ironic09:16
*** etingof has quit IRC09:19
*** Guest85888 is now known as honza09:22
*** dpawar has joined #openstack-ironic09:26
*** dpawar has quit IRC09:27
*** sambetts_ is now known as sambetts09:27
sambettsMorning all09:27
*** etingof has joined #openstack-ironic09:32
*** jtomasek has quit IRC09:38
*** jaganathan has quit IRC09:41
*** jaianshu has joined #openstack-ironic09:46
*** afazekas is now known as afazekas|seek4fo09:48
*** dpawar has joined #openstack-ironic09:51
*** d0ugal has joined #openstack-ironic10:03
*** TuanLA has quit IRC10:06
pas-hamorning sambetts10:06
sambettshey pas-ha10:10
openstackgerritVladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode  https://review.openstack.org/35906110:10
*** hw_wutianwei1 has quit IRC10:27
*** jkilpatr has quit IRC10:34
*** syqian has joined #openstack-ironic10:43
*** ankkumar has quit IRC10:47
*** d0ugal has quit IRC10:48
syqianHi,everyone. is anyone know whether the ironic support to inject the root password for the baremetal server?10:50
*** stendulker has joined #openstack-ironic10:51
*** udesale has quit IRC10:52
*** racedo has joined #openstack-ironic10:52
*** stendulker_ has quit IRC10:54
*** rbudden has joined #openstack-ironic10:54
*** milan has joined #openstack-ironic10:54
*** amarao1 has joined #openstack-ironic10:55
*** stendulker has quit IRC10:56
*** syqian has quit IRC10:57
*** d0ugal has joined #openstack-ironic11:02
*** jkilpatr has joined #openstack-ironic11:02
*** jkilpatr has quit IRC11:07
*** vmud213 has joined #openstack-ironic11:08
aparnavhey vdrok, Can you take look at this patch https://review.openstack.org/#/c/422572/?11:11
patchbotpatch 422572 - ironic - Support SUM based firmware update as clean step fo...11:11
*** syqian has joined #openstack-ironic11:11
sambettssyqian: it is possble via the config drive and cloud-init https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/4/html/End_User_Guide/config-drive.html11:13
syqiansambetts, ok, i will search it, thank you11:18
*** phuongnh has quit IRC11:18
*** jkilpatr has joined #openstack-ironic11:21
*** strigazi has quit IRC11:23
*** strigazi has joined #openstack-ironic11:24
*** fellypefca-afk is now known as fellypefca11:25
*** pcaruana has joined #openstack-ironic11:27
*** udesale has joined #openstack-ironic11:28
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add policy enforcement engine  https://review.openstack.org/50782611:31
fellypefcaGood morning everyone :)11:32
fellypefcarajinir hey, i will discuss with my team about the time slot of  3rd Party CI and answer you ASAP. But I Think that 16:00 UTC it's ok for us11:33
pas-haI kind of suspect 16UTC might not fit IRMC/Fujitsu team (AFAIU most of them are based in Japan)11:35
fellypefcapas-ha Oh! yeah, you're right. 16:00UTC it's midnight in Japan I guess :/11:39
*** mat128 has joined #openstack-ironic11:40
*** jtomasek has joined #openstack-ironic11:41
*** syqian_ has joined #openstack-ironic11:45
*** mat128 has quit IRC11:45
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add policy enforcement engine  https://review.openstack.org/50782611:46
*** syqian has quit IRC11:47
*** syqian_ has quit IRC11:49
*** syqian_ has joined #openstack-ironic11:50
*** dprince has joined #openstack-ironic11:56
*** syqian__ has joined #openstack-ironic11:58
*** syqian_ has quit IRC11:58
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/ironic master: Add default configuration files to data_files  https://review.openstack.org/50619812:02
*** mat128 has joined #openstack-ironic12:03
*** bfournie has joined #openstack-ironic12:03
*** litao__ has quit IRC12:04
*** trown is now known as trown|outtypewww12:05
*** trown|outtypewww is now known as trown|afk12:05
*** mat128 has quit IRC12:06
*** mat128 has joined #openstack-ironic12:08
*** mat128 has quit IRC12:08
*** mat128 has joined #openstack-ironic12:08
*** afazekas|seek4fo is now known as afazekas12:09
*** dpawar has quit IRC12:10
*** jlabarre has joined #openstack-ironic12:11
*** amarao1 has quit IRC12:12
openstackgerritFellype Cavalcante proposed openstack/ironic stable/ocata: Fix multitenancy scenario on OneView Driver  https://review.openstack.org/50763212:12
*** dtantsur|afk is now known as dtantsur12:16
dtantsurmorning ironic12:16
fellypefcaGood morning dtantsur :)12:17
openstackgerritJesse Pretorius (odyssey4me) proposed openstack/ironic master: Add default configuration files to data_files  https://review.openstack.org/50619812:18
*** amarao1 has joined #openstack-ironic12:23
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add policy enforcement engine  https://review.openstack.org/50782612:27
openstackgerritvinay kumar muddu proposed openstack/ironic master: Update proliantutils version for master to "2.4.1"  https://review.openstack.org/50550612:30
fellypefcadtantsur can you take a look? I need you +2 back :) https://review.openstack.org/#/c/502072/12:31
patchbotpatch 502072 - ironic - Documentation for 'oneview' hardware type12:31
*** aparnav has quit IRC12:33
*** yangyapeng has joined #openstack-ironic12:34
*** yangyapeng has quit IRC12:38
*** vmud213 has quit IRC12:38
*** hw_wutianwei1 has joined #openstack-ironic12:42
TheJuliaGood morning12:44
*** gcb has quit IRC12:45
fellypefcaGood morning TheJulia and thak you :)12:47
fellypefcathank* ops12:49
*** jcoufal has joined #openstack-ironic12:51
*** mat128 has quit IRC12:53
openstackgerritMerged openstack/ironic master: Documentation for 'oneview' hardware type  https://review.openstack.org/50207212:53
TheJuliafellypefca: I took care of you :)12:53
fellypefcaTheJulia :D <312:54
dtantsursorry, was being coffeinated :)12:54
TheJuliahehe12:55
TheJuliaI'm getting an early start12:55
TheJuliaand caffeinated at the same time12:55
dtantsurwe had a farewell party yesterday. I woke up at 12:30pm :D12:55
TheJuliaouch :)12:56
fellypefcadtantsur Is it your farewell?12:56
*** dpawar has joined #openstack-ironic12:56
fellypefcaI heard you're moving to Germany12:57
dtantsurfellypefca: yep :)12:57
dtantsurso I was saying bye to local folks12:57
fellypefcadtantsur nice! Oktoberfest o/12:58
dtantsurgot people some free beers, etc :)12:58
TheJuliadtantsur: before I dig into it, the current state of the refarch docs, is that representative of what we discussed at the ptg?12:59
dtantsurTheJulia: I only have the introduction patch up, and it should be up-to-date12:59
dtantsurthe only thing, maybe we want to shrink it a bit, and move some bits to related install guide sections13:00
dtantsurI'll leave it up for reviewers to tell me ;)13:00
*** rloo has joined #openstack-ironic13:00
TheJuliadtantsur: okay, I'm just hoping we get use cases landed soon-ish so I can go point to it in our documentation ;)13:01
TheJuliagood morning rloo13:01
dtantsuryeah, just lemme relocate first :)13:01
jaianshuHello everyone, TheJulia13:01
TheJuliaGood morning13:01
jaianshui have a query w.r.t https://bugs.launchpad.net/ironic/+bug/171632913:01
openstackLaunchpad bug 1716329 in Ironic "[RFE] HPE ilo OOB RAID configuration" [Wishlist,New] - Assigned to Anshul Jain (theanshuljain)13:01
rloohelloooooo and good morning TheJulia, dtantsur, jaianshu13:01
TheJuliajaianshu: sure!13:01
jaianshuactullay here i'm not implementing a new interface13:02
dtantsurmorning rloo, jaianshu13:02
jaianshui'm planning to use existing  RAIDInterface of base class13:02
jaianshuand writing a new olo raid interface13:03
jaianshuwill it require spec?13:03
dtantsurusually no, but there one interesting caveat13:03
dtantsur* there is13:03
jaianshu*ilo raid interface13:03
TheJuliajaianshu: Yes, but if your going to create a new class that will be used as an interface to for raid calls... That begins to enter that territory13:03
dtantsurif your RAID interface only targets a subset of hardware that the iLO hardware type targets, you may need a new hardware type..13:04
jaianshuyes..it will be a new hardware type..13:04
dtantsurwith this in mind, a spec probably won't hurt..13:05
dtantsurTheJulia: wdyt ^^13:05
TheJuliajaianshu: questions like this are why I indicated a spec, since the spec works you through a number of questions we would normally ask/look for during reviewing code. Asking them up front helps in the long run13:05
jaianshusomething like : class IloRAID(base.RAIDInterface):13:06
dtantsurjaianshu: yes, but also a new hardware type (aka driver). class iLOGen5Hardware(iLOHardware) (or something)13:06
dtantsurs/5/10/ but you got the idea13:06
jaianshuyes13:06
dtantsurbecause the ilo hardware type also supports gen8 and 913:07
jaianshusure dtantsur13:08
jaianshuthanks TheJulia, dtantsur..will do that13:08
TheJuliajaianshu: thanks!13:08
*** syqian__ has quit IRC13:10
*** syqian__ has joined #openstack-ironic13:11
dtantsurthanks!13:11
*** jistr is now known as jistr|call13:12
*** jaianshu has quit IRC13:12
*** krtaylor has joined #openstack-ironic13:13
*** syqian__ has quit IRC13:14
*** syqian__ has joined #openstack-ironic13:15
*** cdearborn_ has joined #openstack-ironic13:18
*** trown|afk is now known as trown13:18
*** hw_wutianwei1 has quit IRC13:18
*** cdearborn__ has joined #openstack-ironic13:19
*** cdearborn_ has quit IRC13:22
*** baoli has joined #openstack-ironic13:26
*** chlong has joined #openstack-ironic13:29
*** chlong has quit IRC13:31
*** causten has joined #openstack-ironic13:33
TheJuliaOkay, I'm going to switch laptops and try and get this spec done13:34
dtantsurwhich one?13:34
TheJuliacleanhold which actually should only be like ~40 lines of code and state transitions13:35
TheJuliaoh, and tests13:35
dtantsur sweet!13:35
*** syqian__ has quit IRC13:39
sambettsnice!13:41
openstackgerritMerged openstack/ironic-specs master: Proposal for Queens cycle priorities  https://review.openstack.org/50517313:42
rloobtw TheJulia, did you get around to talking to Interop about deprecation of ironic CLI? Wanted to know if we are good to go ahead with it.13:42
openstackgerritIlya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup  https://review.openstack.org/50785813:42
TheJuliaI ended up double checking the opinion of hogepodge who was leading the discussions at the PTG and he is all for it since it improves consistency.13:43
TheJuliarloo: tl;dr deprecate it! ;)13:43
dtantsur\o/13:44
dtantsuralso \o/ at merged priorities13:44
TheJuliawoot13:44
rlooTheJulia: awesome, thx!13:44
rloodtantsur: i know you want to do a python-ironicclient release soon. was thinking the deprecation could be part of it, but not sure. when were you thinking of doing the release?13:45
dtantsurrloo: after we get vdrok's patch in13:45
rloodtantsur: i believe you wanted vdrok's none auth patch (on my list to do today)13:45
dtantsurwhich does give us some time13:45
rloodtantsur: oh. so i shouldn't review vdrok's patch today then, i should work on deprecation? :)13:46
dtantsurrloo: both, ideally ;)13:46
*** lifeless has quit IRC13:46
dtantsurI won't be able to request a release until next Monday13:46
rloodtantsur: heh, ok, will see how far i get.13:46
rloodtantsur: ah, ok, then i have some time :)13:47
dtantsuryep13:47
hogepodgeTheJulia: +113:48
*** dpawar has quit IRC13:52
*** dpawar has joined #openstack-ironic13:52
*** crushil has joined #openstack-ironic13:53
dtantsurgreat, thanks hogepodge13:54
*** lifeless has joined #openstack-ironic13:54
*** udesale__ has joined #openstack-ironic13:55
*** dpawar_ has joined #openstack-ironic13:55
*** udesale has quit IRC13:56
*** dpawar has quit IRC13:57
*** baoli has quit IRC13:58
*** baoli has joined #openstack-ironic13:59
*** dpawar_ has quit IRC14:00
*** cdearborn__ is now known as cdearborn14:01
*** mat128 has joined #openstack-ironic14:12
*** Goneri has joined #openstack-ironic14:15
*** vsaienk0 has quit IRC14:16
*** alex_xu has quit IRC14:18
*** srobert has quit IRC14:19
*** alex_xu has joined #openstack-ironic14:21
*** mat128 has quit IRC14:23
*** jistr|call is now known as jistr14:27
openstackgerritIlya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup  https://review.openstack.org/50785814:29
dtantsureveryone, the priorities on the whiteboard were updated to match what we agreed on14:29
dtantsurplease provide your status updates14:29
dtantsurI tried to capture what is happening now to my best knowledge14:29
sambettsawesome :D14:29
*** vsaienk0 has joined #openstack-ironic14:33
*** livelace2 has joined #openstack-ironic14:33
*** rbartal has quit IRC14:35
*** udesale__ has quit IRC14:38
openstackgerritIlya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup  https://review.openstack.org/50785814:39
*** makowals has quit IRC14:40
*** mrch has quit IRC14:41
rloodtantsur: our documentation is still using classic drivers. I thought we had switched them to hardware types :( eg: https://docs.openstack.org/ironic/latest/install/enrollment.html14:43
dtantsurthis is another step we need to take14:43
rloodtantsur: you want me to open a bug against it, or is it on your list of things to do?14:44
*** alex_xu has quit IRC14:44
dtantsurrloo: just added to the etherpad. I personally don't need a bug, I treat it as part of the deprecation work.14:44
rloodtantsur: ok, as long as you are tracking it/you are doing it :)14:44
dtantsur:)14:45
*** jaypipes has joined #openstack-ironic14:49
*** alex_xu has joined #openstack-ironic14:49
*** mat128 has joined #openstack-ironic14:56
*** baoli has quit IRC14:56
*** baoli has joined #openstack-ironic14:59
*** mjura has quit IRC15:01
*** yangyapeng has joined #openstack-ironic15:08
*** makowals has joined #openstack-ironic15:10
*** srobert has joined #openstack-ironic15:12
*** makowals has quit IRC15:13
*** rpioso has joined #openstack-ironic15:13
*** chlong has joined #openstack-ironic15:17
*** chas has joined #openstack-ironic15:25
dtantsurgoing a bit earlier today, see you on Monday (hopefully, heh)15:29
dtantsurwhile I'm out, please ping TheJulia about all the things PTL15:29
dtantsurif I'm suddenly out next week, please assume I'm doing some paperwork, and ping TheJulia again :)15:29
*** Sukhdev has joined #openstack-ironic15:30
jlvillalIs the sourceforege issue resolved? I still see it in the whiteboard.15:30
jlvillalThe one that was affecting IPA and ironic-lib15:30
TheJuliajlvillal: I've not looked yet today, I've been heads down on this spec which I might be posting in a few minutes, doing one final read15:31
jlvillalTheJulia, No worries and thanks :)15:31
openstackgerritRuby Loo proposed openstack/python-ironicclient master: Update README  https://review.openstack.org/50789815:32
rloobye dtantsur, good luck with the move!15:32
*** amarao has quit IRC15:33
*** dprince has quit IRC15:34
jlvillalI'm marking the gate as good. I see jobs are passing now for ironic-lib15:34
jlvillaldtantsur, ditto! Have a safe move :)15:34
dtantsuro/15:35
*** dtantsur is now known as dtantsur|afk15:35
*** dprince has joined #openstack-ironic15:35
openstackgerritRuby Loo proposed openstack/python-ironicclient master: Update README  https://review.openstack.org/50789815:36
*** amarao has joined #openstack-ironic15:36
*** amarao1 has quit IRC15:36
*** rcernin has quit IRC15:43
*** crushil_ has joined #openstack-ironic15:43
*** amarao has quit IRC15:49
*** vsaienk0 has quit IRC15:52
*** vsaienk0 has joined #openstack-ironic15:53
*** trown is now known as trown|afk15:53
*** penick has joined #openstack-ironic15:54
*** stendulker has joined #openstack-ironic15:54
openstackgerritRuby Loo proposed openstack/python-ironicclient master: Update README  https://review.openstack.org/50789815:56
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Add cleanhold specification  https://review.openstack.org/50791015:59
*** chas has quit IRC16:02
*** nicodemos has joined #openstack-ironic16:02
TheJuliaapparently sourceforge is having connectivity issues today :(16:03
rlooTheJulia: still? ugh. ^^ jlvillal16:05
*** crushil_ has quit IRC16:05
NobodyCamGood Morning Ironic'ers16:05
*** amarao has joined #openstack-ironic16:06
openstackgerritShivanand Tendulker proposed openstack/ironic-python-agent master: Rescue extension for CoreOS with DHCP tenant networks  https://review.openstack.org/42352116:07
*** r-daneel has joined #openstack-ironic16:12
*** stendulker has quit IRC16:13
*** stendulker has joined #openstack-ironic16:14
*** trown|afk has quit IRC16:18
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083116:20
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add rescue interface field to node-related notifications  https://review.openstack.org/50288916:21
*** crushil has quit IRC16:23
openstackgerritShivanand Tendulker proposed openstack/ironic master: Add rescuewait timeout periodic task  https://review.openstack.org/35315616:23
*** tesseract has quit IRC16:24
openstackgerritShivanand Tendulker proposed openstack/ironic master: Agent driver rescue implementation  https://review.openstack.org/40043716:24
*** marios has quit IRC16:25
*** hjensas has quit IRC16:26
jlvillalrloo: Oh :( And I just updated the whiteboard an hour or so ago16:27
rloojlvillal: undo! :)16:27
jlvillalrloo: Okay16:27
nicodemosgood morning.16:32
TheJuliarloo: jlvillal: yup, still https://twitter.com/sfnet_ops16:32
TheJuliathere was a span where all was good though :)16:32
*** chas has joined #openstack-ironic16:36
*** chas has quit IRC16:41
*** crushil has joined #openstack-ironic16:44
*** stendulker has quit IRC16:44
*** milan has quit IRC16:45
*** chas has joined #openstack-ironic16:46
openstackgerritRuby Loo proposed openstack/python-ironicclient master: Update documentation  https://review.openstack.org/50792716:46
*** derekh has quit IRC16:49
*** vsaienk0 has quit IRC16:49
*** d0ugal has quit IRC16:50
*** e0ne has quit IRC16:52
*** penick has quit IRC16:53
openstackgerritRuby Loo proposed openstack/ironic master: Add missing query parameters to 'next' link for GET /nodes?...  https://review.openstack.org/50628016:53
*** zhangfei has joined #openstack-ironic16:56
openstackgerritRuby Loo proposed openstack/ironic master: Remove 'next' for GET /nodes?limit=1&instance_uuid=  https://review.openstack.org/50620516:56
openstackgerritRuby Loo proposed openstack/ironic master: Add missing query parameters to 'next' link for GET /nodes?...  https://review.openstack.org/50628016:57
*** penick has joined #openstack-ironic16:57
*** penick_ has joined #openstack-ironic17:01
*** penick has quit IRC17:03
*** rama_y has joined #openstack-ironic17:07
*** zhangfei has quit IRC17:08
TheJuliarloo: if you can take a look at https://review.openstack.org/#/c/507910/1/specs/approved/cleanhold-state.rst and let me know if it basically matches what we discussed regarding clean steps at the ptg, I'll go ahead and begin writing the code this afternoon while it is fresh in my head17:11
patchbotpatch 507910 - ironic-specs - Add cleanhold specification17:11
rlooTheJulia: hmm, ok, eating messy lunch. soon...17:12
TheJuliawell, enjoy the lunch first!17:13
TheJuliaI'm on pacific time right now17:13
*** pcaruana has quit IRC17:14
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector master: WiP add policy enforcement engine  https://review.openstack.org/50782617:18
*** trown has joined #openstack-ironic17:21
openstackgerritIlya Etingof proposed openstack/ironic-specs master: IPA to report node activity  https://review.openstack.org/49270217:22
jlvillalTheJulia: Welcome to our wonderful time zone :D17:27
openstackgerritIlya Etingof proposed openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection followup  https://review.openstack.org/50785817:28
*** dpawar has joined #openstack-ironic17:33
*** dpawar has quit IRC17:35
rlooTheJulia: i looked at the spec, when you have time, let's discuss. fwiw, i reviewed the ptg notes (https://etherpad.openstack.org/p/ironic-queens-ptg-contentious-topics) and GET API spec (https://review.openstack.org/#/c/501511)17:38
patchbotpatch 501511 - ironic-specs - spec: Clean Steps that are Available17:38
rlooTheJulia: i am unclear as to how your spec & the get-clean-steps spec work together17:38
rlooTheJulia: at ptg, did folks mention it might be useful to have a 'hold' state so that the node has the ramdisk loaded but doing nothing? (outside of the need for this API to get the clean steps)17:39
rlooTheJulia: was also wondering, what about a 'clean-hold' clean step, and not have a 'clean-hold' state (it'd be 'clean-wait'). or maybe we'd still want a clean-hold state, just thinking maybe for flex, folks might want a clean-hold step to put between/after a particular clean step.17:40
fellypefcaDevstack is too slow :/17:40
*** trown has quit IRC17:46
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function  https://review.openstack.org/50766817:47
openstackgerritJohn L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off  https://review.openstack.org/50729717:47
*** e0ne has joined #openstack-ironic17:49
* sambetts --> home 17:52
sambettsnight all17:52
*** dpawar has joined #openstack-ironic17:52
*** sambetts is now known as sambetts|afk17:52
fellypefcaGood night sambetts|afk :)17:52
*** jtomasek has quit IRC17:53
*** rama_y has quit IRC17:57
openstackgerritChristopher Dearborn proposed openstack/ironic master: python-dracclient driver version update  https://review.openstack.org/50794217:57
*** rama_y has joined #openstack-ironic17:57
openstackgerritChristopher Dearborn proposed openstack/ironic master: python-dracclient driver version update  https://review.openstack.org/50794217:58
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function  https://review.openstack.org/50766817:58
openstackgerritJohn L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off  https://review.openstack.org/50729717:58
*** penick has joined #openstack-ironic17:58
*** penick_ has quit IRC18:01
TheJuliarloo: sorry, I've been on the phone18:04
rlooTheJulia: no worries18:04
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function  https://review.openstack.org/50766818:05
openstackgerritJohn L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off  https://review.openstack.org/50729718:05
*** ralonsoh has quit IRC18:06
rlooTheJulia: I just added my comments to the spec so I don't forget...18:06
rlooTheJulia: so was there a desire from folks to be able to have a 'hold' so they could poke around?18:07
TheJuliasorry, phone still18:07
rlooTheJulia: oh. heh. let me know when you're avail then :)18:07
TheJuliarloo: Okay, finally free18:09
TheJuliarloo: so, tl;dr it was the solution to not touching the state machine from the get of the step information18:09
TheJuliarloo: it was also reached in more groupmind consensus really quickly during that whole discussion and we quickly moved on18:09
rlooTheJulia: but adding a 'hold' *is* touching the state machine?18:09
TheJuliaabsolutely18:10
*** jkilpatr has quit IRC18:10
TheJuliaproblem is we shouldn't have multiple ways of triggering state machine changes18:10
TheJulianor can we build something outside that doesn't reflect the current state but that is doing things18:10
TheJuliaso this kind of decouples the get, makes the get become just what is cached, the user can go to cleanhold, get up to date cleansteps, then trigger manual cleaning with what they want from there18:10
rlooTheJulia: first question. is this 'hold' useful outside of the get-cleaning-steps API? i thought people thought it'd be useful18:10
TheJuliarloo: hold concept is useful in terms of a long running ramdisk18:11
rlooTheJulia: ok, so i think it is worth mentioning that in the spec, because that becomes a usecase/requirement18:11
TheJuliaI guess in theory, we could have a general hold state, but that could also be onfusing18:12
TheJuliaconfusing18:12
TheJuliarloo: I thought I had... hmm18:12
rlooTheJulia: and might someone want a 'hold' to happen after a particular clean step was done?18:12
TheJuliaI wouldn't think a machine would go back to cleanhold, although we could do that with the state machine as long as the step had the ability to go back, but I suspect joining too much on to what is functionally just state machine changes will scope creep and increase complexity18:13
rlooTheJulia: i'm not sure what you mean there. I can envision the 'hold' as a separate clean-step.18:14
TheJuliaSo a separate user definable clean-step that can kick the node back to a cleanhold state?18:14
*** noam_ has joined #openstack-ironic18:14
noam_Hi18:14
TheJuliawhich then a user could retrigger cleaning or exit?18:14
rlooTheJulia: right. what is a cleanhold state anyway. it just means that there is a ramdisk there.18:15
*** pcaruana has joined #openstack-ironic18:15
noam_Someone can elborate the difference between netboot and local?18:15
rlooTheJulia: which i think would mean, that the user would have to 'continue'18:15
TheJuliarloo: basically yeah. Running ramdisk, collected cleansteps.18:15
rlooTheJulia: user would have to explicitly say 'continue'18:15
TheJuliaugh18:16
rlooTheJulia: and then the conductor would go on to the next step if any18:16
TheJuliaI guess a clean continue verb18:16
rlooTheJulia: yes, something like that. i haven't thought out the details. just the concept.18:17
*** Sukhdev has quit IRC18:17
TheJuliaif there is driver internal info stating it can continue, then I guess itcould continue the next cleaning step upon the next heartbeat18:17
TheJuliaYeah, it could just be an alias to kick it to CLEANWAIT18:17
TheJuliaand it could error api side if it is an invalid request18:17
rlooTheJulia: heh, i haven't thought about implementation, i'm sure we can implement ;)18:18
*** ChubYann has joined #openstack-ironic18:18
*** hjensas has joined #openstack-ironic18:18
rlooTheJulia: ok, so if you're good with that, it addresses two of my earlier questions.18:20
rlooTheJulia: now, how does this fit with the get-clean-steps API.18:20
TheJuliarloo: I would prefer that we don't scope creep new mechanics, at least if I'm writing it since I need to get back to some other things to unblock myself18:20
rlooTheJulia: I think we still want a GET .../cleaning/steps API, and it returns the cached steps18:20
TheJuliarloo: and then nothing else is required...18:20
rlooTheJulia: i don't call it 'scope creep new mechanics'. i call it 'support for the feature'18:20
*** racedo has quit IRC18:21
rlooTheJulia: otherwise, the coding might be done in a way that won't allow it to be flexible later18:21
TheJuliaahh, but independent small features can add up and build on each other18:21
rlooTheJulia: so the user can do a GET ../cleaning/steps. How does the user do a 'fetch' to get the steps?18:21
rlooTheJulia: I had proposed a POST ../cleaning/steps to do that, so that there is a symmetry between the get & fetch.18:22
rlooTheJulia: i can't recall if we had decided at the PTG that we didn't or we did like that symmetry18:22
rlooTheJulia: wrt 'independent small features can add up...', yes. but hard to do if you don't architect it right.18:22
rlooTheJulia: there's a diff conceptually between putting the node in a hold state, vs it being a clean step.18:23
*** jkilpatr has joined #openstack-ironic18:23
rlooTheJulia:  let me give you an example. automated vs manual cleaning. all that would have been easier to code if we had planned for both initially.18:23
*** rama_y has quit IRC18:24
*** rama_y has joined #openstack-ironic18:24
TheJuliaphone again18:24
TheJuliaokay18:25
openstackgerritFellype Cavalcante proposed openstack/ironic stable/ocata: Fix multitenancy scenario on OneView Driver  https://review.openstack.org/50763218:25
TheJuliarloo: the user would fetch steps by requesting the cleanhold state. Having a post means a second method of triggering state machine changes, and there should ideally be only one.18:26
TheJuliarloo: agree, hence why I asked you to take a look :)18:26
rlooTheJulia: ok, so we decided that it was ok not to have that symmetry.18:26
rlooTheJulia: my concern with this hold thing, is that we're expecting the user to issue two commands (and wait in between)18:27
rlooTheJulia: in order to get the clean steps18:27
rlooTheJulia: 1. put node in hold state18:27
rloo2. wait til node is in hold state18:27
TheJuliawe did, I remembered the consensus seemed to be that we wanted to maintain existing path for causing things to happen since it would need to be represented back to the user18:27
rloo3. do something to get the node out of hold state18:27
TheJuliaWell18:27
*** rama_y has quit IRC18:27
TheJuliaEssentially having the post is the same18:27
TheJuliabecause you can't let that hang for five minutes or so for IPA to boot and heartbeat in18:28
rlooTheJulia: i *think* the hold was because folks said, if i know i want to do cleaning, but i want to know the clean steps first, i don't want to have to boot the ramdisk twice.18:28
rlooTheJulia: does that ring a bell with you? (i wish i had taken more notes at ptg...)18:29
TheJuliarloo: That was the side effect, that we functionally begin moving towards long running ramdisks and not having multiple boot cycles for things like, refreshing data in the db.18:29
rlooTheJulia: no, the post is different. there is no 'hold' that requires user intervention to get out of18:29
TheJuliarloo: so your post would return what to the user?18:30
rlooTheJulia: nothing18:30
rlooTheJulia: it is async18:30
rlooTheJulia: just like most of the requests to put the node in a provision state18:30
rlooTheJulia: just like the 'hold' would be. what were you planning on returning for that request?18:30
TheJuliaso a user would then have to continue to check the state to see if the machine has booted, aside from getting a last updated field18:31
rlooTheJulia: yup, if they are waiting to get the available clean steps18:31
TheJuliarloo: nothing, but it was going to be via the provision state interface and state machine18:31
rlooTheJulia: just like a user has to wait for a node to get deployed to18:31
TheJuliaexactly, so there is no real difference aside from one follows an existing pattern which was the preference in the room during the discussion18:32
rlooTheJulia: I honestly don't recall that this was the preference18:32
rlooTheJulia: my concern is that for someone that only wants the clean steps, it looks like your proposal is asking them to do the 3 steps i mentioned above.18:33
rlooTheJulia: which may be fine. i have not been given any requirements from any user, as to how they want this to work18:33
TheJuliatrigger clean hold, wait until it hits the cleanhold state, the steps would b e updated at that point and the user would be able to know that they should have refreshed cleanstep information from the get /cleaning/steps endpoint18:34
rlooTheJulia: and then, the user has to issue another request to get it out of hold, right?18:34
*** fellypefca is now known as fellypefca-afk18:35
TheJuliaYes, that is true, they can issue a manual clean or exit the state. They also may wish to leave the node in that powered on state waiting for days or even potentially (and now i feel bad for the earth) weeks18:35
rlooTheJulia: so i think there is value to having the 'hold'. I'm just not sure that it effectively addresses the issue of getting the clean steps (from the user's point of view, although I don't know of any users)18:36
rloos/get/fetch/18:36
rloocaching i guess18:36
TheJuliaEveryone that I know that has asked for a way to get them, wants to turn around and run manual cleaning18:36
rlooi'm not questioning the need for the GET ../clean/steps or the proposed hold thing.18:37
TheJuliai know18:37
rlooTheJulia: i'm just questioning the fetching/caching of clean steps part18:37
*** hjensas has quit IRC18:37
*** rwsu has quit IRC18:38
* TheJulia goes and looks for the instance of irccloud that is using 3.36GB of ram18:38
rlooTheJulia: from the perspective of caching the clean steps, i-the-user might like to issue one command only and come back after coffee to see the clean steps. not have to issue a 'continue' command (or forget to issue that command)18:38
TheJuliarloo: but if we also hook in collection along the way with normal processes such as automated cleaning so that we always have cached clean steps....18:39
TheJuliaThings won't change that often, if an operator changes out the ramdisk with more hardware managers, they will then need a way to refresh that information18:40
rlooTheJulia: huh? we already do this. I mean, we already cache the clean steps, when we start cleaning18:40
TheJuliawe do, then we nuke it upon completion I thought18:41
rlooTheJulia: so i honestly don't recall, i thought someone said that we didn't. but that is ... who cares. we can keep it after that or not, it isn't a big deal.18:41
TheJuliaI only think we nuke it now because I found and fixed a bug in manual cleaning where we were not cleaning stuff up and getting stuck in a loop18:42
rlooTheJulia: ok18:42
TheJuliaif we timed out, we didn't execute the same cleanup steps to that are normally performed upon exit so re-issuing manual clean wouldn't work18:43
TheJuliawow, that was a while ago18:43
rlooTheJulia: so I'm a bit lost. I'm not sure what that has to do with what we are discussing wrt the hold & the fetching/caching of clean steps18:44
TheJuliaI'm just providing background context as to why I think we don't cache them right now unless something has changed, so we would need to always keep them cached.18:45
TheJuliaerr, I think we dont18:45
TheJuliaif that makes sense18:45
rlooTheJulia: yes, for the GET ../cleaning/steps to work, we need to always cache them18:45
rlooTheJulia: all i remember, is that when we start cleaning, we get the clean steps from ipa. easy enough to check what happens when cleaning finishes18:46
rlooTheJulia: anyway, I think I've given you my feedback about the hold'em state. i can recap in the patch.18:48
TheJuliarloo: Agreed, as long as we're careful around logic like on this line https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L313 :)18:49
TheJuliaThat you have, thanks!18:49
rlooTheJulia: yup wrt the code!18:49
*** fellypefca-afk has left #openstack-ironic18:52
*** penick has quit IRC18:53
*** dprince has quit IRC18:56
*** MattMan has quit IRC18:59
*** MattMan has joined #openstack-ironic19:00
*** pcaruana has quit IRC19:00
*** r-daneel has quit IRC19:02
*** r-daneel has joined #openstack-ironic19:03
*** e0ne has quit IRC19:08
*** penick has joined #openstack-ironic19:10
*** jcoufal has quit IRC19:17
*** rama_y has joined #openstack-ironic19:20
*** dprince has joined #openstack-ironic19:20
*** livelace2 has quit IRC19:25
*** dpawar has quit IRC19:26
*** penick_ has joined #openstack-ironic19:27
*** penick has quit IRC19:30
*** rwsu has joined #openstack-ironic19:32
*** ChubYann has quit IRC19:58
*** mat128 has quit IRC20:04
*** ChubYann has joined #openstack-ironic20:12
*** jkilpatr has quit IRC20:14
*** r-daneel has quit IRC20:14
*** r-daneel has joined #openstack-ironic20:15
*** cdearborn has quit IRC20:28
*** rwsu has quit IRC20:52
*** ChubYann has quit IRC20:54
*** noam_ has quit IRC20:54
*** e0ne has joined #openstack-ironic21:00
*** penick_ has quit IRC21:03
*** e0ne has quit IRC21:04
*** e0ne has joined #openstack-ironic21:05
*** penick has joined #openstack-ironic21:07
*** ChubYann has joined #openstack-ironic21:07
*** crushil has quit IRC21:09
*** penick has quit IRC21:11
*** causten has quit IRC21:11
*** rwsu has joined #openstack-ironic21:12
*** LeoBud has joined #openstack-ironic21:13
*** bnemec has quit IRC21:14
*** LeoBud has quit IRC21:20
*** causten has joined #openstack-ironic21:20
*** chas has quit IRC21:21
*** chas has joined #openstack-ironic21:22
*** e0ne has quit IRC21:24
*** chas has quit IRC21:27
*** causten has quit IRC21:27
*** causten has joined #openstack-ironic21:28
*** etingof has quit IRC21:31
*** srobert_ has joined #openstack-ironic21:31
*** srobert has quit IRC21:35
*** srobert_ has quit IRC21:36
*** dprince has quit IRC21:42
*** charcol has joined #openstack-ironic21:43
*** penick has joined #openstack-ironic21:46
*** jlabarre has quit IRC21:46
*** etingof has joined #openstack-ironic21:46
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent master: Build tinyipa with python3  https://review.openstack.org/49538521:51
*** baoli has quit IRC21:54
*** penick has quit IRC21:55
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent master: Build tinyipa with python3  https://review.openstack.org/49538521:55
*** jkilpatr has joined #openstack-ironic21:56
*** causten has quit IRC22:02
*** causten has joined #openstack-ironic22:03
*** causten_ has joined #openstack-ironic22:05
*** cdearborn has joined #openstack-ironic22:06
*** charcol has quit IRC22:06
*** rwsu has quit IRC22:07
*** causten has quit IRC22:07
*** Goneri has quit IRC22:08
*** causten has joined #openstack-ironic22:13
*** causten_ has quit IRC22:16
*** rbudden has quit IRC22:24
*** krtaylor has quit IRC22:30
*** rbudden has joined #openstack-ironic22:38
*** krtaylor has joined #openstack-ironic22:43
*** penick has joined #openstack-ironic22:45
*** jaypipes has quit IRC22:48
*** penick has quit IRC22:52
openstackgerritMerged openstack/ironic stable/pike: Adds more exception handling for ironic-conductor heartbeat  https://review.openstack.org/50737322:56
openstackgerritMerged openstack/ironic master: Cleanup unit tests for ipmittool  https://review.openstack.org/50641322:56
*** charcol has joined #openstack-ironic23:01
*** baoli has joined #openstack-ironic23:18
*** rloo has quit IRC23:21
*** baoli_ has joined #openstack-ironic23:22
*** baoli has quit IRC23:22
*** Sukhdev has joined #openstack-ironic23:32
jlvillalTheJulia: sourceforge says they are now back up as of 5 minutes ago. I could download ipmitool tarball23:33
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function  https://review.openstack.org/50766823:34
openstackgerritJohn L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off  https://review.openstack.org/50729723:34
TheJulia\o/23:34
* TheJulia messages friends that were mentioning it23:35
*** krtaylor has quit IRC23:35
anupnjlvillal, nice, even i was waiting to see when sourceforge gets up23:36
*** causten has quit IRC23:38
*** causten has joined #openstack-ironic23:38
*** srobert has joined #openstack-ironic23:39
*** causten has quit IRC23:43
*** ijw has joined #openstack-ironic23:43
*** ijw has quit IRC23:45
*** ijw has joined #openstack-ironic23:45
jlvillalanupn: I noticed some jobs worked on your Python 3 IPA patch.23:45
jlvillalanupn: Never mind. Those are the coreos jobs.23:47
*** rpioso has quit IRC23:51
*** bnemec has joined #openstack-ironic23:52
*** r-daneel has quit IRC23:58

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