slack-username not respected

[dholbach]

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.

[Starefossen]

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

[dholbach]

@atighineanu Can you help?

[github-actions]

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).

[bengtfredh]

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.

[bengtfredh]

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.