Monday, 2016-04-18

*** bobh has quit IRC00:33
*** bobh has joined #tacker00:49
*** bobh has quit IRC00:56
*** bobh has joined #tacker01:26
*** bobh has quit IRC01:32
openstackgerritTin Lam proposed openstack/python-tackerclient: [WIP][TEST] Adding py3 support to python-tackerclient  https://review.openstack.org/30688601:42
*** gongysh has joined #tacker01:48
*** bobh has joined #tacker02:33
*** gongysh_ has joined #tacker02:42
openstackgerritcaoyue proposed openstack/tacker: Fix the hacking rules  https://review.openstack.org/30689002:42
openstackgerritTin Lam proposed openstack/python-tackerclient: [WIP][TEST] Adding py3 support to python-tackerclient  https://review.openstack.org/30688602:45
*** bobh has quit IRC02:50
openstackgerritcaoyue proposed openstack/tacker: Add debug testenv in tox  https://review.openstack.org/30689302:57
*** dkushwaha has joined #tacker02:57
*** amotoki has joined #tacker03:11
*** amotoki has quit IRC04:11
*** amotoki has joined #tacker04:22
*** amotoki has quit IRC04:30
*** gongysh_ has quit IRC04:34
*** amotoki has joined #tacker04:38
*** gongysh_ has joined #tacker04:50
*** amotoki has quit IRC05:01
*** amotoki has joined #tacker05:09
*** zeih has joined #tacker06:31
openstackgerritcaoyue proposed openstack/tacker: Remove virtualenv management scripts from oslo-incubator  https://review.openstack.org/30692706:44
*** mbound has joined #tacker07:17
*** vishnoianil has quit IRC07:43
*** yifei1 has joined #tacker11:00
*** igordcard has joined #tacker11:04
*** yifei has quit IRC11:05
*** igordcar1 has quit IRC11:05
*** gongysh has quit IRC11:06
*** yifei1 is now known as yifei11:06
*** uck has joined #tacker11:18
*** bobh has joined #tacker11:32
*** mbound has quit IRC12:00
*** bobh has quit IRC12:25
*** mbound has joined #tacker12:30
*** zeih has quit IRC12:36
*** zeih has joined #tacker12:37
*** mageshgv has joined #tacker12:42
*** amotoki has quit IRC12:59
*** trozet has joined #tacker13:11
*** trozet has quit IRC13:14
*** trozet has joined #tacker13:15
*** amotoki has joined #tacker13:19
openstackgerritdharmendra kushwaha proposed openstack/tacker: Prevent update state to PENDING_UPDATE while initials are not done  https://review.openstack.org/29293213:30
*** amotoki has quit IRC13:34
*** amotoki has joined #tacker13:40
*** zeih has quit IRC13:43
*** zeih has joined #tacker13:43
*** amotoki has quit IRC13:49
*** amotoki has joined #tacker13:59
*** Berezin has joined #tacker13:59
*** zeih has quit IRC14:03
*** uck has quit IRC14:09
*** uck has joined #tacker14:10
*** sridhar_ram has joined #tacker14:30
*** sridhar_ram has quit IRC14:31
*** sridhar_ram has joined #tacker14:37
*** bobh has joined #tacker14:48
*** dkushwaha_ has joined #tacker14:48
*** sridhar_ram has quit IRC15:05
*** dkushwaha has quit IRC15:09
*** dkushwaha_ is now known as dkushwaha15:13
dkushwahagongysh_, hi15:32
dkushwahagongysh_, Could you please review the patch-set https://review.openstack.org/#/c/29293215:34
*** bobh has quit IRC15:38
*** uck has quit IRC16:03
*** mbound has quit IRC16:08
*** tbh has joined #tacker16:34
*** bobh has joined #tacker16:40
*** uck has joined #tacker16:48
*** tbh has quit IRC16:48
*** Berezin has quit IRC16:49
*** sridhar_ram has joined #tacker16:54
*** jdowner has joined #tacker17:09
*** dkushwaha has quit IRC17:19
*** jdowner has left #tacker17:20
*** bobh has quit IRC17:23
*** bobh has joined #tacker17:24
*** bobh has quit IRC17:29
*** bobh has joined #tacker17:30
*** sridhar_ram1 has joined #tacker17:32
*** sridhar_ram has quit IRC17:33
*** sripriya has joined #tacker17:48
*** s3wong has joined #tacker17:51
sripriyabobh sridhar_ram1: s3wong: can you pl review https://review.openstack.org/#/c/306271/17:56
bobhsripriya: will do17:56
s3wongsripriya: sure17:56
sripriyabobh: this fix is to make tosca template compatible for heat kilo version17:57
sripriyabobh: s3wong: thanks17:57
sripriyabobh: it also fixes anti_spoofing_protection keyword. as of now , port_security_enabled is the only incompatible attr between kilo and higher versions17:58
s3wongsripriya: yeah, the famous anti_spoof vs anti_spoofing discussion...17:58
* bobh still thinks he was right...17:59
*** lhcheng has joined #tacker17:59
* s3wong sides with bobh, but is obligated to agree with community :-) 17:59
bobhsripriya: I looked at this quick over the weekend (emphasis on quick) - should there be some way of tracking which release the data change affects, in the data structure?18:00
sripriyas3wong: :-) even though they are no different for a developer and is more of editing work, the call for anti_spoof_proection per the boss has to be followed ;)18:00
* bobh senses a revolution in the offing :-)18:01
* sridhar_ram1 thinks of ways how to clamp down this mutiny18:03
sripriyabobh: should we really care which release it was incomatible until we use that to determine if an attr needs to be modified or not. the only we track if a prop is incompatible is querying the resource_type itself if it supports that attr18:04
bobhsripriya: I just finished reading the code I skipped over the weekend, so it makes sense now18:05
sripriyasridhar_ram1: no you cant ;-18:05
sripriyabobh: cool, pl ping back if you have any more questions18:06
bobhsridhar_ram1: chocolate usually works...18:07
bobhsripriya: just wondering if unsupported_res_prop should default to False for safety/readability in the definition of _process_vdu_network_interfaces18:08
sripriyabobh: you mean None? yes that should be added, pl leave a comment on the patch18:09
bobhsripriya: obviously what I meant :-)  will do - otherwise LGTM18:10
*** sridhar_ram1 is now known as sridhar_ram18:10
sripriyabobh: thanks18:11
sridhar_rambobh: any combination of alcohol (wine, beer as suits) + chocolate will even things out..18:11
bobhsridhar_ram: crisis averted...18:12
sridhar_ramon that subject, are you folks open for an informal social after Thursday design summit next week18:12
sripriyasridhar_ram: a treat from the PTL ? :-) ofcourse yes18:15
sridhar_ramsripriya: perhaps.. :)18:17
sridhar_rambobh: sripriya: hang on for that 'anti_spoof{ing}_protection' .... checking some discrepancy in the TOSCA spec going out...18:18
* sripriya starts to think probably bobh's voice was heard by the world18:19
* bobh isn't sure that's necessarily a good thing...18:20
sridhar_rambobh: sripriya: you'll be damned when you hear this..18:26
sridhar_ramif you happen to have access to OASIS access, see this upcoming verion https://www.oasis-open.org/apps/org/workgroup/tosca/download.php/57669/tosca-nfv-v1.0-wd02-rev09.docx18:27
bobhsridhar_ram: I'm sure Nokia has access but I don't have a login18:28
*** mbound has joined #tacker18:28
* sridhar_ram interrupt.. will be back in 15mins18:29
* bobh just submitted the form for a login....18:30
sripriyabobh: and the famous battle of spoof/spoofing continues... :-)18:38
bobhsripriya: let me guess - the spec has it both ways in the text18:38
sripriyabobh: yup!18:39
bobhsridhar_ram: HAH!18:39
bobhsridhar_ram: sorry, couldn't resist...18:41
bobhsripriya: so the code is correct and the spec is wrong :-)18:47
sripriyabobh: well you never know, the final verdict is not out yet :-)18:48
bobhsripriya: but the code matches the spec so clearly the code doesn't need to change :-)18:49
*** uck_ has joined #tacker18:50
*** uck has quit IRC18:50
sripriyabobh: what if the spec is kept consistent with 'anti_spoof_protection' ?18:50
bobhsripriya: then it would break the code - seems like that would just create more work, so it should be consistent with the code18:51
* bobh knows he's really reaching now...18:52
sripriyabobh: haha nice try :-)18:53
* sridhar_ram wanders back in ...19:26
sridhar_rambobh: alas, you are sort-of correct.. couple of examples indeed using 'anti_spoofing_protection' ... however ...19:27
sridhar_rambobh: .. the main schema definition still uses 'anti_spoof_protection' ..19:27
bobhsridhar_ram: so easier to correct one line than several? :-)19:27
sridhar_ramyep19:28
*** sridhar_ram_ has joined #tacker19:28
* bobh is hoping there are several examples and only one definition....19:29
openstackgerritLouis Fourie proposed openstack/tacker-specs: Tacker and networking-sfc Integration  https://review.openstack.org/29077120:01
*** lhcheng has quit IRC20:02
*** vishnoianil has joined #tacker20:07
*** vishnoianil has quit IRC20:13
*** vishnoianil has joined #tacker20:14
sripriyabobh: sridhar_ram: and the final decision is?20:17
sridhar_ramsripriya: bobh: I agree w/ bobh that "anti_spoofing_protection" is the correct usage. I'll create a JIRA bug against the TOSCA standard and fix it in the next release.20:20
sridhar_ramI'm siding with lets-do-the-right-thing rule :)20:21
* sripriya grunts that she has to fix all the files in the patch again :-| 20:21
bobhsripriya: sorry about that...20:22
* sridhar_ram increases the chocolates count for sripriya in next week's social20:22
sripriyabobh: finally! your code stays intact :)20:22
*** vishwanathj has joined #tacker20:22
sripriyasridhar_ram: :-) your treats are already overdue20:23
sridhar_rambobh: on a related topic, we need to have a cross-project design summit session w/ Sahdev / heat-translator team20:23
sridhar_rambobh: have you discussed this w/ spzala ?20:24
bobhsridhar_ram: not directly but I'm sure he'd be happy to20:24
sridhar_rambobh: okay... pinging in heat-translator channel20:25
*** lhcheng has joined #tacker20:31
vishwanathjbobh need help....not sure why my code updates would fail the test_tosca_vnf_multiple_vdu.py and test_vnf_monitoring.py tests http://logs.openstack.org/03/300903/7/check/gate-tacker-dsvm-functional/19565d5/console.html20:31
*** sridhar_ram has quit IRC20:31
bobhvishwanathj: I'll take a look20:32
*** vishnoianil has quit IRC20:36
bobhvishwanathj: wondering if the changes to the 'name' code in heat.py are causing the functional test failures?20:39
bobhvishwanathj: see the note inline20:40
*** zeih has joined #tacker20:51
vishwanathjbobh, I have had name change since the beginning of the patch set20:51
bobhvishwanathj: ok - maybe recheck?  Or can you run the func tests on a local testbed and see if you can reproduce the problem?20:52
vishwanathji did a recheck as well20:52
bobhvishwanathj: not much helpful info in the output20:53
sripriyavishwanathj: are the func testcases failing in every run?20:53
*** sridhar_ram has joined #tacker20:54
vishwanathjnope, patchset 7 was the first time it failed....20:54
vishwanathjI added a new test called test_create_tosca_vnic_normal and other related code changes in response to sridhar's comments20:55
bobhvishwanathj: none of the changes in 7 should affect functional tests20:55
bobhvishwanathj: did you have to rebase between 6 and 7 for changes in master?20:56
vishwanathjbobh hence, I was surprised and clueless20:56
sripriyavishwanathj: looks like a timing issue on jenkins where the devstack exceeded the quota on no. of instanced spawned and running20:56
sripriyavishwanathj: instances*20:56
vishwanathjbobh I did not have to rebase srpriya, i did a recheck twice and saw the same failure, hopefully your analysis is right, I will upload a patchset shortly addressing gongysh comments20:57
openstackgerritvishwanath jayaraman proposed openstack/tacker: Implements SR-IOV support  https://review.openstack.org/30090321:00
*** zeih_ has joined #tacker21:36
*** zeih has quit IRC21:40
*** zeih_ has quit IRC21:40
*** mageshgv has quit IRC21:43
*** mageshgv has joined #tacker21:57
openstackgerritSridhar Ramaswamy proposed openstack/tacker: Ignore: Test to test tackerclient deprecation warning fix  https://review.openstack.org/30750522:07
*** uck has joined #tacker22:10
*** uck_ has quit IRC22:10
sridhar_ramvishwanathj: sripriya: bobh: heads up - huge delay in gate CI results.. for gerrit #307505 ETA is 7 hours :-(22:10
sridhar_ramhttp://status.openstack.org/zuul/22:10
sripriyasridhar_ram: yup there are 601 jobs22:11
sripriyasridhar_ram: super monday :-)22:11
*** mbound has quit IRC22:12
sridhar_ramsripriya: yep.. ! mad craze before the summit22:13
*** mbound has joined #tacker22:15
vishwanathjsridhar_ram, I noticed that, thanks again for the heads up22:19
*** Berezin has joined #tacker22:20
sridhar_ramvishwanathj: sure !22:27
sridhar_rambobh: spzala confirmed Thursday 3:10 slot.. for tosca / heat-translator ..22:28
bobhsridhar_ram: sounds good22:28
sridhar_rambobh: I'm putting you to lead the discussion... one request from you...22:28
sridhar_rambobh: .. can you create an etherpad with all things around tacker-tosca .. ? we will use that to conduct / colloborate during the design session22:29
bobhsridhar_ram: sure, np22:29
sridhar_rambobh: thanks!22:30
sridhar_ramtrozet: ping22:32
s3wongsridhar_ram: from Mac's Xzure IRC client, trozet is transparent, meaning he has been inactive for a while22:33
*** sridhar_ram_ has quit IRC22:44
*** uck_ has joined #tacker22:48
*** uck has quit IRC22:48
*** bobh has quit IRC22:57
sridhar_ramvishwanathj: ping23:05
*** mageshgv has quit IRC23:07
*** bobh has joined #tacker23:13
*** bobh has quit IRC23:14
*** uck_ has quit IRC23:16
*** bobh has joined #tacker23:16
*** mbound has quit IRC23:20
*** dkushwaha has joined #tacker23:37
vishwanathjsridhar_ram what's up?23:39
*** s3wong has quit IRC23:47
sridhar_ramvishwanathj: I reviewing https://review.openstack.org/#/c/300903 .. had a question..23:56
vishwanathjwhat's the question?23:57
sridhar_ramvishwanathj: .. I think I got the answer, which is a CP can have both "management: true" and "type: sriov" ..23:57
sridhar_ramvishwanathj: .. is this assumption, correct ?23:57
vishwanathjyes23:57
sridhar_ramcool.. thanks!23:57
vishwanathjI am adding a test case per your comment23:58

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