Use cdk from node modules (aws-observability#1077)

* Fix error to synthesize * Use CDK from node modules instead of global one
PaurushGarg · Feb 7, 2023 · 4a1740f · 4a1740f
1 parent 3eb8ef2
commit 4a1740f
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/.github/workflows/cdk-deploy.yml b/.github/workflows/cdk-deploy.yml
@@ -39,13 +39,10 @@ jobs:
           key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
           restore-keys: |
             ${{ runner.os }}-node-
-      - name: install cdk
-        run: |
-          npm install -g aws-cdk
       - name: npm install
         working-directory: cdk_infra
         run: |
-          npm i
+          npm install
       - name: run cdk deploy
         working-directory: cdk_infra
         run: |

diff --git a/cdk_infra/Makefile b/cdk_infra/Makefile
@@ -1,3 +1,3 @@
 .PHONY: deploy-infra
 deploy-infra: 
-	cdk deploy --all --require-approval never --concurrency 4
+	npm run cdk deploy -- --all --require-approval never --concurrency 4
diff --git a/cdk_infra/README.md b/cdk_infra/README.md
@@ -87,6 +87,6 @@ The app relies on `CDK_DEFAULT_ACCOUNT` and `CDK_DEFAULT_REGION` to set their [e
 * `npm run build`   compile typescript to js
 * `npm run watch`   watch for changes and compile
 * `npm run test`    perform the jest unit tests
-* `cdk deploy`      deploy this stack to your default AWS account/region
-* `cdk diff`        compare deployed stack with current state
-* `cdk synth`       emits the synthesized CloudFormation template
+* `npm run cdk synth`       emits the synthesized CloudFormation template
+* `npm run cdk deploy`      deploy this stack to your default AWS account/region
+* `npm run cdk diff`        compare deployed stack with current state