| *** amotoki has joined #tacker | 00:00 | |
| *** amotoki has quit IRC | 00:06 | |
| *** amotoki has joined #tacker | 00:12 | |
| *** sripriya has joined #tacker | 00:16 | |
| *** sripriya has quit IRC | 00:28 | |
| *** amotoki has quit IRC | 00:30 | |
| *** sripriya has joined #tacker | 00:35 | |
| *** santoshk has quit IRC | 00:35 | |
| dkushwaha_ | sridhar_ram, hi | 00:39 |
|---|---|---|
| dkushwaha_ | sridhar_ram, Regarding the mistral workflow, I have tried to execute it for Tacker. | 00:40 |
| dkushwaha_ | I have tried it with vnfd create and list operation, and it looks working fine. | 00:41 |
| dkushwaha_ | sridhar_ram, I am planning to create a tacker workflow with complete lifecycle of vnf. And checkin the same into mistral-extra https://github.com/openstack/mistral-extra | 00:43 |
| dkushwaha_ | sridhar_ram, I will be able to do that after mid-cycle(once nsd spec is finalized) . Hope that is fine :) | 00:45 |
| dkushwaha_ | sridhar_ram, And yes we can add support for nsd in mistral workflow during nsd blueprint | 00:47 |
| *** sripriya has quit IRC | 00:48 | |
| *** sripriya has joined #tacker | 00:51 | |
| haiwei | sripriya: hi | 00:51 |
| *** sripriya has quit IRC | 00:52 | |
| *** sripriya has joined #tacker | 00:53 | |
| *** prashantD has quit IRC | 00:57 | |
| sripriya | haiwei: hello | 01:00 |
| *** amotoki has joined #tacker | 01:10 | |
| *** bobh has joined #tacker | 01:28 | |
| haiwei | sripriya, you pinged me yesterday | 01:28 |
| haiwei | sripriya, do you have any issues? :) | 01:29 |
| sripriya | haiwei: wanted to followup with you on https://review.openstack.org/#/c/323192/ as it looks closes to merging and if you are planning to address kanagarajm’s comment/receiving his review since he had few comments on the patch | 01:31 |
| *** gongysh has joined #tacker | 01:32 | |
| haiwei | sripriya, ok, I will work on it soon | 01:32 |
| sripriya | haiwei: thanks | 01:34 |
| *** KanagarajM has joined #tacker | 01:35 | |
| *** xuan_ has joined #tacker | 01:35 | |
| KanagarajM | sripriya, ping | 01:37 |
| sripriya | KanagarajM: pong | 01:37 |
| KanagarajM | sripriya, regarding https://review.openstack.org/329528 I just replied to your comment, can you please have a look | 01:38 |
| *** bobh has quit IRC | 01:40 | |
| sripriya | KanagarajM: willl take a look | 01:40 |
| KanagarajM | sripriya, sure. also https://review.openstack.org/#/c/330666/ | 01:40 |
| KanagarajM | sridhar_ram, ping | 01:41 |
| sridhar_ram | KanagarajM: pong | 01:41 |
| KanagarajM | sridhar_ram, kindly have a look on ^^ | 01:43 |
| sridhar_ram | KanagarajM: replying to the patchset now | 01:43 |
| KanagarajM | sridhar_ram, sure. thanks | 01:44 |
| *** s3wong has quit IRC | 01:46 | |
| sridhar_ram | KanagarajM: what is your availability during the midcycle meetup ? | 01:46 |
| *** gongysh has quit IRC | 01:46 | |
| *** gongysh has joined #tacker | 01:47 | |
| KanagarajM | sridhar_ram, I will try to make me available during the midcycle, hopefully | 01:49 |
| sridhar_ram | dkushwaha_: great, that gives some confidence in using more of mistral in Tacker.. the idea is Tacker NSD feature could be decomposed into dynamically generated mistral workflows that we can invoke using Mistral APIs.. | 01:50 |
| sridhar_ram | KanagarajM: great, thanks! | 01:50 |
| KanagarajM | sridhar_ram, sripriya on patch set , https://review.openstack.org/329528 i would like to go with current one, as the current location is fine now. | 01:51 |
| dkushwaha_ | sridhar_ram, yes, And one idea is to test Tacker through, Mistral workflow | 01:51 |
| sridhar_ram | KanagarajM: i'm fine w/ the location, sripriya ^^ | 01:51 |
| dkushwaha_ | sridhar_ram, a complete lifecycle of vnfd/vnf. Like all crud operation in single workflow, that we can use for gate testing | 01:52 |
| dkushwaha_ | sridhar_ram, what you think? | 01:53 |
| sridhar_ram | dkushwaha_: probably not for gate testing, as it adds one more variability.. we would be exposed to mistral bugs for basic VNFM testing.. which is not good. | 01:54 |
| sridhar_ram | dkushwaha_: however for NSD, it the feature itself is designed with mistral.. it naturally will get tested using mistral as a dependency | 01:54 |
| sridhar_ram | *if the feature | 01:55 |
| dkushwaha_ | sridhar_ram, hmm. yes nice point | 01:55 |
| dkushwaha_ | sridhar_ram, if mistral test all sample workflow on gate(but i dont know the do) | 01:56 |
| dkushwaha_ | sridhar_ram, because tacker actions are not tested on mistral gate | 01:57 |
| dkushwaha_ | sridhar_ram, mistral gate testing should test tacker sample workflow (which going to be added in future) | 01:58 |
| *** KanagarajM_ has joined #tacker | 02:03 | |
| *** KanagarajM has quit IRC | 02:06 | |
| *** KanagarajM__ has joined #tacker | 02:08 | |
| dkushwaha_ | sridhar_ram, Or you mean to execute the complete nsd template through mistral on NSD api request? | 02:09 |
| *** KanagarajM_ has quit IRC | 02:12 | |
| *** sripriya has quit IRC | 02:13 | |
| *** manikanta_tadi has joined #tacker | 02:30 | |
| *** xuan_ has quit IRC | 02:31 | |
| *** Liuqing has joined #tacker | 02:33 | |
| openstackgerrit | Manikantha Srinivas Tadi proposed openstack/tacker-specs: Tacker VNFC support https://review.openstack.org/339798 | 02:50 |
| *** prashantD has joined #tacker | 02:51 | |
| *** Liuqing_ has joined #tacker | 02:52 | |
| openstackgerrit | xu-haiwei proposed openstack/python-tackerclient: Deprecate '--config' option when create/update a vim https://review.openstack.org/323192 | 02:54 |
| *** Liuqing has quit IRC | 02:55 | |
| *** KanagarajM__ has quit IRC | 03:13 | |
| *** derekjhyang has joined #tacker | 03:15 | |
| *** gongysh has quit IRC | 03:46 | |
| *** manikanta_tadi has quit IRC | 03:51 | |
| *** saju_m has joined #tacker | 03:54 | |
| *** shaikapsar has quit IRC | 03:59 | |
| *** links has joined #tacker | 04:03 | |
| *** amotoki has quit IRC | 04:07 | |
| *** KanagarajM__ has joined #tacker | 04:08 | |
| openstackgerrit | Kanagaraj Manickam proposed openstack/tacker: VNF scaling: TOSCA definitions and sample template https://review.openstack.org/329528 | 04:09 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/tacker: VNF scaling: Infra (heat) driver update https://review.openstack.org/330666 | 04:09 |
| openstackgerrit | Kanagaraj Manickam proposed openstack/tacker: VNF scaling: REST API https://review.openstack.org/330667 | 04:09 |
| *** amotoki has joined #tacker | 04:46 | |
| *** neel has joined #tacker | 05:02 | |
| *** prashantD has quit IRC | 05:08 | |
| *** amotoki has quit IRC | 05:10 | |
| *** amotoki has joined #tacker | 05:17 | |
| *** janki has joined #tacker | 05:22 | |
| *** janki has quit IRC | 05:27 | |
| *** janki has joined #tacker | 05:28 | |
| *** amotoki has quit IRC | 05:35 | |
| *** shaikapsar has joined #tacker | 05:39 | |
| *** tbh has joined #tacker | 05:44 | |
| *** amotoki has joined #tacker | 05:54 | |
| *** Liuqing_ has quit IRC | 05:57 | |
| *** Liuqing has joined #tacker | 05:59 | |
| *** Liuqing has quit IRC | 06:02 | |
| *** amotoki has quit IRC | 06:04 | |
| *** Liuqing has joined #tacker | 06:04 | |
| *** gongysh has joined #tacker | 06:09 | |
| *** amotoki has joined #tacker | 06:12 | |
| *** saju_m has quit IRC | 06:23 | |
| *** gongysh has quit IRC | 06:24 | |
| *** gongysh has joined #tacker | 06:24 | |
| *** tbh has quit IRC | 06:26 | |
| *** Ravikiran_K has joined #tacker | 06:29 | |
| gongysh | sridhar_ram, hi | 06:30 |
| openstackgerrit | Janki Chhatbar proposed openstack/python-tackerclient: VNFD legacy template deprecation warning https://review.openstack.org/334581 | 06:31 |
| *** saju_m has joined #tacker | 06:36 | |
| *** tbh has joined #tacker | 06:46 | |
| sridhar_ram | gongysh: hi | 06:58 |
| sridhar_ram | dkushwaha_: the later, complete nsd creation using mistral on ns API request | 07:00 |
| dkushwaha_ | sridhar_ram, yes, make sense. | 07:06 |
| janki | sridhar_ram: gongysh: I have updated the VNFD deprecate warning client patch. Please review. | 07:07 |
| gongysh | janki, look at my comments, thanks | 07:14 |
| *** neel has quit IRC | 07:14 | |
| openstackgerrit | Janki Chhatbar proposed openstack/python-tackerclient: VNFD legacy template deprecation warning https://review.openstack.org/334581 | 07:17 |
| janki | gongysh: looked, edited (_logger.warning) and submitted the patch | 07:18 |
| *** neel has joined #tacker | 07:26 | |
| openstackgerrit | Janki Chhatbar proposed openstack/python-tackerclient: VNFD legacy template deprecation warning https://review.openstack.org/334581 | 07:28 |
| *** gongysh has quit IRC | 07:37 | |
| *** santoshk has joined #tacker | 07:40 | |
| santoshk | KanagarajM__ :Hello | 07:41 |
| santoshk | can you please check updated comment for https://review.openstack.org/#/c/326670/ | 07:41 |
| openstackgerrit | gongysh proposed openstack/tacker: Add domain name info into auth for keystone v3 https://review.openstack.org/343446 | 07:43 |
| *** ksantoshk has joined #tacker | 07:49 | |
| *** santoshk has quit IRC | 07:53 | |
| *** diga has joined #tacker | 07:59 | |
| *** tbh has quit IRC | 08:11 | |
| *** ksantoshk has quit IRC | 08:14 | |
| openstackgerrit | Luka Krajger proposed openstack/python-tackerclient: Readd future import for print function https://review.openstack.org/347184 | 08:25 |
| *** neel has quit IRC | 08:38 | |
| *** amotoki has quit IRC | 08:57 | |
| *** amotoki has joined #tacker | 08:59 | |
| *** amotoki has quit IRC | 09:09 | |
| *** priya has joined #tacker | 09:10 | |
| *** priya has joined #tacker | 09:11 | |
| *** amotoki has joined #tacker | 09:11 | |
| *** mbound has joined #tacker | 09:25 | |
| *** DaveJ__ has joined #tacker | 09:33 | |
| *** saju_m has quit IRC | 09:49 | |
| *** Liuqing has quit IRC | 10:07 | |
| *** tbh has joined #tacker | 10:13 | |
| *** lulei has joined #tacker | 10:23 | |
| *** amotoki has quit IRC | 10:24 | |
| *** amotoki has joined #tacker | 10:25 | |
| lulei | hi, KanagarajM. I got a patch that Use assertEqual() instead of assertDictEqual(). Can you give me some suggestions? Thank you. Here is link:https://review.openstack.org/#/c/346659/ | 10:27 |
| *** trozet has quit IRC | 10:29 | |
| *** amotoki has quit IRC | 10:41 | |
| *** Ravikiran_K has quit IRC | 10:45 | |
| *** diga has quit IRC | 10:56 | |
| *** trozet has joined #tacker | 11:04 | |
| *** amotoki has joined #tacker | 11:07 | |
| *** amotoki has quit IRC | 11:11 | |
| *** Liuqing has joined #tacker | 12:18 | |
| *** Liuqing has quit IRC | 13:06 | |
| *** KanagarajM__ has quit IRC | 13:07 | |
| *** Liuqing has joined #tacker | 13:08 | |
| *** Liuqing has quit IRC | 13:14 | |
| *** Liuqing has joined #tacker | 13:15 | |
| *** Liuqing has quit IRC | 13:19 | |
| *** neel has joined #tacker | 13:29 | |
| *** neel has quit IRC | 13:30 | |
| *** links has quit IRC | 13:44 | |
| *** bobh has joined #tacker | 13:51 | |
| *** bobh has quit IRC | 13:56 | |
| *** bobh has joined #tacker | 13:59 | |
| *** janki has quit IRC | 14:04 | |
| *** Liuqing has joined #tacker | 14:34 | |
| *** bobh has quit IRC | 15:01 | |
| *** mbound has quit IRC | 15:07 | |
| *** janki has joined #tacker | 15:20 | |
| *** Liuqing has quit IRC | 15:29 | |
| *** bobh has joined #tacker | 15:32 | |
| *** KanagarajM has joined #tacker | 15:42 | |
| *** bobh has quit IRC | 15:43 | |
| *** neel has joined #tacker | 15:45 | |
| *** bobh has joined #tacker | 15:47 | |
| KanagarajM | sridhar_ram, ping | 15:50 |
| *** janki has quit IRC | 15:57 | |
| *** neel has quit IRC | 16:03 | |
| *** bobh has quit IRC | 16:05 | |
| *** shaikapsar has joined #tacker | 16:06 | |
| *** mbound has joined #tacker | 16:08 | |
| *** prashantD has joined #tacker | 16:08 | |
| *** mbound has quit IRC | 16:13 | |
| *** scottbarvick has joined #tacker | 16:35 | |
| *** KanagarajM has quit IRC | 16:39 | |
| *** sripriya has joined #tacker | 16:56 | |
| *** KanagarajM has joined #tacker | 16:57 | |
| KanagarajM | sripriya, ping | 16:58 |
| sripriya | KanagarajM: pong | 16:58 |
| KanagarajM | sripriya, do you have some time now to review the scaling patches ? | 16:58 |
| sripriya | KanagarajM: will review it today, did you have any particular comment you wanted to discuss | 16:59 |
| KanagarajM | sripriya, i have incorporated the comments and wanted to close it today, as for next two days we have meetup, we may not get time to look into it | 16:59 |
| KanagarajM | sripriya, so planning to work now :) | 17:00 |
| KanagarajM | sripriya, yes, first two patches, i think all your comments addressed, REST API patch wanted to discuss. i will take it latter once first two patches in . | 17:01 |
| sripriya | KanagaraM: appreciate it :-) just looked at the client patch comment on ScaleCommand | 17:03 |
| sripriya | KanagarajM: ^ | 17:03 |
| sripriya | KanagarajM: i believe it is not a good idea to have a resource specific call scale_vnf in the __init__ file which is handling generic resource calls | 17:04 |
| KanagarajM | sripriya, ok. I will move client thing. | 17:05 |
| sripriya | KanagarajM: please include unit tests for scale command | 17:05 |
| *** bobh has joined #tacker | 17:05 | |
| KanagarajM | sripriya, sure | 17:06 |
| KanagarajM | sripriya, can you look at server side first two patches ? | 17:06 |
| sripriya | KanagarajM; in client patch on rest call for scale, are we going to invoke REST api as '/vnfs/<uuid>/actions' with body containing ‘scale’ resource? | 17:08 |
| KanagarajM | sripriya, yes | 17:08 |
| *** bobh has quit IRC | 17:10 | |
| sripriya | KanagarajM: we need to capture this thoroughly in the user guide since our actual REST api implemenation and blueprint discussion on the API url is different now | 17:12 |
| KanagarajM | sripriya, yes, I will put an patch on spec and capture it in the docs as well, that is the plan. | 17:13 |
| sripriya | KanagarajM: ok, thanks | 17:13 |
| KanagarajM | sripriya, i will do that once we merge the patches , as it make more sense :) | 17:14 |
| KanagarajM | sripriya, are you looking at other two patches ? | 17:23 |
| sripriya | KanagarajM: :-) while i appreciate you probing me to review these patchses and value your effort, please understand significant features like scaling will require more time to review and cant be done in a haste (at least for me) and you should take feedbacks from others as well on the patches (and probe them too ;) ) | 17:29 |
| *** scottbarvick_ has joined #tacker | 17:33 | |
| *** scottbarvick has quit IRC | 17:34 | |
| *** scottbarvick_ is now known as scottbarvick | 17:34 | |
| KanagarajM | sripriya, hey, bascailly i don't like probing others :( but doing because, i want them to get go..... | 17:36 |
| KanagarajM | sripriya, and after seeing your last comment on the heat driver changes, i felt that you are fine with that patch ! ;) | 17:38 |
| sripriya | KanagarajM: i can absolutely understand, of course this is a priority and needs to go in soon! | 17:38 |
| sripriya | KanagarajM: which comment are you referring to, is it on the scale_wait? | 17:39 |
| KanagarajM | sripriya, thanks for understanding . | 17:39 |
| KanagarajM | sripriya, yes | 17:39 |
| sripriya | KanagarajM: :-) regarding scale_wait, we want to get away from that approach for the newer features | 17:41 |
| sripriya | KanagarajM: my comment was the method naming | 17:41 |
| *** prashantD_ has joined #tacker | 17:41 | |
| KanagarajM | sripriya, IMO, let us not make any inconsistency now and we could do the better naming later for all | 17:42 |
| *** prashantD has quit IRC | 17:42 | |
| sripriya | KanagarajM: i know you followed the existing convention for the naming, but we want to get away from that kinda naming as it appears to the user/developer that it is another create or scale method but the only thing it does is to poll the heat stack status | 17:42 |
| *** santoshk has joined #tacker | 17:44 | |
| sripriya | KanagarajM: let us not introduce more effort in refactoring, better to do it the right way than doing it the bad way and later fix it. IMO, it is okay to have these differences since the others will also be fixed later. | 17:44 |
| KanagarajM | sripriya, I feel let it go, as always believe on the consistency than partial betterment ! | 17:44 |
| KanagarajM | sripriya, oh !! hey let us not debate too much on this and its not that complicated refactoring. see, tomorrow if you change your mind to name something else, again the method name to be changed, :) | 17:45 |
| sripriya | KanagarajM: and i see that you have introduced a new driver for scaling which is using the same method convention as old driver which again should give you more freedom to do the right naming | 17:45 |
| KanagarajM | sripriya, ^^ pls | 17:46 |
| sripriya | KanagarajM: it all starts with setting the wrong foot forward ;) and then on it is just downhill, any,ways i’m fine to keep this method naming, but please keep in mind that you need not develop new features which is forced to adhere to existing framework, we are free to be creative :) | 17:50 |
| *** DaveJ__ has quit IRC | 17:50 | |
| KanagarajM | sripriya, hmm !! IMO, i always correct the existing mistake then make new patches consistent. but this scaling case i have not done, as its not that high priority like scaling and my available time became critical fighter now a days if you have looked at earlier patches, i have followed this approach. .... discussion apart ... when i find free time, surely i will land a patch to name the xxx_wait methods with nice-creating-meaningful naming :) | 17:57 |
| sripriya | KanagarajM: yup, thanks for doing that, i need to learn to do these things as well. :-) | 18:07 |
| *** bobh has joined #tacker | 18:07 | |
| KanagarajM | sripriya, sure. :) | 18:07 |
| *** bobh has quit IRC | 18:11 | |
| *** sripriya has quit IRC | 18:27 | |
| *** vishnoianil has quit IRC | 18:29 | |
| *** scottbarvick has quit IRC | 18:29 | |
| *** bobh has joined #tacker | 18:30 | |
| *** tbh has quit IRC | 18:36 | |
| *** sripriya has joined #tacker | 18:38 | |
| *** KanagarajM has quit IRC | 18:41 | |
| *** neel has joined #tacker | 19:06 | |
| *** neel has quit IRC | 19:06 | |
| *** prashantD has joined #tacker | 19:13 | |
| *** prashantD_ has quit IRC | 19:14 | |
| *** vishnoianil has joined #tacker | 19:21 | |
| *** sripriya has quit IRC | 21:03 | |
| *** sripriya has joined #tacker | 21:21 | |
| openstackgerrit | Tim Rozet proposed openstack/tacker: [WIP] Implements VNFFG into NFVO https://review.openstack.org/344522 | 21:42 |
| *** bobh has quit IRC | 21:49 | |
| *** bobh has joined #tacker | 21:52 | |
| *** prashantD_ has joined #tacker | 22:03 | |
| openstackgerrit | Stephen Wong proposed openstack/tacker: VNFFG chain driver abstract driver https://review.openstack.org/347563 | 22:03 |
| *** openstackgerrit has quit IRC | 22:03 | |
| *** openstackgerrit has joined #tacker | 22:04 | |
| *** bobh has quit IRC | 22:05 | |
| *** prashantD has quit IRC | 22:06 | |
| openstackgerrit | Stephen Wong proposed openstack/tacker: [WIP] VNFFG service chain Neutron networking-sfc driver https://review.openstack.org/347568 | 22:19 |
| openstackgerrit | vishwanath jayaraman proposed openstack/tacker: Support purge of soft-deleted resources from DB tables https://review.openstack.org/329652 | 22:34 |
| *** openstackgerrit has quit IRC | 22:48 | |
| *** openstackgerrit has joined #tacker | 22:49 | |
| *** prashantD__ has joined #tacker | 22:54 | |
| *** prashantD_ has quit IRC | 22:58 | |
| *** diga has joined #tacker | 23:52 | |
| *** sripriya has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!