kured

kured copied to clipboard

slack-username is not respected any more.

❯ git grep -i "slack.*username"
README.md:      --slack-username string               slack username for reboot notfications (default "kured")
charts/kured/README.md:| `configuration.slackUsername` | cli-parameter `--slack-username`                                      | `""`                      |
charts/kured/templates/daemonset.yaml:          {{- if .Values.configuration.slackUsername }}
charts/kured/templates/daemonset.yaml:            - --slack-username={{ .Values.configuration.slackUsername }}
charts/kured/values.minikube.yaml:  # slackUsername: ""         # slack username for reboot notfications (default "kured")
charts/kured/values.yaml:  slackUsername: ""          # slack username for reboot notfications (default "kured")
cmd/kured/main.go:      slackUsername                   string
cmd/kured/main.go:      rootCmd.PersistentFlags().StringVar(&slackUsername, "slack-username", "kured",
cmd/kured/main.go:              "slack username for notifications")
kured-ds.yaml:#            - --slack-username=prod

kured on  main via 🐹 v1.16.12 
❯ 

This came up after #368 (along with #469).

I'm a bit unsure what to do. I feel that going with shoutrrr is the right thing to do, but some "old" setups won't work they used to any more.

Looks like channel is not respected either... at least it looks like it in our deployments 😕

@atighineanu Can you help?

This issue was automatically considered stale due to lack of activity. Please update it and/or join our slack channels to promote it, before it automatically closes (in 7 days).

We still have this issue.

• upgraded to chart version 1.12.1
• changed to --notify-url

slack-username is not respected. Message to slack is sendt as Kured.

After some testing I found that this works:

--notify-url=slack://myslackusername@token-a/token-b/token-c

Looks like --slack-username is deprecated too.