08:04:56 #startmeeting tacker 08:04:57 Meeting started Tue Jan 19 08:04:56 2021 UTC and is due to finish in 60 minutes. The chair is yasufum. Information about MeetBot at http://wiki.debian.org/MeetBot. 08:04:58 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 08:05:00 The meeting name has been set to 'tacker' 08:05:42 #link https://etherpad.opendev.org/p/tacker-meeting 08:06:23 Thanks for suggesting topics for the meeting. 08:06:49 Can we start from the first one? 08:07:09 yoshito-ito: Are you joining? 08:09:15 seems he is not here... 08:09:48 manpreet: can you start? 08:10:07 yes, thanks. Please check L319 08:10:47 #topic attach-existing-volume 08:11:37 sorry I'm late 08:12:33 This is regarding adding FT for feature, I have added my queries in subsequent line L331 08:13:15 Seeking for help from team. 08:15:09 I would like to clear your point. 08:15:41 Is there any problem while trying to create a volume? 08:15:55 Or, just asking how to do it? 08:17:38 Yes, would like to know how to create volume ? As for testing purpose I have used cinder CLI, but as I have checked tacker functional testing code seems we are not using cinder client. 08:18:26 Or there is any other way to achieve the same. Sorry for naïve queries. 08:19:02 understand 08:19:11 I think you can refer https://review.opendev.org/c/openstack/tacker/+/760231 for adding new tasks in FT. 08:20:56 Create a new ansible task to create a cinder volume with CLI, to get the volume ID, and to replace the ID with the ID in VNFD. 08:21:28 yoshito-ito: manpreet just asking there are any examples for using cinder from FT code. 08:22:10 And from my quick review, there is no examples in tacker currently. 08:24:19 manpreet: I think you can find some examples from other than tacker, but I am not sure actually. 08:24:46 Can we continue to discuss for implementation on gerrit? 08:25:05 because it can be something detailed... 08:26:06 I'll put some notes for that on the etherpad. I agree that the implementation is too detailed to discuss here. 08:26:34 Thanks yoshito-ito for suggestion will definitely look into it. Yes Yasufum we could discuss on gerrit, as I would work upon the suggestions. 08:26:57 thanks 08:27:02 Thanks for help guys :) 08:28:02 Can we go to the first topic on our etherpad, yoshito-ito? 08:28:16 thanks 08:28:33 #topic FT jobs for new features in W-release 08:29:06 As you know, we have some patches on Gerrit trying to add new FT jobs. 08:29:57 1. (my patch) Spliting existing FT job to Legacy and SOL, and making them stable 08:30:16 2. (from horie-san) Adding FT job for K8s-VIM 08:30:43 3. (from takahashi-san) Adding FT job for separated NFVO 08:31:44 Because our Tacker FT job is now unstable and force us to "recheck" many times, I'd like you to review my patch first to fix it. 08:32:46 The changes are described in the commit massage of https://review.opendev.org/c/openstack/tacker/+/770609 and the etherpad. 08:35:01 So do you agree? or do you have any opinions for that? 08:37:08 I agree with you, partly because the FT exam time was also extended. 08:37:42 +1, It is very helpful to be free from repeated rechecks! 08:38:37 also +1, really helpful for me. 08:40:07 Agree, and I understand that my patch should be updated. I create new "functional_xxx" but "functional/xxx" seems better. 08:41:53 I think so 08:43:44 Thank you. My patch is now under Zuul test due to the merge conflict with the patch enabling Ceilometer again. 08:44:56 It will end in hours. 08:45:57 thank, please review and leave your comment, reviewers 08:47:08 #topic slim-down-etherpad 08:48:26 As you may know, I have updated the contents of our etherpad sometimes, but still mess... 08:49:15 So, I would like to suggest to move large topics to other pages. 08:50:04 I have started from topics clearly take long time. 08:51:52 Sorry but many of them is due to me... Do you mean make new page and move to it? And move large topics or old topics? In my opinion, to move old topics is also necessary. 08:51:59 I would like to also ask to move your topic to another page if it can be large 08:52:28 yes 08:53:15 I roughly made a rule of URL as described in the etherpad, but not mandatory. 08:53:26 Thanks for your help 08:55:00 Thanks 08:56:05 ueha: can we go to your topic? 08:56:11 I'm sorry just before, I added a topic to etherpad. 08:56:19 ok, thanks 08:56:21 OK, please go ahead 08:56:32 #topic Change size of instance_id of vnf table 08:56:46 I am trying to change the Type of instance_id in the vnf table with the following patch for CNF operation. 08:56:55 https://review.opendev.org/c/openstack/tacker/+/768217 08:57:47 mysql has a row limit [1] in each tables, and discusses what to set for the number of characters with yasufumi-san. 08:58:01 [1] https://dev.mysql.com/doc/refman/8.0/en/column-count-limit.html 08:58:38 I think there are two ways: 08:59:16 The first one is to change it to String (12800) or lager, as we are currently discussed with yasufumi-san in the patch. 08:59:40 Secondly, considering the row limit, it is better to use Text in consideration of the future. 09:00:07 Which do you think is better? 09:02:18 yasufumi-san suggested before that it is better to change the size of the String than to make it text. 09:05:10 I think you need to some more explanation about your change. 09:05:43 also we do not have much time... 09:06:34 My point is I think no need to use TEXT type because it is enough remained String type. 09:07:58 I doubt it can be such a large size as not enough for String. 09:09:30 We need to have some limitations if you changes the type to TEXT, such as careless upper or lower 09:10:20 Thanks, I understand the concerns of making it text. 09:11:26 Changing to String(12800), I think enough to store about 100 of k8s resource in instance_id. 09:11:34 I do not still understand your concerns yet. Please continue to discuss. 09:11:37 on gerrit 09:11:50 thansk 09:12:35 OK, thanks 09:13:10 If you have no more comments or topics, I would like close. 09:13:18 sorry,could I have a small topic? 09:13:23 sure 09:13:31 gathering for review on patch https://review.opendev.org/c/openstack/tacker/+/764138 09:14:13 It is a small patch but need to be merged for k8s FT, so I would like to call team to have a review on it, 09:15:49 Currently I added UT code for the patch as yasufum-san mentioned on gerrit, and no more further fix on it 09:16:44 OK. I did not find the test result turns green, sorry. 09:18:11 Looking forward to your review! 09:18:24 and that is all for my topic :) 09:18:51 thanks 09:19:43 seems no more comments 09:20:11 Thank you for today, bye! 09:20:43 thanks, bye 09:20:51 bye! 09:20:58 #endmeeting