Merge pull request #768 from pepperize/blsmth-add-ecr-login-manager

fix: blsmth add ecr login manager
pepperize · Jan 7, 2024 · cd20c13 · cd20c13
2 parents c456bcb + 4545cbd
commit cd20c13
Show file tree

Hide file tree

Showing 4 changed files with 102 additions and 2 deletions.
diff --git a/LICENSE b/LICENSE
diff --git a/src/runner/manager.ts b/src/runner/manager.ts
@@ -201,12 +201,23 @@ export class GitlabRunnerAutoscalingManager extends Construct {
               },
             ],
           }),
+          ECRLogin: PolicyDocument.fromJson({
+            Version: "2012-10-17",
+            Statement: [
+              {
+                Effect: "Allow",
+                Action: ["ecr:BatchGetImage", "ecr:GetAuthorizationToken", "ecr:GetDownloadUrlForLayer"],
+                Resource: "*",
+              },
+            ],
+          }),
         },
       });
 
     this.userData = UserData.forLinux({});
     this.userData.addCommands(
-      `yum update -y aws-cfn-bootstrap` // !/bin/bash -xe
+      `yum update -y aws-cfn-bootstrap`, // !/bin/bash -xe
+      `yum install -y amazon-ecr-credential-helper`
     );
 
     // https://github.com/awslabs/amazon-ecr-credential-helper

diff --git a/test/runner/__snapshots__/manager.test.ts.snap b/test/runner/__snapshots__/manager.test.ts.snap
diff --git a/test/runner/__snapshots__/runner.test.ts.snap b/test/runner/__snapshots__/runner.test.ts.snap