Skip to content

Comments

Update condition for run on PR merge or Push to master#869

Open
mohitrajain wants to merge 2 commits intomasterfrom
update-github-workflow
Open

Update condition for run on PR merge or Push to master#869
mohitrajain wants to merge 2 commits intomasterfrom
update-github-workflow

Conversation

@mohitrajain
Copy link
Contributor

@mohitrajain mohitrajain commented Feb 23, 2026

Update condition for run on PR merge or Push to master

Change type

  • Fix
  • Feature
  • Documentation
  • Security / Upgrade

Basic information

  • THIS CHANGE REQUIRES A DEPLOYMENT PACKAGE RELEASE
  • THIS CHANGE REQUIRES A WIRE-DOCS RELEASE

Testing

  • 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-wiab-staging - WIAB-staging 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
  • I mentioned this PR in Jira, OR I mentioned the Jira ticket in this PR.
  • I mentioned this PR in one of the issues attached to one of our repositories.

Knowledge Transfer

  • An Asciinema session is attached to the Jira ticket.

Motivation

Objective

Reason

Use case

Update condition for run on PR merge or Push to master
@mohitrajain mohitrajain requested review from a team and julialongtin as code owners February 23, 2026 14:24
@sonarqubecloud
Copy link

(github.event_name == 'push' && github.ref == 'refs/heads/master') ||
contains(github.event.pull_request.labels.*.name, 'build-all') ||
contains(github.event.pull_request.labels.*.name, 'build-default') ||
contains(github.event.pull_request.labels.*.name, 'build-wiab-staging')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there a different in build-default and build-wiab-staging?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, infrastructure is different here. The way we are architect-ing resources for wiab-stag is just 7 VMs vs 19 VMs for default profile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants