[Geoserver-devel] Removed Git polling doc exclusion on geoserver-master Jenkins job

GeoServer GeoServer Developer
1

Jody,

I removed the doc exclusion for Git polling on the geoserver-master Jenkins job. Something appears to be horribly wrong with it. The symptoms are that polling *always* detects changes, so there is always a geoserver-master in the queue. Hopefully this will stop the flood of unnecessary builds.

Kind regards,

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

2

Looks like this worked.

On 28/07/16 15:49, Ben Caradoc-Davies wrote:

Jody,

I removed the doc exclusion for Git polling on the geoserver-master
Jenkins job. Something appears to be horribly wrong with it. The
symptoms are that polling *always* detects changes, so there is always a
geoserver-master in the queue. Hopefully this will stop the flood of
unnecessary builds.

Kind regards,

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

3

I have removed the doc and geogig Git polling exclusions on geoserver-master Jenkins. As discussed last year (see below), Git polling exclusions are broken: they cause continual retriggering of the geoserver-master build, waste resources, and delay other builds. Please do not add Git polling exclusions on any Jenkins build.

Kind regards,
Ben.

-------- Forwarded Message --------
Subject: [Geoserver-devel] Removed Git polling doc exclusion on geoserver-master Jenkins job
Date: Thu, 28 Jul 2016 15:49:56 +1200
From: Ben Caradoc-Davies <ben@anonymised.com>
To: Jody Garnett <jody.garnett@anonymised.com>
CC: geoserver-devel@lists.sourceforge.net

Jody,

I removed the doc exclusion for Git polling on the geoserver-master Jenkins job. Something appears to be horribly wrong with it. The symptoms are that polling *always* detects changes, so there is always a geoserver-master in the queue. Hopefully this will stop the flood of unnecessary builds.

Kind regards,

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

4

Ben,

Thanks for the info, I wasn’t aware of the issues.

It seems the exclusions are still in place for the other builds (2.11.x, 2.10.x and 2.9.x). I can remove them on those builds too, unless it is desired to keep them.

···

On Sun, Apr 9, 2017 at 7:56 PM, Ben Caradoc-Davies <ben@anonymised.com> wrote:

I have removed the doc and geogig Git polling exclusions on geoserver-master Jenkins. As discussed last year (see below), Git polling exclusions are broken: they cause continual retriggering of the geoserver-master build, waste resources, and delay other builds. Please do not add Git polling exclusions on any Jenkins build.

Kind regards,
Ben.

-------- Forwarded Message --------
Subject: [Geoserver-devel] Removed Git polling doc exclusion on geoserver-master Jenkins job
Date: Thu, 28 Jul 2016 15:49:56 +1200
From: Ben Caradoc-Davies <ben@anonymised.com>
To: Jody Garnett <jody.garnett@anonymised.com>
CC: geoserver-devel@anonymised.comrge.net

Jody,

I removed the doc exclusion for Git polling on the geoserver-master Jenkins job. Something appears to be horribly wrong with it. The symptoms are that polling always detects changes, so there is always a geoserver-master in the queue. Hopefully this will stop the flood of unnecessary builds.

Kind regards,

–
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/>
New Zealand

Erik Merkle
Software Engineer | Boundless

5

Is this something fixed with newer versions of Jenkins?

···

On 9 April 2017 at 17:56, Ben Caradoc-Davies <ben@anonymised.com> wrote:

I have removed the doc and geogig Git polling exclusions on geoserver-master Jenkins. As discussed last year (see below), Git polling exclusions are broken: they cause continual retriggering of the geoserver-master build, waste resources, and delay other builds. Please do not add Git polling exclusions on any Jenkins build.

Kind regards,
Ben.

-------- Forwarded Message --------
Subject: [Geoserver-devel] Removed Git polling doc exclusion on geoserver-master Jenkins job
Date: Thu, 28 Jul 2016 15:49:56 +1200
From: Ben Caradoc-Davies <ben@anonymised.com>
To: Jody Garnett <jody.garnett@anonymised.com>
CC: geoserver-devel@anonymised.comrge.net

Jody,

I removed the doc exclusion for Git polling on the geoserver-master Jenkins job. Something appears to be horribly wrong with it. The symptoms are that polling always detects changes, so there is always a geoserver-master in the queue. Hopefully this will stop the flood of unnecessary builds.

Kind regards,

–
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/>
New Zealand

–
Jody Garnett

6

Erik,

thanks for pointing that out. I have removed the exclusions for 2.11.x and 2.10.x, even though they have not yet caused trouble. This gives us consistency between the builds.

