Skip to content

docs(deployment): pin image version in Kubernetes example#222

Open
andrerfneves wants to merge 1 commit into
mainfrom
maintenance/fix-k8s-example-image-tag-20260626
Open

docs(deployment): pin image version in Kubernetes example#222
andrerfneves wants to merge 1 commit into
mainfrom
maintenance/fix-k8s-example-image-tag-20260626

Conversation

@andrerfneves

Copy link
Copy Markdown
Owner

Summary

Pins the container image tag from :latest to a specific version in the Kubernetes Deployment manifest example in DEPLOYMENT_DOCKER.md.

Why

The example used image: lightning-decoder:latest, which is a well-known anti-pattern for Kubernetes manifests:

  • Non-reproducible deployments — different cluster operations (scaling, rolling update, pod restart) may pull different image versions
  • Unexpected version changes — a pod restart could silently upgrade to a new version without any manifest change
  • No audit trail:latest doesn't indicate which version is actually running

Pinning to 0.7.1 (the current project version) demonstrates the correct pattern while still being a simple, copyable example.

Changes

  • DEPLOYMENT_DOCKER.md: Changed image: lightning-decoder:latestimage: lightning-decoder:0.7.1 in the Kubernetes Deployment manifest

Test Plan

  • No behavior change — documentation-only fix
  • YAML syntax is valid
  • Version matches the current package.json version

Change image: lightning-decoder:latest to image: lightning-decoder:0.7.1
in the Kubernetes Deployment manifest example. Using :latest in
Kubernetes manifests is an anti-pattern because it makes deployments
non-reproducible and can cause unexpected version changes on pod
restarts. Pinning to a specific version tag follows container best
practices and is consistent with the project's versioning.
@vercel

vercel Bot commented Jun 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lightning-decoder Ready Ready Preview, Comment Jun 26, 2026 10:04pm

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9ba19784ef

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread DEPLOYMENT_DOCKER.md
containers:
- name: lightning-decoder
image: lightning-decoder:latest
image: lightning-decoder:0.7.1 # Update to match your built image version

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Tag the image before referencing it in Kubernetes

When readers follow this page, the documented build commands tag the image as lightning-decoder (which Docker treats as lightning-decoder:latest; Docker's image reference docs state, "If no tag is provided, Docker defaults to latest"). This manifest now references lightning-decoder:0.7.1, so applying it after the documented build steps will look for a tag the instructions never create or push, causing image pull failures unless the reader manually retags the image; either update the build/push examples to create :0.7.1 or keep this manifest aligned with the tag those steps produce.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant