Skip to content

Configura limpeza do sandbox manual#1

Open
anderson-pids wants to merge 3 commits intomainfrom
config-to-nuke-sandbox
Open

Configura limpeza do sandbox manual#1
anderson-pids wants to merge 3 commits intomainfrom
config-to-nuke-sandbox

Conversation

@anderson-pids
Copy link

No description provided.



account-blocklist:
- 1234567890

Choose a reason for hiding this comment

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

Podemos colocar o account id como variável de ambiente em algum lugar? Já que esse repositório é público.

Choose a reason for hiding this comment

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

Outro ponto, aqui não deveria ser o account_id da nossa conta principal? Ao invéz de 1234567890.

# - MSKCluster

accounts:
103334327501:

Choose a reason for hiding this comment

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

aqui também?

- property: UserName
value: "infra-nuke"
IAMRole:
- "MeliuzSandboxCrossAccountAdmin"

Choose a reason for hiding this comment

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

Esse aqui também? Colocar como envvar

- "MeliuzSandboxCrossAccountAdmin"
IAMRolePolicyAttachment:
- property: RoleName
value: "MeliuzSandboxCrossAccountAdmin"

Choose a reason for hiding this comment

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

Assim como esse

- "infra-nuke"
IAMUserPolicyAttachment:
- property: UserName
value: "infra-nuke"

Choose a reason for hiding this comment

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

Esse nome de usuário, podemos colocar como envvar?

value: "infra-nuke"
IAMUserAccessKey:
- property: UserName
value: "infra-nuke"

Choose a reason for hiding this comment

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

Esse nome de usuário, podemos colocar como envvar?

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