Skip to content

Bugfix/170 create temporary admin script hangs #175

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Sgu74 wants to merge 9 commits into
base: 5.1.8/develop
Choose a base branch
from
Open

Bugfix/170 create temporary admin script hangs #175

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
c2cd93b
#170 update makefiles
Sgu74 Jan 21, 2026
a8c9ae4
#170 use 5.1.8/main and 5.1.8/develop for releases
Sgu74 Jan 21, 2026
8b0ae65
#170 update dogu-build-lib
Sgu74 Jan 21, 2026
dffb157
#170 add write permission for the redmine group to the working directory
Sgu74 Jan 22, 2026
2a22045
#170 add function to retry rails script
Sgu74 Jan 22, 2026
07f14f8
#170 fix errors, kill hanging process and catch exit code from backgr…
Sgu74 Jan 23, 2026
48663c2
#170 add changelog and release notes; change time to retry to 180 sec…
Sgu74 Jan 23, 2026
2d3eae1
#170 set owner only to the file that needed it; improved message
Sgu74 Jan 26, 2026
1c977f5
#176 made retry timeout for rails scripts configurable
Sgu74 Jan 29, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
#176 made retry timeout for rails scripts configurable
  • Loading branch information
@Sgu74
Sgu74 committed Jan 29, 2026
commit 1c977f5ffc0fd4af2ec43361038067861cfe882c
6 changes: 6 additions & 0 deletions dogu.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,12 @@
"Name": "rack/bytesize_limit",
"Description": "Configure the maximum byte size of query strings that the redmine web server (rack) allows. The default is 4194304 (4 MiB). Depending on the available processing power setting this parameter to high might cause redmine to soft lock.",
"Optional": true
},
{
"Name": "rails_script_timeout",
"Description": "Define after how many seconds the execution of critical rails scripts in the startup process is terminated and tried again",
"Optional": true,
"Default": "180"
}
],
"Volumes": [
Expand Down
6 changes: 4 additions & 2 deletions resources/util.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -178,11 +178,12 @@ function create_temporary_admin() {
local TMP_ADMIN_RANDOMIZED_STR
TMP_ADMIN_RANDOMIZED_STR="$(doguctl random -l 60)"
TMP_ADMIN_PASSWORD="${TMP_ADMIN_RANDOMIZED_STR}${TMP_ADMIN_PASSWORD_SUFFIX}"
RAILS_TIMEOUT="$(doguctl config rails_script_timeout)"

# In case we are in restart loop to prevent infinite admin users...
remove_last_temporary_admin

railsConsoleRetryOnce 180 "/rails_scripts/create_admin.rb" --username "${TMP_ADMIN_NAME}" --password "${TMP_ADMIN_PASSWORD}" || exit 1
railsConsoleRetryOnce "${RAILS_TIMEOUT}" "/rails_scripts/create_admin.rb" --username "${TMP_ADMIN_NAME}" --password "${TMP_ADMIN_PASSWORD}" || exit 1
doguctl config -e "last_tmp_admin" "${TMP_ADMIN_NAME}"
}

Expand All @@ -194,12 +195,13 @@ function remove_last_temporary_admin() {
local DEFAULT="<empty>"
local LAST_TMP_ADMIN
LAST_TMP_ADMIN="$(doguctl config -e --default "${DEFAULT}" last_tmp_admin)"
RAILS_TIMEOUT="$(doguctl config rails_script_timeout)"

if [ "${LAST_TMP_ADMIN}" != "${DEFAULT}" ]
then
echo "Removing last temporary admin..."
# shellcheck disable=SC1091
railsConsoleRetryOnce 180 "/rails_scripts/remove_user.rb" --username "${LAST_TMP_ADMIN}" || exit 1
railsConsoleRetryOnce "${RAILS_TIMEOUT}" "/rails_scripts/remove_user.rb" --username "${LAST_TMP_ADMIN}" || exit 1
doguctl config --rm last_tmp_admin
fi
}
Expand Down