Monday, 2016-08-08

« Sunday, 2016-08-07 Index Tuesday, 2016-08-09 »
*** dkushwaha has joined #tacker00:07
*** diga has joined #tacker00:41
digasridhar_ram: Hi00:41
sridhar_ramdiga: hi00:41
*** trozet has quit IRC00:42
digasridhar_ram: I am going through the architecture now, It seems we are running everything on one service/port now00:42
digasridhar_ram: I am thinking to propose a distributed architecture like running main services on different ports00:43
digalike taker-api, tacker-nfvo, tacker-placement/vnfm00:44
sridhar_ramdiga: hmm.. why ? for load balancing ?00:45
digasridhar_ram: yes, but in the future this way, it will help us scale & performance wise it will be good00:45
digafor now, we can create tacker-api, tacker-placement/orchestration only00:46
sridhar_ramdiga: standard practice in openstack and other such scenarios it to run a LB like ha_proxy to front end your api service...00:46
sridhar_ramdiga: in fact, someone recently demo'd two tacker service working fine with an LB in the front00:47
digasridhar_ram: okay00:47
sridhar_ramdiga: i don't see a need to break up tacker into such individual service/port now.. rather we need..00:47
sridhar_ramdiga: .. support > 1 api_workers  .. we have a bug and there is a patchset out there to fix it00:48
digasridhar_ram: okay00:48
digasridhar_ram: then in the VNF distributed controller-slave implementation, should we use same architecture ?00:49
sridhar_ramdiga: btw, we have a general work item in this area .. to move to Pecan WSGI framework00:49
digasridhar_ram: yeah, thta's the good framework00:50
digasridhar_ram: if you want, I can take that action item on me, I had implemented WSME+PECAN for magnum00:52
sridhar_ramdiga: sure, that will be great.. sripriya was briefly looking at that at some point.. you may want to touch base w/ her00:53
digasridhar_ram: yeah , sure00:53
*** trozet has joined #tacker00:57
*** diga has quit IRC02:18
openstackgerritSridhar Ramaswamy proposed openstack/tacker: Fix tarball generation error due to missing tacker.conf  https://review.openstack.org/35218802:20
*** dazuko has joined #tacker02:31
*** dazuko has quit IRC02:35
openstackgerritTim Rozet proposed openstack/tacker: [WIP] Implements VNFFG into NFVO  https://review.openstack.org/34452203:33
*** trozet has quit IRC03:44
openstackgerritgongysh proposed openstack/tacker: Rename device into vnf, device_template into vnfd  https://review.openstack.org/35220503:47
*** gongysh has joined #tacker03:52
openstackgerritgongysh proposed openstack/tacker: Remove unused scheduler code  https://review.openstack.org/35221004:11
openstackgerritgongysh proposed openstack/tacker: Device refactor Part1: Rename device db name  https://review.openstack.org/34977604:13
openstackgerritgongysh proposed openstack/tacker: Device refactor Part2: Remove unused scheduler code  https://review.openstack.org/35221004:14
openstackgerritgongysh proposed openstack/tacker: Device refactor Part3: Rename device in codes  https://review.openstack.org/35220504:17
*** Ravikiran_K has joined #tacker04:29
*** Vijayendra_ has quit IRC04:41
*** Vijayendra has joined #tacker04:55
*** janki has joined #tacker05:06
*** santoshk has joined #tacker05:15
openstackgerritJanki Chhatbar proposed openstack/tacker: Change instance id to stack id in devices db  https://review.openstack.org/35008405:17
openstackgerritJanki Chhatbar proposed openstack/tacker: Add VNF resource details to get vnf API  https://review.openstack.org/34083805:26
*** santoshk has quit IRC06:19
jankivishwanathj, ping06:19
openstackgerritJanki Chhatbar proposed openstack/tacker: Add VNF resource details to get vnf API  https://review.openstack.org/34083806:25
jankivishwanathj: I am having similar error as you had about having 2 HEADs06:30
jankivishwanathj, how did you solve it?06:30
jankiI have tried rebasing06:30
*** diga has joined #tacker06:38
openstackgerritJianGang Weng proposed openstack/tacker: Remove unused LOG to keep code clean  https://review.openstack.org/34189906:45
*** lulei has joined #tacker06:53
openstackgerritgongysh proposed openstack/tacker: Device refactor Part3: Rename device in codes  https://review.openstack.org/35220506:56
*** kluka_ has quit IRC07:29
openstackgerritLu lei proposed openstack/tacker: Fix formats for doc's information  https://review.openstack.org/34128107:42
openstackgerritLu lei proposed openstack/tacker: Fix formats for doc's information  https://review.openstack.org/34128107:46
*** saju_m has joined #tacker07:46
-openstackstatus- NOTICE: Gerrit is going to be restarted08:40
*** lulei has quit IRC09:01
*** manikanta_tadi has joined #tacker09:07
*** diga has quit IRC09:32
openstackgerritgongysh proposed openstack/tacker: Device refactor Part3: Rename device in codes  https://review.openstack.org/35220509:35
*** manikanta_tadi has quit IRC10:19
*** lulei has joined #tacker10:23
*** Ravikiran_K has quit IRC10:40
openstackgerritgongysh proposed openstack/tacker: Device refactor Part3: Rename device in codes  https://review.openstack.org/35220510:51
openstackgerritOpenStack Proposal Bot proposed openstack/tacker: Updated from global requirements  https://review.openstack.org/35233711:09
*** zanetworker has joined #tacker11:14
openstackgerritgongysh proposed openstack/tacker: Device refactor Part3: Rename device in codes  https://review.openstack.org/35220511:16
*** zanetworker has quit IRC11:30
*** gongysh has quit IRC12:04
*** amotoki has joined #tacker12:31
*** Ravikiran_K has joined #tacker12:38
*** trozet has joined #tacker12:40
*** uck has joined #tacker12:50
*** amotoki has quit IRC12:56
*** tbh has joined #tacker13:08
*** amotoki has joined #tacker13:13
*** uck has quit IRC13:22
*** amotoki has quit IRC13:27
*** bobh has joined #tacker13:38
*** bobh has quit IRC13:44
*** bobh has joined #tacker13:45
vishwanathjjanki I had to rebase and resolve the merge conflicts13:53
*** Liuqing has joined #tacker14:00
jankivishwanathj: I did rebase14:09
jankiyet facing th issue14:10
jankiwhile resolving merge conflict, which HEAD number did you use?14:10
jankivishwanathj: ^^14:10
*** janki has quit IRC14:14
vishwanathjjanki do you mind sharing the link to the patchset that is having the issue14:33
*** DaveJ__ has joined #tacker14:34
openstackgerritvishwanath jayaraman proposed openstack/tacker: Support purge of soft-deleted resources from DB tables  https://review.openstack.org/32965214:38
*** saju_m has quit IRC14:51
*** amotoki has joined #tacker14:59
*** amotoki has quit IRC14:59
*** amotoki has joined #tacker15:01
*** amotoki has quit IRC15:08
*** amotoki has joined #tacker15:14
*** amotoki has quit IRC15:16
*** amotoki has joined #tacker15:16
*** amotoki_ has joined #tacker15:20
*** amotoki has quit IRC15:21
*** amotoki_ has quit IRC15:22
*** Liuqing has quit IRC15:43
*** Ravikiran_K has quit IRC15:45
*** uck has joined #tacker15:56
*** prashantD has joined #tacker15:56
*** amotoki has joined #tacker15:59
*** amotoki has quit IRC16:02
*** amotoki has joined #tacker16:02
vishwanathjamotoki hi16:07
*** santoshk has joined #tacker16:11
*** Ravikiran_K has joined #tacker16:11
trozetsridhar_ram: ping?16:15
*** amotoki has quit IRC16:15
*** santoshk has quit IRC16:23
openstackgerrityatin proposed openstack/tacker: Replace deprecated LOG.warn with LOG.warning  https://review.openstack.org/34316116:27
*** saju_m has joined #tacker16:47
*** uck has quit IRC17:02
*** uck has joined #tacker17:02
trozetsridhar_ram: there?17:16
*** uck has quit IRC17:23
*** bobh has quit IRC17:34
openstackgerritbharaththiruveedula proposed openstack/tacker-specs: Tacker VNFC support  https://review.openstack.org/33979817:35
*** uck has joined #tacker17:45
*** sripriya has joined #tacker17:45
*** prashantD_ has joined #tacker17:45
*** prashantD has quit IRC17:48
*** uck_ has joined #tacker17:50
*** uck has quit IRC17:50
sridhar_ramtrozet: hi, whats up ?17:50
*** prashantD_ has quit IRC17:50
trozetsridhar_ram: so wanted to talk to you about vnffg, do you have a few minutes?17:51
sridhar_ramtrozet: yes17:51
trozetsridhar_ram: ok so I'm pretty much done coding17:52
trozetsridhar_ram: I removed the constraint that the VNFD had to be created before VNFFGD is created17:52
trozetsridhar_ram: the processing is done at VNFFG create time to determine dependencies missing17:52
sridhar_ramtrozet: cool..17:53
trozetsridhar_ram: so I use Janki's patch to find the CP info...I need to rebase and list that patch as a dependency17:53
trozetsridhar_ram: I also had a comment for s3wong driver, but overall looks like we line up.  I'm passing driver info he needs in the format he wants17:54
sridhar_ramtrozet: great..17:54
sridhar_ramtrozet: is janki's patchset sufficient for your CP info needs ?17:54
trozetsridhar_ram: I believe it is, but I haven't tested the whole thing yet.  Going to do that either today or tomorrow.17:55
trozetsridhar_ram: so one thing I am seeing in CI with my patch are some failures in create_vnfd, etc http://logs.openstack.org/22/344522/13/check/gate-tacker-python27-db-ubuntu-xenial/0b0ba16/console.html17:55
trozetsridhar_ram: It looks like failures are around DB: FOREIGN KEY constraint failed [SQL: u'DELETE FROM vims']17:56
* sridhar_ram is looking up..17:56
trozetoslo_db.exception.DBDuplicateEntry: (sqlite3.IntegrityError) UNIQUE constraint failed: vims.id [SQL: u'INSERT INTO vims (tenant_id, id, type, name, description, placement_attr, shared, is_default, status) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)'] [parameters: ('ad7ebc56538745a08ef7c5e97f8bd437', '6261579e-d6f3-49ad-8bc3-a9cb974778ff', 'openstack', 'fake_vim', 'fake_vim_description', '{"regions": ["RegionOne"]}', 1, 0, 'Active')]17:56
trozetsridhar_ram: thanks.  I don't think I modify any of that stuff, and I include it in the alembic_migration path, so I'm not sure if it is something I broke, or if sometimes those fail in CI?17:57
sridhar_ramtrozet: CI usually never fails randomly in this area..17:57
trozetsridhar_ram: so then I broke something ;)17:57
sridhar_ramtrozet: one thing that is tripping us these days is newer mysql versions with extra error checks17:58
sridhar_ramtrozet: possibility of ur patchset causing is higher ;-)17:58
trozetsridhar_ram: but like that second one I pasted - that is a DuplicateEntry error...not sure how I would make that happen17:59
trozetsridhar_ram: or maybe that duplicate entry is because the previous test failed to delete17:59
sridhar_ramtrozet: hang on, there is a fix that went in recently...to move default vim to db17:59
trozetsridhar_ram: versions/d4f265e8eb9d_add_default_to_vim.py ?18:00
*** s3wong has joined #tacker18:00
sridhar_ramtrozet: yet..18:00
sridhar_ram*yes18:00
trozetsridhar_ram: so yeah, so if you look at alembic HEAD in my patch it is 50712291880018:00
trozetsridhar_ram: and that corresponds to  versions/507122918800_adds_vnffg.py:down_revision = '22f5385a3d4f'18:01
trozetsridhar_ram: and 22f5385a3d4f:  versions/22f5385a3d4f_remove_proxydb.py:Revises: d4f265e8eb9d18:01
trozetsridhar_ram: so my migration script -> remove_proxy -> move default vim...so it should be there I would think18:02
sridhar_ramtrozet: give me a min, running thru this up / down alembic chain ..18:04
trozetsridhar_ram: sure np18:04
trozets3wong: ping?18:05
*** santoshk has joined #tacker18:09
sridhar_ramtrozet: i agree, the chain looks okay18:09
trozetsridhar_ram: it looks like the tests are failing when it tries to do setup for vim18:13
trozet File "tacker/tests/unit/vm/test_plugin.py", line 146, in _insert_dummy_vim18:13
sridhar_ramtrozet: it could be a side effect of the previous test that ran w/o any error ...18:13
sridhar_ramtrozet: test_get_vim_without_defined_default_vim_in_db18:13
trozetsridhar_ram: ah!18:14
sridhar_ramtrozet: hang on, that's not correct..18:15
sridhar_ramtrozet: the FAILED ones are interspersed ..18:15
trozetsridhar_ram: right, but hte fact that one passed means that VIM table is there18:16
trozetsridhar_ram: and passed wthout the default in db18:16
sridhar_ramtrozet: the next possible suspect is .. test_nfvo_plugin.TestNfvoPlugin.test_update_vim18:16
sridhar_ramtrozet: but.. why only in ur patchset .. hmm, you are chosen one ? :)18:16
trozetsridhar_ram: i'm really good at committing crap code?18:17
sridhar_ramtrozet: your code exercises amazing, non-trivial corner cases ?18:17
trozetsridhar_ram: LOL18:18
sridhar_ramtrozet: btw, all test_plugin tests failed18:19
trozetsridhar_ram: test_update_vim deletes the table?18:19
trozetsridhar_ram: 2016-08-08 03:57:24.673350 |       File "tacker/tests/unit/db/base.py", line 41, in clear_tables18:19
trozet2016-08-08 03:57:24.673367 |         conn.execute(table.delete())18:19
trozetits a cleanup failure18:20
sridhar_ramtrozet: that just means, there are references to VIM entry in another table that hasn't got cleaned up18:22
sridhar_ramtrozet: do you've any unit tests in ur patchset ?18:23
trozetsridhar_ram: no I  haven't added them yet18:24
trozetsridhar_ram: I've tested it manually though with openstack, but I didn't test any vim stuff18:24
trozetsridhar_ram: so the likely thing here is somehow I broke the plugin or DB18:24
sridhar_ramtrozet: can u run 'tox -e py27' locally to see if this reproduces18:25
trozetsridhar_ram: yeah, I'm checking one thing real quick18:26
sridhar_ramtrozet: sure, i'll also leave this in for u..18:26
sridhar_ramtrozet: you can run specific tests locally.. 'tox -e py27 test_plugin'18:27
sridhar_ramtrozet: it will help to isolate if this failure is due to side effect from another test18:27
trozetsridhar_ram: cool, thanks.  I did modify the NfvoPluginDB inheritance model, so I'm wondering if there is a super call there that I broke18:28
*** bobh has joined #tacker18:34
sridhar_ramtrozet: ack!18:36
*** bobh has quit IRC18:39
trozetsridhar_ram: they fail when I run them in my dir as well18:41
trozetsridhar_ram: same failures18:41
*** bobh has joined #tacker18:42
*** tbh has quit IRC18:46
trozetsridhar_ram: and i see these tables are created, printed them during the db cleanup: https://paste.fedoraproject.org/404501/18:50
*** DaveJ__ has quit IRC18:51
*** Ravikiran_K has quit IRC18:51
*** uck_ has quit IRC18:57
*** bobh has quit IRC19:03
openstackgerritTung Doan proposed openstack/tacker: Alarm monitor: Update Heat driver  https://review.openstack.org/34772119:09
openstackgerritTung Doan proposed openstack/tacker: Alarm monitor: TOSCA definition  https://review.openstack.org/35118919:14
*** bobh has joined #tacker19:45
*** trozet has quit IRC19:53
*** uck has joined #tacker19:57
*** uck has quit IRC20:03
*** trozet has joined #tacker20:07
openstackgerritSridhar Ramaswamy proposed openstack/tacker: Fix tarball generation error due to missing tacker.conf  https://review.openstack.org/35218820:48
sridhar_rambobh: s3wong: few low hanging review to wrap.. https://review.openstack.org/#/c/350718/ , https://review.openstack.org/#/c/337646/ , https://review.openstack.org/#/c/346296 and https://review.openstack.org/#/c/349694/ .. please help20:53
sridhar_ramsripriya: ^^^20:53
s3wongsridhar_ram: ack20:54
sripriyasridhar_ram: ack20:55
sridhar_rams3wong: sripriya: thanks!20:56
sridhar_ramsripriya: one more, probably in ur alley.. https://review.openstack.org/#/c/343097/20:56
openstackgerritMerged openstack/tacker: Updated from global requirements  https://review.openstack.org/35233720:57
openstackgerritMerged openstack/tacker-horizon: Add domain name field for VIM register  https://review.openstack.org/35071821:02
openstackgerritMerged openstack/tacker: Fixes manual installation doc  https://review.openstack.org/33764621:12
openstackgerritMerged openstack/tacker: Remove execute permission on one file  https://review.openstack.org/34629621:16
openstackgerritMerged openstack/tacker: Always use upper-constraints  https://review.openstack.org/34969421:19
openstackgerritsajuptpm proposed openstack/tacker: doc change default vim registration via CLI  https://review.openstack.org/34309721:22
*** bobh has quit IRC21:23
openstackgerritsajuptpm proposed openstack/tacker: doc change default vim registration via CLI  https://review.openstack.org/34309721:25
*** bobh has joined #tacker21:33
*** sripriya has quit IRC22:01
*** sripriya has joined #tacker22:02
*** uck has joined #tacker22:04
*** bobh has quit IRC22:10
*** uck has quit IRC22:17
openstackgerritsajuptpm proposed openstack/tacker: doc change default vim registration via CLI  https://review.openstack.org/34309722:18
*** saju_m has quit IRC22:25
*** uck has joined #tacker22:57
*** vishwanathj has quit IRC23:20
*** vishnoianil has quit IRC23:59
« Sunday, 2016-08-07 Index Tuesday, 2016-08-09 »

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