Skip to content

fix: Nomad service deploys actually schedule + pull images - #14

Merged
pipozzz merged 1 commit into
mainfrom
fix/nomad-deploy-pulls
Sep 2, 2026
Merged

pipozzz merged 1 commit into
mainfrom
fix/nomad-deploy-pulls

Conversation

@pipozzz

@pipozzz pipozzz commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Found by deploying a service through the UI on a live server. Two blockers, both fixed + verified live (nginx compose → scheduled on Nomad → container running → curl HTTP 200):

  1. nomad CLI missing in the app image — the deploy runs nomad job run inside the container to submit to the control plane's Nomad, but there was no nomad binary (nomad: not found), so nothing scheduled and the deploy falsely reported success. Now installed in the Dockerfile.
  2. Docker driver couldn't pull any image — the Nomad docker plugin's auth.config = /root/.docker/config.json pointed at a missing file, so every pull failed (even public nginx). install.sh + bootstrap now create an empty {"auths":{}} config before Nomad starts.

🤖 Generated with Claude Code

Found by deploying a service through the UI on the live server. Two blockers:

1. nomad CLI missing in the app image. The deploy pipeline runs 'nomad job run'
   to submit to the control plane's own Nomad, but the container had no nomad
   binary -> '/bin/sh: nomad: not found', so nothing was scheduled (and the
   deploy falsely reported success). Install the Nomad CLI in the Dockerfile.

2. Docker driver couldn't pull ANY image. The Nomad docker plugin sets
   auth.config = /root/.docker/config.json, but that file didn't exist, so the
   driver failed every pull ('Failed to open auth config file'), even public
   nginx. install.sh and the bootstrap now create an empty {"auths":{}} config
   before Nomad starts (docker login fills it for private registries later).

Verified live: after these, a compose (nginx) deployed via the UI schedules on
Nomad, the container runs, and curl returns HTTP 200.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@pipozzz
pipozzz merged commit 9e0c945 into main Sep 2, 2026
@pipozzz
pipozzz deleted the fix/nomad-deploy-pulls branch September 2, 2026 09:10
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Thank you for your contribution! Please sign our Contributor License Agreement by posting the following comment:


I have read the CLA Document and I hereby sign the CLA


Peter Gonda seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@github-actions github-actions Bot locked and limited conversation to collaborators Sep 2, 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.

1 participant