Thursday, 2016-08-25

« Wednesday, 2016-08-24 Index Friday, 2016-08-26 »
*** uck has joined #tacker00:11
*** uck has quit IRC00:16
*** KanagarajM has quit IRC00:20
*** sripriya has quit IRC00:24
*** sripriya has joined #tacker00:26
*** alexyang has joined #tacker00:31
*** prashantD has quit IRC00:40
*** ksantoshk has quit IRC01:07
*** santoshk has quit IRC01:07
*** prashantD has joined #tacker01:35
*** sripriya has quit IRC01:39
*** sripriya has joined #tacker01:42
*** sripriya has quit IRC01:44
*** prashantD has quit IRC01:48
openstackgerritLu lei proposed openstack/python-tackerclient: Stop using mox in tackerclient (2)  https://review.openstack.org/35981701:51
*** s3wong has quit IRC02:03
*** xiayu1 has joined #tacker02:14
*** xiayu has quit IRC02:17
*** achatterjee has joined #tacker02:31
*** _achatterjee_ has quit IRC02:32
openstackgerritMerged openstack/tacker: Name of the exception is corrected  https://review.openstack.org/35382802:52
*** prashantD has joined #tacker02:54
*** changzhi has joined #tacker02:55
*** bobh has joined #tacker03:00
openstackgerritTim Rozet proposed openstack/python-tackerclient: Adds client commands for listing a VNF resources  https://review.openstack.org/36019703:10
trozetsridhar_ram:^03:14
openstackgerritLu lei proposed openstack/python-tackerclient: Stop using mox in tackerclient (2)  https://review.openstack.org/35981703:17
*** vishwanathj has joined #tacker03:25
*** vishwanathj is now known as vishwanathj_zzz03:31
*** vishnoianil has quit IRC03:31
*** changzhi has quit IRC03:41
*** gongysh has joined #tacker03:43
*** amotoki has joined #tacker04:06
*** gongysh has quit IRC04:25
*** Ravikiran_K has joined #tacker04:29
*** bobh has quit IRC04:30
*** links has joined #tacker04:37
*** ksantoshk has joined #tacker04:43
*** santoshk has joined #tacker04:43
*** ksk has joined #tacker04:44
*** neel has joined #tacker04:46
*** ksantoshk has quit IRC04:48
*** santoshk has quit IRC04:48
*** santoshk has joined #tacker04:48
*** neel has quit IRC04:53
*** alexyang has quit IRC04:58
*** neel has joined #tacker05:06
*** KanagarajM has joined #tacker05:08
*** prashantD has quit IRC05:15
*** janki has joined #tacker05:23
*** vishwanathj_zzz has quit IRC06:10
*** vishwanathj_zzz has joined #tacker06:10
*** neel has quit IRC06:28
*** neel has joined #tacker06:31
*** vishwanathj_zzz has quit IRC06:32
*** vishwanathj_zzz has joined #tacker06:32
*** vishnoianil has joined #tacker06:35
jankitrozet, ping06:44
*** ossifrath has quit IRC06:57
*** ossifrath has joined #tacker06:57
*** manikanta_tadi has joined #tacker07:03
*** Ravikiran_K has quit IRC07:14
*** vishwanathj_zzz has quit IRC07:19
*** vishwanathj_zzz has joined #tacker07:20
*** neel has quit IRC07:37
openstackgerritsajuptpm proposed openstack/tacker: Validate file size at server side  https://review.openstack.org/34654307:39
*** santoshk has quit IRC07:51
*** ksk has quit IRC07:51
openstackgerritManikantha Srinivas Tadi proposed openstack/tacker: Store the right description of VNF given from Horizon/CLI.  https://review.openstack.org/33103907:59
*** KanagarajM has quit IRC08:03
openstackgerritManikantha Srinivas Tadi proposed openstack/tacker: Store the right description of VNF given from Horizon/CLI.  https://review.openstack.org/33103908:05
*** KanagarajM has joined #tacker08:09
*** vishwanathj_zzz has quit IRC08:10
*** vishwanathj_zzz has joined #tacker08:10
openstackgerritManikantha Srinivas Tadi proposed openstack/python-tackerclient: Created nfvo directory for nfvo specific unit tests in tackerclient.  https://review.openstack.org/34593908:16
*** KanagarajM has quit IRC08:28
*** KanagarajM has joined #tacker08:34
*** neel has joined #tacker08:42
*** openstackgerrit has quit IRC08:48
*** openstackgerrit has joined #tacker08:49
openstackgerritDigambar proposed openstack/python-tackerclient: Deprecate infra_driver and mgmt_driver  https://review.openstack.org/36034508:49
*** diga has joined #tacker08:51
*** neel has quit IRC09:01
openstackgerritDigambar proposed openstack/python-tackerclient: Deprecate infra_driver and mgmt_driver  https://review.openstack.org/36034509:05
*** saju_m has joined #tacker09:25
*** KanagarajM has quit IRC09:34
*** vikasc has left #tacker09:36
openstackgerritDigambar proposed openstack/python-tackerclient: Deprecate infra_driver and mgmt_driver  https://review.openstack.org/36034509:40
openstackgerritgongysh proposed openstack/tacker: Device refactor Part3: Rename device in codes  https://review.openstack.org/35220509:43
openstackgerritsajuptpm proposed openstack/python-tackerclient: [wip]Validate file size before uploading  https://review.openstack.org/34095009:48
openstackgerritgongysh proposed openstack/tacker: Device refactor Part3: Rename device in codes  https://review.openstack.org/35220509:49
*** vishwanathj_zzz has quit IRC09:52
*** vishwanathj_zzz has joined #tacker09:53
*** neel has joined #tacker09:59
openstackgerritgongysh proposed openstack/tacker: Device refactor Part3: Rename device in codes  https://review.openstack.org/35220510:02
openstackgerritgengchc2 proposed openstack/tacker: Translate info-level log messages for LOG.error  https://review.openstack.org/36039710:11
openstackgerritLu lei proposed openstack/python-tackerclient: Fix olso.i18n problems in python-tackerclient  https://review.openstack.org/35530710:13
openstackgerritsajuptpm proposed openstack/python-tackerclient: [wip] Validate file size before uploading  https://review.openstack.org/34095010:16
*** tbh has joined #tacker10:30
openstackgerritLu lei proposed openstack/python-tackerclient: Fix olso.i18n problems in python-tackerclient  https://review.openstack.org/35530710:48
*** neel has quit IRC10:50
*** neel has joined #tacker10:55
*** mpsairam has quit IRC10:55
*** vishwanathj_zzz has quit IRC10:55
*** vishwanathj_zzz has joined #tacker10:56
openstackgerritvenkatamahesh proposed openstack/tacker: [DOC] Update the manual installation guide to rectify errors  https://review.openstack.org/35325811:21
*** xiayu has joined #tacker11:24
*** xiayu1 has quit IRC11:28
*** achatterjee has quit IRC11:37
*** bobh has joined #tacker12:18
*** vishwanathj_zzz has quit IRC12:18
*** vishwanathj_zzz has joined #tacker12:18
*** KanagarajM has joined #tacker12:19
openstackgerritManikantha Srinivas Tadi proposed openstack/tacker-specs: Tacker VNFC support  https://review.openstack.org/33979812:40
*** bobh has quit IRC12:42
*** vishwanathj_zzz is now known as vishwanathj13:00
*** janki has quit IRC13:03
*** janki has joined #tacker13:05
*** vishwanathj has quit IRC13:15
*** vishwanathj has joined #tacker13:16
*** neel has quit IRC13:17
openstackgerritxuan0802 proposed openstack/tacker-specs: Specification for integrating Senlin cluster for VDU into Tacker  https://review.openstack.org/35294313:22
*** links has quit IRC13:22
*** manikanta_tadi has quit IRC13:25
*** janki has quit IRC13:45
*** vishwanathj has quit IRC13:55
*** vishwanathj has joined #tacker13:55
*** janki has joined #tacker14:08
jankitrozet, ping14:12
jankisridhar_ram, ping14:12
trozethi janki14:12
jankitrozet, regarding the patch for vnf-resource-list14:13
jankiI have checked, it works fine except for a minor change. Please see the comments14:13
trozetjanki: did you try my client patch?14:13
jankitrozet, yes  I have checked, it works fine except for a minor change. mentioned that in comments14:14
trozetjanki: i'm surprised :)14:14
trozetjanki: i didnt test it, just unit test, it was pretty late when i got it to pass unit tests and went to bed14:14
jankitrozet, what I want to talk about is the client patch for --show-details that has merged14:15
trozetjanki: for the 2nd comment about doing a vnf-resource-show, we dont need to get that one in for now, we can do it later14:15
jankitrozet, cool, you could do it at a go :)14:15
jankitrozet, agree with doing it later.14:16
trozetjanki: how did you create the release notes on your client patch?  I notice it has a unique id on the file name, I figure youd idnt create the file manually14:16
jankitrozet, reno new 'slug-goes-here'14:17
trozetjanki: ah ok thanks I will try that14:17
jankitrozet, this will create a yaml file. make appropriate changes in the file. be mindfull of the "-" in the file. those are compulsory14:18
trozetjanki: per the other client patches you can just revert right?14:18
jankitrozet, dont know about reverting. i was thinking for deleting earlier patch as a part of this new patch14:19
trozetjanki: it is better to revert14:19
jankitrozet, I believe with this new patch, the older one is irrelevant14:19
jankitrozet, can I revert or it needs special permission?14:20
trozetjanki: yeah on https://review.openstack.org/#/c/354057/ you can click the revert button14:20
trozetjanki: then it will open a new commit, and needs +2 from sridhar_ram14:21
jankitrozet, let me try. give me a minute please14:21
openstackgerritJanki Chhatbar proposed openstack/python-tackerclient: Revert "Creates details API to fetch VNF detials"  https://review.openstack.org/36058314:22
jankitrozet, done. thanks :)14:23
trozetjanki: cool. So you haven't pushed your latest patch to 340838 yet?14:24
openstackgerritJanki Chhatbar proposed openstack/tacker: Add VNF resource details to get vnf API  https://review.openstack.org/34083814:25
jankitrozet, just did. please review at your convinience14:26
trozetjanki: sure will do14:26
trozetjanki: thanks for accommodating the last minute changes, i think they make more sense from a REST perspective14:27
jankitrozet, I agree. the output now looks prettier14:28
jankitrozet, going offline. will log back after reaching home. please ping me in case of any errors/concerns. Thanks :)14:29
*** janki has quit IRC14:29
*** vishwanathj has quit IRC14:44
*** vishwanathj has joined #tacker14:44
*** vishwanathj has quit IRC14:46
*** vishwanathj has joined #tacker14:46
*** neel has joined #tacker14:48
*** openstackgerrit has quit IRC14:48
*** openstackgerrit has joined #tacker14:49
*** KanagarajM has quit IRC15:03
*** xiayu has quit IRC15:14
*** janki has joined #tacker15:14
*** xiayu has joined #tacker15:14
*** vishwanathj has quit IRC15:15
*** vishwanathj has joined #tacker15:16
*** vishwanathj has quit IRC15:18
*** vishwanathj has joined #tacker15:19
*** links has joined #tacker15:20
*** saju_m has quit IRC15:22
sridhar_ramtrozet: janki: thanks for rooting and getting in cleaner APIs.. hats off. I'll do the needful from my side -15:34
*** prashantD has joined #tacker15:36
trozetsridhar_ram: yeah janki left a comment, I'll push a new patch set within next 30 min15:37
jankisridhar_ram, I have requested to revert a patch. https://review.openstack.org/#/c/360583/. This needs to go out in account of new patchset15:37
*** vishwanathj has quit IRC15:44
*** vishwanathj has joined #tacker15:45
*** jchhatbar has joined #tacker15:58
*** janki has quit IRC15:59
*** jchhatbar is now known as janki16:00
*** links has quit IRC16:00
*** uck has joined #tacker16:02
*** xiayu has quit IRC16:02
openstackgerritTim Rozet proposed openstack/python-tackerclient: Adds client commands for listing a VNF resources  https://review.openstack.org/36019716:03
*** xiayu has joined #tacker16:03
*** vishwanathj has quit IRC16:04
*** vishwanathj has joined #tacker16:05
openstackgerritTim Rozet proposed openstack/python-tackerclient: Add client support for VNFFG in NFVO  https://review.openstack.org/34138916:12
jankitrozet, ping16:26
trozetjanki: hi16:26
*** diga has quit IRC16:27
*** vishwanathj has quit IRC16:27
*** vishwanathj has joined #tacker16:28
jankitrozet, just a thought. listvnfresource should work similar to listvnf16:28
jankiI mean without the function boyd and just mentioning the list_columns16:28
trozetjanki: it does, you just have to provide an ID of the VNF16:28
jankitrozet, dont have VPN access to try it out16:29
trozetjanki: those functions i overrode have to be done, so that you can input the ID16:29
jankitrozet, ahh, yes. listvnf doesnot have ids cool :)16:30
jankitrozet, thanks for explaining :)16:32
trozetjanki: np16:32
*** sripriya has joined #tacker16:46
*** janki has quit IRC17:09
*** uck has quit IRC17:09
*** amotoki has quit IRC17:17
*** xiayu has quit IRC17:18
*** xiayu has joined #tacker17:18
*** s3wong has joined #tacker17:22
vishwanathjcores, would  be great if you can review https://review.openstack.org/#/c/358678/ and merge...thanks17:23
*** sripriya has quit IRC17:35
*** uck has joined #tacker18:10
*** uck has quit IRC18:14
*** sripriya has joined #tacker18:15
*** vishnoianil has quit IRC18:17
*** uck has joined #tacker18:24
*** s3wong has quit IRC18:33
*** prashantD_ has joined #tacker18:35
*** prashantD has quit IRC18:38
*** tbh has quit IRC18:44
*** neel has quit IRC18:47
*** vishwanathj has quit IRC18:47
*** vishwanathj has joined #tacker18:48
*** uck_ has joined #tacker18:50
*** uck has quit IRC18:50
openstackgerritvishwanath jayaraman proposed openstack/tacker: Adds functional tests for common services plugin  https://review.openstack.org/35867819:04
*** vishnoianil has joined #tacker19:18
*** sripriya has quit IRC19:31
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is restarting to implement some performance tuning adjustments, and should return to working order momentarily.19:47
*** uck_ has quit IRC19:56
*** sripriya has joined #tacker20:04
*** prashantD has joined #tacker20:05
*** prashantD_ has quit IRC20:07
*** vishwanathj has quit IRC20:19
*** vishwanathj has joined #tacker20:20
trozetsridhar_ram: ping?20:27
*** vishwanathj has quit IRC20:28
*** vishwanathj has joined #tacker20:29
*** saju_m has joined #tacker20:41
*** prashantD has quit IRC20:48
*** prashantD has joined #tacker20:49
*** uck has joined #tacker20:57
*** uck has quit IRC21:02
*** uck has joined #tacker21:09
*** vishwanathj has quit IRC21:16
*** vishwanathj has joined #tacker21:16
*** s3wong has joined #tacker21:20
*** vishwanathj has quit IRC21:26
*** vishwanathj has joined #tacker21:27
sridhar_ramtrozet: hi21:29
trozetsridhar_ram: hi left a comment on gerrit21:29
* sridhar_ram looking up21:30
sridhar_ramtrozet: the answer is yes :)21:30
sridhar_ramtrozet: i wasn21:30
sridhar_ramtrozet: I wasn't clear21:30
trozetsridhar_ram: ok will remove right now21:30
sridhar_ramtrozet: btw, i'm assuming u tested this w/ janki's latest server side patchset ?21:31
trozetsridhar_ram: heh I didn't, by the time I got the thing to pass unit test it was like 11:30pm here and I went to bed.  But woke up and Janki had tested it for me :)21:32
sridhar_ramtrozet: np, super .. that's nice tag teaming !!21:32
sridhar_ramtrozet: I want to land this today.. so that we can request the final new tackerclient release by tomorrow21:33
trozetsridhar_ram: yeah that sounds good21:35
trozetsridhar_ram: then what about this21:36
trozethttps://review.openstack.org/#/c/341389/21:36
sridhar_ramtrozet: of course, ^^^, is the biggee ..21:37
sridhar_rams3wong: ping21:37
s3wongsridhar_ram: pong21:38
sridhar_rams3wong: quick check, did you had a chance to try trozet's patchset w/ you n-sfc driver ?21:38
sridhar_ram*your21:38
s3wongsridhar_ram: my environment kind of crapped out last night21:39
s3wongsridhar_ram: just unstack.sh and stack.sh this morning... hopefully that would work21:39
s3wongsridhar_ram: I got DB error on networking-sfc tables at this point21:39
s3wongsridhar_ram: will need to clean up Neutron DB --- hopefully unstack.sh has done that21:40
sridhar_rams3wong: okay, that would help to validate the client + API.. final tackerclient for newton will be cut by EOD tomorrow21:40
s3wongsridhar_ram: OK21:40
trozetsridhar_ram, s3wong: i think in the meantime, we could agree on the abstract driver?21:40
sridhar_rams3wong: btw, you've added networking-sfc to tacker's requirements.txt ?21:40
s3wongsridhar_ram: I only added Neutron client21:41
s3wongtrozet: correct --- I believe at this point we still don't have an agreement on the name of the key in your flow classifier dictionary...21:41
trozets3wong: what do you mean? like match criteria name keys?21:42
openstackgerritMerged openstack/tacker: Adds functional tests for common services plugin  https://review.openstack.org/35867821:42
*** saju_m has quit IRC21:43
sridhar_rams3wong: while at it, can you please rebase your abstract driver - https://review.openstack.org/34756321:46
sridhar_ramsripriya: can you merge the "details" revert https://review.openstack.org/#/c/360583  .. so that we can proceed w/ the vnf-resource-list ?21:48
sripriyasridhar_ram: done21:50
sridhar_ramsripriya: thanks21:50
openstackgerritTim Rozet proposed openstack/python-tackerclient: Adds client commands for listing a VNF resources  https://review.openstack.org/36019721:51
sridhar_ramtrozet: s3wong : looking at http://specs.openstack.org/openstack/tacker-specs/specs/newton/tacker-vnffg.html#rest-api-impact .. I can see there is a sample ACL policy21:53
sridhar_ramtrozet: s3wong: is this a matter of n-sfc driver map as much as it can to neutron flow-classifier params?21:54
sridhar_ramtrozet: s3wong: for things it doesn't support, it would be fair for the n-sfc driver to return a failure21:55
trozetsridhar_ram, s3wong: I already looked at the spec, and what you had written in your driver.  I think I took them both into consideration when writing the keys21:55
trozeti believe they should match21:55
openstackgerritMerged openstack/python-tackerclient: Revert "Creates details API to fetch VNF detials"  https://review.openstack.org/36058321:55
sridhar_ramtrozet: s3wong : if it matches completely, that's best.. but even in the few cases it doesn't match, let's not let that be a blocker..21:56
trozetsridhar_ram: it looks to match to me:21:57
trozethttps://review.openstack.org/#/c/344522/23/tacker/db/nfvo/db_base/vnffg_db.py21:57
trozetMATCH_DB_KEY_LIST21:57
trozethttps://review.openstack.org/#/c/347568/3/tacker/nfvo/drivers/vnffg/sfc_drivers/networking_sfc/n_sfc.py21:57
trozetFC_MAP21:57
* sridhar_ram looking up21:57
sridhar_ramtrozet: meanwhile, can you rebase https://review.openstack.org/#/c/360197/ ?21:57
trozetsridhar_ram: only issue i see is "neutron_src_port"21:57
sridhar_ramtrozet: for the ^^ perhaps we can just drop "neutron_" prefix in the plugin / db side and let the n-sfc driver take care of handling it ?21:59
trozetsridhar_ram: the neutron word is in the SFC driver, not my plugin change21:59
sridhar_ramtrozet: agreed22:00
trozetsridhar_ram: my equivalent is "network_src_port_id"22:00
sridhar_ramtrozet: thanks :) i was just searching for that22:00
trozetsridhar_ram: tried to keep it abstract from openstack VIM22:01
sridhar_ramtrozet: s3wong: so n-sfc driver just need to convert network_src_port_id -> neutron_src_port22:01
trozetsridhar_ram: yeah let me rebase real quick22:01
trozetsridhar_ram: right22:01
*** vishwanathj has quit IRC22:01
trozetsridhar_ram: the other keys seem to match22:01
sridhar_ramtrozet: okay..22:01
sridhar_ramtrozet: i realize these are not API impacting change.. we can continue to iterate..22:02
*** vishwanathj has joined #tacker22:02
openstackgerritTim Rozet proposed openstack/python-tackerclient: Adds client commands for listing a VNF resources  https://review.openstack.org/36019722:03
sridhar_ramtrozet: trozet: however, I'm curious how well we have tested the instantiation time objects like nfps, sfcs and classifiers ..22:04
trozetsridhar_ram: yeah, I need to work on the changes with VNFFGD22:04
trozetsridhar_ram: well i have unit tests :)22:04
trozetsridhar_ram: so we at least know the plugin methods and the DB works22:04
sridhar_ramtrozet: yes, i'm looking into it just as we speak22:04
sridhar_ramtrozet: :)22:05
trozetsridhar_ram: but i think we could get this one in https://review.openstack.org/#/c/347563/22:05
sridhar_ramtrozet: this is where if n-sfc --returns--> values that doesn't "fit" into the response dicts we will have a problem22:05
trozeti just need s3wong to review, you and probably another committer22:05
trozetsridhar_ram: right, i mock that right now22:06
sridhar_ramtrozet: agree, the abstract driver can go in.. just need a rebase from s3wong22:07
sridhar_ramtrozet: tosca portions will be a separate patchset ?22:07
sridhar_ramtrozet: one comment on the vnf-resource .. see https://review.openstack.org/#/c/360197/4/tackerclient/v1_0/client.py22:13
*** vishwanathj has quit IRC22:17
*** vishwanathj has joined #tacker22:18
*** uck has quit IRC22:22
*** vishwanathj has quit IRC22:23
openstackgerritSripriya Seetharam proposed openstack/python-tackerclient: cli: modify vnfd, param & config attr. to dict obj  https://review.openstack.org/36080622:26
sridhar_ramsripriya: when u get a chance pls rebase https://review.openstack.org/#/c/351431/  .. code changes looks good to me22:30
sridhar_ramsripriya: sorry, that is not high priority.. just in the back of the list.. meanwhile pls help to sign off vnffg .. https://review.openstack.org/#/c/341389/22:31
sridhar_ramsripriya: highest priority right now22:31
openstackgerritSridhar Ramaswamy proposed openstack/tacker: VNFFG  abstract driver  https://review.openstack.org/34756322:31
sripriyasridhar_ram: will do22:33
openstackgerritTim Rozet proposed openstack/python-tackerclient: Adds client commands for listing a VNF resources  https://review.openstack.org/36019722:34
openstackgerritSripriya Seetharam proposed openstack/tacker: modify vnfd, param & config attr. to dict objects  https://review.openstack.org/36080922:34
*** mpsairam has joined #tacker23:00
openstackgerritMerged openstack/python-tackerclient: Adds client commands for listing a VNF resources  https://review.openstack.org/36019723:16
*** sripriya has quit IRC23:30
*** sripriya has joined #tacker23:51
« Wednesday, 2016-08-24 Index Friday, 2016-08-26 »

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