Skip to content
This repository was archived by the owner on Apr 7, 2026. It is now read-only.

add launchpad token action to get tokens - #91

Merged
jamesfwood merged 2 commits into
developfrom
feature/launchpad_tokens
Oct 1, 2025
Merged

add launchpad token action to get tokens#91
jamesfwood merged 2 commits into
developfrom
feature/launchpad_tokens

Conversation

@sliu008

@sliu008 sliu008 commented Oct 1, 2025

Copy link
Copy Markdown
Contributor

This pull request introduces a new reusable GitHub Action for retrieving Launchpad tokens via AWS Lambda and integrates it into the umm_t_sync workflow. The workflow now dynamically fetches tokens for both OPS and UAT environments instead of relying on static secrets. The most important changes are:

New GitHub Action for Token Retrieval:

  • Added .github/workflows/launchpad-token/action.yml, which defines a composite action to obtain a Launchpad token by invoking an AWS Lambda function. This action configures AWS credentials, calls the Lambda, processes the response, and outputs the token securely.

Integration into Workflow:

  • Updated .github/workflows/umm_t_sync.yml to use the new launchpad-token action for both OPS and UAT environments, replacing the use of static secrets with dynamic token retrieval.

Environment Variable Updates:

  • Modified the environment variables in the Run cmr_association_diff command step to use the dynamically retrieved tokens from the previous steps, increasing security and flexibility.

Token Usage in Commands:

  • Updated the cmr_association_diff command invocations to use the new environment variables for tokens instead of referencing the old secrets directly.

@sliu008
sliu008 requested a review from jamesfwood October 1, 2025 20:39
@jamesfwood
jamesfwood merged commit dd65853 into develop Oct 1, 2025
3 checks passed
@jamesfwood
jamesfwood deleted the feature/launchpad_tokens branch October 1, 2025 21:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants