Update depdencies (#105)

* Update depdencies lambda runners, remove depracted methods

* Update dependencies webhook

* Update dependencies sync

* Update webhook, not upgrade ocktokit 7.5.0 due breaking changes

* Upgrade octokit/webhooks

* Add terraform version for examples, upgrade AWS provider to 3.0

* Upgrade jest for webhooks

* Upgrade types/node for syncer

Author: npalm

examples/.terraform-version

@@ -0,0 +1 @@
+0.12.29

examples/default/main.tf

@@ -4,7 +4,7 @@ locals {
 }
 
 resource "random_password" "random" {
-  length = 32
+  length = 28
 }
 
 module "runners" {
@@ -27,12 +27,14 @@ module "runners" {
     webhook_secret = random_password.random.result
   }
 
-  webhook_lambda_zip                = "lambdas-download/webhook.zip"
-  runner_binaries_syncer_lambda_zip = "lambdas-download/runner-binaries-syncer.zip"
-  runners_lambda_zip                = "lambdas-download/runners.zip"
-  enable_organization_runners       = false
-  runner_extra_labels               = "default,example"
+  # webhook_lambda_zip                = "lambdas-download/webhook.zip"
+  # runner_binaries_syncer_lambda_zip = "lambdas-download/runner-binaries-syncer.zip"
+  # runners_lambda_zip                = "lambdas-download/runners.zip"
+  enable_organization_runners = false
+  runner_extra_labels         = "default,example"
+
+  # instance_type = "a1.large"
 
   # disable KMS and encryption
-  # encrypt_secrets = false
+  # encrypt_secrets = true
 }

examples/default/providers.tf

@@ -1,4 +1,4 @@
 provider "aws" {
   region  = local.aws_region
-  version = "2.61"
+  version = "3.0"
 }

examples/permissions-boundary/providers.tf

@@ -1,7 +1,7 @@
 provider "aws" {
   alias   = "terraform_role"
   region  = local.aws_region
-  version = "2.61"
+  version = "3.0"
   assume_role {
     role_arn = data.terraform_remote_state.iam.outputs.role
   }

modules/runner-binaries-syncer/lambdas/runner-binaries-syncer/package.json

@@ -13,9 +13,9 @@
     "dist": "yarn build && cd dist && zip ../runner-binaries-syncer.zip index.js"
   },
   "devDependencies": {
-    "@octokit/rest": "^17.6.0",
+    "@octokit/rest": "^18.0.3",
     "@types/jest": "^26.0.4",
-    "@types/node": "^13.13.4",
+    "@types/node": "^14.0.27",
     "@types/request": "^2.48.4",
     "@typescript-eslint/eslint-plugin": "^2.30.0",
     "@typescript-eslint/parser": "^2.30.0",

modules/runner-binaries-syncer/lambdas/runner-binaries-syncer/yarn.lock

@@ -24,7 +24,7 @@
   },
   "dependencies": {
     "@octokit/auth-app": "^2.4.5",
-    "@octokit/rest": "^17.6.0",
+    "@octokit/rest": "^18.0.3",
     "moment": "^2.25.3",
     "yn": "^4.0.0"
   }

modules/runners/lambdas/runners/package.json

@@ -11,7 +11,7 @@ jest.mock('@octokit/auth-app', () => ({
 const mockOctokit = {
   checks: { get: jest.fn() },
   actions: {
-    listRepoWorkflowRuns: jest.fn(),
+    listWorkflowRunsForRepo: jest.fn(),
     createRegistrationTokenForOrg: jest.fn(),
     createRegistrationTokenForRepo: jest.fn(),
   },
@@ -40,7 +40,7 @@ describe('scaleUp', () => {
     process.env.ENVIRONMENT = 'unit-test-environment';
 
     jest.clearAllMocks();
-    mockOctokit.actions.listRepoWorkflowRuns.mockImplementation(() => ({
+    mockOctokit.actions.listWorkflowRunsForRepo.mockImplementation(() => ({
       data: {
         total_count: 1,
       },
@@ -70,15 +70,15 @@ describe('scaleUp', () => {
 
   it('checks queued workflows', async () => {
     await scaleUp('aws:sqs', TEST_DATA);
-    expect(mockOctokit.actions.listRepoWorkflowRuns).toBeCalledWith({
+    expect(mockOctokit.actions.listWorkflowRunsForRepo).toBeCalledWith({
       owner: TEST_DATA.repositoryOwner,
       repo: TEST_DATA.repositoryName,
       status: 'queued',
     });
   });
 
   it('does not list runners when no workflows are queued', async () => {
-    mockOctokit.actions.listRepoWorkflowRuns.mockImplementation(() => ({
+    mockOctokit.actions.listWorkflowRunsForRepo.mockImplementation(() => ({
       data: { total_count: 0, runners: [] },
     }));
     await scaleUp('aws:sqs', TEST_DATA);

modules/runners/lambdas/runners/src/scale-runners/scale-up.test.ts

@@ -56,7 +56,7 @@ export const scaleUp = async (eventSource: string, payload: ActionRequestMessage
   const environment = process.env.ENVIRONMENT as string;
   const githubAppAuth = await createGithubAppAuth(payload.installationId);
   const githubInstallationClient = await createInstallationClient(githubAppAuth);
-  const queuedWorkflows = await githubInstallationClient.actions.listRepoWorkflowRuns({
+  const queuedWorkflows = await githubInstallationClient.actions.listWorkflowRunsForRepo({
     owner: payload.repositoryOwner,
     repo: payload.repositoryName,
     // @ts-ignore (typing of the 'status' field is incorrect)