Thursday, 2016-07-28

*** hoangcx has quit IRC00:01
*** mbound has quit IRC00:04
*** harshs has quit IRC00:10
openstackgerritMerged openstack/python-ironicclient: Fix coverage target  https://review.openstack.org/34804300:14
*** lujinluo has joined #openstack-ironic00:16
*** wajdi has joined #openstack-ironic00:17
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Add metrics support to IPA  https://review.openstack.org/11998100:23
*** aNupoisc has quit IRC00:31
*** bigjools has quit IRC00:43
*** rama_y has quit IRC00:43
*** bigjools has joined #openstack-ironic00:44
*** hoangcx2 has quit IRC00:50
*** mtanino has joined #openstack-ironic00:55
bodepdis this the right channel for glean questions?00:56
bodepdor is it infra?00:56
*** jjohnson2 has joined #openstack-ironic00:59
*** adu has quit IRC00:59
*** hoangcx has joined #openstack-ironic01:02
*** mbound has joined #openstack-ironic01:03
*** chlong has joined #openstack-ironic01:06
*** mbound has quit IRC01:08
*** adu has joined #openstack-ironic01:09
*** Sukhdev has quit IRC01:18
*** harshs has joined #openstack-ironic01:21
*** ahqa has quit IRC01:22
*** harshs has quit IRC01:25
*** harshs has joined #openstack-ironic01:27
*** jjohnson2 has quit IRC01:37
*** joprovost has joined #openstack-ironic01:49
*** joprovost1 has joined #openstack-ironic02:01
*** joprovost has quit IRC02:03
*** joprovost1 is now known as joprovost02:03
*** ChrisAusten has joined #openstack-ironic02:05
*** baoli has quit IRC02:06
*** amotoki has quit IRC02:10
jrollbodepd: probably infra is best02:12
jrollJayF: I think we should do it02:12
jrollmorgabra: all I really have to say right now is 'yikes', and probably +1 for hitting the ML02:12
*** thrash is now known as thrash|g0ne02:25
*** ayoung has quit IRC02:25
*** adu has quit IRC02:26
*** ayoung has joined #openstack-ironic02:27
*** amotoki has joined #openstack-ironic02:32
*** ayoung has quit IRC02:32
*** spandhe has joined #openstack-ironic02:35
*** amotoki has quit IRC02:36
*** ayoung has joined #openstack-ironic02:36
*** amotoki has joined #openstack-ironic02:40
*** yuanying has quit IRC02:47
*** jrist has quit IRC02:55
*** rloo has quit IRC02:58
*** amotoki has quit IRC03:01
*** adu has joined #openstack-ironic03:01
*** amotoki has joined #openstack-ironic03:10
*** vishwanathj has joined #openstack-ironic03:11
*** ayoung has quit IRC03:14
*** xavierr-afk has quit IRC03:15
*** Nisha has joined #openstack-ironic03:15
*** sdake has joined #openstack-ironic03:15
*** xavierr-afk has joined #openstack-ironic03:15
*** Nisha is now known as Guest5466403:15
*** sabeen has quit IRC03:20
*** sabeen has joined #openstack-ironic03:22
*** lujinluo has quit IRC03:25
*** sdake_ has joined #openstack-ironic03:25
*** sdake has quit IRC03:27
openstackgerritZhenguo Niu proposed openstack/ironic-inspector: Combine multiple warning logs into one in create_ports  https://review.openstack.org/34657803:35
*** mtanino has quit IRC03:39
*** bvandewa has quit IRC03:44
*** sdake_ has quit IRC03:46
*** sdake has joined #openstack-ironic03:46
*** yuanying has joined #openstack-ironic03:50
*** xavierr-afk has quit IRC03:57
*** xavierr-afk has joined #openstack-ironic03:57
*** harshs has quit IRC03:58
*** lujinluo has joined #openstack-ironic03:58
*** links has joined #openstack-ironic04:03
*** harshs has joined #openstack-ironic04:12
openstackgerritShivanand Tendulker proposed openstack/ironic-lib: Support configdrive in iscsi deploy for whole disk images  https://review.openstack.org/23092404:14
openstackgerritNisha Agarwal proposed openstack/ironic: adding back the inspect_ports as an optional parameter  https://review.openstack.org/32523004:15
*** Sukhdev has joined #openstack-ironic04:15
openstackgerritAparna proposed openstack/proliantutils: Add hpssa support only in raid mode  https://review.openstack.org/34797804:19
*** spandhe has quit IRC04:20
*** spandhe_ has joined #openstack-ironic04:20
*** harshs has quit IRC04:28
*** harlowja_at_home has joined #openstack-ironic04:30
*** adu has quit IRC04:32
*** deray has joined #openstack-ironic04:39
*** harshs has joined #openstack-ironic04:48
*** Sukhdev has quit IRC04:58
*** vmud213 has joined #openstack-ironic05:00
derayg'morning Ironicers!05:01
*** jrist has joined #openstack-ironic05:03
*** ChubYann has quit IRC05:15
*** harshs has quit IRC05:23
*** harshs has joined #openstack-ironic05:28
*** bks has joined #openstack-ironic05:31
*** lujinluo has quit IRC05:36
*** lujinluo has joined #openstack-ironic05:36
*** ChrisAusten has quit IRC05:39
*** joprovost has quit IRC05:45
*** joprovost has joined #openstack-ironic05:45
*** amotoki_ has joined #openstack-ironic05:48
*** joprovost has quit IRC05:50
*** amotoki has quit IRC05:51
*** rama_y has joined #openstack-ironic06:00
openstackgerritAparna proposed openstack/proliantutils: Add hpssa support only in raid mode  https://review.openstack.org/34797806:02
*** spandhe_ has quit IRC06:07
*** spandhe has joined #openstack-ironic06:07
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844006:10
*** harlowja_at_home has quit IRC06:16
*** pcaruana has joined #openstack-ironic06:17
*** mjura has joined #openstack-ironic06:23
*** mjura has quit IRC06:32
*** mjura has joined #openstack-ironic06:32
*** rbartal has joined #openstack-ironic06:36
*** tesseract- has joined #openstack-ironic06:42
*** sdake has quit IRC06:42
*** moshele has joined #openstack-ironic06:42
*** rama_y has quit IRC06:45
*** spandhe has quit IRC06:48
*** vishwanathj has quit IRC06:49
*** jtomasek has joined #openstack-ironic07:02
*** Fdaisuke_ has joined #openstack-ironic07:05
*** hoangcx has quit IRC07:05
openstackgerritAparna proposed openstack/proliantutils: Raise exception when HPSSA controller not available  https://review.openstack.org/34779107:07
*** Fdaisuke has quit IRC07:07
*** hoangcx has joined #openstack-ironic07:08
*** hshiina has joined #openstack-ironic07:15
*** ifarkas_away is now known as ifarkas07:15
*** pece has joined #openstack-ironic07:29
*** hoangcx has quit IRC07:34
*** Guest54664 has quit IRC07:39
*** bvandewa has joined #openstack-ironic07:39
*** bvandewa has quit IRC07:40
*** chlong has quit IRC07:47
*** jpich has joined #openstack-ironic07:56
*** bvandewa has joined #openstack-ironic07:56
*** zzzeek has quit IRC08:00
*** hoangcx has joined #openstack-ironic08:00
*** zzzeek has joined #openstack-ironic08:00
*** e0ne has joined #openstack-ironic08:01
*** mgoddard has joined #openstack-ironic08:06
*** _Fdaisuke_ has joined #openstack-ironic08:09
*** Fdaisuke_ has quit IRC08:10
*** hparekh has quit IRC08:11
*** Romanenko_K has joined #openstack-ironic08:13
*** derekh has joined #openstack-ironic08:14
*** stendulker has joined #openstack-ironic08:16
*** lujinluo has quit IRC08:19
*** lucas-dinner is now known as lucasagomes08:21
*** lujinluo has joined #openstack-ironic08:22
*** hparekh has joined #openstack-ironic08:24
*** edand has joined #openstack-ironic08:25
*** harshs has quit IRC08:27
*** yibo has quit IRC08:34
*** yibo has joined #openstack-ironic08:35
*** yibo has quit IRC08:35
*** yibo has joined #openstack-ironic08:36
openstackgerritShivanand Tendulker proposed openstack/ironic: Support configdrive in iscsi deploy for whole disk images  https://review.openstack.org/22511508:37
*** bvandewa has quit IRC08:39
openstackgerritAparna proposed openstack/proliantutils: Raise exception when HPSSA controller not available  https://review.openstack.org/34779108:40
*** sambetts|afk is now known as sambetts08:46
sambettsJayF: I'm happy with making the tinyipa 256 job the default if it works08:47
openstackgerritShivanand Tendulker proposed openstack/ironic-python-agent: Use ironic-lib to create configdrive  https://review.openstack.org/29646609:03
*** daemontool has joined #openstack-ironic09:03
openstackgerritDavid Lenwell proposed openstack/ironic: Add REST API for volume connector operation  https://review.openstack.org/21458609:10
*** e0ne has quit IRC09:17
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add erase_devices_metadata cleaning step  https://review.openstack.org/34416409:17
*** lujinluo has quit IRC09:20
*** hshiina has quit IRC09:22
*** electrofelix has joined #openstack-ironic09:26
openstackgerritMerged openstack/ironic: Add api-ref for new port fields  https://review.openstack.org/32529909:32
*** lujinluo has joined #openstack-ironic09:32
*** e0ne has joined #openstack-ironic09:32
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI  https://review.openstack.org/34819109:33
stendulkerlucasagomes: Hi09:36
lucasagomesstendulker, hi there09:36
stendulkerlucasagomes: regarding review https://review.openstack.org/#/c/344164/09:36
stendulkerlucasagomes: You are throwing exception upon encountering first error on any device09:37
stendulkerlucasagomes: Wont it be better to complete operation for all devices and then throw an exception listing all the devices09:38
stendulkerthis would help user to fix things if any, instead getting incremetal errors on subsequent  run of this clean step09:39
*** electrofelix has quit IRC09:39
lucasagomesstendulker, could be... I will update the patch09:39
stendulkerok. just wanted to check if there is any other rational, will log comment accordingly09:40
lucasagomescool thanks a lot for the review09:40
stendulkertahnk you.09:41
stendulkerthank you :)09:41
*** sabeen has quit IRC09:41
*** fumie has quit IRC09:45
*** lujinluo has quit IRC09:50
*** mgould|afk is now known as mgould09:54
mgouldmorning Ironic!09:54
*** mbound has joined #openstack-ironic09:56
*** lujinluo has joined #openstack-ironic09:57
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Add CRUD notifications to Ironic  https://review.openstack.org/34724210:00
*** electrofelix has joined #openstack-ironic10:05
*** Guest54664 has joined #openstack-ironic10:06
*** hoangcx has quit IRC10:08
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844010:10
vmud213morning mgould10:10
*** yibo has quit IRC10:14
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add erase_devices_metadata cleaning step  https://review.openstack.org/34416410:14
*** hwcomcn has joined #openstack-ironic10:21
*** pionar has joined #openstack-ironic10:22
openstackgerritMerged openstack/python-ironicclient: Add baremetal port show command to OSC plugin  https://review.openstack.org/34473210:30
*** mbound has quit IRC10:32
*** deray_ has joined #openstack-ironic10:32
*** hshiina has joined #openstack-ironic10:35
*** deray has quit IRC10:35
*** milan has quit IRC10:54
*** milan has joined #openstack-ironic10:55
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options  https://review.openstack.org/29731210:56
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions  https://review.openstack.org/23698210:56
*** deray_ has quit IRC11:05
*** deray_ has joined #openstack-ironic11:05
lucasagomesstendulker, re https://review.openstack.org/#/c/230924/11:06
openstackgerritZhenguo Niu proposed openstack/ironic-inspector: Combine multiple warning logs into one in create_ports  https://review.openstack.org/34657811:06
lucasagomesstendulker, how was that releasenote generated? ironic-lib do not have support for it yet11:06
stendulkeroh, i use reno command directly11:07
stendulkernever used ironic version11:07
stendulkerwill remove it11:07
*** deray_ has quit IRC11:08
*** deray_ has joined #openstack-ironic11:08
Romanenko_KThanks for review and merge of "baremetal port show command to OSC plugin"! :)11:08
stendulkerlucasagomes: it was quiet a while.  it was generated when reno was first introduced :(11:08
*** deray_ has quit IRC11:08
lucasagomesstendulker, right, because there's no config for that in ironic-lib11:09
stendulkerlucasagomes: yes11:09
lucasagomesso I bet that release note will conflict with something when we add support for it11:09
stendulkeryes, will clean it up right away11:09
lucasagomesstendulker, perhaps remove it for now and we can add later when reno is avaialble?11:09
stendulkerlucasagomes: thank you for the review11:09
stendulkerlucasagomes: sure thing11:09
lucasagomesstendulker, I'm preparing a patch for it at the moment11:09
stendulkerlucasagomes: ok, that would be useful11:10
TheJuliaGood morning11:10
sambettso/ TheJulia11:11
openstackgerritShivanand Tendulker proposed openstack/ironic-lib: Support configdrive in iscsi deploy for whole disk images  https://review.openstack.org/23092411:11
lucasagomesstendulker, +2;d11:13
lucasagomesTheJulia, morning11:13
stendulkerlucasagomes: thank you :)11:13
*** stendulker has quit IRC11:20
mgouldTheJulia: morning!11:21
openstackgerritAparna proposed openstack/proliantutils: Add a clean step 'hardware_disk_erase'  https://review.openstack.org/33614811:23
jrollmorning y'all11:34
jrolllucasagomes: thanks for catching the resource_class length thing, updating shortly11:34
lucasagomesjroll, morning, no problem :-)11:35
*** vmud213 has quit IRC11:40
*** ccamacho is now known as ccamacho|lunch11:41
*** hshiina has quit IRC11:44
*** hshiina has joined #openstack-ironic11:45
derekhIs there a irc channel for pyghmi related stuff?11:46
TheJulia /win 811:47
TheJuliadoh11:47
openstackgerritJim Rollenhagen proposed openstack/ironic: Add node.resource_class field  https://review.openstack.org/34505511:49
jroll^ with tests :D11:49
jrollderekh: here is best, the main author doesn't seem to be here now, but ask away and we'll try to help11:50
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Add support for node.resource_class  https://review.openstack.org/34508011:54
*** lucasagomes is now known as lucas-hungry11:54
*** chlong has joined #openstack-ironic11:55
derekhjroll: thanks, I'm looking for a release if possible https://bugs.launchpad.net/tripleo/+bug/159929911:55
openstackLaunchpad bug 1599299 in tripleo "openstackbmc still failing with concatenation errors" [High,Triaged]11:55
jrollderekh: ah yeah, that's in our governance, I can roll one now11:56
derekhjroll: great, thanks11:56
*** hoonetorg has quit IRC11:56
jroll... or is it11:57
jrollmaybe not11:57
* jroll does it anyway maybe?11:57
jrollderekh: meh, I need devananda or jarrod to release it, gonna be a bit11:58
derekhjroll: ok11:58
*** lujinluo has quit IRC12:00
*** amotoki_ has quit IRC12:06
*** xavierr-afk is now known as xavierr12:09
openstackgerritMerged openstack/python-ironicclient: Add CLI tests to check JSON response body  https://review.openstack.org/29123912:11
*** mtanino has joined #openstack-ironic12:12
*** amotoki has joined #openstack-ironic12:13
*** thrash|g0ne is now known as thrash12:14
*** MattMan has quit IRC12:17
*** MattMan has joined #openstack-ironic12:17
*** pionar has quit IRC12:21
*** fellypefca has joined #openstack-ironic12:22
*** edand has quit IRC12:24
jrollholding off on the policy stuff for a recheck, multitenant job has never passed on it, quite curious12:26
jroll(devstack errors)12:26
*** hoonetorg has joined #openstack-ironic12:27
*** hwcomcn has quit IRC12:31
*** amotoki has quit IRC12:34
*** amotoki has joined #openstack-ironic12:39
*** links has quit IRC12:40
*** trown|outtypewww is now known as trown12:43
*** amotoki has quit IRC12:47
*** Guest54664 has quit IRC12:48
*** mbound has joined #openstack-ironic12:49
*** thiagop has joined #openstack-ironic12:50
thiagopGood morning, Ironicers12:50
TheJuliagood morning12:51
fellypefcathiagop good morning :)12:51
thiagopTheJulia fellypefca o/12:52
*** baoli has joined #openstack-ironic12:53
*** baoli_ has joined #openstack-ironic12:54
*** ccamacho|lunch is now known as ccamacho12:55
*** baoli has quit IRC12:58
*** hoangcx has joined #openstack-ironic13:02
*** lucas-hungry is now known as lucasagomes13:03
*** ppiela has joined #openstack-ironic13:07
*** rloo has joined #openstack-ironic13:08
*** amotoki has joined #openstack-ironic13:12
openstackgerritMerged openstack/ironic-lib: Support configdrive in iscsi deploy for whole disk images  https://review.openstack.org/23092413:17
*** bks has quit IRC13:19
jrollrloo: mind a re-review on https://review.openstack.org/#/c/345055/ for something lucas found?13:19
jrolllucasagomes: mind reviewing the client patch to go with it? https://review.openstack.org/#/c/345080/13:19
rloojroll: looking...13:19
lucasagomesjroll, not at all, will do in a min13:20
jrollthanks13:20
*** jcoufal has joined #openstack-ironic13:21
*** daemontool has quit IRC13:21
rloojroll: +2'd. I'm going to wait a few hours in case any one else wants to look at it. i or someone else will +A it today I think :)13:24
jrollrloo: thanks :)13:25
jrollrloo: my goal is to get the client release submitted early-ish today so that release team can approve it13:25
jrollthey don't tend to do friday releases13:25
rloojroll: oh, right. monday is probably fine too. nova folks don't really need it this week?13:26
thiagopjroll rloo lucasagomes if you have some review slots to spend on https://review.openstack.org/#/c/286192/, we'd appreciate13:26
jrollrloo: the earlier the better, but monday could be fine13:26
* thiagop don't want it to be pushed back on the last minutes again :/13:27
jrollthiagop: honest question, oneview patches often have lots of co-authors, did 6 people really write that code?13:27
lucasagomesthiagop, will do13:27
thiagopjroll: yup. We were a team of six. Some people put the hands on the code at some point in time, others give maintenance later...13:28
jrollthiagop: interesting13:28
jrollI guess that's 200+ lines each, seems reasonable13:29
jrolljust always curious :)13:29
thiagopjroll: you know, these stuff of the oneview driver hangs a lot before being merged. Maybe because it's a lot of code in a single patch, but... that's it13:29
thiagopjroll: this one is hanging since february... :/13:29
jrollthiagop: well, it's usually because we have other priorities, unfortunately13:29
thiagopjroll: I know that. That's why I'm always looking for breaches on your review agenda :)13:30
jrollhehe13:30
*** wajdi has quit IRC13:34
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Add reno support for release notes management  https://review.openstack.org/34831813:35
lucasagomesshould be a quick review ^ adding reno support to ironic-lib13:35
*** sdake has joined #openstack-ironic13:36
jrolllucasagomes: nice, I was going to file a bug for that today :)13:36
lucasagomesjroll, btw the patch to oslo.utils adding > and < operators to the spec_matcher is already merged13:37
jrollwe'll also need a project-config update13:37
jrolloh nice13:37
lucasagomesjroll, yeah I'm putting a patch up for the project-config now13:37
lucasagomesI just noted the reno support was missing due: https://review.openstack.org/#/c/230924/22/releasenotes/notes/config-drive-support-for-whole-disk-images-in-iscsi-deploy-10d74d74dcca807f.yaml13:37
jrollawesome, thanks man13:37
jrollright, saw that in scrollback13:37
*** ayoung has joined #openstack-ironic13:43
*** jcoufal has quit IRC13:46
*** jcoufal has joined #openstack-ironic13:47
rloojroll, lucasagomes: wrt ironic-lib & reno. I submitted a patch for that awhile ago, and in discussion with someone, we decided it wasn't needed.13:49
rloojroll, lucasagomes: might have been dmitry. I think I asked oslo folks, and they don't use reno.13:49
sambettsI guess if we continue to say that ironic-lib isn't for use outside of Ironic projects then there is noone that needs informing about changes13:50
*** PollyZ has joined #openstack-ironic13:51
rloojroll, lucasagomes: here you go: https://review.openstack.org/#/c/335205/13:51
rloosambetts: yeah, dmitry's point was that it wasn't a user-facing library13:51
lucasagomesrloo, oh hmm it's a good point13:52
*** thiagop is now known as thiagop-afk13:52
lucasagomesrloo, tho even for us it would be easier to know which version includes what by looking at the release notes13:52
lucasagomesthan commits in the code13:52
lucasagomesI don't think it does any harm, not sure... let's see what others think13:53
rloolucasagomes: only harm is extra work for folks i guess. i am not against it cuz obviously i submitted a patch for it, but i also don't see the use of adding extra work given everything that we have to do. clearly, dtantsur|pto was against it :)13:55
lucasagomesrloo, true, I've added a comment to the patch to see what other reviewers think13:59
lucasagomesthanks for pointing it out13:59
rloolucasagomes: thx!14:00
rloolucasagomes: after thinking about it for 10 seconds more, maybe we shouldn't cuz of what sambetts reminded me. the reno's will be available via openstack's release notes url. but it isn't even like oslo's library. it isn't for consumption outside of ironic project.14:01
lucasagomesrloo, hmm true :-/14:02
jrollrloo: good points14:02
lucasagomesyeah let's abandon that patch14:02
jrollthat said reno makes it super easy for me to figure out what version a release needs to be14:02
jrollI'm fine without it though14:02
lucasagomesjroll, yeah I find it easier to figure out what version includes/removes what as well14:03
sambettsperhaps we'll just add a note into the release note that uses the new funtionality in Ironic/IPA?14:03
rloojroll, lucasagomes: i agree. how do the oslo folks deal with it? maybe they have fancy scripts or warped minds (just kidding)14:03
rloosambetts: yes14:04
jrollrloo: they release weekly so that it's not many commits and easy to figure out :P14:04
rloojroll: we can release weekly too :D14:04
*** jvgrant has quit IRC14:04
jrollindeed :)14:04
*** bvandewa has joined #openstack-ironic14:05
*** causten has joined #openstack-ironic14:06
*** thrash is now known as thrash|biab14:07
lucasagomesabandoned14:07
*** jvgrant has joined #openstack-ironic14:08
*** wajdi has joined #openstack-ironic14:08
*** edand has joined #openstack-ironic14:09
*** wajdi_ has joined #openstack-ironic14:11
*** wajdi has quit IRC14:13
*** hshiina has quit IRC14:26
*** wajdi_ has quit IRC14:29
*** jistr is now known as jistr|call14:35
*** thrash|biab is now known as thrash14:36
openstackgerritVladyslav Drok proposed openstack/ironic: Clean up provision ports when reattempting deploy  https://review.openstack.org/34839214:37
*** sabeen has joined #openstack-ironic14:37
*** wajdi has joined #openstack-ironic14:39
*** clenimar has joined #openstack-ironic14:42
*** spandhe has joined #openstack-ironic14:43
*** spandhe_ has joined #openstack-ironic14:45
vdrokmorning all!14:47
*** ijw has joined #openstack-ironic14:47
vdrokanother super easy change that would be good to go into the client release - https://review.openstack.org/34155214:47
*** spandhe has quit IRC14:47
*** spandhe_ is now known as spandhe14:47
*** ijw has quit IRC14:47
*** ijw has joined #openstack-ironic14:47
abashmakHi folks, would someone kindly review this: https://review.openstack.org/#/c/346832/14:48
*** jistr|call is now known as jistr14:49
mgouldvdrok: LGTM14:49
*** rama_y has joined #openstack-ironic14:50
vdrokthx mgould14:51
*** ijw has quit IRC14:52
*** sabeen1 has joined #openstack-ironic14:55
*** sabeen has quit IRC14:57
*** ifarkas is now known as ifarkas_afk14:58
*** causten has quit IRC14:59
*** causten has joined #openstack-ironic14:59
*** rama_y has quit IRC15:01
*** harshs has joined #openstack-ironic15:01
*** mjura has quit IRC15:01
*** rama_y has joined #openstack-ironic15:01
*** causten has quit IRC15:03
*** joprovost has joined #openstack-ironic15:05
openstackgerritYuiko Takada Mori proposed openstack/ironic: Add node serial console installation howto  https://review.openstack.org/29387215:05
*** PollyZ has quit IRC15:06
*** ccamacho is now known as ccamacho|afk15:08
*** jmiu_ is now known as jmiu15:09
*** mgoddard has quit IRC15:09
*** mgoddard_ has joined #openstack-ironic15:09
NobodyCamGood Morning Ironicers15:13
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on  https://review.openstack.org/33000315:15
TheJuliavdrok: re, 341552, is that in parity with osc?15:17
*** harshs has quit IRC15:17
*** openstackgerrit has quit IRC15:18
*** ijw has joined #openstack-ironic15:18
*** openstackgerrit has joined #openstack-ironic15:18
thiagop-afkmorning NobodyCam15:19
NobodyCammorning thiagop-afk15:19
*** yuanying_ has joined #openstack-ironic15:20
*** pcaruana has quit IRC15:22
*** yuanying has quit IRC15:22
*** ijw has quit IRC15:23
*** bvandewa has quit IRC15:27
*** Guest54664 has joined #openstack-ironic15:28
*** PollyZ has joined #openstack-ironic15:30
*** edand has quit IRC15:30
*** ijw has joined #openstack-ironic15:34
openstackgerritMoshe Levi proposed openstack/ironic: Adding InfiniBand Support  https://review.openstack.org/26426315:36
*** harshs has joined #openstack-ironic15:37
devanandagood morning, all.15:38
*** ijw has quit IRC15:39
*** bvandewa has joined #openstack-ironic15:39
*** bvandewa has quit IRC15:39
*** bvandewa has joined #openstack-ironic15:39
*** jistr is now known as jistr|afk15:42
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425715:44
*** e0ne has quit IRC15:45
*** e0ne has joined #openstack-ironic15:46
vdrokTheJulia: iirc there is no port show yet in osc plugin, but will check now15:46
jrollmorning devananda15:48
vdrokTheJulia: ah, there is now15:48
rloovdrok: i think the port show just merged this morning15:48
vdrokTheJulia: but it seems it just outputs whatever is returned by port.py15:48
jrolldevananda: unless jarrod pops up, can you release pyghmi today? it's on the old tagging system and only you two have access. derekh needs it to fix a tripleo bug15:48
vdrokTheJulia: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/osc/v1/baremetal_port.py#L12815:49
vdrokrloo: yup, morning :)15:49
devanandajroll: been at least a year since I did - let me see if I still rememer how15:50
*** ijw has joined #openstack-ironic15:50
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port unset command to OSC plugin  https://review.openstack.org/34740215:50
*** PollyZ has quit IRC15:52
*** mtanino has quit IRC15:54
TheJuliavdrok: thank you :)15:55
*** rloo has quit IRC15:55
*** comstud has quit IRC15:55
*** ijw has quit IRC15:55
*** rloo has joined #openstack-ironic15:55
vdrokyeah, just confirmed locally, np15:55
jrollthis nova patch lgtm if someone else wants to look https://review.openstack.org/#/c/34125315:56
jrollJayF: lucasagomes: I know you two are interested in that interaction (cleaning up failed deploys)15:57
lucasagomesjroll, looking15:57
*** causten has joined #openstack-ironic15:58
*** pece has quit IRC15:58
lucasagomesinteresting...15:59
* lucasagomes really wanted to have a /deploy endpoint to pass all the info for a deployment at once15:59
jrollyeah15:59
jrollI brought that up at nova midcycle15:59
jrollthey want to parallelize some things, so like plug_vifs and any volume stuff would still need to be separate to do those at the same time before deploying16:00
rlooJayF: did that gate troubleshooting session happen on July 13? I missed it.16:02
JayFNo it didn't :X16:02
JayFman, I failboated a lot, that never got on my calendar16:02
rlooJayF: ha ha. I forgot to put it on my calendar too!16:03
rlooJayF: maybe we should try this in Sept or October :)16:03
*** PollyZ has joined #openstack-ironic16:06
*** harshs has quit IRC16:07
*** e0ne has quit IRC16:07
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Dont review  https://review.openstack.org/34845716:08
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Log full config only once in conductor  https://review.openstack.org/34845916:10
cineramarloo: i have a question for you about the volume connection work, if you have time for it16:11
rloocinerama: oh. ugh. you can ask. just in the middle of two things and not sure i can swap my brain to somethign else but can try.16:13
*** harshs has joined #openstack-ironic16:13
cineramarloo: yikes, i know that feel16:14
cineramarloo: anyway this is pretty simple: currently, the RPC API version and the REST API version get an increment for both volume connector and volume target...we were wondering whether it should stay that way, or whether they can both be introduced in the same version16:15
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Dont review  https://review.openstack.org/34845716:17
rloocinerama: same version of ironic? like ironic release?16:17
cineramarloo: same API version16:18
*** rama_y has quit IRC16:18
rloocinerama: the RPC version is diff from the API version.16:18
rloocinerama: if the RPC version is bumped, it doesn't mean the API version is bumped.16:19
*** baoli_ has quit IRC16:19
rloocinerama: i haven't had a chance to look at that volume patch. should i take a quick look?16:21
*** adu has joined #openstack-ironic16:21
*** ijw has joined #openstack-ironic16:21
cineramarloo: that'd be great. all the work is currently under bug/1526231, the first patch in the series is https://review.openstack.org/#/c/200983/16:21
*** mbound has quit IRC16:22
rloocinerama: ok, in a bit. going to grab something to eat first.16:22
cineramarloo: the REST API patches and the client patches are still pretty WIP because there are some differences with the spec we need to address16:22
*** mbound has joined #openstack-ironic16:23
cineramarloo: do eat (i need to after this call i'm on). and thanks for looking16:23
*** ijw has quit IRC16:26
*** Romanenko_K has quit IRC16:28
*** moshele has quit IRC16:29
*** jistr|afk is now known as jistr16:30
abashmakvdrok: thanks for the review, I will address your comments16:31
*** tesseract- has quit IRC16:32
vdrokabashmak: thank you too :)16:32
*** derekh has quit IRC16:39
NobodyCammorning rloo vdrok abashmak cinerama jroll JayF lucasagomes :)16:42
*** jpich has quit IRC16:42
*** aNupoisc has joined #openstack-ironic16:42
abashmakGood morning NobodyCam16:42
JayFo/16:42
jrollhey NobodyCam16:42
NobodyCam:)16:42
lucasagomesNobodyCam, hi there16:43
NobodyCamxavierr: question: when is support for NON-dynamic allocation planed for obsolescence16:43
*** mtanino has joined #openstack-ironic16:45
*** gfhellma has joined #openstack-ironic16:45
vdrokmorning! :)16:45
*** aNupoisc has quit IRC16:46
*** linuxgeek has quit IRC16:46
mgouldmorning NobodyCam16:48
NobodyCammorning mogul :)16:48
NobodyCamgah16:48
lucasagomesfolks, I'm calling it a day16:48
NobodyCammgould: even16:48
NobodyCamlol16:48
lucasagomeshave a great evening all16:48
NobodyCamhave a good night lucasagomes16:48
*** PollyZ has quit IRC16:48
mgouldgood night lucasagomes16:50
mgouldNobodyCam: one of these days you'll spell my name right :-)16:50
openstackgerritMerged openstack/ironic: Add node.resource_class field  https://review.openstack.org/34505516:50
NobodyCamI just have the new laptop "learn" how to do just that :p16:51
mgouldheh16:51
jmiutab completion is a beautiful thing16:51
NobodyCamjmiu: thats what I use but the new mac keeps auto correcting the names :p16:52
jmiuNobodyCam: can you turn that off for specifically irc?16:54
mgouldNobodyCam: https://twitter.com/qntm/status/755460779319685120/photo/116:54
*** amotoki has quit IRC16:54
mgouldhttps://twitter.com/qntm/status/755460779319685120, rather16:55
* sambetts -> home16:56
*** sambetts is now known as sambetts|afk16:56
NobodyCammgould: :)16:56
sambetts|afknight all16:56
NobodyCamnight sambetts|afk16:56
NobodyCamjmiu: that would turn off auto correct safari which would be bad (at least for me) :p16:57
mgouldnight sambetts|afk16:59
*** amotoki has joined #openstack-ironic17:00
*** causten has joined #openstack-ironic17:01
*** PollyZ has joined #openstack-ironic17:01
*** amotoki has quit IRC17:03
*** e0ne has joined #openstack-ironic17:04
*** causten_ has joined #openstack-ironic17:05
*** causten has quit IRC17:05
jmiuNobodyCam: oh so no specifically off for irc17:06
NobodyCamnope17:07
*** loki_ has joined #openstack-ironic17:07
*** trown is now known as trown|lunch17:08
JayFdevananda: is it possible the oslo policy generator is generating YAML17:08
JayFdevananda: instead of JSON?17:08
JayFdevananda: I know YAML is now supported by oslo policy iirc17:08
*** causten has joined #openstack-ironic17:08
*** causten_ has quit IRC17:10
devanandaJayF: does YAML allow items to be separated only by a newline ?17:10
*** Kelso has joined #openstack-ironic17:10
JayFyes17:11
JayFhttp://www.yaml.org/spec/1.2/spec.html#id275996317:11
JayFunless I'm misreading that17:11
*** causten has quit IRC17:12
devanandaok - yea, it's valid YAML17:12
*** causten has joined #openstack-ironic17:12
devanandalacks the opening --- but it seems to pass parse tests for YAML ...17:12
JayFSo perhaps that's what it's doing then?17:13
devanandaperhaps. then we shouldn't call the file *.json17:13
JayF++17:14
*** jcoufal_ has joined #openstack-ironic17:14
devanandaugh. parking expired - bbiab17:14
*** causten has quit IRC17:17
*** jcoufal has quit IRC17:17
*** fragatina has joined #openstack-ironic17:17
*** fragatina has quit IRC17:19
*** mgould is now known as mgould|afk17:20
vdrokgood night everyone!17:20
*** PollyZ has quit IRC17:21
NobodyCamnight vdrok17:21
*** baoli has joined #openstack-ironic17:22
*** rbartal has quit IRC17:25
*** causten has joined #openstack-ironic17:25
jrolloh I'm totally okay with yaml17:26
jrollbecause comments make invalid json \o/17:26
*** fragatina has joined #openstack-ironic17:26
*** fragatina has quit IRC17:27
*** rbartal has joined #openstack-ironic17:27
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Dont review  https://review.openstack.org/34845717:27
*** rama_y has joined #openstack-ironic17:27
*** rbartal has quit IRC17:27
*** rama_y has quit IRC17:28
cineramado we not use ramdisk-image-create for DIB IPA builds anymore? our docs seem to suggest not17:30
cineramadib folks are wondering about it as we are looking at retooling the special casing for the IPA build not creating an image17:33
*** causten has quit IRC17:33
NobodyCamthiagop-afk: wow great stuff with the dynamic allocation stuff. I did leave a question on the review.17:34
*** causten has joined #openstack-ironic17:34
*** causten has quit IRC17:38
*** adu has quit IRC17:38
*** fragatina has joined #openstack-ironic17:39
*** fragatina has quit IRC17:40
*** fragatina has joined #openstack-ironic17:41
*** Kelso has quit IRC17:44
*** Kelso has joined #openstack-ironic17:45
*** fragatina has quit IRC17:46
thiagop-afkNobodyCam: seeing now17:48
NobodyCam:)17:48
*** Kelso has quit IRC17:50
*** maurosr has quit IRC17:52
*** thiagop-afk is now known as thiagop17:53
thiagopNobodyCam: I think that'll depend on the state machine. When cleaning is disabled, the node remains in deployfail or does it move the machine back to available?17:53
thiagopNobodyCam: I have not considered that, indeed..17:54
NobodyCamI think its direct to available which could to confusion17:56
NobodyCams/could/could lead /17:56
thiagopNobodyCam: If it goes to available, the first periodic will take care of it17:57
thiagopNobodyCam: if it reamains in deployfail, we'll need to add a new filter there...17:57
NobodyCamcan you dig in to that17:57
NobodyCamdig / look17:58
* thiagop looks17:59
*** causten has joined #openstack-ironic18:00
NobodyCam:) thou even if the first periodic task would catch the node. it could be (at max) 2xperiodic_check_interval?18:02
NobodyCamthats ten minutes in available when in fact its taken?18:03
thiagopNobodyCam: it depends on the configured time. If an operator expects that these kind of stuff is frequent, he should use a shorter time18:04
thiagopNobodyCam: and if the instance is booted, it'll fail validation18:04
*** causten has quit IRC18:04
*** causten has joined #openstack-ironic18:05
NobodyCamis that documented (or in the process of being documented)?18:05
thiagopNobodyCam: more like "in the process": https://review.openstack.org/#/c/305441/18:07
*** Sukhdev has joined #openstack-ironic18:07
*** trown|lunch is now known as trown18:07
NobodyCamthiagop: Nice!!!! :)18:08
*** causten has quit IRC18:10
*** Sukhdev has quit IRC18:10
*** PollyZ has joined #openstack-ironic18:13
hamzyI have question about IPA hardware managers such as https://github.com/jayofdoom/ipa-example-hardware-managers . I have been looking at diskimage-builder and it isn't obvious to me how I would install a git repo of a python package into the image being built. IIUC the setup.cfg of an installed repo inside of the image would then be picked up by IPA.18:15
JayFI'm not super familiar with DIB, I'd imagine you'd have to do a custom element?18:15
*** Kelso has joined #openstack-ironic18:16
JayFI know there's a prolientutils element, and they used to ship a hardware manager18:16
JayFTheJulia: cinerama ^ can you help hamzy?18:16
hamzyI don't think it would be under package-installs http://docs.openstack.org/developer/diskimage-builder/elements/package-installs/README.html18:16
TheJuliahamzy: looking for an example18:16
*** Sukhdev has joined #openstack-ironic18:17
rajinirsambetts: thiagop: have a question on the thirdparty CI, are you running it with Devstack vm on Devstack?18:17
thiagoprajinir: devstack-gate18:17
rajinirsambetts: thiagop: devstack-gate on the Devstack VM ?18:17
thiagopthiagop: https://github.com/openstack-infra/devstack-gate18:18
thiagoprajinir: ^18:18
TheJuliahamzy: I think this is the magic that will answer your question: https://github.com/openstack/diskimage-builder/blob/master/elements/proliant-tools/install.d/65-proliant-tools-install#L40 and https://github.com/openstack/proliantutils/blob/master/setup.cfg#L2518:19
cineramaJayF: unless i'm missing something that element just installs hpssacli rpm & proliantutils from pip now18:19
JayFYeah that's what I'd imagine18:19
rajinirsambetts: thiagop: I see. We are using devstack-vm-gate, now I see why we are seeing networking issues18:19
JayFall you'd have to do is install the python package in18:19
rajinirsambetts: thiagop: Thanks , let me switch and see if that solves the issues18:19
cineramaJayF: oh hey while you're here do you remember why we stopped using ramdisk-image-create vs disk-image-create for the IPA element?18:20
thiagoprajinir: it really depends on your script and underlying infra. Never used devstack-vm-gate but... maybe :)18:20
JayFcinerama: no idea whatsoever. I have run a dIB IPA image exactly once ever.18:20
cineramaJayF: relevant change here - https://review.openstack.org/#/c/334042/18:20
cineramaheh18:20
*** loki_ has quit IRC18:21
rajinirthiagop: Confused. We are running devstack-gate/devstack-vm-gate-wrap.sh from  https://github.com/openstack-infra/devstack-gate18:21
thiagoprajinir: so you're using devstack-gate18:22
hamzycinerama,  TheJulia, hrm... I want to install an arbitrary git repo of my own manager18:22
rajinirthiagop:yes18:22
TheJuliahamzy: oh18:22
JayFhamzy: you can pip install from a git repository18:22
TheJuliaahh18:22
TheJuliahamzy: what jayF said, and there is a way to do it in dib specificaly18:22
thiagoprajinir: if you describe what's happening, maybe I can give some ideas in how to debug18:22
rajinirthiagop: The dhcp packets are not routed to the tap. This is our main issue18:22
hamzyyeah I was looking for a dib specific way since I am using dib to create the IPA boot image18:22
TheJuliahamzy: example of pulling the package down via git https://github.com/openstack/diskimage-builder/blob/master/elements/simple-init/source-repository-simple-init18:23
TheJuliaor the repo, and then you'd have to install it in your dib element18:23
thiagoprajinir: you can see them arriving at the physical port with tcpdump?18:23
*** causten has joined #openstack-ironic18:23
thiagopcan you**18:23
rajinirthiagop: Would love too. We are running out of ideas and thinnking of abandoning devstak-gate. Any pointer will help18:23
*** causten_ has joined #openstack-ironic18:24
*** ayoung has quit IRC18:24
hamzyTheJulia, thanks... I will try messing around with it18:25
rajinirthiagop: yes, tcpdump shows the packets forwarded until the last step. Hardware Ironic Nodes -> Devstack Host(physicalinterface)->Devstack Slave VM (phrical interface) -> Tap18:25
devanandaJayF: after looking into changing policy.json to policy.yaml, I'm disinclined to do that. oslo things still default to callig the file policy.json in many places18:25
TheJuliahamzy: enjoy!18:25
JayFdevananda: so, call it policy.json even though it's yaml? And maybe add a patch to the policy generator to put a comment at the top saying it's yaml18:26
JayFdevananda: not shipping it is also fine with me18:26
rajinirthiagop: The devstack slave VM networking is what I see is not setup right18:26
*** causten_ has quit IRC18:26
thiagoprajinir: you may need to hack into devstack-gate's (or some of devstack's) options to make it right18:26
devanandaJayF: wdyt about: add tox target, include policy.json.sample, update docs and devstack to indicate that it's not needed18:27
thiagoprajinir: let-me paste you some of the configs I'm using here...18:27
*** causten_ has joined #openstack-ironic18:27
JayFdevananda: +1, and as a bonus we could patch the oslo generator to add a comment at the top saying it's yaml (or perhaps file a bug indicating the confusion and leave it to oslo folks to sort)18:28
*** causten has quit IRC18:28
thiagoprajinir: http://paste.openstack.org/show/543419/18:28
TheJuliadevananda: JayF you know... its not much code to determine if it is yaml or json and load it *ducks*18:28
*** baoli has quit IRC18:29
jrolldevananda: I think the generator can output json18:29
jrolldevananda: I have a present for you, but first am curious if you got around to tagging pyghmi?18:30
rajinirthiagop: Ah, we have something like this http://paste.openstack.org/show/543420/18:30
*** spandhe has quit IRC18:30
devanandajroll: nothing in the --help indicates that it can generate json18:30
jrollreally? :/18:30
devanandajroll: and, no, I haven't.18:30
devanandaI did look at it, but then needed to move my car18:30
jrollok18:30
devanandaI think I just tag -s 1.0.8 && git push 1.0.818:31
rajinirthiagop: looks similar, but when the bridge happens the ip add is del and moved to the ovs bridge, node goes offline18:31
*** causten_ has quit IRC18:31
jrolldevananda: that sounds right18:31
thiagoprajinir: ugh, you are using a single interface VM to run devstack-gate?18:32
rajinirthiagop: yes, is that the issue18:32
rajinirthiagop: Was just thinking about it this morning to add another interface18:32
thiagoprajinir: here we use two different networks.18:32
*** electrofelix has quit IRC18:33
rajinirthiagop: should we give it a try with two different networks?18:33
thiagoprajinir: one interface on the cloud's private network, and another to the physical network that is where the physical blades are18:33
*** vishwanathj has joined #openstack-ironic18:34
rajinirthiagop: and the one you are bridging in the devstack-gate is the physical?18:34
*** catintheroof has joined #openstack-ironic18:34
thiagoprajinir: I think so. If public_interface is the connection jenkins(?!) is using to connect to the node via ssh, it'll break18:34
thiagoprajinir: yup. The other one is just used to ssh commands to devstack18:35
jrolldevananda: anyway, the present: https://review.openstack.org/#/c/348443/18:35
jrollit totally works18:35
jrollthere's one todo in there18:35
rajinirthiagop: ok, will check with my labmanager and add another network and give it a try18:36
thiagoprajinir: good luck18:36
*** fellypefca has quit IRC18:36
rajinirthiagop: Before I go off, did you customize the nodepool to create the vm to add two networks?18:37
thiagoprajinir: yes18:37
rajinirthiagop: Any doc on how to?18:37
rajinirthiagop: Or do we have to hack the nodepool code18:37
thiagoprajinir: not handy, but I can paste it too18:38
rajinirthiagop: cool, that would be very helpful. Thanks a bunch18:38
thiagoprajinir: http://paste.openstack.org/show/543422/18:39
thiagoprajinir: first provider connects only on the private net, second one connects on private AND ironic's18:39
*** rama_y has joined #openstack-ironic18:40
devanandajroll: very interesting. that todo might be a bit of work...18:40
*** gfhellma has quit IRC18:41
jrolldevananda: shouldn't be bad, though maybe a bit db heavy, and this is the worst place to put db heavy things :(18:41
*** ChubYann has joined #openstack-ironic18:41
rajinirthiagop: question, what is the first-provider used for? do you have another job that uses it?18:42
thiagoprajinir: yes. We use it for a power&management only job with fake_oneview. It tests a bunch of operations without real hardware18:43
devanandajroll: agreed. I'll take a look at that with my perf hat on, when you add it18:44
jrolldevananda: I figure it can't be worse than writing compute_nodes records for every node :)18:45
rajinirthiagop: Cool, gives me some clarity and hope. Thank you. Will keep you posted if this works18:45
thiagoprajinir: ;)18:45
*** maurosr has joined #openstack-ironic18:48
devanandajroll: I have pushed a 1.0.8 tag18:48
jrolldevananda: thank you18:49
jrolldevananda: I wonder if you need to do stuff to pypi too18:51
*** gfhellma has joined #openstack-ironic18:51
*** baoli has joined #openstack-ironic18:54
*** ElCoyote_ has quit IRC18:54
*** ElCoyote_ has joined #openstack-ironic18:54
openstackgerritDevananda van der Veen proposed openstack/ironic: Add keystone policy support to Ironic  https://review.openstack.org/32559918:57
*** causten has joined #openstack-ironic18:59
thiagopNobodyCam: yeah Chris, it's what I expected. The driver will go into "deployfail", then nova calls "delete" and the instance is deleted, going thru teard_down to available19:00
NobodyCamso in standalone mode (i.e.. with out nova) we're left in deploy fail?19:01
thiagopjroll devananda , can you  confirm this ^? (we are talking about what happens to the node if it fails deploy and cleaning is disabled)19:01
jrollthat sounds correct19:01
thiagopNobodyCam: yep. So far...19:02
jrollthiagop: in fact, the same thing happens if cleaning is enabled19:02
jrollcleaning won't happen until delete is called19:02
thiagopWould that^ be a problem to bifrost TheJulia?19:02
*** rbudden has joined #openstack-ironic19:02
devanandaTheJulia, NobodyCam, jroll: addressed all of your comments on the policy patch. thanks for the review!19:02
NobodyCam:)19:03
devanandathiagop: that sounds like what I expect -- node is left in deployfail until user initiates something else19:03
devanandathiagop: which could be "delete" or "rebuild", for example19:04
rloocinerama: wrt your question about REST api and RPC apis in the volume connector work, the way it is coded seems fine.19:04
rloocinerama: although i did wonder if you could combine the two REST API patches (volume connector operation & volume target operation) into one patch.19:04
thiagopdevananda jroll gotcha, thanks19:04
jrolldevananda: multitenant job still hasn't passed on that, I'm curious to see if it does this round. couldn't tell if the previous failures were "real", the most recent one was not19:04
rloocinerama: cuz is it useful to use one w/o the other?19:04
jrolldevananda: thanks for fixing though :D19:04
cineramarloo: we were kind of pondering that19:05
cineramarloo: (combining those patches)19:05
rloocinerama: i didn't look at the code but if they are fairly simple, it is fine to combine them. but that's my opinion.19:06
cineramarloo: thanks for the advice. will be good to have your (& others') full review at some point19:07
rloocinerama: at some point. soooo many patches (i guess that is a good thing) and not enough people reviewing :-(19:08
*** jcoufal_ has quit IRC19:13
devanandajroll: multitenant job has been failing on all patches for a bit ... hrmmm19:13
*** harshs has quit IRC19:13
devanandalooks like http://logs.openstack.org/65/321865/14/check/gate-tempest-dsvm-ironic-multitenant-network-nv/a0d8ae6/ was the first run it started failing on19:14
* devananda investigates briefly19:14
jrollah you know what, I bet it got caught in that big refactor19:14
rloojroll: you're going to push out a client release soon?19:16
jrollrloo: yes, thanks for reminder, was waiting for the merge19:16
devananda2016-07-27 01:09:18.814 | + functions-common:git_clone:533           :   echo 'The /opt/stack/new/ironic project was not found; if this is a gate job, add'19:16
devanandathat's the first time I've seen that error19:17
jrolldevananda: seems like we're missing a PROJECTS=openstack/ironic thing19:17
jrollbut, the other jobs don't have it explicit19:17
jrollso idk19:17
devanandayea...19:17
*** Sukhdev has quit IRC19:17
jrolllooks like it includes devstack-virtual-ironic, which does have it19:18
*** jcoufal has joined #openstack-ironic19:18
jrollrloo: zuul didn't pick up the dependent patch merging, I added a +A to kick it19:23
rloojroll: good thing you like to kick :)19:23
* jroll has a strong foot19:24
openstackgerritAkilan Pughazhendi proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros  https://review.openstack.org/34689419:28
devanandajroll: it looks like dsg-setup-workspace isn't including ironic in the builds that are failing19:28
devanandahttp://logs.openstack.org/92/348392/1/check/gate-tempest-dsvm-ironic-multitenant-network-nv/386fc05/logs/devstack-gate-setup-workspace-new.txt.gz19:28
jrollhrm, firefox broke ctrl-f19:28
jrollO_o19:28
devanandaI need to run - delivering soup to a sick friend. bbiab19:29
openstackgerritAkilan Pughazhendi proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros  https://review.openstack.org/34689419:29
*** Kelso has quit IRC19:32
*** Kelso has joined #openstack-ironic19:45
*** jcoufal has quit IRC19:49
* rloo wonders if devananda *made* the soup...19:49
*** jcoufal has joined #openstack-ironic19:50
*** ccamacho has joined #openstack-ironic19:51
openstackgerritHugo Nicodemos proposed openstack/python-oneviewclient: Workaround to OneView pagination  https://review.openstack.org/34853220:00
*** gfhellma1 has joined #openstack-ironic20:01
*** gfhellma has quit IRC20:04
*** baoli has quit IRC20:04
openstackgerritMario Villaplana proposed openstack/ironic: Add power state change notifications  https://review.openstack.org/32186520:07
openstackgerritMario Villaplana proposed openstack/ironic: Add notification base classes and docs  https://review.openstack.org/29846120:07
mariojvfixed the releasenotes test failure above20:07
*** gfhellma1 has quit IRC20:07
*** mbound has quit IRC20:08
*** Kelso has quit IRC20:14
*** adu has joined #openstack-ironic20:14
*** e0ne has quit IRC20:18
openstackgerritMerged openstack/python-ironicclient: Add support for node.resource_class  https://review.openstack.org/34508020:25
*** greghaynes has quit IRC20:25
*** fragatina has joined #openstack-ironic20:27
NobodyCamTheJulia: https://bugs.launchpad.net/ironic/+bug/160752720:28
openstackLaunchpad bug 1607527 in Ironic "ssh power driver can not use a key with a passphrase" [Undecided,New]20:28
*** greghaynes has joined #openstack-ironic20:37
thiagopNobodyCam: answered in the review just to keep record20:39
*** ccamacho has quit IRC20:39
*** ccamacho|afk has quit IRC20:39
NobodyCamthiagop: awesome thank you20:40
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Raise Not Authorized Exception when connection request is forbidden  https://review.openstack.org/34355020:43
*** jralph has joined #openstack-ironic20:45
JayFNobodyCam: I suspect that bug should likely end up wontfix since that driver is going out of tree this cycle20:47
NobodyCam:( they are still going to live in the staging repo. figured I should file it20:48
*** Kelso has joined #openstack-ironic20:48
*** jralph_ has quit IRC20:48
*** Kelso has quit IRC20:49
*** gmmaha has left #openstack-ironic20:57
*** trown is now known as trown|outtypewww20:57
*** gmmaha has joined #openstack-ironic20:58
*** causten has quit IRC21:00
*** causten has joined #openstack-ironic21:01
*** causten_ has joined #openstack-ironic21:02
jrollrloo: dims_ is going to process that client release today21:02
*** causten has quit IRC21:05
rloojroll: sweet!21:06
*** causten_ has quit IRC21:07
*** mtanino has quit IRC21:08
mrdaMorning Ironic21:08
*** causten has joined #openstack-ironic21:08
thiagopmorning mrda21:08
*** mbound has joined #openstack-ironic21:09
thiagopCalling it a day21:09
thiagopgood night folks21:09
*** thiagop has quit IRC21:09
mrdao/21:10
*** causten has quit IRC21:12
*** mbound has quit IRC21:14
*** causten has joined #openstack-ironic21:24
*** wajdi has quit IRC21:24
NobodyCammorning mrda21:28
mrdahey NobodyCam21:29
NobodyCam:)21:30
*** causten has quit IRC21:31
*** causten has joined #openstack-ironic21:31
*** PollyZ has quit IRC21:32
*** causten has quit IRC21:33
*** causten has joined #openstack-ironic21:34
*** baoli has joined #openstack-ironic21:34
*** causten has quit IRC21:39
*** causten has joined #openstack-ironic21:39
*** adu has quit IRC21:46
*** causten has quit IRC21:53
*** harshs has joined #openstack-ironic21:53
openstackgerritChris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP  https://review.openstack.org/34394721:55
*** harshs has quit IRC21:57
*** harshs has joined #openstack-ironic21:58
*** jaybeale has joined #openstack-ironic22:03
*** PollyZ has joined #openstack-ironic22:04
*** jaybeale has quit IRC22:15
openstackgerritDevananda van der Veen proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros  https://review.openstack.org/34689422:21
*** harshs has quit IRC22:27
*** krtaylor has quit IRC22:31
*** harshs has joined #openstack-ironic22:46
*** harshs has quit IRC22:56
*** PollyZ has quit IRC22:56
openstackgerritChris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP  https://review.openstack.org/34394723:01
*** harshs has joined #openstack-ironic23:02
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe]  https://review.openstack.org/30961223:05
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iscsi]  https://review.openstack.org/30961523:16
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe]  https://review.openstack.org/30961223:16
openstackgerritRamamani Yeleswarapu proposed openstack/ironic: Centralize config options - [agent]  https://review.openstack.org/30960323:16
*** fumie has joined #openstack-ironic23:21
openstackgerritChris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP  https://review.openstack.org/34394723:24
*** rbudden has quit IRC23:40
*** hoangcx2 has joined #openstack-ironic23:46
*** hoangcx has quit IRC23:47

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