Yeah, this needs to be dialed in a little tighter. There is a lot that could be
confusing. Some history:

1) there is a list of jobs that we skip in the daily AR run, like any tempest
    job, so in those got marked in the netvirt yaml with daily

2) we used to have a special high-frequency job that covered some netvirt
    jobs and ran multiple times a day. that was removed though:
    https://git.opendaylight.org/gerrit/c/releng/builder/+/81858

Srinivas, can you please dig through the definitions and remove the daily
trigger from anything that is not in the list of jobs to skip. like here:

https://github.com/opendaylight/releng-builder/blob/master/jjb/integration/integration-test-jobs.yaml#L65

If a job is in the list of jobs to skip (it's just matching on job name regex),
then it will still need a trigger like daily or weekly.

Good catch, Luis!

JamO

Guys, I just realized in this patch there is a new daily trigger:

 

schedule: '{schedule-daily}'

 

This has never been required before because the AR already runs daily and executes netvirt tests.

 

So now I see the same test job runs twice a day (because AR + this new trigger):

 

https://jenkins.opendaylight.org/releng/view/distribution/job/netvirt-csit-3node-0cmb-1ctl-2cmp-openstack-queens-upstream-stateful-magnesium/

 

Is this intended or not?

 

BR/Luis

 

On Jun 15, 2020, at 10:49 AM, Jamo Luhrsen <jluhrsen@...> wrote:

 

ok, patch is merged.

Thanks,
JamO

On 6/13/20 8:26 PM, Srinivas Rachakonda wrote:

Hi Jamo,

 

I have reviewed the scripts and it looks fine.

Marked +1 in the review.

 

R.Srinivas

+91-9243478719

 

 

 

On Sat, 13 Jun 2020 at 01:49, Jamo Luhrsen <jluhrsen@...> wrote:

Srinivas,

Can you please check this patch to see if it's in line with what you were expecting?

Thanks,
JamO

#netvirt

On 6/11/20 1:13 AM, Gerrit Code Review wrote:

Patch set 2:Code-Review +1

View Change

To view, visit change 90385. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: releng/builder

Gerrit-Branch: master

Gerrit-Change-Id: Ice7702c633e8f49b2b02f0828d130bb48f762e81

Gerrit-Change-Number: 90385

Gerrit-PatchSet: 2

Gerrit-Owner: Anil Belur <abelur@...>

Gerrit-Reviewer: Bengt Thuree <bthuree@...>

Gerrit-Reviewer: Jamo Luhrsen <jluhrsen@...>

Gerrit-Reviewer: Luis Gomez <ecelgp@...>

Gerrit-Reviewer: Srinivas Rachakonda <srinivas.rachakonda@...>

Gerrit-Reviewer: Thanh Ha (zxiiro) <zxiiro@...>

Gerrit-Reviewer: jenkins-releng <jenkins-releng@...>

Gerrit-Comment-Date: Thu, 11 Jun 2020 08:13:11 +0000

Gerrit-HasComments: No

Gerrit-Has-Labels: Yes

Gerrit-MessageType: comment