Monday, 2016-07-25

*** amotoki has joined #openstack-ironic00:00
*** baoli has quit IRC00:04
*** baoli has joined #openstack-ironic00:08
*** amotoki has quit IRC00:15
*** david-lyle_ has joined #openstack-ironic00:22
*** amotoki has joined #openstack-ironic00:24
*** david-lyle_ has quit IRC00:27
*** amotoki has quit IRC00:28
*** hoangcx has joined #openstack-ironic00:32
*** amotoki has joined #openstack-ironic00:33
*** mbound has joined #openstack-ironic00:47
*** mbound has quit IRC00:52
*** chlong has quit IRC00:54
*** spandhe has quit IRC00:56
*** yibo has joined #openstack-ironic00:58
*** vishwanathj has joined #openstack-ironic01:08
*** chlong has joined #openstack-ironic01:11
*** chlong is now known as chlong_POffice01:21
*** amotoki_ has joined #openstack-ironic01:25
*** amotoki has quit IRC01:28
openstackgerritTang Chen proposed openstack/ironic: Use assertDictEqual() to compare dicts in tests  https://review.openstack.org/34656101:35
openstackgerritTang Chen proposed openstack/python-ironicclient: Use assertDictEqual() to compare dicts in tests  https://review.openstack.org/34656301:52
*** amotoki has joined #openstack-ironic01:57
*** amotoki_ has quit IRC02:00
*** harshs has joined #openstack-ironic02:03
*** PollyZ has joined #openstack-ironic02:12
*** vishwanathj has quit IRC02:13
*** PollyZ has quit IRC02:14
*** tangchen_ has quit IRC02:18
*** harshs has quit IRC02:23
*** david-lyle_ has joined #openstack-ironic02:25
*** hoangcx has quit IRC02:25
*** david-lyle_ has quit IRC02:29
*** tangchen_ has joined #openstack-ironic02:32
*** hoangcx has joined #openstack-ironic02:32
*** harshs has joined #openstack-ironic02:34
*** hoangcx has quit IRC02:37
*** PollyZ has joined #openstack-ironic02:43
*** PollyZ has quit IRC02:48
*** PollyZ has joined #openstack-ironic02:49
*** yuanying has quit IRC02:52
*** PollyZ has quit IRC03:06
*** PollyZ has joined #openstack-ironic03:17
*** PollyZ has quit IRC03:20
*** PollyZ has joined #openstack-ironic03:22
*** harshs has quit IRC03:25
*** Nisha has joined #openstack-ironic03:29
*** Nisha is now known as Guest1885303:29
*** Guest18853 is now known as Nisha103:29
*** ijw_ has quit IRC03:31
*** ijw has joined #openstack-ironic03:32
*** PollyZ has quit IRC03:34
*** PollyZ has joined #openstack-ironic03:38
*** hoangcx has joined #openstack-ironic03:39
*** harshs has joined #openstack-ironic03:40
openstackgerritZhenguo Niu proposed openstack/ironic-inspector: Combine multiple warning logs into one in create_ports  https://review.openstack.org/34657803:41
*** hoangcx has quit IRC03:41
*** vishwanathj has joined #openstack-ironic03:43
*** hoangcx has joined #openstack-ironic03:45
*** appu103_ has joined #openstack-ironic03:47
*** yuanying has joined #openstack-ironic03:47
*** PollyZ has quit IRC03:50
*** harshs has quit IRC04:02
*** PollyZ has joined #openstack-ironic04:06
*** harshs has joined #openstack-ironic04:20
*** PollyZ has quit IRC04:21
*** PollyZ has joined #openstack-ironic04:24
*** david-lyle_ has joined #openstack-ironic04:26
*** david-lyle_ has quit IRC04:32
*** PollyZ has quit IRC04:38
*** PollyZ has joined #openstack-ironic04:46
*** appu103_ is now known as aparnav04:47
*** PollyZ has quit IRC04:50
*** PollyZ has joined #openstack-ironic04:51
*** spandhe has joined #openstack-ironic05:05
*** bvandewa has joined #openstack-ironic05:09
*** PollyZ has quit IRC05:09
*** harshs has quit IRC05:17
*** ChubYann has quit IRC05:22
*** spandhe has quit IRC05:38
*** ijw has quit IRC05:41
*** skramaja has joined #openstack-ironic05:54
*** skramaja has quit IRC05:55
*** mjura has joined #openstack-ironic05:59
*** spandhe has joined #openstack-ironic06:03
*** harshs has joined #openstack-ironic06:04
*** mjura has quit IRC06:05
*** rcernin has joined #openstack-ironic06:05
*** mjura has joined #openstack-ironic06:06
*** edand has joined #openstack-ironic06:07
*** skramaja has joined #openstack-ironic06:10
*** harshs has quit IRC06:11
openstackgerritTang Chen proposed openstack/ironic: Use assertEqual() instead of assertDictEqual()  https://review.openstack.org/34656106:15
*** harshs has joined #openstack-ironic06:22
*** spandhe has quit IRC06:24
*** spandhe has joined #openstack-ironic06:25
openstackgerritTang Chen proposed openstack/ironic: Use assertEqual() instead of assertDictEqual()  https://review.openstack.org/34656106:26
*** rbartal has joined #openstack-ironic06:28
*** david-lyle_ has joined #openstack-ironic06:29
*** david-lyle_ has quit IRC06:33
*** Nisha1 has quit IRC06:34
*** pcaruana has joined #openstack-ironic06:37
*** Nisha1 has joined #openstack-ironic06:38
*** tesseract- has joined #openstack-ironic06:41
*** milan has joined #openstack-ironic06:50
milan#pixiesay Good morning, Ironic! -m flexing06:51
milan??06:51
milanpixie down?06:52
milanGood morning, Ironic!06:52
pas-hamorning Ironic :)06:53
milanmorning lucasagomes, pixie may be on PTO06:53
milanmorning pas-ha06:53
*** spandhe has quit IRC06:54
*** jtomasek has joined #openstack-ironic06:55
*** bvandewa has quit IRC06:57
*** moshele has joined #openstack-ironic07:17
*** ccamacho has joined #openstack-ironic07:18
*** jtomasek has quit IRC07:18
*** jtomasek has joined #openstack-ironic07:19
openstackgerritYuiko Takada Mori proposed openstack/ironic: IPMITool: add IPMISocatConsole and IPMIConsole class  https://review.openstack.org/29387307:41
*** jpich has joined #openstack-ironic07:52
*** jtomasek has quit IRC07:58
*** jtomasek has joined #openstack-ironic07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** Sukhdev has quit IRC08:01
*** fragatina has joined #openstack-ironic08:02
*** jtomasek has quit IRC08:06
*** mgoddard has joined #openstack-ironic08:07
*** mbound has joined #openstack-ironic08:07
*** e0ne has joined #openstack-ironic08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c  https://review.openstack.org/31844008:10
*** fragatina has quit IRC08:10
*** moshele has quit IRC08:11
openstackgerritNisha Agarwal proposed openstack/proliantutils: Discover iscsi_boot and iscsi_iqn  https://review.openstack.org/34165308:22
*** e0ne has quit IRC08:22
*** vishwanathj has quit IRC08:23
*** moshele has joined #openstack-ironic08:23
*** lujinluo has joined #openstack-ironic08:28
*** e0ne has joined #openstack-ironic08:29
*** david-lyle_ has joined #openstack-ironic08:31
*** david-lyle_ has quit IRC08:36
*** lujinluo has quit IRC08:39
*** electrofelix has joined #openstack-ironic08:40
*** derekh has joined #openstack-ironic08:45
*** milan has quit IRC08:55
moshelelucasagomes,  can you please review the Infiniband support code https://review.openstack.org/#/c/264263/ and https://review.openstack.org/#/c/264257/ ?08:55
*** milan has joined #openstack-ironic08:59
*** e0ne has quit IRC09:00
*** e0ne has joined #openstack-ironic09:03
*** jtomasek has joined #openstack-ironic09:05
vdrokmorning ironic!09:06
*** Goneri has joined #openstack-ironic09:10
*** harshs has quit IRC09:15
*** milan has quit IRC09:16
*** sambetts has joined #openstack-ironic09:28
*** mgould|afk is now known as mgould09:28
mgouldmorning Ironic!09:28
sambettsMorning all09:28
mgouldmorning moshele vdrok sambetts pas-ha09:29
*** alex_xu has quit IRC09:31
*** david-lyle_ has joined #openstack-ironic09:32
sambettsHi mgould09:34
*** alex_xu has joined #openstack-ironic09:34
*** david-lyle_ has quit IRC09:36
*** fragatina has joined #openstack-ironic09:47
*** e0ne has quit IRC09:47
*** xiexs has joined #openstack-ironic09:48
*** hwcomcn has joined #openstack-ironic09:50
*** hwcomcn has quit IRC09:51
*** hwcomcn has joined #openstack-ironic09:52
*** fragatina has quit IRC09:54
*** stendulker has joined #openstack-ironic09:55
*** e0ne has joined #openstack-ironic09:55
*** hwcomcn has quit IRC09:55
*** hwcomcn has joined #openstack-ironic09:58
openstackgerritDebayan Ray proposed openstack/ironic: Update f/w versions after ``update_firmware``  https://review.openstack.org/34668309:59
moshelemoring: mgould09:59
*** hwcomcn has quit IRC10:03
*** hwcomcn has joined #openstack-ironic10:04
*** hwcomcn has quit IRC10:05
*** hwcomcn has joined #openstack-ironic10:06
*** hoangcx has quit IRC10:13
*** e0ne has quit IRC10:13
*** e0ne has joined #openstack-ironic10:17
yibohi lucasagomes, would you approve rfe of https://bugs.launchpad.net/ironic/+bug/1582964, thanks10:18
openstackLaunchpad bug 1582964 in Ironic "[RFE] Support multi arch deployments" [Wishlist,In progress] - Assigned to Yibo Cai (cyb70289)10:18
*** ijw has joined #openstack-ironic10:33
*** ijw has quit IRC10:37
openstackgerritJim Rollenhagen proposed openstack/ironic-specs: Add node.resource_class  https://review.openstack.org/34504010:37
jrollwheeee, morning everyone10:37
mgouldmorning jroll10:42
sambettsMorning jroll10:45
openstackgerritJim Rollenhagen proposed openstack/ironic: Add node.resource_class field  https://review.openstack.org/34505510:45
*** fragatina has joined #openstack-ironic10:45
sambettsawesome email btw, sounds like ironic + nova interactions are really going to improve with all these changes10:46
jrollthanks :)10:46
jrollyeah, I think we've got a really solid plan, hope that email explains it well for people without context10:46
sambettsmade sense to me :)10:47
sambettshow did the credit card swinging go for getting the FFE for the driver patches10:47
sambetts?10:48
*** vmud213 has joined #openstack-ironic10:48
Nisha1morning/afternoon/evening to all10:50
jrollsambetts: for networking? talked with folks a bit about it, I'm optimistic but nothing set in stone10:51
sambettsjroll: yeah, thats good news then10:53
*** rbartal has quit IRC10:53
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Add support for node.resource_class  https://review.openstack.org/34508011:05
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Add support for node.resource_class  https://review.openstack.org/34508011:05
*** bks has joined #openstack-ironic11:05
*** stendulker has quit IRC11:07
*** amotoki has quit IRC11:08
*** e0ne has quit IRC11:09
mgouldmorning Nisha111:10
jrollthese look fairly small and have been around for a year: https://review.openstack.org/#/c/201904/ https://review.openstack.org/#/c/338138/11:11
jroll(the second was split out of the first)11:11
openstackgerritSam Betts proposed openstack/ironic-inspector: Add GenericLocalLinkConnectionHook processing hook  https://review.openstack.org/32108211:16
Nisha1mgould, morning11:16
*** fragatina has quit IRC11:19
*** amotoki has joined #openstack-ironic11:23
*** david-lyle_ has joined #openstack-ironic11:34
*** david-lyle_ has quit IRC11:39
*** thrash has quit IRC11:47
*** amotoki has quit IRC11:50
openstackgerritImre Farkas proposed openstack/ironic: DRAC OOB inspection  https://review.openstack.org/34291511:54
jrollholy cow, microversion stuff makes adding a new field pretty painful :/11:54
*** sdake has joined #openstack-ironic11:54
*** skramaja has quit IRC11:55
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port list command to OSC plugin  https://review.openstack.org/34672211:55
openstackgerritZhenguo Niu proposed openstack/ironic-python-agent: Parallel erase disk devices  https://review.openstack.org/34672311:55
*** milan has joined #openstack-ironic11:59
sambettsjroll: yup...12:01
*** skramaja has joined #openstack-ironic12:01
*** skramaja has quit IRC12:02
*** e0ne has joined #openstack-ironic12:02
openstackgerritJim Rollenhagen proposed openstack/ironic: Add node.resource_class field  https://review.openstack.org/34505512:03
jroll 15 files changed, 303 insertions(+), 7 deletions(-)12:03
jrollO_O12:03
sambetts:'(12:04
* jroll now wants to figure out how to make this better12:04
*** skramaja has joined #openstack-ironic12:04
*** krtaylor has joined #openstack-ironic12:05
openstackgerritSam Betts proposed openstack/ironic-inspector: Add GenericLocalLinkConnectionHook processing hook  https://review.openstack.org/32108212:06
*** baoli has quit IRC12:07
sambettsjroll: only way is to revise how we do version negotiation so that we can stop carrying all the horrible hide this field, hide that field code12:08
jrollsambetts: yeah, something like nova's decorator?12:09
jrollhttp://docs.openstack.org/developer/nova/api_microversion_dev.html12:10
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Add new out-of-band properties to ilo drivers  https://review.openstack.org/33813812:10
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Deny chassis with too long description  https://review.openstack.org/34524812:10
sambettsNot seen it before, but something like that decorator would make the overall code nicer, but I'm thinking just in general the whole API interaction with the versions needs work12:12
*** milan has quit IRC12:12
*** bks has quit IRC12:13
jrollsambetts: I'm not too worried about the externals, just the amount of code it takes to do something12:13
jrollhere's an example of adding a field in nova's API https://github.com/openstack/nova/commit/e83e114e12:13
*** trown|outtypewww is now known as trown12:13
jrollmost of it seems like docs/schemas12:13
*** Nisha1 has quit IRC12:15
*** baoli has joined #openstack-ironic12:15
jrollalso, fyi, I saw this in our CI this weekend: https://bugs.launchpad.net/devstack/+bug/160599812:15
openstackLaunchpad bug 1605998 in devstack "Devstack is failing to install libraries that it has to download from git. These libraries are specified in local.conf in parameter 'LIBS_FROM_GIT'." [Undecided,In progress] - Assigned to Spyros Trigazis (strigazi)12:15
jrollseems more global, people are on it12:15
jrolljust a heads up if you get bit by that12:15
*** MattMan has quit IRC12:16
*** MattMan has joined #openstack-ironic12:16
*** d0ugal_ is now known as d0ugal12:17
*** d0ugal has quit IRC12:17
*** d0ugal has joined #openstack-ironic12:17
*** bks has joined #openstack-ironic12:18
sambettsjroll: yeah I think both need to be addressed, maybe v2 will help us out with the external part, but it would be nice to reduce the internal impact of such a change12:19
*** baoli has quit IRC12:20
jrollsambetts: I've always said I think our CLI should default to the latest it knows about, which I think will eliminate most of the grief12:21
*** thrash has joined #openstack-ironic12:22
*** thrash has quit IRC12:22
*** thrash has joined #openstack-ironic12:22
*** hoangcx has joined #openstack-ironic12:27
*** cfarquhar has joined #openstack-ironic12:28
*** cfarquhar has quit IRC12:28
*** cfarquhar has joined #openstack-ironic12:28
*** cfarquhar has quit IRC12:33
*** baoli has joined #openstack-ironic12:36
*** baoli_ has joined #openstack-ironic12:37
*** cfarquhar has joined #openstack-ironic12:38
*** cfarquhar has quit IRC12:38
*** cfarquhar has joined #openstack-ironic12:38
moshelesambetts: if you have time can you please review the infiniband patches https://review.openstack.org/#/c/264263/ and https://review.openstack.org/#/c/264257/12:38
*** baoli has quit IRC12:41
sambettsmoshele: sure I'll add them to my list to review12:45
*** chihhsin has quit IRC12:47
*** jjohnson2__ has joined #openstack-ironic12:47
*** chihhsin has joined #openstack-ironic12:47
moshelesambetts: thanks :)12:48
*** vmud213 has quit IRC12:48
*** fragatina has joined #openstack-ironic12:52
*** xavierr has joined #openstack-ironic12:54
*** rloo has joined #openstack-ironic12:56
*** nicodemos has joined #openstack-ironic12:59
*** joprovost has joined #openstack-ironic12:59
xavierrraise exception.GoodMorning()13:01
*** aparnav has quit IRC13:04
*** amotoki has joined #openstack-ironic13:05
*** snecklifter has quit IRC13:06
*** rloo has quit IRC13:11
*** rloo has joined #openstack-ironic13:12
TheJuliaGood morning everyone13:12
*** jcoufal has joined #openstack-ironic13:13
*** keekz has joined #openstack-ironic13:15
*** jpich_ has joined #openstack-ironic13:18
*** jpich has quit IRC13:21
*** snecklifter has joined #openstack-ironic13:23
sambettso/ xavierr, TheJulia13:24
*** joprovost has quit IRC13:28
*** bvandewa has joined #openstack-ironic13:29
nicodemosgood morning13:30
*** jpich_ is now known as jpich13:31
*** ayoung has joined #openstack-ironic13:32
*** trown is now known as trown|brb13:34
*** thrash has quit IRC13:34
*** skramaja is now known as skramaja_afk13:35
*** david-lyle_ has joined #openstack-ironic13:36
*** snecklifter has quit IRC13:38
* jroll steps away for a while13:38
mgouldmorning xavierr TheJulia13:39
*** mgoddard_ has joined #openstack-ironic13:40
*** david-lyle_ has quit IRC13:41
xavierrsambetts mgould \o13:42
xavierrmorning nicodemos13:42
*** thrash has joined #openstack-ironic13:42
*** thrash has quit IRC13:42
*** thrash has joined #openstack-ironic13:42
mgouldmorning nicodemos13:42
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port list command to OSC plugin  https://review.openstack.org/34672213:43
*** mgoddard__ has joined #openstack-ironic13:43
*** skramaja_afk is now known as skramaja13:44
*** sdake_ has joined #openstack-ironic13:45
*** sdake has quit IRC13:45
*** mgoddard has quit IRC13:45
*** mgoddard_ has quit IRC13:47
*** baoli_ has quit IRC13:48
*** milan has joined #openstack-ironic13:52
*** bks has quit IRC13:53
openstackgerritJulia Kreger proposed openstack/ironic-specs: Boot from Volume - Reference Drivers  https://review.openstack.org/29499513:54
*** trown|brb is now known as trown13:57
*** mgoddard__ has quit IRC14:00
*** mgoddard has joined #openstack-ironic14:00
*** anush_ has joined #openstack-ironic14:00
openstackgerritJulia Kreger proposed openstack/ironic-specs: Boot from Volume - Reference Drivers  https://review.openstack.org/29499514:01
*** hwcomcn has quit IRC14:04
openstackgerritAlexander Bashmakov proposed openstack/ironic: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python. This script is used for making libvirt guest domains during Devstack deployment. New file: create-node.py. Old script kept around for reference.  https://review.openstack.org/34683214:06
*** PollyZ has joined #openstack-ironic14:06
*** fragatina has quit IRC14:08
*** fragatina has joined #openstack-ironic14:08
*** PollyZ has quit IRC14:12
*** PollyZ has joined #openstack-ironic14:13
*** moshele has quit IRC14:16
*** baoli has joined #openstack-ironic14:17
*** moshele has joined #openstack-ironic14:20
*** PollyZ has quit IRC14:22
openstackgerritVladyslav Drok proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/33217714:22
NobodyCamGood morning Ironicers14:23
*** catintheroof has joined #openstack-ironic14:24
*** rama_y has joined #openstack-ironic14:24
*** bvandewa has quit IRC14:24
mgouldmorning NobodyCam14:26
openstackgerritAlexander Bashmakov proposed openstack/ironic: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python.  https://review.openstack.org/34683214:29
*** michauds has joined #openstack-ironic14:30
TheJuliaGood morning NobodyCam14:31
NobodyCammorning mogul :)14:31
NobodyCamgah darn apple autocorrect14:31
mgouldheh14:31
NobodyCammorning mgould14:31
NobodyCamhehehe14:31
mgouldmorning :-)14:31
sambettso/ NobodyCam TheJulia14:31
*** clenimar has joined #openstack-ironic14:32
NobodyCammorning sambetts :)14:32
openstackgerritlokesh s proposed openstack/ironic: Add support for the audit middleware  https://review.openstack.org/27265814:33
*** baoli has quit IRC14:36
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add "erase_devices_metadata_priority" config option  https://review.openstack.org/34417914:36
*** baoli has joined #openstack-ironic14:36
openstackgerritAlexander Bashmakov proposed openstack/ironic: Rewrite devstack/tools/ironic/scripts/create-node.sh in Python.  https://review.openstack.org/34683214:38
*** mtanino has joined #openstack-ironic14:39
*** jcoufal_ has joined #openstack-ironic14:39
*** beekneemech is now known as bnemec14:40
*** baoli has quit IRC14:41
jcookjroll does consistent hashing of computes to Ironic nodes not break non-disruptive upgrades of compute nodes14:41
jcookalso spof14:41
*** vishwanathj has joined #openstack-ironic14:42
jcookthough rehashing "eventually", perhaps quickly enough, would resolve spof14:42
*** jcoufal has quit IRC14:42
*** mjturek1 has joined #openstack-ironic14:43
rlooTheJulia: qq on https://review.openstack.org/#/c/344973/2/doc/source/deploy/adoption.rst. line 136. Did you want it to render with backslash, eg as '{\"boot_option\":...'? or as '{"boot_option": ...'?14:45
NobodyCamGood morning role :)14:45
NobodyCamgrrrr14:46
rlooNobodyCam: morning!14:46
mgouldjcook: what's the spof?14:46
NobodyCamrloo14:46
NobodyCamhehehe14:46
rlooTheJulia: I think I get it. You want it to render with the backslash cuz you used ". Whereas we sometimes use ' & " so that the backslash isn't needed.14:47
jcookmgould compute goes down, no instance actions succeed for that entire segment of nodes associated with it14:47
mgouldoic, thanks14:47
TheJuliarloo: yes, it needs to be rendered to load the json in :(14:48
mgouldour hash ring implementation has code to skip hosts without needing a full rebalance; would that help?14:48
xavierrhey NobodyCam rloo o/14:50
rloohi xavierr14:50
abashmakHello Ironicers. Anybody know how to get14:51
abashmakthe "Dell Ironic CI" build passing on my patch set? https://review.openstack.org/34683214:51
NobodyCammorning xavierr14:52
*** fragatina has quit IRC14:58
*** fragatin_ has joined #openstack-ironic14:58
*** joprovost has joined #openstack-ironic14:59
*** rcernin has quit IRC15:00
*** thiagop has joined #openstack-ironic15:00
*** spandhe has joined #openstack-ironic15:02
thiagopGood morning Ironic15:03
*** milan has quit IRC15:04
vdrokmorning all15:06
vdroksambetts: do you mind if i'll rebase https://review.openstack.org/#/c/314514/11 and fix the comments? I'd like to rebase portgroups patch on top of that15:06
*** mjura has quit IRC15:07
*** spandhe has quit IRC15:08
*** rama_y has quit IRC15:08
sambettsvdrok: just running through the comments now15:10
vdrokcool, thanks15:10
*** PollyZ has joined #openstack-ironic15:11
thiagopsambetts: feeling better? Heard that you were sick last week...15:11
sambettsthiagop: still not 100%, my mind could taking sleeping anymore though so I needed to get up a do something15:13
openstackgerritRuby Loo proposed openstack/ironic: Use 'UUID', not 'uuid' in exception strings  https://review.openstack.org/34688315:13
thiagopsambetts: sometimes is better to react, indeed15:13
vdroksambetts: that sucks, get better soon!15:13
mgouldsambetts: get well soon15:16
*** mtanin___ has joined #openstack-ironic15:17
sambettsvdrok: More than happy for you to rebase that patch :) all the comments look sane enough to me15:17
vdroksambetts: yup, will do :)15:18
*** PollyZ has quit IRC15:18
*** mtanino has quit IRC15:19
*** edand has quit IRC15:20
*** ijw has joined #openstack-ironic15:22
*** harlowja_at_home has joined #openstack-ironic15:22
*** mtanino has joined #openstack-ironic15:24
*** mtanin___ has quit IRC15:25
*** pcaruana has quit IRC15:26
NobodyCammorning thiagop and vdrok15:26
*** skramaja is now known as skramaja_afk15:26
thiagopmorning NobodyCam, sir15:26
NobodyCam:p15:26
vdrokgood morning NobodyCam :)15:29
NobodyCam:)15:29
*** milan has joined #openstack-ironic15:31
openstackgerritVladyslav Drok proposed openstack/ironic: Prevent URL collisions with sub-controllers: nodes/ports  https://review.openstack.org/31451415:32
jrolljcook: "compute goes down, no instance actions succeed for that entire segment of nodes associated with it" is nova's failure model for nova-compute (in other words, the same way it works in virt)15:34
jrolljcook: so if you want to minimize failure as much as possible, run N nova-compute where N is your number of nodes15:34
openstackgerritAkilan Pughazhendi proposed openstack/ironic: Updating dev quickstart to include compatiblity for newest distros  https://review.openstack.org/34689415:35
jrolljcook: as for non-disruptive upgrades, it should be okay, as the message will just hang out on the bus until the compute comes back up15:35
*** PollyZ has joined #openstack-ironic15:36
jrolljcook: anyway, the goal for now is play with it, see where it breaks down, it can't get *worse*, so we iterate15:36
*** fragatin_ has quit IRC15:36
*** milan has quit IRC15:36
*** fragatina has joined #openstack-ironic15:37
*** david-lyle_ has joined #openstack-ironic15:37
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Refactor common checks when instantiating the ipmitool classes  https://review.openstack.org/34689615:38
*** Nisha1 has joined #openstack-ironic15:38
openstackgerritMario Villaplana proposed openstack/ironic: Add power state change notifications  https://review.openstack.org/32186515:39
openstackgerritMario Villaplana proposed openstack/ironic: Add notification base classes and docs  https://review.openstack.org/29846115:39
*** amotoki has quit IRC15:40
*** david-lyle_ has quit IRC15:43
*** rpioso has joined #openstack-ironic15:43
*** baoli has joined #openstack-ironic15:45
jcookjroll ack15:45
*** sdake has joined #openstack-ironic15:46
*** Sukhdev has joined #openstack-ironic15:47
*** skramaja_afk is now known as skramaja15:49
*** sdake_ has quit IRC15:49
*** hshiina has joined #openstack-ironic15:51
thiagopNobodyCam: do you have a review slot to put the Dynamic allocation patch in? :)15:52
openstackgerritStephanie Miller proposed openstack/ironic: Introduce new RPCs to support volume target operations  https://review.openstack.org/28522015:53
TheJuliathiagop: Sorry, didn't get a chance to look at it Friday night, I'll try to look at it after my meetings for the day15:54
thiagopTheJulia: I thank you :)15:55
*** appu103_ has joined #openstack-ironic15:56
NobodyCamthiagop: do you have a link handy15:57
jrollJayF: devananda: rloo: I updated the resource_class spec this morning, if you have time to re-review https://review.openstack.org/#/c/345040/15:57
NobodyCammorning jroll :)15:57
jrollhey NobodyCam :)15:57
thiagopNobodyCam: always: https://review.openstack.org/#/c/286192/15:57
NobodyCam:)15:57
*** mtanino__ has joined #openstack-ironic15:57
rloojroll: will look at it today. Haven't yet read your email.15:57
jrollrloo: cool, just a heads up :)15:58
*** mtanino has quit IRC15:58
*** fragatina has quit IRC15:58
rloojroll: etc, heads up, i'll be away next week, so let me know if there are high (er) priority things that ought to be looked at this week15:58
*** penick has joined #openstack-ironic15:59
jrollrloo: okay, I'll see what I can do. probably re-order trello today15:59
rloojroll: ok, i'm fine if there aren't any hi-prior things ;)15:59
*** penick_ has joined #openstack-ironic16:00
jrollrloo: well, I'm a week out of the loop, still need to catch up :)16:00
*** fragatina has joined #openstack-ironic16:00
*** bvandewa has joined #openstack-ironic16:00
*** edand has joined #openstack-ironic16:01
*** milan has joined #openstack-ironic16:02
devanandamoring, all16:02
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Add parse_root_device_hints to utils.py  https://review.openstack.org/34606816:03
*** penick has quit IRC16:03
*** penick_ is now known as penick16:03
*** Goneri has quit IRC16:04
jrollTheJulia: oh, I remembered why our ML2 plugin "caches" things - it isn't about caching, but rather that it needs to know which rack things are in at port-create time, to assign a vlan (we do vlan to vxlan at the TOR, and so each network needs a unique vlan within a given rack)16:06
jrollhope that sounds like english16:06
*** mtanino has joined #openstack-ironic16:06
TheJuliakind of sounds like english :)16:07
TheJuliajroll: some caching does make sense with the neutron API :)16:07
jrollTheJulia: heh16:08
*** mtanino__ has quit IRC16:08
*** gabriel-bezerra has joined #openstack-ironic16:09
JayFrloo: I approved the RFE for https://review.openstack.org/#/c/344164 if you wanna pull your -216:13
*** e0ne has quit IRC16:14
*** jaybeale has joined #openstack-ironic16:16
rlooJayF: thx. I had some questions about it, did you see?16:16
rlooJayF: about the RFE I mean, not the patch.16:16
JayFNow I do; I had an old copy of the bug up :(16:16
JayFwill unapprove and participate in chat now16:16
rlooJayF: approve is ok if answers to my questions are straightforward.16:17
JayFwell tl;dr: it should be the highest priority of any clean steps16:17
JayFand 2) we do versioning for clean steps16:17
JayFso if you upgrade to a new IPA with this clean step, we should've bumped the HWM version so cleaning would restart16:17
JayFthat's all handled pretty well in the cleaning code :)16:17
*** Goneri has joined #openstack-ironic16:18
rlooJayF: ahh. that is what i was wondering about. do we restart from the beginning of the clean step chain, or restart from the current clean step?16:18
JayFwhen the [{HWM: version}] list of dicts changes *at all* cleaning is restarted from the very beginning16:19
rlooJayF: eg, node is doing the long erase_disk thing; then ipa is updated with this new clean step, we restart and redo the erase-disk again?16:19
JayFyes16:19
JayFwell, actually in reality probably not16:19
JayFsince it's the only clean step16:20
JayFbasically imagine I have clean steps 1, 2 (reboot requested), 3, 416:20
JayFand I replace the IPA image between steps 1 and 216:20
JayFclean step 1 runs, clean step 2 runs, system reboots16:20
JayFnew agent comes up with different clean steps, restarts at 1, 2 (reboot), then 3, 416:20
JayFsince the existing HWM has no reboot_requested cleaning steps, I think it'll never restart today16:21
JayFbecause you run the exact same agent/hwm through the whole cleaning process16:21
*** tesseract- has quit IRC16:21
rlooJayF: so this new clean step, would that be the new #1 clean step?16:21
JayFit'd be the highest priority, so it'd run first16:22
JayFit does mix in with conductor-run cleaning steps, and I ensured in the review of the code it's higher prio than all of those16:22
rlooand where, in list of clean steps, is existing erase_disk clean step?16:22
JayFclose to the end16:22
JayFit's hard to say exactly because different conductor drivers can run different steps16:22
rlooso after your 2 (reboot requested) step?16:23
JayFI know the iLo conductor-side steps are all higher priority than the erase_devices step16:23
JayFwell that 1,2,3,4 was a contrived example :)16:23
JayFfor an agent_ipmitool user, today cleaning is just the 1 erase_Devices step16:23
JayFand this would make it erase_metadata, erase_devices16:23
rlooJayF: ok, let's just use agent_ipmitool then. The node is doing cleaning, there is only one step erase_devices.16:24
JayFin that case, it would not restart16:24
JayFbecause you're running the same agent throughout cleaning16:24
JayFthe "restart cleaning" only comes into effect if *in the middle of cleaning* you get a new agent booted16:24
jrollvsaienk0: added that comment on the nova patch for you :) https://review.openstack.org/29789516:24
JayFwhich only happens if you have cleaning steps taht involve a reboot16:24
JayFand you reboot into a different agent16:24
rloook, good, that' what i wanted to know.16:24
rlooJayF: I wanted to make sure erase_devices was not done twice.16:25
JayFit totally can be in other cases (like our downstream case that happens often)16:25
rlooJayF: so it would only be done twice if someone added a clean step after erase_devices, that had a reboot.16:25
JayFyes16:25
JayFand honestly, I don't think we should be -1 to a change just because that would be a side effect, but in this case it wouldn't happen16:26
rlooJayF: and that, we hope, would be never or rare.16:26
JayFin shipped cleaning, it shouldn't happen often at all16:26
jrollwe should make sure that's clear in the docs, "if you upgrade your agent it may restart ongoing cleaning"16:26
rlooJayF: should that be mentioned in release notes?16:26
JayFit is16:26
JayFjroll: ^16:26
jroll\o/16:26
JayFrloo: Maybe? if you think so please comment on the patch, but IDK what I'd say there16:27
*** harshs has joined #openstack-ironic16:27
*** moshele has quit IRC16:27
rlooJayF: I'm just trying to grok the RFE and any implications of that16:27
JayFrloo: I put a couple of comments sumamrizing the answers to your questions on the bug, I'll let you approve it since you had questions16:28
*** davidlenwell has quit IRC16:28
* lucasagomes reads16:28
lucasagomesrloo, JayF I've just commented on the rfe16:28
lucasagomesthanks for the question and the approval JayF :D16:29
lucasagomesrloo, *16:29
JayFlucasagomes: well I unapproved pending rloo's questions being sufficiently answered16:29
* JayF needs to refresh before doing things from now on, lol16:29
rloolucasagomes: , JayF: ha ha. Am looking now.16:29
lucasagomesJayF, ack, yeah I've just replied to it. Maybe it's not enough, so chime in please :-)16:29
JayFit looks fine to me16:30
rloolucasagomes, JayF: I'm going to add a comment summarizing our IRC discussion, then I'll approve.16:30
JayFthe only caveat to remember is that you'd only get a CLEAN_VERSION_MISMATCH if you changed agents mid-cleaning16:30
JayFwhich would require a reboot mid-cleaning16:30
JayFwhich none of our in-band steps request today16:30
lucasagomestrue16:31
JayFI honestly am now curious about how we handle that in other cases16:31
JayFlike if you update the agent a node would use in the node object16:31
JayFwould ironic actually change to the new agent during cleaning16:31
JayFdownstream, we host our own agents, so it's "easy" to have a version change mid-cleaning16:32
jrollgood question, I feel like it wouldn't16:32
* sambetts -> afk16:32
JayFbut with it setup in the dhcp environment per-node, I don't think it would16:32
*** sambetts is now known as sambetts|afk16:32
jrollbecause the pxe prep does all the tftp stuff16:32
JayFand I don't think that's bad behavior, honestly16:32
lucasagomesJayF, sounds unlekiley, I don't think Ironic will download the new agent mid operation16:32
jrolldownloading images etc16:32
lucasagomesit only does at prepare_ramdisk()16:32
lucasagomeswhich happens at the beginning of the process16:32
JayFlol16:32
lucasagomesbut again, I haven't tested it16:32
JayFyeah; I hadn't ever thought about it until now16:33
*** bvandewa has quit IRC16:34
*** david-lyle_ has joined #openstack-ironic16:35
*** trown is now known as trown|lunch16:36
openstackgerritMerged openstack/bifrost: Remove discover from test-requirements  https://review.openstack.org/34520316:36
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add erase_devices_metadata cleaning step  https://review.openstack.org/34416416:36
rloolucasagomes, JayF approved.16:37
lucasagomesrloo, thanks16:37
lucasagomespatch updated as well ^16:37
JayFrloo: your dillegence helped us realize the version hadn't been bumped, so \o/16:37
rlooJayF: :)16:38
* lucasagomes will take a quick break before the meeting16:39
*** lucasagomes is now known as lucas-afk16:39
*** davidlenwell has joined #openstack-ironic16:40
*** siva_krish has joined #openstack-ironic16:40
TheJuliathiagop, xavierr: by chance did you look into the dynamic allocation change and see why the CI jobs failed?16:41
thiagopTheJulia: oneview CI? It's broken due to power outage on the weekend. I'm fixing it right now16:42
TheJuliathiagop: ok16:42
*** derekh has quit IRC16:43
*** nicodemos has quit IRC16:44
*** bvandewa has joined #openstack-ironic16:45
* thiagop eats lunch quickly before meeting16:47
*** nicodemos has joined #openstack-ironic16:49
*** jpich has quit IRC16:49
*** harshs has quit IRC16:51
*** Sukhdev has quit IRC16:54
*** harlowja_at_home has quit IRC16:54
*** mbound has quit IRC16:55
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port list command to OSC plugin  https://review.openstack.org/34672216:55
*** stendulker has joined #openstack-ironic16:57
*** lucas-afk is now known as lucasagomes17:00
jrollmeeting time :)17:00
*** david-lyle_ has quit IRC17:01
TheJuliathiagop, xavierr: I have a concern that I've noted on the dynamic allocaiton rev17:01
*** PollyZ has quit IRC17:02
thiagopTheJulia: looking...17:02
openstackgerritImre Farkas proposed openstack/ironic: DRAC RAID configuration  https://review.openstack.org/17631917:05
openstackgerritMerged openstack/ironic: Bug fixes and doc updates for adoption  https://review.openstack.org/34497317:05
* mgould is too dopey and tired for the meeting, going home17:06
*** mgould is now known as mgould|afk17:06
TheJuliagoodnight mgould|afk17:06
mgould|afkgood night!17:06
*** david-lyle_ has joined #openstack-ironic17:07
*** rajinir has joined #openstack-ironic17:07
*** ifarkas is now known as ifarkas_afk17:07
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Add baremetal port list command to OSC plugin  https://review.openstack.org/34672217:11
openstackgerritVladyslav Drok proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/33217717:12
*** wajdi has joined #openstack-ironic17:16
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection  https://review.openstack.org/20190417:16
*** moshele has joined #openstack-ironic17:20
*** jaybeale has quit IRC17:21
*** jaybeale has joined #openstack-ironic17:21
*** edand has quit IRC17:22
*** pcaruana has joined #openstack-ironic17:23
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Add new out-of-band properties to ilo drivers  https://review.openstack.org/33813817:25
*** ChubYann has joined #openstack-ironic17:31
*** mbound has joined #openstack-ironic17:33
*** fragatina has quit IRC17:35
*** spandhe has joined #openstack-ironic17:35
*** PollyZ has joined #openstack-ironic17:36
*** ChubYann has quit IRC17:36
*** harshs has joined #openstack-ironic17:37
*** harshs has quit IRC17:37
*** ChubYann has joined #openstack-ironic17:37
*** hshiina has quit IRC17:37
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-python-agent: Fix resolv.conf in tinyipa image build  https://review.openstack.org/34696317:39
*** stendulker_ has joined #openstack-ironic17:39
*** stendulker_ has quit IRC17:39
*** ksumit has joined #openstack-ironic17:39
*** stendulker_ has joined #openstack-ironic17:39
*** e0ne has joined #openstack-ironic17:40
*** trown|lunch is now known as trown17:40
*** moshele has quit IRC17:40
*** sdake has quit IRC17:41
*** electrofelix has quit IRC17:42
*** stendulker has quit IRC17:42
*** harlowja has joined #openstack-ironic17:43
*** harshs has joined #openstack-ironic17:47
*** appu103_ has quit IRC17:50
*** ijw has quit IRC17:56
*** david-lyle_ is now known as david-lyle17:57
xavierrhey TheJulia, I'm removing the translation funtions for NODE_IN_USE_BY_ONEVIEW and SERVER_HARDWARE_ALLOCATION_ERROR constraints, it that ok for you?17:57
*** stendulker_ has quit IRC17:59
TheJuliaxavierr: add a comment as to why it can't be translated17:59
*** fragatina has joined #openstack-ironic17:59
thiagopTheJulia: since these messages aren't logged, I don't think it should be translated at all17:59
thiagopwdyt?18:00
*** stendulker has joined #openstack-ironic18:00
TheJuliait is operator visbible, if an operator can't read english, it might be a little frustrating unless they know what the string means18:00
lucasagomescalling it a day, have a great evening all18:01
*** lucasagomes is now known as lucas-dinner18:01
thiagopg'night lucas-dinner18:01
NobodyCamnight lucas-dinner18:01
TheJuliagoodnight lucas-dinner18:01
xavierrgoodnight dinner18:01
xavierrlucas-dinner18:01
thiagopTheJulia: I see, so remove it and leave a comment, please, xavierr18:01
NobodyCamlol18:02
*** skramaja is now known as skramaja_afk18:02
xavierrlol18:02
TheJuliathiagop: I guess I'm okay with it not being translated, just not exactly thrilled :)18:02
*** stendulker_ has joined #openstack-ironic18:02
thiagopTheJulia: the underlying problem, anyway, is changing the message between releases. We need to take care if we must change it, translated or not18:04
*** fragatina has quit IRC18:04
TheJuliathiagop: yup18:04
*** stendulker has quit IRC18:05
openstackgerritMerged openstack/ironic: Remove duplicate copyright.  https://review.openstack.org/34521318:14
openstackgerritMerged openstack/ironic: Trivial: Fix a trivial flake8 error  https://review.openstack.org/34581918:18
openstackgerritMerged openstack/ironic: Use assertEqual() instead of assertDictEqual()  https://review.openstack.org/34656118:18
*** ravig has joined #openstack-ironic18:20
*** mbound has quit IRC18:23
*** mbound has joined #openstack-ironic18:24
*** mbound has quit IRC18:25
* vdrok leaves too18:25
vdrokgood night!18:26
thiagopnigth vdrok18:26
thiagopnight*18:26
NobodyCamnight vdrok :)18:27
*** wajdi has quit IRC18:28
*** ravig has quit IRC18:29
*** wajdi has joined #openstack-ironic18:29
*** harshs has quit IRC18:29
openstackgerritXavier proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers  https://review.openstack.org/28619218:29
xavierrTheJulia ^^^18:30
TheJuliathank you xavierr18:30
*** ayoung has quit IRC18:30
xavierrTheJulia ;)18:32
*** bodepd has joined #openstack-ironic18:32
bodepdI have a specific bifrost question. It comes with examples for DIB with a list of elements that can be applied. I see that it does not enable cloud-init, so I'm confused about how the ip addresses that are set in the database get applied to the hosts.18:33
*** ijw has joined #openstack-ironic18:34
bodepdIf I use the default image building scripts, it doesn't get network, if I try to add cloud-init, it uses plain dhcp and doesn't get the ip address that was setup during the enroll process18:34
TheJuliabodepd: I'll be with you in one moment, wrapping up a review real quick18:35
*** ahqa has joined #openstack-ironic18:38
*** ijw has quit IRC18:38
*** e0ne has quit IRC18:39
bodepdTheJulia: thanks!18:39
*** michauds has quit IRC18:42
*** spandhe has quit IRC18:43
cineramahey bodepd so we use simple-init by default, which uses glean to apply network configuration stored in a configuration drive18:44
TheJuliaokay18:44
TheJuliabodepd: what cinerama said :)18:44
*** sdake has joined #openstack-ironic18:44
TheJuliabodepd: we set network information into the configuration drive, it is something that cloud-init doesn't really support, at least as of the last time I checked18:46
cineramabodepd, we also had a user try to use cloud-init and simple-init at the same time and iirc things got kind of confused so i wouldn't recommend that18:46
TheJuliabodepd: we also support static dhcp assignments with dnsmasq, if you use cloud-init that migth be an option18:49
*** ravig_ has joined #openstack-ironic18:49
*** Sukhdev has joined #openstack-ironic18:50
ravig_Hello everyone18:50
ravig_I have a question, may I know, who is working on this : https://review.openstack.org/#/c/207278/18:50
*** harshs has joined #openstack-ironic18:52
TheJuliaravig_: You mean who pushed the patch up?18:53
rlooravig_: lin.tan@intel.com. although i thought he said something about not working on it.18:54
NobodyCamHi ravig_ looks like that was lint an, but it appears that its not been worked on for a while18:54
NobodyCams/lint/lintan/18:54
NobodyCam:p18:54
NobodyCamI will get this new system trained yet18:55
ravig_Yeah, I have question for the ipxe part, should I contact them directly?18:55
PollyZhi got a question, I found a bug about hte microversion-parser and I submitted a bug report, should I work on that untill someone approves this bug?18:56
rlooravig_: yes, best to ask him directly.18:57
*** michauds has joined #openstack-ironic18:58
rlooravig_: i believe he mentioned that there was a problem with the ipxe solution18:59
*** ahqa has quit IRC18:59
jrollPollyZ: yes, you can work on bugs before they are triaged18:59
jrollPollyZ: if you want to link me to it, I can do that now18:59
*** PollyZ has quit IRC19:00
*** bvandewa_ has joined #openstack-ironic19:00
*** bvandewa has quit IRC19:04
*** harshs has quit IRC19:04
*** bvandewa_ has quit IRC19:05
*** harshs has joined #openstack-ironic19:05
*** harshs has quit IRC19:06
*** ravig_ has quit IRC19:07
*** Nisha1 has quit IRC19:12
*** PollyZ has joined #openstack-ironic19:13
*** stendulker_ has quit IRC19:15
*** jcoufal_ has quit IRC19:15
*** mbound has joined #openstack-ironic19:25
*** jaybeale has quit IRC19:29
*** chlong_POffice has quit IRC19:30
*** mbound has quit IRC19:31
*** chlong_POffice has joined #openstack-ironic19:31
*** spandhe has joined #openstack-ironic19:33
*** bvandewa has joined #openstack-ironic19:36
thiagop#info: OneView CI is officially back19:37
*** bvandewa_ has joined #openstack-ironic19:38
TheJulia\o/19:40
*** bvandewa has quit IRC19:42
openstackgerritJim Rollenhagen proposed openstack/ironic: Testing: nova with multitenant networks  https://review.openstack.org/34700419:57
*** harshs has joined #openstack-ironic19:58
*** ijw has joined #openstack-ironic19:59
*** harshs has quit IRC20:00
*** ijw has quit IRC20:00
NobodyCamthiagop: w00t :)20:06
openstackgerritJulia Kreger proposed openstack/ironic-specs: Boot from Volume - Reference Drivers  https://review.openstack.org/29499520:08
*** ahqa has joined #openstack-ironic20:12
*** PollyZ_ has joined #openstack-ironic20:16
*** sdake has quit IRC20:16
*** PollyZ has quit IRC20:16
*** rodrigofm has joined #openstack-ironic20:21
thiagopjroll: I'm trying to do that thing of removing Ironic tests from tempest here, but a wild question appeared: tests from nova such as https://github.com/openstack/tempest/blob/master/tempest/api/compute/admin/test_baremetal_nodes.py should be moved to Ironic's tree?20:26
jrollthiagop: those can stay, it's a silly ironic proxy from when we deprecated nova-baremetal20:27
jroll(which is going away soon)20:27
*** bvandewa_ has quit IRC20:28
thiagopjroll: ack20:28
*** jaybeale has joined #openstack-ironic20:30
jrollugh, wsmeeeeeeee20:37
*** bvandewa has joined #openstack-ironic20:37
JayFI was going to try and add statds metrics support (statds backending to console) to devstack20:39
JayFbut there's no statsd package for ubuntu20:40
JayF[insert four letter words here]20:40
* TheJulia suspects that the line would be truncated by your irc client20:40
JayFsomething like that20:40
jrollJayF: curl | sh20:40
JayFfedora has the package too20:40
jroller I guess just curl -o /usr/bin/statsd20:40
JayF...20:41
* TheJulia blinks20:41
JayFvery no?20:41
jrolllol20:41
JayFI'm just saying, statsd + console backend20:41
JayFis the perfect testnig setup for statsd in devstack20:41
JayFand dirt simple ... in fedora20:41
jrollthey do just ship a binary though, right?20:41
JayFI mean, I'm sure they do? but at that point it becomes less of a good thing to do20:41
JayFi.e. it's not just "add a package to debs + rpms, write simple config, start it in a screen"20:42
*** milan has quit IRC20:43
clif_hJayF: jroll does it make sense to make `cache_image` a core API endpoint if caching is only currently supported by the agent driver? mariojv brought it up on my image caching spec https://review.openstack.org/#/c/310594/820:43
JayFclif_h: Maybe the better question is; why shouldn't/can't we support caching in the iscsi driver?20:43
jrollclif_h: is there a reason it couldn't be supported by iscsi?20:43
mariojvi thought since agent heartbeat and lookup were getting promoted, it'd make sense20:43
jrollyeah that20:43
jrollmariojv++20:44
jrollI swear I commented similarly a while ago20:44
clif_hidk anything about iscsi20:44
clif_hor are you saying why couldn't be supported by Foo driver?20:45
clif_hwhere Foo is any Ironic driver20:45
jrollright20:45
*** bvandewa has quit IRC20:46
jrollgiven there's only two deploy drivers, foo == iscsi20:46
clif_hoh, ok20:46
clif_haren't there three?20:46
clif_hpxe?20:46
jrolliscsi basically exposes the hard disk from the node as an iscsi mount, ironic connects to it and writes stuff20:46
jrollpxe_* is actually iscsi20:46
jrollbut was poorly named in the past20:46
clif_hI see20:46
clif_hI'm just going off of https://wiki.openstack.org/wiki/Ironic/Drivers20:47
jroll>.>20:47
clif_h<.<20:47
* jroll wonders why that has both20:47
clif_hdocs are always out of date/alignment with code :>20:48
*** joprovost has quit IRC20:48
jrollwiki isn't doc!20:49
jroll:P20:49
ahqaHi, I wanted to know if DIB supports ubuntu xenial?20:50
clif_hDIB?20:51
TheJuliaclif_h: we're fairly good about getting people to land doc with changes, or have the doc nearly ready to go before we land the feature20:51
TheJuliadiskimage-builder20:51
JayFclif_h: diskimage-builder20:51
clif_hoh20:51
JayFahqa: that's a better question for tripleo IRC20:52
TheJuliaahqa: I do not know :(  cinerama may know20:52
cineramahey ahqa you should be able to use xenial20:52
JayFahqa: well, TheJulia is right if you mean "ironic's ramdisk built with dib", if you're asking about dib generically, you should ask tripleo :)20:52
cineramaahqa: most of us dib folk hang out in #openstack-infra and #tripleo (though we'll hopefully have our own IRC channel really soon)20:53
cineramaahqa: oh assuming you mean "can i build a dib image of xenial"20:53
JayFblargh :( /me realizes how much of a pita it is to test IPA in devstack20:54
clif_hso is it ok to only initially support 'cache_image' in the agent driver, if I'm spec-ing it to be a core api endpoint?20:54
jrollclif_h: yeah20:54
ahqacinerama Yes, I am trying to build a dib image of xenial and xenail doesn't come with python2 installed by default.20:55
*** chlong_POffice has quit IRC20:56
cineramaahqa, how recent of a diskimage-builder are you using?20:56
cineramaahqa, the reason i ask is that we have some changes that support distros which don't include python 2 but they're reasonably recent, so check your version - 1.18.0 or later should work20:58
openstackgerritClif Houck proposed openstack/ironic-specs: Add spec for image caching to nodes via agent driver  https://review.openstack.org/31059420:59
*** amotoki has joined #openstack-ironic20:59
cineramaahqa, anyway if you've checked the version and you're still having trouble, please pastebin your output and i'll have a look21:03
*** bvandewa has joined #openstack-ironic21:07
*** bvandewa has quit IRC21:07
rloojroll: do you have a few minutes to discuss node.resource_class?21:07
rloojroll: if there are 10K ironic nodes, does that mean there are 10K resource providers in nova?21:08
jrollrloo: kinda, in a meeting but yes21:08
jrollrloo: correct21:08
mrdaMorning Ironic21:09
rloojroll: and what if node.resource_class gets changed? the nova resource_provider will get updated somehow?21:09
JayFrloo: with the resource tracker loop21:09
JayFrloo: similar to today if you modified # of vcpus in node properties21:09
*** chlong_POffice has joined #openstack-ironic21:09
*** trown is now known as trown|outtypewww21:09
rlooJayF: OH. ok, i thought with the upcoming placement API, the resource tracker loop wouldn't be needed.21:09
jrollwhat jay said21:09
jrollrloo: so in newton, the resource tracker will put the data there21:10
JayFrloo: no, we're still using that for now, BUT it gives us the option with Ironic to push data directly into the placement API21:10
jrollin ocata, we'll start pushing data through the placement API21:10
JayF^ that21:10
*** rodrigofm has quit IRC21:11
thiagopmorning mrda21:11
mrdao/21:12
NobodyCammorning mrda :)21:12
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Add metrics support to IPA  https://review.openstack.org/11998121:12
rlooJayF, jroll: OK, got it. so it doesn't really matter that node.resource_class is initially populated with something. as long as the field exists to do the dance in nova21:12
JayFthis ^ code fails tests because apparently _download_image isn't ever getting called behind the metrics context manager :(21:13
mrdarloo: lol21:13
JayFrloo: well, it'll need to be populated in advance of an ocata upgrade21:13
*** jjohnson2__ has quit IRC21:13
*** ijw has joined #openstack-ironic21:13
jrollrloo: right, we'll want operators to populate that between newton and ocata releases of nova21:13
JayFrloo: so for instance, an operator upgrading to ocata would want to, X hours in advance, ensure all nodes have node.resource_class populated as desirted21:13
rlooJayF: well, that depends. it isn't clear what happens if that value is NULL.21:13
*** ijw has joined #openstack-ironic21:13
mrdaJayF: Is there plans for a script to generate that info, as a pre-O population?21:13
JayFrloo: it just means we don't have those ironic nodes as resource providers.21:14
JayFmrda: It's an operator setting, not really something we can divine21:14
JayFmrda: so in that case any script would be more or less a for loop to update all nodes21:14
rlooJayF: oh, that isn't clear in the spec. I mean, Jim said in the spec patch: 'Yeah, there's some edge cases here that I haven't fully formed my thoughts on - I'd like to chat with the Nova folks about that.'21:15
mrdamight still be useful, especially if there we could guesstimate sane defaults21:15
*** fragatina has joined #openstack-ironic21:15
JayFrloo: my response is based on what I understood from our conversations at the mid-cycle, if that's not true I have questions too :)21:16
JayFmrda: the operator also has to change their nova flavor to a value that matches21:16
JayFmrda: so nothing we do can really be predictive there21:16
rlooJayF: OK, we should make a note that it is an outstanding question.21:16
mrdaoh, that's true21:16
*** Goneri has quit IRC21:16
JayFmrda: because you have to make the nova flavor say "I want one of these resource class: foo as part of flavor: chicken", then you have to make sure you have nodes in resource_class: foo21:16
JayFmrda: so a circular bootstrapping problem21:17
rlooJayF: one last question (for now). the example, line 92: https://review.openstack.org/#/c/345040/8/specs/approved/node-resource-class.rst21:17
mrdayeah, after I coffee I want to thing about this some more - it'd be nice to ease transition on upgrade, just a matter of how to do that...21:17
rlooJayF: i don't think that is true cuz baremetal-gold has extraspecs boot_mode bios21:18
rlooJayF: and node1 has capabilities uefi.21:18
rlooJayF: but I'll add a comment about that. I might just be tired.21:19
JayFrloo: I think (will double check) that bios boot is also considered capable in ironic21:19
JayF*always21:19
*** morgabra has quit IRC21:19
JayFrloo: but that is worthy of a comment b/c at a minimum the example is confusing21:19
*** ijw has quit IRC21:19
rlooJayF: yeah, I'd just delete that capability thing i think.21:19
JayFwell capabilities still working was a big topic at the mid cycle21:20
*** ijw has joined #openstack-ironic21:20
JayFbut it may just be overkill for the spec21:20
*** morgabra has joined #openstack-ironic21:20
jrollright, I wanted to express to folks that capabilities will still work21:20
jrollmaybe I'll make it something less implicit21:20
jrollrloo: if you comment on that I can fix it21:20
*** harshs has joined #openstack-ironic21:21
*** morgabra has quit IRC21:28
*** morgabra has joined #openstack-ironic21:28
*** siva_krish has quit IRC21:36
*** siva_krish has joined #openstack-ironic21:40
*** bnemec has quit IRC21:40
*** spandhe has quit IRC21:42
*** moshele has joined #openstack-ironic21:44
*** fragatin_ has joined #openstack-ironic21:49
*** bnemec has joined #openstack-ironic21:50
*** fragatin_ has quit IRC21:51
*** fragatin_ has joined #openstack-ironic21:52
*** fragatina has quit IRC21:53
*** PollyZ_ has quit IRC21:53
*** bvandewa has joined #openstack-ironic21:57
*** bvandewa has quit IRC22:01
*** PollyZ has joined #openstack-ironic22:01
*** baoli has quit IRC22:03
thiagopI'm calling it a day22:08
thiagopgood night!22:08
*** thiagop has quit IRC22:08
*** ahqa has quit IRC22:09
*** gfhellma has joined #openstack-ironic22:10
openstackgerritChris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP  https://review.openstack.org/34394722:12
openstackgerritJim Rollenhagen proposed openstack/ironic: Add node.resource_class field  https://review.openstack.org/34505522:13
*** ijw has quit IRC22:14
*** thrash is now known as thrash|g0ne22:17
*** ijw has joined #openstack-ironic22:20
*** fragatina has joined #openstack-ironic22:23
*** bvandewa has joined #openstack-ironic22:23
*** harshs has quit IRC22:24
*** fragatin_ has quit IRC22:25
*** harshs has joined #openstack-ironic22:26
*** bvandewa has quit IRC22:27
*** ksumit has quit IRC22:28
*** wajdi_ has joined #openstack-ironic22:32
*** bvandewa has joined #openstack-ironic22:33
*** moshele has quit IRC22:35
*** bcornec1 has quit IRC22:35
*** wajdi has quit IRC22:35
*** wajdi_ has quit IRC22:36
*** bcornec1 has joined #openstack-ironic22:37
*** ijw has quit IRC22:38
*** ahqa has joined #openstack-ironic22:38
*** ijw has joined #openstack-ironic22:38
*** ijw has quit IRC22:40
*** oomichi has quit IRC22:41
*** siva_krish has quit IRC22:41
*** michauds has quit IRC22:42
rlooTheJulia: if you're around and have time, let me know if I'm wrong or not: https://review.openstack.org/#/c/297895/9/nova/virt/ironic/driver.py22:48
rlooTheJulia: am going to take off. will check with you or jroll tomorrow about that.22:49
*** oomichi has joined #openstack-ironic22:50
*** harshs has quit IRC22:50
*** lucas-dinner has quit IRC22:51
*** PollyZ has quit IRC22:55
*** spandhe has joined #openstack-ironic22:57
*** lucasagomes has joined #openstack-ironic22:57
*** jaybeale has quit IRC23:01
TheJuliarloo: if it is None, the API should represent whatever default is defined, or flat if neutron is the default dhcp provider, otherwise it is noop... so I don't think anything is needed on the nova side of things23:03
*** ksumit has joined #openstack-ironic23:05
* devananda returns from lunch and PT to get some revies and coding done23:13
*** amotoki has quit IRC23:13
*** anush_ has quit IRC23:15
*** rpioso has quit IRC23:23
*** rpioso has joined #openstack-ironic23:23
NobodyCamwow zuul appears to be quite backed up23:23
*** rpioso has quit IRC23:23
*** fragatina has quit IRC23:23
*** fragatina has joined #openstack-ironic23:24
*** amotoki has joined #openstack-ironic23:24
openstackgerritChris Krelle proposed openstack/bifrost: WIP: initial work to add ISC-DHCP  https://review.openstack.org/34394723:24
*** chlong_POffice has quit IRC23:25
*** ksumit has quit IRC23:26
*** amotoki has quit IRC23:29
*** bcornec1 has quit IRC23:31
*** oomichi has quit IRC23:31
*** jaybeale has joined #openstack-ironic23:31
*** bcornec1 has joined #openstack-ironic23:33
*** jaybeale has quit IRC23:39
*** ahqa_ has joined #openstack-ironic23:40
*** oomichi has joined #openstack-ironic23:40
*** jaybeale has joined #openstack-ironic23:41
*** Sukhdev has quit IRC23:41
*** chlong_POffice has joined #openstack-ironic23:42
*** ahqa has quit IRC23:43
*** Sukhdev has joined #openstack-ironic23:43
*** jaybeale has quit IRC23:44
*** abcxyz has joined #openstack-ironic23:46
*** anush_ has joined #openstack-ironic23:46
*** hoangcx has quit IRC23:48
*** harshs has joined #openstack-ironic23:58
*** jaybeale has joined #openstack-ironic23:58

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