Drone: Refactor version branch pipeline logic (#28531)

Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>

.drone.yml

@@ -875,19 +875,6 @@ steps:
   depends_on:
   - package
 
-- name: publish-storybook
-  image: grafana/grafana-ci-deploy:1.2.6
-  commands:
-  - printenv GCP_KEY | base64 -d > /tmp/gcpkey.json
-  - gcloud auth activate-service-account --key-file=/tmp/gcpkey.json
-  - gsutil -m rsync -d -r ./packages/grafana-ui/dist/storybook gs://grafana-storybook/latest
-  - gsutil -m rsync -d -r ./packages/grafana-ui/dist/storybook gs://grafana-storybook/${DRONE_TAG}
-  environment:
-    GCP_KEY:
-      from_secret: gcp_key
-  depends_on:
-  - build-storybook
-
 - name: copy-packages-for-docker
   image: grafana/build-container:1.2.28
   commands:
@@ -951,18 +938,6 @@ steps:
   - test-backend
   - test-frontend
 
-- name: release-npm-packages
-  image: grafana/build-container:1.2.28
-  commands:
-  - ./node_modules/.bin/lerna bootstrap
-  - echo "//registry.npmjs.org/:_authToken=$${NPM_TOKEN}" >> ~/.npmrc
-  - ./scripts/build/release-packages.sh ${DRONE_TAG}
-  environment:
-    NPM_TOKEN:
-      from_secret: npm_token
-  depends_on:
-  - end-to-end-tests
-
 - name: upload-packages
   image: grafana/grafana-ci-deploy:1.2.6
   commands:
@@ -984,6 +959,31 @@ steps:
   - mysql-integration-tests
   - postgres-integration-tests
 
+- name: release-npm-packages
+  image: grafana/build-container:1.2.28
+  commands:
+  - ./node_modules/.bin/lerna bootstrap
+  - echo "//registry.npmjs.org/:_authToken=$${NPM_TOKEN}" >> ~/.npmrc
+  - ./scripts/build/release-packages.sh ${DRONE_TAG}
+  environment:
+    NPM_TOKEN:
+      from_secret: npm_token
+  depends_on:
+  - end-to-end-tests
+
+- name: publish-storybook
+  image: grafana/grafana-ci-deploy:1.2.6
+  commands:
+  - printenv GCP_KEY | base64 -d > /tmp/gcpkey.json
+  - gcloud auth activate-service-account --key-file=/tmp/gcpkey.json
+  - gsutil -m rsync -d -r ./packages/grafana-ui/dist/storybook gs://grafana-storybook/latest
+  - gsutil -m rsync -d -r ./packages/grafana-ui/dist/storybook gs://grafana-storybook/${DRONE_TAG}
+  environment:
+    GCP_KEY:
+      from_secret: gcp_key
+  depends_on:
+  - build-storybook
+
 services:
 - name: postgres
   image: postgres:12.3-alpine
@@ -1644,16 +1644,6 @@ steps:
   depends_on:
   - package
 
-- name: publish-storybook
-  image: grafana/grafana-ci-deploy:1.2.6
-  commands:
-  - echo Testing release
-  environment:
-    GCP_KEY:
-      from_secret: gcp_key
-  depends_on:
-  - build-storybook
-
 - name: copy-packages-for-docker
   image: grafana/build-container:1.2.28
   commands:
@@ -1711,17 +1701,6 @@ steps:
   - test-backend
   - test-frontend
 
-- name: release-npm-packages
-  image: grafana/build-container:1.2.28
-  commands:
-  - ./node_modules/.bin/lerna bootstrap
-  - echo "//registry.npmjs.org/:_authToken=$${NPM_TOKEN}" >> ~/.npmrc
-  environment:
-    NPM_TOKEN:
-      from_secret: npm_token
-  depends_on:
-  - end-to-end-tests
-
 - name: upload-packages
   image: grafana/grafana-ci-deploy:1.2.6
   commands:
@@ -1743,6 +1722,27 @@ steps:
   - mysql-integration-tests
   - postgres-integration-tests
 
+- name: release-npm-packages
+  image: grafana/build-container:1.2.28
+  commands:
+  - ./node_modules/.bin/lerna bootstrap
+  - echo "//registry.npmjs.org/:_authToken=$${NPM_TOKEN}" >> ~/.npmrc
+  environment:
+    NPM_TOKEN:
+      from_secret: npm_token
+  depends_on:
+  - end-to-end-tests
+
+- name: publish-storybook
+  image: grafana/grafana-ci-deploy:1.2.6
+  commands:
+  - echo Testing release
+  environment:
+    GCP_KEY:
+      from_secret: gcp_key
+  depends_on:
+  - build-storybook
+
 services:
 - name: postgres
   image: postgres:12.3-alpine

scripts/release.star

@@ -56,7 +56,7 @@ def release_npm_packages_step(edition, ver_mode):
         'commands': commands,
     }
 
-def get_steps(edition, ver_mode):
+def get_steps(edition, ver_mode, publish):
     steps = [
         lint_backend_step(edition),
         codespell_step(),
@@ -70,22 +70,25 @@ def get_steps(edition, ver_mode):
         e2e_tests_server_step(),
         e2e_tests_step(),
         build_storybook_step(edition=edition, ver_mode=ver_mode),
-        publish_storybook_step(edition=edition, ver_mode=ver_mode),
         copy_packages_for_docker_step(),
-        build_docker_images_step(edition=edition, ver_mode=ver_mode, publish=True),
-        build_docker_images_step(edition=edition, ver_mode=ver_mode, ubuntu=True, publish=True),
+        build_docker_images_step(edition=edition, ver_mode=ver_mode, publish=publish),
+        build_docker_images_step(edition=edition, ver_mode=ver_mode, ubuntu=True, publish=publish),
         postgres_integration_tests_step(),
         mysql_integration_tests_step(),
-        release_npm_packages_step(edition=edition, ver_mode=ver_mode),
-        upload_packages_step(edition=edition, ver_mode=ver_mode),
     ]
+    if publish:
+        steps.extend([
+            upload_packages_step(edition=edition, ver_mode=ver_mode),
+            release_npm_packages_step(edition=edition, ver_mode=ver_mode),
+            publish_storybook_step(edition=edition, ver_mode=ver_mode),
+        ])
     windows_steps = get_windows_steps(edition=edition, ver_mode=ver_mode)
 
     return steps, windows_steps
 
-def get_oss_pipelines(trigger, ver_mode):
+def get_oss_pipelines(trigger, ver_mode, publish):
     services = integration_test_services()
-    steps, windows_steps = get_steps(edition='oss', ver_mode=ver_mode)
+    steps, windows_steps = get_steps(edition='oss', ver_mode=ver_mode, publish=publish)
     return [
         pipeline(
             name='oss-build-{}'.format(ver_mode), edition='oss', trigger=trigger, services=services, steps=steps,
@@ -97,34 +100,37 @@ def get_oss_pipelines(trigger, ver_mode):
         ),
     ]
 
-def get_enterprise_pipelines(trigger, ver_mode):
+def get_enterprise_pipelines(trigger, ver_mode, publish):
     services = integration_test_services()
-    steps, windows_steps = get_steps(edition='enterprise', ver_mode=ver_mode)
+    steps, windows_steps = get_steps(edition='enterprise', ver_mode=ver_mode, publish=publish)
     return [
         pipeline(
-            name='enterprise-build-{}'.format(ver_mode), edition='enterprise', trigger=trigger, services=services, steps=steps,
-            ver_mode=ver_mode,
+            name='enterprise-build-{}'.format(ver_mode), edition='enterprise', trigger=trigger, services=services,
+            steps=steps, ver_mode=ver_mode,
         ),
         pipeline(
-            name='enterprise-windows-{}'.format(ver_mode), edition='enterprise', trigger=trigger, steps=windows_steps, platform='windows',
-            depends_on=['enterprise-build-{}'.format(ver_mode)], ver_mode=ver_mode,
+            name='enterprise-windows-{}'.format(ver_mode), edition='enterprise', trigger=trigger, steps=windows_steps,
+            platform='windows', depends_on=['enterprise-build-{}'.format(ver_mode)], ver_mode=ver_mode,
         ),
     ]
 
-def release_pipelines():
-    ver_mode = 'release'
-
+def release_pipelines(ver_mode='release', trigger=None):
     services = integration_test_services()
+    if not trigger:
         trigger = {
             'ref': ['refs/tags/v*',],
         }
 
+    publish = ver_mode in ('release', 'test-release',)
+
     # The release pipelines include also enterprise ones, so both editions are built for a release.
     # We could also solve this by triggering a downstream build for the enterprise repo, but by including enterprise
     # in OSS release builds, we simplify the UX for the release engineer.
-    oss_pipelines = get_oss_pipelines(ver_mode=ver_mode, trigger=trigger)
-    enterprise_pipelines = get_enterprise_pipelines(ver_mode=ver_mode, trigger=trigger)
+    oss_pipelines = get_oss_pipelines(ver_mode=ver_mode, trigger=trigger, publish=publish)
+    enterprise_pipelines = get_enterprise_pipelines(ver_mode=ver_mode, trigger=trigger, publish=publish)
 
