Friday, 2015-09-18

*** naohirot has joined #openstack-ironic00:01
*** achanda_ has joined #openstack-ironic00:02
*** achanda has quit IRC00:06
*** derekh has joined #openstack-ironic00:06
*** achanda_ has quit IRC00:08
*** garthb has quit IRC00:10
openstackgerritRuby Loo proposed openstack/ironic: Refactor deploy_utils methods  https://review.openstack.org/22226400:10
*** shoutm has quit IRC00:17
*** bizarrochristy has joined #openstack-ironic00:19
*** shoutm has joined #openstack-ironic00:20
*** harshs has joined #openstack-ironic00:21
*** alexpilotti has quit IRC00:22
*** mtanino has quit IRC00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-ironic00:23
openstackgerritRuby Loo proposed openstack/ironic: Agent Inband RAID configuration available in cleaning  https://review.openstack.org/22493800:25
rloojroll, JoshNang: for inband raid/cleaning: https://review.openstack.org/#/c/224938/00:26
rloojroll, JoshNang: there is a patch that ^ depends on. And it occurred to me that I didn't finish reviewing that bit of code from ramesh00:26
rloojroll, JoshNang: there was something funky about how the RAID clean steps got called, cuz it had to get the steps from the ipa hw manager, and i didn't get around to figuring out if it all worked.00:27
rloojroll, JoshNang: do with it what you want. I'll touch base with you next week.00:28
*** rloo has quit IRC00:28
*** shoutm_ has joined #openstack-ironic00:29
*** shoutm has quit IRC00:32
openstackgerritBrian Moss proposed openstack/ironic-specs: [ironic-specs] Update links to Cloud Admin Guide  https://review.openstack.org/22494000:34
*** derekh has quit IRC00:38
*** dims_ has joined #openstack-ironic00:38
*** david-lyle has quit IRC00:40
*** dims_ has quit IRC00:43
*** dims_ has joined #openstack-ironic00:43
*** david-lyle has joined #openstack-ironic00:44
*** trown is now known as trown|outttypeww01:05
*** penick has quit IRC01:05
*** blakec has quit IRC01:05
*** shuquan has joined #openstack-ironic01:25
*** baoli has joined #openstack-ironic01:26
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/22461701:27
*** dims_ has quit IRC01:27
*** dims_ has joined #openstack-ironic01:29
*** dims_ has quit IRC01:37
jrollJoshNang: if you know what rloo meant up there, pls to translate :)01:42
* jroll sends emails01:42
*** bizarrochristy has quit IRC01:48
*** chenglch has joined #openstack-ironic01:51
*** baoli has quit IRC01:52
*** puranamr has joined #openstack-ironic02:07
*** sdake_ has joined #openstack-ironic02:10
*** sdake has quit IRC02:13
*** bizarrochristy has joined #openstack-ironic02:17
*** achanda has joined #openstack-ironic02:28
*** zhenguo has joined #openstack-ironic02:29
*** shoutm_ has quit IRC02:38
*** baoli has joined #openstack-ironic02:39
*** bizarrochristy has quit IRC02:43
*** bizarrochristy has joined #openstack-ironic02:44
*** bizarrochristy has quit IRC02:47
*** yog_ has quit IRC02:49
*** Haomeng|2 has joined #openstack-ironic02:49
*** shoutm has joined #openstack-ironic02:49
*** Haomeng has quit IRC02:52
*** harshs has quit IRC02:59
*** blakec has joined #openstack-ironic03:05
*** baoli has quit IRC03:05
*** baoli has joined #openstack-ironic03:08
*** shuquan has quit IRC03:15
*** baoli has quit IRC03:17
*** dims_ has joined #openstack-ironic03:22
*** achanda has quit IRC03:27
*** dims_ has quit IRC03:28
*** sdake has joined #openstack-ironic03:29
*** sdake_ has quit IRC03:32
*** harshs has joined #openstack-ironic03:33
openstackgerritMerged openstack/ironic: Update descriptions in RAID config schema  https://review.openstack.org/22429903:36
*** garthb has joined #openstack-ironic03:36
*** sdake_ has joined #openstack-ironic03:38
*** sdake_ has quit IRC03:39
*** sdake_ has joined #openstack-ironic03:40
*** VikasC has joined #openstack-ironic03:41
*** sdake has quit IRC03:41
*** yog_ has joined #openstack-ironic03:42
*** garthb has quit IRC03:42
*** penick has joined #openstack-ironic03:45
*** puranamr has quit IRC03:45
*** sdake_ is now known as sdake03:48
*** shoutm_ has joined #openstack-ironic04:00
*** shoutm has quit IRC04:02
*** puranamr has joined #openstack-ironic04:16
*** puranamr has quit IRC04:20
*** lazy_prince has joined #openstack-ironic04:27
*** harshs has quit IRC04:29
*** achanda has joined #openstack-ironic04:40
*** shoutm has joined #openstack-ironic04:55
*** harshs has joined #openstack-ironic04:55
*** shoutm_ has quit IRC04:59
*** pradipta has joined #openstack-ironic05:14
*** shoutm_ has joined #openstack-ironic05:17
*** shoutm has quit IRC05:17
*** shoutm has joined #openstack-ironic05:20
*** shoutm_ has quit IRC05:22
*** harshs has quit IRC05:22
*** vishwana_ has quit IRC05:26
*** vinbs has joined #openstack-ironic05:27
*** coolsvap|holiday is now known as coolsvap05:28
*** Marga_ has quit IRC05:30
*** openstackgerrit has quit IRC05:31
*** openstackgerrit has joined #openstack-ironic05:32
*** Marga_ has joined #openstack-ironic05:48
*** sdake has quit IRC05:49
*** Marga_ has quit IRC05:49
*** puranamr has joined #openstack-ironic05:59
*** shoutm has quit IRC06:03
*** puranamr has quit IRC06:04
*** shoutm has joined #openstack-ironic06:06
*** Marga_ has joined #openstack-ironic06:10
*** Marga_ has quit IRC06:16
*** Marga_ has joined #openstack-ironic06:17
*** jroll has quit IRC06:27
*** eglute has quit IRC06:27
*** jroll has joined #openstack-ironic06:33
*** eglute has joined #openstack-ironic06:33
*** shoutm has quit IRC06:40
*** e0ne has joined #openstack-ironic06:42
*** e0ne has quit IRC06:45
*** shoutm has joined #openstack-ironic06:45
*** e0ne has joined #openstack-ironic06:48
*** lsmola has joined #openstack-ironic06:50
*** e0ne has quit IRC06:55
*** achanda has quit IRC06:57
*** e0ne has joined #openstack-ironic06:59
*** Marga_ has quit IRC06:59
*** Marga_ has joined #openstack-ironic07:00
*** karimb has joined #openstack-ironic07:02
*** e0ne has quit IRC07:10
*** Marga_ has quit IRC07:14
*** VikasC has quit IRC07:20
*** HenryG has quit IRC07:21
*** MattMan has joined #openstack-ironic07:25
*** karmatronic has joined #openstack-ironic07:34
*** ifarkas has joined #openstack-ironic07:34
*** Marga_ has joined #openstack-ironic07:35
*** Marga_ has quit IRC07:35
*** Marga_ has joined #openstack-ironic07:35
*** e0ne has joined #openstack-ironic07:37
*** karimb has quit IRC07:38
*** dtantsur|afk is now known as dtantsur07:41
dtantsurMorning Ironic07:42
betherlyMorning dtantsur :)07:44
betherlyMorning ironic07:44
dtantsuro/07:44
*** e0ne has quit IRC07:44
*** romainh has joined #openstack-ironic07:46
openstackgerritAnton Arefiev proposed openstack/ironic-specs: Policy improvements  https://review.openstack.org/22177807:53
*** athomas has joined #openstack-ironic07:56
*** karmatronic has quit IRC07:57
*** karimb has joined #openstack-ironic07:58
*** VikasC has joined #openstack-ironic07:59
*** lucasagomes has joined #openstack-ironic08:02
*** lucasagomes_ has joined #openstack-ironic08:02
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Fix rolling upgrades by implementing indirection_api  https://review.openstack.org/22407908:03
*** lucasagomes has quit IRC08:04
*** lucasagomes_ is now known as lucasagomes08:04
*** jistr has joined #openstack-ironic08:07
*** dtantsur is now known as dtantsur|brb08:11
*** HenryG has joined #openstack-ironic08:12
*** derekh has joined #openstack-ironic08:15
*** HenryG has quit IRC08:16
lucasagomesjroll, jlvillal hah damn the tripleo gate is pretty flaky right now! Thanks for all the rechecks08:26
lucasagomesit keeps failing for random stuff08:26
lucasagomesand good morning all!08:26
derekhlucasagomes: the tripleo jobs isn't why your patch hasn't merged (they are not in the gate)08:29
lucasagomesderekh, yeah not the tripleo-ci08:29
lucasagomesI mean the dib gate jobs08:29
*** dims_ has joined #openstack-ironic08:30
lucasagomesderekh, you know if the mirror problem is gone?08:30
*** HenryG has joined #openstack-ironic08:30
derekhlucasagomes: cool, ;-)  btw I know the tripleo test is busted also because of a package rename last night08:31
lucasagomesderekh, oh :-(08:32
lucasagomesderekh, there's a fix on the way? I plan to do some work on the tripleo-ci today to make it use the IPA ramdisk instead of the bash one08:33
lucasagomessince now we basically deprecated the bash ramdisk already08:33
derekhlucasagomes: I'm working on it, basically the switchover from ironic-discovery to ironic-inspector is causing us trouble, I'll let you know how far I got but it would be quick08:34
*** dims_ has quit IRC08:34
lucasagomesderekh, ok, thanks much08:35
openstackgerritBob Ball proposed openstack/ironic: Handle unquoted node names from virt types  https://review.openstack.org/22477008:35
*** shoutm has quit IRC08:36
*** VikasC has quit IRC08:39
*** blakec has quit IRC08:43
*** ndipanov has joined #openstack-ironic08:43
*** chlong has quit IRC08:47
*** romcheg has joined #openstack-ironic08:50
*** blakec has joined #openstack-ironic08:54
*** shuquan has joined #openstack-ironic09:05
*** ndipanov is now known as ndipanoff09:11
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Allow abort for CLEANWAIT states  https://review.openstack.org/20155209:17
openstackgerritBlake Caldwell proposed openstack/ironic: Override PXE kernel cmdline via Glance property  https://review.openstack.org/17822909:18
openstackgerritWang Wei proposed openstack/ironic: Fix incorrect urls  https://review.openstack.org/22507509:18
*** dtantsur|brb is now known as dtantsur09:19
*** blakec has quit IRC09:24
*** e0ne has joined #openstack-ironic09:27
*** pelix has joined #openstack-ironic09:29
openstackgerritSam Betts proposed openstack/ironic-inspector: Add alembic migrations for the inspector database  https://review.openstack.org/22366509:29
*** dims_ has joined #openstack-ironic09:32
*** dims_ has quit IRC09:36
openstackgerritZhenguo Niu proposed openstack/ironic: Check image size before provisioning for agent driver  https://review.openstack.org/21525409:49
*** dims_ has joined #openstack-ironic09:50
*** zhenguo has quit IRC09:50
*** bcornec has quit IRC09:51
*** shuquan has quit IRC09:54
*** pradipta has quit IRC09:56
*** naohirot has quit IRC09:58
*** romcheg has quit IRC10:02
*** athomas has quit IRC10:03
*** athomas has joined #openstack-ironic10:08
openstackgerritSam Betts proposed openstack/ironic-inspector: Add alembic migrations for the inspector database  https://review.openstack.org/22366510:15
*** romainh has quit IRC10:19
*** romcheg has joined #openstack-ironic10:23
openstackgerritMerged openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/22467010:24
*** rameshg87 has joined #openstack-ironic10:32
*** chenglch has quit IRC10:33
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Add "logs" and "extra-hardware" inspection collectors  https://review.openstack.org/22509210:33
sambettsMorning all o/ just realised I was lurking without saying Hi10:37
openstackgerritMerged openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/22467110:38
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Add logging for _rest_op method in RIS  https://review.openstack.org/21963610:41
lucasagomessambetts, morning10:41
*** vinbs has quit IRC10:42
sambettso/ lucasagomes10:43
lucasagomes(-:10:46
* lucasagomes watching the dib gate!10:46
dtantsurhi sambetts, lucasagomes!10:49
lucasagomesdtantsur, hi there10:49
sambettsdtantsur o/ the alembic patch is working with the dvsm gate :D10:51
dtantsurawesome!10:52
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: [WIP] Expose serial, wwn and vendor on the BlockDevice object  https://review.openstack.org/22510210:53
*** karimb has quit IRC10:53
*** karimb has joined #openstack-ironic10:54
dtantsurlucasagomes, this ^^ touches root device hints, could you have a look at the direction where it's going?10:54
lucasagomesdtantsur, hi there, sure10:54
sambettsdtantsur: https://blueprints.launchpad.net/ironic-inspector/+spec/rules should we update the progress on this one, all patches in the white board have merged10:56
*** romcheg has quit IRC10:58
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/22461510:58
lucasagomes<openstackgerrit> Merged openstack/diskimage-builder: Deprecate the deploy-ironic element  https://review.openstack.org/20907911:09
lucasagomesw00t!11:09
dtantsur\o/11:09
*** e0ne has quit IRC11:10
dtantsursambetts, we didn't implement all the plugins. I'll post a patch today with a few remaining11:11
sambettsdtantsur: Ah ok :) I'm just trying to look at the extra_hardware eDeploy thing, trying to find example data11:12
dtantsuroh well11:12
dtantsursambetts, it looks like [ ['interface', 'eth1', 'ip', '1.2.3.4'], ['interface', 'eth1', 'mac', 'aa:bb:cc:dd:ee:ff'], ...]11:13
dtantsura huge list of lists11:13
*** amotoki has joined #openstack-ironic11:14
sambettsdtantsur: do you think something like { 'interfaces': { 'eth1' : {'ip':1.2.3.4, 'mac':'aa:bb:cc' } } is a good format to transpose it too ?11:16
*** thrash|g0ne is now known as thrash11:16
dtantsursambetts, exactly (under some "extra" key, I guess)11:17
*** sinval has quit IRC11:17
*** sinval has joined #openstack-ironic11:19
sambettsdtantsur: does it need an extra key? doesn't the fact its being put in swift indicate its extra data?11:24
dtantsursambetts, I mean, put it in 'extra' key in the whole data. As to swift, we should put there unconverted data, and deprecate it.11:25
dtantsursambetts, we have a generic API for getting data from swift now11:25
*** jaypipes has joined #openstack-ironic11:28
sambettsdtantsur: so no change in the ExtraHardwareHook?11:29
*** stendulker has joined #openstack-ironic11:32
*** e0ne has joined #openstack-ironic11:33
dtantsursambetts, except for converting?11:33
dtantsurI think no11:33
*** lucasagomes is now known as lucas-hungry11:35
sambettsdtantsur: so I'll convert introspection_data['data'], and insert it in to introspection_data['extra'] but still send introspection_data['data'] to swift not the converted data?11:36
dtantsursambetts, that was my plan11:36
*** baoli has joined #openstack-ironic11:36
dtantsuryes11:36
sambettsdtantsur: How will that work when we decide to rerun the rules? Will the data go through the hooks first?11:38
*** baoli_ has joined #openstack-ironic11:39
dtantsursambetts, I was just thinking about rerunning the rules :)11:39
dtantsurpreviously I wanted to rerun just rules11:39
dtantsurbut today I thought that we'd better store the original data as it was received from the ramdisk, and rerun the whole processing chain11:39
dtantsurbut that's for M11:39
dtantsur:)11:39
sambettsok :)11:40
*** baoli has quit IRC11:41
sambettsI'll convert the live data, and store the original data, the orignal key is 'data' should I create a new 'extra' key or just store the data then override the 'data' key?11:44
openstackgerritShivanand Tendulker proposed openstack/ironic: Config drive support for whole disk images for iscsi based deploy  https://review.openstack.org/22511511:44
*** bcornec has joined #openstack-ironic11:48
dtantsursambetts, I think just a new 'extra' key11:49
*** yog_ has quit IRC11:50
sambettsdtantsur: People would still be able to use introspection_data['data'] in their rule field, would that work?11:52
openstackgerritRamakrishnan G proposed openstack/ironic: Agent Inband RAID configuration available in cleaning  https://review.openstack.org/22493811:53
dtantsursambetts, no, it won't. well, probably we should drop 'data' key after storing and converting..11:53
sambettsdtantsur: Ok :)11:54
rameshg87dtantsur: lucas-hungry: ^^ can you please let me know your thoughts on the above patch11:54
*** stendulker has quit IRC11:55
* rameshg87 will be back after 2 hours11:56
*** rameshg87 has quit IRC11:56
*** romainh has joined #openstack-ironic11:56
*** trown|outttypeww is now known as trown12:01
*** puranamr has joined #openstack-ironic12:02
*** chlong has joined #openstack-ironic12:06
*** puranamr has quit IRC12:06
*** chlong has quit IRC12:07
*** chlong has joined #openstack-ironic12:07
TheJuliagood morning12:12
dtantsurmorning TheJulia, happy Friday12:13
TheJuliaIt is a good thing it is Friday!12:17
dtantsurdefinitely12:19
* sambetts agrees12:22
*** lsmola has quit IRC12:22
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/22461612:22
trowngood morning sambetts dtantsur TheJulia Happy Friday indeed12:30
dtantsurmorning trown12:31
trowndtantsur: tripleo CI was somehow using old discoverd package with inspector client.... So when I "fixed" packaging of inspector client to not allow them to be installed together...kaboom!12:31
dtantsuroh lol12:31
dtantsurwell, they *may* work together, but I don't guarantee it12:31
dtantsurin a sense that they both define one setuptools entry point12:32
trownya, I am confused how that was working...but there were lots of strange things going on wrt inspector/discoverd12:32
*** [1]cdearborn has joined #openstack-ironic12:32
trownlike both yum installing and pip installing inspector client12:33
derekhtrown: ok, so that was me that did that, I shoehorned it all together to make it work. Since we switched over the instack ci upstream I've been removing all the workarounds but hadn't gotten to that one yet12:35
trowndtantsur: tripleoclient was also using discoverd python client for the "baremetal bulk" stuff...12:35
*** lsmola has joined #openstack-ironic12:35
trownderekh: ya, I think moving all this stuff from rdo-manager to openstack was bound to be bumpy12:36
dtantsurtrown, yeah, I'm reviewing your change now :)12:37
*** lucas-hungry is now known as lucasagomes12:39
*** jaypipes is now known as leakypipes12:48
lucasagomestrown, heh happens12:59
openstackgerritMerged openstack/ironic-inspector: Smart root disk selection including support for root device hints  https://review.openstack.org/21827113:02
*** puranamr has joined #openstack-ironic13:03
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands  https://review.openstack.org/17251713:03
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode command  https://review.openstack.org/21672713:03
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Introduce openstackclient plugin  https://review.openstack.org/17167213:03
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create  https://review.openstack.org/17246113:03
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add provision state command  https://review.openstack.org/20611913:03
*** cdearborn has joined #openstack-ironic13:05
*** puranamr has quit IRC13:07
openstackgerritMerged openstack/ironic: Help patch authors to remember to update version docs  https://review.openstack.org/22490813:13
*** [1]cdearborn has quit IRC13:14
*** romcheg has joined #openstack-ironic13:15
trowndtantsur: for the inpector reqs patch, that now says that having >=1.2.0 is valid, but so is >=1.0.1?13:21
trownas in projects can opt in to >=1.2.013:22
dtantsurtrown, this patch only makes our gate use 1.2.0 instead of 1.0.1. it does not affect anything else13:22
dtantsurtrown, well, ideally, no. you can stop by #openstack-relmgr-office to chat with folks on how to treat it. But in theory you should be ready to use 1.0.1.13:22
trowndtantsur: I am trying to understand from tripleoclient perspective...13:23
trownie. is it allowed to have >=1.2.0 in the requirements.txt there13:23
dtantsurtrown, no. not in liberty13:23
trownok... :(13:23
*** harshs has joined #openstack-ironic13:25
*** dansmith is now known as superdan13:35
jrollmorning everyone :)13:37
jrollFYI I'm around for about the next 1.5 hours, then I'm off for the day13:38
lucasagomesjroll, TheJulia trown morning!13:40
jrollhey lucasagomes :)13:40
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Allow abort for CLEANWAIT states  https://review.openstack.org/20155213:40
* lucasagomes just fixed merge conflicts ^13:40
lucasagomesjroll, right if u want to take a look ^ it had a +2 from ruby and I just fixed few nits (added a new test and two typos on the docstring)13:40
jrollsure thing13:41
dtantsurmorning jroll!13:42
jrollhey dtantsur \o13:42
lucasagomesta much!13:43
*** zhenguo has joined #openstack-ironic13:43
*** sinval has quit IRC13:46
*** openstackgerrit has quit IRC13:46
*** openstackgerrit has joined #openstack-ironic13:47
*** sinval has joined #openstack-ironic13:49
*** vinbs has joined #openstack-ironic13:51
*** vinbs has quit IRC13:51
*** rameshg87 has joined #openstack-ironic13:52
*** sinval has quit IRC13:57
*** sinval has joined #openstack-ironic13:58
jlvillallucasagomes, I kept doing rechecks on diskimage-builder until I went to bed.  Looks like you finally got it merged :)14:03
ifarkasjroll, morning!14:03
*** [1]cdearborn has joined #openstack-ironic14:03
ifarkasjroll, regarding https://bugs.launchpad.net/ironic/+bug/145449214:03
openstackLaunchpad bug 1454492 in Ironic "openwsman library is not greenthread-aware, leads to driver lockup with both DRAC and AMT drivers" [Medium,Triaged] - Assigned to Imre Farkas (ifarkas)14:03
ifarkasjroll, the fix involves pretty much a rewrite of the drac driver, I don't think it's likely to land for 4.2.014:04
jrollifarkas: hey, morning14:04
jrollifarkas: I agree, I left it there with high hopes :)14:04
jrollI can untarget it though14:04
lucasagomesjlvillal, heh yeah! Thanks much14:04
ifarkasjroll, makes sense, I am actually working on it right now14:05
jrollifarkas: cool, I'd love to make this a library so AMT can also use it14:05
jrollor common, at least14:05
* jroll removes milestone14:06
ifarkasjroll, +114:06
*** gabriel-1 is now known as gabriel-bezerra14:06
ifarkasjroll, do you think it makes sense for the library to also live under openstack/ironic?14:06
jrollifarkas: I'm fine with it either way, whatever you think is best14:06
jrollifarkas: sdague's amt library might be a good start on this, too14:07
*** liliars has joined #openstack-ironic14:08
ifarkasjroll, yeah, I started working on something similar: https://github.com/ifarkas/wsmanclient14:08
*** gabriel-bezerra is now known as gabrielbezerra14:09
ifarkasjroll, it uses lxml+requests, so doesn't need to contain everything as a template14:09
jrollifarkas: oh, neat :)14:09
openstackgerritSam Betts proposed openstack/ironic-inspector: Convert eDeploy data so that rules can process it  https://review.openstack.org/22516814:10
*** smoriya has quit IRC14:11
*** karimb has quit IRC14:11
*** mtanino has joined #openstack-ironic14:16
*** cdearborn has quit IRC14:18
openstackgerritRamakrishnan G proposed openstack/ironic: Agent Inband RAID configuration available in cleaning  https://review.openstack.org/22493814:19
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add missing plugins for introspection rules  https://review.openstack.org/22517314:20
*** dims_ has quit IRC14:20
*** dims_ has joined #openstack-ironic14:21
jrolllucasagomes: +2 on your abort patch, mind a followup patch (or another rev) that adds 1.13 to api docs?14:24
jrolllucasagomes: also that's marked partial-bug, are there other patches up for the rest? or what needs to be done there?14:24
lucasagomesjroll, it already does https://review.openstack.org/#/c/201552/20/doc/source/webapi/v1.rst14:25
lucasagomesjroll, oh, yeah it should close that bug :-(14:25
jrolllucasagomes: oops, it took me too long to review the rest and I forgot that was there :P14:25
lucasagomeswe can manually mark it as fix commited once it merge14:25
jrollif you want to update commit msg I can quickly +2 again14:25
lucasagomesjroll, that works too14:25
lucasagomes1 sec14:25
jrollor actually I'll do it14:25
jrollor not14:25
jrolllol14:25
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Allow abort for CLEANWAIT states  https://review.openstack.org/20155214:26
lucasagomesboom14:26
lucasagomesdone14:26
jrollthanks14:27
*** sdake has joined #openstack-ironic14:27
*** jroll is now known as jroll|dupe14:31
*** jroll|dupe is now known as brickednick14:31
*** david-lyle has quit IRC14:31
*** brickednick is now known as jroll14:31
jrollsuperdan: would you mind reviewing this at some point? https://review.openstack.org/#/c/224079/614:34
*** w_ has joined #openstack-ironic14:34
openstackgerritBob Ball proposed openstack/ironic: Handle unquoted node names from virt types  https://review.openstack.org/22477014:36
superdanjroll: added to my tab queue14:37
*** olaph has quit IRC14:37
jrollthanks14:37
jlvillallucasagomes, I thought the regex was much simpler than the suggested change. I commented to that effect14:39
lucasagomesjlvillal, yeah, I wonder that too... Usually people don't like regex14:41
lucasagomesbut that one is simple enough, I will keep it]14:41
lucasagomeskeep it*14:41
jlvillallucasagomes, I like regex :)14:42
*** shoutm has joined #openstack-ironic14:42
lucasagomesheh14:42
openstackgerritShinn'ya Hoshino proposed openstack/ironic: Fix a bug error by passwords includes just numbers  https://review.openstack.org/22038514:43
*** w_ has quit IRC14:45
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Proxy configuration for agent images downloading  https://review.openstack.org/22518114:45
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Fix rolling upgrades by implementing indirection_api  https://review.openstack.org/22407914:46
*** zhenguo has quit IRC14:48
sambettslucasagomes, jroll: Is there any way I can squeeze this into the 4.2.0 release?14:50
sambettshttps://review.openstack.org/#/c/219253/14:50
jrollsambetts: D:14:51
sambettsjroll: I thought that might be your answer :-P14:51
jrollthat's uh, pretty large14:51
dtantsur1305 LOC, wow14:51
jrollI suspect not, 4.2 is 6 days away14:51
jrollomg it has docs <314:52
openstackgerritShinn'ya Hoshino proposed openstack/ironic: Test for a bug error by passwords includes just numbers  https://review.openstack.org/22038514:52
dtantsurjroll, on a second sight it doesn't look too big and complex. I'll have a look at it after a short break14:52
dtantsursambetts, ^^14:53
jrolldtantsur: sambetts yeah, I'm not strictly opposed to it, it isn't risky14:53
jrolljust a matter of reviewer time14:53
sambettsYeah, I got a whole bunch of reviews then I think it slipped through the net as we tried to get 4.1.0 cut14:54
*** harshs has quit IRC14:54
jrollalright, I'm basically out for the rest of the day14:55
jrollhave a good weekend folks :)14:55
sambettsjroll: Cya! Have a good weekend o/14:55
*** harshs has joined #openstack-ironic14:55
sambettsThanks dtantsur :D14:56
openstackgerritJulia Kreger proposed openstack/bifrost: WIP: Include token value for no-auth mode  https://review.openstack.org/22518714:57
lucasagomessambetts, I will review it!14:57
sambetts:D lucasagomes thanks!14:58
openstackgerritSam Betts proposed openstack/ironic-inspector: Add alembic migrations for the inspector database  https://review.openstack.org/22366514:59
lucasagomesjroll, enjoy the rest of the day14:59
*** jistr is now known as jistr|call15:02
*** dims_ is now known as dimsum__15:03
*** harshs has quit IRC15:03
openstackgerritShinn'ya Hoshino proposed openstack/ironic: Fix a bug error by passwords includes just numbers  https://review.openstack.org/22038515:14
*** olaph has joined #openstack-ironic15:22
*** tsekiyama has joined #openstack-ironic15:26
*** Nisha has joined #openstack-ironic15:30
*** blakec has joined #openstack-ironic15:34
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Fix rolling upgrades by implementing indirection_api  https://review.openstack.org/22407915:35
*** r-daneel has joined #openstack-ironic15:39
openstackgerritMerged openstack/ironic: Document GET ...raid/logical_disk_properties  https://review.openstack.org/22435615:40
*** romainh has quit IRC15:40
*** lazy_prince has quit IRC15:42
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add missing plugins for introspection rules  https://review.openstack.org/22517315:43
dtantsursambetts, this ^^ I believe is the last part of 'rules' blueprint15:44
sambetts:) awesome15:44
*** harshs has joined #openstack-ironic15:44
*** sinval has quit IRC15:46
*** blakec has quit IRC15:46
*** mgoddard1 has joined #openstack-ironic15:47
*** sinval has joined #openstack-ironic15:48
*** Marga_ has quit IRC15:48
*** mgoddard has quit IRC15:49
*** blakec has joined #openstack-ironic15:49
Nishadtantsur, other core reviewers....could you please review https://review.openstack.org/#/c/217068/. This is the last patch of the merged spec https://review.openstack.org/19347815:51
*** krtaylor has quit IRC15:56
*** mgoddard1 has quit IRC15:56
dtantsurNisha, I'll take a look on Monday, it's a bit late today for me15:56
dtantsurand by the way, I'm going now. Have a nice weekend everyone!15:57
*** mgoddard has joined #openstack-ironic15:57
trownhave a great weekend dtantsur15:58
dtantsurtrown, please have a look at https://review.openstack.org/225168 maybe you know more about edeploy stuff...16:01
*** dtantsur is now known as dtantsur|afk16:01
trowndtantsur|afk: will do. sambetts thanks for that ^16:02
*** trown is now known as trown|lunch16:04
*** rameshg87 has left #openstack-ironic16:04
*** e0ne has quit IRC16:05
*** puranamr has joined #openstack-ironic16:05
*** blakec has quit IRC16:09
*** Nisha_away has joined #openstack-ironic16:10
*** Sukhdev has joined #openstack-ironic16:11
*** Nisha has quit IRC16:13
*** krtaylor has joined #openstack-ironic16:14
*** garthb has joined #openstack-ironic16:15
*** w_ has joined #openstack-ironic16:16
*** romcheg has quit IRC16:16
*** olaph has quit IRC16:16
*** Marga_ has joined #openstack-ironic16:17
*** puranamr has quit IRC16:18
openstackgerritBob Ball proposed openstack/ironic: Handle unquoted node names from virt types  https://review.openstack.org/22477016:23
jlvillallucasagomes: Do you know how to abort a deploy in progress?  Besides just waiting for it to time out.16:26
lucasagomesjlvillal, if it's DEPLOYING I'm afraid you can not right now :-(16:26
jlvillallucasagomes: :(   Okay thanks.16:26
lucasagomesjlvillal, I have to get back to the spec implementing it16:27
lucasagomesjlvillal, you can abort a node in DEPLOYWAIT tho16:27
jlvillalOh, maybe I am in DEPLOYWAIT16:27
lucasagomesjlvillal, ironic node-set-provision-state UUID deleted16:27
*** Sukhdev has quit IRC16:27
jlvillallucasagomes: Thanks. I will try that in my testing.16:27
*** derekh has quit IRC16:28
lucasagomesjlvillal, one thing is, if a node is in DEPLOYING and the conductor dies, once it starts again it will clean that up16:28
lucasagomesand mark the nodes as failed because it can't resume16:28
lucasagomesso... if you restart it while deploying you will "abort" it16:28
*** jistr|call is now known as jistr16:28
lucasagomesnote: ^ unless it's a testing env, or a very constraint env I don't suggest anyone doing that16:29
*** Marga_ has quit IRC16:29
*** Sukhdev_ has joined #openstack-ironic16:29
*** jistr has quit IRC16:30
*** Sukhdev_ has quit IRC16:31
*** lsmola has quit IRC16:32
jlvillallucasagomes: For me it is a testing environment. Thanks.16:33
lucasagomescool16:33
lucasagomesso that will work for u16:33
jlvillallucasagomes: Regarding the deprecated ramdisk. Does that mean IPA supports AMT?16:34
*** Nisha_away has quit IRC16:34
* jlvillal does not know yet know a lot how the various ramdisks work...16:35
lucasagomesjlvillal, amt is just the driver to power control the nodes right?16:35
jlvillallucasagomes: Yep.16:36
*** Marga_ has joined #openstack-ironic16:36
lucasagomesthe ramdisk don't have to deal or care about who power on/off the machines16:36
jlvillalThanks16:36
lucasagomesit will be loaded once the machine power on, in the amt I suppose it will always be network boot? (I don't think it suppports something like virtual media)16:36
jlvillallucasagomes: Yeah, I am currently setting it up and using the old ramdisk.16:37
jlvillalJust wondering if I should switch16:37
*** Nisha_away has joined #openstack-ironic16:37
lucasagomesjlvillal, it will be around for some time still16:38
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/22461416:38
lucasagomesyou can switch when convenient I would say16:38
jlvillallucasagomes: Thanks for your help.  And enjoy your weekend!16:38
lucasagomesjlvillal, and should be easy enough. It should be a drop-in replacement really, no extra config16:38
jlvillallucasagomes: I'll give it a shot. Hopefully today :)16:39
lucasagomesjlvillal, thanks! yeah I will have my freedom soon!16:39
*** hemna has joined #openstack-ironic16:40
jlvillal:)16:40
* jlvillal wonders if he is the only one annoyed by unit tests that test functions by calling a function which calls the function that we are trying to test...16:41
*** Sukhdev_ has joined #openstack-ironic16:43
*** olaph has joined #openstack-ironic16:43
lucasagomesjlvillal, lol ur not alone on that16:43
jlvillallucasagomes: I'm happy to hear that :)16:43
*** w_ has quit IRC16:44
lucasagomesand I'm going to call it a day folks! Have a great night everyone16:44
lucasagomesenjoy the weekend16:44
jlvillallucasagomes: Ciao!16:44
lucasagomessee y'all16:44
*** athomas has quit IRC16:45
jlvillalBobBall: Hope you don't mind my random comments in your patch :)16:46
jlvillalBobBall: Thanks for the patch!16:46
*** MattMan has left #openstack-ironic16:48
*** Sukhdev_ has quit IRC16:51
*** Sukhdev has joined #openstack-ironic16:52
* jlvillal wonders if it is documented that ironic conductor does a sudo and thus need to setup NOPASSWD for sudo. It probably is documented and he missed it.16:56
*** w_ has joined #openstack-ironic16:57
*** harshs_ has joined #openstack-ironic16:58
*** Sukhdev has quit IRC16:58
*** ifarkas has quit IRC16:59
*** olaph has quit IRC17:00
*** achanda has joined #openstack-ironic17:01
*** harshs has quit IRC17:01
*** harshs_ is now known as harshs17:01
*** Sukhdev_ has joined #openstack-ironic17:02
openstackgerritNaohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaAgentDeploy by applying new BootInterface  https://review.openstack.org/22157717:02
*** Sukhdev_ has quit IRC17:03
*** trown|lunch is now known as trown17:04
*** Sukhdev has joined #openstack-ironic17:05
*** Sukhdev has quit IRC17:06
openstackgerritMerged openstack/ironic: Check image size before provisioning for agent driver  https://review.openstack.org/21525417:07
*** Sukhdev_ has joined #openstack-ironic17:08
*** absubram has joined #openstack-ironic17:09
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/22461417:09
openstackgerritMerged openstack/ironic: Fix incorrect urls  https://review.openstack.org/22507517:16
*** Sukhdev_ has quit IRC17:17
*** Sukhdev has joined #openstack-ironic17:18
*** w_ has quit IRC17:19
*** Sukhdev has quit IRC17:20
*** Sukhdev has joined #openstack-ironic17:20
*** w_ has joined #openstack-ironic17:20
*** romcheg has joined #openstack-ironic17:22
*** amotoki has quit IRC17:26
*** e0ne has joined #openstack-ironic17:26
*** Sukhdev has quit IRC17:26
*** Sukhdev has joined #openstack-ironic17:28
*** achanda has quit IRC17:30
*** Sukhdev has quit IRC17:33
*** hemna has quit IRC17:36
*** Nisha_brb has joined #openstack-ironic17:39
*** e0ne has quit IRC17:41
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279617:41
*** Nisha_away has quit IRC17:41
*** pelix has quit IRC17:43
*** romcheg has quit IRC17:43
*** romcheg has joined #openstack-ironic17:44
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279617:44
*** Sukhdev has joined #openstack-ironic17:45
*** e0ne has joined #openstack-ironic17:49
*** boris-42 has quit IRC17:50
*** Sukhdev has quit IRC17:51
*** romcheg1 has joined #openstack-ironic17:52
*** achanda has joined #openstack-ironic17:52
*** w_ has quit IRC17:52
*** romcheg has quit IRC17:53
*** w_ has joined #openstack-ironic17:53
openstackgerritSam Betts proposed openstack/ironic: Add Cisco IMC PXE Driver  https://review.openstack.org/21925317:56
*** Nisha_brb has quit IRC17:57
*** harlowja has quit IRC17:58
*** harlowja has joined #openstack-ironic17:58
*** Sukhdev has joined #openstack-ironic17:59
sambettslucasagomes: ^^ thanks for your comments18:01
sambettsNight all o/ See everyone Monday18:02
BobBalljlvillal: Always love random comments TBH ;)18:03
jlvillal:)18:03
jlvillalBobBall: I replied, my mistake. Not seeing the 'skipped' part of the diff.18:06
jlvillalOnce I showed the lines above the change I could actually see the function name. Duh on my part.18:06
*** penick_ has joined #openstack-ironic18:08
*** praneshp has joined #openstack-ironic18:09
BobBallnp :)18:10
*** penick has quit IRC18:10
*** penick_ is now known as penick18:10
openstackgerritBob Ball proposed openstack/ironic: Handle unquoted node names from virt types  https://review.openstack.org/22477018:14
*** BobBall is now known as BobBall_AWOL18:14
*** ijw has joined #openstack-ironic18:17
*** BobBall_AWOL is now known as BobBall18:19
*** w_ is now known as olaph18:21
*** ijw has quit IRC18:22
betherlyOff for the weekend! Have a good one everyone18:24
*** Sukhdev has quit IRC18:27
*** w_ has joined #openstack-ironic18:28
*** Sukhdev has joined #openstack-ironic18:29
*** olaph has quit IRC18:31
*** BobBall is now known as BobBall_AWOL18:32
*** olaph has joined #openstack-ironic18:33
*** w_ has quit IRC18:34
*** Sukhdev has quit IRC18:35
*** e0ne has quit IRC18:39
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279618:41
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279618:46
*** romcheg1 has quit IRC18:52
*** romcheg has joined #openstack-ironic18:55
*** harshs has quit IRC18:56
*** achanda has quit IRC18:57
*** e0ne has joined #openstack-ironic18:58
*** r-daneel has quit IRC18:59
jlvillalNobodyCam: Did you ever setup a system using AMT with Ironic?18:59
jlvillalOr anyone else :)18:59
*** w_ has joined #openstack-ironic19:06
*** Sukhdev_ has joined #openstack-ironic19:06
*** olaph has quit IRC19:09
*** ijw has joined #openstack-ironic19:11
*** w_ has quit IRC19:15
*** Sukhdev_ has quit IRC19:16
*** ijw has quit IRC19:16
*** e0ne has quit IRC19:17
*** olaph has joined #openstack-ironic19:25
*** cdearborn has joined #openstack-ironic19:38
*** [1]cdearborn has quit IRC19:39
*** absubram has quit IRC19:40
openstackgerritMerged openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279619:43
*** sdake has quit IRC19:46
*** sdake has joined #openstack-ironic19:53
*** cdearborn has quit IRC19:55
*** ijw has joined #openstack-ironic19:57
*** Haomeng has joined #openstack-ironic19:58
*** Haomeng|2 has quit IRC20:00
*** derekh has joined #openstack-ironic20:04
*** liliars has quit IRC20:05
*** cdearborn has joined #openstack-ironic20:05
*** derekh has quit IRC20:06
*** bnemec has quit IRC20:15
*** bnemec has joined #openstack-ironic20:15
gabrielbezerraHi folks!20:23
gabrielbezerraIf I put a config option of a driver as required, is it going to prevent ironic from working in case the user does not enable such driver?20:24
anteayajroll: did you make it to the diversity meeting yesterday?20:24
*** gabrielbezerra is now known as gbezerra20:25
*** gbezerra is now known as gabriel-bezerra20:25
anteayajroll: skimming the logs, I see you made it noone talked about geo diversity though so it got skipped :(20:27
gabriel-bezerraI'm referring specifically to my comment on line 36 of this review https://review.openstack.org/#/c/191822/29/ironic/drivers/modules/oneview/common.py20:27
anteayajroll: thanks for trying, I appreciate it20:27
gabriel-bezerrajlvillal: IIRC, devananda did it on his presentation last summit, in Vancouver.20:29
gabriel-bezerrajlvillal: he used it with bifrost. Maybe TheJulia can help you.20:30
*** intr1nsic has joined #openstack-ironic20:31
*** romcheg has quit IRC20:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/22461420:42
*** krtaylor has quit IRC20:52
*** romcheg has joined #openstack-ironic20:58
*** w_ has joined #openstack-ironic20:59
*** blakec has joined #openstack-ironic21:01
*** olaph has quit IRC21:02
*** olaph has joined #openstack-ironic21:03
*** w_ has quit IRC21:06
*** lucasagomes has quit IRC21:11
*** chlong has quit IRC21:14
*** harshs has joined #openstack-ironic21:15
jlvillalgabriel-bezerra: Thanks. Yeah I saw that. Was hoping to ask some questions from someone who had done it :)21:22
*** dims_ has joined #openstack-ironic21:29
*** dimsum__ has quit IRC21:32
*** krtaylor has joined #openstack-ironic21:35
*** trown is now known as trown|outttypeww21:36
gabriel-bezerrajlvillal: np21:41
*** blakec has quit IRC21:49
*** blakec has joined #openstack-ironic21:51
openstackgerritMario Villaplana proposed openstack/ironic-python-agent: Adds more functional tests for commands  https://review.openstack.org/22424721:54
*** romcheg has quit IRC22:08
openstackgerritMario Villaplana proposed openstack/ironic: Fix typo in vendor method dev documentation  https://review.openstack.org/22535122:10
*** romcheg has joined #openstack-ironic22:21
*** w_ has joined #openstack-ironic22:27
*** olaph has quit IRC22:29
*** garthb has quit IRC22:33
jlvillalJoshNang: Any chance you could look at: https://review.openstack.org/215854 ?22:36
*** w_ is now known as olaph22:37
JoshNangjlvillal: sure22:37
jlvillalJoshNang: Thanks :)22:37
*** sdake_ has joined #openstack-ironic22:40
*** sdake has quit IRC22:43
*** openstackgerrit has quit IRC22:46
*** openstackgerrit has joined #openstack-ironic22:47
*** sdake has joined #openstack-ironic22:49
*** sdake_ has quit IRC22:51
*** achanda has joined #openstack-ironic22:52
*** dims_ has quit IRC22:56
*** cdearborn has quit IRC23:12
*** baoli_ has quit IRC23:16
*** leakypipes has quit IRC23:22
*** dimsum__ has joined #openstack-ironic23:23
openstackgerritMerged openstack/ironic-python-agent: Create a SerializableComparable class  https://review.openstack.org/21585423:47
*** praneshp has quit IRC23:54
*** davideag_ has quit IRC23:56

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