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

EREGCSC-2902 - add app context for removal #1500

Merged
merged 2 commits into from
Dec 12, 2024
Merged

Conversation

addis-samtek
Copy link
Contributor

@addis-samtek addis-samtek commented Dec 12, 2024

Resolves # https://jiraent.cms.gov/browse/EREGCSC-2902

Description
Adds deployment workflow for CDK infrastructure, targeting the dev environment upon merge to main branch.

Changes

  • Configures sequential deployment of Lambda functions in specific order:
    1. ZIP-based Lambdas (Redirect & Maintenance APIs)
    2. Text Extractor
    3. FR Parser
    4. ECFR Parser
  • Uses matrix strategy for environment management
  • Implements proper dependency chain between deployments
  • Configures OIDC authentication for AWS

Verification Steps

  1. Merge PR to main branch
  2. Verify in GitHub Actions that workflow triggers automatically
  3. Confirm all stacks deploy successfully in order
  4. Verify in AWS Console that all resources are created with proper configurations

Note: Initial deployment targets dev environment only. Future environments can be added by extending the environment matrix.
This Resolves pipeline failure for removal of ephemeral env by adding app context to cdk remove

Copy link

✨ See the Django Site in action

Copy link

✨ See the Django Site in action

Copy link
Contributor

@PhilR8 PhilR8 left a comment

Choose a reason for hiding this comment

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

LGTM

@addis-samtek addis-samtek merged commit d2d5e04 into main Dec 12, 2024
25 checks passed
@brittag brittag changed the title add app context for removal EREGCSC-2902 - add app context for removal Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants