Tuesday, 2017-04-11

*** amotoki has joined #tacker00:20
openstackgerritMerged openstack/python-tackerclient master: Updated from global requirements  https://review.openstack.org/45453700:24
openstackgerritMerged openstack/tacker master: Updated from global requirements  https://review.openstack.org/45454400:25
*** yifei has joined #tacker00:42
openstackgerritMerged openstack/tacker master: Tacker server throws exception when member action contains UPDATE string.  https://review.openstack.org/45241600:53
*** tung_doan has joined #tacker01:21
*** s3wong has quit IRC01:25
*** diga has joined #tacker02:34
*** avishnoi has quit IRC02:37
*** diga has quit IRC02:48
*** diga has joined #tacker02:49
*** dkushwaha has quit IRC02:51
*** dkushwaha has joined #tacker03:02
*** diga has quit IRC03:03
*** diga has joined #tacker03:10
*** KanagarajM has joined #tacker03:19
*** amotoki has quit IRC03:26
*** amotoki has joined #tacker03:40
*** amotoki has quit IRC03:50
*** amotoki has joined #tacker03:58
*** diga has quit IRC03:58
*** amotoki has quit IRC04:13
*** trinaths has joined #tacker04:16
*** amotoki has joined #tacker04:29
*** diga has joined #tacker04:37
*** diga has quit IRC04:44
openstackgerritTrinath Somanchi proposed openstack/tacker-specs master: Tacker support in python-openstackclient  https://review.openstack.org/45518805:25
*** mohankumar_ has joined #tacker05:32
*** janki has joined #tacker05:39
*** avishnoi has joined #tacker05:48
*** manikanta_tadi has joined #tacker06:21
*** jchhatbar has joined #tacker06:25
*** janki has quit IRC06:25
*** jchhatbar is now known as janki06:25
*** manikanta_tadi has quit IRC06:29
*** mohankumar_ has quit IRC06:32
*** trinaths has quit IRC06:41
*** Aqsa has joined #tacker07:00
*** Aqsa has quit IRC07:08
*** trinaths has joined #tacker07:30
trinathssridhar_ram: Hi07:32
trinathsdkushwaha: ping07:32
dkushwahatrinaths, hi07:49
trinathsdkushwaha: please look into https://review.openstack.org/#/c/449956/, this fix must go in P1.07:53
trinathsdkushwaha: pike-107:54
dkushwahatrinaths, sure, will review it07:55
trinathsdkushwaha: thanks a lot. :)07:56
*** mohankumar_ has joined #tacker08:15
*** derekjhyang has joined #tacker08:43
*** janki has quit IRC08:44
*** janki has joined #tacker08:45
*** bobmel has joined #tacker09:01
*** openstackgerrit has quit IRC09:03
*** trinaths has left #tacker10:27
*** trinaths has joined #tacker11:09
trinathsdkushwaha: ping11:09
trinathsdkushwaha: the FT must return success, since it caught exception and we its assertEqual.11:10
*** jchhatbar has joined #tacker11:12
*** janki has quit IRC11:12
trinathsdkushwaha: I get an exception when I re run the FT again, please see, http://paste.openstack.org/show/606136/11:13
*** jchhatbar has quit IRC11:16
*** jchhatbar has joined #tacker11:16
dkushwahatrinaths, you are getting this error because creating vim with same name, which alrady exist11:19
trinathsdkushwaha: yes11:19
trinathsdkushwaha: in the FT we dont get this error since, we are asserting the exception raised.11:20
trinathsdkushwaha: w.r.t your comments on https://review.openstack.org/#/c/449956/, the FT will result success since we assert equal and get the exception.11:21
dkushwahatrinaths, my point is , we are not allowed to create the new vim with same name(vim already exist), but as per fix we can update the vim name which can be same as existing one11:21
trinathsdkushwaha: okay.11:21
dkushwahatrinaths, which should not be allowed11:22
trinathsdkushwaha:  we are not allowed to create the new vim with same name(vim already exist), but as per fix we can update the vim name which can be same as existing one - is handled.11:22
trinathsdkushwaha:  https://review.openstack.org/#/c/449956/24/tacker/tests/functional/nfvo/test_vim.py L7011:23
trinathsdkushwaha: Do you mean that, we need to validate duplicate entry at https://review.openstack.org/#/c/449956/24/tacker/db/nfvo/nfvo_db.py L186 ?11:29
dkushwahatrinaths, I understand, In L75 we are creating with the updated name and it will raise exception, that is perfectly correct. But suppose we have 2 vims which names are "vim1" and "vim2".  As per this fix, if we update the name of first vim(i.e. "vim1") with  the new name "vim2" . It will be success and we will have 2 vims with the same name11:30
trinathsdkushwaha: that must fail, with exception,at https://review.openstack.org/#/c/449956/24/tacker/db/nfvo/nfvo_db.py L18611:31
dkushwahatrinaths, Ok I will verify again and will share my execution steps after few hours.11:35
trinathsdkushwaha: okay sir. thank you very much :)11:36
dkushwahatrinaths, leaving office now, catch you later :)11:37
*** jchhatbar is now known as janki11:56
*** KanagarajM has quit IRC12:03
*** amotoki has quit IRC12:11
*** amotoki has joined #tacker12:23
*** amotoki has quit IRC12:39
*** amotoki has joined #tacker12:43
*** amotoki has quit IRC12:49
trinathsdkushwaha: ping13:00
*** openstackgerrit has joined #tacker13:01
openstackgerritTrinath Somanchi proposed openstack/tacker master: Tacker Server side changes for UPDATE VIM operation.  https://review.openstack.org/44995613:01
trinathsdkushwaha: I have added another testcase, test_vim_duplicate_entry in test_vim.py to address your review comments in P2413:03
*** bobh has joined #tacker13:05
*** trinaths has left #tacker13:07
*** amotoki has joined #tacker13:12
*** Vijayendra has joined #tacker13:26
*** janki has quit IRC13:48
*** KanagarajM has joined #tacker13:53
-openstackstatus- NOTICE: latest base images have mistakenly put python3 in some places expecting python2 causing widespread failure of docs patches - fixes are underway14:28
-openstackstatus- NOTICE: we have rolled back centos-7, fedora-25 and ubuntu-xenial images to the previous days release. Feel free to recheck your jobs now.14:48
*** KanagarajM has quit IRC15:17
*** mohankumar_ has quit IRC15:37
*** arturt_ has joined #tacker15:43
*** trinaths has joined #tacker15:58
*** arturt_ has quit IRC15:58
*** arturt_ has joined #tacker15:59
*** arturt_ has quit IRC16:00
*** arturt_ has joined #tacker16:00
*** arturt_ has quit IRC16:05
*** arturt_ has joined #tacker16:05
trinathsdkushwaha: ping16:07
*** arturt_ has quit IRC16:12
*** arturt_ has joined #tacker16:13
*** mohankumar has joined #tacker16:18
*** arturt_ has quit IRC16:26
*** trinaths has left #tacker16:31
*** amotoki has quit IRC17:04
*** mohankumar has quit IRC17:05
*** mohankumar has joined #tacker17:09
*** mohankumar has quit IRC17:43
*** Aqsa has joined #tacker17:55
*** s3wong has joined #tacker18:04
*** avishnoi has quit IRC18:11
*** arturt_ has joined #tacker18:43
*** bobh has quit IRC18:59
*** avishnoi has joined #tacker19:12
*** avishnoi has quit IRC19:42
*** avishnoi has joined #tacker19:42
openstackgerritdharmendra kushwaha proposed openstack/tacker master: Adds unique constraint on deleted_at  https://review.openstack.org/45542119:49
*** bobh has joined #tacker20:00
*** Aqsa has quit IRC20:03
*** bobh has quit IRC20:05
*** bobh has joined #tacker20:33
*** arturt_ has quit IRC20:34
*** bobh has quit IRC22:18
openstackgerritSridhar Ramaswamy proposed openstack/tacker master: [WIP] Unify OpenStack VIM driver  https://review.openstack.org/45048722:25

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