chore: Point stage deploy at the renamed ECS service - #755
Merged
Merged
Conversation
Incubator is renaming the stage service from civic-tech-jobs-fs-stage to civic-tech-jobs-fullstack-stage (hackforla/incubator#227). Without this, the redeploy step would target a service that no longer exists.
ale210
added a commit
to hackforla/incubator
that referenced
this pull request
Sep 13, 2026
Closes #227. Spells the civic-tech-jobs stage container's application type out as `fullstack` again, reversing `1f13c57`. The `fs` abbreviation existed only to fit the target group's 32-character limit, which the container module now handles itself (#231), so the service, log group, security group and IAM names can match the ECR repository and Postgres roles. **This apply has downtime.** An ECS service name forces replacement, so `stage.civictechjobs.org` goes down while the service is destroyed and recreated. What the plan should show, and nothing else: - ECS service, task definition, log group, security group, IAM role and task policy replaced, all renamed `civic-tech-jobs-fs-stage` → `civic-tech-jobs-fullstack-stage` - target group `civic-tech-jobs-fs-stage-145` → `ctj-fs-stage-145` as `+/-` (create before destroy) — the full name would be 35 characters, so the ladder drops to rung 2; listener rule 200 updated in place - **no change** to `module.civic_tech_jobs_stage_database` — it already uses `fullstack`, so its roles, passwords and SSM parameters must not appear. If they do, stop. Also updates the container module's header comment (and README) to say civic-tech-jobs now falls to rung 2 alongside civic-tech-index. **The CivicTechJobs deploy workflow names the old service** and is fixed in hackforla/CivicTechJobs#755. That only takes effect when their `develop` is next released to `main`, so their next release needs to be coordinated with this apply, or its redeploy step will target a service that does not exist.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related to hackforla/incubator#227 (no CivicTechJobs issue).
Changes
.github/workflows/deploy-stage.yml:SERVICE_NAMEcivic-tech-jobs-fs-stage→civic-tech-jobs-fullstack-stage. The DevOps CoP is renaming the stage ECS service in Undo the fs abbreviation in civic-tech-jobs incubator#236 so its name matches thecivic-tech-jobs-fullstackECR repository.Timing matters. This workflow only runs on a push to
main, so this change does nothing until the nextdevelop→mainrelease — and that release will be the workflow's first run. If a release reachesmainwithout this change after the incubator apply, or with it before the apply, the image build and push still succeed but the redeploy step fails because the service it names does not exist. Check with the DevOps CoP before releasing.Screenshots, if applicable
N/A — CI change only.