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

bug in default override json #703

Open
ocofaigh opened this issue Feb 8, 2024 · 4 comments
Open

bug in default override json #703

ocofaigh opened this issue Feb 8, 2024 · 4 comments
Assignees

Comments

@ocofaigh
Copy link
Member

ocofaigh commented Feb 8, 2024

The override.json files in the patterns all have a bug in them:

"archive_rule": {
  "rule_id": "a-bucket-arch-rule",
  "enable": true,
  "days": 0,
  "type": "GLACIER"
}

This will give the error:
image

Seems the expected value it Glacier and not all caps like in the override json GLACIER

@ocofaigh
Copy link
Member Author

ocofaigh commented Feb 8, 2024

@akocbek look like you added this in #687 so assigning to you

@ocofaigh
Copy link
Member Author

ocofaigh commented Feb 8, 2024

@akocbek it looks like you have set other things in the override.json that should not be set. The override json should always be set up to the default values, but I see you added things like "allowed_ip": ["10.10.10.10/24"] which we should not have as a default value.

@akocbek
Copy link
Contributor

akocbek commented Feb 9, 2024

#704

@akocbek
Copy link
Contributor

akocbek commented Feb 12, 2024

we had to remove activity_tracking and metrics_monitoring because they require valid CRN (e.q. activity_tracker_crn) which we don't have it at that stage. If we remove activity_tracker_crn or metrics_monitoring_crn only then we get the following:

│ The given value is not suitable for module.roks_landing_zone.module.landing_zone.var.cos declared at ../../variables.tf:480,1-15: element 0: attribute "buckets": element 0: attribute "metrics_monitoring":
│ attribute "metrics_monitoring_crn" is required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants