Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates to deploy PDS Nucleus in MCP Test and Prod environments #98

Merged
merged 32 commits into from
Jun 20, 2024

Conversation

ramesh-maddegoda
Copy link
Contributor

🗒️ Summary

This pull request updates Terraform scripts, IAM policy files, Lambda functions and documentation to deploy PDS Nucleus in MCP Test and Prod environments.

♻️ Related Issues

#89

…mentation to deploy PDS Nucleus in MCP Test and Prod environments.

Refer to the issue:
#89
@jordanpadams
Copy link
Member

@ramesh-maddegoda note: the secrets workflow is failing so the secrets baseline may need to be regenerated

@nutjob4life
Copy link
Member

@ramesh-maddegoda FYI, to create a new secrets baseline:

$ detect-secrets scan --disable-plugin AbsolutePathDetectorExperimental \
    --exclude-files '\.secrets..*' \
    --exclude-files '\.git.*' \
    --exclude-files '\.pre-commit-config\.yaml' \
    --exclude-files '\.mypy_cache' \
    --exclude-files '\.pytest_cache' \
    --exclude-files '\.tox' \
    --exclude-files '\.venv' \
    --exclude-files 'venv' \
    --exclude-files 'dist' \
    --exclude-files 'build' \
    --exclude-files '.*\.egg-info' \
    > .secrets.baseline

Copy link
Member

@nutjob4life nutjob4life left a comment

Choose a reason for hiding this comment

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

Whoa … this was an enormous pull request! +2628, -1547 😅

Let me know if I should clarify any remarks/questions. Thanks in advance!

.secrets.baseline Show resolved Hide resolved
password = ChangeMe!
```

18. Use the PDS Data Upload Manager (DUM) tool to upload files to pds_nucleus_staging_bucket.
Copy link
Member

Choose a reason for hiding this comment

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

Whew! I opened these up in a Markdown viewer. Thanks for a really complete set of instructions 😌

ENV LD_LIBRARY_PATH="/cfitsio-4.4.0/lib"

COPY entrypoint.sh /usr/local/bin/entrypoint.sh
ENTRYPOINT ["bash", "/usr/local/bin/entrypoint.sh"]
Copy link
Member

Choose a reason for hiding this comment

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

Ditto

terraform/terraform-modules/ecs-ecr/variables.tf Outdated Show resolved Hide resolved
terraform/terraform-modules/ecs-ecr/variables.tf Outdated Show resolved Hide resolved
sensitive = true
}

#variable "pds_nucleus_config_init_ecr_image_path" {
Copy link
Member

Choose a reason for hiding this comment

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

Ditto

}


#variable "pds_nucleus_s3_to_efs_copy_ecr_image_path" {
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

terraform/terraform-modules/ecs-ecr/variables.tf Outdated Show resolved Hide resolved
terraform/terraform-modules/mwaa-env/requirements.txt Outdated Show resolved Hide resolved
@ramesh-maddegoda
Copy link
Contributor Author

Whoa … this was an enormous pull request! +2628, -1547 😅

Let me know if I should clarify any remarks/questions. Thanks in advance!

Thank a lot for your review!

@ramesh-maddegoda
Copy link
Contributor Author

@ramesh-maddegoda FYI, to create a new secrets baseline:

$ detect-secrets scan --disable-plugin AbsolutePathDetectorExperimental \
    --exclude-files '\.secrets..*' \
    --exclude-files '\.git.*' \
    --exclude-files '\.pre-commit-config\.yaml' \
    --exclude-files '\.mypy_cache' \
    --exclude-files '\.pytest_cache' \
    --exclude-files '\.tox' \
    --exclude-files '\.venv' \
    --exclude-files 'venv' \
    --exclude-files 'dist' \
    --exclude-files 'build' \
    --exclude-files '.*\.egg-info' \
    > .secrets.baseline

@jordanpadams and @nutjob4life , I created a new secret baseline.

@jordanpadams jordanpadams merged commit 304a2c9 into main Jun 20, 2024
1 check passed
@jordanpadams jordanpadams deleted the mcp-deployment-terraform branch June 20, 2024 18:30
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.

4 participants