Merge pull request #836 from wireapp/wpb-22405-git-lable

WPB-22405: Add level based approach to run CI deployment pipeline on a PR

Author: sghosh23

.github/pull_request_template.md

@@ -21,6 +21,15 @@ Fixes ${ISSUE_URL}
 * [ ] I ran/applied the changes myself, in a test environment.
 * [ ] The CI job attached to this repo will test it for me.
 
+#### Offline Build CI (label-based)
+Add one or more labels to trigger offline builds:
+- `build-default` - Full production build (ansible, terraform, all packages)
+- `build-demo` - Demo/WIAB build
+- `build-min` - Minimal build (fastest, essential charts only)
+- `build-all` - Run all three builds
+
+**Note:** No builds run by default. Add a label to trigger CI.
+
 ### Tracking
 
 * [ ] I added a new entry in an appropriate subdirectory of `changelog.d`

.github/workflows/offline.yml

@@ -6,9 +6,10 @@
 #   - min: Minimal deployment
 #
 # Build Optimization via PR Labels:
-#   - No label: Builds default + demo + min (all profiles)
-#   - 'demo-only': Builds only demo profile
-#   - 'min-only': Builds only min profile
+#   - No label: No builds run (must add label to trigger builds)
+#   - 'build-default': Builds only default profile
+#   - 'build-demo': Builds only demo profile
+#   - 'build-min': Builds only min profile
 #   - 'build-all': Explicitly builds all profiles (useful for workflow changes)
 #
 # Push to master/develop: Always builds all profiles regardless of labels
@@ -21,6 +22,7 @@ on:
       - "*.md"
       - "**/*.md"
   pull_request:
+    types: [synchronize, reopened, labeled]
     branches: [master, develop]
     paths-ignore:
       - "*.md"
@@ -30,11 +32,9 @@ jobs:
   build-default:
     name: Build default profile
     if: |
-      !contains(github.event.head_commit.message, 'skip ci') &&
-      (github.event_name == 'push' ||
-       contains(github.event.pull_request.labels.*.name, 'build-all') ||
-       (!contains(github.event.pull_request.labels.*.name, 'demo-only') &&
-        !contains(github.event.pull_request.labels.*.name, 'min-only')))
+      github.event_name == 'push' ||
+      contains(github.event.pull_request.labels.*.name, 'build-all') ||
+      contains(github.event.pull_request.labels.*.name, 'build-default')
     runs-on:
       group: wire-server-deploy
     outputs:
@@ -118,11 +118,9 @@ jobs:
   build-demo:
     name: Build demo profile
     if: |
-      !contains(github.event.head_commit.message, 'skip ci') &&
-      (github.event_name == 'push' ||
-       contains(github.event.pull_request.labels.*.name, 'build-all') ||
-       contains(github.event.pull_request.labels.*.name, 'demo-only') ||
-       !contains(github.event.pull_request.labels.*.name, 'min-only'))
+      github.event_name == 'push' ||
+      contains(github.event.pull_request.labels.*.name, 'build-all') ||
+      contains(github.event.pull_request.labels.*.name, 'build-demo')
     runs-on:
       group: wire-server-deploy
     steps:
@@ -181,11 +179,9 @@ jobs:
   build-min:
     name: Build min profile
     if: |
-      !contains(github.event.head_commit.message, 'skip ci') &&
-      (github.event_name == 'push' ||
-       contains(github.event.pull_request.labels.*.name, 'build-all') ||
-       contains(github.event.pull_request.labels.*.name, 'min-only') ||
-       !contains(github.event.pull_request.labels.*.name, 'demo-only'))
+      github.event_name == 'push' ||
+      contains(github.event.pull_request.labels.*.name, 'build-all') ||
+      contains(github.event.pull_request.labels.*.name, 'build-min')
     runs-on:
       group: wire-server-deploy
     steps:

changelog.d/3-deploy-builds/wpb-22405-git-lable

@@ -0,0 +1 @@
+Changed: offline workflow to require explicit labels for PR builds (build-default, build-demo, build-min, build-all); PRs without labels no longer trigger builds