Wednesday, 2017-03-22

*** trinaths1 has joined #tacker00:11
*** uck has quit IRC00:15
*** bobh has joined #tacker00:16
*** Aqsa has quit IRC00:18
openstackgerritMerged openstack/tacker master: Update cirros image to 0.3.5 disk image in sample files  https://review.openstack.org/44747100:29
openstackgerritMerged openstack/tacker master: Provide a tool to generate a VNFC image  https://review.openstack.org/44584800:38
openstackgerritTrinath Somanchi proposed openstack/python-tackerclient master: Refactor mox references to use the mock library in pythontackerclient  https://review.openstack.org/42981300:58
*** gongysh has joined #tacker01:33
*** s3wong has quit IRC01:43
*** trinaths1 has left #tacker02:00
*** KanagarajM has joined #tacker02:08
*** vishnoianil has quit IRC02:13
*** bobh has quit IRC02:13
*** KanagarajM has quit IRC02:23
*** tbachman has quit IRC02:26
*** xianghui has joined #tacker02:34
*** bobh has joined #tacker02:35
*** bobh has quit IRC03:00
*** links has joined #tacker03:12
*** links has quit IRC03:38
*** links has joined #tacker03:52
*** diga has joined #tacker04:01
*** Zhou_Zhihong has joined #tacker04:06
*** diga has quit IRC04:18
*** diga has joined #tacker04:18
*** links has quit IRC04:33
sridhar_ramgongysh: ping, tacker meeting time?04:35
gongyshit is not one hour later?04:35
sridhar_ramgongysh: oops, daylight saving .. duh04:35
gongyshso what is the time over there now?04:36
*** links has joined #tacker04:49
*** tbachman has joined #tacker05:00
*** trinaths has joined #tacker05:01
*** YanXing_an has joined #tacker05:03
openstackgerritTrinath Somanchi proposed openstack/python-tackerclient master: Refactor mox references to use the mock library in pythontackerclient  https://review.openstack.org/42981305:05
*** KanagarajM has joined #tacker05:17
openstackgerritTrinath Somanchi proposed openstack/python-tackerclient master: Refactor mox references to use the mock library in pythontackerclient  https://review.openstack.org/42981305:24
*** tbh has joined #tacker05:28
gongyshone minute to meeting channel05:29
openstackgerritTrinath Somanchi proposed openstack/python-tackerclient master: Refactor mox references to use the mock library in pythontackerclient  https://review.openstack.org/42981305:58
openstackgerritTrinath Somanchi proposed openstack/python-tackerclient master: Refactor mox references to use the mock library in pythontackerclient  https://review.openstack.org/42981306:24
*** mohankumar has joined #tacker06:30
sridhar_ramtrinaths: there is already a partial effort on osc .. we can start from there06:30
openstackgerritTrinath Somanchi proposed openstack/python-tackerclient master: Refactor mox references to use the mock library in pythontackerclient  https://review.openstack.org/42981306:30
trinathssridhar_ram: oh okay can you please point me the commit.06:32
sridhar_ramtrinaths: https://review.openstack.org/#/c/374715/06:32
sridhar_ramtrinaths: https://blueprints.launchpad.net/tacker/+spec/tacker-openstackclient-plugin06:33
trinathssridhar_ram: okay. thank you.06:33
sridhar_ramtrinaths: urw06:33
* sridhar_ram signing off for the evening06:33
gongyshsridhar_ram, good night.06:34
gongyshKanagarajM hi06:34
trinathssridhar_ram: is mox->mock fix dependant on this osc fix ?06:34
sridhar_ramtrinaths: it shouldn't06:35
gongyshtrinaths, not06:35
sridhar_ramgongysh: thanks06:35
openstackgerritTrinath Somanchi proposed openstack/python-tackerclient master: Refactor mox references to use the mock library in pythontackerclient  https://review.openstack.org/42981306:35
trinathssridhar_ram: okay.06:35
*** tung_doan_ has quit IRC06:48
*** YanXing_an has left #tacker06:55
tbhsridhar_ram, https://review.openstack.org/#/c/438211/  ccan you please review?07:08
tbhdkushwaha, ^^07:08
KanagarajMgongysh, hi07:15
gongyshKanagarajM, hi07:15
*** bobmel has joined #tacker07:15
gongyshabout the unique index name, https://review.openstack.org/#/c/439719/07:16
KanagarajMgongysh, is the format used for  name of index is importnat07:17
gongyshKanagarajM, yes. with this,  DuplicateEntry exception will contain the columns of the index, so we can tell which unique index the code is touching.07:18
*** xianghui has quit IRC07:18
KanagarajMi try to test the code using tacker command it was working. so let me know if we need to really change the naming format07:18
gongyshKanagarajM, Yes, I know the code can work. but we can make it more robust.07:19
gongyshto think about too unique index on one table.07:19
KanagarajMgongysh, you mean, the index name should be part of the DuplicateEntry exception ?07:19
gongyshhow do you tell which unique index you are working on if DuplicateEntry happens?07:20
*** bobmel has quit IRC07:20
KanagarajMwhich i observed while testing. but i skippe the index name in DuplicateEntry exception as its internal and user should not be known about it07:20
gongyshI don't think the index name will be there, but the columns will be in DuplicateEntry.07:20
gongyshKanagarajM, you can check the columns in DuplicateEntry object.07:21
KanagarajMDuplicateEntry already having the column names07:21
*** diga has quit IRC07:21
KanagarajMits DuplicateEntity and i already added required column name with values07:21
gongyshKanagarajM, but you did not check it all: https://review.openstack.org/#/c/439719/8/tacker/db/vnfm/vnfm_db.py07:22
KanagarajMok i got what do you mean ..... you want to capture the message from the sqlalchmy eception and put them in the DuplicateEntity execption07:23
KanagarajMi will give anoth patch with this change. thanks gongysh07:23
gongyshthe standard way is: name the index with  uniq_<tablename>0column10column207:24
gongyshand then to tell DuplicateEntity contains just column1 and column2 .07:25
gongyshKanagarajM, we just need to abide by the practice here.07:25
*** Zhou_Zhihong has quit IRC07:26
KanagarajMgongysh, you mean, the column values are not required to be reported in the DuplicateEntity, just colum names are sufficient ?07:27
KanagarajMbut with values it would be more evident for user to understand.07:28
gongyshI means: use column names to tell the DuplicateEntity exception is for your index, not others.07:28
gongyshsystem may have many unique index on a table.07:29
KanagarajMbut same has been reproted to user07:29
gongysh        except DBDuplicateEntry:07:30
gongysh            raise exceptions.DuplicateEntity(07:30
gongysh                _type="nsd",07:30
gongysh                entry=dict(name=vnfd.get('name'),07:30
gongysh                           tenant_id=tenant_id))07:30
gongyshfor example. this is for name and tenant_id duplication.07:30
KanagarajMyes, i tried to keep them like this so that its useful info to user.07:30
gongyshif I need to add one more unique index on nsd table for example, "descript" and 'id' , what do you do?07:31
gongyshthis time, we should tell user, the id and descri is duplicated.07:32
gongyshnot name and tenant_id.07:32
KanagarajMexactly that is what missing, we will get the list of column names from the DBDuplicateEntry and pass it to DuplicateEntity07:32
KanagarajMi will add them in the next patch as mentioned earlier :)07:33
gongyshand also to change the index name please.07:33
gongyshthis is openstack way.07:33
KanagarajMsure. ok07:34
gongyshtrinaths, https://bugs.launchpad.net/python-tackerclient/+bug/167490607:53
openstackLaunchpad bug 1674906 in python-tackerclient "failed to update is-default attribute of vim" [Undecided,New]07:53
trinathsgongysh: ok. looking into it07:59
trinathsgongysh: when can --config-file be enforced?08:00
gongyshjust make them all optional.08:00
trinathsgongysh: ok08:01
gongyshand have a functional test.08:01
gongyshif server reports error, file bug on tacker server.08:01
gongyshthanks08:01
*** Aqsa has joined #tacker08:01
trinathsgongysh: ok08:01
trinathsgongysh: for all the commands or only vim based commands?08:05
gongyshtrinaths, only vim-update08:05
trinathsgongysh: ok08:06
*** diga has joined #tacker08:06
*** amotoki has joined #tacker08:20
trinathsgongysh: as I see the code, --config-file is the keystone based file.08:41
trinathsgongysh: with all auth credentials.08:41
gongyshyes08:42
trinathsgongysh: dont we need that file while updating the vim info ?08:42
gongyshtrinaths, we can just update name, description08:43
gongyshor something08:43
trinathsgongysh: agree. but with all those params, config-file with keystone credentails is required.08:45
gongyshtrinaths, why?08:46
trinathsgongysh:  since those credentials are used to authenticate at server and update the data.08:46
trinathsgongysh: see, def args2body_vim at v1_0/nfvo/vim_utils.py08:47
gongyshtrinaths, I know, you can pass that if user not pass in config-file08:49
gongyshthis is just for target vim, not the tacker's credential.08:49
trinathsgongysh: ok08:50
*** trinaths has left #tacker09:15
openstackgerritgongysh proposed openstack/tacker master: Continue to avoid temporary connection error to target VIM  https://review.openstack.org/44846609:31
openstackgerritgongysh proposed openstack/tacker master: Continue to avoid temporary connection error to target VIM  https://review.openstack.org/44846609:34
*** gongysh has quit IRC09:36
digahi09:49
openstackgerritMerged openstack/tacker master: Access the controller's mistral endpoint instead of VIM  https://review.openstack.org/43821110:33
*** tbh has quit IRC11:40
*** gongysh has joined #tacker11:49
openstackgerritgongysh proposed openstack/tacker master: move the left cirros 3.4 into 3.5 disk image  https://review.openstack.org/44852711:57
*** trinaths has joined #tacker11:58
trinathsgongysh: hi12:08
gongyshhi12:09
trinathsgongysh: wrt to your comments, at test_shell.py, I get a mispatch while I give the arguments. http://paste.openstack.org/show/603745/ so I'm assertig whether the function itself is called or not12:09
trinathsgongysh: auth and session params12:09
trinathsgongysh: in the actual call.12:10
trinathss/mispatch/mismatch/12:11
gongyshtrinaths,  show me the code position12:12
gongyshor your code12:13
trinathsgongysh: my code, http://paste.openstack.org/show/603748/12:13
trinathsgongysh: this works well, http://paste.openstack.org/show/603749/12:15
trinathsgongysh: without args12:15
gongyshtrinaths, without args,  the test does not make sense.12:16
trinathsgongysh: agreed. but how to generate auth and session ?12:18
gongyshthat is easy. the auth and session is not the main role. so you can just use mock.ANY12:19
trinathsgongysh: agree. with too I get this error. http://paste.openstack.org/show/603750/12:21
gongyshtrinaths, pleas compare two, you should add some in expected part.12:23
trinathsgongysh: yes, auth and session12:25
trinathsgongysh: auth and session traceable in expected call12:29
trinathsgongysh: got a fix.12:30
trinathsgongysh: let me test that and get back to you12:31
gongyshdkushwaha, hi12:33
*** tbachman has left #tacker13:06
openstackgerritgongysh proposed openstack/tacker master: Continue waiting to avoid temporary connection error with target VIM  https://review.openstack.org/44846613:24
*** janki has joined #tacker13:31
*** jchhatbar has joined #tacker13:34
*** gongysh has quit IRC13:35
*** janki has quit IRC13:38
*** KanagarajM has quit IRC13:41
*** jchhatbar has quit IRC13:47
*** janki has joined #tacker13:48
openstackgerritgongysh proposed openstack/tacker master: Continue waiting to avoid temporary connection error with target VIM  https://review.openstack.org/44846613:50
*** luisnho223 has joined #tacker14:01
luisnho223hi everyone. Im new to tacker and i have some doubts. I just installed devstack with tacker and i want to connect net0, net1 and net_management to my private network that i created14:03
luisnho223It is needed to launch VNF instances?14:03
*** bobh has joined #tacker14:03
*** bobh has quit IRC14:04
*** bobh has joined #tacker14:04
*** Aqsa has quit IRC14:05
*** Aqsa has joined #tacker14:08
openstackgerritgongysh proposed openstack/tacker master: Put vnfm cases into discovered case list  https://review.openstack.org/44860114:16
*** diga has quit IRC14:20
*** links has quit IRC14:21
*** trinaths has left #tacker14:23
openstackgerritgongysh proposed openstack/tacker master: Continue waiting to avoid temporary connection error with target VIM  https://review.openstack.org/44846614:24
luisnho223hi everyone. Im new to tacker and i have some doubts. I just installed devstack with tacker and i want to connect net0, net1 and net_management to my private network that i created. It is needed to launch VNF instances?14:26
*** janki has quit IRC14:41
*** Aqsa has quit IRC15:15
*** bobmel_ has joined #tacker15:37
*** Aqsa has joined #tacker16:15
*** Aqsam has joined #tacker16:16
*** luisnho223 has quit IRC16:17
*** Aqsa has quit IRC16:19
*** tbh has joined #tacker16:21
*** bobmel has joined #tacker16:46
*** links has joined #tacker16:48
*** bobmel_ has quit IRC16:50
*** uck has joined #tacker16:52
*** bobmel has quit IRC17:00
*** uck_ has joined #tacker17:21
*** uck has quit IRC17:21
*** links has quit IRC17:39
*** uck_ has quit IRC18:04
*** uck has joined #tacker18:04
*** trinaths has joined #tacker18:18
*** bobmel has joined #tacker18:34
*** bobmel has quit IRC18:34
*** bobmel has joined #tacker18:34
*** uck_ has joined #tacker18:49
*** uck has quit IRC18:49
*** vishnoianil has joined #tacker18:49
*** Aqsam has quit IRC18:55
*** bobmel has quit IRC19:33
*** trinaths has left #tacker19:43
*** Aqsa has joined #tacker19:51
*** uck_ has quit IRC19:56
*** tbh has quit IRC20:12
*** vishwanathj has quit IRC20:18
*** vishwanathj has joined #tacker20:18
*** uck has joined #tacker20:19
*** bobmel has joined #tacker20:34
*** bobmel has quit IRC20:39
*** Aqsa has quit IRC21:08
*** bobmel has joined #tacker21:35
*** bobmel has quit IRC21:40
*** Aqsa has joined #tacker21:43
*** bobmel has joined #tacker21:46
*** bobmel has quit IRC21:51
*** uck has quit IRC22:40
*** uck has joined #tacker22:40
*** bobmel has joined #tacker22:47
*** bobmel has quit IRC22:52
*** Aqsa has quit IRC23:10
*** bobh has quit IRC23:42

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