Tuesday, 2016-09-13

*** prashantD_ has joined #tacker00:03
*** n-harada has joined #tacker00:04
*** prashantD has quit IRC00:07
*** mbound has quit IRC00:16
*** n-harada has quit IRC00:22
*** sripriya has joined #tacker00:24
sridhar_rams3wong: ping00:27
*** dkushwaha has joined #tacker00:28
*** uck has quit IRC00:30
*** vishnoianil has quit IRC00:31
sridhar_rams3wong: are you planning to attend tomorrow's weekly meeting? it will be good to run the state of FFG across everyone00:33
*** n-harada has joined #tacker00:35
openstackgerritDigambar proposed openstack/tacker-specs: Implement API frameowrk for taker  https://review.openstack.org/36851100:39
*** yifei has joined #tacker00:42
*** vishwana_ has quit IRC00:48
*** n-harada has quit IRC00:51
s3wongsridhar_ram: hello, sorry, was in a meeting00:55
s3wongsridhar_ram: I should be able to attend tomorrow morning00:55
*** dkushwaha has quit IRC00:57
*** prashantD_ has quit IRC00:58
*** dkushwaha has joined #tacker00:58
dkushwahasripriya, Could you please give a +2 on https://review.openstack.org/#/c/346557/ , if it is ok :)01:01
*** n-harada has joined #tacker01:03
*** gongysh has joined #tacker01:06
gongyshsridhar_ram, hi01:08
gongyshwhich gerrit review is that error coming from?01:08
sripriyadkushwaha: not sure why you are fixing it ?01:12
sripriyadkushwaha: i think it is good to handle body=None in our client calls01:13
*** mbound has joined #tacker01:16
dkushwahasripriya, yea right, but the definitions with body=None looks no meaning here, it should be required.01:20
*** mbound has quit IRC01:22
sripriyadkushwaha: having body as an optional arg with default set to None will allow the client to take appropriate calls when user send these args01:22
sripriyadkushwaha: moreover the actual post calls still do not mandate body and set it to None when not passed01:22
sripriyadkushwaha: i dont think we should not do this change01:23
sripriyadkushwaha: argh, i think *01:23
dkushwahasripriya, make sense.01:25
dkushwahasripriya, Thanks for clarification :)01:26
sripriyadkushwaha: no prob.01:26
openstackgerritMerged openstack/tacker: Move functional tests from commonservices directory  https://review.openstack.org/36769701:32
*** yifei1 has joined #tacker01:33
*** yifei has quit IRC01:34
*** yifei1 is now known as yifei01:34
openstackgerritSripriya Seetharam proposed openstack/tacker: Derive infra driver using VIM type  https://review.openstack.org/36759901:35
*** s3wong has quit IRC01:37
*** n-harada has quit IRC01:44
gongyshsripriya, hi01:47
sripriyagongysh: hello01:47
gongyshsripriya, is  newton stable of the tacker repo created?01:48
sripriyagongysh: not yet, the release is on 15th01:48
gongyshsripriya, if not,  we cannot remove the deprecated yet.01:49
sripriyagongysh: yes we can remove deprecations after the release is cut01:49
gongyshhttps://review.openstack.org/#/c/367599/4 should go into ocata version.01:50
*** bobh has quit IRC01:51
gongyshcan you mark it as WIP so that it should not be merged into newton version? or we can keep the deprecation statements and remove them in an independent patch.01:51
openstackgerritdharmendra kushwaha proposed openstack/tacker: Allow vdu (VM) names to be specified as a parameter  https://review.openstack.org/36354901:53
sripriyagongysh: this is not removing the deprecation , see https://review.openstack.org/#/c/367599/4/tacker/vm/plugin.py@15401:53
sripriyagongysh: the deprecation message is moved from db to plugin file01:54
*** n-harada has joined #tacker01:56
openstackgerritSripriya Seetharam proposed openstack/tacker: Rename heat unit tests to openstack  https://review.openstack.org/36762301:59
*** diga has joined #tacker02:04
digasridhar_ram: Hi02:04
digasridhar_ram: about VNFFG, as trozet is on leave, whom should I contact. I am starting e2e testing today02:05
sridhar_ramdiga: contact s3wong02:11
digasridhar_ram: okay02:11
digasridhar_ram: did you get a chance to look at https://review.openstack.org/#/c/368511/02:12
openstackgerritdharmendra kushwaha proposed openstack/tacker: Upgrade keystone version from v2.0 t v3 in functional test cases  https://review.openstack.org/36861402:25
*** diga has quit IRC02:33
sridhar_ramdiga: I briefly looked at it, need to spend more time before commenting02:49
*** vishwanathj has joined #tacker02:58
*** n-harada has quit IRC03:13
openstackgerritdharmendra kushwaha proposed openstack/tacker: Allow vdu (VM) names to be specified as a parameter  https://review.openstack.org/36354903:16
*** prashantD_ has joined #tacker03:20
*** n-harada has joined #tacker03:25
openstackgerritvenkatamahesh proposed openstack/tacker: [DOC] Update the manual installation guide to rectify errors  https://review.openstack.org/35325803:40
*** sripriya has quit IRC03:42
openstackgerritvenkatamahesh proposed openstack/tacker: Clean up the imports according to hacking standards  https://review.openstack.org/35960803:46
*** haiwei has joined #tacker03:55
openstackgerritdharmendra kushwaha proposed openstack/tacker: Allow vdu (VM) names to be specified as a parameter  https://review.openstack.org/36354903:56
openstackgerritvenkatamahesh proposed openstack/tacker: [py35] Fix for jenkins-gate-py35 error  https://review.openstack.org/36665603:59
openstackgerritvenkatamahesh proposed openstack/tacker: Fix the Attribute keyname in nova.py  https://review.openstack.org/36092804:05
openstackgerritSridhar Ramaswamy proposed openstack/tacker: [WIP] Provide support for security groups for ConnectionPoints  https://review.openstack.org/33425604:10
openstackgerritSridhar Ramaswamy proposed openstack/tacker: Provide support for security groups for ConnectionPoints  https://review.openstack.org/33425604:11
*** n-harada has quit IRC04:14
*** KanagarajM has joined #tacker04:16
openstackgerritvishwanath jayaraman proposed openstack/tacker: Ensures common services plugin is always loaded  https://review.openstack.org/36914204:22
vishwanathjgongysh hi ... would appreciate your review of ^^^^^ patchset and ensure the approach is right04:24
*** vishwanathj is now known as vishwanathj_zzz04:26
*** n-harada has joined #tacker04:26
*** KanagarajM has quit IRC04:33
sridhar_ramgongysh: can you review https://review.openstack.org/#/c/363352/ ?04:33
*** KanagarajM has joined #tacker04:34
gongyshsridhar_ram, if we remove the anti_poof to enable the port-security, will it impact the ping or http ping monitor?04:41
sridhar_ramgongysh: hmm, it shouldn't .. the default secgrp does allow all pkts to reach the VM04:42
*** n-harada has quit IRC04:42
gongyshsridhar_ram, so it will impact. we should allow ping or http ping packet into VDU VM.04:42
*** hparekh has joined #tacker04:43
tung_doansridhar_ram: ping04:49
sridhar_ramtung_doan: pong04:50
tung_doansridhar_ram: thanks.. regarding to merging alarm_abstract driver with original "abstract_driver".. if it has any impact?04:51
tung_doansridhar_ram: I strongly agree, we have two type: Pollling and Alarm.. but with alarm, we will have mandatory functions04:52
sridhar_ramtung_doan: you need to merge alarm_abstract_driver with existing .. https://github.com/openstack/tacker/blob/master/tacker/vnfm/monitor_drivers/abstract_driver.py04:53
tung_doansridhar_ram: ceilometer, monasca will have some mandatory functions like create_alarm(), but doesn;t make sense aith ping, httpping04:53
sridhar_ramtung_doan: for now, given we are cutting too close, i'd suggest you to add stub methods in ping and http_ping driver for the methods in the alaram_abstract_driver04:54
*** n-harada has joined #tacker04:54
sridhar_ramtung_doan: yes, we can clean this up later.. for now just stub them out in the ping and http-ping monitor drivers04:54
tung_doansridhar_ram: thanks..04:55
gongyshsridhar_ram, after these features merged, I want to merge https://review.openstack.org/#/c/363351/ rename vm/plugin.py into nfvm/plugin.py so that we will not bring 'device' into ocata version.04:56
sridhar_ramgongysh: absolutely.. i was waiting for newton feature patchsets to merge first04:59
sridhar_ramgongysh: btw, checking on the default icmp rule ...04:59
sridhar_ramgongysh: looks the default sec group has ipv4 and ipv6 all for the ingress .. but not icmp05:00
gongyshsridhar_ram, me too. so I am not in a hurry to upload new patchset.05:00
*** prashantD_ has quit IRC05:00
sridhar_ramgongysh: btw, infra driver from vim is an important fix.. need to get this into newton05:00
sridhar_ramgongysh: https://review.openstack.org/#/c/367599/05:01
gongyshsridhar_ram, no, the default rules in security group is for exgres direction. for ingress direction, there is no rules for outsider to access VMs.05:01
openstackgerritvenkatamahesh proposed openstack/tacker: [py35] Fix for jenkins-gate-py35 error  https://review.openstack.org/36665605:02
gongyshsridhar_ram, en, I will reexamine the VIM infra_driver patch after new patchset.05:02
gongyshvishwanathj_zzz, hi05:03
gongyshvishwanathj_zzz, wake up.05:04
sridhar_ram:)05:04
sridhar_ramgongysh: what is this remote_group_id in the neutron sec group ?05:05
gongyshremote_group_id means all the ports that have the specified security group.05:06
openstackgerritvenkatamahesh proposed openstack/tacker: [py35] Fix for jenkins-gate-py35 error  https://review.openstack.org/36665605:06
gongyshsridhar_ram, If port a,b are using SG A, and in SG B, if we have a rule to say access icmp ping from remote_sg_id SG A,  that means port a, b can ping.05:07
openstackgerritKanagaraj Manickam proposed openstack/tacker: Devref for vnf scaling feature  https://review.openstack.org/35920605:09
openstackgerritKanagaraj Manickam proposed openstack/tacker: Devref for vnf scaling feature  https://review.openstack.org/35920605:10
*** KanagarajM_ has joined #tacker05:14
*** KanagarajM has quit IRC05:14
tung_doansridhar_ram: gonggysh: alarm monitor worked well now... waiting for review so that we can wrap it up soon.. thanks05:15
*** hparekh_ has joined #tacker05:16
*** hparekh_ has quit IRC05:16
*** hparekh has quit IRC05:27
*** KanagarajM_ has quit IRC05:36
*** n-harada has quit IRC05:39
openstackgerritKanagaraj Manickam proposed openstack/tacker: VNF scaling: Functional test  https://review.openstack.org/36357305:39
*** sripriya has joined #tacker05:42
*** n-harada has joined #tacker05:51
*** janki has joined #tacker05:51
sridhar_ramgongysh: thanks, i'll remove anti_spoofing true for monitor sample05:54
*** KanagarajM has joined #tacker05:59
openstackgerritgengchc2 proposed openstack/tacker: Translate log messages  https://review.openstack.org/36041006:04
tung_doangongysh: ping06:05
openstackgerritKanagaraj Manickam proposed openstack/tacker: Devref for vnf scaling feature  https://review.openstack.org/35920606:05
gongyshtung_doan, pong06:07
*** n-harada has quit IRC06:07
gongyshtung_doan, I have some comments on https://review.openstack.org/#/c/365435/21/tacker/alarm_receiver.py06:08
tung_doangongysh: frankly, we didnot have a ideal solution to get token in alarm_receiver.. Meanwhile, hardcode user/passwork to get token seems not good06:08
tung_doanregarding to your second comment... do you have any suggestion?06:09
tung_doangongysh: thanks for your review...really appreciate it06:09
gongyshtung_doan, why not use config items in tacker.conf,  I remember we have done this way in a certain patchset, but you discarded them in new patchset.06:10
*** KanagarajM has quit IRC06:12
tung_doangongysh: already did like this (I knew that this solution was already used in tacker)... but we will must set default password in devstack06:13
openstackgerritSridhar Ramaswamy proposed openstack/tacker: Introduce TOSCA 'order' property for connection points  https://review.openstack.org/36335206:13
tung_doangongysh: if we could use this solution.. happy to get your inputs06:14
gongyshtung_doan, use the config items is the right way.06:14
tung_doangongysh: got it... thanks :)06:15
gongyshtung_doan, in devstack, you can use the way we have used to generate keystone configuration in devstack.06:16
*** n-harada has joined #tacker06:19
tung_doangongysh: my concern is that if someone change the default password in devstack.. alarm_receiver will not get token...06:20
*** sripriya has quit IRC06:21
gongyshtung_doan, what do you mean?06:21
tung_doangongysh: therefore, I already wrote note in alarm monitoring usage guide06:21
*** diga has joined #tacker06:21
gongyshtung_doan, if someone changed the password in keystone, he/she must change it in tacker.conf accordingly06:22
tung_doangongysh: got it.. but what you mean "the way we have used to generate keystone configuration in devstack"?06:23
*** KanagarajM has joined #tacker06:25
gongyshtung_doan, configure_auth_token_middleware $conf_file $TACKER_ADMIN_USERNAME $TACKER_AUTH_CACHE_DIR $section06:29
gongyshtung_doan, https://github.com/openstack/tacker/blob/master/devstack/lib/tacker#L32906:29
*** n-harada has quit IRC06:33
tung_doangongysh: will try.. thanks06:34
*** hparekh has joined #tacker06:35
gongyshtung_doan, maybe configure_auth_token_middleware $conf_file ALARM_USER,  $TACKER_AUTH_CACHE_DIR $alarm_section06:35
tung_doanthen what we can do with it? could you elaborate?06:36
openstackgerritvenkatamahesh proposed openstack/tacker: [py35] Fix for jenkins-gate-py35 error  https://review.openstack.org/36665606:38
dkushwahatung_doan, could you please test on your system, don't know i am failing again06:40
tung_doandkushwaha: will test now..06:44
*** KanagarajM has quit IRC06:44
*** n-harada has joined #tacker06:45
tung_doandkushwaha: also, please refresh your system.. code already worked well in my sys.. but will check again06:45
*** hparekh_ has joined #tacker06:47
*** hparekh has quit IRC06:50
*** hparekh__ has joined #tacker06:56
openstackgerritgengchc2 proposed openstack/tacker: Translate log messages  https://review.openstack.org/36041006:57
*** hparekh_ has quit IRC06:59
*** hparekh_ has joined #tacker07:01
*** n-harada has quit IRC07:03
*** hparekh__ has quit IRC07:05
*** n-harada has joined #tacker07:15
*** hparekh__ has joined #tacker07:19
*** hparekh_ has quit IRC07:23
openstackgerritdharmendra kushwaha proposed openstack/tacker: Alarm monitor: All in one  https://review.openstack.org/36543507:28
*** n-harada has quit IRC07:29
openstackgerritvenkatamahesh proposed openstack/tacker: [py35] Fix for jenkins-gate-py35 error  https://review.openstack.org/36665607:31
*** n-harada has joined #tacker07:41
*** Ravikiran_K has joined #tacker07:46
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #tacker07:49
openstackgerritSantosh Kodicherla proposed openstack/tacker: Functional test for static ip support and anti_spoof_protection fields in tosca template  https://review.openstack.org/32410107:53
*** n-harada has quit IRC07:57
*** dkushwaha has quit IRC07:59
*** mbound has joined #tacker08:05
*** n-harada has joined #tacker08:09
*** dkushwaha has joined #tacker08:14
*** n-harada has quit IRC08:23
*** hparekh_ has joined #tacker08:28
*** amotoki has joined #tacker08:32
*** hparekh__ has quit IRC08:32
*** hparekh__ has joined #tacker08:35
*** n-harada has joined #tacker08:36
*** hparekh_ has quit IRC08:39
*** amotoki has quit IRC08:44
*** n-harada has quit IRC08:52
openstackgerritvenkatamahesh proposed openstack/tacker: [py35] Fix for jenkins-gate-py35 error  https://review.openstack.org/36665608:53
*** hparekh_ has joined #tacker08:54
openstackgerritdharmendra kushwaha proposed openstack/tacker: Alarm monitor: All in one  https://review.openstack.org/36543508:55
*** hparekh__ has quit IRC08:57
*** Vijayendra has joined #tacker09:00
*** hparekh_ has quit IRC09:03
*** n-harada has joined #tacker09:04
*** amotoki has joined #tacker09:05
*** n-harada has quit IRC09:19
*** gongysh has quit IRC09:22
*** n-harada has joined #tacker09:31
*** amotoki has quit IRC09:37
*** amotoki has joined #tacker09:46
*** mfedosin has joined #tacker09:47
*** n-harada has quit IRC09:48
*** amotoki has quit IRC09:56
*** vishwana_ has joined #tacker09:56
*** vishwanathj_zzz has quit IRC09:58
*** n-harada has joined #tacker10:00
*** amotoki has joined #tacker10:07
*** n-harada has quit IRC10:15
*** n-harada has joined #tacker10:27
*** janki has quit IRC10:30
*** n-harada has quit IRC11:01
*** diga has quit IRC11:13
*** n-harada has joined #tacker11:13
*** n-harada has quit IRC11:44
*** amotoki has quit IRC11:49
*** bobh has joined #tacker11:54
*** n-harada has joined #tacker11:56
*** tung_doan is now known as tung_doan_afk12:13
*** dkushwaha has quit IRC12:24
*** bobh has quit IRC12:25
*** n-harada has quit IRC12:27
*** n-harada has joined #tacker12:38
*** n-harada has quit IRC12:53
*** n-harada has joined #tacker13:05
bryan_atthi, can anyone help debug a Heat (mitaka/stable) error caused by Tacker (mitaka/stable) creating a CP1.properties 'port_security_enabled' flag (not present in blueprint), resulting "Resource CREATE failed: BadRequest: resources.CP1: Unrecognized attribute(s) 'port_security_enabled' Neutron server " ?13:17
*** n-harada has quit IRC13:20
*** amotoki has joined #tacker13:26
*** amotoki has quit IRC13:28
*** n-harada has joined #tacker13:32
*** Vijayendra has quit IRC13:49
*** amotoki has joined #tacker13:59
*** dkushwaha has joined #tacker14:26
openstackgerritdharmendra kushwaha proposed openstack/tacker: Implement Alarm monitor  https://review.openstack.org/36543514:30
openstackgerritMerged openstack/tacker: Introduce TOSCA 'order' property for connection points  https://review.openstack.org/36335214:40
*** amotoki has quit IRC14:44
*** amotoki has joined #tacker14:44
*** Ravikiran_K has quit IRC14:49
*** amotoki has quit IRC14:52
*** n-harada has quit IRC14:54
*** diga has joined #tacker15:05
*** n-harada has joined #tacker15:06
*** bobh has joined #tacker15:06
*** tbh has joined #tacker15:07
*** bobh has quit IRC15:12
*** bobh has joined #tacker15:12
dkushwahasridhar_ram, tbh, please review the patch https://review.openstack.org/#/c/363549/15:18
tbhdkushwaha, sure, doing it now15:19
dkushwahatbh, thanks :)15:20
*** LamT_ has joined #tacker15:20
sridhar_ramdkushwaha: done15:31
*** n-harada has quit IRC15:39
dkushwahasridhar_ram, thanks :)15:45
*** sripriya has joined #tacker15:46
tbhsridhar_ram, qq15:50
sridhar_ramtbh: shoot15:51
*** n-harada has joined #tacker15:52
tbhsridhar_ram, regarding the https://review.openstack.org/#/c/363549/, though it is straighforward, I have one doubt. As we already specifiying VDU name like VDU1 etc., in the template then why we need to explicitly mention "name" property. It looks more specific to infra_driver15:53
*** KanagarajM has joined #tacker15:53
sridhar_ramtbh: this is more a "friendly name" that is useful for cloud admins to identify the VMs spawned by Tacker..15:53
sridhar_ramtbh: without that the VM is named by infra_driver (by Heat driver) is some cryptic way.. that is still going to be the default behavior..15:54
tbhsridhar_ram, ok15:54
*** n-harada has quit IRC15:55
*** vishwana_ is now known as vishwanathj15:58
openstackgerritMerged openstack/tacker: Allow vdu (VM) names to be specified as a parameter  https://review.openstack.org/36354916:02
openstackgerritKanagaraj Manickam proposed openstack/tacker: Devref for vnf scaling feature  https://review.openstack.org/35920616:05
*** s3wong has joined #tacker16:10
*** KanagarajM has quit IRC16:11
*** mfedosin has quit IRC16:22
*** uck has joined #tacker16:23
*** vishnoianil has joined #tacker16:26
*** mfedosin has joined #tacker16:35
*** vishwanathj_ has joined #tacker16:44
*** Ravikiran_K has joined #tacker16:44
*** s3wong has quit IRC17:00
*** diga has quit IRC17:04
*** sripriya has quit IRC17:04
openstackgerritMerged openstack/tacker: VNF scaling: Functional test  https://review.openstack.org/36357317:07
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being restarted now to address current performance problems, but should return to a working state within a few minutes17:12
*** tbh has quit IRC17:14
*** mfedosin has quit IRC17:22
*** uck_ has joined #tacker17:31
*** uck has quit IRC17:32
*** uck has joined #tacker17:42
*** uck_ has quit IRC17:42
*** mbound has quit IRC17:42
*** bobh has quit IRC17:43
*** amotoki has joined #tacker17:53
*** s3wong has joined #tacker17:56
*** amotoki has quit IRC17:58
*** sripriya has joined #tacker18:06
sripriyabryan_att: ping18:07
bryan_attsripriya: hi18:08
sripriyabryan_att: reg: heat error, i’m assuming you enabled port_security extension as mentioned in prerequisites section of manual installation guide: http://docs.openstack.org/developer/tacker/install/manual_installation.html#pre-requisites18:10
sripriyabryan_att: can you confirm?18:10
sripriyabryan_att: refer step 4)18:11
bryan_attNot specifically - where is that file (folder) ?18:11
sripriyabryan_att: it will be under /etc/neutron/plugins/ml2/ml2_conf.ini18:12
bryan_attlet me check. this is something that the installers would do (JOID, Apex) in OPNFV.18:12
sripriyabryan_att: this extension is required for tacker, it can be made part of installer configuration for neutron18:14
bryan_attI'm checking with the installer teams. One other question though is whether there is a Tacker (stable/mitaka) compatibility issue between Heat  6.0.1 to 6.0.0 ? I'm seeing the Heat issue in 6.0.0.18:16
bryan_attsripriya: is this the right settings? https://www.irccloud.com/pastebin/5d3cfCKR/18:18
bryan_attsripriya: That's what's set for Apex, which works fine.18:19
bryan_attsripriya: Looks like it's not enabled by default in JOID. I will do that manually and retest.18:22
sripriyabryan_att: you do not see the issue on 6.0.1?18:23
sripriyabryan_att: also, are you using the tosca templates or the old deprecated templates?18:23
bryan_attI haven't tested the behavior with the flag off in 6.0.1 (Apex). I'll try to see if setting the flag fixes this in JOID (6.0.0). I'm using TOSCA templates - see https://git.opnfv.org/cgit/models/tree/tests (vHello_Tacker.sh) and the blueprint at https://git.opnfv.org/cgit/models/tree/tests/blueprints/tosca-vnfd-hello-world-tacker18:26
sripriyabryan_att: got it18:30
*** bobh has joined #tacker18:43
*** uck_ has joined #tacker18:47
*** uck has quit IRC18:47
*** bobh has quit IRC18:48
*** bobh has joined #tacker18:54
sridhar_rambobh: sripriya: s3wong: pls review https://review.openstack.org/#/c/334256/19:04
sripriyasridhar_ram: ack19:06
*** vishwanathj_ has quit IRC19:06
sridhar_ramsripriya: thanks19:06
*** uck has joined #tacker19:15
*** uck_ has quit IRC19:15
*** mike_m has quit IRC19:16
*** xiayu has quit IRC19:16
*** priya__ has quit IRC19:16
*** vishnoianil has quit IRC19:17
*** dkushwaha has quit IRC19:17
*** achatterjee has quit IRC19:17
*** bryan_att has quit IRC19:17
*** Ravikiran_K has quit IRC19:17
*** haiwei has quit IRC19:17
*** tung_doan_afk has quit IRC19:17
*** bobh has quit IRC19:17
*** LamT_ has quit IRC19:17
*** mbirru has quit IRC19:17
*** Qiming has quit IRC19:17
*** amit213 has quit IRC19:17
*** sridhar_ram has quit IRC19:17
*** s3wong has quit IRC19:17
*** sripriya has quit IRC19:18
*** yifei has quit IRC19:18
*** igordcard has quit IRC19:18
*** afranc has quit IRC19:18
*** veena__ has quit IRC19:18
*** trozet has quit IRC19:18
*** ossifrath has quit IRC19:18
*** uck has quit IRC19:18
*** openstackgerrit has quit IRC19:18
*** xiayu has joined #tacker19:23
*** mike_m has joined #tacker19:23
*** trozet has joined #tacker19:23
*** yifei has joined #tacker19:23
*** sridhar_ram has joined #tacker19:23
*** priya__ has joined #tacker19:23
*** s3wong has joined #tacker19:23
*** veena__ has joined #tacker19:23
*** vishnoianil has joined #tacker19:23
*** dkushwaha has joined #tacker19:23
*** achatterjee has joined #tacker19:23
*** igordcard has joined #tacker19:23
*** uck has joined #tacker19:23
*** openstackgerrit has joined #tacker19:23
*** afranc has joined #tacker19:23
*** bobh has joined #tacker19:23
*** mbirru has joined #tacker19:24
*** Qiming has joined #tacker19:24
*** sripriya has joined #tacker19:24
*** ossifrath has joined #tacker19:41
*** Ravikiran_K has joined #tacker19:49
*** bryan_att has joined #tacker19:54
*** amit213 has joined #tacker19:59
*** uck_ has joined #tacker19:59
*** LamT_ has joined #tacker20:00
*** uck has quit IRC20:01
*** Ravikiran_K has quit IRC21:03
openstackgerritSripriya Seetharam proposed openstack/tacker: Derive infra driver using VIM type  https://review.openstack.org/36759921:10
*** amit213 has quit IRC21:27
*** mbirru has quit IRC21:27
*** Qiming has quit IRC21:27
*** mbirru has joined #tacker21:28
*** Qiming has joined #tacker21:28
sripriyasridhar_ram: ping21:29
*** amit213 has joined #tacker21:34
*** santoshk has joined #tacker21:41
sridhar_ramsripriya: hi21:57
*** bobh has quit IRC21:58
sripriyasridhar_ram: https://review.openstack.org/#/c/367599/ git mv does not apply for heat file since we are retaining that file even after renaming the old heat file to openstack21:58
sridhar_ramsripriya: hmm.. i understand..21:59
sridhar_ramsripriya: wondering if there are any "workarounds"21:59
sripriyasridhar_ram: i tried with git add/rm , that does not work as well22:00
sripriyasridhar_ram: git is smart to identify git mv as just basically modifying the existing file and not renaming/adding new file22:02
sridhar_ramsripriya: one crazy idea ..22:02
sripriyasridhar_ram: split them in to separate patches?22:02
sridhar_ramsripriya: git mv heat.py -> openstack.py and use a different name for heat driver .. say "vim_heat.py" so that there is no clash22:03
sridhar_ramsripriya: i thought abt two separate patch as well.. thats another option.. in addition to ^^^22:03
sripriyasridhar_ram: that will change the entrypoint in setup.cfg22:03
sridhar_ramsripriya: i guess .. we need the 2nd patch to move it back to heat.py22:04
sridhar_rami know it is an hassle... but, IMO, leaving the git history is worth for the project for this file22:04
sridhar_ramthis file pretty much defined the project for a long time :)22:05
sridhar_ram*preserving22:05
sripriyasridhar_ram: are you saying keep other references of heat untouched but just rename the file to vim_heat?22:06
sridhar_ramsripriya: if setup.cfg entry point changes for heat driver, does it have any impact on tacker.conf  ?22:06
sridhar_ramsripriya: if no impact, above vim_heat.py workaround is sufficient22:07
sripriyasridhar_ram: no impact on tacker.conf22:07
sridhar_ramsripriya: okay, that shd be workable ?22:08
sripriyasridhar_ram: the patch with just the renaming of heat file to vim_heta may look a bit weird22:09
sripriyasridhar_ram: thats the only thing :)22:10
sripriyavim_heat*22:10
sridhar_ramwith this plan, we don't need the 2nd rename patchset..22:11
sridhar_ramjust in your patchset, have the entry point like "heat = tacker.vnfm.infra_drivers.heat.vim_heat:DeviceHeat"22:11
sripriyasridhar_ram: the second patch is handling the testcases for openstack22:13
sripriyasridhar_ram: why dont we need the 2nd patch?22:14
sridhar_ramsripriya: by 2nd patch, are you referring to https://review.openstack.org/#/c/367623/ ?22:15
sripriyasridhar_ram: yes22:15
sridhar_ramsripriya: oh, i wasn't talking abt that.. i was referring to the 2nd patch to move back the vim_heat.py file name back to heat.py22:16
sripriyasridhar_ram: ah got it, yes we dont need it, but i thought i would rename it back in https://review.openstack.org/#/c/367623/ otherwise testcases need to modified to reflect this new path22:17
sridhar_ramsripriya: sure, that will tidy things up22:17
sripriyasridhar_ram: also, are you expecting test_heat history to be preserved?22:18
sridhar_ramsripriya: IMO, that is not important22:21
sripriyasridhar_ram: ok22:21
*** uck_ has quit IRC22:43
*** uck has joined #tacker22:54
openstackgerritSantosh Kodicherla proposed openstack/tacker: Functional test for static ip support and anti_spoof_protection fields in tosca template  https://review.openstack.org/32410123:20
openstackgerritSripriya Seetharam proposed openstack/tacker: [WIP] Derive infra driver using VIM type  https://review.openstack.org/36759923:30
*** gongysh has joined #tacker23:31
*** santoshk has quit IRC23:37
*** uck_ has joined #tacker23:38
*** uck has quit IRC23:39
*** uck_ has quit IRC23:58

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