Skip to content

chore: change slack channel to helm doc upgrade notify workflow #642

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.

Sign up for GitHub

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

Merged
guimoreirar merged 9 commits into from
Aug 4, 2026
Merged

chore: change slack channel to helm doc upgrade notify workflow #642

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
5f7eac7
chore(release): backmerge main into develop [skip ci]
lerian-studio Aug 1, 2026
6d0d947
fix(helm-upgrade-doc): rename SLACK_CHANNEL_DEVOPS to SLACK_CHANNEL_OPS
guimoreirar Aug 4, 2026
66b7835
chore: change slack channel to helm doc upgrade notify workflow (#641)
guimoreirar Aug 4, 2026
78e97f1
fix(helm-upgrade-doc): use generic SLACK_CHANNEL secret for notificat…
guimoreirar Aug 4, 2026
4395d24
fix(helm-upgrade-doc): use generic SLACK_CHANNEL secret for notificat…
guimoreirar Aug 4, 2026
45a8964
docs(helm-upgrade-doc): update Slack channel secret to generic SLACK_…
guimoreirar Aug 4, 2026
600e758
docs(helm-upgrade-doc): clarify SLACK_CHANNEL naming under secrets: i…
guimoreirar Aug 4, 2026
79ea937
docs(helm-upgrade-doc): update Slack channel secret to generic SLACK_…
guimoreirar Aug 4, 2026
96084d8
Merge branch 'main' into develop
guimoreirar Aug 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions .github/workflows/helm-upgrade-doc.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,14 @@ name: "Helm Upgrade Doc"
# upgrade-doc:
# uses: LerianStudio/github-actions-shared-workflows/.github/workflows/helm-upgrade-doc.yml@v1
# secrets: inherit
#
# Slack channel: the notification channel is a GENERIC `SLACK_CHANNEL` secret, so
# switching channels never requires editing this shared workflow. With
# `secrets: inherit` it uses the caller's `SLACK_CHANNEL`. To point at a different
# channel, map any channel secret to it in the CALLER (instead of inherit):
# secrets:
# SLACK_CHANNEL: ${{ secrets.SLACK_CHANNEL_OPS }} # or any channel secret
# # ...plus the other secrets this workflow requires...
Comment thread
guimoreirar marked this conversation as resolved.

on:
workflow_call:
Expand Down Expand Up @@ -58,7 +66,7 @@ on:
required: false
SLACK_BOT_TOKEN_HELM:
required: false
SLACK_CHANNEL_DEVOPS:
SLACK_CHANNEL:
required: false
SLACK_GROUP_TECH_SUPPORT:
required: false
Expand Down Expand Up @@ -92,7 +100,7 @@ jobs:
anthropic-api-key: ${{ secrets.ANTHROPIC_API_KEY }}
openrouter-api-key: ${{ secrets.OPENROUTER_API_KEY }}
slack-bot-token: ${{ secrets.SLACK_BOT_TOKEN_HELM }}
slack-channel: ${{ secrets.SLACK_CHANNEL_DEVOPS }}
slack-channel: ${{ secrets.SLACK_CHANNEL }}
slack-group-ops: ${{ secrets.SLACK_GROUP_TECH_SUPPORT }}
charts-root: ${{ inputs.charts_root }}
docs-subdir: ${{ inputs.docs_subdir }}
Expand Down
10 changes: 9 additions & 1 deletion docs/helm-upgrade-doc.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,9 +106,17 @@ All secrets are passed via `secrets: inherit`. The caller repository must have t
| `ANTHROPIC_API_KEY` | No | Anthropic API key. When set, takes priority over OpenRouter |
| `OPENROUTER_API_KEY` | No | OpenRouter API key. Used when `ANTHROPIC_API_KEY` is not set |
| `SLACK_BOT_TOKEN_HELM` | No | Slack bot token for PR review notifications |
| `SLACK_CHANNEL_DEVOPS` | No | Slack channel ID to send notifications |
| `SLACK_CHANNEL` | No | Slack channel ID to send notifications. Generic — map any channel secret to it in the caller (e.g. `SLACK_CHANNEL: ${{ secrets.SLACK_CHANNEL_OPS }}`) |
| `SLACK_GROUP_TECH_SUPPORT` | No | Slack group ID to mention in notifications (e.g. ops team) |

> **Slack channel with `secrets: inherit`:** `inherit` matches by name — it does **not**
> rename `SLACK_CHANNEL_OPS` to `SLACK_CHANNEL`. The caller must have a secret named
> exactly `SLACK_CHANNEL`, otherwise `slack-channel` is empty and the Slack step is
> skipped. To target a channel whose secret has a different name, replace
> `secrets: inherit` with an explicit mapping listing every required secret plus
> `SLACK_CHANNEL: ${{ secrets.SLACK_CHANNEL_OPS }}` (you cannot combine `inherit`
> with an explicit `secrets:` block).

Comment thread
guimoreirar marked this conversation as resolved.
> **Note:** At least one of `ANTHROPIC_API_KEY` or `OPENROUTER_API_KEY` must be provided.

## Outputs
Expand Down
Loading