Skip to content
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.

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

Send notification after each completed run #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

moschlar
Copy link
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
Copy link
Collaborator

djmaze commented Feb 27, 2024

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

@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
Copy link
Collaborator

djmaze commented Feb 24, 2025

@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

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

@djmaze
Copy link
Collaborator

djmaze commented Feb 26, 2025

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

@virtualex-itv
Copy link

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