Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-39630

ATH should run against latest LTS versions of Jenkins

    XMLWordPrintable

    Details

    • Similar Issues:
    • Sprint:
      Blue Ocean 1.1

      Description

      In Scope

      • For every day we should be running the ATH against all supported LTS versions.

        Attachments

          Activity

          Hide
          jamesdumay James Dumay added a comment -

          Ivan Meredith can you pow-wow with Michael Neale before picking this one up?

          Show
          jamesdumay James Dumay added a comment - Ivan Meredith can you pow-wow with Michael Neale before picking this one up?
          Hide
          michaelneale Michael Neale added a comment -

          Yes as discussed yesterday, we don't have the physical resources to test permiutations of this.

           

          We could have weekly jobs to test things, possibly, parametrised? but this is out of our reach right now. The alternaitve is to not test against an older base but a recent LTS (but develop against older one) to get protection of both. 

          Show
          michaelneale Michael Neale added a comment - Yes as discussed yesterday, we don't have the physical resources to test permiutations of this.   We could have weekly jobs to test things, possibly, parametrised? but this is out of our reach right now. The alternaitve is to not test against an older base but a recent LTS (but develop against older one) to get protection of both. 
          Hide
          jamesdumay James Dumay added a comment -

          Michael Neale and I discussed an approach yesterday. Best catch with him Ivan Meredith

          Show
          jamesdumay James Dumay added a comment - Michael Neale and I discussed an approach yesterday. Best catch with him Ivan Meredith
          Hide
          imeredith Ivan Meredith added a comment -

          I've created a job called blueocean-weekly-ath. It is currently building just one branch, and it should test against 2.7.3 and 2.46.3. Assuming that goes well I will make it work on all major LTS versions.

          Note I'm not using parrallel nodes to build these, to save on those limited reources. But ATH itself only takes 20ishmins per run I believe.

          Show
          imeredith Ivan Meredith added a comment - I've created a job called blueocean-weekly-ath. It is currently building just one branch, and it should test against 2.7.3 and 2.46.3. Assuming that goes well I will make it work on all major LTS versions. Note I'm not using parrallel nodes to build these, to save on those limited reources. But ATH itself only takes 20ishmins per run I believe.
          Hide
          michaelneale Michael Neale added a comment -

          This needs something like: https://github.com/jenkinsci/blueocean-plugin/pull/1098 to work

           

          as for reasons we don't yet understand, LTS seems to cause things to load up incorrectly regarding this transitive dependency, and thus blue ocean won't load (this may warrant it's own invesitgation... hard to say). 

          Show
          michaelneale Michael Neale added a comment - This needs something like: https://github.com/jenkinsci/blueocean-plugin/pull/1098  to work   as for reasons we don't yet understand, LTS seems to cause things to load up incorrectly regarding this transitive dependency, and thus blue ocean won't load (this may warrant it's own invesitgation... hard to say). 
          Hide
          vivek Vivek Pandey added a comment -

          Ivan Meredith Michael Neale as part of investigation in https://issues.jenkins-ci.org/browse/JENKINS-44533. This diff makes ATH work with 2.46.3 LTS:

          diff --git a/acceptance-tests/runner/runtime-plugins/runtime-deps/pom.xml b/acceptance-tests/runner/runtime-plugins/runtime-deps/pom.xml
          index bf9fe526..9f606c4d 100644
          --- a/acceptance-tests/runner/runtime-plugins/runtime-deps/pom.xml
          +++ b/acceptance-tests/runner/runtime-plugins/runtime-deps/pom.xml
          @@ -64,6 +64,11 @@
                       <artifactId>fake-realm</artifactId>
                       <version>1.0-SNAPSHOT</version>
                   </dependency>
          +        <dependency>
          +            <groupId>org.jenkins-ci.plugins</groupId>
          +            <artifactId>plain-credentials</artifactId>
          +            <version>1.3</version>
          +        </dependency>
               </dependencies>
               <profiles>
                   <profile>
          diff --git a/acceptance-tests/runner/scripts/maven.sh b/acceptance-tests/runner/scripts/maven.sh
          index 98593889..e2fbd78a 100644
          --- a/acceptance-tests/runner/scripts/maven.sh
          +++ b/acceptance-tests/runner/scripts/maven.sh
          @@ -25,9 +25,10 @@ assemble-plugins() {
               pushd runner/runtime-plugins
               mvn clean install -B -DskipTests -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn
               pushd runtime-deps
          +    mkdir -p target/plugins
          +    cp -f $AGGREGATOR_DIR/target/plugins/*.hpi target/plugins
               mvn hpi:assemble-dependencies -B -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn
               popd
               popd
          -    cp -f $AGGREGATOR_DIR/target/plugins/*.hpi ./runner/runtime-plugins/runtime-deps/target/plugins
          }

          It also fixes a bug where we let runtime-deps overwrite plugin versions brought in from blueocean-aggregator, I think that was intent.

           

          Show
          vivek Vivek Pandey added a comment - Ivan Meredith Michael Neale  as part of investigation in https://issues.jenkins-ci.org/browse/JENKINS-44533 . This diff makes ATH work with 2.46.3 LTS: diff --git a/acceptance-tests/runner/runtime-plugins/runtime-deps/pom.xml b/acceptance-tests/runner/runtime-plugins/runtime-deps/pom.xml index bf9fe526..9f606c4d 100644 --- a/acceptance-tests/runner/runtime-plugins/runtime-deps/pom.xml +++ b/acceptance-tests/runner/runtime-plugins/runtime-deps/pom.xml @@ -64,6 +64,11 @@              <artifactId>fake-realm</artifactId>              <version>1.0-SNAPSHOT</version>          </dependency> +        <dependency> +            <groupId>org.jenkins-ci.plugins</groupId> +            <artifactId>plain-credentials</artifactId> +            <version>1.3</version> +        </dependency>      </dependencies>      <profiles>          <profile> diff --git a/acceptance-tests/runner/scripts/maven.sh b/acceptance-tests/runner/scripts/maven.sh index 98593889..e2fbd78a 100644 --- a/acceptance-tests/runner/scripts/maven.sh +++ b/acceptance-tests/runner/scripts/maven.sh @@ -25,9 +25,10 @@ assemble-plugins() {      pushd runner/runtime-plugins      mvn clean install -B -DskipTests -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn      pushd runtime-deps +    mkdir -p target/plugins +    cp -f $AGGREGATOR_DIR/target/plugins/*.hpi target/plugins      mvn hpi:assemble-dependencies -B -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn      popd      popd -    cp -f $AGGREGATOR_DIR/target/plugins/*.hpi ./runner/runtime-plugins/runtime-deps/target/plugins } It also fixes a bug where we let runtime-deps overwrite plugin versions brought in from blueocean-aggregator, I think that was intent.  
          Hide
          michaelneale Michael Neale added a comment -

          AS suggested on https://issues.jenkins-ci.org/browse/JENKINS-44533 - perhaps reordering the deps is the way to go. 

          Show
          michaelneale Michael Neale added a comment - AS suggested on https://issues.jenkins-ci.org/browse/JENKINS-44533  - perhaps reordering the deps is the way to go. 

            People

            Assignee:
            imeredith Ivan Meredith
            Reporter:
            jamesdumay James Dumay
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: