*** abhishekk is now known as akekane|home | 05:12 | |
*** akekane|home is now known as abhishekk | 05:12 | |
*** h_asahin1 is now known as h_asahina | 08:01 | |
ueha | Hi, tacker team. | 08:02 |
---|---|---|
h_asahina | hi | 08:02 |
ueha | Does anyone know if yasufum and tsc-takahashi will participate? | 08:02 |
masazumi_ota | hi | 08:02 |
masaki-ueno | hi | 08:02 |
manpreetk | hi | 08:02 |
yasufum | hi team | 08:03 |
ueha | hi | 08:03 |
yasufum | #startmeeting tacker | 08:04 |
opendevmeet | Meeting started Tue Nov 16 08:04:02 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:04 |
opendevmeet | The meeting name has been set to 'tacker' | 08:04 |
esto-aln | Hello Everyone, Takahashi-san is in another meeting and cannot participate today. | 08:04 |
yasufum | OK, thanks. | 08:06 |
yasufum | There are three topics on etherpad today. | 08:06 |
yasufum | #link https://etherpad.opendev.org/p/tacker-meeting | 08:06 |
yasufum | I am not sure who is the owner of the first item. | 08:07 |
h_asahina | Sorry, it's me. | 08:08 |
h_asahina | I forgot to write my name. | 08:08 |
yasufum | and it is helpful to change the color of hightlight. | 08:08 |
yasufum | Anyway, could you start your topic? | 08:09 |
h_asahina | ok. | 08:09 |
h_asahina | As I wrote here, I recently found the bug that VNF Instantiaion with Legacy API fails when heat tries to bind `net_mgmt` port to a server. | 08:10 |
h_asahina | First of all, I'd like to confirm does anyone have the same problem | 08:11 |
h_asahina | If nobody face this problem, I'd like to ask you whether we should fix this soon or not. | 08:13 |
h_asahina | As it's a legacy API, I guess there aren't so many people using it. | 08:14 |
yasufum | Have you confirmed the error is caused in takcer, or underlying component, might be heat or neutron? | 08:16 |
h_asahina | I think the problem happens between Nova and Neutron | 08:17 |
yasufum | Is it neutron considering the error log? | 08:17 |
yasufum | OK, i see. | 08:17 |
h_asahina | yes | 08:17 |
h_asahina | I've confirmed it by manually runnning `openstack server ...` command | 08:18 |
h_asahina | So, it might be a matter of the OpenStack deployment, and not Tacker's bug. | 08:19 |
h_asahina | but, at least, the environment using local.conf in the tacker repository incurs this problem in my environment. | 08:20 |
yasufum | matter of the OpenStack deployment > I think so... | 08:22 |
yasufum | Anyway, I will try the same on my env. | 08:23 |
yasufum | so that we will discuss later. | 08:24 |
h_asahina | sounds good. thanks | 08:24 |
yasufum | Any other comment, or go to the next topic? | 08:24 |
ueha | Do you mean that it can happen with SOL compliant APIs with similar settings? | 08:25 |
h_asahina | No. This problem doesn't happen when using SOL complicant APIs. | 08:26 |
h_asahina | though I'm not sure the reason why... | 08:27 |
ueha | Okay I see, thanks. | 08:27 |
h_asahina | thanks | 08:28 |
yasufum | Thanks for your comment. | 08:29 |
yasufum | The second topic is mine. | 08:31 |
yasufum | We have found that fenix plugin is not activated in the examples of local.conf with k8s | 08:31 |
yasufum | while the error of previous fenix was fixed by ueha. | 08:32 |
yasufum | Fenix is supported in tacker officially, so it should be included in the example basically. | 08:33 |
yasufum | I would like to hear your comment if you opposite this change. | 08:33 |
yasufum | Do you think there is any problem if fenix is included in k8s example? | 08:34 |
ueha | I thinks it should be included and it may be no problem. | 08:36 |
yasufum | OK, thanks. I will do that. | 08:36 |
yasufum | any other comment? | 08:37 |
yasufum | good | 08:38 |
yasufum | go to the final topic | 08:38 |
yasufum | ueha: can you share your topic? | 08:39 |
ueha | Sure, this is about the CI error we discussed last week. | 08:39 |
ueha | The FT error that I reported last week has been resolved in patch [1], but there is another error in gate job.. | 08:40 |
ueha | [1] https://review.opendev.org/c/openstack/tacker/+/817183 | 08:40 |
ueha | Failure in "tox -e docs" that showing following error: "undefined label: types_typedecorator" | 08:41 |
ueha | This error caused by updating SQLAlchemy version to 1.4.27 on Nov 11. | 08:41 |
ueha | This error of tox-docs job seems to be problem between SQLAlchemy and Sphinx, so I think there are following resolutions: | 08:42 |
ueha | 1. Error point is db/types.py. Therefore this error can be fixed by excluding db/types.py from sphinx-apidoc. and I posted the patch [2] and test is passed. | 08:43 |
ueha | [2] https://review.opendev.org/c/openstack/tacker/+/817953 | 08:43 |
ueha | 2. Exclude "SQLAlchemy 1.4.27" from tacker's requirements and openstack/requrements. | 08:44 |
ueha | Since the gate of many patches is stopped, first of all, I think that 1. should be implemented as a workaround in tacker. | 08:45 |
ueha | Do you have any other good idea or comment? | 08:45 |
ueha | If there is no problem with solution 1 in particular, I would like you to review patch [2]. | 08:47 |
yasufum | Is something dropped from API docs if you add 'db/types.py' in apidoc_excluded_paths? | 08:47 |
yasufum | for [2] | 08:48 |
ueha | I think it dropped the tacker.db.types pages of documentation -> https://docs.openstack.org/tacker/latest/contributor/api/tacker.db.types.html | 08:49 |
yasufum | My another concern is there is an possibility the same problem is happened in other projects although nothing on any projects from my short survey. | 08:53 |
yasufum | [2] looks good to me as an temporary and local fix for tacker. | 08:54 |
ueha | sure | 08:54 |
ueha | I think it should be fixed with Sphinx or SQLAlchemy. | 08:55 |
yasufum | But I also think it does not take times so much to fix global requirements in ‘requirements’ project. | 08:55 |
yasufum | If the team agree to do so. | 08:56 |
yasufum | It’s just adding the excluded version. | 08:56 |
yasufum | Although I am still not sure it is ideal way for fixing. | 08:57 |
h_asahina | I'm concerned that the problem happens again in the future when the SQLAlchemy is updated. | 08:58 |
yasufum | It can be avoided by using lower thant 1.4.27, but we cannot use any other updates in the later versions… | 09:00 |
ueha | Yes I'm concerned same things. If it isn't fixed in SQLAlchemy or Sphinx, we are going to keep updating requirements. | 09:00 |
ueha | and as yasufum says, we can't use the latest version SQLAlchemy... uum.. | 09:02 |
h_asahina | I guess we might have to contact with SQLAlchemy or Sphinx community to deal with this problem. Is it possible? | 09:03 |
yasufum | I think you can do that, but it is something difficult to ask to stop using `:ref:` attribute. | 09:05 |
yasufum | Anyway, in my opinion, [2] is acceptable although we will lost a API doc for `tacker.db.types` | 09:07 |
yasufum | because our top priority is to recover the gate test. | 09:07 |
yasufum | as a temporary solution. | 09:08 |
h_asahina | I agree | 09:08 |
ueha | I think so. thanks | 09:08 |
yasufum | thanks | 09:10 |
yasufum | please continue to discuss for the details on gerrit review if you have any comment | 09:11 |
yasufum | It is the end of the time for today’s meeting. | 09:13 |
yasufum | So, I would like to wrap up this meeting if you have no more comemnts. | 09:13 |
ueha | Sorry, I have one thing. | 09:14 |
yasufum | ok | 09:14 |
ueha | next week (Nov 23), Japan is national holiday. | 09:14 |
ueha | so, is next weekly meeting cancel? | 09:15 |
yasufum | Considering not so many people joining from other than | 09:17 |
yasufum | japan, it seems OK | 09:17 |
ueha | I see, thanks | 09:17 |
yasufum | I would like to make an announcement for that because of “my absence”. | 09:18 |
yasufum | and ask someone to host the meeting if anyone want to have a meeting at the day. | 09:19 |
yasufum | thanks for your good suggestion! | 09:21 |
ueha | :) | 09:21 |
yasufum | Is there any other comment? | 09:21 |
yasufum | good | 09:22 |
yasufum | Let’s close this meeting. | 09:22 |
yasufum | Bye! | 09:22 |
ueha | thanks, bye! | 09:22 |
masaki-ueno | bye | 09:22 |
manpreetk | bye | 09:22 |
esto-aln | bye | 09:22 |
h_asahina | bye | 09:22 |
yasufum | #endmeeting | 09:22 |
opendevmeet | Meeting ended Tue Nov 16 09:22:56 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:22 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/tacker/2021/tacker.2021-11-16-08.04.html | 09:22 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/tacker/2021/tacker.2021-11-16-08.04.txt | 09:22 |
opendevmeet | Log: https://meetings.opendev.org/meetings/tacker/2021/tacker.2021-11-16-08.04.log.html | 09:22 |
*** soniya29|ruck is now known as soniya29|ruck|afk | 11:54 | |
*** soniya29|ruck|afk is now known as soniya29|ruck | 13:40 | |
*** akekane_ is now known as abhishekk | 16:43 | |
*** tosky is now known as Guest6054 | 18:05 | |
*** tosky_ is now known as tosky | 18:05 | |
*** tosky_ is now known as tosky | 18:48 | |
*** tosky is now known as Guest6070 | 22:42 | |
*** tosky_ is now known as tosky | 22:42 | |
*** tosky is now known as Guest6073 | 23:07 | |
*** tosky_ is now known as tosky | 23:07 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!