Skip to content

fix: set Rome env vars for SSM#1532

Open
kevinkim-ogp wants to merge 1 commit intotrunk/ai-actionsfrom
fix/rome-env-vars
Open

fix: set Rome env vars for SSM#1532
kevinkim-ogp wants to merge 1 commit intotrunk/ai-actionsfrom
fix/rome-env-vars

Conversation

@kevinkim-ogp
Copy link
Copy Markdown
Contributor

@kevinkim-ogp kevinkim-ogp commented Apr 8, 2026

TL;DR

Replaced generic PAIR Rome API keys with service-specific keys for AI Builder and Pair Action components.

What changed?

Removed the generic PAIR_ROME_PUBLIC_KEY and PAIR_ROME_SECRET_KEY environment variables and added four new service-specific environment variables:

  • PAIR_ROME_AI_BUILDER_PUBLIC_KEY and PAIR_ROME_AI_BUILDER_SECRET_KEY for AI Builder service authentication
  • PAIR_ROME_PAIR_ACTION_PUBLIC_KEY and PAIR_ROME_PAIR_ACTION_SECRET_KEY for Pair Action service authentication

How to test?

  1. Deploy the updated ECS configuration
  2. Verify that AI Builder and Pair Action services can authenticate successfully with their respective API keys
  3. Confirm that no services are attempting to use the removed generic PAIR Rome keys

Post-deployment

  • Delete plumber-pair-rome-public-key and plumber-pair-rome-secret-key from parameter store

Copy link
Copy Markdown
Contributor Author

kevinkim-ogp commented Apr 8, 2026

@kevinkim-ogp kevinkim-ogp marked this pull request as ready for review April 8, 2026 16:51
@kevinkim-ogp kevinkim-ogp requested a review from a team as a code owner April 8, 2026 16:51
@kevinkim-ogp kevinkim-ogp changed the title fix: set Rome env vars for SMS fix: set Rome env vars for SSM Apr 8, 2026
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.

1 participant