| opendevreview | Yasufumi Ogawa proposed openstack/tacker master: Migrate rootwrap to privsep https://review.opendev.org/c/openstack/tacker/+/831433 | 00:36 |
|---|---|---|
| opendevreview | Aldinson C. Esto proposed openstack/tacker master: Add a Sample VNFD Package which uses the Ansible Driver https://review.opendev.org/c/openstack/tacker/+/830948 | 01:02 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support CNF update with MgmtDriver https://review.opendev.org/c/openstack/tacker/+/830963 | 01:19 |
| opendevreview | Yi Feng proposed openstack/tacker master: [WIP] FT of multi tenants for CNF https://review.opendev.org/c/openstack/tacker/+/830985 | 01:23 |
| opendevreview | Merged openstack/tacker master: Fix usageState of VNF package update https://review.opendev.org/c/openstack/tacker/+/832188 | 01:43 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support Basic Lifecycle Operations of CNF v2 API https://review.opendev.org/c/openstack/tacker/+/830946 | 01:55 |
| opendevreview | Itsuro Oda proposed openstack/tacker master: Fix subprocess usage https://review.opendev.org/c/openstack/tacker/+/832743 | 02:30 |
| opendevreview | Itsuro Oda proposed openstack/tacker master: Fix subprocess usage https://review.opendev.org/c/openstack/tacker/+/832743 | 02:40 |
| opendevreview | Masaki Oyama proposed openstack/tacker master: Fix grant usage in scaling out https://review.opendev.org/c/openstack/tacker/+/832748 | 03:22 |
| opendevreview | Masaki Oyama proposed openstack/tacker master: Fix grant usage in scaling out https://review.opendev.org/c/openstack/tacker/+/832748 | 03:24 |
| opendevreview | Yasufumi Ogawa proposed openstack/tacker master: [WIP] Introduce vagrant for devstack https://review.opendev.org/c/openstack/tacker/+/830881 | 03:55 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support Basic Lifecycle Operations of CNF v2 API https://review.opendev.org/c/openstack/tacker/+/830946 | 04:06 |
| opendevreview | Masaki Oyama proposed openstack/tacker master: Fix grant usage in scaling out https://review.opendev.org/c/openstack/tacker/+/832748 | 05:04 |
| opendevreview | Masaki Oyama proposed openstack/tacker master: [WIP] Add a VNF package sample for practical use cases https://review.opendev.org/c/openstack/tacker/+/832226 | 05:06 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support CNF update with MgmtDriver https://review.opendev.org/c/openstack/tacker/+/830963 | 05:07 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support CNF update with MgmtDriver https://review.opendev.org/c/openstack/tacker/+/830963 | 05:38 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support CNF update with MgmtDriver https://review.opendev.org/c/openstack/tacker/+/830963 | 05:44 |
| opendevreview | Masaki Oyama proposed openstack/tacker master: [WIP] Add a VNF package sample for practical use cases https://review.opendev.org/c/openstack/tacker/+/832226 | 06:41 |
| opendevreview | Yi Feng proposed openstack/tacker master: FT of scale/modify v2 APIs https://review.opendev.org/c/openstack/tacker/+/829156 | 07:24 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support ChangeCurrentVNFPackage for VNF of v2 API https://review.opendev.org/c/openstack/tacker/+/826249 | 07:33 |
| opendevreview | Yasufumi Ogawa proposed openstack/tacker master: [WIP] Introduce vagrant for devstack https://review.opendev.org/c/openstack/tacker/+/830881 | 08:05 |
| opendevreview | Merged openstack/tacker master: Do not set ineffective [alarm_auth] url https://review.opendev.org/c/openstack/tacker/+/832200 | 08:20 |
| opendevreview | Yi Feng proposed openstack/tacker master: [WIP] FT of multi tenants for CNF https://review.opendev.org/c/openstack/tacker/+/830985 | 08:38 |
| opendevreview | Tetsuro Kaneko proposed openstack/tacker master: Add documents for enhance operations in V2 https://review.opendev.org/c/openstack/tacker/+/831341 | 09:11 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support Basic Lifecycle Operations of CNF v2 API https://review.opendev.org/c/openstack/tacker/+/830946 | 09:25 |
| ueha | manpreetk: Hi, I would appreciate it if you could tell me how is the progress on the issue of missing the tenant_id in the vnf_resource? | 09:38 |
| opendevreview | renu rani proposed openstack/tacker master: Heal and scale method support of lcm_user_data https://review.opendev.org/c/openstack/tacker/+/825294 | 09:46 |
| opendevreview | Yasufumi Ogawa proposed openstack/tacker master: Introduce vagrant for devstack https://review.opendev.org/c/openstack/tacker/+/830881 | 09:57 |
| opendevreview | Yasufumi Ogawa proposed openstack/tacker master: Introduce vagrant for devstack https://review.opendev.org/c/openstack/tacker/+/830881 | 10:04 |
| manpreetk | ueha: Hi, Please give me some time, I will look into it and share my findings with you. Thanks | 10:11 |
| opendevreview | Merged openstack/tacker master: Fix subprocess usage https://review.opendev.org/c/openstack/tacker/+/832743 | 10:22 |
| opendevreview | Yi Feng proposed openstack/tacker master: Support ChangeCurrentVNFPackage for VNF of v2 API https://review.opendev.org/c/openstack/tacker/+/826249 | 10:38 |
| ueha | manpreetk: Thank you! Could you tell me when you think it will be? And we are also investigating, so if we find something, I will share it. | 10:39 |
| manpreetk | ueha: Hi, please find some high level investigation results for OpenStack as VIM, as I am not much aware of kubernetes code, https://paste.opendev.org/show/bNOiGeO9io6NLd5KBFPy/ | 11:54 |
| manpreetk | ueha: I haven't tested these changes and investigation is in progress. Thanks. | 11:56 |
| ueha | manpreetk: Thanks for your investigation! As a result of my investigation, the change of Object and Db model is the same. | 12:03 |
| ueha | I think the following changes are necessary at the kubernetes_driver: | 12:03 |
| ueha | 4. "tacker/vnfm/infra_drivers/kubernetes/kubernetes.py" (L1640~) | 12:03 |
| ueha | def _create_vnf_resource(self, context, vnf_instance, file_content_dict): | 12:03 |
| ueha | vnf_resource = vnf_resource_obj.VnfResource( | 12:03 |
| ueha | context=context) | 12:03 |
| ueha | vnf_resource.vnf_instance_id = vnf_instance.id | 12:03 |
| ueha | metadata = file_content_dict.get('metadata', {}) | 12:04 |
| ueha | vnf_resource.resource_name = metadata.get('name', ' ') | 12:04 |
| ueha | vnf_resource.resource_type = ','.join([ | 12:04 |
| ueha | file_content_dict.get('apiVersion', ''), | 12:04 |
| ueha | file_content_dict.get('kind', '')]) | 12:04 |
| ueha | vnf_resource.resource_identifier = '' | 12:04 |
| ueha | vnf_resource.resource_status = '' | 12:04 |
| ueha | + vnf_resource.tenant_id = vnf_instance.tenant_id | 12:04 |
| ueha | return vnf_resource | 12:04 |
| manpreetk | ueha:Thanks for sharing , yes I believe openstack_driver and kubernetes_driver changes will be enough to fix the issue. | 12:09 |
| ueha | manpreetk; Sure! Thanks too | 12:11 |
| manpreetk | Most welcome, please let me know in case u need my help, I will be here :) | 12:12 |
| ueha | Thank you! I will try in my local environment, but my env is crush... and I am struggling to reconstruct under proxy.. | 12:14 |
| manpreetk | No worries, meanwhile I will try to run this patch locally in case found issue would share with you. | 12:16 |
| ueha | Could you post a patch? If I can't test locally, I will try it with Zuul. Thanks. | 12:19 |
| manpreetk | Sure can do that. | 12:21 |
| ueha | Thanks a lot! RC1 is close, but let's do our best! :) | 12:22 |
| opendevreview | Manpreet Kaur proposed openstack/tacker master: Add specific tenant to vnf resources https://review.opendev.org/c/openstack/tacker/+/832793 | 13:28 |
| manpreetk | ueha: I have pushed patch with desired changes https://review.opendev.org/c/openstack/tacker/+/832793 | 13:28 |
| ueha | manpreetk: Thanks for your quick work! | 13:31 |
| manpreetk | ueha: No worries, I have added UT but please let me know if something more is needed. | 13:32 |
| ueha | I will confirm it. | 13:36 |
| manpreetk | Thanks :) | 13:36 |
| ueha | I've confirm that unit test for kubernetes_driver has passed in my local env! thank you. | 13:47 |
| manpreetk | Welcome ueha san. | 13:51 |
| opendevreview | Merged openstack/tacker master: Update release notes of multi tenant policy in LCM https://review.opendev.org/c/openstack/tacker/+/832686 | 14:03 |
| opendevreview | Ayumu Ueha proposed openstack/tacker master: [DNM] test for FT of multitenant CNF https://review.opendev.org/c/openstack/tacker/+/832802 | 14:54 |
| ueha_ | manpreetk: I've confirm that termination CNF with member role user (user: demo) was successful after applying patch/832793! | 15:13 |
| manpreetk | ueha: Great!! | 15:14 |
| ueha_ | XD | 15:14 |
| ueha_ | Thanks! | 15:14 |
| ueha_ | oops, sorry. I have added db migration file. | 15:18 |
| ueha_ | like https://review.opendev.org/c/openstack/tacker/+/799022/27/tacker/db/migration/alembic_migrations/versions/d6ae359ab0d6_add_tenant_id_to_lcm_subscriptions_and_.py | 15:18 |
| ueha_ | I think it needs to be added to patch/832793. | 15:19 |
| ueha_ | manpreetk: could you add the file? | 15:21 |
| manpreetk | ueha: Yes i missed that, my bad. Sure will add that. | 15:37 |
| ueha_ | manpreetk: thank you! | 15:39 |
| opendevreview | Ayumu Ueha proposed openstack/tacker master: [DNM] test for FT of multitenant CNF https://review.opendev.org/c/openstack/tacker/+/832802 | 16:11 |
| opendevreview | Manpreet Kaur proposed openstack/tacker master: Add specific tenant to vnf resources https://review.opendev.org/c/openstack/tacker/+/832793 | 16:44 |
| opendevreview | Yasufumi Ogawa proposed openstack/tacker master: Introduce vagrant for devstack https://review.opendev.org/c/openstack/tacker/+/830881 | 16:47 |
| opendevreview | Yasufumi Ogawa proposed openstack/tacker master: [WIP] Cleanup logs in tests https://review.opendev.org/c/openstack/tacker/+/783768 | 17:09 |
| opendevreview | Ayumu Ueha proposed openstack/tacker master: [DNM] test for FT of multitenant CNF https://review.opendev.org/c/openstack/tacker/+/832802 | 17:20 |
| opendevreview | Merged openstack/tacker master: Heal and scale method support of lcm_user_data https://review.opendev.org/c/openstack/tacker/+/825294 | 18:29 |
| opendevreview | Ayumu Ueha proposed openstack/tacker master: Remove singleton class in FakeServerManager class https://review.opendev.org/c/openstack/tacker/+/831566 | 23:31 |
| opendevreview | Ayumu Ueha proposed openstack/tacker master: FT for multi tenant policy in LCM https://review.opendev.org/c/openstack/tacker/+/823965 | 23:31 |
| opendevreview | Ayumu Ueha proposed openstack/tacker master: [WIP] FT of multi tenants for CNF https://review.opendev.org/c/openstack/tacker/+/830985 | 23:31 |
| opendevreview | Ayumu Ueha proposed openstack/tacker master: [DNM] test for FT of multitenant CNF https://review.opendev.org/c/openstack/tacker/+/832802 | 23:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!