Skip to content

Conversation

alexandertgtalbot
Copy link

Description

Add flow log support taking heavy inspiration from the terraform-aws-vpc module.

Motivation and Context

Flow logs are required auditing and troubleshooting purposes.

Breaking Changes

None.

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

- Add conditional flow logging mimicing the style of the following
  Terraform module: https://github.com/terraform-aws-modules/terraform-aws-vpc.
@alexandertgtalbot alexandertgtalbot changed the title Feature: Add Flow log Support feat: Add Flow Log Support Jul 12, 2025
@luminajames
Copy link

luminajames commented Jul 15, 2025

I would like to see this implemented as well.

Not having flow logging enabled leads to a fail of the AWS Foundational Security Best Practices v1.0.0 standard. It probably shows as an issue in other tools as well.

Copy link

This PR has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this PR will be closed in 10 days

@github-actions github-actions bot added the stale label Aug 15, 2025
@mohag
Copy link

mohag commented Aug 18, 2025

+1 or whatever we need to get the annoying bot that things that ignoring problems solves them to go away

@github-actions github-actions bot removed the stale label Aug 19, 2025
Copy link

This PR has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this PR will be closed in 10 days

@github-actions github-actions bot added the stale label Sep 19, 2025
@mohag
Copy link

mohag commented Sep 19, 2025

This might need changes if region support is added. (the maintainer does not seem positive about that though)

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

Successfully merging this pull request may close these issues.

3 participants