Add option to test notification

[stepdg]

Since notify.sh is dependent on values passed along by the main script you cannot set up notify.sh and run it alone to test notification will work. I set it up a few days ago and sure enough I haven't had any image updates. I suppose I could force a lower version then switch to :latest but I still think this would be a nice option. -o or some other switch.

Disclaimer: Developer knows I am a dummy dumb so this may be way off the rails.

[mag37]

I've been thinking of this too, I think it's reasonable (even if you're nuts 😅).

Though my thoughts was to just make a minor test script that calls the notification with dummy data.

And sometimes when I just want to see output I modify the line at 405, adding a ! (not equal):

## from
    if [[ "$LocalHash" = *"$RegHash"* ]]; then
## to
    if [[ "$LocalHash" != *"$RegHash"* ]]; then

That flips the logic partly and show the ones with no updates as they've got updates.

[stepdg]

Solid idea!

[mag37]

This is a rough one, but does the trick:

templateTester.sh

#!/usr/bin/env bash

PrintReleaseURL=true
PrintMarkdownUrl=false
GotUpdates=( "dummy-one" "caddy" "dozzle" "homer" "mqtt" )
UrlsList="urls.list"

ScriptArgs=( "$@" )
ScriptPath="$(readlink -f "$0")"
ScriptWorkDir="$(dirname "$ScriptPath")"
NotifyTemplate=${ScriptArgs[0]:="notify.sh"}
Notify=true

printHelp() {
  printf "Use with specifying filename of notification template (need to be in same directory)\n"
  printf "Example: ./templateTester.sh notify_test1.sh\n"
  printf "Optionally also have urls.list in the same directory for testing.\n"
  exit 1
}

releasenotes() {
  unset Updates
  for update in "${GotUpdates[@]}"; do
    found=false
    while read -r container url; do
      if [[ "$update" == "$container" ]] && [[ "$PrintMarkdownURL" == true ]]; then Updates+=("- [$update]($url)"); found=true;
      elif [[ "$update" == "$container" ]]; then Updates+=("$update  ->  $url"); found=true;
      fi
    done < "${ScriptWorkDir}/${UrlsList}"
    if [[ "$found" == false ]] && [[ "$PrintMarkdownURL" == true ]]; then Updates+=("- $update  ->  url missing");
    elif [[ "$found" == false ]]; then Updates+=("$update  ->  url missing");
    else continue;
    fi
  done
}

[[ "${ScriptArgs[0]}" == "-h" ]] && printHelp

if [[ -s "${ScriptWorkDir}/${NotifyTemplate}" ]]; then
  source "${ScriptWorkDir}/${NotifyTemplate}"
else printHelp
fi

if [[ -n ${GotUpdates[*]:-} ]]; then
  printf "\n--- Dummy containers ---\n"
  if [[ -s "${ScriptWorkDir}/${UrlsList}" ]] && [[ "$PrintReleaseURL" == true ]]; then releasenotes; else Updates=("${GotUpdates[@]}"); fi
  printf "%s\n" "${Updates[@]}"
  [[ "$Notify" == true ]] && { type -t send_notification &>/dev/null && send_notification "${GotUpdates[@]}" || printf "\nCould not source notification function.\n"; }
fi

[mag37]

Though there might be some upcoming changes to notifications (see #156) so I will hold on adding this to main for a bit.

[vorezal]

I apparently unintentionally threw a bit of a wrench in this. But while I was working on notify_v2 I was running a lot of tests and added some logic temporarily that skipped all of the container checks. Maybe we could use that in combination with populating dummy containers and notify_v2 to create a flag that does this though? There are a lot of flags already, but we could treat it like a dry-run/dry-run with dummy images. Perhaps -b/-B for bench-test/bench-test with dummy images and update notifications? They could also be used together with -u to perform a kind of self-update only execution.

Does any of that sound useful or am I just complicating things unnecessarily to solve problems that don't exist? Alternatively I think all we would have to do is switch the NotifyTemplate variable to pull notify_v2.sh, either by default or as a fallback, to get this to work with the new notification feature mostly as-is.

[mag37]

Haha well, no problem. I think due to the wrapper it would be easier/cleaner to implement it now anyway.

A dry-run kind of functionality sounds very reasonable. Maybe it doesn't need dummy containers but just use all local containers from something like

docker ps --format '{{.Names}}'

But force-skip any container checks and update logic.

[yoyoma2]

I like the dry-run concept like tailscale --dry-run. Go through the motions exactly but don't actually update anything when it comes to that step.

You know how -e can prevent updating specific containers that need it. How about an opposite option to fake specific container(s) as having a need to update. Useful for testing with dry-run when no updates are pending.