Skip to content

Feat/#55: 변수 처리 방식 변경#62

Merged
yunhoch0i merged 2 commits intomainfrom
feat/#55-1
Jul 29, 2025
Merged

Feat/#55: 변수 처리 방식 변경#62
yunhoch0i merged 2 commits intomainfrom
feat/#55-1

Conversation

@yunhoch0i
Copy link
Contributor

@yunhoch0i yunhoch0i commented Jul 29, 2025

#️⃣ Related Issues

#55
#61
#63

📝 Work Summary

terraform apply에서 account_id가 노출되는 문제를 해결하기 위해 Secret를 통해 변수를 받는 방식으로 변경

Screenshot (Optional)

스크린샷 2025-07-29 오후 4 35 37

@github-actions
Copy link

[Terraform Plan Summary]

항목
Status success
Directory operation-team-account/runtime-verification/inspector
Executed At 2025-07-29 07:37:54 UTC

Plan Output

Plan failed

Plan Error (if any)


Error: No value for required variable

  on variables.tf line 1:
   1: variable "prod_account_id" {

The root module input variable "prod_account_id" is not set, and has no
default value. Use a -var or -var-file command line argument to provide a
value for this variable.

@github-actions
Copy link

[Terraform Plan Summary]

항목
Status success
Directory management-team-account/inspector-delegation/organizations
Executed At 2025-07-29 07:37:58 UTC

Plan Output

Plan failed

Plan Error (if any)


Error: No value for required variable

  on variables.tf line 1:
   1: variable "operation_account_id" {

The root module input variable "operation_account_id" is not set, and has no
default value. Use a -var or -var-file command line argument to provide a
value for this variable.

@github-actions
Copy link

github-actions bot commented Jul 29, 2025

💰 Infracost report

Monthly estimate generated

Estimate details (includes details of unsupported resources)
──────────────────────────────────
1 project has no cost estimate change.
Run the following command to see its breakdown: infracost breakdown --path=/path/to/code

──────────────────────────────────
2 cloud resources were detected:
∙ 0 were estimated
∙ 2 are not supported yet, see https://infracost.io/requested-resources:
  ∙ 1 x aws_inspector2_delegated_admin_account
  ∙ 1 x aws_inspector2_enabler
This comment will be updated when code changes.

@github-actions
Copy link

[Terraform Plan Summary]

항목
Status success
Directory management-team-account/inspector-delegation/organizations
Executed At 2025-07-29 07:45:00 UTC

Plan Output

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # aws_inspector2_delegated_admin_account.this will be created
  + resource "aws_inspector2_delegated_admin_account" "this" {
      + id                  = (known after apply)
      + relationship_status = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Plan Error (if any)

(no errors)

@github-actions
Copy link

[Terraform Plan Summary]

항목
Status success
Directory operation-team-account/runtime-verification/inspector
Executed At 2025-07-29 07:44:59 UTC

Plan Output

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # aws_inspector2_delegated_admin_account.prod_account will be created
  + resource "aws_inspector2_delegated_admin_account" "prod_account" {
      + id                  = (known after apply)
      + relationship_status = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Plan Error (if any)

(no errors)

Copy link
Contributor

@rnjsdbwlsqwer rnjsdbwlsqwer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

secret으로 변수 받아오는 것 확인했습니다.

@yunhoch0i yunhoch0i merged commit e6bc899 into main Jul 29, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants