Skip to content

Conversation

@nhsdevws
Copy link
Contributor

@nhsdevws nhsdevws commented Jul 9, 2025

Summary

  • Routine Change
  • ❗ Breaking Change
  • 🤖 Operational or Infrastructure Change
  • ✨ New Feature
  • ⚠️ Potential issues that might be caused by this change

Add any other relevant notes or explanations here. Remove this line if you have nothing to add.

Reviews Required

  • Dev
  • Test
  • Tech Author
  • Product Owner

Review Checklist

ℹ️ This section is to be filled in by the reviewer.

  • I have reviewed the changes in this PR and they fill all or part of the acceptance criteria of the ticket, and the code is in a mergeable state.
  • If there were infrastructure, operational, or build changes, I have made sure there is sufficient evidence that the changes will work.
  • I have ensured the changelog has been updated by the submitter, if necessary.

@nhsdevws nhsdevws force-pushed the VED-355-Mesh-Terraform-Mailbox branch from f232199 to b8fa16c Compare July 14, 2025 18:18
@nhsdevws nhsdevws changed the title DRAFT Ved 355 mesh terraform mailbox VED-355 mesh terraform mailbox Jul 14, 2025
@nhsdevws nhsdevws self-assigned this Jul 14, 2025
@nhsdevws nhsdevws marked this pull request as ready for review July 14, 2025 18:19
mfjarvis
mfjarvis previously approved these changes Jul 15, 2025
@sonarqubecloud
Copy link

@mfjarvis
Copy link
Contributor

Requires these SSM parameters which I don't have permission to create:

/imms-dev-mesh/mesh/MESH_CA_CERT
/imms-dev-mesh/mesh/MESH_CLIENT_CERT
/imms-dev-mesh/mesh/MESH_CLIENT_KEY
/imms-dev-mesh/mesh/MESH_SHARED_KEY
/imms-dev-mesh/mesh/MESH_URL
/imms-dev-mesh/mesh/MESH_VERIFY_SSL

/imms-dev-mesh/mesh/mailboxes/X26OT303/MAILBOX_PASSWORD

Waiting on Paul to create them, then will test.

@mfjarvis mfjarvis merged commit 890ead7 into master Jul 16, 2025
8 checks passed
@mfjarvis mfjarvis deleted the VED-355-Mesh-Terraform-Mailbox branch July 16, 2025 09:56
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