Friday, 2013-11-01

« Thursday, 2013-10-31 Index Saturday, 2013-11-02 »
NobodyCamhave a safe halloween everyone!00:13
*** matsuhashi has joined #openstack-ironic00:25
HaomengEnjoy halloween everyone:)00:26
Haomeng:)00:51
*** nosnos has joined #openstack-ironic01:09
*** sjing has joined #openstack-ironic01:24
*** jbjohnso has quit IRC01:32
sjingMorning, Ironic01:45
Haomengg'morning Jing:)01:52
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Change Port create API to EAFP  https://review.openstack.org/5453702:08
*** matsuhashi has quit IRC02:43
*** matsuhashi has joined #openstack-ironic02:43
*** matsuhas_ has joined #openstack-ironic02:44
*** matsuhashi has quit IRC02:44
*** michchap has quit IRC03:04
*** michchap has joined #openstack-ironic03:05
*** matsuhas_ has quit IRC03:20
*** matsuhashi has joined #openstack-ironic03:21
*** matsuhashi has quit IRC03:26
*** rloo has quit IRC03:43
*** matsuhashi has joined #openstack-ironic04:28
*** rushiagr has joined #openstack-ironic04:38
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Expose created_at/updated_at properties in the REST API  https://review.openstack.org/5362706:03
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Expose created_at/updated_at properties in the REST API  https://review.openstack.org/5362706:20
*** michchap has quit IRC06:30
openstackgerritHaomeng,Wang proposed a change to openstack/python-ironicclient: Enable created_at/updated_at for port-show/chassis-show  https://review.openstack.org/5399106:34
*** rushiagr has quit IRC06:34
*** michchap has joined #openstack-ironic06:37
*** michchap has quit IRC06:40
*** michchap has joined #openstack-ironic06:40
*** romcheg has joined #openstack-ironic06:48
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: ipmitool driver SHOULD accept empty username or password  https://review.openstack.org/5488607:45
*** matsuhashi has quit IRC08:08
*** matsuhashi has joined #openstack-ironic08:08
*** romcheg has quit IRC08:10
*** matsuhashi has quit IRC08:12
*** matsuhashi has joined #openstack-ironic08:16
*** martyntaylor has joined #openstack-ironic08:19
openstackgerritSun Jing proposed a change to openstack/ironic: Add tftp_server pattern in ironic.conf  https://review.openstack.org/5489108:31
*** matsuhashi has quit IRC08:34
*** jistr has joined #openstack-ironic08:39
*** sjing has quit IRC08:44
*** creiht_ has joined #openstack-ironic08:54
*** creiht has quit IRC08:58
*** Haomeng|2 has joined #openstack-ironic09:01
*** matsuhashi has joined #openstack-ironic09:02
*** Haomeng has quit IRC09:02
*** yuriyz has joined #openstack-ironic09:02
*** matsuhashi has quit IRC09:10
*** matsuhashi has joined #openstack-ironic09:11
*** matsuhashi has quit IRC09:15
Haomeng|2yuriyz: ping09:15
yuriyzpong09:15
Haomeng|2about https://review.openstack.org/#/c/54886/09:16
Haomeng|2how do you think we should use a 'space' as the empty password to avoid default python string value 'None'?09:16
Haomeng|2yuriyz, thanks for your comments, I will try to fix, will be back later...09:17
*** ndipanov_gone is now known as ndipanov09:19
yuriyzI think answer will be in man ipmitools, i didnt look it09:19
*** matsuhashi has joined #openstack-ironic09:20
*** derekh has joined #openstack-ironic09:21
*** jistr has quit IRC09:24
yuriyzHaomeng, I think we should drop -U / -f ipmitool option for empty username / password http://ipmitool.sourceforge.net/manpage.html09:28
*** romcheg has joined #openstack-ironic09:31
*** romcheg has left #openstack-ironic09:31
*** jistr has joined #openstack-ironic09:38
*** jistr has quit IRC09:39
*** jistr has joined #openstack-ironic09:40
*** liteIRC has joined #openstack-ironic09:47
*** lucasagomes has joined #openstack-ironic09:52
*** yuriyz has quit IRC09:57
*** matsuhashi has quit IRC10:25
*** liteIRC has quit IRC10:29
*** nosnos has quit IRC10:32
Haomeng|2thank you yuriyz, will follow you10:48
openstackgerritA change was merged to openstack/ironic: Fix i18N compliance  https://review.openstack.org/5468711:09
*** michchap has quit IRC11:18
*** michchap has joined #openstack-ironic11:21
*** mwagner has joined #openstack-ironic11:27
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add driver interfaces validation  https://review.openstack.org/5459311:44
*** romcheg1 has joined #openstack-ironic11:52
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Replace __metaclass__  https://review.openstack.org/5491312:10
*** lucasagomes is now known as lucas-hungry12:22
*** jdob has joined #openstack-ironic12:26
*** liteIRC has joined #openstack-ironic12:29
martyntaylorlucas-hungry: ping12:34
*** liteIRC has quit IRC12:37
*** michchap has quit IRC12:49
*** romcheg has joined #openstack-ironic12:51
*** michchap has joined #openstack-ironic12:53
*** michchap_ has joined #openstack-ironic12:54
*** michchap_ has quit IRC12:54
*** michchap_ has joined #openstack-ironic12:56
*** romcheg1 has quit IRC12:56
*** michchap has quit IRC12:58
*** jdob has quit IRC12:59
*** jdob has joined #openstack-ironic12:59
*** romcheg has quit IRC12:59
*** jbjohnso has joined #openstack-ironic13:18
max_loburHi linggao, lucasgomes, devananda, lifeless, yuriyz13:34
max_loburI've posted my comment to instance_uuid discussion https://review.openstack.org/#/c/53262/ . Ready to start one more lap of great discussion =)13:34
max_loburThank you all for the comments13:35
*** romcheg has joined #openstack-ironic13:37
*** jianingy1afk has joined #openstack-ironic13:37
*** jianingy_afk has quit IRC13:39
*** lucas-hungry is now known as lucasagomes13:40
lucasagomesmartyntaylor, pong :)13:40
martyntaylorlucasagomes: just got on  a call mate, can I ping again in 30mins13:41
lucasagomesmartyntaylor, sure :) np13:41
*** jdob has quit IRC13:41
*** jdob has joined #openstack-ironic13:41
*** romcheg has left #openstack-ironic13:42
*** liteIRC has joined #openstack-ironic13:54
*** liteIRC has quit IRC14:03
*** jbjohnso has quit IRC14:07
NobodyCamgood mornign Ironic14:14
max_loburmorning NobodyCam14:17
NobodyCamhey hey max_lobur how are you this fine friday14:19
max_loburgreat, thanks! truly fine friday :)14:19
max_loburand you?14:19
*** linggao has joined #openstack-ironic14:25
NobodyCamTGIF!14:25
NobodyCamyou said it14:25
max_lobur:)14:26
*** linggao has quit IRC14:27
*** linggao has joined #openstack-ironic14:27
*** yuriyz has joined #openstack-ironic14:34
*** ndipanov has quit IRC14:46
*** ndipanov has joined #openstack-ironic14:48
linggaogood morning, max_lobur and lifeless.14:51
linggaogood morning all.14:51
NobodyCammorning linggao :)14:51
linggaomorning NobodyCam14:51
NobodyCam:)14:51
linggaomax_lobur, I saw your last comment on the instance_uuid.14:52
linggaoSo this is what I am going to do:14:52
linggao.../v1/nodes/?instance_uuid=<instance_uuid>  to return a collection of one node.14:53
linggaoor empty.14:53
max_loburnot expanded collection right?14:53
linggaothat's my question,14:54
linggaoironic node-show --instance-uuid <uuid> needs details.14:54
max_loburah:) I think it shouldn't be expanded. I'd expect expanded collection only for details14:55
linggaoshall I make 2 calls for that command?14:55
max_loburnot sure about 2nd call. I proposed this in my last commend but didn't back any feedback yet14:55
linggaoyes.14:55
linggaolifeless has comments on the CLI side, so I also need to work out with him to see what is his expectation.14:56
*** creiht_ is now known as creiht14:56
linggaoare u there lifeless?14:56
max_loburlet's see what devananda and lucasagomes and others say about 2nd call. Meybe we'll came up with some better way14:57
linggaomax_lobur, the  /v1/nodes/?instance_associated=<[True|False]> part14:57
linggaoCan the values be more flexiable like true, false, True, False?14:58
*** yuriyz has quit IRC14:58
max_loburor just  /v1/nodes/?associated=<[True|False]> if we define those property14:59
max_loburI think yes they can, but need to try to find something similar14:59
max_loburand see how they did this14:59
linggaomax_lobur the property part is confusing me. where do you define the property?15:02
linggao@property15:02
linggaodef associated(self):15:02
linggao    return self.instance_uuid != null)15:02
NobodyCamquick walkies... brb15:04
max_loburyeah, we're working with db models so it's better not to place the properties there15:05
*** jianingy_afk has joined #openstack-ironic15:06
max_loburanyway the property is not mandatory, I posted it just for better understanding of the ?associated=<[True|False]> filter15:06
linggaook. thanks max_lobur. I'll do the implementation now and check it in soon.15:07
*** jianingy1afk has quit IRC15:08
max_loburso do you want to proceed with two API calls?15:08
linggaoyes15:09
max_loburI mean if it's still in discussion than it wouldn't it better to wait why we get something that everyone accept15:09
max_lobur*while we get15:10
linggaothis one seems what davananda liked on IRC and you summarized in your comments.15:11
max_loburtrue. I think everyone will be on-line in 2 or less hours so they will be able to respond15:13
linggaook, I'll wait a bit.15:13
*** jbjohnso has joined #openstack-ironic15:13
max_loburperhaps someone had a chance to think of it at night and have something better:)15:14
linggaolol15:14
linggaowonder what is lifeless15:14
linggao's time zoon.15:14
linggaozone15:14
linggaowe need to make sure all the APIs are defined, REST. CLI and internal. then I can proceed.15:15
max_loburyea15:16
NobodyCamlinggao: lifeless is in NZ15:16
NobodyCamwell australia15:17
linggaoSo it is 4am in the morning there.15:18
NobodyCamya15:18
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Deal with unicode strings  https://review.openstack.org/5494215:42
lucasagomesmax_lobur, linggao I liked the proposed idea15:59
lucasagomescommented on the review15:59
NobodyCammorning lucasagomes16:00
lucasagomesv1/nodes?instance_associated=... or even just v1/nodes?associated=... looks good to me16:00
lucasagomesNobodyCam, hey ya16:00
lucasagomesNobodyCam, how are u ?16:00
max_loburhi lucasagomes16:01
max_loburcool16:01
linggaomorning lucasagomes.16:01
max_loburlinggao also proposed to have flexible True|False or true|false or even 0|1 maybe16:01
max_loburwhat do you think?16:01
linggaoyes.16:01
NobodyCamlucasagomes: doing good. looking at all thestuff I have to get ready to start packing16:01
NobodyCampip install /opt/stack/nova16:02
NobodyCamdoh ww16:02
devanandag'morning, all16:02
NobodyCammorning devananda16:02
linggaomorning davananda.16:02
lucasagomesdevananda, morning16:02
max_loburg'moning devananda16:02
lucasagomesmax_lobur, hmm I don't really would use 0|1, I don't see any benefits16:03
lucasagomestrue|false, or True|False or TRUE|FALSE grand, since we should convert it internally16:03
max_loburyep, agree16:04
NobodyCam2013-11-01 16:03:54,546.546 9423 TRACE nova.openstack.common.threadgroup     nm = ironic.node.NodeManager(self._get_client())16:04
NobodyCam2013-11-01 16:03:54,546.546 9423 TRACE nova.openstack.common.threadgroup AttributeError: 'module' object has no attribute 'node'16:04
linggaoas to if  the result in /v1/nodes/?instance_uuid=<instance_uuid> get expended,16:05
devanandaNobodyCam: perhaps the "import .. as ironic" is importing the wrong thing16:05
devanandaNobodyCam: not deep enough16:05
linggaosince node-show --instance-uuid <uuid> needs it,  shall we expand it?, otherwise, 2 calls needs to be made.16:06
lucasagomesNobodyCam, r you testing the nova ironic driver? cool I wanna test it as well!16:06
lucasagomeslinggao, hmm, I would put the API in the first plans here, the cli just use it16:06
lucasagomesin the API it doesnt make sense to expand it16:07
lucasagomessince it's not consistent with the result of the other collections16:07
max_lobur+16:07
linggaook, make sense.16:07
NobodyCamyep16:07
lucasagomeslinggao, so I would ask, is node-show --instance-uuid what we are looking for here16:07
lucasagomesor maybe node-list?16:07
devanandafwiw guys, the WIP i have up of nova-ironic driver is untested by me and has no unit tests even ...16:08
NobodyCamlucasagomes: 2013-11-01 16:03:54,546.546 9423 TRACE nova.openstack.common.threadgroup     nm = ironic.node.NodeManager(self._get_client())16:08
devanandait's goign to break in a million ways16:08
NobodyCam2013-11-01 16:03:54,546.546 9423 TRACE nova.openstack.common.threadgroup AttributeError: 'module' object has no attribute 'node'16:08
NobodyCamwhat is up with my kyb16:08
NobodyCamlucasagomes: https://github.com/NoBodyCam/nova-ironic16:08
NobodyCam:-p16:08
lucasagomesNobodyCam, wow! u already started an nova-ironic element16:09
lucasagomesawesome!16:09
NobodyCamhttps://github.com/NoBodyCam/nova-ironic/blob/master/nova-ironic/install.d/89-nova-ironic-services16:10
NobodyCam:-p it deff NOT ready for prime time16:10
NobodyCamoh wait16:10
NobodyCamI have a change to push16:11
lucasagomesNobodyCam, I wouldn't expect it to work at the first try as well... but having an element for a WIP patch that's being way ahead the curve anyway :D16:11
lucasagomesgood stuff16:12
linggaolucasagomes, it should show the same result as if with uuid.  http://paste.openstack.org/show/50395/16:12
* NobodyCam hopes the plane has power as his laptop lasts all of an hour on batt16:13
*** ndipanov has quit IRC16:13
devanandak, i'm'a walk out for a good cup'o coffee. bbiab16:14
lucasagomeslinggao, I see...16:15
linggaolucasagomes, node-show shows one node, node-list shows a collection.  Now with my another patch, the CLI for instance uuid looks like this:16:15
linggaonode-show --instance-uuid <uuid>  shows one node.16:16
lucasagomesone thing is that the cli uses the lib (which is also used by our nova ironic driver) and the lib uses the api... so I don't see any problem in doing 2 requests when node-show --instance_uuid is requested16:16
linggaonode-list --associated   shows all the associated nodes.16:16
lucasagomesinternally you would list filtering by that uuid, and then you would get that node if found and return to the user16:16
lucasagomesjust like a node-shode <node uuid>16:16
linggaonode-list --unassociated  shows all unassociated nodes16:17
linggaoyes.  node-show [--instance-uuid} <id>,   and node-list [--[un]associated]16:17
lucasagomeslinggao, got it16:17
lucasagomeswhat you think about sending two requests when instance_uuid is provided?16:18
lucasagomesI don't think it would do any harm16:18
lucasagomesthat's the workflow of our API as well16:18
lucasagomesfilter by the instance uuid to find the node uuid16:18
linggaoyes, ok.16:18
lucasagomesand if u need more detail you would GET that node uuid16:18
lucasagomesmore detail about that node*16:19
lucasagomesmax_lobur, ^ thoughts?16:19
max_lobur+16:19
max_lobur:)16:20
max_loburI mean I'm totally agre16:20
max_loburbetter to have consistent API and two calls then something weird in API16:20
lucasagomesyea16:21
lucasagomesI think 2 calls are common for things like that16:21
lucasagomesthere's even a http return code for that16:21
lucasagomes30316:21
linggaolucasagomes, max_lobut, so that's a parallel discussion, what's the internal API  look like.   https://review.openstack.org/#/c/53485/16:22
lucasagomeshttp://stackoverflow.com/questions/2427518/which-http-redirect-status-code-is-best-for-this-rest-api-scenario16:22
NobodyCambbt... brb16:29
max_loburlinggao, I'm looking to CLI patch set. will add my comments16:33
linggaomax_lobur, thanks.16:34
* linggao lunch time16:40
*** romcheg has joined #openstack-ironic17:13
*** romcheg has left #openstack-ironic17:14
max_loburlinggao, lucasagomes I voted for node-list [--associated <true|false>] instead of  node-list [--[un]associated] just to be consistent with internal API and REST API. And also I gave an example that we could have something like node-list --powered-on <true|false> in future, so proceeding with two mutually exclusive flags for each such filter will be not very convenient. But if you guys think it's unlikely to have something similar to 'associated'17:20
max_loburand two flags are more handy, than I'm ok with node-list [--[un]associated]17:20
max_lobur*voted in review for CLI patch set17:21
max_loburdon't want to restart debates, just wanted to let you know that I'm OK with both ways for CLI17:22
* linggao is back17:33
linggaohi max_lobur, yes, I see your point. thanks.17:33
lucasagomesmax_lobur, ack :) I think I'm also happy with any17:34
max_lobur:)17:35
linggaomax_lobur, we had the debates on it, and people aggreed that they do not want to put attribute values in the commad line.17:35
max_loburthan it's OK as is ;)17:35
linggaoso the conclusion was either  use sepearate commands like node-list-unassociated, or put them in one command with flags17:36
linggaomax_lobur, I think lifeless has problem with internal function.  like the paramters for get, and show.17:38
*** linggao_ has joined #openstack-ironic17:39
* linggao is still here.17:39
NobodyCam:-p17:40
max_loburlinggao, let's make a fresh patch with everything concluded till now, and then see what can be improoved17:41
max_loburI think It will be pretty consistent and everyone will liked it :)17:41
linggaomax_lobur, yes. good idea. there are too many reviews. I am confused  :-).17:42
max_loburyea:) a wall of comments17:43
NobodyCam++17:43
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Add node-set-power-state to cli  https://review.openstack.org/5335617:44
*** derekh has quit IRC17:45
lifelessCOMMMMMMENTS!17:45
max_loburmorning lifeless :)17:46
lifelessmorning :)17:46
*** jistr has quit IRC17:48
NobodyCammorning lifeless :)17:49
linggaomorning lifeless.17:56
NobodyCamHoodies are confirmed in. Atc's Please email nobodycam@gmail.com as to the best way to get said hoodies to you..18:03
*** jdob has quit IRC18:04
*** jdob has joined #openstack-ironic18:10
lucasagomesNobodyCam, a-ha! cool18:15
devanandathey're being shipped to HKG18:20
NobodyCamanteaya: you around?18:22
NobodyCamanteaya: 5.1.1 <anita.kuno@enovance.com>: Recipient address rejected: enovance.com18:23
NobodyCambrb18:27
max_loburEveryone have a good weekend! And safe travel for those who going to HKG! :)18:37
*** blamar has quit IRC18:44
clarkbNobodyCam: anteaya is at hp now18:45
*** blamar has joined #openstack-ironic18:58
NobodyCam:)18:59
NobodyCamlucasagomes: http://www.youtube.com/watch?v=KPjfH-5vu8c19:00
* lucasagomes clicks19:00
NobodyCam:-P19:01
lucasagomeshahaha nice19:03
NobodyCamthats old school easter egg19:03
lucasagomeshaha oh yea19:04
NobodyCam:)19:04
lucasagomeshttps://bugs.launchpad.net/ubuntu/+source/apt/+bug/5612519:07
lucasagomeslol19:07
NobodyCamlol19:07
lucasagomesalright I'm done for today19:09
lucasagomeshave a g night everyone19:09
lucasagomessee some of you guys in hk :)19:09
lucasagomeshave a safe trip!19:10
*** nobodycamtablet has joined #openstack-ironic19:12
*** lucasagomes has quit IRC19:14
nobodycamtablethttp://paste.openstack.org/show/tBuCqxwAp6lZJXoOfrTD/19:15
openstackgerritlinggao proposed a change to openstack/ironic: Supports get node by instance uuid in API  https://review.openstack.org/5326219:56
NobodyCamjust a snip from compute log... http://paste.openstack.org/show/SbcSaeaRukC6s5mzP9su/20:26
*** jdob has quit IRC20:47
*** dwi has left #openstack-ironic20:50
*** rloo has joined #openstack-ironic20:54
openstackgerritRuby Loo proposed a change to openstack/ironic: Changes power_state and adds last_error field  https://review.openstack.org/5446620:59
*** kobier has quit IRC21:18
*** martyntaylor has left #openstack-ironic21:19
*** jbjohnso has quit IRC21:21
*** kobier has joined #openstack-ironic21:26
*** kobier has quit IRC21:30
NobodyCamlooks like walkies time21:43
*** martyntaylor has joined #openstack-ironic21:49
*** romcheg has joined #openstack-ironic21:49
*** romcheg has quit IRC21:50
*** martyntaylor1 has joined #openstack-ironic21:50
*** martyntaylor has quit IRC21:50
NobodyCamdevananda: you about?22:17
NobodyCamdevananda: http://paste.openstack.org/show/JlCwzM049e1fqgf5QcAT/22:18
*** linggao has quit IRC22:35
*** jbjohnso has joined #openstack-ironic22:45
*** jbjohnso has quit IRC23:02
*** jbjohnso has joined #openstack-ironic23:06
devanandajsut for a minute23:06
devanandaNobodyCam: i see ....23:07
devanandaNobodyCam: so it's talkign to Ironic. Add some nodes to Ironic. You should see them appear in Nova as compute resources if get_available_resources is implemented correctly, I think23:08
devanandaNobodyCam: also - that's awesome!23:08
*** jbjohnso has quit IRC23:11
NobodyCam:-)23:14
Haomeng|2NobodyCam: I response your mail ready:) I will NOT being attend the Hong Kong summit.23:18
Haomeng|2:)23:19
NobodyCamHaomeng|2: I do not actually have them. I will get them at the summit so we'll get thing going then23:20
NobodyCamthings*23:20
Haomeng|2ok, got:)23:22
Haomeng|2np23:22
*** Haomeng has joined #openstack-ironic23:29
*** Haomeng|2 has quit IRC23:29
*** rloo has quit IRC23:44
« Thursday, 2013-10-31 Index Saturday, 2013-11-02 »

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