Skip to content

Commit

Permalink
Bugfix: Messaging Clarity (#146)
Browse files Browse the repository at this point in the history
* Bad wording in plextrac stop command

* Updated migration container message to be more clear

* Missing space
  • Loading branch information
Michael-Burke authored Jun 26, 2024
1 parent 3db9462 commit 21525e0
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion src/_manage_postgres.sh
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,8 @@ function mod_check_etl_status() {
local migration_exited=$(docker inspect --format '{{.State.Status}}' `docker ps -a | grep migrations 2>/dev/null | awk '{print $1}'` || migration_exited="exited")
fi
if [ $(date +%s) -gt $endTime ]; then
die "Migration container has been running for over 5 minutes or is still running. Exiting..."
error "Migration container has been running for over 5 minutes or is still running. Please ensure they complete or fail before taking further action with the PlexTrac Manager Utility. You can check on the logs by running 'docker compose logs -f couchbase-migrations'"
die "Exiting PlexTrac Manager Utility."
fi
if [ "$CONTAINER_RUNTIME" == "podman" ]; then
for s in / - \\ \|; do printf "\r\033[K$s $(podman inspect --format '{{.State.Status}}' migrations) -- $(podman logs migrations 2> /dev/null | tail -n 1 -q)"; sleep .1; done
Expand Down
2 changes: 1 addition & 1 deletion src/_stop.sh
Original file line number Diff line number Diff line change
Expand Up @@ -31,5 +31,5 @@ function mod_stop() {
compose_client stop
fi
info "-----"
info "PlexTrac stopped. It's now safe to update and restart"
info "PlexTrac stopped. It's now safe to update the OS and restart"
}

0 comments on commit 21525e0

Please sign in to comment.