Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix: Error in count condition of HTTP-Api-Gateway #60

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Mar 14, 2025
Merged

Fix: Error in count condition of HTTP-Api-Gateway #60

merged 9 commits into from
Mar 14, 2025

Conversation

Aatishsharma77
Copy link
Contributor

@Aatishsharma77 Aatishsharma77 commented Apr 5, 2024 •
edited by h1manshu98
Loading

what

Fixed count condition for module "kms_key" {}, data "aws_iam_policy_document" "cloudwatch" {} in main.tf
This was causing below error on terraform plan

│ Error: Invalid index
│ 
│   on ../../../main.tf line 543, in module "kms_key":
│  543:   policy              = data.aws_iam_policy_document.cloudwatch[0].json
│     ├────────────────
│     │ data.aws_iam_policy_document.cloudwatch is empty tuple
│ 
│ The given key does not identify an element in this collection value: the collection has no elements.

@clouddrove-ci clouddrove-ci self-assigned this Apr 5, 2024
@Aatishsharma77 Aatishsharma77 added the bug Something isn't working label Apr 5, 2024
@Aatishsharma77 Aatishsharma77 changed the title Fix: Enable of kms_key in HTTP Api-Gateway Fix: Error in count condition of HTTP-Api-Gateway Apr 8, 2024
h1manshu98
h1manshu98 previously approved these changes Apr 10, 2024
View reviewed changes
Copy link
Contributor

@h1manshu98 h1manshu98 left a comment

Choose a reason for hiding this comment

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

LGTM!

clouddrove-ci
clouddrove-ci previously approved these changes Mar 14, 2025
View reviewed changes
@clouddrove-ci clouddrove-ci dismissed stale reviews from h1manshu98 and themself via e477d9b March 14, 2025 12:17
@clouddrove-ci clouddrove-ci merged commit 2d0cee3 into master Mar 14, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@h1manshu98 h1manshu98 h1manshu98 left review comments

@clouddrove-ci clouddrove-ci clouddrove-ci left review comments

@anmolnagpal anmolnagpal Awaiting requested review from anmolnagpal anmolnagpal is a code owner

@themaniskshah themaniskshah Awaiting requested review from themaniskshah

@13archit 13archit Awaiting requested review from 13archit

@vaibhav7797 vaibhav7797 Awaiting requested review from vaibhav7797

@VishwajitNagulkar VishwajitNagulkar Awaiting requested review from VishwajitNagulkar

Assignees

@Aatishsharma77 Aatishsharma77

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Aatishsharma77 @h1manshu98 @clouddrove-ci