Skip to content

Maintenance: Lint/validate examples Iac #1459

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

Closed
1 of 2 tasks
jeromevdl opened this issue Oct 9, 2023 · 3 comments
Closed
1 of 2 tasks

Maintenance: Lint/validate examples Iac #1459

jeromevdl opened this issue Oct 9, 2023 · 3 comments
Assignees
Labels
feature-request New feature or request good-first-issue Good for newcomers maintenance

Comments

@jeromevdl
Copy link
Contributor

Summary

We should validate the example IaC:

  • validate the template (lint), ex: sam validate --lint
  • validate that it "works", ex: sam build (we already have the compilation with maven/gradle)

Not only for SAM but for others too.

Why is this needed?

Make sure the provided templates (IaC) are working.

Which area does this relate to?

Governance, Tests

Solution

Use sam validate --lint (see https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/validate-cfn-lint.html and https://github.com/aws-cloudformation/cfn-lint#config-file) + sam build for SAM.

See also https://awslabs.github.io/serverless-rules/cfn-lint/

For terraform, use terraform validate and tflint.

Acknowledgment

@jeromevdl jeromevdl added feature-request New feature or request good-first-issue Good for newcomers maintenance labels Oct 9, 2023
@jasoniharris
Copy link
Contributor

Please assign this to me.

@jeromevdl
Copy link
Contributor Author

@jeromevdl
Copy link
Contributor Author

Also, if you could extract this plus what you will do for SAM in a pr_iac_lint.yml, so that we don't have all these if java 11 everywhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature or request good-first-issue Good for newcomers maintenance
Projects
None yet
Development

No branches or pull requests

2 participants