Skip to content
This repository was archived by the owner on Nov 22, 2025. It is now read-only.

Commit 9dbef48

Browse files
pacphiclaude
andcommitted
fix(ci): pass region parameter to setup-fly-test-env action
Resolves volume region mismatch causing deployment failures where fly.toml configured one region but volume created in another. The setup-fly-test-env action prepares fly.toml via prepare-fly-config.sh, which needs the region parameter to configure the correct primary_region. Previously, only deploy-fly-app received the region parameter, creating volumes in a different region than fly.toml expected. Fixed in 6 workflow files: - api-compliance.yml - dependency-chain-tests.yml - extension-combinations.yml - manifest-operations-tests.yml - per-extension.yml - protected-extensions-tests.yml Fixes error: "New machine in group 'app' needs an unattached volume named 'test_data' in region 'sjc'" when deploying to non-default regions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 2b4e7ef commit 9dbef48

6 files changed

Lines changed: 6 additions & 0 deletions

.github/workflows/api-compliance.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ jobs:
5959
app-prefix: "${{ inputs.test_app_prefix }}-api"
6060
extension-name: ${{ matrix.extension }}
6161
fly-api-token: ${{ secrets.FLYIO_AUTH_TOKEN }}
62+
region: ${{ inputs.region }}
6263
vm-memory: "4096"
6364
vm-cpu-kind: "shared"
6465
vm-cpu-count: "2"

.github/workflows/dependency-chain-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ jobs:
5252
app-prefix: ${{ inputs.test_app_prefix }}
5353
extension-name: "deps"
5454
fly-api-token: ${{ secrets.FLYIO_AUTH_TOKEN }}
55+
region: ${{ inputs.region }}
5556
vm-memory: "4096"
5657
vm-cpu-kind: "shared"
5758
vm-cpu-count: "2"

.github/workflows/extension-combinations.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,7 @@ jobs:
9090
app-prefix: ${{ inputs.test_app_prefix }}
9191
extension-name: ${{ matrix.combination.name }}
9292
fly-api-token: ${{ secrets.FLYIO_AUTH_TOKEN }}
93+
region: ${{ inputs.region }}
9394
vm-memory: "16384"
9495
vm-cpu-kind: "performance"
9596
vm-cpu-count: "4"

.github/workflows/manifest-operations-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ jobs:
5252
app-prefix: ${{ inputs.test_app_prefix }}
5353
extension-name: "manifest"
5454
fly-api-token: ${{ secrets.FLYIO_AUTH_TOKEN }}
55+
region: ${{ inputs.region }}
5556
vm-memory: "2048"
5657
vm-cpu-kind: "shared"
5758
vm-cpu-count: "1"

.github/workflows/per-extension.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,7 @@ jobs:
114114
app-prefix: ${{ inputs.test_app_prefix }}
115115
extension-name: ${{ matrix.extension.name }}
116116
fly-api-token: ${{ secrets.FLYIO_AUTH_TOKEN }}
117+
region: ${{ inputs.region }}
117118

118119
# NEW: Using composite action for deployment
119120
- name: Deploy test environment

.github/workflows/protected-extensions-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ jobs:
5252
app-prefix: ${{ inputs.test_app_prefix }}
5353
extension-name: "protected"
5454
fly-api-token: ${{ secrets.FLYIO_AUTH_TOKEN }}
55+
region: ${{ inputs.region }}
5556
vm-memory: "2048"
5657
vm-cpu-kind: "shared"
5758
vm-cpu-count: "1"

0 commit comments

Comments
 (0)