| yasufum | Hi tacker team. | 08:00 |
|---|---|---|
| ueha | hi | 08:00 |
| masaki-ueno | hi | 08:01 |
| takahashi-tsc | hi | 08:01 |
| manpreetk | hi | 08:01 |
| yasufum | #startmeeting tacker | 08:02 |
| opendevmeet | Meeting started Tue Sep 6 08:02:06 2022 UTC and is due to finish in 60 minutes. The chair is yasufum. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:02 |
| opendevmeet | The meeting name has been set to 'tacker' | 08:02 |
| yasufum | #link https://etherpad.opendev.org/p/tacker-meeting | 08:03 |
| yasufum | there are two items on the etherpad today. | 08:03 |
| yasufum | The first one is mine. | 08:04 |
| yasufum | #topic releasenotes coverage | 08:04 |
| yasufum | In the last meeting, I've mentioned to check our releasenotes. | 08:05 |
| yasufum | It's because some features or bug fixes were missed to be merged in the latest releases. | 08:06 |
| yasufum | So, I'd like to check them before the end of this release. | 08:07 |
| yasufum | As described on the etherpad, roughtly 29 changes have no releasenotes | 08:08 |
| yasufum | which has "Implement" or "Closes-Bug" tag in its commit message. | 08:08 |
| yasufum | It's better to cover such a change in the releasenotes without tiny ones. | 08:09 |
| yasufum | The detailed results of the survey is here. | 08:10 |
| yasufum | #link https://etherpad.opendev.org/p/tacker-meeting-20220906-list-all-changes-in-zed | 08:10 |
| yasufum | So, please check your patches again and upload a releasenotes if it seems better to do so. | 08:11 |
| yasufum | that's all | 08:11 |
| yasufum | Do you have any comment? | 08:11 |
| ueha | Thanks for your survey and sharing, I will tell my colleagues and post releasenote if necessary. | 08:12 |
| yasufum | thanks :) | 08:12 |
| takahashi-tsc | No comments from my side, we'll confirm the list and make release note if required. | 08:12 |
| yasufum | OK, thanks. | 08:12 |
| yasufum | So, can we go to the next item? | 08:12 |
| yasufum | #topic About posted patch dependencies for RC1 | 08:13 |
| ueha | Sorry, I have a question about deadline of post releasenote. | 08:13 |
| yasufum | oops | 08:13 |
| ueha | We aim to post by RC1? or Final Release..? | 08:13 |
| yasufum | I wonder it's must not be critical changes and no need to have long time for reviewing. | 08:14 |
| ueha | Yes, I think so too. | 08:14 |
| yasufum | How about Sep 12? It's the first day of RC1 target week. | 08:16 |
| yasufum | I mean the deadline of uploading releasenotes. | 08:16 |
| takahashi-tsc | OK, I'll share the schedule with my team. | 08:17 |
| ueha | I agree, we should post it by Sep 12 and merge it by RC1. | 08:17 |
| yasufum | Thanks. I'd like to add the conclusion on my item on the ehterpad. | 08:18 |
| yasufum | OK, go to the second topic from ueha. | 08:19 |
| ueha | sure, I will share about posted patch dependencies for RC1. | 08:19 |
| ueha | Since RC1 (Sep 16) is approaching, we want to prevent a large merge conflict from occurring just before RC1. | 08:20 |
| ueha | It is assumed that merge conflicts will occur with CNF refactoring/enhancement patch, but there will be no major impact. | 08:20 |
| ueha | But, basically, it is related to the changes of the patch below, so I would like you to review it so that it can be merged as soon as possible. | 08:21 |
| ueha | * CNF v2 API enhance and refactor | https://review.opendev.org/c/openstack/tacker/+/855117 | 08:21 |
| ueha | * [WIP] Helm chart support for CNF v2 API | https://review.opendev.org/c/openstack/tacker/+/855579 | 08:21 |
| ueha | (The last one is WIP. | 08:21 |
| ueha | And it is clear that the some patches posted by FJ and the following UT improvement patch cause conflicts, so we want to merge them earlier. | 08:22 |
| ueha | Improve Unit Test for V2 API | https://review.opendev.org/c/openstack/tacker/+/848565 | 08:22 |
| ueha | That's all from my side. | 08:22 |
| ueha | Do you have any comment or question? | 08:23 |
| yasufum | thanks | 08:23 |
| yasufum | I also do that ASAP. | 08:25 |
| yasufum | Is there any concerns about dependency? No? | 08:26 |
| yasufum | good | 08:26 |
| ueha | As we are considering the affected point, there is nothing in particular at the moment. | 08:27 |
| yasufum | ok, good news for us :) | 08:27 |
| ueha | thank you :) | 08:28 |
| takahashi-tsc | Sorry please let me clarify the request. We should merge 855117 and 855579 earlier than other related patches such as 848565 | 08:28 |
| takahashi-tsc | Is my understanding correct? | 08:28 |
| ueha | No, I think 848565 is in the same level as 855117 and 855579. | 08:30 |
| takahashi-tsc | OK, so we should merge 855117, 855579 and 848565bearlier than other patches, right? | 08:31 |
| ueha | related patches I said is other CNF patch posted by FJ. | 08:31 |
| ueha | Yes | 08:31 |
| takahashi-tsc | Understood, thanks! | 08:31 |
| ueha | :) | 08:32 |
| yasufum | takahashi-tsc: Thanks for the comment. | 08:33 |
| yasufum | It seems enough for the topic. | 08:34 |
| yasufum | ueha: Thnak you for your mention. | 08:34 |
| yasufum | So, do you have any other topic than on the etherpad? | 08:35 |
| ma-ooyama | sorry, I have a topi. Is it OK? | 08:35 |
| yasufum | Sure, please go ahead. | 08:35 |
| ma-ooyama | Thanks. We are implementing the tool for DB migration, is FT implementation required? | 08:35 |
| ma-ooyama | https://review.opendev.org/c/openstack/tacker-specs/+/848410 | 08:36 |
| ma-ooyama | We think the migration needs to be tested to ensure that it can be done succesfully, but the tool we are implementing is not tacker's main funtion, we want to confirm that if it is OK to test the tool by FT. | 08:36 |
| yasufum | I'm not sure exactly, but no need to have a test, especially a functional test, in CI | 08:38 |
| yasufum | as similar to other tools. | 08:38 |
| yasufum | ma-ooyama: It will be implemented as a feature of tacker-db-manage, right? | 08:39 |
| ma-ooyama | Yes. | 08:40 |
| yasufum | We don't have any test for the tool currently. | 08:40 |
| ma-ooyama | Understood. So we will implement only unit test. | 08:41 |
| yasufum | IMO, it's better to implement some validation feature for your update to ensure it works correctly. | 08:43 |
| yasufum | For example, dry-run for checking arguments or so. | 08:43 |
| yasufum | Although it depends on your implementation. | 08:43 |
| yasufum | Any other comment on the topic? | 08:44 |
| ma-ooyama | Thanks. I understood that the test or feature to validate tool is needed, but FT is not needed. | 08:45 |
| yasufum | OK, it seems enough for the topic. | 08:47 |
| yasufum | So, let's close this meeting if no one has any othere comment topic. | 08:48 |
| yasufum | Good | 08:49 |
| yasufum | Thanks for joining, bye! | 08:49 |
| ueha | Thanks, bye | 08:49 |
| takahashi-tsc | thanks, bye | 08:49 |
| ma-ooyama | thanks, bye | 08:49 |
| masaki-ueno | bye | 08:49 |
| Ramona-ho-xu | thanks,bye | 08:49 |
| yasufum | #endmeeting | 08:49 |
| opendevmeet | Meeting ended Tue Sep 6 08:49:49 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 08:49 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/tacker/2022/tacker.2022-09-06-08.02.html | 08:49 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/tacker/2022/tacker.2022-09-06-08.02.txt | 08:49 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/tacker/2022/tacker.2022-09-06-08.02.log.html | 08:49 |
| *** Ramona-ho-xu is now known as honjinx | 08:51 | |
| *** honjinx is now known as ho-xu | 08:51 | |
| *** dasm is now known as Guest2115 | 13:31 | |
| *** Guest2115 is now known as dasm | 14:02 | |
| *** dasm is now known as dasm|off | 22:56 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!