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

feat: support scan_type #113

Merged
merged 16 commits into from
Nov 8, 2023
Merged

feat: support scan_type #113

merged 16 commits into from
Nov 8, 2023

Conversation

dudymas
Copy link
Contributor

@dudymas dudymas commented Oct 9, 2023

what

  • add scanning_configuration as child module

why

  • support scan_type settings
  • should be deployed as a per-account global, rather than per-ecr-repo

Notes

examples/complete/main.tf Outdated Show resolved Hide resolved
modules/scanning_configuration/outputs.tf Outdated Show resolved Hide resolved
@dudymas
Copy link
Contributor Author

dudymas commented Oct 9, 2023

/terratest

@dudymas dudymas force-pushed the feat/scan_config/added branch from 273ed96 to 66b31d9 Compare October 9, 2023 15:50
@dudymas
Copy link
Contributor Author

dudymas commented Oct 9, 2023

/terratest

@dudymas
Copy link
Contributor Author

dudymas commented Oct 10, 2023

/terratest

@dudymas
Copy link
Contributor Author

dudymas commented Oct 10, 2023

/terratest

@dudymas
Copy link
Contributor Author

dudymas commented Oct 10, 2023

/terratest

@dudymas
Copy link
Contributor Author

dudymas commented Nov 1, 2023

/terratest

@dudymas
Copy link
Contributor Author

dudymas commented Nov 1, 2023

/terratest

@dudymas
Copy link
Contributor Author

dudymas commented Nov 1, 2023

/terratest

@dudymas
Copy link
Contributor Author

dudymas commented Nov 1, 2023

/terratest

@dudymas dudymas marked this pull request as ready for review November 1, 2023 18:55
@dudymas dudymas requested review from a team as code owners November 1, 2023 18:55
@dudymas dudymas requested a review from srhopkins November 1, 2023 18:55
@dudymas dudymas requested a review from florian0410 November 1, 2023 18:55
main.tf Outdated Show resolved Hide resolved
@dudymas
Copy link
Contributor Author

dudymas commented Nov 1, 2023

/terratest

@@ -0,0 +1,4 @@
output "configuration" {
description = "The ECR registry scanning configuration"
value = aws_ecr_registry_scanning_configuration.default[*]
Copy link
Member

Choose a reason for hiding this comment

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

this will make the configuration output a list.
We can improve it like this:

value       = one(aws_ecr_registry_scanning_configuration.default[*])

Copy link
Member

@aknysh aknysh left a comment

Choose a reason for hiding this comment

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

see comments

@dudymas
Copy link
Contributor Author

dudymas commented Nov 7, 2023

/terratest

@dudymas dudymas merged commit 1bdedff into main Nov 8, 2023
@dudymas dudymas deleted the feat/scan_config/added branch November 8, 2023 12:46
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.

Requesting to add scan_type to aws_ecr_registry_scanning_configuration
3 participants