Skip to content

Terraform apply is broken #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
stephanosio opened this issue May 17, 2022 · 2 comments
Closed

Terraform apply is broken #27

stephanosio opened this issue May 17, 2022 · 2 comments
Assignees
Labels
bug Something isn't working priority: low Low impact/importance task

Comments

@stephanosio
Copy link
Member

stephanosio commented May 17, 2022

Terraform apply fails when applying the zephyr_runner configurations:
https://github.com/zephyrproject-rtos/terraform-aws-github-runner/tree/production/examples/zephyr

module.runners.module.webhook.aws_iam_role_policy.webhook_logging: Refreshing state... [id=ubuntu-action-webhook-lambda-role:ubuntu-lambda-logging-policy]
╷
│ Error: error reading S3 Bucket (ubuntu-dist-iz4lj75v5mz7gur7dw4byrh2): BadRequest: Bad Request
│     status code: 400, request id: TC89N2FARJF8M9YC, host id: MH4ZlzkMwdNhYLihYWrME1x01EZbll8E2aXMx/NFnraLHZjjv/z5IGSdoZP/R1fl0seHuQj9zZc=
│
│   with module.runners.module.runner_binaries.aws_s3_bucket.action_dist,
│   on ../../modules/runner-binaries-syncer/main.tf line 5, in resource "aws_s3_bucket" "action_dist":
│    5: resource "aws_s3_bucket" "action_dist" {
│
╵
╷
│ Error: Your query returned no results. Please change your search criteria and try again.
│
│   with module.runners.module.runners.data.aws_ami.runner,
│   on ../../modules/runners/main.tf line 26, in data "aws_ami" "runner":
│   26: data "aws_ami" "runner" {
│

Potentially related issues:

@stephanosio stephanosio added bug Something isn't working priority: high High impact/importance task labels May 17, 2022
@stephanosio stephanosio added priority: low Low impact/importance task and removed priority: high High impact/importance task labels Sep 13, 2022
@stephanosio
Copy link
Member Author

Re-assigning "low" priority since the current infrastructure is both functional and maintainable without the help of Terraform and #43 will likely make this task obsolete.

@stephanosio
Copy link
Member Author

This is no longer relevant since we have new Actions Runner Controller-based runner deployment (see #55) and are no longer using philips-labs/terraform-aws-github-runner.

@stephanosio stephanosio closed this as not planned Won't fix, can't repro, duplicate, stale Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority: low Low impact/importance task
Projects
None yet
Development

No branches or pull requests

2 participants