| yasufum-o | Hi, tacker team. | 08:00 |
|---|---|---|
| ueha | hi | 08:00 |
| yasufum-o | #startmeeting tacker | 08:01 |
| opendevmeet | Meeting started Tue Sep 5 08:01:41 2023 UTC and is due to finish in 60 minutes. The chair is yasufum-o. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:01 |
| opendevmeet | The meeting name has been set to 'tacker' | 08:01 |
| yasufum-o | #link https://etherpad.opendev.org/p/tacker-meeting | 08:02 |
| yasufum-o | So, two items on the etherpad today. | 08:03 |
| yasufum-o | #topic Sharing about current Zuul Centos jobs error | 08:04 |
| yasufum-o | Let's start from the first item. | 08:04 |
| yasufum-o | Renu: Are you joining? | 08:05 |
| takahashi-tsc | Oops, I have confirmed with Renu and she is joining incorrect channel. | 08:07 |
| takahashi-tsc | Sorry but please wait a minutes... | 08:07 |
| yasufum-o | ok | 08:07 |
| renu | Hello everyone | 08:08 |
| yasufum-o | hi | 08:08 |
| renu | This is for the topic "Sharing about current Zuul Centos jobs error (Renu, 2023/09/05 IRC)" | 08:08 |
| renu | We observed an issue realted ensure-java role for our zuul build job | 08:09 |
| renu | Bug https://bugs.launchpad.net/tacker/+bug/2033755 has been opened for the same. | 08:11 |
| renu | we would like to know Is there a problem with our policy? | 08:15 |
| renu | please share your opinion | 08:16 |
| yasufum-o | thanks | 08:18 |
| yasufum-o | Is it just a problem in the playbook and not in tacker itself? | 08:19 |
| yasufum-o | #link https://opendev.org/zuul/zuul-jobs/src/commit/be0415e556aa183dc425a386cb07e0f681f5ca56/roles/ensure-java | 08:19 |
| renu | yes .. ensure-java is a role | 08:21 |
| renu | need to fix in playbook of zuul job | 08:21 |
| yasufum-o | Although I'm not sure the exact behavior of "package:" directive, is there any other alternative for centos env? | 08:22 |
| renu | The fix required in "https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/ensure-java" | 08:22 |
| yasufum-o | yeah, I think it might be so. | 08:22 |
| ueha | renu: I have a question and suggestion. | 08:24 |
| ueha | If the "ensure-java" role is required only by the compliance test job, then there is a way to make it run only in the compliance test job. | 08:24 |
| ueha | If "ensure-java" role isn't really need for other jobs, can't you do it that way? | 08:25 |
| renu | I think need to open bug in zuul-jobs and in parallel fix to install java in tox of compliance test | 08:25 |
| renu | Ueha san , we agree with your point. | 08:26 |
| ueha | however having said that, I think the "ensure-java" role should be fixed for the future. | 08:29 |
| ueha | But if you don't use it in our test by my suggestion, I think you can give it a lower priority. | 08:30 |
| renu | Ueha san , currently this is part of tacker/playbooks/devstack/pre.yaml which is first step for environment preparation. | 08:31 |
| renu | will it be ok to comment it in tacker/playbooks/devstack/pre.yaml and fix in tox of compliance. | 08:31 |
| renu | In parallel will open an issue in zuul | 08:32 |
| ueha | sorry, does `in tox of complience` mean the "in another patch (fixing for complience patch)"? | 08:33 |
| ueha | like https://review.opendev.org/c/openstack/tacker/+/888757 ? | 08:33 |
| ueha | or new patch? | 08:34 |
| renu | Fix in patch https://review.opendev.org/c/openstack/tacker/+/852125 , itself | 08:36 |
| ueha | Thanks. Yeah, I think it is okay to fix in CentOS testing patch. | 08:37 |
| renu | In parallel will open bug in zuul and accordingly update tacker open bug https://bugs.launchpad.net/tacker/+bug/2033755 | 08:37 |
| renu | Thanks Ueha san for your input. | 08:38 |
| ueha | You're welcome. | 08:39 |
| ueha | yasufum-o: Is that policy okay? If you have any comments, please let us know. | 08:40 |
| yasufum-o | Can I confirm how to fix the issue? | 08:41 |
| yasufum-o | Just dropping the task "ensure-java"? | 08:41 |
| yasufum-o | for a while? | 08:42 |
| ueha | I think "dropping the ensure-java role" is not correct. | 08:43 |
| ueha | "Fix to run "ensure-java" role when only complience test job" is correct. | 08:44 |
| ueha | by adding env value of centos jobs to .zuul.yaml, and add a condition to be run "ensure-java" role. | 08:45 |
| ueha | https://review.opendev.org/c/openstack/tacker/+/852125/22/playbooks/devstack/pre.yaml#24 | 08:46 |
| ueha | renu: Please let me know if my understanding is wrong. | 08:47 |
| renu | yes , correct | 08:47 |
| renu | will make an environment variable if centos env then ensure-java role will not run . The java will be installed from compliance test in tox.ini . | 08:48 |
| yasufum-o | OK, making it conditional, agree. | 08:48 |
| yasufum-o | Any other comment, or go to the next topic? | 08:50 |
| renu | only one confirmation can we do fix in patch https://review.opendev.org/c/openstack/tacker/+/852125 itself | 08:50 |
| yasufum-o | LGTM | 08:51 |
| renu | ok thanks | 08:51 |
| yasufum-o | So, second topic is just a request | 08:53 |
| yasufum-o | RC1 target week is Sep 11-15 | 08:54 |
| yasufum-o | There are several patches are still remained, | 08:55 |
| yasufum-o | Let's us know if any other patches should be fixed soon | 08:57 |
| yasufum-o | Anyway, many thanks for your help for reviewing. | 08:58 |
| yasufum-o | OK, it seems enough for today, but is there any other comment? | 09:00 |
| yasufum-o | Or close this meeting? | 09:00 |
| ueha | takahashi-tsc: There are some patches that I have posted and are ready for review, so please kindly review them. | 09:01 |
| takahashi-tsc | ueha: Sire, sorry for the late... | 09:01 |
| ueha | No problem, thank you :) | 09:02 |
| yasufum-o | thx | 09:02 |
| ueha | That's all from myside, thank you. | 09:02 |
| yasufum-o | good | 09:02 |
| yasufum-o | Let's close the meeting. | 09:02 |
| yasufum-o | Thanks for joining, bye! | 09:03 |
| ueha | thanks, bye | 09:03 |
| yasufum-o | #endmeeting | 09:03 |
| opendevmeet | Meeting ended Tue Sep 5 09:03:38 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:03 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/tacker/2023/tacker.2023-09-05-08.01.html | 09:03 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/tacker/2023/tacker.2023-09-05-08.01.txt | 09:03 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/tacker/2023/tacker.2023-09-05-08.01.log.html | 09:03 |
| *** dasm is now known as Guest1929 | 14:57 | |
| *** Guest1929 is now known as dasm | 15:02 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!