Tuesday, 2022-03-08

*** soniya29 is now known as soniya29|ruck05:21
yasufum-ohi tacker team08:00
yasufum-o#startmeeting tacker08:02
opendevmeetMeeting started Tue Mar  8 08:02:14 2022 UTC and is due to finish in 60 minutes.  The chair is yasufum-o. Information about MeetBot at http://wiki.debian.org/MeetBot.08:02
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.08:02
opendevmeetThe meeting name has been set to 'tacker'08:02
yasufum-oThere are five topics today.08:03
yasufum-oI'd like to share my items, 2nd and 3rd, because it's just announcements.08:04
yasufum-o2nd one is about yoga release of python-tackerclient.08:04
yasufum-oThere are some patches dropped in stable08:05
yasufum-oSo, update for statle is required.08:06
yasufum-oI'm not still sure the reason, but I would like to upload from NTT.08:06
yasufum-o3rd item is about RC1..08:07
yasufum-oPlease keep in mind the deadline.08:07
yasufum-oany comment?08:08
yasufum-oSo, can we go to the 1st item next?08:09
uehayes, 1st one is my topic.08:09
yasufum-othx, go ahead.08:09
uehaWe found a error when we implementing multi-tenant FT on the CNF side.08:10
uehaWe operated to termination with member role resulted in the following error:08:10
ueha  Error: Entity namespace for "vnf_resources" has no property "tenant_id"08:10
uehaI think this error happens not only in CNF but also in VNF because it happens in the common processing unit [1].08:10
ueha  [1] https://opendev.org/openstack/tacker/src/branch/master/tacker/vnflcm/vnflcm_driver.py#L677-L67808:11
uehaIn order to terminate successfully with member role, should we add a `tenant_id` to `vnf_resource` as well as vnf_lcm_subscriptions or vnf_lcm_op_occs like patch [2]? or any other fix?08:11
ueha  [2] https://review.opendev.org/c/openstack/tacker/+/79902208:11
uehaw-juso or manpreetk: What do you think about this error and fix? and could you post patch for this error?08:11
uehaDidn't it happen when you tested in a local environment?08:12
uehathat's all from myside, thanks,08:12
uehaw-juso and manpreetk: any answer?08:14
manpreetkueha: Thanks for sharing. As you are aware VNF instantiation for member role is not working, therefore such scenario was not tested. But yes we could investigate the issue and upload patch.08:17
manpreetkBut again testing might be a issue, or may be on hold until VNF instantiation is enable for member role.08:18
uehaI see.. Do you think it will work on the RC1 deadline?08:20
manpreetkahh that might be difficult as need to investigate.08:21
uehaIn that case, we can't merge because the CNF multi-tenant test doesn't pass. umm..08:22
uehaShould we write in the issue of release note that the operation in member role does not work normally in Y version?08:23
uehawhat do you think?08:24
manpreetkueha: Yes as per my shared topic, member role user is unable to instantiate VNF and the solution might need some more time.08:24
uehaOK, let's discuss it again on your topic.08:25
uehaCNF seems to work well if the above problem is solved.08:26
uehaSo, the problem is whether to fix only CNF in Y release..08:27
uehaThanks, any other comments? If not, let's move on to the next topic.08:29
yasufum-omanpreetk: please share your topic08:31
manpreetkPlease check L4708:32
manpreetkCurrently the proposed VNF instantiation solution for member role is not working. More investigation is required in the following areas:08:33
manpreetkIdentify blocking policies through source code.08:33
manpreetkDebug the complete workflow with member role user with respect to infra setups like policy modification for nova, heat, etc (any other blocking policy).08:33
manpreetkSomehow this activity would consume a good amount of time and could be addressed in the upcoming release. 08:33
manpreetkPlease share your opinion, thats all from my side.08:35
takahashi-tscIMO, higher priority is to resolve the issue which Ueha-san shared. And very sorry but to resolve Heat role issue needs much time and not in time in Yoga release...08:36
yasufum-oueha: Do you have any suggestion for the fix?08:41
uehaNo, I don't have..08:42
ueharelease note of `bp/multi-tenant-policy` has been merged: https://review.opendev.org/c/openstack/tacker/+/83003708:43
uehaIf it can't make it to Y release, do we have to correct this?08:43
h_asahinaisn't possible to simply try to add the lacked property to vnf_resources? ueha:08:44
uehaWe haven't tried it yet, but I think it's possible. but I don't know if we have time to do it..08:47
uehaI'll check with my colleague.08:47
h_asahinaBTW, multitenat functionality is already available for admin role users? I think the problem only appears for member role users.08:48
h_asahinaIn other words, is it possible to release multi-tenant for admin role users in Yoga and exntend it for member role users in Zeta?08:51
uehamanpreetk: what do you think?08:55
manpreetk+1 yes i agree with h_asahina.08:55
h_asahinathank you two. I think that is a realistic goal. At least, we can provide multi-tenant for admin users.08:57
yasufum-omanpreetk: Does it satistify the requirements in spec?08:58
yasufum-o#link https://specs.openstack.org/openstack/tacker-specs/specs/yoga/multi-tenant-policy.html08:58
manpreetkyasufum-o: No this does not satisfy spec requirement, but we could edit release note with missing functionality and partially implement in Y release.09:02
yasufum-oI understand we don't have any alternative options for the issue for now.09:06
yasufum-oAnyway, we should talk about this spec in the next vPTG.09:07
yasufum-oGo on to the next topic09:09
ma-ooyamaSure. I would like to talk to you about two topics related to the sample VNF package I am working on https://review.opendev.org/c/openstack/tacker/+/832226.09:09
ma-ooyamaFirst, the sample can't be instantiated because of the bug of https://bugs.launchpad.net/tacker/+bug/1948925.09:10
ma-ooyamaI made a patch for this bug on https://review.opendev.org/c/openstack/tacker/+/832224, and would like you to review this.09:10
ma-ooyamaSecond, the sample can't be scaled out because of the bug of https://bugs.launchpad.net/tacker/+bug/1963900.09:11
ma-ooyamaI think it is difficult to have the bug fixed before yoga release. So I want to confirm whether the following is OK.09:11
ma-ooyama- In Yoga release, scale operation won't be supported in this sample.09:11
ma-ooyama- Add the explanation that this sample can't support scale operation now to user guide or read me.09:11
ma-ooyama- Fix the bug in future release, and then remove the explanation above.09:12
ma-ooyamaWhat do you think about this?09:13
yasufum-odo you have any comment?09:15
uehaI see the patch (https://review.opendev.org/c/openstack/tacker/+/832226) include scalable deployment flavour VNFD and BaseHot.09:16
uehaDo you mean that you leave this VNFD and BaseHot as they are and write "not support" in document or README?09:18
ma-ooyamaI think those VNFD and BaseHot is still located in the sample, but adding explanation that scale out operation is not supported now. 09:19
uehaI am wondering if there is no problem with putting the sample that we know won't work.09:22
uehaCan't you insert it after fixing the bug? What do other think?09:22
yasufum-oma-ooyama: Is there any reason why it's must be included in yoga?09:24
ma-ooyamaSorry, I originaly thought it is a matter of tacker implementation and so whole sample package can be merged.09:27
ma-ooyamaBut I understood your opinion.09:28
ma-ooyamaI understood it is better to include scalable deployment flavour after scale oparation is suceed.09:30
ma-ooyamaIn this case, is there something to do against losing the part of usecases in yoga release.09:32
ma-ooyamaI mean where should I explain that the scale flavour is implemented in future.09:34
uehaI think commit message and document/README? I don't know if there is any other place to write..09:36
uehaany other ideas?09:37
yasufum-oI'm not sure the purpose, but seems ok.09:39
yasufum-oma-ooyama: any other question?09:42
ma-ooyamaSorry I want to confirm that the sample that doesn't include scalable deployment flavour can be merged in yoga release?09:43
yasufum-oDo you mean only rst files will be remained in this patch?09:44
yasufum-otakahashi-tsc: oops, I've found the 6th item appears.09:46
ma-ooyamaI mean the deployment flavour of ha can be remained, or whole sample should be merged after scale operation works.09:46
takahashi-tscSorry, I forget to write my topic before this meeting. So I just annouce at this meeting today.09:47
yasufum-otakahashi-tsc: got it09:48
yasufum-oma-ooyama: I think it's acceptable, but please check the details in your updated patch.09:48
yasufum-ofor former one.09:49
yasufum-oueha: do you agree?09:49
uehaYes, I agree.09:50
ma-ooyamaI understood. Thank you for a lot of comment.09:50
yasufum-oyou're welcome!09:50
yasufum-oSo, the last topic09:51
yasufum-otakahashi-tsc: can you share it shortly?09:51
takahashi-tscSorry for the late, I think we have no time ... so shout annoucement.09:51
takahashi-tscPlease see https://etherpad.opendev.org/p/tacker-meeting#L60709:52
takahashi-tscWe discussed O-RAN SC activities before. I'm also interested in RAN area, so I participated O-RAN SC meeting.09:52
takahashi-tscBasically, O-RAN SC members allow Tacker community to contribute for O-RAN SC. So I want to start contribute.09:53
takahashi-tscAnd, I'm also looking for members who work with me. Today we have not enough time, but I'd like to continue to discuss how to proceed.09:53
takahashi-tscThat's all, thanks.09:54
yasufum-oThanks for sharing your progress. 09:55
yasufum-odo you have any comment?09:57
uehaWe will support but the deployment system has not been established and the internal participation procedure has not been completed. We will participate as soon as we are ready!09:59
yasufum-oI hope we'll continue the discussion for expanding our activity.10:00
yasufum-oWe should close this meeting today.10:00
yasufum-oThanks for joining, bye!10:00
manpreetkThanks, bye.10:01
uehathanks, bye10:01
opendevmeetMeeting ended Tue Mar  8 10:01:27 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)10:01
opendevmeetMinutes:        https://meetings.opendev.org/meetings/tacker/2022/tacker.2022-03-08-08.02.html10:01
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/tacker/2022/tacker.2022-03-08-08.02.txt10:01
opendevmeetLog:            https://meetings.opendev.org/meetings/tacker/2022/tacker.2022-03-08-08.02.log.html10:01
*** dasm|off is now known as dasm14:35
*** h_asahin1 is now known as h-asahina14:59
*** h-asahina is now known as h_asahina14:59
*** h_asahina is now known as h-asahina15:00
*** h-asahina is now known as h_asahina15:00
*** hemna8 is now known as hemna15:25
*** hemna7 is now known as hemna15:47
*** hemna0 is now known as hemna16:43
*** hemna7 is now known as hemna17:05

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!