Skip to content

chore: remove the migrated dogfood template sources - #29469

Merged
jdomeracki-coder merged 8 commits into
mainfrom
jd/remove-dogfood-template-sources
Sep 18, 2026
Merged

jdomeracki-coder merged 8 commits into
mainfrom
jd/remove-dogfood-template-sources

Conversation

@jdomeracki-coder

@jdomeracki-coder jdomeracki-coder commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Deletes the dogfood template sources that were migrated to the infrastructure repository, leaving only the image build inputs in this repo.

Stacked on #29400 (which removes the deploy_template job and dogfood/main.tf); depends on coder/dogfood#551 being applied first. #29468 stacks on top of this branch.

Changes

  • Delete dogfood/coder/main.tf, dogfood/coder-envbuilder/, dogfood/vscode-coder/main.tf, and the template READMEs and guide
  • Delete dogfood/coder/devcontainer.json: its only reader was the retired coder-envbuilder template, and it referenced a Dockerfile removed in chore: upgrade to ubuntu 26.04 #24267
  • Add dogfood/README.md documenting what remains here and where template changes go
  • Drop the now-empty dependabot entries (/dogfood/coder-envbuilder docker, dogfood/*/ terraform)

Kept: dogfood/coder/ubuntu-22.04/, dogfood/coder/ubuntu-26.04/, dogfood/coder/Makefile, and dogfood/vscode-coder/Dockerfile. Image builds, tags, and scripts/dogfood/* are unchanged; the contract with the templates is the published Docker tag.

Checks: actionlint, shellcheck, compute-base-sha.sh/compute-final-sha.sh still resolve, docs lint and make lint/emdash clean.


Generated by Coder Agents on behalf of @jdomeracki-coder.

Template provisioning now lives in the infrastructure repository. The dogfood
workflow keeps building and publishing the dogfood images.

Generated by Coder Agents.
The dogfood workspace templates are now provisioned from the infrastructure
repository. Keep only the image build inputs here and document the split.

Generated by Coder Agents.
@jdomeracki-coder jdomeracki-coder self-assigned this Sep 17, 2026
@jdomeracki-coder
jdomeracki-coder marked this pull request as ready for review September 17, 2026 09:33
@coderagents

coderagents Bot commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Documentation Check

Updates Needed

  • docs/about/contributing/backend.md (line 80) - The repository tour says the dogfood directory holds the "Terraform definition of the dogfood cluster deployment." This PR deletes all Terraform sources (dogfood/coder/main.tf, dogfood/coder-envbuilder/, dogfood/vscode-coder/main.tf), leaving only image build inputs and the new dogfood/README.md. The entry should describe the directory accordingly (e.g., the build inputs for Coder's dogfood workspace images, with templates living in the Coder infrastructure repository).

Automated review via Coder Agents

Comment thread dogfood/README.md Resolved
@@ -52,7 +52,7 @@ fi
# Helper: run a make target inside the image.
#
# Mounts /home/coder/ as a single named volume to mirror the dogfood
# workspace template (dogfood/coder/main.tf), so caches (Go modules,
# workspace template, so caches (Go modules,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: looks a bit awkward without reflowing the rest of the comment

Base automatically changed from jd/remove-dogfood-template-workflow to main September 18, 2026 07:19
@github-actions

github-actions Bot commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Docs preview

Check off each page once it's been reviewed. If a page changes in a later push, its checkbox clears automatically so it gets a fresh look. Pages not yet wired into the docs navigation aren't listed here.

@jdomeracki-coder
jdomeracki-coder merged commit 95a3f01 into main Sep 18, 2026
32 checks passed
@jdomeracki-coder
jdomeracki-coder deleted the jd/remove-dogfood-template-sources branch September 18, 2026 07:55
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants