Skip to content

fix(org-policies,docs): add essential contacts policy parameters and document blueprint deployer permissions - #210

Open
aghassemlouei wants to merge 3 commits into
mainfrom
fix/org-policy-and-blueprint-deployer-docs
Open

fix(org-policies,docs): add essential contacts policy parameters and document blueprint deployer permissions#210
aghassemlouei wants to merge 3 commits into
mainfrom
fix/org-policy-and-blueprint-deployer-docs

Conversation

@aghassemlouei

Copy link
Copy Markdown
Collaborator

Description

This pull request addresses essential contacts org policy parameters and documents deployer permissions for database and serverless blueprints:

Fixes #108
Fixes #115

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: SI-5, IR-6, AC-6

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 Moderate, FedRAMP High, IL5, etc.).

Testing

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

Testing Performed

  • Validated YAML syntax in platform_policy.yaml.
  • Verified README markdown formatting and links in PostgreSQL and App Engine blueprints.

@aghassemlouei aghassemlouei added bug Something isn't working documentation Improvements or additions to documentation Priority - Low Minor polish, typo fixes, or technical debt; nice to have; pick up when time permits Level of Effort - Low Quick, well-defined tasks with no unknowns; takes a few hours up to one day to complete labels Aug 28, 2026
@Calvin-Cheng1
Calvin-Cheng1 self-requested a review August 29, 2026 04:28

@Calvin-Cheng1 Calvin-Cheng1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

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

Labels

bug Something isn't working documentation Improvements or additions to documentation Level of Effort - Low Quick, well-defined tasks with no unknowns; takes a few hours up to one day to complete Priority - Low Minor polish, typo fixes, or technical debt; nice to have; pick up when time permits

Projects

None yet

3 participants