Tuesday, 2015-11-10

*** thrash is now known as thrash|g0ne00:00
*** Sukhdev has joined #openstack-ironic00:00
*** spandhe has quit IRC00:01
zer0c00lMy co-worker pointed me towards this https://review.openstack.org/#/c/241040/00:01
zer0c00lLooks like zookeeper is now default for all devstack installs00:02
zer0c00lIt's just centos7 couldn't fnd it00:02
*** Sukhdev has quit IRC00:02
*** ndipanov has joined #openstack-ironic00:06
*** naohirot has joined #openstack-ironic00:09
*** Sukhdev has joined #openstack-ironic00:11
*** harshs has joined #openstack-ironic00:13
*** achanda has joined #openstack-ironic00:16
*** harlowja has joined #openstack-ironic00:20
Haomengsturivnyi: I reviewed, and agree with lucas, anastasia , and dmitry's comments, so please address and will review again, thank you00:20
*** achanda has quit IRC00:25
*** mtanino has quit IRC00:28
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: support multi-nodes input for node-set-provision-state  https://review.openstack.org/24230100:28
*** Sukhdev has quit IRC00:30
*** Sukhdev has joined #openstack-ironic00:30
*** achanda has joined #openstack-ironic00:30
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: support multi-nodes input for node-set-maintenance  https://review.openstack.org/24229900:31
*** dims_ has quit IRC00:33
openstackgerritJinxing Fang proposed openstack/ironic: refine the ironic installation guide  https://review.openstack.org/24299000:33
*** jamielennox is now known as jamielennox|away00:33
*** dims has joined #openstack-ironic00:36
openstackgerritJinxing Fang proposed openstack/ironic: refine the ironic installation guide  https://review.openstack.org/24299000:39
lintanmorning Ironic00:50
*** dims_ has joined #openstack-ironic00:51
*** dims has quit IRC00:51
TheJuliagood morning lintan01:04
lintanhi TheJulia o/01:04
*** hemna is now known as hemnafk01:08
*** Sukhdev has quit IRC01:10
openstackgerritRuby Loo proposed openstack/ironic-lib: Remove tempdir config, add tempdir parameter  https://review.openstack.org/24337801:13
rloomorning lintan01:14
lintanHello rloo o/01:15
*** jobewan has joined #openstack-ironic01:19
*** garthb_ has quit IRC01:19
*** shuquan has joined #openstack-ironic01:26
*** zhenguo has joined #openstack-ironic01:28
*** achanda has quit IRC01:29
*** jobewan has quit IRC01:52
*** shuquan has quit IRC01:59
*** dims_ has quit IRC02:01
*** jobewan has joined #openstack-ironic02:04
Haomengmorning lintan TheJulia rloo :)02:05
*** dims has joined #openstack-ironic02:05
openstackgerritZhenguo Niu proposed openstack/ironic: Validate console port number in a valid range  https://review.openstack.org/24338702:06
*** phschwartz has quit IRC02:09
*** phschwartz has joined #openstack-ironic02:09
*** lintan has quit IRC02:11
*** MattMan has quit IRC02:14
*** MattMan has joined #openstack-ironic02:14
*** jobewan has quit IRC02:16
*** praneshp has quit IRC02:22
*** lintan has joined #openstack-ironic02:22
openstackgerritZhenguo Niu proposed openstack/ironic: Check shellinabox process during stopping console  https://review.openstack.org/24231902:25
*** jobewan has joined #openstack-ironic02:29
*** jamielennox|away is now known as jamielennox02:34
*** Sukhdev has joined #openstack-ironic02:38
openstackgerritZhenguo Niu proposed openstack/ironic: Validate console port number in a valid range  https://review.openstack.org/24338702:38
*** Sukhdev has quit IRC02:38
*** hemna has joined #openstack-ironic02:40
*** baoli has quit IRC02:50
*** hemna has quit IRC02:51
*** bapalm has quit IRC02:54
*** hemna has joined #openstack-ironic03:02
*** bapalm has joined #openstack-ironic03:02
*** hemna has quit IRC03:03
*** baoli has joined #openstack-ironic03:07
*** hemna has joined #openstack-ironic03:10
*** rloo has quit IRC03:11
*** dims has quit IRC03:13
*** hemna has quit IRC03:14
*** yuanying has quit IRC03:17
*** hemna has joined #openstack-ironic03:18
*** hemna has quit IRC03:19
*** shuquan has joined #openstack-ironic03:32
*** Haomeng has quit IRC03:39
openstackgerritZhenguo Niu proposed openstack/ironic: Validate console port number in a valid range  https://review.openstack.org/24338703:44
*** shuquan has quit IRC03:50
openstackgerritMerged openstack/ironic: iboot: add wait loop for pstate to activate  https://review.openstack.org/24098903:56
*** links has joined #openstack-ironic03:59
*** harshs has quit IRC04:01
openstackgerritZhenguo Niu proposed openstack/ironic: Check shellinabox process during stopping console  https://review.openstack.org/24231904:02
*** david-lyle has joined #openstack-ironic04:07
*** jobewan has quit IRC04:09
*** yuanying has joined #openstack-ironic04:14
*** jerrygb has quit IRC04:28
*** baoli has quit IRC04:35
*** jerrygb has joined #openstack-ironic04:36
*** Sukhdev has joined #openstack-ironic04:39
*** Sukhdev has quit IRC04:48
*** amotoki has joined #openstack-ironic04:50
*** Marga_ has quit IRC04:54
*** jamielennox is now known as jamielennox|away05:01
*** Haomeng has joined #openstack-ironic05:14
*** jerrygb has quit IRC05:20
*** jerrygb has joined #openstack-ironic05:23
*** jerrygb has quit IRC05:23
*** garthb has joined #openstack-ironic05:24
*** kozhukalov_ has joined #openstack-ironic05:24
*** harshs has joined #openstack-ironic05:27
*** jobewan has joined #openstack-ironic05:27
*** Marga_ has joined #openstack-ironic05:40
*** kozhukalov_ has quit IRC05:43
*** harshs has quit IRC05:43
*** Marga_ has quit IRC05:44
*** garthb has quit IRC05:45
*** harshs has joined #openstack-ironic05:46
*** achanda has joined #openstack-ironic06:03
*** ig0r__ has quit IRC06:05
*** ukalifon1 has joined #openstack-ironic06:10
*** ig0r__ has joined #openstack-ironic06:10
*** kalpase has joined #openstack-ironic06:10
*** jerrygb has joined #openstack-ironic06:24
*** jerrygb has quit IRC06:28
*** Marga_ has joined #openstack-ironic06:35
*** baoli has joined #openstack-ironic06:38
*** Marga_ has quit IRC06:38
*** Marga_ has joined #openstack-ironic06:39
*** baoli has quit IRC06:43
zhenguojlvillal: are you around?06:51
openstackgerritChangBo Guo(gcb) proposed openstack/ironic: test:clean up wrong value in method CONF.set_override  https://review.openstack.org/24343006:53
mrdazhenguo: I think it might be a bit late for jlvillal :)07:03
zhenguomrda: oh, thanks mrda.07:03
*** kozhukalov_ has joined #openstack-ironic07:06
*** alexpilotti has joined #openstack-ironic07:15
*** jobewan has quit IRC07:18
*** jerrygb has joined #openstack-ironic07:25
*** achanda has quit IRC07:27
*** Marga_ has quit IRC07:27
*** jerrygb has quit IRC07:29
*** alexpilotti has quit IRC07:35
*** openstackgerrit has quit IRC07:46
*** openstackgerrit has joined #openstack-ironic07:47
openstackgerritZhenguo Niu proposed openstack/ironic: Check shellinabox process during stopping console  https://review.openstack.org/24231907:53
*** lsmola has joined #openstack-ironic07:58
*** zigo has quit IRC08:01
*** zigo has joined #openstack-ironic08:03
*** ifarkas has joined #openstack-ironic08:13
*** pas-ha has quit IRC08:20
*** romainh has joined #openstack-ironic08:21
*** ukalifon has joined #openstack-ironic08:23
*** ukalifon1 has quit IRC08:26
*** jerrygb has joined #openstack-ironic08:26
*** jerrygb has quit IRC08:31
*** openstackgerrit has quit IRC08:31
*** pas-ha has joined #openstack-ironic08:31
*** openstackgerrit has joined #openstack-ironic08:32
*** dtantsur|afk is now known as dtantsur08:33
dtantsurMorning Ironic08:33
*** lucas-afk is now known as lucasagomes08:36
lucasagomesmorning dtantsur08:36
dtantsuro/08:36
*** mbound has joined #openstack-ironic08:38
*** mbound has quit IRC08:38
*** itooon has joined #openstack-ironic08:41
*** itooon has quit IRC08:41
sturivnyiHaomeng: Thanks. I will.08:43
ifarkasdtantsur, morning, could you please review https://review.openstack.org/#/c/214160/ ?08:47
dtantsurmorning, sure, will do08:47
ifarkasthanks08:48
Haomengsturivnyi: welcome, and thanks for your patch08:50
*** achanda has joined #openstack-ironic08:57
*** achanda has quit IRC09:02
ifarkasdtantsur, thanks for the review, I fixed the conditional09:07
dtantsurok, will take a look soon09:07
dtantsurlucasagomes, Haomeng, please see a ML thread about the OSC provision command, would be interesting to hear your opinions09:12
lucasagomesdtantsur, will take a look09:12
openstackgerritJinxing Fang proposed openstack/ironic: refine the ironic installation guide  https://review.openstack.org/24299009:18
*** derekh has joined #openstack-ironic09:21
*** mbound has joined #openstack-ironic09:25
openstackgerritRicardo Carrillo Cruz proposed openstack/bifrost: Fix dnsmasq restart requiring sudo privileges  https://review.openstack.org/24348209:25
*** rcarrillocruz has joined #openstack-ironic09:25
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node  https://review.openstack.org/23950609:27
*** jerrygb has joined #openstack-ironic09:27
*** subscope has joined #openstack-ironic09:29
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add agent_iboot entrypoint  https://review.openstack.org/24312309:31
*** lintan has quit IRC09:31
*** jerrygb has quit IRC09:32
openstackgerritZhenguo Niu proposed openstack/ironic: Validate console port number in a valid range  https://review.openstack.org/24338709:33
*** lintan has joined #openstack-ironic09:33
*** harshs has quit IRC09:34
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add documentation how to run functional tests  https://review.openstack.org/24086609:39
*** jistr has joined #openstack-ironic09:45
*** athomas has quit IRC09:47
*** kozhukalov_ has quit IRC09:48
zhenguolucasagomes: can you please have a look at https://review.openstack.org/#/c/240781/09:51
openstackgerritTan Lin proposed openstack/ironic: Isolate ir-api from DB  https://review.openstack.org/24349709:51
lucasagomeszhenguo, hi there, will do09:51
lucasagomesgimme some min09:51
zhenguolucasagomes: thanks09:52
*** mattymo has joined #openstack-ironic09:54
mattymoyuriyz, ping09:54
*** athomas has joined #openstack-ironic09:54
yuriyzhi mattymo09:54
*** naohirot has quit IRC09:54
mattymoyuriyz, do you know about obtaining a list of physical interfaces in python?09:55
mattymoromcheg pointed me to you^09:55
yuriyzone moment09:55
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port  https://review.openstack.org/24094409:56
yuriyzmattymo i use parsing output of "ip" utility https://github.com/openstack/fuel-agent/blob/master/fuel_agent/utils/utils.py#L351-L36609:58
yuriyzdon't know a better way09:58
mattymothat's just grabbing 'ip' of an interface09:59
yuriyzyes this example for mac-ip09:59
mattymothis doesn't have anything in common with physical/virtual interfaces09:59
*** kozhukalov_ has joined #openstack-ironic09:59
*** zhenguo has quit IRC10:00
*** achanda has joined #openstack-ironic10:00
openstackgerritJinxing Fang proposed openstack/ironic: refine the ironic installation guide  https://review.openstack.org/24299010:01
yuriyzmattymo probably you can use "ip link list" and filter by names or by macs (virtual interfaces usually have "special" macs)10:04
mattymoyuriyz, that's not a very pythonic route.. nothing using a python lib?10:04
yuriyzmattymo AFAIK no because this is OS-specific10:05
*** achanda has quit IRC10:05
*** ijw has joined #openstack-ironic10:07
*** mkovacik has joined #openstack-ironic10:19
Haomengdtantsur: ok, let me check10:21
pas-hamattymo, https://pypi.python.org/pypi/netifaces - could that help?10:23
*** jamielennox|away is now known as jamielennox10:24
*** Marga_ has joined #openstack-ironic10:25
*** subscope has quit IRC10:26
*** jerrygb has joined #openstack-ironic10:28
mattymopas-ha, ok that lists all. how do you tell if it is a real physical one or a virtual?10:28
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Follow up patch for 8c3e102fc5736bfcf98525ebab59b6598a69b428  https://review.openstack.org/24352110:29
pas-hamattymo, what is then a trait that differs for virtual and physical ones and can be asserted? I'm not sure there is a generic one, you'd have to make some naming/pattern assumptions I guess10:31
*** jerrygb has quit IRC10:34
*** mgould has joined #openstack-ironic10:34
*** ig0r__ has quit IRC10:40
*** electrofelix has joined #openstack-ironic10:43
mattymopas-ha, yeah10:50
*** baoli has joined #openstack-ironic10:56
sambettsMorning all o/11:00
*** baoli has quit IRC11:01
*** dims has joined #openstack-ironic11:05
openstackgerritMerged openstack/ironic-specs: Add ShellinaboxConsole to SSH driver  https://review.openstack.org/24078111:09
dtantsurmorning sambetts11:13
*** dims has quit IRC11:21
*** romainh has left #openstack-ironic11:21
*** dims has joined #openstack-ironic11:22
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client: Allow several UUID's in 'introspection start'  https://review.openstack.org/24354111:25
sambettsMorning dtantsur, I responded to your OSC email :)11:26
dtantsurcool!11:26
*** sinval has joined #openstack-ironic11:36
sinvalgood morning o/11:36
dtantsurmorning sinval11:37
dtantsursambetts, ifarkas, wdyt about https://blueprints.launchpad.net/ironic-inspector/+spec/list-introspections ?11:38
ifarkasifarkas, +1, that would help debugging11:41
*** subscope has joined #openstack-ironic11:41
ifarkasdtantsur, ^ :-)11:41
*** jaypipes has joined #openstack-ironic11:42
openstackgerritMiles Gould proposed openstack/ironic: Get mandatory patch attrs from WSME properties  https://review.openstack.org/24020211:52
*** achanda has joined #openstack-ironic11:54
*** marios_ is now known as marios11:55
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Properly order node deletion from cache  https://review.openstack.org/24355311:55
dtantsurTheJulia, please have a look if this ^^ fixes your problem with rerunning introspection11:55
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add FirmwareInterface for BIOS/Firmware configuration  https://review.openstack.org/24317612:02
openstackgerritMerged openstack/ironic: Validate the input of properties of nodes.  https://review.openstack.org/21550512:02
*** thrash|g0ne is now known as thrash12:05
*** romainh has joined #openstack-ironic12:07
openstackgerritJinxing Fang proposed openstack/ironic: refine the ironic installation guide  https://review.openstack.org/24299012:10
*** lucasagomes is now known as lucas-hungry12:13
*** ps231 has joined #openstack-ironic12:15
*** arturas has joined #openstack-ironic12:16
sambettsdtantsur: I think we should tackle it as part of this https://blueprints.launchpad.net/ironic-inspector/+spec/node-states12:21
ps231hello i was wondering if anyone could help with the guidelines for committing to the ironic project?12:22
dtantsursambetts, well, they're not strictly related, I'd say12:23
*** achanda has quit IRC12:23
sambettsps231: https://wiki.openstack.org/wiki/How_To_Contribute12:23
dtantsurps231, hi, everyone here, I guess :) what's your problem?12:23
sambettsdtantsur: I just think we'll end up re-writing the work we do for list-introspections when we change how we store the states12:24
*** smoriya has quit IRC12:24
dtantsursambetts, not sure, but anyway I plan to do states work first :)12:24
ps231the work to be committed is related the ironic inspector, i was wondering if I need to include the same functionallity to IPA as I already have for a normal baremetal boot?12:25
sambettsWhat sort of change is it, have you added additional functionality to the DIB version of the inspector ramdisk?12:28
dtantsurps231, could you give us more background. What are you trying to achieve?12:28
ps231basically it will add add to the current discovery, by adding some network information onto the ironic port12:29
ps231there is a new element to the discoverd ramdisk, and then the inspector plugin simply makes use of this new information12:30
dtantsurps231, is this work synced with the ongoing ironic-neutron collaboration on tenant network separation?12:31
*** jerrygb has joined #openstack-ironic12:32
sambettshttps://review.openstack.org/#/c/241242/ <- current LLDP based work12:32
dtantsuroh, I have to review it12:33
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add Sanity tests for testing actions with Driver  https://review.openstack.org/24123012:36
ps231sambetts, yeah thats the one12:36
*** jerrygb has quit IRC12:37
ps231gonna be adjusting a few things for it, but wondering the need for IPA to be included12:37
sambettsironic inspector now supports the use of IPA as the inspection ramdisk, so if you want your information to be returned in cases where people are using IPA for inspection that you'll need to check if IPA supports it and add it if it doesn'y12:38
sambettsdoesn't *12:38
dtantsurps231, IPA has some functions for LLDP, but they're not wired in. you have to update https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/inspector.py I guess12:39
ps231dtantsur, yeah most probably12:40
sambettsps231: are you William?12:42
ps231no, philip12:42
sambettsah, ok, just trying to match lp names to IRC nicks :)12:43
ps231hehe yeah i should have made it a bit clearer :P12:43
*** sambetts is now known as sambetts-hungry12:45
ifarkasdtantsur, could you please review https://review.openstack.org/#/c/230445/ as well?12:46
dtantsursure12:46
*** romainh has quit IRC12:51
*** dims has quit IRC12:53
*** baoli has joined #openstack-ironic12:53
mkovacikguys, is https://blueprints.launchpad.net/ironic-inspector/+spec/abort-introspection still relevant? I'd take it.12:54
dtantsurmkovacik, it is, feel free to12:55
mkovacikdtantsur, ack12:55
*** romainh has joined #openstack-ironic12:56
*** dims has joined #openstack-ironic12:56
ifarkasdtantsur, thanks for the review! one question: what do you mean by a reference to the namedtuple in the docstring?12:56
*** romainh has left #openstack-ironic12:57
dtantsurifarkas, I was thinking about just mentioning like "returns Job object" :) not sure about this one though, -1 is for another issue12:58
ifarkasdtantsur, ok, makes sense12:58
*** dprince has joined #openstack-ironic13:00
ifarkasdtantsur, it's fixed13:06
dtantsurthanks13:06
dtantsurmkovacik, feel free to assign yourself to https://blueprints.launchpad.net/ironic-inspector/+spec/abort-introspection or lemme know if you don't have enough rights to do it13:07
*** kalpase has left #openstack-ironic13:08
mkovacikdtantsur, seems I could use some privileges elevation ;)13:08
openstackgerritDmitry Tantsur proposed openstack/ironic: Rename test_conductor_utils.py to test_utils.py  https://review.openstack.org/24357513:09
dtantsurmkovacik, what's your lp nick? seems like only admins and the owner can assign blueprints13:09
mkovacikdtantsur, vetrisko13:10
dtantsurdone13:11
mkovacikdtantsur, ack, thanks13:12
openstackgerritDmitry Tantsur proposed openstack/ironic: Move common code from ironic.conductor.manager to ironic.conductor.utils  https://review.openstack.org/23251713:12
*** lucas-hungry is now known as lucasagomes13:15
*** subscope has quit IRC13:17
*** achanda has joined #openstack-ironic13:23
*** jerrygb has joined #openstack-ironic13:24
*** baoli has quit IRC13:25
*** daemontool has quit IRC13:25
*** baoli has joined #openstack-ironic13:26
*** jerrygb has quit IRC13:29
*** achanda has quit IRC13:30
*** sambetts-hungry is now known as sambetts13:31
*** subscope has joined #openstack-ironic13:34
*** absubram has joined #openstack-ironic13:36
TheJuliadtantsur: I will look when I wake up :)13:36
*** jefabo has joined #openstack-ironic13:37
*** absubram_ has joined #openstack-ironic13:37
* dtantsur hands a big cup of coffee to TheJulia 13:37
TheJuliadtantsur: merci beaucoup13:39
*** absubram has quit IRC13:40
*** absubram_ is now known as absubram13:40
*** trown|outttypeww is now known as trown13:44
devanandamorning, all13:51
mgoulddevananda, morning!13:52
dtantsurmorning devananda13:56
lucasagomesmgould, devananda  TheJulia  sambetts ifarkas morning!13:57
mgouldlucasagomes, morning!13:57
mgouldwait, you're in the same time zone as me, aren't you? Afternoon!13:57
lucasagomesmgould, hah yes! But we always can use the UGT13:58
lucasagomeshttp://www.total-knowledge.com/~ilya/mips/ugt.html13:58
mgouldaha, I hadn't encountered that convention before13:59
* devananda is waiting 4hrs for a flight to D.C.14:02
TheJulia:(14:03
TheJuliagood morning devananda lucasagomes14:03
devanandaAA doesn't allow same day flight changes any more, apparently, unless you pay them more money14:03
TheJuliaugh...14:03
lucasagomesmgould, yeah, UGT rocks14:03
dtantsurI was waiting 6hrs in Frankfurt when going back from Vancouver... extremely boring14:07
*** links has quit IRC14:08
*** [1]cdearborn has joined #openstack-ironic14:11
*** jdkeenleyside has joined #openstack-ironic14:12
*** rloo has joined #openstack-ironic14:16
*** daemontool has joined #openstack-ironic14:18
*** absubram has quit IRC14:18
openstackgerritZhenguo Niu proposed openstack/ironic: Add console feature to ssh driver  https://review.openstack.org/24145314:18
*** Marga_ has quit IRC14:19
*** absubram has joined #openstack-ironic14:19
*** arturas has quit IRC14:22
*** Marga_ has joined #openstack-ironic14:23
*** absubram has quit IRC14:23
*** ndipanov has quit IRC14:24
jdkeenleysidei am trying to install ironic-bifrost on Ubuntu 15.10 but ironic-conductor service fails to start with error "file not found". I don't see anything in /etc/systemd related to ironic. Any suggestions?14:30
TheJuliaHmmm14:30
TheJuliaDoes the install's output show anything with regards to systemd?14:31
*** mkovacik_ has joined #openstack-ironic14:31
TheJuliajdkeenleyside: we've not explictly tested on 15.10, sounds like I need to fire it up14:32
*** mkovacik has quit IRC14:34
sambettso/ lucasagomes14:35
*** ukalifon has quit IRC14:36
*** [1]cdearborn has quit IRC14:38
jdkeenleysideIs bifrost tested on Ubuntu 15.04?14:39
*** aweeks has quit IRC14:41
*** aweeks has joined #openstack-ironic14:42
openstackgerritRuby Loo proposed openstack/ironic-lib: Remove tempdir config, add tempdir parameter  https://review.openstack.org/24337814:45
*** arturas has joined #openstack-ironic14:47
liliarsmorning all (:14:48
openstackgerritZhenguo Niu proposed openstack/ironic: Check shellinabox process during stopping console  https://review.openstack.org/24231914:48
krtaylormorning ironic14:50
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port  https://review.openstack.org/24094414:50
* krtaylor is sick today, but will try to crank out a revision on the ci spec14:51
*** Marga_ has quit IRC14:52
liliarsmorning krtaylor, oh hope you feel better soon "/14:56
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add simple table structure Ironic CLI tests  https://review.openstack.org/23857714:57
*** daemontool has quit IRC14:58
*** amotoki_ has joined #openstack-ironic15:00
*** links has joined #openstack-ironic15:01
*** mbound has quit IRC15:05
*** links has quit IRC15:05
devanandajroll: is there a reason you can think of to separate node.properties from node.capabilities, as we move to storing these in key/value format?15:07
dtantsurdevananda, you mean why do it or why NOT do it? :) currently updating capabilities is racy for all 3rdparties (inspector is one of them)15:08
devanandadtantsur: racy how?15:08
dtantsurdevananda, to add new capability you have to fetch all, parse, update and set back15:08
dtantsur2 processes doing it in parallel will conflict15:09
devanandaahh. yes.15:09
devanandawe could resolve that by using etag id's in API responses15:09
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add Sanity tests for testing actions with Driver  https://review.openstack.org/24123015:10
dtantsurdevananda, also we won't have to copy-paste capability-handling code everywhere :) I've seen it in ironic itself, inspector and tripleo already15:10
*** mbound has joined #openstack-ironic15:10
devanandadtantsur: https://en.wikipedia.org/wiki/HTTP_ETag15:10
dtantsuryeah, I know this thing, thanks15:10
devanandak k15:10
*** mbound has quit IRC15:11
dtantsuractually, having etags is a good thing even if we split away capabilities..15:12
*** mbound has joined #openstack-ironic15:12
devanandaupdates to any resource will be racy until we implement etags, so I don't think that raciness affects my question15:12
devanandaand yah, I think having etags would be a good thing in general :)15:12
*** ijw has quit IRC15:13
dtantsurwell, then my answer to the initial question would be "for simplicity for 3rd parties using capabilities" :)15:13
devanandadtantsur: so, if I follow, you'd prefer to keep capabilities as a separate resource (separate from properties) because that is simpler for 3rd parties?15:14
dtantsuryep15:15
devanandacool15:15
dtantsuranother question for folks: what would you think about operations in client that operate on all nodes or all nodes in particular state?15:18
dtantsur(this question is driven by the current tripleo discussions)15:18
devanandadtantsur: wdyt about search API, though? separating the two means potential duplication of keys and more complex semantics for searching15:18
dtantsurdevananda, it depends on whether we expect people to use capabilities in search. if we do, then having them inside a string is out of question15:19
devanandawe do15:19
devanandai'm rewriting the spec that moves them out of the JSON fields15:19
devanandaso assume they wont be in a string, for this discussion, but rather in a set of (key,value) pairs associated with the node15:19
dtantsurmmm, maybe I didn't get your right initially15:20
*** jefabo has quit IRC15:20
devanandastoring properties separately from capabilities, for the same node, means that we could have node.properties['foo']='horse' and node.capabilities['foo']='camel'15:20
dtantsurdevananda, do you suggest we just create a new table and put both properties and capabilities there?15:20
devanandawhich, frankly, we could have today15:20
devanandadtantsur: yah. have you seen the spec / discussion on this already?15:21
devananda20464115:21
dtantsurdevananda, I've seen this spec before the summit, I guess it changed since then15:21
devanandadtantsur: I posted some lengty comments after the summit. and I'm working on the next rev of the spec based on those now15:21
devanandadtantsur: I think we need separate tables for properties & capabilities15:22
dtantsur+115:22
devanandahowever, I'm fretting over the REST API implications of searching both15:22
devanandawe essentially have two namespaces for metadata, and as a user, I don't necessarily care about that15:23
devananda"find me a node that has 64GB RAM and an NVIDIA GPU"15:23
* dtantsur rereads the spec and the comments15:23
devanandais a reasonable request, IMO, but to satisfy that, the user will need to know that RAM is stored in 'properties' and 'video card' is stored in 'capabilities'15:24
*** achanda has joined #openstack-ironic15:27
lucasagomesdtantsur, btw https://review.openstack.org/#/c/232571/15:29
lucasagomesthis allows having capabilites as a dictionary15:29
lucasagomesnot sure if we want to go that way tho, if we split the capabilities it won't be needed15:29
dtantsurwell, this conflicts with Jim's suggestion15:29
*** achanda has quit IRC15:32
devanandado you have an example of current non-dict capabilities value?15:32
*** jerrygb has joined #openstack-ironic15:33
* rloo always gets confused as to what is a 'property' vs a 'capability'.15:33
devanandaalso, wdyt of: 1) matching only by exact match of any property or capability value, or 2) allowing user to specify operators like <, >, !=, etc15:33
devanandarloo: me too15:33
devanandarloo: afaict, the only difference is the way Nova handles the namespacing15:34
rlooperhaps someone could clarify/describe them both so it is clear.15:34
devanandaand so we've been putting nearly everything in capabilities just so that Nova filters can match flavors <> nodes15:34
rloootherwise, I'd vote for having one table that includes both15:34
rloooh... that nova thing.15:35
dtantsuryeah, having only one table does not sound too bad.. I initially thought you plan on having capabilities under properties as a JSON15:35
rloodevananda: you have two +2s for one table :)15:36
*** mbound has quit IRC15:36
devanandarloo: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L31515:36
rloodevananda: we're going to change the scheduling part, aren't we? I don't know the details of the multiple compute hosts stuff.15:37
*** jerrygb has quit IRC15:37
devanandarloo: exactly15:37
rlooso let's do it 'right-er'15:37
devanandathis whole thing is to allow us to stop relying on the nova scheduler. iow, we won't need to do this matching at L31515:37
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node  https://review.openstack.org/23950615:37
dtantsurall looks good, I'm only worried about multi-node claims. I hope we don't plan to have multi-node locks for them..15:38
devanandahowever, we need to provide similar support for flavors-matching-nodes15:38
devanandawhich the current rev of the spec doesn't cover at all15:38
*** bradjones is now known as bradjones|away15:38
devanandadtantsur: I think we need multi-node claims. but I think it's fine to do without changing TaskManager at all15:38
devanandaok - I'll continue with a single table15:38
dtantsuryeah, let's address the current comments and see how it shapes15:39
devanandahopefully I'll finish this draft by EOD. it's going to be very different than thecurrent rev15:39
dtantsursounds promising :) I'll have a look tomorrow morning then15:40
*** garthb has joined #openstack-ironic15:41
*** mbound has joined #openstack-ironic15:45
*** arturas has quit IRC15:47
sturivnyiHaomeng, lucasagomes , dtantsur: I've fixed, please review https://review.openstack.org/#/c/238577/15:48
liliarslucasagomes, dtantsur: had the same issue formatting capabilities into a dict in lots of places when developing the ov driver, sooo there is this function we put in utils.py to help other folks at least handle such case with less duplicated code https://github.com/openstack/ironic/blob/7680d384b8cc0b768152af0892453f258c50dc24/ironic/drivers/utils.py#L22215:48
liliarsmight be helpful for now (:15:48
lucasagomesliliars, nice one! Thanks15:48
lucasagomessturivnyi, will do in a min15:48
sturivnyilucasagomes: Great. Thanks =)15:49
dtantsurliliars, it won't help in inspector and tripleo cases, as they are separate projects. but yeah, we've copy pasted it some time ago :)15:49
devanandaliliars: thanks. will help inthe data migration I'm going to need to do15:49
*** subscope has quit IRC15:51
*** mjturek1 has quit IRC15:55
*** mtanino has joined #openstack-ironic15:57
*** mjturek1 has joined #openstack-ironic15:58
*** daemontool has joined #openstack-ironic16:03
liliarslucasagomes, dtantsur, devananda: cool! glad to help (: also, if we change capabilities to not be a string anymore I'm oh so happily +1 it. will take a look at the spec tomorrow, thanks! (:16:05
*** jerrygb has joined #openstack-ironic16:07
*** hemnafk is now known as hemna16:09
*** jerrygb has quit IRC16:12
*** jerrygb has joined #openstack-ironic16:13
*** jerrygb_ has joined #openstack-ironic16:15
*** degorenko is now known as _degorenko16:15
*** jerrygb has quit IRC16:17
*** praneshp has joined #openstack-ironic16:17
*** amotoki_ has quit IRC16:19
openstackgerritMerged openstack/ironic: Rename test_conductor_utils.py to test_utils.py  https://review.openstack.org/24357516:23
*** smcginnis has left #openstack-ironic16:25
openstackgerritJulia Kreger proposed openstack/bifrost: Initial support for installation of ironic-inspector  https://review.openstack.org/22993016:26
openstackgerritJulia Kreger proposed openstack/bifrost: Initial addition of an inspection module  https://review.openstack.org/23274516:28
*** jamielennox has quit IRC16:28
openstackgerritJulia Kreger proposed openstack/bifrost: Additional inspection role and adding role to NV job script  https://review.openstack.org/23274616:28
*** arturas has joined #openstack-ironic16:34
NobodyCammornign Ironic folks16:36
mgouldNobodyCam, morning!16:36
NobodyCam:)16:37
liliarsmorning NobodyCam (:16:37
NobodyCammornign :)16:37
openstackgerritMerged openstack/bifrost: Minor CONTRIBUTNG text fix  https://review.openstack.org/24210816:38
*** subscope has joined #openstack-ironic16:38
*** arturas has quit IRC16:39
*** arturas has joined #openstack-ironic16:39
*** subscope has quit IRC16:39
*** ps231 has quit IRC16:40
dtantsurmorning NobodyCam16:40
*** jamielennox has joined #openstack-ironic16:41
johnthetubaguyjroll: just looking at the nova spec on tenant networking support, does that change radically post summit?16:44
*** arturas has quit IRC16:44
*** subscope has joined #openstack-ironic16:44
JayFjohnthetubaguy: we're in castle this week so he might be higher latency than usual (as you can see, as he didn't respond in 3 seconds, :P)16:44
johnthetubaguyJayF: heh true, I forgot about that16:45
*** mgoddard_ has joined #openstack-ironic16:46
johnthetubaguyJayF: it would be fun if we had tele presence device things that remote folks wonder around the place...16:46
JayFjohnthetubaguy: are /you/ in the castle? If so you're welcome to come say hi to the team :)16:46
johnthetubaguyJayF: no, afraid not, not been there for a while actually16:47
*** mkovacik_ has quit IRC16:47
*** mgoddard has quit IRC16:48
openstackgerritJulia Kreger proposed openstack/bifrost: Revised prep role README.md for clarity  https://review.openstack.org/23061216:51
openstackgerritMerged openstack/ironic: Follow up patch for 8c3e102fc5736bfcf98525ebab59b6598a69b428  https://review.openstack.org/24352116:54
*** subscope has quit IRC16:54
*** baoli has quit IRC16:55
*** mgoddard has joined #openstack-ironic16:55
*** mgoddard_ has quit IRC16:55
NobodyCamhey dtantsur :) and morning JayF16:58
sambettsHi and bye NobodyCam o/ cya everyone I'm calling it a day17:03
*** trown is now known as trown|brb17:03
dtantsurg'night sambetts17:03
dtantsurI'm also calling it a day for now17:03
*** mbound has quit IRC17:03
*** daemontool has quit IRC17:04
mgouldg'night sambetts, dtantsur17:05
*** arturas has joined #openstack-ironic17:05
lucasagomesNobodyCam, morning17:08
lucasagomesdtantsur, sambetts g'night17:08
*** dtantsur is now known as dtantsur|afk17:09
*** mbound has joined #openstack-ironic17:12
*** baoli has joined #openstack-ironic17:13
*** garthb_ has joined #openstack-ironic17:14
*** garthb has quit IRC17:15
*** trown|brb is now known as trown17:16
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Ansible deploy driver  https://review.openstack.org/24194617:17
*** mbound has quit IRC17:18
*** dims has quit IRC17:27
*** dims has joined #openstack-ironic17:28
*** achanda has joined #openstack-ironic17:30
jrolljohnthetubaguy: it has not changed significantly, that I'm aware of17:31
lucasagomesfolks I'm calling a day17:34
lucasagomeshave a g'night everyone!17:34
jrollnight lucas :)17:35
*** achanda has quit IRC17:36
mgouldgnight lucasagomes!17:37
liliarsgood night lucasagomes, hopefully *not* lots of jet lag to still catch up on ((:17:37
lucasagomesliliars, mostly good! I will try to not fall sleep before 00:00 today17:40
lucasagomesasleep*17:40
*** lucasagomes is now known as lucas-afk17:40
liliarsso I'm thinking about adding a section in the install docs to include guidelines on how to setup swift as a storage backend in the ironic side (for agent deployment things), as we need our users to know this but I think other drivers might too17:43
liliarsI saw some documentation related to that in the ilo drivers docs, tried to ping someone here but the ones I remember are not online (: does someone else have an opinion about that? would it be interesting for the install guide17:44
liliars?17:44
*** jistr has quit IRC17:48
rlooliliars: if you think the info would be useful and it isn't documented eg in swift documentation, then sure.17:48
liliarsrloo: I saw documentation regarding this in the glance side, but not in the ironic side (only the ilo specific one). will just recheck if swift has something on it, thanks17:52
*** BobBall is now known as BobBall_AWOL17:54
*** derekh has quit IRC17:55
NobodyCammorning rloo jroll17:55
jrollhiya NobodyCam :)17:56
jrollliliars: that seems useful - funny story, that's exactly what I just set up for a new thing and I didn't know it off the top of my head17:57
BadCubhowdy folks!17:57
devanandamornin, BadCub !17:58
BadCubhey hey devananda hows things and stuff?17:58
krotscheckDurm dee durm dee durm.17:58
devanandaBadCub: airports. that is all.17:58
BadCubfun fun!17:59
krotscheckSo, anyone want to review javascript? We need at least one +1 for betherly to +A this patch chain.https://review.openstack.org/#/c/229055/17:59
* BadCub looks at ^^17:59
NobodyCam:)18:00
liliarsjrool: whoop, I'm on it then! (:18:00
liliarsmorning BadCub18:01
BadCubhey hey liliars :)18:01
*** lsmola has quit IRC18:01
*** garthb_ has quit IRC18:02
*** garthb_ has joined #openstack-ironic18:02
*** krtaylor has quit IRC18:03
*** willsama has joined #openstack-ironic18:06
*** jerrygb has joined #openstack-ironic18:06
rloomorning NobodyCam, jroll, BadCub and everyone else18:06
BadCubhey hey rloo :)18:06
*** jerrygb_ has quit IRC18:09
*** rebase has joined #openstack-ironic18:09
rlooliliars: btw, awhile ago Dmitry had started a thread about re-org/splitting up the install guide18:09
rlooliliars: http://lists.openstack.org/pipermail/openstack-dev/2015-September/074196.html18:11
krotscheckBadCub: Danke :)18:11
rlooliliars: not sure what the status is, but might help wrt where to put the swift info if you decide to add it18:11
BadCubkrotscheck: yw :)18:11
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add Sanity tests for testing actions with Driver  https://review.openstack.org/24123018:15
openstackgerritMerged openstack/python-ironicclient: Add simple table structure Ironic CLI tests  https://review.openstack.org/23857718:15
*** krtaylor has joined #openstack-ironic18:16
liliarsrloo: oh ok, will take a look at it. thanks (:18:17
*** trown is now known as trown|flu_shot18:18
liliarsrloo: yeah just saw it, I was also thinking the install docs are already a bit long.. seems like a good idea, thanks for pointing it out18:21
rlooliliars: yw. I am hoping that maybe you'll take up the challenge :)18:21
rlooliliars: although it would be good to have some idea of how/if/when our documentation can be incorporated with the rest/main openstack docs.18:22
*** harshs has joined #openstack-ironic18:23
*** kozhukalov_ has quit IRC18:23
*** achanda has joined #openstack-ironic18:25
krotscheckWOOO BADCUB THANKOOOOOU18:25
sinvallol18:25
krotscheckbetherly: I have another +1 on https://review.openstack.org/#/c/229055/ :)18:25
BadCubyw  krotscheck18:25
*** thrash is now known as thrash|biab18:27
* mgould -> home; good night!18:27
liliarsrloo: yeah we have to plan that. but also I think there are some things we def *need* to work on before incorporating the docs, so lets align deadlines with needs18:27
liliarsrloo: plus I'm glad to take a good challenge (:18:28
*** mgould has quit IRC18:28
liliarsnight mgould18:28
rlooliliars: thanks liliars! :D18:28
*** bnemec has quit IRC18:29
*** garthb__ has joined #openstack-ironic18:31
*** garthb_ has quit IRC18:31
*** praneshp_ has joined #openstack-ironic18:36
liliarsrloo: np (:18:36
*** arturas has quit IRC18:36
*** praneshp has quit IRC18:38
*** praneshp_ is now known as praneshp18:38
*** kozhukalov_ has joined #openstack-ironic18:39
*** arturas has joined #openstack-ironic18:41
*** achanda has quit IRC18:41
*** arturas has quit IRC18:47
*** arturas has joined #openstack-ironic18:48
*** arturas has quit IRC18:50
*** arturas has joined #openstack-ironic18:50
*** kozhukalov_ has quit IRC18:54
openstackgerritMerged openstack/ironic: Refactor iscsi_ilo driver to use new boot interface  https://review.openstack.org/21653818:54
*** Sukhdev has joined #openstack-ironic18:56
*** arturas has quit IRC18:56
*** arturas has joined #openstack-ironic18:57
*** thrash|biab is now known as thrash18:57
*** Marga_ has joined #openstack-ironic18:57
*** baoli has quit IRC18:58
*** trown|flu_shot is now known as trown18:59
*** arturas has quit IRC19:01
*** jerrygb_ has joined #openstack-ironic19:01
*** arturas has joined #openstack-ironic19:02
*** achanda has joined #openstack-ironic19:03
*** baoli has joined #openstack-ironic19:03
*** jerrygb has quit IRC19:04
*** caiobo has joined #openstack-ironic19:05
*** jerrygb has joined #openstack-ironic19:05
*** jerrygb_ has quit IRC19:08
*** achanda has quit IRC19:09
*** achanda has joined #openstack-ironic19:12
*** achanda has quit IRC19:13
*** electrofelix has quit IRC19:13
*** dprince has quit IRC19:21
*** ijw has joined #openstack-ironic19:21
*** _degorenko has quit IRC19:22
*** _degorenko has joined #openstack-ironic19:27
*** subscope has joined #openstack-ironic19:39
*** amotoki has quit IRC19:42
openstackgerritJarrod Johnson proposed openstack/pyghmi: Avoid abandoning retry when SOL packet is not ACK  https://review.openstack.org/24379719:46
openstackgerritMerged openstack/pyghmi: Avoid abandoning retry when SOL packet is not ACK  https://review.openstack.org/24379719:51
*** baoli has quit IRC19:51
*** daemontool has joined #openstack-ironic19:51
*** rebase has quit IRC19:52
*** arturas has quit IRC19:56
*** baoli has joined #openstack-ironic19:56
*** arturas has joined #openstack-ironic19:58
*** baoli has quit IRC20:02
*** athomas has quit IRC20:03
*** baoli has joined #openstack-ironic20:03
*** baoli has quit IRC20:04
*** baoli has joined #openstack-ironic20:05
*** baoli has quit IRC20:08
*** ijw has quit IRC20:10
mrdaMorning Ironic20:16
BadCubhey hey mrda20:17
sinvalgood morning mrda20:18
*** mbound has joined #openstack-ironic20:18
*** r-daneel has joined #openstack-ironic20:18
mrdaMorning BadCub and sinval20:19
*** jpeeler has joined #openstack-ironic20:22
*** lifeless has quit IRC20:26
*** dprince has joined #openstack-ironic20:29
openstackgerritSinval Vieira Mendes Neto proposed openstack/ironic: Add whole disk image creation command to Installation Guide  https://review.openstack.org/24381920:41
*** baoli has joined #openstack-ironic20:42
*** bnemec has joined #openstack-ironic20:47
*** arturas has quit IRC20:48
*** Sukhdev has quit IRC20:49
*** jerrygb has quit IRC20:51
*** kozhukalov_ has joined #openstack-ironic20:53
*** jerrygb has joined #openstack-ironic20:59
*** jerrygb has quit IRC20:59
*** jerrygb has joined #openstack-ironic20:59
*** kozhukalov_ has quit IRC21:01
*** ifarkas has quit IRC21:03
*** lifeless has joined #openstack-ironic21:10
*** sacharya has joined #openstack-ironic21:10
NobodyCammorning mrda21:10
*** jerrygb_ has joined #openstack-ironic21:12
*** jerrygb has quit IRC21:16
mrdahey NobodyCam21:16
*** jerrygb_ is now known as jerrygb21:17
*** kozhukalov_ has joined #openstack-ironic21:17
*** Marga_ has quit IRC21:22
NobodyCam:)21:23
openstackgerritJulia Kreger proposed openstack/bifrost: WIP: Set variables for Ubuntu 15.10  https://review.openstack.org/24382721:24
*** jamielennox is now known as jamielennox|away21:25
*** kozhukalov_ has quit IRC21:26
*** rebase has joined #openstack-ironic21:26
*** jdkeenleyside has left #openstack-ironic21:29
openstackgerritJulia Kreger proposed openstack/bifrost: Initial support for installation of ironic-inspector  https://review.openstack.org/22993021:30
*** dprince has quit IRC21:31
openstackgerritJulia Kreger proposed openstack/bifrost: Initial addition of an inspection module  https://review.openstack.org/23274521:33
*** baoli has quit IRC21:38
*** sacharya has quit IRC21:44
*** rvasilets___ has joined #openstack-ironic21:46
*** rvasilets___ has quit IRC21:46
*** rvasilets___ has joined #openstack-ironic21:47
*** trown is now known as trown|outttypeww21:49
*** Marga_ has joined #openstack-ironic21:51
*** jerrygb has quit IRC21:57
*** thrash is now known as thrash|bbl22:17
*** jamielennox|away is now known as jamielennox22:24
*** mkovacik has joined #openstack-ironic22:25
*** boris-42 has joined #openstack-ironic22:30
*** baoli has joined #openstack-ironic22:32
*** rebase has quit IRC22:37
*** rebase has joined #openstack-ironic22:37
*** spandhe has joined #openstack-ironic22:44
*** caiobo_ has joined #openstack-ironic22:51
*** harshs has quit IRC22:52
*** BobBall_AWOL has quit IRC22:53
*** BobBall_AWOL has joined #openstack-ironic22:55
*** Sukhdev has joined #openstack-ironic23:06
*** mkovacik has quit IRC23:07
*** jerrygb has joined #openstack-ironic23:08
*** dims has quit IRC23:15
Haomengsturivnyi: ok , thanks23:15
*** caiobo_ has quit IRC23:17
*** garthb__ has quit IRC23:22
*** garthb__ has joined #openstack-ironic23:23
davidlenwellhttps://youtu.be/6GVJpOmaDyU23:45
*** Sukhdev has quit IRC23:53

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