Skip to content

Fix undefined variable LOADED_MODULE_NAME in disable_rds.sh and disable_tipc.sh#322

Open
Denflorr wants to merge 2 commits into
ovh:masterfrom
Denflorr:patch-1
Open

Fix undefined variable LOADED_MODULE_NAME in disable_rds.sh and disable_tipc.sh#322
Denflorr wants to merge 2 commits into
ovh:masterfrom
Denflorr:patch-1

Conversation

@Denflorr

@Denflorr Denflorr commented Apr 3, 2026

Copy link
Copy Markdown

This PR fixes a bug in disable_rds.sh and disable_tipc.sh where the script references an undefined variable: LOADED_MODULE_NAME.

When running the script, the following error occurs:
disable_rds.sh: line 60: LOADED_MODULE_NAME: unbound variable
disable_tipc.sh: line 60: LOADED_MODULE_NAME: unbound variable

The correct variable used in the script is MODULE_NAME. This change replaces LOADED_MODULE_NAME with MODULE_NAME to resolve the issue.

This fix allows the script to run without triggering an unbound variable error.

Denflorr added 2 commits April 3, 2026 11:34
…_rds.sh

This fixes a bug in disable_rds.sh where the script references an undefined variable: LOADED_MODULE_NAME.

When running the script, this results in the following error:
disable_rds.sh: line 60: LOADED_MODULE_NAME: unbound variable

The correct variable available in the script is MODULE_NAME. This change replaces LOADED_MODULE_NAME with MODULE_NAME to fix the issue.

This allows the script to run correctly without triggering an unbound variable error.
…_tipc.sh

This fixes a bug in disable_tipc.sh where the script references an undefined variable: LOADED_MODULE_NAME.

When running the script, this results in the following error:
disable_tipc.sh: line 60: LOADED_MODULE_NAME: unbound variable

The correct variable available in the script is MODULE_NAME. This change replaces LOADED_MODULE_NAME with MODULE_NAME to fix the issue.

This allows the script to run correctly without triggering an unbound variable error.
@george-petrakis

Copy link
Copy Markdown

@Denflorr fixed this issue in all the files it occurs, #323

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.

2 participants