I do not know the precise circumstances that cause a build flood, but it seems that removing the exclusions stops it once started. This worked both last time and this time. There are many bugs filed against Jenkins git-plugin. I did not find an exact match for this situation (ignores and continuous retriggering):
https://issues.jenkins-ci.org/browse/JENKINS-6658?jql=project%20%3D%20JENKINS%20AND%20component%20%3D%20git-plugin%20AND%20text%20~%20"ignore"

Also, I see no reason to exclude docs and geogig as developers may be building them locally. Leaving them included will allow Jenkins to detect breakage.

Kind regards,
Ben.

On 11/04/17 03:38, Erik Merkle wrote:

Ben,

Thanks for the info, I wasn't aware of the issues.

It seems the exclusions are still in place for the other builds (2.11.x,
2.10.x and 2.9.x). I can remove them on those builds too, unless it is
desired to keep them.

Erik Merkle
Software Engineer | Boundless <http://boundlessgeo.com/&gt;

<http://d32wfbeasdaw38.cloudfront.net/img/Boundless_Logo.png&gt;

On Sun, Apr 9, 2017 at 7:56 PM, Ben Caradoc-Davies <ben@anonymised.com> wrote:

I have removed the doc and geogig Git polling exclusions on
geoserver-master Jenkins. As discussed last year (see below), Git polling
exclusions are broken: they cause continual retriggering of the
geoserver-master build, waste resources, and delay other builds. Please do
not add Git polling exclusions on any Jenkins build.

Kind regards,
Ben.

-------- Forwarded Message --------
Subject: [Geoserver-devel] Removed Git polling doc exclusion on
geoserver-master Jenkins job
Date: Thu, 28 Jul 2016 15:49:56 +1200
From: Ben Caradoc-Davies <ben@anonymised.com>
To: Jody Garnett <jody.garnett@anonymised.com>
CC: geoserver-devel@lists.sourceforge.net

Jody,

I removed the doc exclusion for Git polling on the geoserver-master
Jenkins job. Something appears to be horribly wrong with it. The symptoms
are that polling *always* detects changes, so there is always a
geoserver-master in the queue. Hopefully this will stop the flood of
unnecessary builds.

Kind regards,

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

7

Unknown. I have not found a bug report that matches our situation. As I mentioned in my previous email, I do not see why we are excluding commits to docs and geogig in the first place. Shall we just leave them in?

Kind regards,
Ben.

On 11/04/17 08:53, Jody Garnett wrote:

Is this something fixed with newer versions of Jenkins?

--
Jody Garnett

On 9 April 2017 at 17:56, Ben Caradoc-Davies <ben@anonymised.com> wrote:

I have removed the doc and geogig Git polling exclusions on
geoserver-master Jenkins. As discussed last year (see below), Git polling
exclusions are broken: they cause continual retriggering of the
geoserver-master build, waste resources, and delay other builds. Please do
not add Git polling exclusions on any Jenkins build.

Kind regards,
Ben.

-------- Forwarded Message --------
Subject: [Geoserver-devel] Removed Git polling doc exclusion on
geoserver-master Jenkins job
Date: Thu, 28 Jul 2016 15:49:56 +1200
From: Ben Caradoc-Davies <ben@anonymised.com>
To: Jody Garnett <jody.garnett@anonymised.com>
CC: geoserver-devel@lists.sourceforge.net

Jody,

I removed the doc exclusion for Git polling on the geoserver-master
Jenkins job. Something appears to be horribly wrong with it. The symptoms
are that polling *always* detects changes, so there is always a
geoserver-master in the queue. Hopefully this will stop the flood of
unnecessary builds.

Kind regards,

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

8

I removed the exclusions on 2.9.x as well, even though it is unsupported.

On 11/04/17 11:07, Ben Caradoc-Davies wrote:

I have removed the exclusions for 2.11.x
and 2.10.x, even though they have not yet caused trouble. This gives us
consistency between the builds.

--
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/&gt;
New Zealand

9

For docs at least, there was an effort to separate the doc build and the code build so if a doc change was made we would only build the docs, and if a code change was made we would only build the code. Consequently, the docs are excluded from the polling for the main builds.

See the *-live-docs builds for the doc portion of this.

Torben

···

On Mon, Apr 10, 2017 at 4:15 PM, Ben Caradoc-Davies <ben@anonymised.com…4191…> wrote:

I removed the exclusions on 2.9.x as well, even though it is unsupported.

On 11/04/17 11:07, Ben Caradoc-Davies wrote:

I have removed the exclusions for 2.11.x
and 2.10.x, even though they have not yet caused trouble. This gives us
consistency between the builds.

–
Ben Caradoc-Davies <ben@anonymised.com>
Director
Transient Software Limited <http://transient.nz/>
New Zealand

Check out the vibrant tech community on one of the world’s most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot

Geoserver-devel mailing list
Geoserver-devel@anonymised.com.366…sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel