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

Skip to content

Update variables.tf#127

Open
johnlakeee wants to merge 1 commit into
mainfrom
johnlakeee-patch-7
Open

Update variables.tf#127
johnlakeee wants to merge 1 commit into
mainfrom
johnlakeee-patch-7

Conversation

@johnlakeee

Copy link
Copy Markdown
Collaborator

Description

Added the logging_bucket_retention variable defaulting to 365 (1 year) to meet regulatory requirements out-of-the-box (e.g. NIST AU-9).

Updated the description of the log_sinks variable to reference the new 365-day default retention period.

Fixes #7

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Deployment & Compliance Impact

  • Applicable Regimes:
    • US Region Restricted (e.g., Access Policy constraint)
    • FedRAMP Moderate
    • FedRAMP High
    • DoD IL4
    • DoD IL5
    • General / All
  • NIST 800-53r5 Controls: (If this PR helps satisfy or modifies control implementations, list them here)

Checklist

Code Quality & Reusability

  • My code adheres to the Maximize Reusability principle. I have not redefined common elements and have reused existing base configurations and modules where possible.
  • I have checked that no existing module or configuration in modules/ or fast/ can be leveraged for this change.
  • My code follows the established naming conventions outlined in documentation/naming-convention.md.

Documentation

  • I have updated the README.md of the modified module or blueprint.
  • I have added/updated documentation for inputs (variables) and outputs.

Security

  • My change adheres to GCP security best practices and the principle of least privilege.
  • I have ensured compliance with the targeted regime (FedRAMP High, IL5, etc.).

Testing

  • I have tested my changes locally.
  • I have included details of my testing in this PR.

Testing Performed

Please describe the tests that you ran to verify your changes.

Added the logging_bucket_retention variable defaulting to 365 (1 year) to meet regulatory requirements out-of-the-box (e.g. NIST AU-9).

Updated the description of the log_sinks variable to reference the new 365-day default retention period.
@johnlakeee johnlakeee added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] Documentation - Audit Logging

3 participants