Wednesday, 2016-06-01

NobodyCammorning mrda00:01
mrdaHey NobodyCam00:02
NobodyCamHow the weather down under00:02
mrdaGettgin cold, for Adelaide.  It's 64F outside.  Brrr!00:03
NobodyCamlol only 101F here today 113 by Friday :)00:13
JayFoh god that sounds awful00:13
*** dims has quit IRC00:14
*** sacharya has quit IRC00:15
*** hoangcx has joined #openstack-ironic00:17
*** dims has joined #openstack-ironic00:19
*** praneshp has quit IRC00:20
NobodyCamhehehe00:21
NobodyCamin the house it's a nice 73F00:22
*** garthb_ has quit IRC00:23
*** sdake_ has joined #openstack-ironic00:29
*** sdake has quit IRC00:30
mrda:)00:31
mrdaAlso, come y'all move your timezones closer to me?  It'd really be helpful.00:31
*** rama_y has quit IRC00:32
*** mtanino has quit IRC00:41
*** piet has joined #openstack-ironic00:49
*** Sukhdev has quit IRC00:55
*** sdake has joined #openstack-ironic00:56
*** ijw has quit IRC00:56
*** ijw has joined #openstack-ironic00:57
*** sdake_ has quit IRC00:59
*** piet has quit IRC01:00
*** sdake has quit IRC01:02
*** piet has joined #openstack-ironic01:03
*** spandhe has quit IRC01:06
*** ChrisAusten has joined #openstack-ironic01:09
*** sacharya has joined #openstack-ironic01:15
*** piet has quit IRC01:17
*** YiboCai has joined #openstack-ironic01:20
*** DCWilliams_VA has joined #openstack-ironic01:20
*** DCWilliams_VA is now known as Guest9945301:20
*** sacharya has quit IRC01:21
*** Guest99453 is now known as dcwilliams29id01:22
*** ijw has quit IRC01:23
*** dcwilliams29id is now known as dcwilliamsva01:23
*** dcwilliamsva has quit IRC01:29
*** sdake has joined #openstack-ironic01:41
*** sabeen has joined #openstack-ironic01:41
*** sdake has quit IRC01:43
*** sdake has joined #openstack-ironic01:47
*** sdake_ has joined #openstack-ironic01:48
*** sdake has quit IRC01:51
*** chenke has joined #openstack-ironic02:16
*** sacharya has joined #openstack-ironic02:16
*** DCWilliams_VA has joined #openstack-ironic02:21
*** DCWilliams_VA is now known as Guest3530702:21
*** rama_y has joined #openstack-ironic02:25
*** yuikotakadamori has quit IRC02:33
openstackgerritYibo Cai proposed openstack/ironic: Support multi arch deployment  https://review.openstack.org/32263202:34
*** yuanying has quit IRC02:50
*** rloo has quit IRC03:00
*** Guest35307 has quit IRC03:04
*** ayoung has quit IRC03:09
*** DCWilliams_VA has joined #openstack-ironic03:12
*** DCWilliams_VA is now known as Guest5755203:13
*** sabeen has quit IRC03:15
openstackgerritMerged openstack/ironic: Add one use case for configdrive  https://review.openstack.org/32309903:17
*** piet has joined #openstack-ironic03:22
*** Guest57552 has quit IRC03:23
zhenguo_devananda: I restored the patch to gate on ironic.config.sample generation, as we have already switched to oslo-config-generator, could you please have a look at this https://review.openstack.org/#/c/238837/ again?03:25
*** piet has quit IRC03:36
openstackgerritZhenguo Niu proposed openstack/ironic: Clear target_power_state on conductor startup  https://review.openstack.org/31009003:36
openstackgerritShivanand Tendulker proposed openstack/ironic-specs: Baremetal provisioning in UEFI secure boot mode  https://review.openstack.org/23027403:38
*** appu103_ has joined #openstack-ironic03:46
*** yuanying has joined #openstack-ironic03:47
*** links has joined #openstack-ironic04:00
*** vmud213 has joined #openstack-ironic04:02
*** ChrisAusten has quit IRC04:10
*** appu103_ is now known as aparnav04:13
*** david-lyle has quit IRC04:21
*** Sukhdev has joined #openstack-ironic04:26
*** rama_y has quit IRC04:27
*** piet has joined #openstack-ironic04:29
*** flwang1 has quit IRC04:30
*** flwang1 has joined #openstack-ironic04:36
*** spandhe has joined #openstack-ironic04:39
*** spandhe_ has joined #openstack-ironic04:40
*** ishant has joined #openstack-ironic04:42
*** spandhe has quit IRC04:44
*** spandhe_ is now known as spandhe04:44
*** piet has quit IRC04:45
*** milan has quit IRC04:55
openstackgerritYuiko Takada Mori proposed openstack/ironic-specs: Nova compatible serial console support  https://review.openstack.org/31950504:55
*** ishant has quit IRC04:56
*** ishant has joined #openstack-ironic04:58
*** rcernin has quit IRC05:04
*** ishant is now known as ishant|away05:10
*** ishant|away is now known as ishant05:17
*** ChubYann has quit IRC05:21
*** praneshp has joined #openstack-ironic05:22
*** phuongnh has joined #openstack-ironic05:23
*** moshele has joined #openstack-ironic05:24
*** _vs has joined #openstack-ironic05:24
*** _vsaienko has joined #openstack-ironic05:24
*** mtanino has joined #openstack-ironic05:25
*** Sukhdev has quit IRC05:26
*** _vsaienko has quit IRC05:26
*** _vsaienko has joined #openstack-ironic05:26
*** Sukhdev has joined #openstack-ironic05:30
*** mtanino has quit IRC05:31
*** Sukhdev_ has joined #openstack-ironic05:33
*** spandhe has quit IRC05:34
*** Sukhdev has quit IRC05:34
*** spandhe has joined #openstack-ironic05:37
*** spandhe has quit IRC05:39
*** rcernin has joined #openstack-ironic05:43
*** _vsaienko has quit IRC05:46
*** _vs has quit IRC05:46
*** ohamada has joined #openstack-ironic05:47
*** _vsaienko has joined #openstack-ironic05:48
*** _vs has joined #openstack-ironic05:48
*** sdake has joined #openstack-ironic05:52
*** sdake_ has quit IRC05:55
*** spandhe has joined #openstack-ironic05:57
*** milan has joined #openstack-ironic06:04
milanmorning Ironic06:04
*** rbartal has joined #openstack-ironic06:05
milan#pixiesay milan woke up soon today -m happy06:05
PixieBootsʕ^ᴥ^ʔ: milan woke up soon today06:05
milanmorning rbartal o/06:05
rbartalmilan, good morning :)06:06
milan:)06:06
*** yolanda_ has quit IRC06:09
*** yolanda has joined #openstack-ironic06:09
*** Sukhdev_ has quit IRC06:14
*** Sukhdev has joined #openstack-ironic06:15
*** sdake has quit IRC06:15
*** _vsaienko has quit IRC06:17
*** _vs has quit IRC06:17
*** mjura has joined #openstack-ironic06:18
*** vishwanathj has joined #openstack-ironic06:19
*** Sukhdev has quit IRC06:19
*** _vsaienko has joined #openstack-ironic06:20
*** _vs has joined #openstack-ironic06:20
*** yolanda_ has joined #openstack-ironic06:22
*** yolanda_ has quit IRC06:22
*** yolanda_ has joined #openstack-ironic06:22
*** vishwanathj has quit IRC06:23
*** vishwanathj has joined #openstack-ironic06:24
*** vishwana_ has joined #openstack-ironic06:25
*** yolanda has quit IRC06:25
*** yolanda_ has quit IRC06:26
*** yolanda has joined #openstack-ironic06:26
*** mjura has quit IRC06:27
chenkehi, jroll06:27
*** praneshp has quit IRC06:28
*** vishwanathj has quit IRC06:29
openstackgerritTan Lin proposed openstack/ironic: Improve the readability of configuration drive doc part  https://review.openstack.org/32367506:29
*** vishwana_ is now known as vishwanathj06:30
chenkehi, jroll!  i see u implement the features--https://blueprints.launchpad.net/nova/+spec/ironic-networks-support, i have some questions about that, could u call me here while u are available?06:32
*** _vsaienko has quit IRC06:34
*** _vs has quit IRC06:34
moshelejroll, and other ironic cores/reviewer - can you review again the adding support for infiniband spec https://review.openstack.org/#/c/265635/06:34
*** spandhe has quit IRC06:38
*** _vsaienko has joined #openstack-ironic06:39
*** _vs has joined #openstack-ironic06:39
*** mjura has joined #openstack-ironic06:39
*** vishwanathj has quit IRC06:45
*** _vsaienko has quit IRC06:47
*** _vs has quit IRC06:47
*** vishwanathj has joined #openstack-ironic06:47
openstackgerritTan Lin proposed openstack/ironic: A follow-up patch of 19ef56cf00a78871e0d30a160b07f280488ddd0c  https://review.openstack.org/32368006:47
*** divya_ has joined #openstack-ironic06:51
*** _vsaienko has joined #openstack-ironic06:52
*** _vs has joined #openstack-ironic06:52
divya_GM Vsaienko06:53
openstackgerritTan Lin proposed openstack/ironic: Add a doc about appending kernel parameters to boot instances  https://review.openstack.org/32084806:55
*** _vsaienko has quit IRC06:58
*** _vs has quit IRC06:58
*** _vsaienko has joined #openstack-ironic06:58
*** _vs has joined #openstack-ironic06:58
openstackgerritTan Lin proposed openstack/ironic: Improve the readability of configuration drive doc part  https://review.openstack.org/32367507:01
*** _vsaienko has quit IRC07:02
*** vishwana_ has joined #openstack-ironic07:02
*** _vs has quit IRC07:02
*** vishwanathj has quit IRC07:03
*** tesseract has joined #openstack-ironic07:03
*** _vsaienko has joined #openstack-ironic07:04
*** _vs has joined #openstack-ironic07:04
*** slagle has quit IRC07:06
*** _vs has quit IRC07:08
*** _vsaienko has quit IRC07:08
*** _vs has joined #openstack-ironic07:08
*** _vsaienko has joined #openstack-ironic07:08
*** ifarkas has joined #openstack-ironic07:11
*** _vs has quit IRC07:14
*** _vsaienko has quit IRC07:14
*** _vsaienko has joined #openstack-ironic07:17
*** _vs has joined #openstack-ironic07:17
ishantsambetts|afk, cool . Thanks a lot :)07:25
*** DCWilliams_VA has joined #openstack-ironic07:25
*** amotoki has joined #openstack-ironic07:26
*** DCWilliams_VA is now known as Guest7510107:26
*** vishwana_ has quit IRC07:26
*** pcaruana has joined #openstack-ironic07:26
*** _vsaienko has quit IRC07:28
*** _vs has quit IRC07:28
*** _vsaienko has joined #openstack-ironic07:29
*** _vs has joined #openstack-ironic07:29
*** Guest75101 has quit IRC07:30
*** e0ne has joined #openstack-ironic07:32
*** _vs has quit IRC07:33
*** _vsaienko has quit IRC07:33
betherlymorning all!07:35
*** dtantsur|afk is now known as dtantsur07:35
dtantsurMorning Ironic07:35
*** _vsaienko has joined #openstack-ironic07:36
*** _vs has joined #openstack-ironic07:36
ifarkasmorning all!07:38
*** _vsaienko has quit IRC07:38
*** _vs has quit IRC07:38
ifarkasmilan, regarding the tempest patch, do you want to address the nits in the current patch or do a follow-up?07:39
milanifarkas,  hi07:40
dtantsurfollow-up please07:40
milandtantsur, betherly morning07:40
dtantsurthis patch is around for waaaay too long07:40
dtantsurmorning ifarkas, milan07:40
ifarkasdtantsur, +107:40
ifarkasapproving it then07:40
dtantsur(if nits are really nits, that is)07:40
milanyeah I'd say there's no show stopper there07:40
milanaarefiev, wdyt?07:41
dtantsurmilan, now that it's approved, please move the tempest job to the check queue07:41
*** _vsaienko has joined #openstack-ironic07:41
*** _vs has joined #openstack-ironic07:41
dtantsurdid you work with project config ever?07:42
*** daemontool_ has joined #openstack-ironic07:42
milandtantsur, nope07:42
dtantsurmilan, essentially move this line https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L7019 to after line 701307:43
* milan looks07:43
dtantsuralso thanks for working on this, aarefiev, milan!07:44
milandtantsur, np! :)07:45
milandtantsur, what's the difference between the check: and gate: in that file?07:46
dtantsurmilan, check is run on every new patchset, gate is run after the patch is approved07:46
dtantsurmilan, also check runs on every patch independently, gate stacks them in the order they would merge07:47
milanshould we have the test in both then probably?07:47
dtantsurmilan, no, it's still non-voting. we don't have non-voting tests in the gate pipeline07:47
milanah OK07:47
dtantsurbecause you can't act on a non-voting failure in the gate pipeline - the change will be already merged when you see it07:48
dtantsursee http://docs.openstack.org/infra/system-config/zuul.html#overview for more details on pipelines etc07:48
*** _vsaienko has quit IRC07:48
*** _vsaienko has joined #openstack-ironic07:49
milandtantsur, thanks07:49
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:01
*** xek has joined #openstack-ironic08:01
*** _vsaienko has quit IRC08:03
*** fragatin_ has joined #openstack-ironic08:03
*** _vs has quit IRC08:03
*** _vs has joined #openstack-ironic08:03
*** _vsaienko has joined #openstack-ironic08:03
*** fragatina has quit IRC08:07
vsaienkomorning milan, dtantsur, ifarkas betherly and all!08:07
milandtantsur, https://review.openstack.org/32370808:07
milanmorning vsaienko :)08:07
*** fragatin_ has quit IRC08:07
*** fragatina has joined #openstack-ironic08:08
dtantsurhey vsaienko08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
*** moshele has quit IRC08:11
betherlyo/ dtantsur ifarkas milan vsaienko :)08:16
milanbetherly, hi :)08:16
ifarkasmorning vsaienko, betherly and all  :-)08:17
openstackgerritMerged openstack/ironic-inspector: Tempest: add basic test  https://review.openstack.org/29392608:18
openstackgerritMerged openstack/ironic-inspector: Fix py3 issue in functional tests  https://review.openstack.org/32152108:18
lucasagomesmorning all08:18
betherlyo/ lucasagomes :)08:18
milanhey lucasagomes :)08:18
dtantsurmorning betherly, lucasagomes08:18
dtantsurmilan, aarefiev, so our next target is tempest for discovery, right?08:19
milandtantsur, yeah that and/or the grenade for inspector08:19
dtantsuryeah, probably both in paralle08:20
milandtantsur, +108:20
*** mkoderer has quit IRC08:22
*** kromanenko_ has joined #openstack-ironic08:22
*** kromanenko__ has joined #openstack-ironic08:22
*** Romanenko_K has joined #openstack-ironic08:22
aarefievdtantsur: yeah, I have working wip08:25
dtantsurawesome08:25
aarefievbut it's raw08:25
*** mkoderer has joined #openstack-ironic08:26
aarefievdtantsur: did I miss something https://review.openstack.org/#/c/323726/08:29
dtantsurouch, I think it's the 2nd one :)08:30
dtantsurhttps://review.openstack.org/32370808:31
*** MattMan has quit IRC08:31
*** MattMan has joined #openstack-ironic08:31
dtantsuranyway, I need to move to the office, so please figure out which one to leave :)08:31
*** dtantsur is now known as dtantsur|bbl08:31
aarefievheh, ok, didn't see it, it's identical08:31
*** electrofelix has joined #openstack-ironic08:36
openstackgerritVasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy  https://review.openstack.org/26915708:37
openstackgerritVasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port  https://review.openstack.org/25636508:37
openstackgerritVasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849608:37
openstackgerritVasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624408:37
openstackgerritVasyl Saienko proposed openstack/ironic: Create common neutron module  https://review.openstack.org/31739008:37
openstackgerritVasyl Saienko proposed openstack/ironic: Add network_interface node field DB, OBJ  https://review.openstack.org/31739108:37
openstackgerritVasyl Saienko proposed openstack/ironic: Refactor ironic enroll-node code  https://review.openstack.org/25636408:37
openstackgerritVasyl Saienko proposed openstack/ironic: Add configure_provision_network function  https://review.openstack.org/25636708:37
openstackgerritVasyl Saienko proposed openstack/ironic: Update Ironic VM network connection  https://review.openstack.org/25636608:37
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643208:37
openstackgerritVasyl Saienko proposed openstack/ironic: Add 'neutron' network driver  https://review.openstack.org/31739308:37
openstackgerritVasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326208:37
openstackgerritVasyl Saienko proposed openstack/ironic: Add network_interface API  https://review.openstack.org/31739208:37
openstackgerritVasyl Saienko proposed openstack/ironic: Add network_drivers  https://review.openstack.org/28585208:37
openstackgerritVasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation  https://review.openstack.org/25859608:37
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack  https://review.openstack.org/29352008:37
*** dmk0202 has joined #openstack-ironic08:38
*** derekh has joined #openstack-ironic08:47
*** dmk0202 has quit IRC08:48
mgould|afkmorning Ironic!08:49
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic-inspector-specs: Generic PCI Resource spec  https://review.openstack.org/31541308:51
*** sacharya has quit IRC08:57
*** daemontool_ has quit IRC08:58
*** daemontool has joined #openstack-ironic08:59
vsaienkomorning mgould09:00
mgould|afkmorning vsaienko09:01
*** _vsaienko has quit IRC09:08
sambetts|afkMorning all :)09:09
*** sambetts|afk is now known as sambetts09:09
*** daemontool has quit IRC09:11
*** daemontool_ has joined #openstack-ironic09:11
*** fragatin_ has joined #openstack-ironic09:11
*** _vs has quit IRC09:12
*** fragatina has quit IRC09:15
*** _vsaienko has joined #openstack-ironic09:15
*** _vs has joined #openstack-ironic09:15
*** bharathk has joined #openstack-ironic09:16
*** fragatin_ has quit IRC09:16
*** jtomasek has joined #openstack-ironic09:22
*** vishwanathj has joined #openstack-ironic09:24
*** fragatina has joined #openstack-ironic09:27
*** fragatina has quit IRC09:31
*** ohamada has quit IRC09:32
*** tdurakov has joined #openstack-ironic09:32
tdurakovhi09:32
sambettsgood morning tdurakov09:33
mgould|afkmorning tdurakov sambetts09:34
tdurakovsambetts: there is a thread in ml http://lists.openstack.org/pipermail/openstack-dev/2016-May/096292.html so, It would be great to gather feedback from ironic team09:34
tdurakovmgould|afk ^09:35
sambettsjroll: ^09:36
*** mgould|afk is now known as mgould09:37
mgouldtdurakov: state machines are a Good Thing, but we'll need to think about how we'll make changes to the state machine in the future09:39
mgouldbecause introducing new states to the main Ironic state machine is an ongoing World Of Pain (TM)09:39
sambetts^ ++09:40
tdurakovmgould: yes, that's why I'll be happy you answered the thread09:41
mgouldtdurakov: cool, will do09:42
tdurakovthanks in advance:)09:42
*** _vsaienko has quit IRC09:47
*** _vsaienko has joined #openstack-ironic09:47
mgouldtdurakov: done09:51
*** tangchen has joined #openstack-ironic09:52
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: WIP: Discovery tempest test  https://review.openstack.org/32376909:52
*** milan has quit IRC09:54
openstackgerritVasyl Saienko proposed openstack/ironic: Move create-node.sh to create_vms func  https://review.openstack.org/32285009:54
openstackgerritVasyl Saienko proposed openstack/ironic: Refactor enroll-node code  https://review.openstack.org/32319009:54
openstackgerritVasyl Saienko proposed openstack/ironic: TEMP PATCH.  https://review.openstack.org/32377009:54
*** ifarkas has quit IRC09:55
*** vdrok has joined #openstack-ironic09:56
*** sacharya has joined #openstack-ironic09:57
lucasagomesJayF, good question on the logs spec, made me think about it and also look at swift to see what it can offer09:57
vdrokmorning ironic, lucasagomes09:58
lucasagomesvdrok, hi there09:58
*** ifarkas has joined #openstack-ironic09:58
*** YiboCai has quit IRC09:58
*** milan has joined #openstack-ironic09:59
mgouldmorning milan09:59
*** daemontool_ has quit IRC10:00
*** sacharya has quit IRC10:03
*** vishwanathj has quit IRC10:03
openstackgerritTan Lin proposed openstack/ironic: [PoC]Ironic rolling upgrade framework - Make service compatible  https://review.openstack.org/30635710:03
sambettsvsaienko: having real issues working out whats going on with my CI and https://review.openstack.org/#/c/323190/, I'm watching my zuul logs and it says 323190 depends on changes [] :/10:03
vsaienkomorning vdrok, lucasagomes, sambetts10:04
vdrokmorning mgould sambetts and vsaienko10:05
lucasagomesvsaienko, morning :-)10:05
vdroksambetts, also sometimes this happens - https://review.openstack.org/322932, see how many times cisco ci is run, is it expected?10:06
*** dtantsur|bbl is now known as dtantsur10:06
dtantsurmorning sambetts, mgould10:06
vsaienkosambetts: I have prepared temp patch that will add changes to devstack10:06
vdrokmorning dtantsur10:06
dtantsurmorning vdrok10:06
openstackgerritVasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy  https://review.openstack.org/26915710:06
openstackgerritVasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port  https://review.openstack.org/25636510:06
openstackgerritVasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849610:06
openstackgerritVasyl Saienko proposed openstack/ironic: Add configure_provision_network function  https://review.openstack.org/25636710:06
openstackgerritVasyl Saienko proposed openstack/ironic: Update Ironic VM network connection  https://review.openstack.org/25636610:06
openstackgerritVasyl Saienko proposed openstack/ironic: DO NOT REVIEW  https://review.openstack.org/29643210:06
openstackgerritVasyl Saienko proposed openstack/ironic: Add 'neutron' network driver  https://review.openstack.org/31739310:06
openstackgerritVasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326210:06
openstackgerritVasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation  https://review.openstack.org/25859610:06
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack  https://review.openstack.org/29352010:06
vdrokheh10:06
sambettsvsaienko: :/ not again... I'm not sure if my CI keeps rechecking itself because we mention the recheck command in the comment, but it shouldn't do it10:08
sambettsvsaienko: I also think that my CI seems to rerun things if I tweak again downstream10:08
sambettsanything *10:08
sambettse.g. update the job10:09
*** _vs has quit IRC10:18
*** _vs has joined #openstack-ironic10:18
vsaienkosambetts: I'm sorry for the spam10:21
sambettsvsaienko: no problem, just gets me more data on my CI :-P10:21
*** yolanda has quit IRC10:22
*** yolanda has joined #openstack-ironic10:23
*** hoangcx has quit IRC10:25
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA upon a deploy failure  https://review.openstack.org/32351110:31
openstackgerritTan Lin proposed openstack/ironic: [PoC]Refactor configdrive into a new field  https://review.openstack.org/30635810:31
*** _vs has quit IRC10:36
*** _vs has joined #openstack-ironic10:38
vsaienkosambetts: merging multitenancy patches will solve all problems :)10:41
sambettsvsaienko: heh ;)10:41
sambettsits a crazy long chain now10:41
vsaienkosambetts: it is really big feature10:45
sambettsYup :-P10:46
openstackgerritMerged openstack/ironic-specs: follows cycle-with-intermediary release model  https://review.openstack.org/31818810:46
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA upon a deploy failure  https://review.openstack.org/32351110:47
jrollchenke: just ask your questions, I might not be the right person to answer so you should ask for the group :)10:51
jrollmorning everyone10:51
lucasagomesjrist, hi there10:51
*** vishwanathj has joined #openstack-ironic10:51
lucasagomesjroll, ^10:51
sambettsjroll: Morning10:51
lucasagomesjrist, heh again, sorry and morning too :D10:51
openstackgerritSam Betts proposed openstack/ironic-inspector: Add generic LLDP processing hook  https://review.openstack.org/32108210:52
*** ifarkas has quit IRC10:52
vdrokjroll, morning. so what is the result of midcycle poll?10:53
jrollvdrok: oh, good question! I'll email out today10:55
* jroll adds to his list10:55
*** vishwanathj has quit IRC10:56
*** _vs has quit IRC10:59
*** _vs has joined #openstack-ironic11:00
*** rbartal has quit IRC11:01
*** ishant is now known as ishant|away11:01
sambettsvsaienko: I'm working on trying to fix the nova side of this bug -> https://bugs.launchpad.net/ironic/+bug/153707611:11
openstackLaunchpad bug 1537076 in Ironic "Timed out waiting for Nova hypervisor-stats count >= 1 due to Nova Unable to establish connection to http://127.0.0.1:35357/v2.0/tokens" [Critical,Fix released] - Assigned to Dmitry Tantsur (divius)11:11
sambettsvsaienko: https://review.openstack.org/#/c/320040/11:11
*** slagle has joined #openstack-ironic11:13
sambettsvsaienko: I saw you put up a fix for the grenade stuff, I wondered if you had seen it recently, because the trace in that bug is v.old, the code triggering it doesn't even exist in stable/mitaka11:14
openstackgerritvinay kumar muddu proposed openstack/ironic-python-agent: Wait for at least one interface before node lookup  https://review.openstack.org/30350311:15
*** lucasagomes is now known as lucas-hungry11:16
divya_Hi  mat12811:21
divya_am facing issue in openvswitch agent - its not starting11:21
divya_when it starts nova compute going down11:22
*** vishwanathj has joined #openstack-ironic11:22
divya_can u pour some ur comments on this11:22
*** aparnav has quit IRC11:26
dtantsurmorning jroll11:29
*** bharathk has quit IRC11:30
milanmorning mgould sambetts jroll vdrok :)11:30
vdrokgood morning milan11:30
*** vishwanathj has quit IRC11:30
*** vishwanathj has joined #openstack-ironic11:31
*** _vs has quit IRC11:33
*** _vs has joined #openstack-ironic11:34
*** links has quit IRC11:34
vsaienkosambetts: I saw this issue on grenade jobs11:35
*** vishwanathj has quit IRC11:35
*** ifarkas has joined #openstack-ironic11:36
sambettsvsaienko: do you have a trace that I can look at ?11:37
jrollsambetts: tdurakov: I think that josh and laski already said anything I would have to say, but I may end up replying eventually :)11:37
jrollre: state machine thing11:37
*** _vs has quit IRC11:39
*** _vs has joined #openstack-ironic11:40
vsaienkosambetts: http://logs.openstack.org/09/311109/25/experimental/gate-grenade-dsvm-ironic/58a7a45/logs/new/screen-n-cpu.txt.gz11:42
sambettsvsaienko: awesome :D11:42
openstackgerritVladyslav Drok proposed openstack/ironic: Add API reference for portgroups  https://review.openstack.org/32279611:43
*** baoli has joined #openstack-ironic11:44
mat128hi divya_ (and jroll, mgould, sambetts, vdrok, milan, dtantsur)11:46
mat128divya_: got any logs?11:46
vdrokmorning mat128 :)11:46
milanmat128, hi :)11:46
*** links has joined #openstack-ironic11:47
mgouldmorning mat12811:49
*** _vs has quit IRC11:51
*** _vsaienko has quit IRC11:51
*** _vsaienko has joined #openstack-ironic11:53
*** _vs has joined #openstack-ironic11:53
*** sacharya has joined #openstack-ironic11:54
*** phuongnh has quit IRC11:56
divya_hi mat128 am not able to open logs11:56
divya_the server become very slow11:56
openstackgerritSam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url  https://review.openstack.org/31614911:58
openstackgerritSam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions  https://review.openstack.org/31576611:58
openstackgerritSam Betts proposed openstack/ironic: Prevent URL collisions with sub-controllers: nodes/ports  https://review.openstack.org/31451411:58
*** baoli_ has joined #openstack-ironic11:59
*** sacharya has quit IRC11:59
mat128divya_: ouch, any OOB you can use to see what's going on?12:01
mat128out of band*12:01
*** baoli has quit IRC12:02
*** nico-afk has quit IRC12:04
*** _vs has quit IRC12:05
*** _vsaienko has quit IRC12:05
divya_mat128 : Timed out waiting for nova-conductor.  Is it running? Or did this service start before nova-conductor?  Reattempting establishment of nova-conductor connection...12:06
divya_openvswitch am not able to open the file12:06
dtantsurhi mat12812:06
mat128o/12:06
* dtantsur spends the whole time fighting with his new laptop, so he's not overly responsive12:06
*** _vsaienko has joined #openstack-ironic12:07
mat128Joys of upgrading laptops12:07
*** _vs has joined #openstack-ironic12:07
*** jcoufal has joined #openstack-ironic12:08
*** jtomasek has quit IRC12:11
*** lucas-hungry is now known as lucasagomes12:12
*** xavierr_afk is now known as xavierr12:14
*** divya__ has joined #openstack-ironic12:15
*** openstackgerrit has quit IRC12:18
*** openstackgerrit has joined #openstack-ironic12:18
*** DCWilliams_VA has joined #openstack-ironic12:19
*** rbudden has joined #openstack-ironic12:19
*** DCWilliams_VA is now known as Guest2973512:19
*** Guest29735 has quit IRC12:20
*** jtomasek has joined #openstack-ironic12:20
*** jjohnson2 has joined #openstack-ironic12:21
*** dprince has joined #openstack-ironic12:23
divya__when i restart my neutron openvswitch, the nova-comp goes donw12:24
divya__what could be the issue12:24
divya__vsaienko, mat2812:25
divya__anybody can suggest12:25
divya__mat12812:26
mat128divya__: never troubleshooted openvswitch, but do you see anything happening when you start neutron?12:26
mat128divya__: either in the neutron logs, or the nova logs12:26
mat128maybe the api is getting hammered?12:27
*** trown|outtypewww is now known as trown12:27
divya__mat128, neutron is gettng started12:28
vsaienkodovya__: how you restart openvswitch?12:29
divya__q-agt using rejoin12:30
vsaienkodivya__ so you just kill q-agt process and than start it back?12:30
divya__let me try vsaienko12:31
vsaienkodivya__ could you provide exact shell command?12:31
divya__i use rejoin.stack.sh12:33
divya__my problem is i am able to start q-agt but nova-compu stops when openvswitch is started12:33
divya__any configuration dependent??12:34
thiagopgood morning Ironic12:34
sambettsIsn't there a log or something?12:34
sambettsa traceback, anything to indicate why its failing?12:35
divya__i am not able to open logs as well.. it gets hangs12:35
sambettshow are you trying to open the logs?12:36
divya__using vi editoru12:36
sambettsif you bring up the screen using screen -x, and go to the n-cpu window, you should be able to see the error if its died for some reason12:37
mat128divya__: maybe it's a network problem? when you start openvswitch it takes over your management IP then you lose access?12:38
vsaienkodivya, rejoin stack has been removed in Mitaka community do not using it anymore12:38
divya__let me try12:40
vsaienkodivya__ to restart specific process you have to open stack screen, go to specific tab press CTRL+C, wait for process termination and then start it back, as it was pointed by sambetts12:40
divya__yea vsaienko i am taking from old release12:40
divya__how to restart in mitaka12:40
vsaienkodivya__ if you using screen ^^, if do not using screen than just kill process and run command12:41
mat128divya__: if you are using devstack, you could send us your local.conf and maybe we could reproduce?12:41
*** itamarl has joined #openstack-ironic12:42
vsaienkomat128, divya__ rejoin-stack.sh has been removed due to this reason, it is not working as expected. I doubt that local.con will help here. Please have look https://review.openstack.org/#/c/291453/12:46
*** PollyZ_ has joined #openstack-ironic12:47
sambettsvsaienko: do you have a way of patching in a version of ironicclient into your grenade setup so you can test to see if a change fixes that nova issue?12:50
vsaienkosambetts, you are talking about Newton version or commit that is on review?12:51
*** Goneri has joined #openstack-ironic12:52
vsaienkosambetts: there is one of possible ways https://review.openstack.org/#/c/296432/42/devstack/lib/ironic12:52
sambettsvsaienko: ah I see12:54
openstackgerritSam Betts proposed openstack/python-ironicclient: Catch RetriableConnectionFailures from KAuth and retry  https://review.openstack.org/32385112:55
sambettsvsaienko: ^12:55
*** lindycoder has joined #openstack-ironic12:56
*** piet has joined #openstack-ironic13:03
openstackgerritVasyl Saienko proposed openstack/ironic: Patch to test Grenade.  https://review.openstack.org/31110913:04
openstackgerritVasyl Saienko proposed openstack/ironic: Update resources subnet CIDR  https://review.openstack.org/31708213:04
openstackgerritVasyl Saienko proposed openstack/ironic: Disable disk_config compute-feature-enabled in tempest  https://review.openstack.org/32198213:04
openstackgerritVasyl Saienko proposed openstack/ironic: Make sure create_ovs_taps creates unique taps  https://review.openstack.org/31910113:04
TheJuliao/13:05
* TheJulia is a zombie13:05
*** rloo has joined #openstack-ironic13:05
NobodyCamZombie is not alone in that feeling13:07
*** afaranha has joined #openstack-ironic13:07
*** wajdi has quit IRC13:07
vsaienkosambetts: https://review.openstack.org/#/c/31110913:08
NobodyCamgood morning Ironicers13:08
vsaienkogood morning NobodyCam13:08
NobodyCammorning vsaienko13:08
dtantsurmorning TheJulia, NobodyCam13:09
NobodyCammorning dtantsur :) how goes the day13:09
*** sdake has joined #openstack-ironic13:10
TheJuliagood morning NobodyCam13:10
dtantsurNobodyCam, could be better, but could be even worse :) and for you?13:10
NobodyCamGood Morning TheJulia (zombie)13:10
sambettsvsaienko: Oh awesome, we'll see if it works :D13:12
NobodyCamdtantsur: same.. Will be a long day.. Have a presentation I'm doing later today13:12
*** ayoung has joined #openstack-ironic13:12
*** parlos has joined #openstack-ironic13:14
TheJuliaSpeaking of yesterdays topics, the lack of upper bounds breaking fresh installs that are not in the gate.... Anyone have the bug IDs for those handy?13:14
*** vmud213 has quit IRC13:17
dtantsurTheJulia, you mean https://bugs.launchpad.net/bugs/1586979 ?13:17
openstackLaunchpad bug 1586840 in oslo.messaging "duplicate for #1586979 amqp 2.0 breaks the kombu driver" [Undecided,In progress] - Assigned to Kirill Bespalov (k-besplv)13:17
TheJuliathanks dtantsur13:20
mgouldmorning NobodyCam TheJulia13:21
NobodyCammorning mgould :)13:21
rloohi and good morning dtantsur, vsaienko, sambetts, mgould, NobodyCam, TheJulia13:22
dtantsurmorning rloo13:22
*** joprovost has joined #openstack-ironic13:22
TheJuliagood morning rloo13:22
NobodyCamGood Morning rloo sambetts :)13:22
sambettso/ rloo, TheJulia, NobodyCam13:22
TheJuliagood morning sambetts13:24
*** piet has quit IRC13:27
vsaienkogood morning rloo, TheJulia13:29
thiagopmorning NobodyCam TheJulia rloo13:29
NobodyCammorning thiagop :)13:29
sambettsmat128: WRT a node being called vendor_passthru, I don't think a node can be called vendor_passthru, at least I think when I tried to do it told me it was an invalid name :/13:30
sambettsjroll: do you know ^13:30
mat128sambetts: were you attempting to fix this%?13:30
mat128I felt so because vendor_passthru was removed from the list of reserved names13:31
NobodyCammorning mat12813:31
mat128morning NobodyCam13:31
sambettsmat128: I think vendor_passthru doesn't pass the real name validation, there are some regex it has to pass13:31
sambettsthey prevent things like punctuation in the names13:31
mat128I know for a fact that there is a regex on the driver name13:31
*** divya__ has quit IRC13:32
mat128sambetts: your commented test, does it pass when you uncomment?13:32
sambettsif i remember correctly I think node names have to be valid hostnames or something13:32
sambettsmat128: I don't think it did, which is why I wanted to confirm I wasn't being dumb :-P13:33
mat128ahh13:33
mat128sambetts: so it fails not because of routing issues but because of the regex?13:33
sambettsyeah, when it tries to create the node to test against13:34
*** sdake has quit IRC13:34
vdrokmorning TheJulia NobodyCam and rloo13:36
NobodyCammorning vdrok :)13:37
openstackgerritHugo Nicodemos proposed openstack/python-oneviewclient: Remove insecure connection warnings  https://review.openstack.org/32387313:39
*** _vsaienko has quit IRC13:39
*** _vs has quit IRC13:39
sambettsmat128: what I wanted to check though, is that there is a if pecan.request.version.minor < versions.MINOR_10_UNRESTRICTED_NODE_NAME check in on of the fucntions that seems to change if underscore is allowed or not13:39
*** _vsaienko has joined #openstack-ironic13:40
*** _vs has joined #openstack-ironic13:41
*** mgoddard_ has joined #openstack-ironic13:44
openstackgerritImre Farkas proposed openstack/ironic: [WIP] DRAC RAID configuration  https://review.openstack.org/17631913:45
vdroksambetts, left a comment on ironicclient patch. reason why I'd prefer it do retries in keystoneauth itself is that I think we should move from httpclient to keystoneauth admin_token auth method at some point, and all this _with_retries logic will be removed too13:47
jrollsambetts: dunno off the top of my head, no13:47
*** mgoddard has quit IRC13:47
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/32388913:48
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/32389013:48
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/32306613:48
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/32389113:48
sambettsvdrok: yeah i looked into it, and there is no way (right now) to inject a higher retry count into the point that is failing in that traceback you sent me :'( that would have been the ideal solution I guess13:50
*** nicodemos has joined #openstack-ironic13:52
*** links has quit IRC13:52
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/32391613:53
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/32300313:53
*** sdake has joined #openstack-ironic13:55
*** mtanino has joined #openstack-ironic13:55
*** sacharya has joined #openstack-ironic13:56
*** ametts has joined #openstack-ironic13:58
sambettsmat128: hmmm ... weird... so the reason that my test seems to be failing is because when the tests are running pecan.request.version.minor < versions.MINOR_10_UNRESTRICTED_NODE_NAME returns True...13:59
sambettsmat128: you would have thought we should be running the newest version...13:59
*** sacharya has quit IRC14:00
*** joprovost1 has joined #openstack-ironic14:02
*** joprovost has quit IRC14:04
*** joprovost1 is now known as joprovost14:04
vdroksambetts, so just adding connect_retries=CONNECT_RETRIES in here https://review.openstack.org/#/c/323851/1/ironicclient/common/http.py@525 does not work? it should... or we can do connect_retries = max_retries during init of session client here https://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L61414:05
vdrokif adding it to _RETRY_EXCEPTIONS works, ^ should work too14:06
*** _vs has quit IRC14:07
*** _vsaienko has quit IRC14:07
*** rloo has quit IRC14:08
*** rloo has joined #openstack-ironic14:08
openstackgerritVladyslav Drok proposed openstack/ironic: Add API reference for portgroups  https://review.openstack.org/32279614:09
*** _vsaienko has joined #openstack-ironic14:11
*** _vs has joined #openstack-ironic14:11
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Returns CPU flags in the CPU inventory  https://review.openstack.org/30829214:12
dtantsurrloo, updated ^^^14:12
rloodtantsur: thx, +2'd.14:14
openstackgerritVasyl Saienko proposed openstack/ironic: Refactor enroll-node code  https://review.openstack.org/32319014:14
openstackgerritVasyl Saienko proposed openstack/ironic: TEMP PATCH.  https://review.openstack.org/32377014:14
sambettsvdrok: the problem is https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L382 leads to https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/v2.py#L64 which calls https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L586 which calls https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L289 and14:15
sambettsit doesn't pass connect_retries through :(14:15
openstackgerritOpenStack Proposal Bot proposed openstack/python-oneviewclient: Updated from global requirements  https://review.openstack.org/32394314:17
vdroksambetts, aha, I see, thanks for clarifying!14:18
vdrokyea, that's bad :(14:18
sambettsthe only way is the catch it on the outside :( adding it to _RETRY_EXCEPTIONS seemed to be the neatest way to fix the issue, and if it works I'd like to backport it to stable/mitaka so we can get rid of the need to restart nova in the grenade stuff14:19
*** joprovost has quit IRC14:21
sambettsmat128: Worked out whats going on, the test itself is limiting the API version to 1.5, and at that time node names had to be hostname compatible which meant no underscores, since 1.10 we've unrestricted it though14:23
mat128sambetts: ooh14:24
mat128sambetts: does that mean we break other tests when bumping that to 1.10?14:24
mat128and/or tests are missing for anything between 1.5 and 1.10?14:24
openstackgerritRuby Loo proposed openstack/ironic: NOTIFICATION_TRANSPORT should be global  https://review.openstack.org/32352614:25
*** d0ugal has quit IRC14:25
*** itamarl has quit IRC14:26
sambettsmat128: 1.5 was when we first introduced node names so I think its on purpose, but it does mean that none of those tests are running against the newer API versions although they should be backward compatible, I was planning to make it so that that specifc test ran on 1.1014:27
*** ametts has quit IRC14:28
*** sabeen has joined #openstack-ironic14:31
*** _vs has quit IRC14:31
*** _vsaienko has quit IRC14:32
*** itamarl has joined #openstack-ironic14:32
*** _vsaienko has joined #openstack-ironic14:34
*** _vs has joined #openstack-ironic14:34
sambettsmat128, jroll: Should our unit tests ever be limiting themselves to a specifc API version? i.e. if we add a new minor version shouldn't we be unit testing to ensure that that an older feature works even if the API version in the request is MINOR_MAX_VERSION?14:35
*** d0ugal has joined #openstack-ironic14:35
mat128sambetts: we should still prove the old version behaves as it was behaving14:35
jroll^14:36
sambettsyeah :/ should we be testing older features against newer API version though?14:37
jrollit certainly wouldn't hurt14:37
jrollI just don't know if we want for x in range(max_version) in every unit test :)14:38
mat128sambetts, jroll: we can have a test class that derives from another ;)14:38
mat128runs the same test against new api version, plus adds it's own bunch of tests14:38
jrollsure14:39
sambettse.g. TestNodesControllerV1_10(TestNodesControllerV1_9) ??14:39
mat128yup14:40
mat128and if something from the past isn,t supported anymore14:40
mat128you ``def bleh(self): pass`` or something14:40
mat128so it givesyou a clear view of what is supported and what changed14:40
sambettswe'd have to add new ones for every minor version we add though right? even if it doesn't affect the nodes controller14:40
*** _vsaienko has quit IRC14:41
mat128sambetts: hmm14:41
openstackgerritDmitry Tantsur proposed openstack/ironic-specs: The driver composition reform  https://review.openstack.org/18837014:41
dtantsurplz review ^^^14:41
*** _vs has quit IRC14:41
mat128yeah, but as you said, it'll simply derive from the other one14:41
mat128sambetts: not adding any test if nothing changed14:41
sambettsmat128: just thinking then any API bump would have to add a bunch of classes to the tests that did nothing14:42
*** mgoddard_ has quit IRC14:42
*** ametts has joined #openstack-ironic14:42
*** mgoddard has joined #openstack-ironic14:42
*** itamarl has quit IRC14:43
sambettsI think we actually only really need to test API version that added the feature, and API max version, because if a version between the to breaks anything then it should be caught along the way right?14:43
mat128"and API max version"14:44
mat128sambetts: that means you would have a TestNodesControllerVMAX() ?14:44
mat128interesting14:44
mat128if it broke, you would know, and if you add soetmhign, you have to subclass14:45
mat128sounds good14:45
*** _vsaienko has joined #openstack-ironic14:46
*** _vs has joined #openstack-ironic14:46
sambettswhen the feature is added api version and vmax are equal, and then if someone comes along and in their patch adds a new feature and bumps then we'll be testing both, and if it breaks then it shouldn't merge, meaning that we'll be catching the problems along the way instead of retesting old versions that ideally shouldn't be changing14:47
*** d0ugal has quit IRC14:48
*** d0ugal has joined #openstack-ironic14:48
*** d0ugal has quit IRC14:48
*** d0ugal has joined #openstack-ironic14:49
*** vmud213 has joined #openstack-ironic14:49
dtantsurjroll, isn't https://review.openstack.org/323190 a good example of why we would need an import-from-file command in ironicclient? ;)14:50
*** krtaylor has joined #openstack-ironic14:50
*** vmud213 has quit IRC14:51
*** vmud213 has joined #openstack-ironic14:51
jrolldtantsur: you think we should build a json object in bash instead?14:51
*** e0ne has quit IRC14:52
dtantsurjroll, with 'jq' is not even that hard. and anyway we have to build some file there, I don't want us to parse it in bash as well14:52
dtantsuralso you argument was "a script with node-create is enough". does it apply here?14:52
*** spandhe has joined #openstack-ironic14:53
jrolldtantsur: my argument was for a python script14:54
dtantsurjroll, so should we apply it here too?14:54
dtantsurcause this patch is doing what you said is unnecessary: importing ironic nodes from a file14:54
* dtantsur would love our devstack plugin to be written in Python, but that's another topic14:55
jrolldtantsur: I never said importing nodes from a file was unnecessary, I said that everyone would have a different input format that needs to be translated anyway14:55
dtantsurwell, JSON/YAML following Ironic node format is something which is easy to agree on, and that's essentially what this patch invents, but in INI format14:56
jrolldtantsur: anyway, if you remember, I said I wasn't blocking it, I just didn't think it was very important compared to other things going on14:56
jrollthe common format is easy to agree on, but everyone's vendor/cmdb/whatever will have a different format, and ops will need a translation script anyway14:57
dtantsuryeah.. I just wanted to know what you think after seeing this patch.. cause I was nearly convinced we should not do it14:57
jrollI think INI syntax is silly for something like this14:57
jrollalso think bash is silly14:57
jroll:)14:57
*** spandhe has quit IRC14:57
dtantsur+100 for both14:57
NobodyCammorning jroll :)14:57
jrollI'm still indifferent as to "ironic node-enroll-from-file" command14:58
jrollhey NobodyCam :)14:58
NobodyCam:)14:58
*** yolanda_ has joined #openstack-ironic14:58
*** yolanda has quit IRC15:00
openstackgerritLucas Alvares Gomes proposed openstack/virtualbmc: Set IPMI session timeout for the virtual BMCs  https://review.openstack.org/31602215:01
*** yolanda_ is now known as yolanda15:01
*** vmud213 has quit IRC15:01
*** cinerama has joined #openstack-ironic15:02
*** pcaruana has quit IRC15:04
openstackgerritVladyslav Drok proposed openstack/ironic: Add API reference for portgroups  https://review.openstack.org/32279615:05
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Remove support for the old bash ramdisk  https://review.openstack.org/31922715:07
dtantsursambetts, please take a look when you some time ^^^15:07
sambettsdtantsur: will do :D15:08
*** rcernin has quit IRC15:09
*** d0ugal has quit IRC15:10
rlooJayF: wrt oslo.policy, are you going to work on this RFE: https://bugs.launchpad.net/ironic/+bug/1526752 ?15:10
openstackLaunchpad bug 1526752 in Ironic "[RFE] Policy Improvements" [Wishlist,Confirmed]15:10
JayFrloo: aye15:10
rlooJayF: thx. Please assign to yourself :)15:11
JayFaye15:11
JayFI think https://review.openstack.org/#/c/221778/ should probably just be abandoned?15:11
*** d0ugal has joined #openstack-ironic15:12
NobodyCammorning JayF15:12
jrollJayF: yeah, or you could update it, either way15:12
rlooJayF: probably. maybe ping Anton?15:12
jrollI can abandon if you like15:12
jrollI pinged him previously, he said he isn't working on it15:12
JayFI can abandon it too, I'm core on specs15:12
JayFjust wanted to make sure folks were onboard15:12
jrollah right15:12
jrollI don't care as long as a spec for this gets done :)15:12
rloook, abandon it then if there isn't anything useful there.15:12
rlooJayF: I mean, in your spec, see if there are any relevant comments, whatever in this spec.15:13
JayFI won't let the little history that exists be ignored when I develop my spec :)15:14
JayFI love not having to start from zero15:14
rloothx JayF15:14
*** yolanda_ has joined #openstack-ironic15:14
*** yolanda has quit IRC15:14
*** yolanda_ is now known as yolanda15:16
*** tesseract has quit IRC15:18
*** milan has quit IRC15:18
vsaienkojroll, rloo please have look https://review.openstack.org/#/c/323190/15:20
rloovsaienko: is that urgent? (in middle of something)15:21
vsaienkorloo, no it is not urgent. I would like to know your opinion about it.15:22
rloovsaienko: OH. I cannot. get. distracted. which. would. be. easy. to. do. cuz. i. don't. want. to. do. what. i. am. currently. doing. So maybe later today...15:23
sambetts:-P15:24
*** wajdi has joined #openstack-ironic15:24
thiagoplol15:25
*** joprovost has joined #openstack-ironic15:28
dtantsurjroll, vsaienko, I'm trying to avoid having too much logic in the devstack plugin. If we need wrappers around our CLI, we need to start improving it. Just another example: I would prefer a --wait flag for node-set-provision-state instead of everyone inventing their own version.15:30
dtantsurjroll, vsaienko, this "just create a wrapper" approach hit us hard in tripleo when we've found out that people try to avoid large parts of our CLI and use wrappers instead.15:31
*** _vsaienko has quit IRC15:34
*** _vs has quit IRC15:34
*** sacharya has joined #openstack-ironic15:34
*** _vsaienko has joined #openstack-ironic15:35
*** _vs has joined #openstack-ironic15:35
JayFhttps://review.openstack.org/#/c/313800/ is the review to change ironic jobs to use ipxe+tinyipa -- I addressed rloo's comments and just rebased it so it will pass tests again. Would love Ironic reviews15:36
*** mbound has joined #openstack-ironic15:36
JayFand is very relevant to the QA meeting happening in 84 minutes :)15:36
*** divya__ has joined #openstack-ironic15:37
*** divya__ has quit IRC15:37
JayFdtantsur: including the ram requirement?15:42
JayFdtantsur: I wasn't going to touch the inspector jobs without talking to you, but if you're sure you can inspect in 384mb of ram I'll change that and repush15:43
dtantsurit's a good question. lets push to a separate patch which is easier to revert (and I'll check it first)15:43
JayFah, then let me revert my change I just did15:43
JayFlol15:44
dtantsurI'm pretty sure it's possible, but.. no hurry :)15:44
dtantsurthanks15:44
dtantsurJayF, well, hold on15:44
sambettsvsaienko: this is the log from the grenade run you set up for me, http://logs.openstack.org/09/311109/55/experimental/gate-grenade-dsvm-ironic/5174f70/logs/new/screen-n-cpu.txt.gz looking at the trace back it didn't pull in my patch correctly15:44
dtantsurok, you did it already, nevermind :)15:44
* dtantsur can't make up his mind so quickly15:44
JayFdtantsur: https://review.openstack.org/313800 + https://review.openstack.org/32399415:46
*** _vsaienko has quit IRC15:46
*** _vs has quit IRC15:46
dtantsurthanks!15:46
JayFnp at all15:47
JayFthanks for reviewing15:47
JayFI know it'll need to be soaked in Ironic +1s before infra will land it15:47
dtantsurhehe :) sometimes they move surprisingly fast15:48
thiagopJayF: the "no -src" job shouldn't be running on both coreos and tinyipa too?15:48
JayFI'm not sure what you're referring to? We run -src jobs against IPA today; my change renames them to coreos-src and tinyipa-src and runs both ramdisks on every ipa change15:50
JayFI'd hope to add DIB to that soon15:50
thiagopJayF: https://review.openstack.org/#/c/313800/8/jenkins/jobs/projects.yaml@253615:51
JayFif you look at line 2576 + 258115:51
JayFyou'll see the new versions of those jobs15:51
thiagopJayF: it doesn't have a coreos nor tinyipa in the name of the job15:51
*** _vsaienko has joined #openstack-ironic15:51
*** _vs has joined #openstack-ironic15:51
JayFthiagop: that's the removal of the old job. You have to look at the other line numbers to see the new ones.15:52
thiagopJayF: I thought that the -src job builds the ipa ramdisk from source and the "no -src" downloaded it from tarballs.o.o, maybe I'm just being ignorant15:53
JayFthiagop: that's correct. The -src jobs run on IPA, and the non-src jobs run on Ironic.15:53
*** rajinir has joined #openstack-ironic15:54
JayFWe will no longer have a CoreOS non-source job after this change15:54
JayFI explain all of this pretty well in the commit messsage; basicaly IPA tests all ramdisks, then Ironic tests aganist a prebuilt copy of one of them15:54
*** Sukhdev has joined #openstack-ironic15:54
*** e0ne has joined #openstack-ironic15:54
*** sdake has quit IRC15:54
JayFso IPA owns and tests the relationship with its ramdisks, so all Ironic cares about is that there's a working ramdisk at all (which is why we don't, and never will, test Ironic proper against all ramdisks)15:55
JayFtl;dr: there's no need for non-src ramdisk specific jobs15:55
*** cdearborn has joined #openstack-ironic15:57
thiagopJayF: I see your point, thanks for explaining15:57
sambettsJayF: havn't we had problems with prebuilt ramdisks being broken by during the build process because of the missing deps on the build machine on the post job, that were only caught by the non-src jobs on ironic?15:57
mat128sambetts: but those are fixed now15:58
*** penick has quit IRC15:58
mat128sambetts: we have builds of both CoreOS and TinyIPA15:58
JayFI think sambetts is saying that we got free "monitoring" of if the upload job was working15:58
mat128sambetts: I think the only case where you will build from source is if you Depends-on a source change in IPA15:58
JayFbecause Ironic jobs would fail15:59
mat128Ahh15:59
mat128yes15:59
JayFif we aren't using a prebuilt CoreOS ramdisk in Ironic jobs, how do we know it built properly?15:59
sambettsexactly15:59
JayFMy answer is: That's a great problem we should address with real engineering instead of just being lucky enough to catch it15:59
JayFlol15:59
sambettshahaha XD15:59
JayFI mean that kinda seriously, right?15:59
JayFThe info about success of post jobs is accessible, nobody checks it and nobody alerts on it15:59
sambettsyeah, I agree15:59
sambettsI would love the post job to post back onto the job it was triggered from... or something16:00
JayFI don't know how it's possible to do that16:00
JayFthat'd probably be a decent ML post, on how to detect failure of post jobs16:00
JayFI bet there's tooling that I'm not familiar with16:00
JayFsambetts: ^ you wanna ask the question or shall I?16:01
sambettsJayF: we'd also have to run the tempest tests in the post job too to ensure everything is working as expected :/16:01
JayFsambetts: the post job doesn't even spawn a full devstack16:01
JayFsambetts: it's literally just a trusty image that runs the build16:01
sambettsyup16:01
*** penick has joined #openstack-ironic16:01
JayFI think the first concern should be: if the job fails, know it failed16:02
*** joprovost has quit IRC16:02
JayFsecond concern is: how do we know the uploaded ramdisks work16:02
JayFperhaps as a step two to this project-config pr we should add experimental and/or periodic, non-src coreos jobs?16:02
JayFI just think that's a workaround in reality, because we should be able to trust the post job did a right thing16:03
JayFand not have to eat resources periodically to confirm it16:03
sambettsJayF: is there any reason why we couldn't ditch the post job, and have the gate job upload the image it ran the tempest tests aginst upload the image?16:03
JayFsambetts: I built the post job based on what infra told me to get an image working16:03
JayF(the coreos one, I mean)16:04
JayFif there's a better way to do it, I'm game to having it changed; but I think none of these are good reasons to block my project-config pr16:04
JayFjust more work we need to continue doing16:04
*** mjura has quit IRC16:05
sambettsyup, I mean right now there is always going to be the period of time between IPA merge, and Post job building and uploading, where Ironic jobs are running against an outdated version of IPA16:06
JayFthat's true for *any* package Ironic depends on16:06
JayFincluding new ironic-lib releases16:06
sambettsthat time can be quite long if the post job queue gets long, which it has been getting resently16:06
sambettsrecently*16:06
JayFit's just more awkward because we don't explicitly manage the versinos16:07
JayFsambetts: I'm serious; lets write all this down and ask Infra if there are better solutions. I don't know what they are but it sounds like this is a case that might be handled better with existing tooling16:08
JayFI love the idea of the tempest job passing on a gate causing the ramdisk to upload16:08
JayFbut I think it's hard to do that in a way where if tinyipa succeeed and coreos fails that it works16:08
JayFi.e. you have to wait until it passes all tests before doing the upload16:08
JayFand the jobs are sufficiently split I'm not sure that's possible16:08
sambettsJayF: you can do sub-jobs I discovered that today when playing with my CI, e.g. run this job only if this job passes sort of thing16:10
sambettsJayF: http://docs.openstack.org/infra/zuul/zuul.html#projects in their example it run the UT jobs only if the project-merge job passes16:11
JayFinteresting16:11
JayFyou have my moral support in going down this path :)16:11
JayFyou'll excuse me if I don't wanna have more infra changes to rebase over and over without review for a month16:11
JayFgonna wait until I get this one in before I bite off anymore16:12
JayFlol16:12
*** fragatina has joined #openstack-ironic16:12
sambettshehe I know what thats like16:12
*** sacharya has quit IRC16:13
dtantsursee you tomorrow16:13
JayFo/16:13
*** dtantsur is now known as dtantsur|afk16:13
*** fragatina has quit IRC16:17
vsaienkosambetts look your fix to ironicclient did the trick16:18
*** sdake has joined #openstack-ironic16:19
sambettsvsaienko: oh? the grenade job failed though?16:19
lucasagomesvsaienko, dtantsur|afk jroll re nodes from file, I do have a PoC that I created long ago about it. Do you guys want me to refactor that quickly and submit as an external command for ironic?16:20
vsaienkosambetts: yeah it is failed http://logs.openstack.org/09/311109/55/experimental/gate-grenade-dsvm-ironic/5174f70/logs/new/screen-n-cpu.txt.gz#_2016-06-01_15_23_58_85116:20
lucasagomeswhich could later be used in the client16:20
sambettsvsaienko: this line: 2016-06-01 15:23:58.851 7828 ERROR oslo_service.service   File "/usr/local/lib/python2.7/dist-packages/ironicclient/common/http.py", line 519, in _http_request16:21
sambetts2016-06-01 15:23:58.851 7828 ERROR oslo_service.service     raise_exc=False, **kwargs)16:21
sambettsvsaienko: shows me that its using the wrong version of Ironic client16:21
sambettsbecause that code isn't on 519 any more16:21
vsaienkosambetts: hmm16:22
sambettsthat code is on L525/616:22
vsaienkolucasagomes: if you have a time it would be good16:22
sambettsso I don't think its using my change16:22
lucasagomesvsaienko, ack, should be quick... 1 sec16:22
*** Romanenko_K has quit IRC16:23
*** kromanenko_ has quit IRC16:23
*** sdake has quit IRC16:23
*** kromanenko__ has quit IRC16:23
vsaienkosambetts: yeah, the client from master has been installed16:24
sambettsvsaienko: even the ironicclient master doesn't have that code there anymore?16:27
vdroksambetts, it is 1.3.0 release16:29
vdrokinstalled from pypi16:29
*** ijw has joined #openstack-ironic16:29
sambettsvsaienko: yeah, so its installed it off pypi, 1.3.1 is latest on pypi16:30
sambettsand that is closer to master16:31
vsaienkosambetts: let me try one thing16:31
*** ijw has quit IRC16:32
*** sdake has joined #openstack-ironic16:32
openstackgerritVasyl Saienko proposed openstack/ironic: Patch to test Grenade.  https://review.openstack.org/31110916:39
vsaienkosambetss: ^^16:39
*** jtomasek has quit IRC16:41
*** spandhe has joined #openstack-ironic16:42
sambettsvsaienko: heh interesting :-p16:43
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Import inventory from file (PoC)  https://review.openstack.org/32401516:44
lucasagomesvsaienko, dtantsur|afk jroll ^16:45
sambettslucasagomes: does it do node properties too?16:46
lucasagomessambetts, yup, extra and all16:46
lucasagomessambetts, just look athe schema16:46
sambettssweet :) I had a look at the paste but none of the examples has node properties16:46
lucasagomessambetts, yeah, the example inventory is succinct for readbility16:47
sambettslucasagomes: looks awesome :D16:47
lucasagomessambetts, yeah, that small yaml will create 1 chassis, 4 nodes and 5 ports16:48
*** e0ne has quit IRC16:48
lucasagomesI can work and make it a proper script, unless vsaienko also likes it and want to take it over16:48
jrolllucasagomes: we'd need pykwalify in global-requirements too, I think16:49
lucasagomesjroll, oh that would be handy16:49
lucasagomesjroll, to be fair I also think that it should ultimetely for to the ironicclient, but, starting as a script in devstack is a first step I guess16:50
*** rpioso has joined #openstack-ironic16:50
*** vishwanathj has joined #openstack-ironic16:50
jrolllucasagomes: right, either way I think we need it in g-r16:51
lucasagomesjroll, yeah, shouldn't be a problem... It's MIT license16:51
lucasagomesjroll, let'se see what vsaienko thinks about it first16:51
jrollsure16:51
*** trown is now known as trown|lunch16:52
*** derekh has quit IRC16:53
*** vishwanathj has quit IRC16:54
openstackgerritMerged openstack/python-oneviewclient: Updated from global requirements  https://review.openstack.org/32394316:57
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Import inventory from file (PoC)  https://review.openstack.org/32401516:57
*** PollyZ_ has quit IRC17:00
lucasagomesJayF, btw, I've updated the spec for the logs. Thanks for the suggestions, now it includes adding expiration time and other neat stuff17:00
*** vishwanathj has joined #openstack-ironic17:00
JayFI saw that, will look back over it17:00
lucasagomescool, thanks17:01
*** links has joined #openstack-ironic17:02
*** baoli_ has quit IRC17:03
*** mbound has quit IRC17:03
*** praneshp has joined #openstack-ironic17:04
lucasagomescalling it a day folks17:04
lucasagomeshave a great evening, talk to you all tomorrow!17:04
* mgould too: good night, everyone!17:04
*** mgould is now known as mgould|afk17:04
*** lucasagomes is now known as lucas-dinner17:05
*** vishwanathj has quit IRC17:06
*** Goneri has quit IRC17:11
*** tangchen has quit IRC17:14
*** tangchen has joined #openstack-ironic17:14
*** jcoufal_ has joined #openstack-ironic17:15
*** sacharya has joined #openstack-ironic17:15
*** baoli has joined #openstack-ironic17:15
*** vishwanathj has joined #openstack-ironic17:16
*** jcoufal has quit IRC17:18
*** ChubYann has joined #openstack-ironic17:19
*** vishwanathj has quit IRC17:20
NobodyCamnight all17:23
_vsaienkolucasagomes please review https://review.openstack.org/#/c/299318/17:28
vdroklucas-dinner, just so you get this notification, no rush :) ^^17:31
*** leifmadsen has left #openstack-ironic17:35
*** links has quit IRC17:36
*** Sukhdev has quit IRC17:38
*** Goneri has joined #openstack-ironic17:39
*** electrofelix has quit IRC17:39
openstackgerritStephanie Miller proposed openstack/bifrost: Unify testing scripts  https://review.openstack.org/29742417:42
openstackgerritStephanie Miller proposed openstack/bifrost: Make booleans in templates explicit  https://review.openstack.org/32115117:42
vdrokgood night!17:53
*** vdrok has quit IRC17:53
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: Remove insecure connection warnings  https://review.openstack.org/32387318:02
*** sambetts is now known as sambetts|afk18:04
*** moshele has joined #openstack-ironic18:05
openstackgerritSam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url  https://review.openstack.org/31614918:05
openstackgerritSam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions  https://review.openstack.org/31576618:05
*** trown|lunch is now known as trown18:05
jlvillalvdrok-afk, vsaienko Are you still around?18:08
openstackgerritAndre Aranha proposed openstack/python-oneviewclient: Remove insecure connection warnings  https://review.openstack.org/32387318:14
lucas-dinnervdrok-afk, sorry, was away :-) +118:18
openstackgerritMerged openstack/ironic-python-agent: Returns CPU flags in the CPU inventory  https://review.openstack.org/30829218:21
*** e0ne has joined #openstack-ironic18:26
vdrok-afkjlvillal: kind of, on the way home18:26
vdrok-afkMorning :)18:26
jlvillalvdrok-afk, I was wondering if you minded if I broke up the patch chain. And used depends-on instead.18:26
jlvillalSo that individual patches can land if they are approved.18:27
jrollyou can only have one depends-on per project in the commit msg18:27
jlvillaljroll, Not in my experience18:27
jrolljlvillal: hrm, vasyl pointed this out to me once, in the code even18:27
jrollCI just loops over them doing git checkout18:28
jrollI could be wrong, who knows18:28
jlvillaljroll, Hmmm. Before I had unpacked them all and did depends-on. And grenade was passing.18:28
jlvillalSo that is why I thought it worked.18:28
jrollhm, not sure18:29
jrollanyway, if something is blocking another, let's just get it landed :)18:29
vdrok-afkjlvillal: if it works, I don't mind :)18:29
jlvillal:)18:29
*** PollyZ_ has joined #openstack-ironic18:30
jlvillaljroll, I think you saw all that. Sounds like it can use Depends-On18:34
jrollyeah18:34
jrollweird :)18:34
* jlvillal will go work on unpacking the patch chain18:35
jlvillalvdrok-afk, ^^^18:35
NobodyCammorning jlvillal18:35
*** mbound has joined #openstack-ironic18:35
jlvillalNobodyCam, Good morning :)18:35
NobodyCam:)18:35
openstackgerritJohn L. Villalovos proposed openstack/ironic: Make sure create_ovs_taps creates unique taps  https://review.openstack.org/31910118:37
*** e0ne has quit IRC18:37
openstackgerritJohn L. Villalovos proposed openstack/ironic: Update resources subnet CIDR  https://review.openstack.org/31708218:38
openstackgerritJohn L. Villalovos proposed openstack/ironic: Disable disk_config compute-feature-enabled in tempest  https://review.openstack.org/32198218:39
*** mbound has quit IRC18:40
*** krtaylor has quit IRC18:41
*** e0ne has joined #openstack-ironic18:44
*** milan has joined #openstack-ironic18:46
*** ifarkas has quit IRC18:50
openstackgerritJohn L. Villalovos proposed openstack/ironic: Patch to test Grenade.  https://review.openstack.org/31110918:50
openstackgerritJohn L. Villalovos proposed openstack/ironic: Patch to test Grenade.  https://review.openstack.org/31110918:52
*** [1]cdearborn has joined #openstack-ironic18:53
*** rama_y has joined #openstack-ironic18:56
*** ChubYann has quit IRC18:56
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/32389019:01
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/32389119:01
*** cdearborn has quit IRC19:04
*** krtaylor has joined #openstack-ironic19:06
*** ChubYann has joined #openstack-ironic19:09
*** mbound has joined #openstack-ironic19:12
*** Sukhdev has joined #openstack-ironic19:30
*** moshele has quit IRC19:31
*** e0ne has quit IRC19:35
*** nicodemos is now known as nico-afk19:39
*** e0ne has joined #openstack-ironic19:40
*** praneshp has quit IRC19:41
*** e0ne has quit IRC19:43
*** e0ne has joined #openstack-ironic19:48
mat128JayF, sambetts|afk: back from appointment, but uploading in gate is a no-no, gate can fail and will be retriggered, how do you manage this? :)19:57
mat128we have a similar setup downstream but we rely exclusively on binaries and it makes for interesting cases19:57
JayFmat128: that's why I suggested he email and ask infra for new ideas :)19:57
mat128yeah19:57
*** Sukhdev has quit IRC19:59
*** SiRiuS__ has joined #openstack-ironic20:01
*** david-lyle has joined #openstack-ironic20:05
*** sabeen has quit IRC20:06
*** _vsaienko has quit IRC20:08
*** _vs has quit IRC20:08
*** sabeen has joined #openstack-ironic20:08
*** e0ne has quit IRC20:09
*** SiRiuS__ has quit IRC20:10
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Bugfix+removing invalid tests+functional at gate  https://review.openstack.org/32408820:11
openstackgerritGabriel Bezerra proposed openstack/python-oneviewclient: Bugfix+removing invalid tests+functional at gate  https://review.openstack.org/32408820:14
*** lucas-dinner has quit IRC20:19
openstackgerritJay Faulkner proposed openstack/ironic-specs: Update metrics spec to reflect review comments  https://review.openstack.org/32115320:19
jlvillaljroll, rloo devananda NobodyCam TheJulia: https://review.openstack.org/#/c/319101/20:21
jlvillalIt has two +2 already. I modified the code so I don't want to +W20:22
jlvillalIt is Grenade related.20:22
jrollquite the 'review me' ping :P20:22
jlvillal:)20:22
* jroll +W20:23
jlvillalThanks!20:23
mat128jlvillal: who sets local ovs_tap=ovs-${tag_id}-tap1 to anything other than -tap1 ?20:23
mat128jlvillal: tag_id, my bad :)20:24
jlvillalmat128, Yep :)20:24
* mat128 silently goes away :)20:24
mat128haha20:24
mat128totally missed it20:24
mat128looked like the constant was simply extracted20:24
*** milan has quit IRC20:24
*** lucasagomes has joined #openstack-ironic20:25
*** milan has joined #openstack-ironic20:27
JayFmat128: So tomorrow or Friday, wanna make an appointment and pair up on a first stab at the policy spec?20:28
mat128both are pretty busy for me but friday sounds better20:29
*** praneshp has joined #openstack-ironic20:29
*** lucasagomes has quit IRC20:29
JayFFriday my calendar is clear; and I usually work from 8a-5p PST so name your time and I can send out an invitation20:30
mat128I'm busy after 10a PST, so 8a to 10a PST works for me20:32
JayFLets put it on the calendar for 9a-10a, that way I'll be well awake and we're timeboxed a bit better20:33
mat128If you want an event in your calendar; mat128@gmail.com20:33
mat128good20:33
JayF^ Anyone interested in policy is welcome to join, just ask and i'll add you to an invite20:33
* jroll steps away for a bit20:34
rama_yHi JayF: is this meeting related to Keystone policy support?20:34
JayFYes; but it's not a meeting, it's a working session20:34
JayFto be explicit :)20:34
rama_yJayF: I am interested in joining20:35
*** fragatin_ has joined #openstack-ironic20:35
*** fragatin_ has quit IRC20:35
JayFif you can give me an email I'll gladly add you to the invite.20:35
*** fragatina has joined #openstack-ironic20:36
rama_yJayF: ramamani.yeleswarapu@intel.com20:36
JayFrama_y: are you working with/for OSIC?20:36
*** lucasagomes has joined #openstack-ironic20:36
rama_yJayF: in the same broader group20:37
JayFrama_y: ah, I'm working on OSIC for Rackspace, hence why I'm stirring up the policy stuff20:38
JayFrama_y: given you were working on that + config centralization I thought there might be a connection :)20:38
rama_yJayF: thanks!20:40
rama_yJayF: thanks for the invite.20:40
JayFno problem; I never object to help :)20:40
*** ijw_ has joined #openstack-ironic20:44
NobodyCamjlvillal: will take a look in a bit20:46
*** bharath has joined #openstack-ironic20:47
jlvillalNobodyCam, It has been reviewed already. Thanks though :)20:48
*** ijw_ has quit IRC20:49
*** sdake has quit IRC20:49
*** david-lyle has quit IRC20:51
NobodyCamlol :p20:51
*** ayoung has quit IRC20:55
*** bharath has quit IRC20:56
openstackgerritMerged openstack/python-oneviewclient: Bugfix+removing invalid tests+functional at gate  https://review.openstack.org/32408820:57
*** Sukhdev has joined #openstack-ironic20:58
*** PollyZ_ has quit IRC20:58
*** PollyZ_ has joined #openstack-ironic20:59
*** david-lyle has joined #openstack-ironic21:01
*** trown is now known as trown|outtypewww21:01
openstackgerritMerged openstack/ironic: Make sure create_ovs_taps creates unique taps  https://review.openstack.org/31910121:04
*** baoli has quit IRC21:04
*** xhku has joined #openstack-ironic21:09
*** _vsaienko has joined #openstack-ironic21:10
*** _vs has joined #openstack-ironic21:10
*** xhku has quit IRC21:11
*** xhku_ has quit IRC21:11
_vsaienkojlvillal good morning :)21:11
*** xhku has joined #openstack-ironic21:12
_vsaienkojlvillal do you know if multiple depends-on to the same project works?21:12
jroll_vsaienko: it does, confirmed by infra folks21:13
*** Goneri has quit IRC21:13
_vsaienkojroll what if patches can't be merged automatically?21:13
*** jjohnson2 has quit IRC21:13
jrollTIL zuul v3 gives us in-repo job config, that's goign to be amazing \o/21:13
jroll_vsaienko: I would assume not21:14
_vsaienkojroll that is really nice21:14
JayFjroll: speaking of job config, would love your reivew on https://review.openstack.org/#/c/313800/ https://review.openstack.org/#/c/323994/21:15
*** xhku has quit IRC21:16
jrollJayF: reviewing21:17
*** xhku has joined #openstack-ironic21:17
jrollJayF: -1 on the big one, just one thing I think21:19
JayFjroll: the answer is no21:19
jrollorly?21:19
JayFjroll: the default is 1024, which works, and would only be updated by a devstack commit to that stable branch21:19
JayFjroll: so why make the job more complex?21:19
jrolloh default is 1024?21:20
*** xhku has quit IRC21:20
jrollno, https://github.com/openstack-dev/devstack/blob/stable/liberty/lib/ironic#L7821:20
*** xhku has joined #openstack-ironic21:20
JayFah I checked master21:20
jroll:)21:21
JayFgive me 2m and you can revote21:21
JayFthat is an easy fix21:21
*** xhku has quit IRC21:22
jrollyep, standing by :)21:22
*** xhku has joined #openstack-ironic21:22
*** mbound has quit IRC21:22
JayFjroll: k, pushed new patchset for https://review.openstack.org/#/c/313800/21:24
*** xhku has quit IRC21:26
*** _vsaienko has quit IRC21:26
*** _vs has quit IRC21:26
jrollJayF: +121:27
*** xhku has joined #openstack-ironic21:27
* jroll out for the night, see y'all tomorrow21:30
NobodyCamnight jroll21:30
thiagopnight jroll21:30
thiagopI'm going too21:30
thiagopsee ya21:30
*** thiagop has quit IRC21:31
JayFo/21:31
TheJuliagoodnight21:31
*** xhku has quit IRC21:33
*** xhku has joined #openstack-ironic21:34
openstackgerritMerged openstack/ironic: Disable disk_config compute-feature-enabled in tempest  https://review.openstack.org/32198221:40
NobodyCamJayF: quick question. Have you seen issues with passing multiple steps in manual cleaning IE: http://paste.openstack.org/show/2kegZtWIsMSajZQhK1Fe/21:41
JayFI have never used manual cleaning21:42
JayFbut I'll gladly take a look :)21:42
JayFdo you not have to pass in priority for manual cleaning?21:42
JayFI guess not, after reading the docs21:43
JayFNobodyCam: I presume that should work, looking at it.21:43
NobodyCamJayF: ya that blows up: Invalid clean_steps: Additional properties are not allowed ('priority' was unexpected)21:43
JayFpriority was unexpected?21:44
JayFthat has to be a bug21:44
JayFoh21:44
JayFNobodyCam: what error do you get with *your pasted request body*21:44
NobodyCamit appeared to be running create before the delete21:45
JayFstrange; but it should be reproducable in a unit test case I think21:45
JayFif you wanna prove the bad behavior21:45
NobodyCamstill testing here21:45
JayFI can assure you this bug doesn't exist in automated cleaning21:46
JayFheh21:46
TheJulia:)21:47
*** _vs has joined #openstack-ironic21:48
*** _vsaienko has joined #openstack-ironic21:48
*** jcoufal_ has quit IRC21:48
*** lindycoder has quit IRC21:49
JayFoh rloo21:49
JayFre: the mysterious pxe_libvirt experimental job21:49
rlooJayF: uh?21:49
JayFthat driver is apparently in ironic-staging-drivers21:49
rlooJayF: no way!21:50
JayFit was removed from project-config recently, it got rebased out of my project-config change21:50
rlooJayF: mystery solved :)21:50
NobodyCamIf I pass single steps it works: 2016-06-01 21:48:54.532 19996 INFO ironic.conductor.manager [req-ab71cad1-3dfd-45fd-8c21-19ba48b0f82a - - - - -] Node b2235c35-15d0-4781-9a60-609588651d51 cleaning complete21:50
*** sdake has joined #openstack-ironic21:50
JayFhttps://github.com/openstack/ironic-staging-drivers/blob/master/ironic_staging_drivers/libvirt/__init__.py21:50
rlooJayF: I was actually trying to remember the name of that git repo. Will add it to our wiki.21:52
*** dprince has quit IRC21:52
*** Sukhdev has quit IRC21:57
*** sdake_ has joined #openstack-ironic22:01
*** david-lyle has quit IRC22:01
*** sdake has quit IRC22:03
*** wajdi_ has joined #openstack-ironic22:04
*** mtanino has quit IRC22:06
*** wajdi has quit IRC22:07
mrdaMorning Ironic22:07
*** wajdi_ has quit IRC22:08
NobodyCammorning mrda22:09
*** PollyZ_ has quit IRC22:13
*** PollyZ has joined #openstack-ironic22:15
*** david-lyle has joined #openstack-ironic22:16
*** _vsaienko has quit IRC22:18
*** _vs has quit IRC22:18
*** [1]cdearborn has quit IRC22:19
*** _vsaienko has joined #openstack-ironic22:21
*** _vs has joined #openstack-ironic22:21
*** sdake_ has quit IRC22:21
*** mbound has joined #openstack-ironic22:23
*** mbound has quit IRC22:28
*** ametts has quit IRC22:30
*** sdake has joined #openstack-ironic22:31
*** _vs has quit IRC22:33
*** _vsaienko has quit IRC22:34
*** _vsaienko has joined #openstack-ironic22:35
mrdao/22:35
*** mtanino has joined #openstack-ironic22:35
*** _vs has joined #openstack-ironic22:35
*** Sukhdev has joined #openstack-ironic22:37
*** intr1nsic has quit IRC22:43
*** intr1nsic has joined #openstack-ironic22:45
*** sacharya has quit IRC22:45
*** sabeen has quit IRC22:46
*** rpioso has quit IRC22:48
*** praneshp has quit IRC22:51
openstackgerritJohn L. Villalovos proposed openstack/ironic: WIP: Patch to test Grenade.  https://review.openstack.org/31110922:55
oomichihi, can anyone take a look at a tempest patch https://review.openstack.org/#/c/322656/ for ironic test?22:58
oomichihelps from ironic team would be great :)22:58
*** Goneri has joined #openstack-ironic23:00
*** Goneri has quit IRC23:05
*** _vs has quit IRC23:12
*** _vsaienko has quit IRC23:12
*** praneshp has joined #openstack-ironic23:13
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/32389123:16
*** PollyZ has quit IRC23:17
*** amotoki has quit IRC23:38
*** mtanino has quit IRC23:49
*** phschwartz has quit IRC23:52
*** sdake_ has joined #openstack-ironic23:55
openstackgerritStephanie Miller proposed openstack/bifrost: Config drive spec uses network_data.json  https://review.openstack.org/31586223:56
*** phschwartz has joined #openstack-ironic23:56
*** sdake has quit IRC23:58
*** david-lyle has quit IRC23:59

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