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

Replace or Remove Mattermost notification from Github Actions #22407

Closed
SDawley opened this issue Jul 27, 2023 · 4 comments
Closed

Replace or Remove Mattermost notification from Github Actions #22407

SDawley opened this issue Jul 27, 2023 · 4 comments
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. severity/P2 Has a minor but important impact to the usage or development of the system.

Comments

@SDawley
Copy link
Contributor

SDawley commented Jul 27, 2023

Is your task related to a problem? Please describe

Release actions are currently all failing at the Mattermost notification stage since mattermost is no more.

Example: https://github.com/eclipse/che/actions/runs/5682971423/job/15403670412

Describe the solution you'd like

For now, we should remove the stage from all relevant github actions. Once we decide on a replacement we can update them again.

Describe alternatives you've considered

No response

Additional context

No response

@SDawley SDawley added the kind/task Internal things, technical debt, and to-do tasks to be performed. label Jul 27, 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 Jul 27, 2023
@SDawley SDawley changed the title Replace Remove Mattermost notification from Github Actions Replace or Remove Mattermost notification from Github Actions Jul 27, 2023
@nickboldt
Copy link
Contributor

some ideas for possible replacements:

  • post to RH External Slack the same way we did to MM (presumably using a similar GH action + auth token, like https://github.com/marketplace/actions/slack-notify ?)
  • log events to che-release repo in a single replaced-with-every release release.log.txt file
  • log events to che-release repo, but put tag the repo at the start of the release, and put the release.log.txt in the 7.yy.x branch so we have unique histories for each release

@Kasturi1820 Kasturi1820 added 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 Jul 28, 2023
SDawley added a commit to che-incubator/che-code that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit to eclipse-che/che-machine-exec that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit to eclipse-che/che-dashboard that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit to eclipse-che/che-server that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit that referenced this issue Aug 17, 2023
#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit to eclipse-che/che-plugin-registry that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit to eclipse-che/che-devfile-registry that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit to eclipse-che/che-operator that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
tolusha pushed a commit to eclipse-che/che-operator that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
tolusha pushed a commit to eclipse-che/che-server that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
RomanNikitenko pushed a commit to che-incubator/che-code that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit to eclipse-che/che-release that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
SDawley added a commit to che-incubator/che-code that referenced this issue Aug 17, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
svor pushed a commit to eclipse-che/che-plugin-registry that referenced this issue Aug 21, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
svor pushed a commit to eclipse-che/che-devfile-registry that referenced this issue Aug 21, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
azatsarynnyy pushed a commit to che-incubator/che-code that referenced this issue Aug 22, 2023
eclipse-che/che#22407
Comment out the mattermost notification so it doesn't fail the release workflow, but leaving it in the file so I remember to replace it.
@SDawley
Copy link
Contributor Author

SDawley commented Sep 21, 2023

Update:
Planning on using https://github.com/slackapi/slack-github-action to replace mattermost. Testing using the #eclipse-che-ci channel in slack.

@SDawley
Copy link
Contributor Author

SDawley commented Sep 21, 2023

I've made a che-release-bot for the ECD-tools slack which I'll use to test notifications. If we also want to post to the redhat-internal slack we'll need to get that approved.

@che-bot
Copy link
Contributor

che-bot commented Mar 19, 2024

Issues go stale after 180 days of inactivity. lifecycle/stale issues rot after an additional 7 days of inactivity and eventually close.

Mark the issue as fresh with /remove-lifecycle stale in a new comment.

If this issue is safe to close now please do so.

Moderators: Add lifecycle/frozen label to avoid stale mode.

@che-bot che-bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 19, 2024
@che-bot che-bot closed this as completed Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. 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

4 participants