Skip to content

Send notification after each completed run#120

Open
moschlar wants to merge 1 commit into
masterfrom
notify-after-each-run
Open

Send notification after each completed run#120
moschlar wants to merge 1 commit into
masterfrom
notify-after-each-run

Conversation

@moschlar

Copy link
Copy Markdown
Collaborator

Uses apprise sidecar url, if configured, to send an "info" notification after each run of the update_services loop.

Closes: #95

Uses apprise sidecar url, if configured, to send an "info" notification
after each run of the update_services loop.

Closes: #95
@djmaze

djmaze commented Feb 27, 2024

Copy link
Copy Markdown
Collaborator

Sorry for the late response. Not sure if everyone would want this. Maybe we could make it optional via an env variable?

@virtualex-itv

Copy link
Copy Markdown

@djmaze - i think this would be a great addition as a an env variable. Additionally, i think you should include at least an initial notification just to confirm that it is up and running and properly connected to the messaging endpoint...just like watchtower does right now. Watchtower sends a quick notification so why not have Shepherd do the same?

@djmaze

djmaze commented Feb 24, 2025

Copy link
Copy Markdown
Collaborator

@virtualex-itv Watchtower sends an initial notification? Upon start of the container? Or upon every update cycle? I guess the first one does not really have much value because you won't know if it is suddenly not working anymore.

@virtualex-itv

Copy link
Copy Markdown

@djmaze - correct. Upon initial start up post-deployment and every time there's an image updated.

@djmaze

djmaze commented Feb 26, 2025

Copy link
Copy Markdown
Collaborator

Yeah, but then let's better put it in front of an additional env variable. Something like NOTIFY_ON_UPDATE.

@virtualex-itv

Copy link
Copy Markdown

That would work too. Some type of option is better than no option at all. 🦾

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Request: Notification on completion

3 participants