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