Skip to content

Add .gitignore and pgcopydb infrastructure templates#37

Open
teknogeek0 wants to merge 2 commits intomainfrom
add/gitignore-and-templates-readme
Open

Add .gitignore and pgcopydb infrastructure templates#37
teknogeek0 wants to merge 2 commits intomainfrom
add/gitignore-and-templates-readme

Conversation

@teknogeek0
Copy link
Copy Markdown
Contributor

Summary

  • Add .gitignore covering OS files, env/secrets, Terraform state, pgcopydb migration artifacts, and editor files
  • Add pgcopydb migration instance templates (AWS CloudFormation, AWS Terraform, GCP Terraform) for provisioning instances with pgcopydb pre-installed
  • Add top-level pgcopydb-templates/README.md indexing the three provisioning options

Test plan

  • Verify .gitignore excludes expected files (git status no longer shows .DS_Store, .env, etc.)
  • Verify template READMEs render correctly on GitHub
  • Spot-check Terraform and CloudFormation templates for correctness

Add .gitignore covering OS files, env/secrets, Terraform state,
pgcopydb migration artifacts, and editor files. Add top-level
README for pgcopydb-templates to index the three provisioning
options.
CloudFormation and Terraform templates for provisioning migration
instances on AWS and GCP with pgcopydb pre-installed.
Copy link
Copy Markdown
Contributor

@rcrowley rcrowley left a comment

Choose a reason for hiding this comment

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

LGTM

### Prerequisites
- The IAM role from Step 1 must be deployed first
- VPC ID and a subnet ID where the instance should run
- The restored database from Step 3 must be accessible from this subnet
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

On second thought, these prerequisites don't make any sense to me.

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.

2 participants