Wednesday, 2015-09-16

openstackgerritJosh Gachnang proposed openstack/ironic: Support Zapping in the conductor  https://review.openstack.org/22198900:03
JoshNangthat should cover all the test cases for zapping ^00:04
jrollnice, thanks!00:07
jrollI'm mostly done for the night, have a good one all00:07
JoshNango/00:07
*** shoutm has joined #openstack-ironic00:10
*** trown is now known as trown|outttypeww00:10
*** blakec has quit IRC00:11
openstackgerritMerged openstack/ironic: Agent supports post-clean-step operations  https://review.openstack.org/22228700:16
*** tsekiyama has quit IRC00:18
*** devlaps has quit IRC00:18
*** BadCub_ has joined #openstack-ironic00:18
*** smoriya has joined #openstack-ironic00:20
*** dims_ has quit IRC00:20
*** bapalm_ has joined #openstack-ironic00:20
*** liliars_ has joined #openstack-ironic00:21
*** BadCub has quit IRC00:21
*** bapalm has quit IRC00:21
*** lintan_ has quit IRC00:21
*** sinval has quit IRC00:21
rlooJoshNang: thx for the revision(s). will look at your zapping patch tomorrow morning!00:22
*** rloo has quit IRC00:22
*** mtanino has quit IRC00:22
*** mgoddard has quit IRC00:22
JoshNangrloo: thanks! i'll be around to fix up the issues you find :)00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-ironic00:23
*** bradjones has quit IRC00:25
*** lintan has joined #openstack-ironic00:25
*** shoutm_ has joined #openstack-ironic00:28
*** betherly_ has joined #openstack-ironic00:28
*** shoutm has quit IRC00:29
*** persia_ has joined #openstack-ironic00:29
*** slagle_ has joined #openstack-ironic00:31
*** mtaylor has joined #openstack-ironic00:31
*** gabriel-1 has joined #openstack-ironic00:33
*** JoshNang_ has joined #openstack-ironic00:34
*** slagle has quit IRC00:35
*** jaypipes has quit IRC00:35
*** JoshNang has quit IRC00:35
*** gabriel-bezerra has quit IRC00:35
*** MattMan has quit IRC00:36
*** mordred has quit IRC00:36
*** persia has quit IRC00:36
*** mjturek1 has quit IRC00:36
*** mikal has quit IRC00:36
*** betherly has quit IRC00:36
*** betherly_ is now known as betherly00:36
*** JoshNang_ is now known as JoshNang00:36
*** mikal has joined #openstack-ironic00:36
*** mjturek1 has joined #openstack-ironic00:36
*** jaypipes has joined #openstack-ironic00:39
*** bizarrochristy has joined #openstack-ironic00:40
*** MattMan has joined #openstack-ironic00:42
*** shoutm_ has quit IRC00:46
*** shoutm has joined #openstack-ironic00:48
*** MattMan has quit IRC01:00
*** dims_ has joined #openstack-ironic01:05
*** Sukhdev has quit IRC01:09
*** korekhov has joined #openstack-ironic01:10
*** praneshp has quit IRC01:14
*** BadCub_ is now known as BadCub01:16
*** r-daneel has quit IRC01:17
*** lazy_prince has joined #openstack-ironic01:18
*** thrash is now known as thrash|g0ne01:18
*** garthb has quit IRC01:19
*** killer_prince has joined #openstack-ironic01:27
*** lazy_prince has quit IRC01:30
*** lazy_prince has joined #openstack-ironic01:30
*** killer_prince has quit IRC01:31
*** dims_ has quit IRC01:34
*** baoli has joined #openstack-ironic01:36
*** bizarrochristy has quit IRC01:38
*** bizarrochristy has joined #openstack-ironic01:38
*** bizarrochristy has quit IRC01:43
*** Haomeng has quit IRC01:49
*** zhenguo has joined #openstack-ironic01:50
*** wshao has quit IRC01:51
*** korekhov has quit IRC02:02
*** shoutm has quit IRC02:05
lintanhi Haomeng02:06
*** yog_ has joined #openstack-ironic02:08
*** shoutm has joined #openstack-ironic02:13
*** harshs has quit IRC02:16
*** puranamr has joined #openstack-ironic02:17
*** Haomeng has joined #openstack-ironic02:22
*** zhenguo has quit IRC02:28
openstackgerritTan Lin proposed openstack/ironic: Replace metaclass registry with explicit opt-in registry from oslo  https://review.openstack.org/21758402:32
*** zhenguo has joined #openstack-ironic02:33
*** lazy_prince has quit IRC02:34
*** lazy_prince has joined #openstack-ironic02:34
openstackgerritTan Lin proposed openstack/ironic: Replace metaclass registry with explicit opt-in registry from oslo  https://review.openstack.org/21758402:34
*** achanda has joined #openstack-ironic02:35
*** mdbooth has quit IRC02:40
*** bizarrochristy has joined #openstack-ironic02:41
*** bizarrochristy has quit IRC02:42
*** mdbooth has joined #openstack-ironic02:46
*** puranamr has quit IRC02:50
*** lazy_prince has quit IRC02:51
*** puranamr has joined #openstack-ironic03:11
*** puranamr has quit IRC03:15
*** flwang1 has joined #openstack-ironic03:15
*** saripurigopi has joined #openstack-ironic03:29
saripurigopiMorning Ironic03:29
*** boris-42 has quit IRC03:30
*** flwang1 has quit IRC03:30
*** puranamr has joined #openstack-ironic03:34
*** chenglch has joined #openstack-ironic03:36
*** rameshg87 has joined #openstack-ironic03:39
*** david-lyle has joined #openstack-ironic03:41
*** shoutm has quit IRC03:48
*** shoutm_ has joined #openstack-ironic03:49
*** boris-42 has joined #openstack-ironic03:49
*** boris-42 has quit IRC03:49
*** boris-42 has joined #openstack-ironic03:49
*** puranamr has quit IRC03:51
Haomengsaripurigopi: morning:)03:56
saripurigopiHi Haomeng03:56
Haomengsaripurigopi: :)03:56
*** amotoki has joined #openstack-ironic03:56
*** naohirot has joined #openstack-ironic04:05
*** baoli has quit IRC04:10
openstackgerritYuiko Takada proposed openstack/ironic-inspector: [WIP]Support IPA for devstack-plugin  https://review.openstack.org/22356304:13
*** Marga_ has quit IRC04:16
*** praneshp has joined #openstack-ironic04:24
*** coolsvap has joined #openstack-ironic04:28
*** sdake has joined #openstack-ironic04:33
*** sdake_ has joined #openstack-ironic04:34
*** sdake has quit IRC04:38
*** wshao has joined #openstack-ironic04:38
*** david-lyle has quit IRC04:42
*** david-lyle has joined #openstack-ironic04:45
*** achanda has quit IRC04:46
*** achanda has joined #openstack-ironic04:51
*** sdake_ has quit IRC04:56
*** amit213 has quit IRC04:59
*** sdake has joined #openstack-ironic04:59
*** amit213 has joined #openstack-ironic04:59
openstackgerritRamakrishnan G proposed openstack/ironic: Add support for in-band cleaning in ISCSIDeploy  https://review.openstack.org/22089805:10
*** sdake_ has joined #openstack-ironic05:10
*** sdake has quit IRC05:14
*** garthb has joined #openstack-ironic05:14
*** praneshp_ has joined #openstack-ironic05:15
*** praneshp has quit IRC05:18
*** praneshp_ is now known as praneshp05:18
*** Nisha has joined #openstack-ironic05:22
*** lazy_prince has joined #openstack-ironic05:24
*** Sukhdev_ has joined #openstack-ironic05:32
*** vinbs has joined #openstack-ironic05:35
*** baoli has joined #openstack-ironic05:41
*** lsmola has joined #openstack-ironic05:42
*** shoutm_ has quit IRC05:44
*** baoli has quit IRC05:46
*** stendulker has joined #openstack-ironic05:57
*** ukalifon has joined #openstack-ironic05:57
openstackgerritYuiko Takada proposed openstack/ironic-inspector: [WIP]Support rootwrap  https://review.openstack.org/22393006:08
openstackgerritYuiko Takada proposed openstack/ironic-inspector: Support IPA for devstack-plugin  https://review.openstack.org/22356306:09
*** shoutm has joined #openstack-ironic06:12
openstackgerritMerged openstack/ironic: Allow tftpd usage of '--secure' by using symlinks  https://review.openstack.org/22282706:20
*** praneshp has quit IRC06:26
*** praneshp has joined #openstack-ironic06:28
*** flwang1 has joined #openstack-ironic06:32
*** wshao has quit IRC06:35
*** e0ne has joined #openstack-ironic06:37
*** e0ne has quit IRC06:39
*** marios has joined #openstack-ironic07:01
*** marios has joined #openstack-ironic07:01
*** praneshp has quit IRC07:04
*** achanda has quit IRC07:06
*** ifarkas has joined #openstack-ironic07:07
*** garthb has quit IRC07:12
*** boris-42 has quit IRC07:19
*** rameshg87 has quit IRC07:19
*** marios has quit IRC07:19
*** zhenguo has quit IRC07:19
*** getvasanth has joined #openstack-ironic07:24
*** marios has joined #openstack-ironic07:24
*** zhenguo has joined #openstack-ironic07:24
*** rameshg87 has joined #openstack-ironic07:25
*** boris-42 has joined #openstack-ironic07:25
*** Sukhdev_ has quit IRC07:30
*** flwang1 has quit IRC07:50
vdrokmorning ironic07:58
yuikotakadavdrok, morning :)08:01
vdrokmorning yuikotakada08:01
*** romainh has joined #openstack-ironic08:01
betherlyMorning ironic08:03
*** e0ne has joined #openstack-ironic08:03
Haomengmorning vdrok, yuikotakada betherly:08:06
yuikotakadabetherly, Haomeng, morning :)08:06
vdrokmorning betherly, Haomeng08:06
Haomengyuikotakada: :)08:06
Haomengvdrok: :)08:06
betherlyHi Haomeng, yuikotakada, vdrok :)08:08
Haomengbetherly: :)08:09
*** getvasanth has quit IRC08:11
*** sdake has joined #openstack-ironic08:11
*** sdake_ has quit IRC08:12
openstackgerritYuiko Takada proposed openstack/ironic-inspector: [WIP]Support rootwrap  https://review.openstack.org/22393008:14
*** lucasagomes has joined #openstack-ironic08:15
*** yog_ has quit IRC08:15
*** jistr has joined #openstack-ironic08:19
*** e0ne has quit IRC08:24
*** mgoddard has joined #openstack-ironic08:26
*** getvasanth has joined #openstack-ironic08:26
*** getvasanth has quit IRC08:27
*** getvasanth has joined #openstack-ironic08:28
*** Nisha has quit IRC08:32
*** shoutm has quit IRC08:33
sambettsMorning all o/08:34
sambettsyuikotakada: https://review.openstack.org/#/c/223665 Can I get some feed back that I'm going about it the right way08:35
*** alexpilotti has joined #openstack-ironic08:35
lucasagomessambetts, morning08:35
sambettsHey lucasagomes08:35
*** ndipanov has quit IRC08:35
yuikotakadasambetts, lucasagomes, morning :)08:36
lucasagomesyuikotakada, morning (-:08:36
yuikotakadasambetts, oh, getting start alembic? great! Yeah, I will check it otherwise perhaps it will take time08:38
lucasagomesapparently the python-ironicclient gate is broken :-(08:38
yuikotakadalucasagomes, you have ever posted some blueprints and specs in Ironic, right?08:39
lucasagomesyuikotakada, yes08:39
sambettsyuikotakada: I've only really got the experience from the Neutron migration work to go on, so I thought I would get some feedback before I implement it completely wrong and have to start over08:39
*** dlpartain has joined #openstack-ironic08:41
yuikotakadalucasagomes, I'm also thinking about posting a blueprint, so that I'd like to know something like flow... Can I follow https://wiki.openstack.org/wiki/Ironic/Specs_Process ?08:42
*** Nisha has joined #openstack-ironic08:42
*** derekh has joined #openstack-ironic08:42
yuikotakadalucasagomes, thanks, you mean, at fitrst should I post my idea to community  as a blueprint and discuss?08:42
lucasagomesyuikotakada, yes exactly that's the flow08:42
*** ndipanov has joined #openstack-ironic08:43
lucasagomesyuikotakada, no you can post the blueprint+spec directly08:43
lucasagomesand have the discussions in the review08:43
*** dlpartain has left #openstack-ironic08:44
lucasagomesusually people discuss first if it's something controversial or there are too many ways to solve the same problem so they want to filter ideas before proposing one08:44
yuikotakadalucasagomes, oh, should I write a spec together?08:44
lucasagomesbut doesn't matter08:44
lucasagomesyuikotakada, yeah, actually the spec is the important part, the blueprint is just so we can keep track of it as part of the releases (we use launchpad to track features)08:45
yuikotakadalucasagomes, I see, thanks! :)08:45
*** e0ne has joined #openstack-ironic08:46
*** pelix has joined #openstack-ironic08:46
lucasagomesnp :-)08:46
lucasagomesjroll, I would like to, I will see if I find some time to get back to that patch08:53
*** kbyrne has joined #openstack-ironic08:53
*** getvasanth has quit IRC08:56
*** degorenko has joined #openstack-ironic09:01
*** getvasanth has joined #openstack-ironic09:04
*** athomas has joined #openstack-ironic09:04
*** romcheg has joined #openstack-ironic09:05
*** dtantsur|afk is now known as dtantsur09:06
lucasagomesdtantsur, around? I think this breaks inpection https://review.openstack.org/#/c/21550509:06
dtantsurMorning Ironic, hi yuikotakada, lucasagomes, sambetts and others :)09:06
lucasagomesthe concept of it09:06
* dtantsur is looking09:07
yuikotakadadtantsur, morning :)09:07
*** getvasanth has quit IRC09:07
lucasagomesdtantsur, oh I see you already looked09:07
sambettsHey dtantsur, I've got a question about your patches deprecation patch09:08
dtantsursambetts, yep09:08
*** getvasanth has joined #openstack-ironic09:08
sambettsdtantsur: your assertCalledWithPatch function, I understand what it does but why do you need it over something like assert_any_call ?09:09
dtantsursambetts, because patches come in random order, this function sorts them all and compares09:09
dtantsuressentially, it depends on a (random) dict order09:10
dtantsurlucasagomes, so that patch on its own should not break exception, but what people suggest might09:11
lucasagomesdtantsur, yeah I missed the continue there09:11
yuikotakadasambetts, I have no experience about alembic migration.. .>_< I'll try09:12
sambettsdtantsur: ^ https://review.openstack.org/#/c/22366509:13
sambettsdtantsur: I started alembic work, but I would like some feedback that I'm doing it the right way09:13
dtantsursambetts, oh, that's nice. I will have a look today (just woke up, sorry :)09:14
sambettsdtantsur: or at least in the right direction09:14
dtantsursambetts, yuikotakada, I'm not sure how the conversation on the ML about deadlines will go, but still I'll appreciate some reviews to python-ironic-inspector-client patches I've put up09:14
sambettsdtantsur: Ok, will do, I've been working off the milestone page this morning as you suggested09:15
dtantsurcool!09:15
dtantsurthat's also important. I'm not sure what's more important, with the client we probably missed deadlines...09:15
* dtantsur needs a clone to do client work while he's doing API work...09:16
* dtantsur brb09:16
*** dlpartain1 has joined #openstack-ironic09:29
*** sdake has quit IRC09:30
*** dlpartain has joined #openstack-ironic09:30
*** killer_prince has joined #openstack-ironic09:33
*** dlpartain1 has quit IRC09:33
*** dims_ has joined #openstack-ironic09:35
*** lazy_prince has quit IRC09:36
*** dlpartain has quit IRC09:38
rameshg87dtantsur: hi, any idea in which tz trown|outttypeww is in ?09:43
dtantsurrameshg87, east coast09:44
dtantsurwill appear in ~ 3-4 hours09:44
rameshg87okay09:44
rameshg87dtantsur: wondering if you have some idea about https://bugs.launchpad.net/diskimage-builder/+bug/149624809:45
openstackLaunchpad bug 1496248 in diskimage-builder "building ironic-agent ramdisk with fedora element fails" [Undecided,New]09:45
rameshg87dtantsur: have you built an ironic-agent ramdisk using fedora element in dib after sep 309:46
dtantsurrameshg87, I've fixed it recently, could you retry with the latest DIB?09:46
rameshg87dtantsur: for your work for inspector09:48
*** bigjools has quit IRC09:49
ifarkasdtantsur, hi, re: your comment on https://review.openstack.org/#/c/218271/09:53
ifarkasdtantsur, doesn't the feature freeze mentioned on the mailing list applies to both client and server?09:54
ifarkasdtantsur, just trying to understand...09:54
dtantsurifarkas, ironic does not comply with general OpenStack feature freeze, neither does inspector09:55
ifarkasdtantsur, cool, that's what I thought. but then I don't understand why it was even brought up09:56
ifarkasnevermind09:56
dtantsurwell, Doug is speaking for general schedule pov... and I agree that is sucks that we're doing everything so late in cycle09:57
*** naohirot has quit IRC09:57
dtantsurbut for inspector I'd prefer to make a bit raw liberty release with features, rather than a useless one09:57
dtantsurwe're a young project, and we have to move fast09:58
ifarkaslol, I hope they will appreciate that argument :-)10:00
ifarkasbut I am fine with merging what we can10:00
openstackgerritNisha Agarwal proposed openstack/ironic: Implement take_over for iscsi_ilo driver  https://review.openstack.org/21706810:00
*** chenglch has quit IRC10:06
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Extend the state machine spec to cover DEPLOYWAIT and abort  https://review.openstack.org/22400810:10
Nisharameshg87, lucasagomes dtantsur please review above patch. ^^^10:13
*** flwang1 has joined #openstack-ironic10:16
*** flwang1 has left #openstack-ironic10:22
openstackgerritMerged openstack/ironic-inspector: Add mock for check_call to all firewall tests  https://review.openstack.org/22350310:26
rameshg87dtantsur: sure..thanks..let me give a try10:27
dtantsurlucasagomes, if you have a second of time, could you review a tiny IPA patch https://review.openstack.org/#/c/222605/ please?10:30
lucasagomesdtantsur, hi there yup10:30
*** thrash|g0ne is now known as thrash10:33
lucasagomesdtantsur, mind if I update the commit message? The change id there is not querable in git (only gerrit)10:34
lucasagomesthat should be the git hash10:34
lucasagomesso one can git show <hash>10:34
dtantsurlucasagomes, as you wish :)10:34
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Follow-up to inspection patch 096830414b  https://review.openstack.org/22260510:36
lucasagomesdtantsur, done10:37
dtantsurthnx!10:37
openstackgerritYuiko Takada proposed openstack/ironic-specs: Get executable events API  https://review.openstack.org/22402210:37
openstackgerritZhenguo Niu proposed openstack/ironic: Check image size before provisioning for agent driver  https://review.openstack.org/21525410:38
openstackgerritYuiko Takada proposed openstack/ironic-specs: Get executable events API  https://review.openstack.org/22402210:39
dtantsuryuikotakada, nice catch ^^ but I would not call it "executable events". we call it "states", so it should be like nodes/UUID/states/transitions or something like that. WDYT?10:39
yuikotakadadtantsur, thank you for your advice :D Does "states" mean events? like provide, active? Yeah, I'm not sure about words especially.10:40
dtantsuryuikotakada, I think we call it "transitions"10:41
* dtantsur brb10:42
yuikotakadadtantsur, I see, thanks!10:42
*** bigjools has joined #openstack-ironic10:42
*** vinbs has quit IRC10:48
sambettsdtantsur, yuikotakada: In regards to yuiko's comments on the API discovery patch, I was just reading the "Unique URLs / Redirection Behavior" section of this doc -> http://flask.pocoo.org/docs/0.10/quickstart10:48
openstackgerritNisha Agarwal proposed stackforge/proliantutils: Fix ribcl to return correct number of processors  https://review.openstack.org/22167810:49
sambettsdtantsur, yuikotakada: Looking at all our current routing, non of our endpoints will work with a trailing slash10:49
sambettsdtantsur, yuikotakada: so the question is do we want them too10:50
sambetts?10:50
dtantsursambetts, I don't care much. we can fix all URL's separately10:53
*** Nisha has quit IRC10:53
sambettsdtantsur: Yeah, I was thinking maybe add a wishlist item or something that covers them all10:54
dtantsursambetts, ifarkas, yuikotakada, as to client patches: we should land them asap no matter how the discussion on the ML goes. I can make a stable/liberty branch from every commit, not only HEAD. So if Doug tells me to use 1.1.0, I'll make stable/liberty from it anyway.10:54
dtantsursambetts, ++10:54
yuikotakadasambetts, maybe, it's ok with as is. I just not sure this behavior is correct or not and then I wanted to know the answer :)10:54
sambettsyuikotakada: :) I wasn't sure either10:55
*** e0ne has quit IRC10:56
yuikotakadasambetts, :) and how do you think about updating HTTP-API.rst ? It would be user friendly, I think10:56
yuikotakadadtantsur, ok, I will review client patches tomorrow. Today is too late already :)10:57
dtantsuryuikotakada, yeah, no prob :) I'm just clarifying the situation that we don't have to wait for resolution to land patches10:58
*** itsuugo has joined #openstack-ironic11:04
*** mgoddard1 has joined #openstack-ironic11:08
*** mgoddard has quit IRC11:08
*** itsuugo has quit IRC11:11
*** mtaylor is now known as mordred11:12
*** lazy_prince has joined #openstack-ironic11:13
*** e0ne has joined #openstack-ironic11:13
*** killer_prince has quit IRC11:16
*** itsuugo has joined #openstack-ironic11:19
*** MattMan has joined #openstack-ironic11:20
openstackgerritMerged openstack/ironic-python-agent: Fixes incorrect description URL  https://review.openstack.org/22365711:21
*** lucasagomes is now known as lucas-hungry11:23
*** ukalifon has left #openstack-ironic11:25
*** itsuugo has quit IRC11:26
*** mgoddard1 has quit IRC11:40
*** mgoddard has joined #openstack-ironic11:41
openstackgerritRamakrishnan G proposed openstack/ironic: Add support for in-band cleaning in ISCSIDeploy  https://review.openstack.org/22089811:43
*** yuikotakada has quit IRC11:47
*** stendulker has quit IRC11:49
*** boris-42 has quit IRC11:50
*** trown|outttypeww is now known as trown11:51
*** pelix has quit IRC11:51
trowngood morning rameshg87 ... still need me?11:52
*** pelix has joined #openstack-ironic11:53
*** romcheg has quit IRC11:54
*** romcheg1 has joined #openstack-ironic11:55
*** baoli has joined #openstack-ironic11:57
*** baoli has joined #openstack-ironic11:58
dtantsurmorning trown11:58
openstackgerritMerged openstack/ironic: Add config option to override url for links  https://review.openstack.org/22364011:58
*** baoli_ has joined #openstack-ironic11:59
trowngood morning dtantsur I will do some reviews for the inspector client today :), I will try to get a patch up for the data storage too, but that may be a stretch12:00
*** baoli has quit IRC12:03
*** itsuugo has joined #openstack-ironic12:05
openstackgerritSam Betts proposed openstack/ironic-inspector: Add API Discovery to Ironic Inspector  https://review.openstack.org/21506712:05
dtantsurtrown, thanks12:05
sambettsdtantsur: I've updated HTTP-API.rst in the API discovery patch12:07
dtantsurawesome!12:09
dtantsurI'll rereview once I finish with introspection rules client patch12:09
rameshg87trown: thanks, dtantsur just helped me out. I am just checking if it works12:17
rameshg87trown: will keep you posted12:17
rameshg87trown: good morning btw :)12:17
openstackgerritMerged openstack/ironic-python-agent: Follow-up to inspection patch 096830414b  https://review.openstack.org/22260512:17
trown:)12:20
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for introspection rules  https://review.openstack.org/22309612:20
*** nicodemos has joined #openstack-ironic12:22
*** itsuugo has quit IRC12:22
*** lucas-hungry is now known as lucasagomes12:23
*** jaypipes has quit IRC12:26
*** jaypipes_ has joined #openstack-ironic12:26
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: inspection: prepare for future deprecations  https://review.openstack.org/22305412:27
openstackgerritRamakrishnan G proposed openstack/ironic: Refactor deploy_utils methods  https://review.openstack.org/22226412:27
* rameshg87 goes home12:28
*** rameshg87 has quit IRC12:28
*** e0ne has quit IRC12:38
*** slagle_ is now known as slagle12:39
dtantsurtrown, if you didn't start your client part, and don't plan on right now, I can overtake it. I'm surprisingly free this afternoon :D12:41
trowndtantsur: cool, I have not started it. I am 50/50 whether I would get to it today.12:43
trowndtantsur: thanks12:43
dtantsurok, I'll ping you for review instead :)12:44
*** adam_g has quit IRC12:44
dtantsurI really feel like finishing client today12:44
*** thiagop has joined #openstack-ironic12:44
thiagopGood morning Ironic12:44
thiagopwhere can I find the list of priority reviews?12:44
*** wendar has quit IRC12:47
*** wendar has joined #openstack-ironic12:47
*** dims_ has quit IRC12:47
*** persia_ has quit IRC12:48
*** comstud has quit IRC12:50
*** persia has joined #openstack-ironic12:50
*** pelix has quit IRC12:50
*** adam_g has joined #openstack-ironic12:50
*** comstud has joined #openstack-ironic12:51
*** adam_g has quit IRC12:51
*** adam_g has joined #openstack-ironic12:51
lucasagomesthiagop, https://etherpad.openstack.org/IronicWhiteBoard12:52
lucasagomesthiagop, you may want to check https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo/edit?pli=1#gid=1338037272 as well12:52
*** rloo has joined #openstack-ironic12:52
thiagoptks lucasagomes12:52
lucasagomesyw12:55
*** smoriya has quit IRC12:55
*** shoutm has joined #openstack-ironic12:58
*** dims_ has joined #openstack-ironic12:58
*** saripurigopi has quit IRC13:00
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for getting introspection data  https://review.openstack.org/22407513:02
dtantsurtrown, ^^13:02
dtantsurtrown, I didn't test it yet, as my devstack is ancient and does not have store_data option... will rebuild now13:04
trowndtantsur: ya my testing is kind of blocked by this whole delorean nonsense, but I am getting a local delorean up now13:05
trownI should probably just get a devstack env up too13:05
*** cdearborn has joined #openstack-ironic13:07
dtantsurI can't avoid it, for upstream development devstack is still a must13:07
*** e0ne has joined #openstack-ironic13:09
*** pelix has joined #openstack-ironic13:13
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Fix upgrades by implementing indirection_api  https://review.openstack.org/22407913:14
*** xek has joined #openstack-ironic13:14
openstackgerritRuby Loo proposed openstack/ironic: Expose versioning information on GET / endpoint  https://review.openstack.org/20901213:15
*** itsuugo has joined #openstack-ironic13:17
rloohi dtantsur, trown, lucasagomes, thiagop13:18
lucasagomesrloo, hi there13:18
dtantsurrloo, o/13:18
rloodtantsur: what do you think the importance is of this bug? i think you wanted the fix soonish? https://bugs.launchpad.net/ironic/+bug/148917213:18
openstackLaunchpad bug 1489172 in Ironic "GET / endpoint doesn't provide versioning information" [Undecided,In progress] - Assigned to Ruby Loo (rloo)13:18
trowno/ rloo13:19
dtantsurrloo, not so much, but we needed it to provide detail versioning information to the ironicclient; then have something like `ironic version-supported 1.10`, then use it in devstack13:19
*** krtaylor has quit IRC13:20
rloodtantsur: ok. i saw another bug that was high that seemed similar: https://bugs.launchpad.net/ironic/+bug/147574413:21
openstackLaunchpad bug 1475744 in Ironic "driver properties not discoverable in REST API" [High,In progress] - Assigned to Anusha (anusha-iiitm)13:21
*** blakec has joined #openstack-ironic13:21
*** blakec has quit IRC13:22
xekrloo, dtantsur, lucasagomes  please look at https://bugs.launchpad.net/ironic/+bug/149381613:25
openstackLaunchpad bug 1493816 in Ironic "Rolling-upgrade: lack of indirection API" [Undecided,In progress] - Assigned to Grzegorz Grasza (xek)13:25
xekI hope this could be merged in this release, because we would have to backport it to enable upgrades in Mitaka13:25
*** getvasanth has quit IRC13:26
lucasagomesxek, there's an effort right now to move to oslo.objects13:26
lucasagomesit's not completed yet13:26
xeklucasagomes, I know13:26
lucasagomesxek, yeah I'm afraid the priority for it is also not high13:27
lucasagomes:-/13:27
* lucasagomes maybe we should bump it13:27
dtantsurifarkas, sambetts, could you please review https://review.openstack.org/#/c/223490 and https://review.openstack.org/#/c/223096/ (the 3rd patch pending testing)? would be really awesome to merge it today13:27
xekI re-implemented two oslo functions to accomodate the lack of oslo-style object registration13:27
ifarkasdtantsur, sure13:27
rloolucasagomes: would it be 'faster' to migrate to oslo.objects? I'm not quite sure where we're at with that.13:28
lucasagomesrloo, there are a couple of patches that tan lin have put up13:28
lucasagomesrloo, I dunno exactly how much is left tho, I need to take a look at it13:28
xekrloo, the indirection API is something that has to be implemented regardless of oslo integration13:28
lucasagomesrloo, fyi the bug tracking the changes is this one https://bugs.launchpad.net/ironic/+bug/146123913:28
openstackLaunchpad bug 1461239 in Ironic "replace objects with oslo.versionedobjects" [Low,In progress] - Assigned to Tan Lin (tan-lin-good)13:28
rloolucasagomes: yeah, but the bug doesn't say/indicate what's left. it just shows patches :)13:29
lucasagomesxek, ++13:29
rloolucasagomes: it would be good to fix the indirection API.13:29
lucasagomesindeed13:29
rlooi think it is 'high' priority cuz there's no other way right now to do rolling upgrades w/o downtime, is there?13:30
xekrloo, I'm afraid not13:30
lucasagomesyeah lets mark it as high and try to fix it soon13:31
xeklucasagomes, I already submitted a patch, it's similar to the one that was recently merged in Magnum13:32
lucasagomesxek, ah thanks! ok13:32
rloojroll: ^^ another high bug that we think we should try to get in for L: https://bugs.launchpad.net/ironic/+bug/149381613:32
openstackLaunchpad bug 1493816 in Ironic "Rolling-upgrade: lack of indirection API" [High,In progress] - Assigned to Grzegorz Grasza (xek)13:32
*** itsuugo has quit IRC13:39
*** achanda has joined #openstack-ironic13:40
*** achanda has quit IRC13:41
*** amotoki has quit IRC13:42
openstackgerritZhenguo Niu proposed openstack/ironic: Check image size before provisioning for agent driver  https://review.openstack.org/21525413:42
*** mtanino has joined #openstack-ironic13:42
*** zhenguo has quit IRC13:43
thiagopmorning rloo13:44
rloothiagop: :)13:46
*** nicodemos has quit IRC13:54
lucasagomesrloo, btw deleted vs delete14:02
lucasagomesrloo, I'm all about creating an alias for the deleted verb14:02
lucasagomesso user can use "delete" or "deleted", we could even bump the API later on and remove the "deleted"14:02
lucasagomesor simply bump it and remove the "deleted"14:02
BobBallOK - so now I don't understand how the ironic nodes work... The ram filter is now excluding all hosts because they have 0 RAM available, but the hosts clearly have RAM and a bulk introspection correctly found 6G: http://paste.openstack.org/show/464737/14:02
rloolucasagomes: yeah, I think we should support 'delete'. not sure if/when we can deprecate 'deleted'.14:03
*** thiagop has quit IRC14:03
lucasagomesrloo, yeah14:04
*** liliars_ has quit IRC14:04
lucasagomesrloo, if we bump the api version we can introduce delete14:04
lucasagomesnot sure if that worth tho, that's the problem14:04
*** stendulker has joined #openstack-ironic14:04
lucasagomesBobBall, what's the nova hypervisor-stats command says?14:04
rloolucasagomes: i was thinking we might want to revisit the state machine and see what is missing etc. but don't know when that might happen.14:05
BobBallAll 0s...14:05
rloolucasagomes: i think it is worth supporting 'delete' though.14:05
rloolucasagomes: but not a high priority given the other stuff we should get done in the next week or so.14:06
lucasagomesBobBall, are you using the IronicHostManager in nova? for the scheduling?14:06
lucasagomesrloo, yeah leave it for later, I added it to my todo here14:06
lucasagomesBobBall, http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-to-use-the-bare-metal-service14:07
rloolucasagomes: you could open a bug for it, see if anyone else wants to do it.14:07
lucasagomesrloo, ++ good idea, will do14:07
BobBallCurious lucasagomes... No.  But this is using RHEL OSP so I'm very confused as to how it works without using IronicHostManager14:09
openstackgerritVladyslav Drok proposed openstack/ironic: Add IPMINative vendor methods to *IPMINative drivers  https://review.openstack.org/22409914:10
lucasagomesBobBall, one thing that I can see in ur paste there that is wrong14:10
lucasagomesBobBall, is that you didn't set the deploy_kernel nor the deploy_ramdisk14:10
lucasagomesBobBall, if you do a ironic node-validate <node uuid>14:10
BobBallHumz.  Ok, thanks.  I'll look into that too14:11
BobBallThanks for the pointers14:11
lucasagomesBobBall, no problem, you tried to deploy that node and it failed with NoValidHost right?14:12
BobBallRight - because the Ram filter filtered it out14:12
*** r-daneel has joined #openstack-ironic14:12
lucasagomesif so, you may want to check the nova-compute logs to see why it failed14:12
lucasagomesok14:12
lucasagomesyeah fix the deploy_kernel and deploy_ramdisk, see if that correct stuff for u14:13
BobBallThanks14:13
NobodyCamgood morning Ironicers14:15
*** Marga_ has joined #openstack-ironic14:16
*** yog_ has joined #openstack-ironic14:16
openstackgerritMerged openstack/bifrost: Cleanups on role README files  https://review.openstack.org/22329714:16
*** lazy_prince has quit IRC14:17
NobodyCammorning rloo lucasagomes dtantsur sambetts jlvillal jroll BobBall and everyone else not listed here14:17
dtantsurmorning NobodyCam14:18
lucasagomesNobodyCam, hi there, good morning14:18
openstackgerritMerged openstack/bifrost: Correct pxe template for CoreOS config drive  https://review.openstack.org/22353414:18
jlvillalGood morning, NobodyCam rloo lucasagomes dtantsur sambetts jroll BobBall and ...14:19
lucasagomesjlvillal, hi there, good mroning14:19
lucasagomesmorning*14:19
dtantsurjlvillal, o/14:19
jlvillaldtantsur, o/  \o  o/ o/  :)14:20
NobodyCammornign :)14:21
dtantsurtrown, sambetts, ifarkas, rebuilt my devstack and tested https://review.openstack.org/#/c/224075/ as well, should be good to land14:25
ifarkasdtantsur, ok, adding to my review list14:25
dtantsurthanks :) sorry for the rush, but I'd really like to get a properly functional client for liberty14:26
*** sdake_ has joined #openstack-ironic14:27
*** alexpilotti has quit IRC14:29
rloomorning jlvillal, NobodyCam14:30
NobodyCam:)14:30
jlvillal:)14:31
openstackgerritMerged openstack/python-ironic-inspector-client: Add a proper client object  https://review.openstack.org/22349014:31
NobodyCam\o\ \o/ /o/14:31
*** garthb has joined #openstack-ironic14:31
*** openstackgerrit has quit IRC14:31
*** mgoddard has quit IRC14:32
*** openstackgerrit has joined #openstack-ironic14:32
* NobodyCam wounders if could make ascii version of y.m.c.a. lol14:32
*** mgoddard has joined #openstack-ironic14:32
openstackgerritJulia Kreger proposed openstack/bifrost: Add warning text to Debian package list  https://review.openstack.org/22412914:36
*** Marga_ has quit IRC14:36
TheJuliaNobodyCam: I'm guessing... yes.14:36
*** Marga_ has joined #openstack-ironic14:37
*** david-lyle has quit IRC14:37
jrollmorning everyone14:38
jrollrloo: thank you, added that bug to 4.214:39
openstackgerritMerged openstack/ironic: Add support for inband raid configuration agent ramdisk  https://review.openstack.org/19823814:39
*** vinbs has joined #openstack-ironic14:40
NobodyCammorning jroll :)14:40
jrollheya NobodyCam :)14:40
dtantsurjroll, morning14:40
NobodyCam:)14:40
openstackgerritMerged openstack/bifrost: Move dnsmasq to the end of the Debian package list  https://review.openstack.org/22381314:42
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for introspection rules  https://review.openstack.org/22309614:46
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for getting introspection data  https://review.openstack.org/22407514:47
*** mgoddard has quit IRC14:48
lucasagomesTheJulia, jroll good morning!14:48
TheJuliagood morning lucasagomes14:48
*** mgoddard has joined #openstack-ironic14:49
jrollmorning lucasagomes, dtantsur :)14:50
jrollironicclient gate should be good to go now, btw14:50
lucasagomeso/14:50
NobodyCamw00t14:52
*** vinbs has quit IRC14:57
*** vinbs has joined #openstack-ironic14:57
trowndtantsur: I got a devstack env up using the inspector plugin... where does the client code get put?14:57
dtantsurtrown, it's installed via pip by default. I just make a local chechout and test on it.14:58
*** dims_ has quit IRC14:58
trowndtantsur: ah that makes sense, thanks14:58
*** dims_ has joined #openstack-ironic14:58
*** jistr is now known as jistr|call14:58
*** boris-42 has joined #openstack-ironic15:01
*** openstackgerrit has quit IRC15:01
*** openstackgerrit has joined #openstack-ironic15:02
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for getting introspection data  https://review.openstack.org/22407515:05
BobBalllucasagomes: So it seems that the issue is that the extra_specs require capabilities:boot_option = local but the compute filter says that there are no capabilities to retrieve.  Any hints on that one?15:13
BobBallSorry for all these questions :)15:13
openstackgerritRamakrishnan G proposed openstack/ironic: Refactor agent {prepare,tear_down}_cleaning into deploy_utils  https://review.openstack.org/22089515:14
openstackgerritMerged openstack/ironic-inspector: Add API Discovery to Ironic Inspector  https://review.openstack.org/21506715:14
*** MattMan has quit IRC15:14
lucasagomesBobBall, heh yes. So basically the capabilities from the filter has to match with some node(s)15:15
*** MattMan has joined #openstack-ironic15:15
BobBallYes, but why might the node not have any capabilities - that's the bit I don't understand15:15
lucasagomesBobBall, looking at ur paste there's no capabilities set for that node, so you either: 1) use a flavor without a capability or 2) set it to the node15:15
lucasagomesBobBall, this is totally manual set15:15
*** nicodemos has joined #openstack-ironic15:15
lucasagomesyou can do15:16
*** dlpartain has joined #openstack-ironic15:16
lucasagomesironic node-update <node uuid> properties/capabilities="boot_option:local"15:16
openstackgerritRamakrishnan G proposed openstack/ironic: Add support for in-band cleaning in ISCSIDeploy  https://review.openstack.org/22089815:16
lucasagomesBobBall, or u can remove from flavor via: nova flavor-key baremetal unset capabilities:boot_option15:17
BobBallOkies.  Thanks.  I think this is all because I'm using a new virt_type for ssh.py and so can't create the nodes in the UI... I'll add the property to the nodes.15:17
*** thiagop has joined #openstack-ironic15:18
lucasagomesBobBall, yeah, sometimes UIs makes things easier for the normal uses cases15:18
lucasagomesbut once you divert a little from it ...15:18
BobBallIt's times like this that I'm glad I'm in a padded cell.  Easier on the head.15:19
lucasagomeslol15:19
* lucasagomes always wanted a punching bag in his house15:20
*** jaypipes_ is now known as jaypipes15:23
*** Marga_ has quit IRC15:24
*** Sukhdev_ has joined #openstack-ironic15:26
*** dlpartain has left #openstack-ironic15:28
*** shoutm has quit IRC15:30
*** lazy_prince has joined #openstack-ironic15:30
*** jistr|call is now known as jistr15:30
*** slo___ has joined #openstack-ironic15:30
*** david-ly_ has joined #openstack-ironic15:35
*** david-ly_ is now known as david-lyle15:41
*** Sukhdev_ has quit IRC15:44
*** Sukhdev has joined #openstack-ironic15:47
*** krtaylor has joined #openstack-ironic15:51
openstackgerritJulia Kreger proposed openstack/bifrost: Add warning text to Debian package list  https://review.openstack.org/22412915:52
*** tsekiyama has joined #openstack-ironic15:52
*** jistr has quit IRC15:54
*** devlaps has joined #openstack-ironic15:54
openstackgerritStephanie Miller proposed openstack/bifrost: Add warning text to Debian package list  https://review.openstack.org/22412915:55
*** garthb has quit IRC15:55
*** romcheg1 has quit IRC16:00
sambettsdtantsur: How does ironic's client get the nice formating for things like node-show??16:03
dtantsursambetts, it doesn't. JSON fields are outputted as they are with random wrapping, which gives some pain when trying to work with them16:04
sambettsdtantsur: I'm just looking at the shell code and I can't work out how and where that rendering happens, I assume its when take_action returns something right?16:05
dtantsursambetts, it's done somewhere in openstackclient. If we need it to look well, we probably have to return a string with \n and \t16:06
sambettsdtantsur: and the the openstack client thing deals with making it look nice?16:06
sambettsdtantsur: thats sad :(16:06
trownI think openstack client has something for that16:07
dtantsursambetts, last time I checked it didn't even do wrapping :) not that ironic client does not use openstackclient, it uses stuff from oslo-incubator16:07
*** ifarkas has quit IRC16:08
sambettsdtantsur: apprently openstackclient used the prettytable module to prettify16:08
trownah that16:08
*** Marga_ has joined #openstack-ironic16:09
*** marios has quit IRC16:09
openstackgerritRamakrishnan G proposed openstack/ironic: Add support for in-band cleaning in IloVirtualMediaIscsiDeploy  https://review.openstack.org/22417416:09
dtantsursambetts, trown, ok, I've checked and it doesn't it even wrap things16:10
*** marios has joined #openstack-ironic16:10
trowndtantsur: :(16:10
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Allow abort for CLEANWAIT states  https://review.openstack.org/20155216:12
lucasagomesrloo, when you have time (no rush, I'm still running tests on it) ^16:12
rloolucasagomes: ok, will look in a few minutes16:12
lucasagomesthat refactors part of the _do_next_clean_step() and put in the continue_node_cleaning() which is prior to changing the state of the node16:13
*** mjturek1 has quit IRC16:14
lucasagomesrloo, oh wait I forgot the logs again...16:15
rloolucasagomes: no worries, i can wait :)16:15
*** mgoddard has quit IRC16:16
*** romainh has left #openstack-ironic16:19
*** mgoddard has joined #openstack-ironic16:19
*** praneshp has joined #openstack-ironic16:21
openstackgerritMerged openstack/ironic-specs: Update nodes-tagging spec  https://review.openstack.org/19293516:21
*** sdake_ is now known as sdake16:23
*** cdearborn has quit IRC16:26
*** garthb has joined #openstack-ironic16:28
openstackgerritLilia Sampaio proposed openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279616:28
dtantsursambetts, so, what do you think about figuring it out later on?16:29
sambettsdtantsur: I would like to have the show command there even if it doesn't look too pretty, I think its something at least I would find useful, you can always just pass in "-f json" to view the raw json if the table view is ugly16:32
dtantsursambetts, note that "doesn't look too pretty" actually means "very ugly on any long conditions and/or actions"16:33
trownyou could always curl the API directly to get raw json16:33
trownya...like unreadable16:34
sambettswith OSC you can pass in several types of formatter, csv etc16:34
sambettsyamls another one16:34
sambettsand then I think we should bring up the wrapping issue wuth the OSC guys16:35
*** lazy_prince has quit IRC16:35
sambettsour command need only return the data, and improvments to OSC's pretty printing will just get picked up16:36
dtantsursambetts, even wrapper raw JSON if of somewhat limited use... I was thinking among the lines of creating separate commands for getting conditions/actions as a table16:36
dtantsur* wrapped16:36
*** praneshp_ has joined #openstack-ironic16:36
*** praneshp has quit IRC16:37
*** praneshp_ is now known as praneshp16:37
sambettsdtantsur: Even then your likely to run into wrapping issues on any long field values / descriptions16:39
dtantsursambetts, well, field is unlikely to occupy 100+ symbols, and conditions/actions don't have descriptions16:41
dtantsurbut yeah, this is why I wanted to defer any kind of show command: I don't know a sane way to do it16:41
dtantsurand I can't invent it given limited amount of time before we must call something liberty16:42
*** trown is now known as trown|lunch16:44
sambettsdtantsur: I just see all these different formatters http://docs.openstack.org/developer/cliff/list_commands.html#list-output-formatters and think that not including the show command because of just one of them (admittedly its the default one) might be a mistake, and it might not look bad if someones got a wide terminal :-p16:45
dtantsurhehe16:46
*** lekha is now known as lekha||away16:47
dtantsursambetts, so you don't like the idea of having separate tables, do you?16:47
dtantsursambetts, and if so, may I make it a separate patch tomorrow morning? this one is already big enough16:49
sambettsdtantsur: Not a big fan, I just think you'll just end up making 3 requests/commands everytime, because the rule, condition and action data's aren't really useful by without their sister data16:50
*** cdearborn has joined #openstack-ironic16:50
dtantsursambetts, gotcha. so what about splitting these patches?16:51
openstackgerritMerged openstack/ironic-inspector: Deprecate returning patches from plugins  https://review.openstack.org/22098716:51
sambettsdtantsur: making it a follow on?16:51
dtantsursambetts, yep. I'll work on it tomorrow morning.16:51
sambettsdtantsur: my only concern with that is the whole does that make the client version a new version thing? but I guess as its an addition not a modification or deletetion it should be fine16:52
dtantsursambetts, yeah, we won't change anything existing, so it's completely fine16:53
sambettsdtantsur: Ok yup happy with that, I'll +W the current one then16:55
dtantsurthanks16:55
openstackgerritNisha Agarwal proposed openstack/ironic: Implement take_over for iscsi_ilo driver  https://review.openstack.org/21706816:55
dtantsurleaving for a day now, see you tomorrow16:58
*** dtantsur is now known as dtantsur|afk16:58
*** derekh is now known as derekh_afk16:59
*** Sukhdev has quit IRC17:01
*** romcheg has joined #openstack-ironic17:02
openstackgerritMerged openstack/python-ironicclient: Remove unneeded param['detail'] setting  https://review.openstack.org/22224917:04
NobodyCamnight dtantsur|afk17:04
sambettsI'm off too, night all o/17:04
*** athomas has quit IRC17:04
*** praneshp has quit IRC17:05
openstackgerritMerged openstack/ironic-python-agent: Added home-page value in setup.cfg file  https://review.openstack.org/22376917:05
*** vinbs has quit IRC17:06
*** yog_ has quit IRC17:06
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Allow abort for CLEANWAIT states  https://review.openstack.org/20155217:08
*** achanda has joined #openstack-ironic17:09
lucasagomesrloo, now should be all good, I added the distinction when the clean step was aborted after or not for the last_error17:09
lucasagomesand with that, I'm going to call it a day folks17:09
lucasagomeshave a great night everyone!17:09
rloothx lucasagomes. probably won't get to it for 30 min or so.17:09
mariojv\o17:09
rloolucasagomes: night!17:10
lucasagomesrloo, no rush at all! Take ur time17:10
NobodyCamhave a good night lucasagomes17:11
lucasagomesyou too17:11
*** lucasagomes is now known as lucas-dinner17:11
BobBallIs there a way to get ironic to delay after changing the power state of a host?17:12
BobBallPower on command is being sent successfully then it seems less than 2 seconds later it checks if the power state is now On, but actually in my case the power state is still transitioning to on17:12
*** lsmola has quit IRC17:13
*** nicodemos has quit IRC17:16
*** e0ne has quit IRC17:19
*** yog_ has joined #openstack-ironic17:19
mariojvhas anyone ever seen a "Connection aborted" error when running ipa functional tests?17:20
mariojvi've been adding a few extra tests and can't reproduce with 1 test. it might have to do with running the tests multiple times consecutively with very little time in between, though.17:21
*** stendulker has quit IRC17:21
mariojvit only appears intermittently17:21
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Smart root disk selection including support for root device hints  https://review.openstack.org/21827117:22
*** stendulker has joined #openstack-ironic17:22
*** stendulker_ has joined #openstack-ironic17:24
*** stendulker has quit IRC17:24
mariojvnever mind - i think it had to do with the time we wait for IPA to come up17:28
*** mjturek1 has joined #openstack-ironic17:31
*** stendulker_ has quit IRC17:33
*** mjturek1 has quit IRC17:33
rlooJoshNang: will you have time to work on the zapping patch today? And/or would it help to have someone work on it with you?17:41
*** yog_ has quit IRC17:43
*** yog_ has joined #openstack-ironic17:46
*** trown|lunch is now known as trown17:46
*** pelix has quit IRC17:47
* jlvillal learned something new about decorators. Decorators that take arguments are only called once during the decoration process17:48
*** pelix has joined #openstack-ironic17:49
* jlvillal feels like https://s-media-cache-ak0.pinimg.com/originals/b2/8a/3a/b28a3a52ee99399a5389e758f7de87b6.gif after reading way too much about decorators :)17:50
JoshNangrloo: already working on it!17:50
thiagoplol17:51
JoshNangthanks for the comments. sorry for the sloppy first patchset..17:51
rlooJoshNang: cool!17:51
*** pelix has quit IRC17:51
*** sdake_ has joined #openstack-ironic17:54
*** praneshp has joined #openstack-ironic17:57
*** sdake has quit IRC17:58
*** e0ne has joined #openstack-ironic17:59
openstackgerritJohn L. Villalovos proposed openstack/ironic: Remove comment about exception decorator  https://review.openstack.org/22423818:15
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata  https://review.openstack.org/22424018:15
*** openstackgerrit has quit IRC18:16
*** openstackgerrit has joined #openstack-ironic18:17
*** e0ne has quit IRC18:18
*** e0ne has joined #openstack-ironic18:20
*** Sukhdev_ has joined #openstack-ironic18:22
*** mgoddard has quit IRC18:23
*** achanda has quit IRC18:25
*** david-lyle has quit IRC18:30
*** boris-42 has quit IRC18:30
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424718:34
mariojvlekha||away: you may be interested in the functional testing patch above18:35
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424718:39
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424718:39
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424718:40
*** romcheg has quit IRC18:46
*** romcheg has joined #openstack-ironic18:46
*** yog_ has quit IRC18:48
*** slo___ has quit IRC18:49
openstackgerritJohn L. Villalovos proposed openstack/ironic: Convert functools.wraps() usage to six.wraps()  https://review.openstack.org/22426319:03
*** achanda has joined #openstack-ironic19:12
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Fix func test job broken by Inspector func tests changes  https://review.openstack.org/22426819:19
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for introspection rules  https://review.openstack.org/22309619:20
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Support for getting introspection data  https://review.openstack.org/22407519:21
*** harshs has joined #openstack-ironic19:30
*** e0ne has quit IRC19:35
*** achanda_ has joined #openstack-ironic19:40
*** achanda has quit IRC19:40
*** sdake has joined #openstack-ironic19:49
*** sdake_ has quit IRC19:52
*** openstackstatus has quit IRC20:02
*** mestery has quit IRC20:02
*** openstackstatus has joined #openstack-ironic20:03
*** ChanServ sets mode: +v openstackstatus20:03
*** mestery has joined #openstack-ironic20:03
*** sambetts has quit IRC20:05
*** derekh_afk is now known as derekh20:06
*** sambetts has joined #openstack-ironic20:10
*** derekh has quit IRC20:11
*** Sukhdev_ has quit IRC20:13
*** olaph has joined #openstack-ironic20:19
mariojvis this bug actually valid? https://bugs.launchpad.net/ironic/+bug/1481665 i thought you could just pass async=True to the decorator and acquire a shared lock to get that functionality20:20
openstackLaunchpad bug 1481665 in Ironic "Exclusive locking should be optional in vendor passthru" [Wishlist,Triaged]20:20
jrollvendor passthru always takes a lock20:21
rloomariojv: i believe it is still valid. did you look at the conductor code to verify?20:21
mariojvi'm checking now20:21
jrollyeah, imbw :)20:22
jrollhttps://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L49520:22
jrollso yeah, always exclusive20:22
mariojvah, i was looking at ironic/drivers/base20:23
mariojvthanks20:23
jrollnp20:23
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix string formatting issues  https://review.openstack.org/22286220:23
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Remove DictMatches custom matcher from unit tests  https://review.openstack.org/22428720:23
jrollI hate that, I really wish heartbeats didn't lock20:23
*** ukalifon has joined #openstack-ironic20:25
JoshNangjroll: that timeout i was getting on those tests? must have been tox 1.6. had to upgrade to 1.8 for nova tests last night, works fine now20:26
jrolloh wird20:26
jrollweird, too20:26
openstackgerritMerged openstack/ironic: Replace metaclass registry with explicit opt-in registry from oslo  https://review.openstack.org/21758420:34
*** ndipanov has quit IRC20:37
*** lucas-dinner has quit IRC20:40
*** puranamr has joined #openstack-ironic20:42
*** puranamr has quit IRC20:47
*** sinval has joined #openstack-ironic20:51
openstackgerritRuby Loo proposed openstack/ironic: Update descriptions in RAID config schema  https://review.openstack.org/22429920:55
*** ijw_ has joined #openstack-ironic21:01
jlvillalrloo: jroll: Any idea why Jenkins would only have four jobs listed on this patch?  https://review.openstack.org/#/c/224287/21:01
jlvillalSeems odd to me...21:01
rloojlvillal: dunno and it doesn't seem right.21:02
jlvillalrloo: Maybe Jenkins doesn't do the other tests since there a dependent patch?21:02
jlvillalrloo: I think they were pushed together. So maybe Jenkins is smart...21:02
jlvillalI will go with that theory for now :)21:02
rloojlvillal: doubtful. i'm in a meeting now.21:03
jrolljlvillal: we no longer run tempest etc on tests-only changes21:03
jrollsame for docs changes21:03
jlvillaljroll: Oh. Thanks.21:03
*** ukalifon has quit IRC21:04
jrolljlvillal: https://review.openstack.org/#/c/221086/21:04
jlvillalAh, very new change :)21:04
jlvillaljroll: Thanks for the info.21:04
jrollnp :)21:05
jrollnow we can say ironic is green because we save gate resources :P21:05
jlvillalheh :)21:05
rloojroll: awesome. that's great.21:06
rloojroll: do we want that for all the ironic-related projects, or just ironic?21:07
jrollrloo: I suppose it wouldn't hurt for the other projects :)21:07
rloojroll: let's go green aaaaalllllllll the way :D21:08
jroll:D21:08
JayFno tempest on docs changes21:10
JayFmeans I'll make more docs changes21:10
*** ijw_ has quit IRC21:14
*** ijw_ has joined #openstack-ironic21:15
*** puranamr has joined #openstack-ironic21:16
*** harshs has quit IRC21:25
*** boris-42 has joined #openstack-ironic21:26
*** Sukhdev has joined #openstack-ironic21:27
*** puranamr has quit IRC21:28
JayFCan I get a 3rd opinion on the comment here? https://review.openstack.org/#/c/220313/21:35
*** trown is now known as trown|outttypeww21:35
jrollJayF: anything specifically?21:37
jrolloh heh21:37
jrollI'd say :param *args: positional arguments to pass to processutils.execute21:38
jrollditto for **kwargs, s/positional/keyword21:38
JayFoooh that's a GREAT idea21:38
jrollidk about UnknownArgumentError but it can definitely raise ProcessExecutionError21:38
jrolland that'21:38
jrollthat's good to have to not have to dig into oslo to find out what that can raise21:39
*** harshs has joined #openstack-ironic21:45
*** achanda_ has quit IRC21:45
*** krtaylor has quit IRC21:46
jlvillalJayF: Looks like it could raise OSError too21:47
jlvillalJayF: Based on the function below...21:48
mariojvis UnknownArgumentError real? i don't think i've seen it before21:54
*** harshs has quit IRC21:54
mariojvi'm thinking that's an ironic but not an ipa thing?21:55
*** david-lyle has joined #openstack-ironic21:56
*** achanda has joined #openstack-ironic21:59
mariojvactually, the commenter is right, it might raise that error: https://github.com/openstack/oslo.concurrency/blob/24cf1005467f5d59db83b3c59bdfb58590079010/oslo_concurrency/processutils.py#L18922:00
*** ijw_ has quit IRC22:01
openstackgerritJosh Gachnang proposed openstack/ironic: Add Zapping states to node and API  https://review.openstack.org/22194922:05
openstackgerritJosh Gachnang proposed openstack/ironic: Support Zapping in the conductor  https://review.openstack.org/22198922:05
*** achanda has quit IRC22:07
*** achanda has joined #openstack-ironic22:09
*** cdearborn has quit IRC22:10
*** thiagop has quit IRC22:16
*** achanda_ has joined #openstack-ironic22:17
*** achanda has quit IRC22:17
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Ensure all methods in util.py have docstrings.  https://review.openstack.org/22031322:21
JayFjroll: ^ used your strategy, thanks22:22
JayFmariojv: ^ also thanks for being my lazyweb, lol22:22
JayFI realized my response comment was never posted on there22:22
mariojvnp :)22:22
JayFso you guys had no idea what I was talkign about at first22:22
JayFheh22:22
JayFbut it's all fixed now I hope22:23
anteayaanyone in here involved in working on the Grenoble mid-cycle in Feb? I'm curious about how hard it was to organize22:23
NobodyCamanteaya: devananda handled that22:23
JayFanteaya: I think BadCub and devananda did most of the heavy lifting22:24
anteayacool, thanks I will find them22:24
JayFGOOD AFTERNOON NobodyCam :P22:24
NobodyCam:)22:24
NobodyCamhey hey JayF :)22:24
JayFThree things certain in life: Death, Taxes, and a "good morning" ping from NobodyCam making me feel guilty for getting into IRC late :P22:24
NobodyCamlol22:25
NobodyCamhehehe22:25
NobodyCam*(blush)*22:25
BadCubJayF: anteaya Devananda was more involved in the Grenoble mid-cycle, but I did all the planning and coordinating for the Seattle Mid-Cycle we just held.22:26
BadCubJay22:27
BadCubJayF: LOL ^^22:27
BadCubNobodyCam: leave good morning messages everywhere. They are like inspirational posters!22:27
NobodyCamlol22:27
*** Sukhdev has quit IRC22:28
*** david-lyle has quit IRC22:29
*** krtaylor has joined #openstack-ironic22:32
anteayaBadCub: ah thank you22:38
anteayawondering why we only have 1 out of 21 meetups outside the states22:38
jlvillalanteaya: Suggest choosing low-cost location for outside of US.22:40
BadCubanteaya: I am not sure. I think it kinda comes up for vote on a location that is conducive to getting the most amount of attendees? I am just guessing there.22:40
jlvillalAt least for me probably easier to get approval to stay in Prague vs Paris :)22:40
BadCubanteaya: are you looking to host an Ironic mid-cycle?22:40
anteayano check the infra backscroll22:41
anteayafixed the sprints wikipage22:41
anteayaonly 1 out of 21 sprints are outside the states22:41
anteayanot so good for diversity22:41
anteayaI'm hearing taht the person organizing needs to be local22:41
anteayafort collins had 4 meetup22:41
anteayas22:41
anteayabecause there is a person here (at the qa sprint) willing to organize it22:42
BadCubI would assume there is some logistics involved in the planning. I can only speak for what I know from planning for Ironic, not other groups.22:42
anteayaright22:44
anteayaand site specific stuff22:44
anteayawhich someone not in that country often decides takes too much time to figure out22:44
anteayahence 20 meetups in the states22:44
BadCubThat could be the case. I know finding the location for our sprint in Seattle was a chore in itself.22:45
anteayayeah22:45
anteayawhich is why 4 meetups were in fort collins this year22:45
anteayait is easy to contact the organizer and they organize it22:46
anteayathe ptl has to do very little22:46
BadCubYep. And we held ours in a hotel banquet room. We had no real assistance from any planners22:46
anteayawhich personally is my preference22:46
anteayabut you need someone willing to organize it22:47
jlvillalBadCub: I thought the Seattle mid-cycle was great :)22:47
anteayaand they have to be native to the country it is held in22:47
BadCubthanks jlvillal :) I thought it was pretty rock'in too22:47
anteayaI had to write a letter so someone could get a visa for canada22:47
anteayacouldn't do that for another country22:47
BadCubanteaya: yeah, finding someone who is willing to organize is a bit more tricky. And yes, visa issues can be a pain.22:47
*** romcheg has quit IRC22:48
*** dims_ has quit IRC22:48
BadCubI actually had fun organizing the sprint for Ironic.22:48
anteayaawesome!22:49
anteayaI'm glad to hear that22:49
anteaya:)22:49
anteayathanks22:49
BadCubI think folks may be scared of the perceived amount of work it may take. But once you do one, it is actually pretty easy22:49
BadCubyw :)22:49
*** Sukhdev_ has joined #openstack-ironic22:51
anteaya:)22:52
JayFjroll: What ever happened to the long running agents spec? cc: mariojv22:54
jrollit grew some crust23:00
jrollJayF: https://review.openstack.org/#/c/102405/23:00
jrollanteaya: of course this will depend on the team, but we didn't have a very good turnout for the grenoble midcycle :(23:01
anteayajroll: oh?23:01
anteayajroll: how many folks attended?23:02
anteayayeah attendance is a statistic I don't have23:02
jrollanteaya: I don't know, because I wasn't there either :)23:02
anteayajroll: ah23:02
jrollhowever I know it was roughly 1/3 of the core team23:02
anteayawhy didn't you attend?23:02
openstackgerritMerged openstack/ironic: Remove comment about exception decorator  https://review.openstack.org/22423823:02
jrollwell...23:02
jrollsince core attendance was low, we scheduled *another* midcycle in SF23:03
jrolland my employer preferred I went to that23:03
anteayaafter grenoble?23:03
jrollcorrect23:03
anteayaor it was scheduled before grenoble?23:03
jrollit was scheduled before, sorry, after rough RSVP numbers were in23:03
anteayadid you fork your own mid-cycle?23:03
anteayaright23:03
anteayaso you forked your own mid-cycle23:04
anteayathat colours things a bit23:04
jrollkind of :)23:04
jrolldeva organized both23:04
anteayasounds like the second was due to pressure23:04
jrollanteaya: it was due to sads23:04
anteayadue to sads?23:04
anteayacan you expand?23:05
anteayaI feel I am missing detail23:05
jrolldeva was sad that many people were missing the midcycle due to budget constraints and such23:05
anteayawas that the reason given for low rsvps?23:05
anteayabudget23:05
jrollanteaya: http://lists.openstack.org/pipermail/openstack-dev/2014-December/053618.html23:06
jrollI'm not sure23:06
anteayathank you23:06
* anteaya reads23:06
jrollthat was the reason more people from my company could not come23:06
anteayabudget23:06
jrollyes.23:07
*** ijw_ has joined #openstack-ironic23:07
anteayaokay thank you23:07
jrollanteaya: you're welcome, this sort of thing is interesting to me, so let me know if you want to talk more about it :)23:08
anteayaoh sure23:10
anteayaactually fungi thought having someone bring it up at tomorrow's diversity working group meeting would be good23:11
anteayaI don't think I will be available23:11
anteayabut happy if someone with thoughts on the matter shared the stats of 1 of 21 midcycles outside the states for liberty23:11
anteayahttps://wiki.openstack.org/wiki/Sprints23:12
anteayaand that ironic's attempt to have it in europe ended up forking the mid-cycle23:13
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Ensure all methods in util.py have docstrings.  https://review.openstack.org/22031323:13
JayFjlvillal: ^ thanks for catching the typo, fixed now.23:13
anteayaso rather than getting everyone together somewhere you had two groups23:13
anteayawhich is not really the point23:13
anteayafrom my perspective23:13
JayFjroll: do you think we should revive long running agents for M? I feel like the topic of when agents are running comes up more and more (thinking clean_step API as one of many)23:14
jrollanteaya: I could probably join, is there more info on the meeting somewhere? time etc?23:14
anteayaso maybe companies just expect them to be in the states23:14
anteayaand if they are apply pressure until they are in the states23:14
JayFanteaya: it's incredibly difficult to get 2x international trips approved this year23:14
anteayaif they are not23:14
JayFanteaya: and given that choice, we tend to choose summit23:14
anteayaoh I hear that23:14
* JayF manager of jroll's team23:14
anteayabudget is a concern for sure23:14
JayFlike I try very hard not to advocate hard for moving things to the US so we can come23:14
anteayabut folks in europe have to do 2X internations trips to attend mid-cycles23:15
JayFbut realistically, I've shipped people on my team to openstack meetups more than I've shipped my team all to the same place to work together23:15
JayFlol23:15
jrollJayF: I'd like to revive that in general, though I don't think it really solves any of those problems since it will need to be optional :)23:15
jroll(the agent thing, not the travel thing) :P23:15
JayFjroll: I think it forces us to solve those problems in a generic way other pieces can consume23:15
anteayaI'm not saying their are not limitations23:15
*** Sukhdev_ has quit IRC23:15
anteayabut the result is we have less eurpoean and apac devs sharing the experience23:16
anteayawhich affects diversity23:16
anteayaI have no solutions23:16
anteayajust collecting data23:16
jrollyep, totally agree23:16
jrollanteaya: I would totally speak on ironic's behalf in that meeting tomorrow if I have the info :)23:17
anteayajroll: well you have what I have23:17
jroll(or convince deva to do it, since he was at both, heh)23:17
jrollall I know is "tomorrow"23:17
anteayathis kicked off when I fixed the wikipage23:17
anteayajroll: fungi mentioned the diversity working group23:17
JayFanteaya: I'd really love to think about virtual meetup formats23:17
jrollaccording to the wiki they haven't had a meeting since july https://wiki.openstack.org/wiki/Diversity23:17
anteayajroll: you want to ask fungi in -infra he may know more23:18
jrolloh I'm an idiot, I read backwards23:18
jrolltomorrow 1700 utc23:18
* jroll marks his calendar23:18
JayFanteaya: Timezones are still a problem, but I think having people doing more collaboration over video rather than flying people out are a great potential solution fort his23:18
anteayaJayF: https://wiki.openstack.org/wiki/VirtualSprints23:18
anteayaJayF: that is the format23:18
anteayahelp yourself23:18
anteayawell we have yet to find an open source video tool23:19
anteayainfra is open to suggestions if you find one23:19
jrollright, so there's also the question of midcycle vs sprint, where I believe the tendency is midcycle is to talk about work and sprint is to do work23:19
anteayathe qa sprint has been both23:19
jrollthe former needs to be syncronous, the latter does not (as much)23:19
jrolloh, neat23:19
jrolldid folks make an attempt to work the same timezone?23:20
anteayawe designed the layout for a dashboard tool23:20
anteayawe are here23:20
anteayain fort collins23:20
anteayaso we all got here round about 923:20
anteayaand it is after 5 on the last day and still working23:20
jrollso it is a partially-virtual sprint?23:20
anteayano23:20
jrolloh, I see, misread23:21
anteayahttps://wiki.openstack.org/wiki/QA/CodeSprintLibertyFortCollins#How_to_access_the_code_sprint_remotely23:21
anteayafolks signed up to attend remotely23:21
anteayaprompting mtreinish to write that paragraph23:21
jrollcool23:21
jrollis the remote bit working well?23:21
anteayawe got a lot done on the dashboard because we were in person23:21
anteayafor those who choose a virtual sprint it suits their needs23:22
anteayafor instance I ran one on documentation23:22
anteayawe all knew what needed to be done23:22
anteayabut never had the time to do it23:22
anteayano design questions were needing to be hashed out23:22
anteayajust content written23:22
anteayaso we wrote content for 2 days23:22
anteayaand were successful23:23
jrollI see, cool23:23
anteayadepends on the need23:23
JayFWe have had similar openstack bug squashing events inside Rackspace+with partners23:23
anteayano way could we have designed the dashboard views virtually23:23
anteayaright23:23
anteayaso in person is necessary sometiems23:23
anteayaand virtual works sometimes23:23
JayFanteaya: is the real blocker for doing good design work remotely the lack of an OSS video platform?23:23
anteayaand european and apac devs miss out on the in person things disproportionately23:24
anteayano23:24
anteayait is standing beside someone23:24
anteayaand listening to the pace of their breathing23:24
anteayaand having beers with them later23:24
jrollunfortunately, this is true23:24
JayFanteaya: I really, really wish we could find a way around that.23:24
anteayafolks who try to find a way around the value of direct human interaction often end up being disappointed23:25
anteayait adds value to life23:25
jrollJayF: the nova midcycle had someone on video during the midcycle and it wasn't terribly helpful, it's unclear if that was poor hangout quality or the 50:1 ratio of in-person:video23:25
anteayafor some it is the value of life23:25
JayFanteaya: Oh, I completely agree that it's better/more fun/etc, I just don't think every company shipping people who work on openstack somewhere 4 times a year is sustainable either23:25
*** romcheg has joined #openstack-ironic23:26
anteayaI don't have a horse in that race23:27
anteayaand have no interest in being dragged into that one23:27
anteayaso I'll pass23:27
anteayathe point is diversity23:27
jrollyeah23:28
jrollbut if we can't get some of the largest companies (HP, red hat) to send everyone working on the project to these things23:28
jrollit's only going to be worse with other smaller companies :(23:28
jrolland company diversity is important too23:28
JayFjroll: you made my pointbetter than I did23:29
jrollheh, I didn't even realize that was your point :)23:30
anteayaso your argument is taht hp and red hat should pay travel expenses for folks who are not their employees?23:30
jrollno23:31
jrollI'm saying if HP and red hat can't afford it, how will the little guys?23:31
* anteaya listens again23:31
anteayaah okay23:32
anteayaI don't have an answer to taht question23:32
anteayaexcpet the little guys have employees in europe too23:32
jrolloh, I know, just pointing out that it's yet another problem23:32
jrollsure23:32
*** dims_ has joined #openstack-ironic23:32
*** baoli_ has quit IRC23:37
anteayayes23:37
anteayaI have to go offline now and have dinner23:37
anteayajroll JayF thank you for the chat23:37
jrollok, have a good night :)23:37
anteayait is very valuable23:37
jrollthank you!23:37
JayFnp, enjoy your dinner23:37
anteayathanks you too23:37
anteaya:)23:37
anteayaJayF: thanks :)23:37
*** chlong has quit IRC23:41
*** romcheg has quit IRC23:45
*** Sukhdev has joined #openstack-ironic23:52

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