Skip to content

feat(cdk): Provision ECS infrastructure for tag page rendering CODE#16321

Draft
akash1810 wants to merge 3 commits into
mainfrom
aa/tag-rendering-ecs
Draft

feat(cdk): Provision ECS infrastructure for tag page rendering CODE#16321
akash1810 wants to merge 3 commits into
mainfrom
aa/tag-rendering-ecs

Conversation

@akash1810

@akash1810 akash1810 commented Jul 3, 2026

Copy link
Copy Markdown
Member

Note

Recommended to review commit by commit.

What does this change?

Why?

Screenshots

Before After
before after

@akash1810 akash1810 added the feature Departmental tracking: work on a new feature label Jul 3, 2026
Comment on lines +15 to +21
container:
permissions:
packages: write
needs: [production-container]
uses: ./.github/workflows/container.yml
with:
production-image-digest: ${{ needs.production-container.outputs.imageDigest }}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This container step is what runs cdk synth. This container is published to GitHub Packages and is used by the Commercial CI checks. I think there's an opportunity to simplify CI (as a separate task) by having the Commercial CI checks use the "production container". This would mean running cdk synth in a discrete step too, for improved readability.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

akash1810 added 2 commits July 3, 2026 09:49
Update tag page rendering CODE to provision ECS infrastructure but route no traffic to it.
@akash1810 akash1810 force-pushed the aa/tag-rendering-ecs branch from b010457 to d34b39d Compare July 3, 2026 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant