Skip to content

Add Kamal hook support#3

Open
mintyPT wants to merge 1 commit into
dvassallo:mainfrom
mintyPT:add-kamal-hook-support
Open

Add Kamal hook support#3
mintyPT wants to merge 1 commit into
dvassallo:mainfrom
mintyPT:add-kamal-hook-support

Conversation

@mintyPT

@mintyPT mintyPT commented Jul 26, 2026

Copy link
Copy Markdown

Summary:

  • detect known Kamal hook files under .kamal/hooks and pass generated executable copies via hooks_path
  • keep source hooks untouched so doctor does not see chmod-induced checkout dirtiness
  • cover hook behavior with unit tests, e2e ops assertions, and docs updates

Tests:

  • env GOCACHE=/tmp/singleserver-go-build go test ./...
  • bash -n internal/singleserver/deploy_script.sh test/e2e/run.sh test/e2e/lib/cases.sh test/e2e/lib/host.sh
  • env E2E_DISTROS=ubuntu E2E_CASES= E2E_COMMAND_COVERAGE=1 ./test/e2e/run.sh

E2E note:
I ran the Ubuntu e2e suite successfully with command coverage enabled. That run exercised the operational app scenario, detected .kamal/hooks/pre-deploy and .kamal/hooks/post-deploy, verified Single Server reported hooks=pre-deploy,post-deploy, and confirmed both hook marker files were written during deploy.

@mintyPT mintyPT closed this Jul 26, 2026
@mintyPT
mintyPT deleted the add-kamal-hook-support branch July 26, 2026 21:26
@mintyPT
mintyPT restored the add-kamal-hook-support branch July 26, 2026 21:32
@mintyPT mintyPT reopened this Jul 26, 2026
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