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