Skip to content

Add terraform script for complete deployment of the Registry in the Dev Account#475

Open
tloubrieu-jpl wants to merge 32 commits into
developfrom
terraform_for_dev
Open

Add terraform script for complete deployment of the Registry in the Dev Account#475
tloubrieu-jpl wants to merge 32 commits into
developfrom
terraform_for_dev

Conversation

@tloubrieu-jpl
Copy link
Copy Markdown
Member

@tloubrieu-jpl tloubrieu-jpl commented Feb 23, 2026

🗒️ Summary

This PR initiates a terraform script which deployed opensearch and the needed policies to start.

A follow up action is going to be able to initialize the registry with registry-mgr and the reference dataset.

It is ready to be merged as it is used for deployment in dev with validated integration tests as a basis for the ISRO node creation.

⚙️ Test Data and/or Report

Integration test validated.

♻️ Related Issues

🤓 Reviewer Checklist

Reviewers: Please verify the following before approving this pull request.

Documentation and PR Content

  • Documentation: README, Wiki, or inline documentation (Sphinx, Javadoc, Docstrings) have been updated to reflect these changes.
  • Issue Traceability: The PR is linked to a valid GitHub Issue
  • PR Title: The PR title is "user-friendly" clearly identifying what is being fixed or the new feature being added, that if you saw it in the Release Notes for a tool, you would be able to get the gist of what was done.

Security & Quality

  • SonarCloud: Confirmed no new High or Critical security findings.
  • Secrets Detection: Verified that the Secrets Detection scan passed and no sensitive information (keys, tokens, PII) is exposed.
  • Code Quality: Code follows organization style guidelines and best practices for the specific language (e.g., PEP 8, Google Java Style).

Testing & Validation

  • Test Accuracy: Verified that test data is accurate, representative of real-world PDS4 scenarios, and sufficient for the logic being tested.
  • Coverage: Automated tests cover new logic and edge cases.
  • Local Verification: (If applicable) Successfully built and ran the changes in a local or staging environment.

Maintenance

  • Backward Compatibility: Confirmed that these changes do not break existing downstream dependencies or API contracts (or that breaking changes are clearly documented).

@tloubrieu-jpl tloubrieu-jpl requested a review from a team as a code owner February 23, 2026 17:50
@tloubrieu-jpl tloubrieu-jpl marked this pull request as draft February 23, 2026 17:50
@tloubrieu-jpl tloubrieu-jpl requested a review from a team February 24, 2026 20:38
@tloubrieu-jpl tloubrieu-jpl marked this pull request as ready for review May 6, 2026 23:35
Comment thread terraform/main.tf
common_tags = var.common_tags
}

# TL: not ready for integration as we have a deadline for the ISRO node creation in dev, step done manuallyt
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@tloubrieu-jpl I feel seen! 😁 (Which is also how some people pronounce my name 🤣)

Thank you for doing this 👍

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.

Create an IAM policy for tenant data operator

2 participants