Skip to content

docs: fix the Quick Start health check step - #2

Merged
krlex merged 1 commit into
mainfrom
hardening-followup-2026-07
Jul 25, 2026
Merged

docs: fix the Quick Start health check step#2
krlex merged 1 commit into
mainfrom
hardening-followup-2026-07

Conversation

@krlex

@krlex krlex commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

The published Quick Start health check did not work as written. Found while validating the documented install path on 2026-07-23.

`./scripts/healthcheck-web.sh` cannot run on the host — the file is not
executable and its first command is `forail-manage`, which only exists inside
the backend image. The scripts under scripts/ are the container probes Compose
mounts at /etc/forail/ and runs there.

Point step 4 at `docker compose ps` and document the manual form,
`docker compose exec forail-web bash /etc/forail/healthcheck-web.sh`. Both
verified against a stack brought up from the documented .env.example.
@krlex
krlex merged commit 07c6724 into main Jul 25, 2026
1 check passed
krlex added a commit that referenced this pull request Jul 25, 2026
The Fixed section said in-cluster job execution "now works out of the box",
which contradicts Breaking changes #2 in the same document: forail-task ships
non-privileged, and project updates and control-plane jobs run through podman
inside that pod. Without the opt-in every job dies moments after launch with

  mount /var/lib/containers/storage/overlay: permission denied

and the UI shows nothing but a project or job stuck in Pending. The RBAC and
receptor worktype that section describes were genuinely missing and are
genuinely fixed -- they are just not the whole story, and a reader who stopped
there would install a Forail that cannot run anything.

Also set the release date to the day the images were actually published.
@krlex
krlex deleted the hardening-followup-2026-07 branch July 30, 2026 14:06
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