Skip to content

[docker] Fix ERLANG_EXITS typo in healthcheck script - #3130

Open
jacky-w-j-li wants to merge 1 commit into
signalwire:masterfrom
jacky-w-j-li:fix-healthcheck-erlang-typo
Open

jacky-w-j-li wants to merge 1 commit into
signalwire:masterfrom
jacky-w-j-li:fix-healthcheck-erlang-typo

Conversation

@jacky-w-j-li

@jacky-w-j-li jacky-w-j-li commented Aug 23, 2026

Copy link
Copy Markdown

The Docker healthcheck script docker/base_image/healthcheck.sh contains a variable name typo: ERLANG_EXITS should be ERLANG_EXIST.

Root cause

Line 35 assigns the variable as ERLANG_EXITS, and line 37 references it as $ERLANG_EXITS. While the script is internally consistent (the same misspelled name is used in both places, so the script works), the name is clearly intended to be ERLANG_EXIST to match the sibling variable KAZOO_EXIST on line 34, and to reflect the actual meaning (checking whether the module exists).

Fix

Rename ERLANG_EXITS to ERLANG_EXIST in both the assignment (line 35) and the conditional check (line 37).

Testing

  • Verified the script logic is unchanged: the variable is renamed consistently in both the assignment and the usage.
  • No behavioral change — the script was already working because the same (misspelled) name was used throughout.

Resolves: #3126

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.

Typo in variable name: ERLANG_EXITS should be ERLANG_EXIST

1 participant