-
Bug
-
Resolution: Done
-
Normal
-
None
-
1.6.0
-
1
-
False
-
-
False
-
Release Note Not Required
-
-
-
RHDH Install 3278
Description of problem:
As caught in one of my recent review comments, the scripts used as entrypoints for the compose containers do not fail if there is an error, which can lead to unexpected behaviors.
- https://github.com/redhat-developer/rhdh-local/blob/main/prepare-and-install-dynamic-plugins.sh
- https://github.com/redhat-developer/rhdh-local/blob/main/wait-for-plugins-and-start.sh
They should fail sooner rather than later.
Prerequisites (if any, like setup, operators/versions):
Steps to Reproduce
- Add a call to a command that does not exist in the containers (like `yq`)
- Start the app with "podman compose up" or "docker compose up"
Actual results:
- The logs show that the command does not exist, but this doesn't prevent the containers from starting
[rhdh] | /opt/app-root/src/wait-for-plugins-and-start.sh: line 46: yq: command not found [rhdh] | Waiting for dynamic-plugins-root/app-config.dynamic-plugins.yaml to be created by install-dynamic-plugins container ... [rhdh] | Debugger listening on ws://0.0.0.0:9229/c019b082-c0d3-44a0-aa8a-e0985a5712ce [rhdh] | For help, see: https://nodejs.org/en/docs/inspector [rhdh] | 2025-06-24T12:29:24.064Z developer-hub-init info Starting Developer Hub backend [rhdh] | (node:1) [UNDICI-EHPA] Warning: EnvHttpProxyAgent is experimental, expect them to change at any time. [rhdh] | (Use `node --trace-warnings ...` to show where the warning was created) [rhdh] | Loading config from MergedConfigSource{FileConfigSource{path="/opt/app-root/src/app-config.yaml"}, FileConfigSource{path="/opt/app-root/src/app-config.example.yaml "}, FileConfigSource{path="/opt/app-root/src/app-config.example.production.yaml"}, FileConfigSource{path="/opt/app-root/src/dynamic-plugins-root/app-config.dynamic-plugins.yaml"}, FileConfigS ource{path="/opt/app-root/src/generated/app-config.patched.yaml"}, EnvConfigSource{count=0}} [...]
Expected results:
In case of any error (unless explicitly caught and handled in the script), the scripts should error out.
Reproducibility (Always/Intermittent/Only Once):
Always