+    pipelines = oss_pipelines + enterprise_pipelines
+    if publish:
         publish_pipeline = pipeline(
             name='publish-{}'.format(ver_mode), trigger=trigger, edition='oss', steps=[
                 {
@@ -146,8 +152,7 @@ def release_pipelines():
             ], depends_on=[p['name'] for p in oss_pipelines + enterprise_pipelines], install_deps=False,
             ver_mode=ver_mode,
         )
-
-    pipelines = oss_pipelines + enterprise_pipelines + [publish_pipeline,]
+        pipelines.append(publish_pipeline)
 
     pipelines.append(notify_pipeline(
         name='notify-{}'.format(ver_mode), slack_channel='grafana-ci-notifications', trigger=trigger,
@@ -164,8 +169,8 @@ def test_release_pipelines():
         'event': ['custom',],
     }
 
-    oss_pipelines = get_oss_pipelines(ver_mode=ver_mode, trigger=trigger)
-    enterprise_pipelines = get_enterprise_pipelines(ver_mode=ver_mode, trigger=trigger)
+    oss_pipelines = get_oss_pipelines(ver_mode=ver_mode, trigger=trigger, publish=True)
+    enterprise_pipelines = get_enterprise_pipelines(ver_mode=ver_mode, trigger=trigger, publish=True)
 
     publish_cmd = './bin/grabpl publish-packages --edition {{}} --dry-run {}'.format(test_release_ver)
 

scripts/version.star

 load(
-    'scripts/lib.star',
-    'build_image',
-    'pipeline',
-    'lint_backend_step',
-    'codespell_step',
-    'shellcheck_step',
-    'test_backend_step',
-    'test_frontend_step',
-    'build_backend_step',
-    'build_frontend_step',
-    'build_plugins_step',
-    'package_step',
-    'e2e_tests_server_step',
-    'e2e_tests_step',
-    'build_storybook_step',
-    'copy_packages_for_docker_step',
-    'build_docker_images_step',
-    'postgres_integration_tests_step',
-    'mysql_integration_tests_step',
-    'get_windows_steps',
-    'benchmark_ldap_step',
-    'ldap_service',
-    'frontend_metrics_step',
-    'upload_packages_step',
-    'notify_pipeline',
-    'integration_test_services',
+    'scripts/release.star',
+    'release_pipelines',
 )
 
 ver_mode = 'version-branch'
 
-def get_steps(edition):
-    steps = [
-        lint_backend_step(edition),
-        codespell_step(),
-        shellcheck_step(),
-        test_backend_step(),
-        test_frontend_step(),
-        build_backend_step(edition=edition, ver_mode=ver_mode),
-        build_frontend_step(edition=edition, ver_mode=ver_mode),
-        build_plugins_step(edition=edition, sign=True),
-        package_step(edition=edition, ver_mode=ver_mode),
-        e2e_tests_server_step(),
-        e2e_tests_step(),
-        build_storybook_step(edition=edition, ver_mode=ver_mode),
-        copy_packages_for_docker_step(),
-        build_docker_images_step(edition=edition, ver_mode=ver_mode),
-        build_docker_images_step(edition=edition, ver_mode=ver_mode, ubuntu=True),
-        postgres_integration_tests_step(),
-        mysql_integration_tests_step(),
-    ]
-    windows_steps = get_windows_steps(edition=edition, ver_mode=ver_mode)
-
-    return steps, windows_steps
-
-def get_oss_pipelines(trigger):
-    services = integration_test_services()
-    steps, windows_steps = get_steps(edition='oss')
-    return [
-        pipeline(
-            name='oss-build-{}'.format(ver_mode), edition='oss', trigger=trigger, services=services, steps=steps,
-            ver_mode=ver_mode,
-        ),
-        pipeline(
-            name='oss-windows-{}'.format(ver_mode), edition='oss', trigger=trigger, steps=windows_steps,
-            platform='windows', depends_on=['oss-build-{}'.format(ver_mode)], ver_mode=ver_mode,
-        ),
-    ]
-
-def get_enterprise_pipelines(trigger):
-    services = integration_test_services()
-    steps, windows_steps = get_steps(edition='enterprise')
-    return [
-        pipeline(
-            name='enterprise-build-{}'.format(ver_mode), edition='enterprise', trigger=trigger, services=services,
-            steps=steps, ver_mode=ver_mode,
-        ),
-        pipeline(
-            name='enterprise-windows-{}'.format(ver_mode), edition='enterprise', trigger=trigger, steps=windows_steps,
-            platform='windows', depends_on=['enterprise-build-{}'.format(ver_mode)], ver_mode=ver_mode,
-        ),
-    ]
-
 def version_branch_pipelines():
-    services = integration_test_services()
-    trigger = {
+    return release_pipelines(ver_mode=ver_mode, trigger={
         'ref': ['refs/heads/v*',],
-    }
-
-    oss_pipelines = get_oss_pipelines(trigger=trigger)
-    enterprise_pipelines = get_enterprise_pipelines(trigger=trigger)
-
-    pipelines = oss_pipelines + enterprise_pipelines
-
-    pipelines.append(notify_pipeline(
-        name='notify-{}'.format(ver_mode), slack_channel='grafana-ci-notifications', trigger=trigger,
-        depends_on=[p['name'] for p in pipelines],
-    ))
-
-    return pipelines
+    })