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

Trigger slack notifications from chectl release action #22551

Closed
nickboldt opened this issue Sep 25, 2023 · 6 comments
Closed

Trigger slack notifications from chectl release action #22551

nickboldt opened this issue Sep 25, 2023 · 6 comments
Assignees
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@nickboldt
Copy link
Contributor

nickboldt commented Sep 25, 2023

Describe the solution you'd like

replace https://github.com/eclipse-che/che-release/blob/main/.github/workflows/release-send-mattermost-announcement.yml with an action that notifies: ECD slack channel #forum-che-release that a release is done.

WIP in https://github.com/eclipse-che/che-release/blob/addSlack/.github/workflows/release-send-slack-announcement.yml

Message should be short, with link to release notes for that version.

Ths should be triggered when a new chectl release is completed, from https://github.com/che-incubator/chectl/blob/main/.github/workflows/release-announce.yml (but could also be triggered by workflow)

Additional context

Rather than writing a bot/webhook/app, using built in slack notifications work pretty well.

https://github.com/integrations/slack#configuration

See also:

@nickboldt nickboldt added the kind/task Internal things, technical debt, and to-do tasks to be performed. label Sep 25, 2023
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Sep 25, 2023
@nickboldt nickboldt changed the title https://github.com/che-incubator/chectl/blob/main/.github/workflows/release.yml trigger slack notifications from chectl release action Sep 25, 2023
@l0rd l0rd added area/ci CI build and releases, PR testing, & whitelabel/productization issues severity/P2 Has a minor but important impact to the usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Sep 25, 2023
@l0rd l0rd changed the title trigger slack notifications from chectl release action Trigger slack notifications from chectl release action Sep 25, 2023
@nickboldt
Copy link
Contributor Author

nothing showing up in https://app.slack.com/client/TMU5A34SW/C05SD64M85R yet

asked for slack integration to be added to the che-incubator repos (starting with chectl) so we can at least watch for events there.

@nickboldt
Copy link
Contributor Author

PR notification working:
image
and updates automatically as tests pass/fail too:
image

@nickboldt
Copy link
Contributor Author

nickboldt commented Oct 13, 2023

image

working: issues, pulls, releases, branches/tags, issues:

/github subscribe che-incubator/chectl pulls

  • includes link to the PR, repo, and tests; slack message updates automatically as changes happen / workflows run

/github subscribe che-incubator/chectl releases

/github subscribe che-incubator/chectl branches

  • includes link to the branch/tag and repo on creation (and a notification if deleted too)

/github subscribe eclipse/che issues

  • includes a link to the issue, repo and threaded comments under it (if comments enabled); otherwise just creation and closure are reported

also working: comments and deployments (probably overkill / too verbose):

/github subscribe eclipse/che issues comments

  • includes a link to the issue, repo and threaded comments under it; also creation and closure are reported

/github subscribe che-incubator/chectl deployments

  • includes link to the action that deployed the gh page

Everything is working including workflow notifications:

image

@nickboldt
Copy link
Contributor Author

nickboldt commented Oct 13, 2023

Events to track (currently set up in the #forum-che-release channel on ECD slack:

/github subscribe che-incubator/chectl releases workflows:{event:"workflow_dispatch","push" branch:"main"}  
/github unsubscribe che-incubator/chectl issues pulls commits deployments branches

/github subscribe eclipse-che/che-operator workflows:{event:"workflow_dispatch","push" branch:"main"}
/github unsubscribe eclipse-che/che-operator issues pulls commits releases deployments branches

/github subscribe eclipse-che/che-release workflows:{event:"workflow_dispatch","push" branch:"main"}
/github unsubscribe eclipse-che/che-release issues pulls commits releases deployments branches

/github subscribe eclipse/che issues releases +label:'status/need-triage'
/github unsubscribe eclipse/che pulls commits deployments branches

/github subscribe list features

Events to track (currently set up in the #forum-che channel on ECD slack:

/github subscribe eclipse/che issues releases +label:'status/need-triage'
/github unsubscribe eclipse/che pulls commits deployments branches

/github subscribe list features

README updated at https://github.com/eclipse-che/che-release/blob/main/README.md#notifications

@ibuziuk
Copy link
Member

ibuziuk commented Nov 8, 2023

@nickboldt can we close?

@nickboldt
Copy link
Contributor Author

if you're happy with the outcome here, then yes. Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci CI build and releases, PR testing, & whitelabel/productization issues kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Development

No branches or pull requests

5 participants