*** Dinesh__Bhor has joined #tacker | 00:28 | |
*** Dinesh__Bhor has quit IRC | 00:31 | |
*** gongysh has joined #tacker | 01:10 | |
*** YanXing_an has joined #tacker | 01:13 | |
*** openstackgerrit has joined #tacker | 01:17 | |
openstackgerrit | LongKB proposed openstack/python-tackerclient master: Implement Tacker Client to support VNF cluster features https://review.openstack.org/526364 | 01:17 |
---|---|---|
*** gongysh has quit IRC | 01:20 | |
*** links has joined #tacker | 01:28 | |
*** links has quit IRC | 01:29 | |
*** daidv has joined #tacker | 01:33 | |
*** Dinesh__Bhor has joined #tacker | 01:38 | |
*** Dinesh__Bhor has quit IRC | 01:40 | |
*** Li_Jiale has joined #tacker | 01:40 | |
*** YanXing_an has quit IRC | 01:57 | |
*** gongysh has joined #tacker | 02:10 | |
*** links has joined #tacker | 02:12 | |
*** Dinesh__Bhor has joined #tacker | 02:23 | |
*** Dinesh__Bhor has quit IRC | 02:23 | |
*** Dinesh__Bhor has joined #tacker | 02:28 | |
*** Dinesh__Bhor has quit IRC | 02:29 | |
*** nmathew has joined #tacker | 02:32 | |
*** nmathew has quit IRC | 02:33 | |
*** nmathew has joined #tacker | 02:33 | |
*** nmathew has quit IRC | 02:51 | |
*** trinaths has quit IRC | 03:00 | |
*** masaito has joined #tacker | 03:37 | |
openstackgerrit | gongysh proposed openstack/tacker master: Implement update VNFFG functionality https://review.openstack.org/531817 | 03:48 |
openstackgerrit | Merged openstack/tacker master: Add document and release note for containerised vnf feature https://review.openstack.org/536143 | 03:52 |
*** openstackgerrit has quit IRC | 04:04 | |
*** YanXing_an has joined #tacker | 04:19 | |
*** YanXingAn has joined #tacker | 04:21 | |
*** YanXingAn has quit IRC | 04:24 | |
*** YanXingAn has joined #tacker | 04:28 | |
Li_Jiale | o/ | 04:32 |
*** Li_Jiale has left #tacker | 04:33 | |
*** Li_Jiale has joined #tacker | 04:34 | |
gongysh | mardim, hi | 04:34 |
phuoc | Li_Jiale hi | 04:34 |
YanXing_an | #openstack-meeting | 04:34 |
gongysh | mardim, hi | 04:36 |
*** YanXingAn has quit IRC | 04:36 | |
phuoc | gongysh, I think he is not awake now | 04:37 |
*** daidv has quit IRC | 04:44 | |
*** janki has joined #tacker | 04:46 | |
phuoc | masaito, hi | 05:04 |
*** janki has quit IRC | 05:05 | |
*** janki has joined #tacker | 05:06 | |
gongysh | phuoc, https://review.openstack.org/#/c/538586/5 | 05:14 |
gongysh | have you tested this? | 05:14 |
phuoc | I tested it, but I think we should wait mardim patch | 05:14 |
*** masaito has quit IRC | 05:15 | |
phuoc | because it can cause conflict when some files change concurrent | 05:15 |
gongysh | ok | 05:20 |
*** openstackgerrit has joined #tacker | 05:24 | |
openstackgerrit | Merged openstack/tacker master: Zuul: Remove project name https://review.openstack.org/539764 | 05:24 |
gongysh | dkushwaha, phuoc, who is writing the proposal sfc between openstack and container? | 05:28 |
openstackgerrit | Merged openstack/tacker master: Remove use of unsupported TEMPEST_SERVICES variable https://review.openstack.org/536814 | 05:29 |
openstackgerrit | Merged openstack/tacker master: Fix role name for OS::Nova::Flavor https://review.openstack.org/532469 | 05:29 |
phuoc | gongysh, he is Trinh, my colleague | 05:29 |
gongysh | I think we can consider the cluster feature across openstack vim and k8s vim. | 05:29 |
phuoc | he is starting to contribute to Tacker | 05:30 |
gongysh | since our first topic is talking about sfc | 05:30 |
gongysh | the third one can focus on cluster across vims. | 05:30 |
phuoc | gongysh, we will add that feature | 05:31 |
dkushwaha | gongysh, you mean a separate session for cluster feature, right? | 05:33 |
dkushwaha | or to combine in the same? | 05:33 |
gongysh | dkushwaha, I mean the third one at etherpad | 05:36 |
gongysh | the third one can talk about cluster across vims. | 05:36 |
gongysh | not to focus vnffg at 3rd topic. | 05:37 |
dkushwaha | got it | 05:37 |
openstackgerrit | Merged openstack/tacker master: Imported Translations from Zanata https://review.openstack.org/538568 | 05:41 |
*** jchhatbar has joined #tacker | 06:19 | |
*** jchhatbar has quit IRC | 06:20 | |
*** jchhatbar has joined #tacker | 06:21 | |
*** janki has quit IRC | 06:21 | |
*** dangtrinhnt1 has joined #tacker | 06:35 | |
*** trinaths has joined #tacker | 06:46 | |
longkb | hi gongysh, can I register my cluster patch in CFP? | 07:06 |
longkb | in 4th session | 07:06 |
*** egonzalez has joined #tacker | 07:07 | |
gongysh | sure | 07:08 |
gongysh | I hope you guys do this n 3rd session. | 07:08 |
longkb | Ok, I will push it in 3rd session | 07:10 |
longkb | Thanks | 07:10 |
openstackgerrit | gongysh proposed openstack/python-tackerclient master: add reno note for vim osc commands https://review.openstack.org/541591 | 07:19 |
gongysh | dkushwaha, hi | 07:23 |
dkushwaha | gongysh, hi | 07:23 |
gongysh | phuoc dkushwaha longkb , don't forget to register your session at openstack site. | 07:23 |
dkushwaha | gongysh, sure | 07:24 |
openstackgerrit | gongysh proposed openstack/python-tackerclient master: Add reno note for classifier name field in its list command https://review.openstack.org/541593 | 07:28 |
openstackgerrit | gongysh proposed openstack/python-tackerclient master: Add reno note for vnffg template updation command https://review.openstack.org/541594 | 07:32 |
openstackgerrit | gongysh proposed openstack/python-tackerclient master: Add reno note for cert_verify in vim config file https://review.openstack.org/541596 | 07:37 |
gongysh | dkushwaha, please merge some reno patches of tacker client | 07:40 |
gongysh | https://review.openstack.org/#/q/project:openstack/python-tackerclient | 07:40 |
gongysh | YanXing_an, please | 07:40 |
dkushwaha | gongysh, sure | 07:40 |
*** YanXing_an has quit IRC | 07:44 | |
phuoc | mardim, are you there? | 07:51 |
mardim | phuoc, yes hello | 07:51 |
openstackgerrit | Dimitrios Markou proposed openstack/tacker master: Implement update VNFFG functionality https://review.openstack.org/531817 | 07:53 |
phuoc | I have some problem with your patch | 07:53 |
phuoc | I will note it in review | 07:53 |
*** YanXing_an has joined #tacker | 07:54 | |
mardim | phuoc, ok | 07:54 |
openstackgerrit | gongysh proposed openstack/python-tackerclient master: Implement Tacker Client to support VNF cluster features https://review.openstack.org/526364 | 07:57 |
gongysh | dkushwaha, lets review reno patches of client. | 07:58 |
openstackgerrit | Merged openstack/python-tackerclient master: Implement Tacker Client to support VNF cluster features https://review.openstack.org/526364 | 08:13 |
openstackgerrit | Merged openstack/python-tackerclient master: Add reno note for classifier name field in its list command https://review.openstack.org/541593 | 08:13 |
openstackgerrit | Merged openstack/python-tackerclient master: Add reno note for cert_verify in vim config file https://review.openstack.org/541596 | 08:17 |
phuoc | mardim, https://github.com/openstack/tacker/blob/master/tacker/nfvo/nfvo_plugin.py#L435 | 08:18 |
phuoc | After I change from 'id' to 'instance_id', it works fine to me | 08:18 |
mardim | phuoc, I saw you review you are totally right | 08:18 |
mardim | phuoc, my mistake | 08:19 |
mardim | phuoc, I confused id with instance_id thre | 08:19 |
mardim | it was a type | 08:19 |
mardim | typo thanks a lot | 08:19 |
mardim | :) | 08:19 |
phuoc | it is okie | 08:19 |
mardim | I will fix it now | 08:19 |
mardim | phuoc, ^ | 08:19 |
phuoc | sure, I will give you my +1 later :-) | 08:20 |
*** dangtrinhnt1 has left #tacker | 08:20 | |
openstackgerrit | Merged openstack/python-tackerclient master: Add reno note for vnffg template updation command https://review.openstack.org/541594 | 08:21 |
*** dangtrinhnt1 has joined #tacker | 08:22 | |
openstackgerrit | Dimitrios Markou proposed openstack/tacker master: Implement update VNFFG functionality https://review.openstack.org/531817 | 08:23 |
mardim | phuoc, I updated the patch | 08:28 |
mardim | phuoc, Can you please check again ? :) | 08:28 |
phuoc | mardim, I see | 08:29 |
phuoc | I will check one more time :-) | 08:29 |
mardim | phuoc, Thanks i appreciate :) | 08:30 |
mardim | phuoc, If it is working please add a comment to the review so evreyone knows that evrything are ok | 08:30 |
phuoc | I just see your patch has problem with pep8 | 08:35 |
phuoc | :-) | 08:35 |
phuoc | please check it | 08:35 |
mardim | phuoc, where did you see that ? | 08:35 |
mardim | phuoc, Do you have a link ? | 08:35 |
phuoc | from Zuul | 08:36 |
mardim | phuoc, I am not seeing any ZUUL gates in my latest patch | 08:36 |
phuoc | maybe I am wrong | 08:36 |
phuoc | :-) | 08:36 |
phuoc | sorry | 08:36 |
mardim | phuoc, Are you sure you are looking the latest ? | 08:37 |
openstackgerrit | Merged openstack/python-tackerclient master: Update reno for stable/queens https://review.openstack.org/540023 | 08:37 |
mardim | phuoc, the right patch is the one with the 42 number | 08:38 |
mardim | fyi ^ :) | 08:38 |
phuoc | from patch set 40 | 08:38 |
mardim | I think I fixed the problem with pep* that caused in patchset 40 :) | 08:38 |
mardim | pep8* | 08:38 |
phuoc | Ahh, I see | 08:39 |
phuoc | mardim, after that, please see https://review.openstack.org/#/c/538586/5 | 08:46 |
phuoc | :-) | 08:46 |
phuoc | Thanks | 08:46 |
mardim | phuoc, Yes sure I will give me a minute :) | 08:47 |
openstackgerrit | gongysh proposed openstack/python-tackerclient master: Revert "Add reno note for vnffg template updation command" https://review.openstack.org/541617 | 08:54 |
openstackgerrit | Merged openstack/python-tackerclient master: Revert "Add reno note for vnffg template updation command" https://review.openstack.org/541617 | 09:06 |
openstackgerrit | gongysh proposed openstack/python-tackerclient master: Revert "Add reno note for cert_verify in vim config file" this should be in queens branch This reverts commit e557ecde7274da77c7aa6fa77227f8eb2e5f9a9e. https://review.openstack.org/541625 | 09:11 |
openstackgerrit | gongysh proposed openstack/python-tackerclient master: Revert "Add reno note for classifier name field in its list command" it should be at stable/queens branch This reverts commit df80486f32cf0011faadcbbc73e595e4afd4364c. https://review.openstack.org/541627 | 09:13 |
openstackgerrit | Merged openstack/python-tackerclient master: Revert "Add reno note for cert_verify in vim config file" this should be in queens branch This reverts commit e557ecde7274da77c7aa6fa77227f8eb2e5f9a9e. https://review.openstack.org/541625 | 09:28 |
openstackgerrit | Merged openstack/python-tackerclient master: Revert "Add reno note for classifier name field in its list command" it should be at stable/queens branch This reverts commit df80486f32cf0011faadcbbc73e595e4afd4364c. https://review.openstack.org/541627 | 09:28 |
mardim | gongysh, phuoc I tested the patch too I think is ok https://review.openstack.org/#/c/531817 | 09:53 |
gongysh | mardim, ok | 09:54 |
gongysh | when it passes CI, I will +2 | 09:54 |
mardim | gongysh, sounds good thanks | 09:54 |
*** YanXing_an has quit IRC | 09:56 | |
openstackgerrit | dharmendra kushwaha proposed openstack/python-tackerclient master: Complete VNFD osc commands https://review.openstack.org/540000 | 10:29 |
*** gongysh has quit IRC | 10:33 | |
*** trinaths has quit IRC | 11:30 | |
*** Li_Jiale has quit IRC | 11:52 | |
*** Nuri has joined #tacker | 13:00 | |
*** Nuri has quit IRC | 13:05 | |
dangtrinhnt1 | Hi guys, This patch of mardim was abandoned since Oct last year https://review.openstack.org/#/c/510003/ Is there any reason or that? | 13:08 |
mardim | dangtrinhnt1, Hello | 13:12 |
mardim | dangtrinhnt1, it is abandoned because I am planning to continue it | 13:12 |
mardim | into the future because right now this is not a high priority patch | 13:13 |
mardim | so when I finish with high priority patches I will update it :) | 13:13 |
dangtrinhnt1 | Ah, thanks for the updates. I just want to gather information for my bp :D | 13:15 |
mardim | ok :) | 13:15 |
*** gongysh has joined #tacker | 13:15 | |
dangtrinhnt1 | This patch is not high priority now but can anyone review this? https://review.openstack.org/#/c/540310/ (add dhcp, dns, and qos services for OpenWRT VNFM mgmt driver) I just want to finish this to work on my next patches. Thanks! :) | 13:22 |
openstackgerrit | Merged openstack/tacker master: Implement update VNFFG functionality https://review.openstack.org/531817 | 13:29 |
gongysh | dangtrinhnt1, hi | 13:38 |
gongysh | we will not merge new feature patch until queens branch is cut. | 13:39 |
*** jchhatba_ has joined #tacker | 13:45 | |
dangtrinhnt1 | gongysh, hi. Thanks for the info. So now Can I wait until rocky for the patch merged (maybe)? Is it what you mean? Thanks. | 13:47 |
*** jchhatbar has quit IRC | 13:48 | |
gongysh | dangtrinhnt1, you can do patches, but we will not merge until the queens branch is out. | 13:48 |
gongysh | dangtrinhnt1, this will happen in a few days. | 13:48 |
dangtrinhnt1 | Ah, I got it. Thanks :) | 13:49 |
*** links has quit IRC | 14:09 | |
*** daidv_ has joined #tacker | 14:13 | |
openstackgerrit | Trinh Nguyen proposed openstack/tacker master: Add dnsmasq and qos support for OpenWRT VNFM management driver https://review.openstack.org/540310 | 14:13 |
openstackgerrit | Trinh Nguyen proposed openstack/tacker-specs master: [WIP] SFC support for mixed type VNFs (VM & container) https://review.openstack.org/541735 | 14:49 |
openstackgerrit | Trinh Nguyen proposed openstack/tacker-specs master: [WIP] Prometheus plugin for Application & Container Monitoring in Tacker VNFM https://review.openstack.org/540416 | 14:51 |
openstackgerrit | Nguyen Hai proposed openstack/tacker-specs master: [WIP] VNF Marketplace https://review.openstack.org/541743 | 15:11 |
openstackgerrit | Cong Phuoc Hoang proposed openstack/tacker master: Support creating VNFFG with no flow-classifier https://review.openstack.org/538586 | 15:14 |
*** gongysh has quit IRC | 15:23 | |
*** egonzalez has quit IRC | 15:59 | |
*** jchhatba_ has quit IRC | 16:00 | |
*** mardim has quit IRC | 16:33 | |
openstackgerrit | Cong Phuoc Hoang proposed openstack/tacker master: Support creating VNFFG with no flow-classifier https://review.openstack.org/538586 | 17:40 |
*** bobh has joined #tacker | 19:35 | |
openstackgerrit | LongKB proposed openstack/tacker master: Implement Tacker Server to support VNF cluster features https://review.openstack.org/529808 | 21:07 |
*** bobh has quit IRC | 22:40 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!