Repository navigation
Task 6: Remote state - #12
Closed
FPurichaya wants to merge 1 commit into
Closed
FPurichaya wants to merge 1 commit into
FPurichaya wants to merge 1 commit into
Conversation
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
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.
Task
Task 6: Remote state
What changed
.nvmrcand the README prerequisite line match that version.apps/api/Dockerfileuse the official imagenode:24.22.1-bookworm-slim, pinned to the manifest-list digestsha256:55fb5f5a527ea2c5c4a95f963e48ef90aa38c7b346054fdc8ee9957a2b55d205.docker buildx imagetools inspect node:24.22.1-bookworm-slimreports media typeapplication/vnd.oci.image.index.v1+jsonand includeslinux/amd64andlinux/arm64.node src/index.jsas thenodeuser withNODE_ENV=production, includessql/001_counters.sql, and healthchecksGET /healthwith Node's built-infetch.How to verify
imagetools inspectdigest:sha256:55fb5f5a527ea2c5c4a95f963e48ef90aa38c7b346054fdc8ee9957a2b55d205(OCI image index).docker historyfor the final stage, newest first:docker inspect:User=node,Cmd=["node","src/index.js"]. The environment containsNODE_ENV=productionandNODE_VERSION=24.22.1. It does not containDATABASE_URL.Inside the image: uid 1000, Node v24.22.1,
sql/001_counters.sqlpresent. No.gitor.envfile.docker run --rm devops-task-api:task1exits 1 withMissing required environment variable DATABASE_URL.GET /healthstays unhealthy until Postgres and Valkey are wired in Task 3. The healthcheck does not callPOST /api/clicks.Security checklist
latestimage tags and no floating action tags (@v4,@main)permissionsare set explicitly and are no broader than the job needsapps/behavior is unchanged, except operational configuration this task allows.terraform.lock.hclis committed0.0.0.0/0on the EKS API or the web load balancerExceptions
Use the API image Node.js 24.22.1 as it is the current patch on the 24 LTS line.