Standardize "release" wording 38/16038/5
authorJessica Wagantall <jwagantall@linuxfoundation.org>
Fri, 28 Jun 2019 22:14:52 +0000 (15:14 -0700)
committerJessica Wagantall <jwagantall@linuxfoundation.org>
Tue, 9 Jul 2019 23:02:56 +0000 (16:02 -0700)
Using "releases" and "release" in different places
is becoming a little confusing. Standardize to "release"
to match lftools command and the majority of the exisiting
wording.
Using "releases" for the list of tech team releases does
makes sense to be kept that way, for example:
releases/1.1.1.yaml

Issue: RELENG-2131
Change-Id: I9a9a548947cc1acbe3e61bb8fa6532a89fb17e16
Signed-off-by: Jessica Wagantall <jwagantall@linuxfoundation.org>
.jjb-test/lf-release-jobs.yaml
docs/jjb/lf-macros.rst
docs/jjb/lf-release-jobs.rst
jjb/lf-macros.yaml
jjb/lf-release-job-groups.yaml
jjb/lf-release-jobs.yaml
releasenotes/notes/standardize_release_version_job-81eb03ef2c798827.yaml [new file with mode: 0644]

index 26a3dd3..73f6ec3 100644 (file)
@@ -1,6 +1,6 @@
 ---
 - project:
-    name: '{project-name}-releases'
+    name: '{project-name}-release'
     jobs:
       - '{project-name}-gerrit-release-jobs'
 
index 6772eee..32aa2be 100644 (file)
@@ -247,8 +247,8 @@ check-info-votes
 
 Calls shell script to validate votes on a change to an INFO.yaml
 
-lf-releases
------------
+lf-release
+----------
 
 releases lftools.ini (required)
 needed to push to nexus.
index 8565235..87932f8 100644 (file)
@@ -27,8 +27,8 @@ configfiles with a [nexus] section for auth.
 Macros
 ======
 
-lf-releases
------------
+lf-release
+----------
 
 Release verify and merge jobs are the same except for their scm, trigger, and
 builders definition. This anchor is the common template.
@@ -53,7 +53,7 @@ Runs:
 
 
 :Template Name:
-    - {project-name}-releases-merge-{stream}
+    - {project-name}-release-merge-{stream}
 
 :Comment Trigger: remerge
 
@@ -94,7 +94,7 @@ is available on the job.
 
 
 :Template Names:
-    - {project-name}-releases-verify-{stream}
+    - {project-name}-release-verify-{stream}
 
 :Comment Trigger: recheck|reverify
 
index 82dc840..1e697cf 100644 (file)
       - shell: !include-raw-escape: ../shell/check-info-votes.sh
 
 - builder:
-    name: lf-releases
+    name: lf-release
     builders:
       - config-file-provider:
           files:
index 41ba20a..27bdf24 100644 (file)
@@ -6,5 +6,5 @@
     # for any project using self releases.
 
     jobs:
-      - gerrit-releases-verify
-      - gerrit-releases-merge
+      - gerrit-release-verify
+      - gerrit-release-merge
index c384e5d..dc3807a 100644 (file)
@@ -11,8 +11,8 @@
 # COMMON FUNCTIONS #
 ####################
 
-- lf_releases_common: &lf_releases_common
-    name: lf-releases-common
+- lf_release_common: &lf_release_common
+    name: lf-release-common
 
     ######################
     # Default parameters #
@@ -51,8 +51,8 @@
 # RELEASE VERIFY #
 ##################
 
-- lf_releases_verify: &lf_releases_verify
-    name: lf-releases-verify
+- lf_release_verify: &lf_release_verify
+    name: lf-release-verify
 
     ######################
     # Default parameters #
               variable: SIGUL_PKI
       - shell: !include-raw-escape: ../shell/sigul-configuration.sh
       - shell: !include-raw-escape: ../shell/sigul-install.sh
-      - lf-releases
+      - lf-release
 
 - job-template:
-    name: '{project-name}-releases-verify-{stream}'
-    id: gerrit-releases-verify
-    <<: *lf_releases_common
+    name: '{project-name}-release-verify-{stream}'
+    id: gerrit-release-verify
+    <<: *lf_release_common
     # yamllint disable-line rule:key-duplicates
-    <<: *lf_releases_verify
+    <<: *lf_release_verify
 
     scm:
       - lf-infra-gerrit-scm:
 # RELEASE MERGE #
 #################
 
-- lf_releases_merge: &lf_releases_merge
-    name: lf-releases-merge
+- lf_release_merge: &lf_release_merge
+    name: lf-release-merge
 
     ######################
     # Default parameters #
               variable: SIGUL_PKI
       - shell: !include-raw-escape: ../shell/sigul-configuration.sh
       - shell: !include-raw-escape: ../shell/sigul-install.sh
-      - lf-releases
+      - lf-release
 
 - job-template:
-    name: '{project-name}-releases-merge-{stream}'
-    id: gerrit-releases-merge
-    <<: *lf_releases_common
+    name: '{project-name}-release-merge-{stream}'
+    id: gerrit-release-merge
+    <<: *lf_release_common
     # yamllint disable-line rule:key-duplicates
-    <<: *lf_releases_merge
+    <<: *lf_release_merge
 
     scm:
       - lf-infra-gerrit-scm:
diff --git a/releasenotes/notes/standardize_release_version_job-81eb03ef2c798827.yaml b/releasenotes/notes/standardize_release_version_job-81eb03ef2c798827.yaml
new file mode 100644 (file)
index 0000000..a90c096
--- /dev/null
@@ -0,0 +1,18 @@
+---
+upgrade:
+  - |
+    This change will require to update lf-release-job calls.
+    Update from using "{project-name}-releases-merge-{stream}",
+    "{project-name}-releases-verify-{stream}" to "{project-name}-release-merge-{stream}",
+    "{project-name}-release-verify-{stream}".
+    No upgrade need to be done if using "{project-name}-gerrit-release-jobs"
+    group.
+
+fixes:
+  - |
+    Using "releases" and "release" in different places is becoming confusing.
+    Standardize to "release" to match lftools command and the majority of the
+    exisiting wording.
+
+    Use "releases" for the list of tech team releases and trggers since it is
+    intuitive there. For example "releases/1.1.1.yaml"