From: Thanh Ha (zxiiro) <thanh.ha@linuxfoundation.org>
Date: Wed, 13 Feb 2019 17:52:56 +0000 (+0000)
Subject: Merge "Update lf-maven-stage macro with missing parts"
X-Git-Tag: v0.31.0~1
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=d499fb48afd7c2e2f317018b21e0620ba681bd98;hp=9668001c567d8b17af9fff029b2e50533b09f252;p=releng%2Fglobal-jjb.git

Merge "Update lf-maven-stage macro with missing parts"
---

diff --git a/jjb/lf-macros.yaml b/jjb/lf-macros.yaml
index 08bb637c..2b6cf10f 100644
--- a/jjb/lf-macros.yaml
+++ b/jjb/lf-macros.yaml
@@ -184,11 +184,15 @@
           condition-kind: boolean-expression
           condition-expression: '{mvn-central}'
           steps:
+            - lf-provide-maven-settings:
+                global-settings-file: '{mvn-global-settings}'
+                settings-file: '{mvn-settings}'
             - lf-infra-create-netrc:
                 server-id: 'ossrh'
             - inject:
                 properties-content: 'OSSRH_PROFILE_ID={ossrh-profile-id}'
             - shell: !include-raw-escape: ../shell/maven-central.sh
+            - lf-provide-maven-settings-cleanup
 
 - builder:
     name: lf-maven-install
diff --git a/jjb/lf-maven-jobs.yaml b/jjb/lf-maven-jobs.yaml
index 46a078a0..f82d6c6a 100644
--- a/jjb/lf-maven-jobs.yaml
+++ b/jjb/lf-maven-jobs.yaml
@@ -24,10 +24,16 @@
 - builder:
     name: lf-maven-stage
     builders:
+      - lf-provide-maven-settings:
+          global-settings-file: '{mvn-global-settings}'
+          settings-file: '{mvn-settings}'
+      - lf-infra-create-netrc:
+          server-id: '{mvn-staging-id}'
       # include-raw-escape fails due to JJB bug
-      - shell: !include-raw:
+      - shell: !include-raw-escape:
           - ../shell/common-variables.sh
           - ../shell/maven-stage.sh
+      - lf-provide-maven-settings-cleanup
 
 - builder:
     name: lf-update-java-alternatives
@@ -716,17 +722,20 @@
       - lf-provide-maven-settings:
           global-settings-file: '{mvn-global-settings}'
           settings-file: '{mvn-settings}'
-      - lf-infra-create-netrc:
-          server-id: '{mvn-staging-id}'
       - shell: !include-raw-escape: ../shell/maven-patch-release.sh
       - lf-maven-build:
           mvn-goals: '{mvn-goals}'
       - lf-sigul-sign-dir:
           sign-artifacts: '{sign-artifacts}'
           sign-dir: '$WORKSPACE/m2repo'
-      - lf-maven-stage
+      - lf-maven-stage:
+          mvn-global-settings: '{mvn-global-settings}'
+          mvn-settings: '{mvn-settings}'
+          mvn-staging-id: '{mvn-staging-id}'
       - lf-maven-central:
           mvn-central: '{mvn-central}'
+          mvn-global-settings: '{mvn-global-settings}'
+          mvn-settings: '{mvn-settings}'
           ossrh-profile-id: '{ossrh-profile-id}'
       - lf-provide-maven-settings-cleanup
 
diff --git a/releasenotes/notes/refactor-5d0c339718d7aaca.yaml b/releasenotes/notes/refactor-5d0c339718d7aaca.yaml
new file mode 100644
index 00000000..2306484c
--- /dev/null
+++ b/releasenotes/notes/refactor-5d0c339718d7aaca.yaml
@@ -0,0 +1,16 @@
+---
+upgrade:
+  - |
+    Jobs using the **lf-maven-stage** macro now need to update to the new
+    usage. Preparation calls to **lf-provide-maven-settings**,
+    **lf-infra-create-netrc**, and **lf-provide-maven-settings-cleanup** are no
+    longer necessary to prepare the **lf-maven-stage** macro.
+
+    Usage:
+
+    .. code-block:: yaml
+
+       - lf-maven-stage:
+          mvn-global-settings: 'global-settings'
+          mvn-settings: 'settings'
+          mvn-staging-id: 'staging profile id'