Skip to content

Conversation

@drernie
Copy link
Member

@drernie drernie commented Dec 3, 2025

Summary

  • Add warning in CloudFormation section about not modifying Quilt-provided templates
  • Add warning in Terraform section about not modifying Quilt-provided modules and templates
  • Clarify that customizations may affect service agreement coverage
  • Direct users to contact their account manager for supported customization options

Context

Enterprise customers need to understand that customizing Quilt-provided CloudFormation templates or Terraform modules can negatively impact their service agreement coverage. These warnings provide gentle but clear guidance while offering a path forward through account manager consultation.

🤖 Generated with Claude Code

Greptile Overview

Greptile Summary

Added important warnings in the CloudFormation and Terraform deployment sections to inform enterprise customers that customizing Quilt-provided templates or modules may negatively impact service agreement coverage. The warnings are well-placed, consistent with existing documentation style, and provide a clear path forward through account manager consultation.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk - documentation-only changes with clear messaging
  • Score reflects that this is a straightforward documentation update that adds important customer-facing warnings without any code changes, deployment logic modifications, or technical risk
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
docs/Catalog/Installation.md 5/5 Added clear warnings in CloudFormation and Terraform sections about not modifying Quilt-provided templates/modules and directing users to contact account managers for customization needs

Sequence Diagram

sequenceDiagram
    participant User as Enterprise Customer
    participant Docs as Installation Documentation
    participant CF as CloudFormation/Terraform
    participant Support as Quilt Account Manager
    
    User->>Docs: Read deployment instructions
    Docs->>User: Display warning about template modification
    
    alt Customer needs customization
        User->>Support: Contact for customization options
        Support->>User: Provide supported customization path
        User->>CF: Deploy with approved customizations
    else Customer follows standard path
        User->>CF: Deploy with unmodified templates
    end
    
    CF->>User: Deployment complete with full service agreement coverage
Loading

Add warnings in the Installation documentation to clarify that
Quilt-provided CloudFormation templates and Terraform modules should not
be modified, as customizations may cause deployment issues and affect
service agreement coverage. Direct users to contact their account
manager for supported customization options.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@codecov
Copy link

codecov bot commented Dec 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 36.71%. Comparing base (11258ec) to head (fa1998e).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4648   +/-   ##
=======================================
  Coverage   36.71%   36.71%           
=======================================
  Files         865      865           
  Lines       35433    35433           
  Branches     6238     6241    +3     
=======================================
  Hits        13009    13009           
  Misses      21153    21153           
  Partials     1271     1271           
Flag Coverage Δ
api-python 91.66% <ø> (ø)
catalog 21.82% <ø> (ø)
py-shared 98.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@drernie
Copy link
Member Author

drernie commented Dec 3, 2025

Vir was trying to hack our VPCs to share stacks, and were (pleasantly!) surprised when I told them they shouldn't do that, and need to push back on their IT department. We should probably make this official...

@drernie drernie enabled auto-merge December 3, 2025 05:50
@drernie drernie added this pull request to the merge queue Dec 11, 2025
Merged via the queue into master with commit d6dd34c Dec 11, 2025
46 checks passed
@drernie drernie deleted the doc-do-not-customize branch December 11, 2025 20:15
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.

3 participants