*** bobh has quit IRC | 00:33 | |
*** bobh has joined #tacker | 00:49 | |
*** bobh has quit IRC | 00:56 | |
*** bobh has joined #tacker | 01:26 | |
*** bobh has quit IRC | 01:32 | |
openstackgerrit | Tin Lam proposed openstack/python-tackerclient: [WIP][TEST] Adding py3 support to python-tackerclient https://review.openstack.org/306886 | 01:42 |
---|---|---|
*** gongysh has joined #tacker | 01:48 | |
*** bobh has joined #tacker | 02:33 | |
*** gongysh_ has joined #tacker | 02:42 | |
openstackgerrit | caoyue proposed openstack/tacker: Fix the hacking rules https://review.openstack.org/306890 | 02:42 |
openstackgerrit | Tin Lam proposed openstack/python-tackerclient: [WIP][TEST] Adding py3 support to python-tackerclient https://review.openstack.org/306886 | 02:45 |
*** bobh has quit IRC | 02:50 | |
openstackgerrit | caoyue proposed openstack/tacker: Add debug testenv in tox https://review.openstack.org/306893 | 02:57 |
*** dkushwaha has joined #tacker | 02:57 | |
*** amotoki has joined #tacker | 03:11 | |
*** amotoki has quit IRC | 04:11 | |
*** amotoki has joined #tacker | 04:22 | |
*** amotoki has quit IRC | 04:30 | |
*** gongysh_ has quit IRC | 04:34 | |
*** amotoki has joined #tacker | 04:38 | |
*** gongysh_ has joined #tacker | 04:50 | |
*** amotoki has quit IRC | 05:01 | |
*** amotoki has joined #tacker | 05:09 | |
*** zeih has joined #tacker | 06:31 | |
openstackgerrit | caoyue proposed openstack/tacker: Remove virtualenv management scripts from oslo-incubator https://review.openstack.org/306927 | 06:44 |
*** mbound has joined #tacker | 07:17 | |
*** vishnoianil has quit IRC | 07:43 | |
*** yifei1 has joined #tacker | 11:00 | |
*** igordcard has joined #tacker | 11:04 | |
*** yifei has quit IRC | 11:05 | |
*** igordcar1 has quit IRC | 11:05 | |
*** gongysh has quit IRC | 11:06 | |
*** yifei1 is now known as yifei | 11:06 | |
*** uck has joined #tacker | 11:18 | |
*** bobh has joined #tacker | 11:32 | |
*** mbound has quit IRC | 12:00 | |
*** bobh has quit IRC | 12:25 | |
*** mbound has joined #tacker | 12:30 | |
*** zeih has quit IRC | 12:36 | |
*** zeih has joined #tacker | 12:37 | |
*** mageshgv has joined #tacker | 12:42 | |
*** amotoki has quit IRC | 12:59 | |
*** trozet has joined #tacker | 13:11 | |
*** trozet has quit IRC | 13:14 | |
*** trozet has joined #tacker | 13:15 | |
*** amotoki has joined #tacker | 13:19 | |
openstackgerrit | dharmendra kushwaha proposed openstack/tacker: Prevent update state to PENDING_UPDATE while initials are not done https://review.openstack.org/292932 | 13:30 |
*** amotoki has quit IRC | 13:34 | |
*** amotoki has joined #tacker | 13:40 | |
*** zeih has quit IRC | 13:43 | |
*** zeih has joined #tacker | 13:43 | |
*** amotoki has quit IRC | 13:49 | |
*** amotoki has joined #tacker | 13:59 | |
*** Berezin has joined #tacker | 13:59 | |
*** zeih has quit IRC | 14:03 | |
*** uck has quit IRC | 14:09 | |
*** uck has joined #tacker | 14:10 | |
*** sridhar_ram has joined #tacker | 14:30 | |
*** sridhar_ram has quit IRC | 14:31 | |
*** sridhar_ram has joined #tacker | 14:37 | |
*** bobh has joined #tacker | 14:48 | |
*** dkushwaha_ has joined #tacker | 14:48 | |
*** sridhar_ram has quit IRC | 15:05 | |
*** dkushwaha has quit IRC | 15:09 | |
*** dkushwaha_ is now known as dkushwaha | 15:13 | |
dkushwaha | gongysh_, hi | 15:32 |
dkushwaha | gongysh_, Could you please review the patch-set https://review.openstack.org/#/c/292932 | 15:34 |
*** bobh has quit IRC | 15:38 | |
*** uck has quit IRC | 16:03 | |
*** mbound has quit IRC | 16:08 | |
*** tbh has joined #tacker | 16:34 | |
*** bobh has joined #tacker | 16:40 | |
*** uck has joined #tacker | 16:48 | |
*** tbh has quit IRC | 16:48 | |
*** Berezin has quit IRC | 16:49 | |
*** sridhar_ram has joined #tacker | 16:54 | |
*** jdowner has joined #tacker | 17:09 | |
*** dkushwaha has quit IRC | 17:19 | |
*** jdowner has left #tacker | 17:20 | |
*** bobh has quit IRC | 17:23 | |
*** bobh has joined #tacker | 17:24 | |
*** bobh has quit IRC | 17:29 | |
*** bobh has joined #tacker | 17:30 | |
*** sridhar_ram1 has joined #tacker | 17:32 | |
*** sridhar_ram has quit IRC | 17:33 | |
*** sripriya has joined #tacker | 17:48 | |
*** s3wong has joined #tacker | 17:51 | |
sripriya | bobh sridhar_ram1: s3wong: can you pl review https://review.openstack.org/#/c/306271/ | 17:56 |
bobh | sripriya: will do | 17:56 |
s3wong | sripriya: sure | 17:56 |
sripriya | bobh: this fix is to make tosca template compatible for heat kilo version | 17:57 |
sripriya | bobh: s3wong: thanks | 17:57 |
sripriya | bobh: it also fixes anti_spoofing_protection keyword. as of now , port_security_enabled is the only incompatible attr between kilo and higher versions | 17:58 |
s3wong | sripriya: yeah, the famous anti_spoof vs anti_spoofing discussion... | 17:58 |
* bobh still thinks he was right... | 17:59 | |
*** lhcheng has joined #tacker | 17:59 | |
* s3wong sides with bobh, but is obligated to agree with community :-) | 17:59 | |
bobh | sripriya: 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 |
sripriya | s3wong: :-) 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 mutiny | 18:03 | |
sripriya | bobh: 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 attr | 18:04 |
bobh | sripriya: I just finished reading the code I skipped over the weekend, so it makes sense now | 18:05 |
sripriya | sridhar_ram1: no you cant ;- | 18:05 |
sripriya | bobh: cool, pl ping back if you have any more questions | 18:06 |
bobh | sridhar_ram1: chocolate usually works... | 18:07 |
bobh | sripriya: just wondering if unsupported_res_prop should default to False for safety/readability in the definition of _process_vdu_network_interfaces | 18:08 |
sripriya | bobh: you mean None? yes that should be added, pl leave a comment on the patch | 18:09 |
bobh | sripriya: obviously what I meant :-) will do - otherwise LGTM | 18:10 |
*** sridhar_ram1 is now known as sridhar_ram | 18:10 | |
sripriya | bobh: thanks | 18:11 |
sridhar_ram | bobh: any combination of alcohol (wine, beer as suits) + chocolate will even things out.. | 18:11 |
bobh | sridhar_ram: crisis averted... | 18:12 |
sridhar_ram | on that subject, are you folks open for an informal social after Thursday design summit next week | 18:12 |
sripriya | sridhar_ram: a treat from the PTL ? :-) ofcourse yes | 18:15 |
sridhar_ram | sripriya: perhaps.. :) | 18:17 |
sridhar_ram | bobh: 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 world | 18:19 | |
* bobh isn't sure that's necessarily a good thing... | 18:20 | |
sridhar_ram | bobh: sripriya: you'll be damned when you hear this.. | 18:26 |
sridhar_ram | if 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.docx | 18:27 |
bobh | sridhar_ram: I'm sure Nokia has access but I don't have a login | 18:28 |
*** mbound has joined #tacker | 18:28 | |
* sridhar_ram interrupt.. will be back in 15mins | 18:29 | |
* bobh just submitted the form for a login.... | 18:30 | |
sripriya | bobh: and the famous battle of spoof/spoofing continues... :-) | 18:38 |
bobh | sripriya: let me guess - the spec has it both ways in the text | 18:38 |
sripriya | bobh: yup! | 18:39 |
bobh | sridhar_ram: HAH! | 18:39 |
bobh | sridhar_ram: sorry, couldn't resist... | 18:41 |
bobh | sripriya: so the code is correct and the spec is wrong :-) | 18:47 |
sripriya | bobh: well you never know, the final verdict is not out yet :-) | 18:48 |
bobh | sripriya: but the code matches the spec so clearly the code doesn't need to change :-) | 18:49 |
*** uck_ has joined #tacker | 18:50 | |
*** uck has quit IRC | 18:50 | |
sripriya | bobh: what if the spec is kept consistent with 'anti_spoof_protection' ? | 18:50 |
bobh | sripriya: then it would break the code - seems like that would just create more work, so it should be consistent with the code | 18:51 |
* bobh knows he's really reaching now... | 18:52 | |
sripriya | bobh: haha nice try :-) | 18:53 |
* sridhar_ram wanders back in ... | 19:26 | |
sridhar_ram | bobh: alas, you are sort-of correct.. couple of examples indeed using 'anti_spoofing_protection' ... however ... | 19:27 |
sridhar_ram | bobh: .. the main schema definition still uses 'anti_spoof_protection' .. | 19:27 |
bobh | sridhar_ram: so easier to correct one line than several? :-) | 19:27 |
sridhar_ram | yep | 19:28 |
*** sridhar_ram_ has joined #tacker | 19:28 | |
* bobh is hoping there are several examples and only one definition.... | 19:29 | |
openstackgerrit | Louis Fourie proposed openstack/tacker-specs: Tacker and networking-sfc Integration https://review.openstack.org/290771 | 20:01 |
*** lhcheng has quit IRC | 20:02 | |
*** vishnoianil has joined #tacker | 20:07 | |
*** vishnoianil has quit IRC | 20:13 | |
*** vishnoianil has joined #tacker | 20:14 | |
sripriya | bobh: sridhar_ram: and the final decision is? | 20:17 |
sridhar_ram | sripriya: 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_ram | I'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 | |
bobh | sripriya: sorry about that... | 20:22 |
* sridhar_ram increases the chocolates count for sripriya in next week's social | 20:22 | |
sripriya | bobh: finally! your code stays intact :) | 20:22 |
*** vishwanathj has joined #tacker | 20:22 | |
sripriya | sridhar_ram: :-) your treats are already overdue | 20:23 |
sridhar_ram | bobh: on a related topic, we need to have a cross-project design summit session w/ Sahdev / heat-translator team | 20:23 |
sridhar_ram | bobh: have you discussed this w/ spzala ? | 20:24 |
bobh | sridhar_ram: not directly but I'm sure he'd be happy to | 20:24 |
sridhar_ram | bobh: okay... pinging in heat-translator channel | 20:25 |
*** lhcheng has joined #tacker | 20:31 | |
vishwanathj | bobh 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.html | 20:31 |
*** sridhar_ram has quit IRC | 20:31 | |
bobh | vishwanathj: I'll take a look | 20:32 |
*** vishnoianil has quit IRC | 20:36 | |
bobh | vishwanathj: wondering if the changes to the 'name' code in heat.py are causing the functional test failures? | 20:39 |
bobh | vishwanathj: see the note inline | 20:40 |
*** zeih has joined #tacker | 20:51 | |
vishwanathj | bobh, I have had name change since the beginning of the patch set | 20:51 |
bobh | vishwanathj: ok - maybe recheck? Or can you run the func tests on a local testbed and see if you can reproduce the problem? | 20:52 |
vishwanathj | i did a recheck as well | 20:52 |
bobh | vishwanathj: not much helpful info in the output | 20:53 |
sripriya | vishwanathj: are the func testcases failing in every run? | 20:53 |
*** sridhar_ram has joined #tacker | 20:54 | |
vishwanathj | nope, patchset 7 was the first time it failed.... | 20:54 |
vishwanathj | I added a new test called test_create_tosca_vnic_normal and other related code changes in response to sridhar's comments | 20:55 |
bobh | vishwanathj: none of the changes in 7 should affect functional tests | 20:55 |
bobh | vishwanathj: did you have to rebase between 6 and 7 for changes in master? | 20:56 |
vishwanathj | bobh hence, I was surprised and clueless | 20:56 |
sripriya | vishwanathj: looks like a timing issue on jenkins where the devstack exceeded the quota on no. of instanced spawned and running | 20:56 |
sripriya | vishwanathj: instances* | 20:56 |
vishwanathj | bobh 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 comments | 20:57 |
openstackgerrit | vishwanath jayaraman proposed openstack/tacker: Implements SR-IOV support https://review.openstack.org/300903 | 21:00 |
*** zeih_ has joined #tacker | 21:36 | |
*** zeih has quit IRC | 21:40 | |
*** zeih_ has quit IRC | 21:40 | |
*** mageshgv has quit IRC | 21:43 | |
*** mageshgv has joined #tacker | 21:57 | |
openstackgerrit | Sridhar Ramaswamy proposed openstack/tacker: Ignore: Test to test tackerclient deprecation warning fix https://review.openstack.org/307505 | 22:07 |
*** uck has joined #tacker | 22:10 | |
*** uck_ has quit IRC | 22:10 | |
sridhar_ram | vishwanathj: sripriya: bobh: heads up - huge delay in gate CI results.. for gerrit #307505 ETA is 7 hours :-( | 22:10 |
sridhar_ram | http://status.openstack.org/zuul/ | 22:10 |
sripriya | sridhar_ram: yup there are 601 jobs | 22:11 |
sripriya | sridhar_ram: super monday :-) | 22:11 |
*** mbound has quit IRC | 22:12 | |
sridhar_ram | sripriya: yep.. ! mad craze before the summit | 22:13 |
*** mbound has joined #tacker | 22:15 | |
vishwanathj | sridhar_ram, I noticed that, thanks again for the heads up | 22:19 |
*** Berezin has joined #tacker | 22:20 | |
sridhar_ram | vishwanathj: sure ! | 22:27 |
sridhar_ram | bobh: spzala confirmed Thursday 3:10 slot.. for tosca / heat-translator .. | 22:28 |
bobh | sridhar_ram: sounds good | 22:28 |
sridhar_ram | bobh: I'm putting you to lead the discussion... one request from you... | 22:28 |
sridhar_ram | bobh: .. can you create an etherpad with all things around tacker-tosca .. ? we will use that to conduct / colloborate during the design session | 22:29 |
bobh | sridhar_ram: sure, np | 22:29 |
sridhar_ram | bobh: thanks! | 22:30 |
sridhar_ram | trozet: ping | 22:32 |
s3wong | sridhar_ram: from Mac's Xzure IRC client, trozet is transparent, meaning he has been inactive for a while | 22:33 |
*** sridhar_ram_ has quit IRC | 22:44 | |
*** uck_ has joined #tacker | 22:48 | |
*** uck has quit IRC | 22:48 | |
*** bobh has quit IRC | 22:57 | |
sridhar_ram | vishwanathj: ping | 23:05 |
*** mageshgv has quit IRC | 23:07 | |
*** bobh has joined #tacker | 23:13 | |
*** bobh has quit IRC | 23:14 | |
*** uck_ has quit IRC | 23:16 | |
*** bobh has joined #tacker | 23:16 | |
*** mbound has quit IRC | 23:20 | |
*** dkushwaha has joined #tacker | 23:37 | |
vishwanathj | sridhar_ram what's up? | 23:39 |
*** s3wong has quit IRC | 23:47 | |
sridhar_ram | vishwanathj: I reviewing https://review.openstack.org/#/c/300903 .. had a question.. | 23:56 |
vishwanathj | what's the question? | 23:57 |
sridhar_ram | vishwanathj: .. I think I got the answer, which is a CP can have both "management: true" and "type: sriov" .. | 23:57 |
sridhar_ram | vishwanathj: .. is this assumption, correct ? | 23:57 |
vishwanathj | yes | 23:57 |
sridhar_ram | cool.. thanks! | 23:57 |
vishwanathj | I am adding a test case per your comment | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!