Skip to content

Commit b676588

Browse files
authored
Update condition for run on PR merge or Push to master (#869)
* Update condition for run on PR merge or Push to master Update condition for run on PR merge or Push to master * fix: wpb-24362 update wiab-stag artifact hash, fixed offline.yml workflow and updated wiab-staging docs
1 parent 95636c0 commit b676588

File tree

4 files changed

+21
-6
lines changed

4 files changed

+21
-6
lines changed

.github/workflows/offline.yml

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,14 @@
22
#
33
# This workflow builds offline deployment artifacts for different profiles:
44
# - default: Production deployment (includes external charts, ansible, terraform)
5-
# - demo: Demo/WIAB deployment (includes databases-ephemeral)
5+
# - build-wiab-staging: Wire-in-a-box (wiab-stag) a production like deployment (includes external charts, ansible, terraform)
6+
# - wiab-dev: Wire-in-a-box dev deployment (includes databases-ephemeral)
67
# - min: Minimal deployment
78
#
89
# Build Optimization via PR Labels:
910
# - No label: No builds run (must add label to trigger builds)
1011
# - 'build-default': Builds only default profile
11-
# - 'build-demo': Builds only demo profile
12+
# - 'build-dev': Builds only demo profile
1213
# - 'build-wiab-staging' - Builds only wiab-staging profile
1314
# - 'build-min': Builds only min profile
1415
# - 'build-all': Explicitly builds all profiles (useful for workflow changes)
@@ -33,6 +34,7 @@ jobs:
3334
build-default:
3435
name: Build default profile
3536
if: |
37+
(github.event_name == 'push' && github.ref == 'refs/heads/master') ||
3638
contains(github.event.pull_request.labels.*.name, 'build-all') ||
3739
contains(github.event.pull_request.labels.*.name, 'build-default') ||
3840
contains(github.event.pull_request.labels.*.name, 'build-wiab-staging')
@@ -77,6 +79,7 @@ jobs:
7779
name: Verify default profile
7880
needs: build-default
7981
if: |
82+
(github.event_name == 'push' && github.ref == 'refs/heads/master') ||
8083
contains(github.event.pull_request.labels.*.name, 'build-all') ||
8184
contains(github.event.pull_request.labels.*.name, 'build-default')
8285
runs-on:
@@ -116,6 +119,7 @@ jobs:
116119
name: Verify wiab staging profile
117120
needs: build-default
118121
if: |
122+
(github.event_name == 'push' && github.ref == 'refs/heads/master') ||
119123
contains(github.event.pull_request.labels.*.name, 'build-all') ||
120124
contains(github.event.pull_request.labels.*.name, 'build-wiab-staging')
121125
runs-on:
@@ -176,11 +180,12 @@ jobs:
176180
DOCKER_LOGIN: '${{ secrets.DOCKER_LOGIN }}'
177181

178182
# Build demo profile
179-
build-demo:
183+
build-dev:
180184
name: Build demo profile
181185
if: |
186+
(github.event_name == 'push' && github.ref == 'refs/heads/master') ||
182187
contains(github.event.pull_request.labels.*.name, 'build-all') ||
183-
contains(github.event.pull_request.labels.*.name, 'build-demo')
188+
contains(github.event.pull_request.labels.*.name, 'build-dev')
184189
runs-on:
185190
group: wire-server-deploy
186191
steps:
@@ -239,6 +244,7 @@ jobs:
239244
build-min:
240245
name: Build min profile
241246
if: |
247+
(github.event_name == 'push' && github.ref == 'refs/heads/master') ||
242248
contains(github.event.pull_request.labels.*.name, 'build-all') ||
243249
contains(github.event.pull_request.labels.*.name, 'build-min')
244250
runs-on:

ansible/inventory/demo/wiab-staging.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,4 @@ wiab-staging:
66
ansible_user: 'demo'
77
ansible_ssh_private_key_file: "~/.ssh/id_ed25519"
88
vars:
9-
artifact_hash: e5e1919173f6fe2b56034a57e2dc59e8343e8b8e
9+
artifact_hash: f1f624256bdab0f9f76158c7f45e0618ee641237

changelog.d/4-docs/fix-wiab-docs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Fixed: Fixed wiab-staging documentation

offline/wiab-staging.md

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ Since the inventory is ready, please continue with the following steps:
128128
### Environment Setup
129129

130130
- **[Making tooling available in your environment](docs_ubuntu_22.04.md#making-tooling-available-in-your-environment)**
131-
- Source the `bin/offline-env.sh` shell script to set up a `d` alias that runs commands inside a Docker container with all necessary tools for offline deployment.
131+
- Source the `bin/offline-env.sh` shell script by running `source bin/offline-env.sh` to set up a `d` alias that runs commands inside a Docker container with all necessary tools for offline deployment.
132132

133133
- **[Generating secrets](docs_ubuntu_22.04.md#generating-secrets)**
134134
- Run `./bin/offline-secrets.sh` to generate fresh secrets for Minio and coturn services. This creates two secret files: `ansible/inventory/group_vars/all/secrets.yaml` and `values/wire-server/secrets.yaml`.
@@ -147,6 +147,14 @@ Since the inventory is ready, please continue with the following steps:
147147
- `CERT_MASTER_EMAIL`: email used by cert-manager for ACME registration.
148148
- `HOST_IP`: public IP that matches your DNS A record (auto-detected if empty).
149149

150+
**TLS / certificate behavior (cert-manager vs. Bring Your Own):**
151+
- By default, `bin/helm-operations.sh` runs `deploy_cert_manager`, which installs cert-manager and configures a Let’s Encrypt (HTTP-01) issuer for the ingress charts.
152+
- If you **do not** want Let’s Encrypt / cert-manager (for example, you are using **[Bring Your Own certificates](docs_ubuntu_22.04.md#acquiring--deploying-ssl-certificates)** or you cannot satisfy HTTP-01 requirements), disable this step by commenting out the `deploy_cert_manager` call inside `bin/helm-operations.sh`.
153+
- After disabling cert-manager, ensure your ingress is configured with your own TLS secret(s) as described in the TLS documentation below.
154+
155+
**To run the automated helm chart deployment**:
156+
`d ./bin/helm-operations.sh`
157+
150158
**Charts deployed by the script:**
151159
- External datastores and helpers: `cassandra-external`, `elasticsearch-external`, `minio-external`, `rabbitmq-external`, `databases-ephemeral`, `reaper`, `fake-aws`, `demo-smtp`.
152160
- Wire services: `wire-server`, `webapp`, `account-pages`, `team-settings`, `smallstep-accomp`.

0 commit comments

Comments
 (0)