Skip to content
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

Add new permissions for Amazon Elasticache to coralogix-metric-integr… #170

Merged
merged 4 commits into from
Dec 12, 2024

Conversation

zhrebicek
Copy link
Contributor

@zhrebicek zhrebicek commented Dec 5, 2024

…ation-policy

Description

New permissions needed for PIPEV2-2208

  • first commit is the thing
  • second one is fixed order and markdown format

How Has This Been Tested?

Yes by adding the roles to role in our AWS account

Checklist:

  • I have updated the relevant component changelog(s)

@zhrebicek zhrebicek requested a review from a team as a code owner December 5, 2024 17:18
@zhrebicek zhrebicek requested a review from a team December 5, 2024 17:18
kolov
kolov previously approved these changes Dec 12, 2024
@zhrebicek zhrebicek dismissed stale reviews from RafalSumislawski and kolov via 354aaa6 December 12, 2024 15:29
@zhrebicek
Copy link
Contributor Author

Fixed the indents:

λ aws cloudformation deploy --capabilities CAPABILITY_IAM  CAPABILITY_NAMED_IAM --template-file template.yaml --stack-name test-disabled-STS --parameter-overrides AWSAccount=staging RoleName=test_disabled_STS ExternalId=1234567

while scanning for the next token
found character '\t' that cannot start any token
  in "<unicode string>", line 117, column 20:
                      -	elasticache:DescribeCacheClusters
                       ^ (line: 117)
λ aws cloudformation deploy --capabilities CAPABILITY_IAM  CAPABILITY_NAMED_IAM --template-file template.yaml --stack-name test-disabled-STS --parameter-overrides AWSAccount=staging RoleName=test_disabled_STS ExternalId=1234567

Waiting for changeset to be created..
Waiting for stack create/update to complete

@zhrebicek zhrebicek merged commit 835cac5 into master Dec 12, 2024
4 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.

4 participants