Skip to content

Commit

Permalink
remove useless dependencies in deployment tests, turn on pull request…
Browse files Browse the repository at this point in the history
… trigger to check the action works
  • Loading branch information
emileten committed Feb 21, 2024
1 parent b4de035 commit 9bd845b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 14 deletions.
10 changes: 4 additions & 6 deletions .github/workflows/deploy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ name: Test deployment

on:
merge_group:
branches: [ main ]

jobs:
build_package_and_deploy:
Expand Down Expand Up @@ -54,11 +55,8 @@ jobs:
cd integration_tests/cdk
npx cdk synth --debug --all --require-approval never
# deploy the stack and grab URLs for testing
# deploy the stack
npx cdk deploy --ci --all --require-approval never
echo "ingestor_url=$(aws cloudformation describe-stacks --stack-name eoapi-cdk-integration-test-pgSTAC-infra --query "Stacks[0].Outputs[?starts_with(OutputKey, 'stacingestor')].OutputValue | [0]" --output text)" >> $GITHUB_OUTPUT
echo "stac_api_url=$(aws cloudformation describe-stacks --stack-name eoapi-cdk-integration-test-pgSTAC-infra --query "Stacks[0].Outputs[?starts_with(OutputKey, 'pgstacapi')].OutputValue | [0]" --output text)" >> $GITHUB_OUTPUT
echo "titiler_pgstac_api_url=$(aws cloudformation describe-stacks --stack-name eoapi-cdk-integration-test-pgSTAC-infra --query "Stacks[0].Outputs[?starts_with(OutputKey, 'titilerpgstac')].OutputValue | [0]" --output text)" >> $GITHUB_OUTPUT
deactivate
cd -
Expand All @@ -67,11 +65,11 @@ jobs:
run: |
cd integration_tests/cdk
# run this only if we find a 'cdk.out' directory, which means there might be things to tear down
# see https://github.com/aws/aws-cdk/issues/24946
rm -f cdk.out/synth.lock
if [ -d "cdk.out" ]; then
cd -
source .deployment_venv/bin/activate
cd integration_tests/cdk
# see https://github.com/aws/aws-cdk/issues/24946
rm -f cdk.out/synth.lock
npx cdk destroy --ci --all --force
fi
8 changes: 0 additions & 8 deletions integration_tests/cdk/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,15 +1,7 @@
aws-cdk-lib>=2.99.1
aws_cdk.aws_cognito_identitypool_alpha>=2.99.0a0
aws-cdk.aws-apigatewayv2-alpha>=2.99.0a0
constructs>=10.0.0,<11.0.0
pydantic==2.0.2
pydantic-settings==2.0.1
black==22.3.0
boto3==1.24.15
boto3-stubs[cognito-idp,cognito-identity]
flake8==4.0.1
click==8.1.3
requests==2.28.0
python-dotenv==1.0.0
pyyaml==6.0
types-PyYAML==6.0.12.10

0 comments on commit 9bd845b

Please sign in to comment.