Skip to content

Commit 496aabf

Browse files
committed
better ci and correct timeout
1 parent 23a0649 commit 496aabf

File tree

4 files changed

+15
-7
lines changed

4 files changed

+15
-7
lines changed

.github/workflows/test-aws.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,8 @@ jobs:
4949
runs-on: ubuntu-latest
5050
# Note: Triggered automatically when approved PRs enter GitHub's merge queue
5151
# Smart path filtering ensures tests run only when AWS infrastructure changes
52+
env:
53+
DURATION_SECONDS: 14400 # 4 hours
5254

5355
steps:
5456
- name: Checkout code
@@ -61,7 +63,7 @@ jobs:
6163
role-session-name: GitHubActions-AWS-Tests
6264
aws-region: ${{ vars.AWS_REGION || 'us-east-1' }}
6365
# Max duration of the role is 8 hours as per IAM role configuration in .github/setup/aws/main.tf
64-
role-duration-seconds: 14400 # 4 hours
66+
role-duration-seconds: ${{ env.DURATION_SECONDS }}
6567

6668
- name: Verify AWS Authentication
6769
run: |
@@ -102,4 +104,4 @@ jobs:
102104
SKIP_setup_materialize_disk_disabled: ${{ github.event.inputs.test_stage == 'network-only' && 'true' || '' }}
103105
run: |
104106
echo "Running tests"
105-
go test -timeout 240m -run TestStagedDeploymentSuite -v
107+
go test -timeout ${{ env.DURATION_SECONDS }}s -run TestStagedDeploymentSuite -v

.github/workflows/test-azure.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,8 @@ jobs:
4848
test-azure:
4949
name: Azure Infrastructure Tests
5050
runs-on: ubuntu-latest
51+
env:
52+
DURATION_SECONDS: 14400 # 4 hours
5153

5254
steps:
5355
- name: Checkout code
@@ -59,6 +61,8 @@ jobs:
5961
role-to-assume: ${{ vars.GA_AWS_IAM_ROLE }}
6062
role-session-name: GitHubActions-Azure-S3Backend
6163
aws-region: ${{ vars.TF_TEST_S3_REGION || 'us-east-1' }}
64+
# Max duration of the role is 8 hours as per IAM role configuration in .github/setup/aws/main.tf
65+
role-duration-seconds: ${{ env.DURATION_SECONDS }}
6266

6367
- name: Verify AWS Authentication (for S3 backend)
6468
run: |
@@ -122,4 +126,4 @@ jobs:
122126
echo ""
123127
echo "🧪 Starting Go tests..."
124128
125-
go test -timeout 240m -run TestStagedDeploymentSuite -v -count=1
129+
go test -timeout ${{ env.DURATION_SECONDS }}s -run TestStagedDeploymentSuite -v

.github/workflows/test-gcp.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,8 @@ jobs:
4747
test-gcp:
4848
name: GCP Infrastructure Tests
4949
runs-on: ubuntu-latest
50+
env:
51+
DURATION_SECONDS: 14400 # 4 hours
5052

5153
steps:
5254
- name: Checkout code
@@ -58,13 +60,15 @@ jobs:
5860
role-to-assume: ${{ vars.GA_AWS_IAM_ROLE }}
5961
role-session-name: GitHubActions-GCP-S3Backend
6062
aws-region: ${{ vars.TF_TEST_S3_REGION || 'us-east-1' }}
63+
# Max duration of the role is 8 hours as per IAM role configuration in .github/setup/aws/main.tf
64+
role-duration-seconds: ${{ env.DURATION_SECONDS }}
6165

6266
- name: Authenticate to Google Cloud (Service Account Impersonation)
6367
uses: google-github-actions/auth@v2
6468
with:
6569
workload_identity_provider: ${{ secrets.GCP_WORKLOAD_IDENTITY_PROVIDER }}
6670
service_account: ${{ secrets.GCP_SERVICE_ACCOUNT_EMAIL }}
67-
access_token_lifetime: 14400s # 4 hours for long-running tests
71+
access_token_lifetime: ${{ env.DURATION_SECONDS }}s # 4 hours for long-running tests
6872

6973
- name: Set up Cloud SDK
7074
uses: google-github-actions/setup-gcloud@v2
@@ -120,4 +124,4 @@ jobs:
120124
echo ""
121125
echo "🧪 Starting Go tests..."
122126
123-
go test -timeout 240m -run TestStagedDeploymentSuite -v -count=1
127+
go test -timeout ${{ env.DURATION_SECONDS }}s -run TestStagedDeploymentSuite -v

test/gcp/staged_deployment_test.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -332,7 +332,6 @@ func (suite *StagedDeploymentSuite) setupMaterializeConsolidatedStage(stage, sta
332332

333333
// GKE Configuration
334334
"namespace": TestGKENamespace,
335-
"skip_nodepool": false,
336335
"materialize_node_type": machineType,
337336
"min_nodes": TestGKEMinNodes,
338337
"max_nodes": TestGKEMaxNodes,
@@ -367,7 +366,6 @@ func (suite *StagedDeploymentSuite) setupMaterializeConsolidatedStage(stage, sta
367366
// Storage Configuration
368367
"storage_bucket_versioning": TestStorageBucketVersioning,
369368
"storage_bucket_version_ttl": TestStorageBucketVersionTTL,
370-
"enable_bucket_encryption": true,
371369

372370
// Cert Manager Configuration
373371
"cert_manager_install_timeout": TestCertManagerInstallTimeout,

0 commit comments

Comments
 (0)