Fix builder steps in lf-maven-docker-stage 72/15672/2
authorJessica Wagantall <jwagantall@linuxfoundation.org>
Tue, 14 May 2019 18:59:14 +0000 (11:59 -0700)
committerJessica Wagantall <jwagantall@linuxfoundation.org>
Tue, 14 May 2019 19:02:43 +0000 (12:02 -0700)
lf-maven-versions-plugin builder step needs to run before
maven-patch-release.sh as this second script contains a
condition to confirm if the maven vesions plugin was selected
as a way to remove the 'SNAPSHOT' pattern from the pom.xml
files.
lf-maven-docker-stage was based on lf-maven-stage and it
seems that these particular builder steps were switched in
place accidentally.

Change-Id: I30ac4af2a7566cb2c1072412a738f38d4f3318fb
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>
jjb/lf-maven-jobs.yaml
releasenotes/notes/lf-maven-docker-versions-fix-b57870964f161873.yaml [new file with mode: 0644]

index 7c47aec..aa6904b 100644 (file)
       - lf-provide-maven-settings:
           global-settings-file: '{mvn-global-settings}'
           settings-file: '{mvn-settings}'
-      - shell: !include-raw-escape: ../shell/maven-patch-release.sh
       - lf-maven-versions-plugin:
           maven-versions-plugin: '{maven-versions-plugin}'
           mvn-version: '{mvn-version}'
           mvn-pom: '{mvn-pom}'
           mvn-settings: '{mvn-settings}'
+      - shell: !include-raw-escape: ../shell/maven-patch-release.sh
       - inject:
           properties-content: |
               CONTAINER_PULL_REGISTRY={container-public-registry}
diff --git a/releasenotes/notes/lf-maven-docker-versions-fix-b57870964f161873.yaml b/releasenotes/notes/lf-maven-docker-versions-fix-b57870964f161873.yaml
new file mode 100644 (file)
index 0000000..d00cb25
--- /dev/null
@@ -0,0 +1,9 @@
+---
+fixes:
+  - |
+    lf-maven-versions-plugin builder step needs to run before maven-patch-release.sh
+    as this second script contains a condition to confirm if the maven vesions
+    plugin was selected as a way to remove the 'SNAPSHOT' pattern from the pom.xml
+    files.
+    lf-maven-docker-stage was based on lf-maven-stage and it seems that these particular
+    builder steps were switched in place accidentally.