Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Fix matrix rooms #7143

Merged
merged 10 commits into from
Jul 28, 2023
Merged

Fix matrix rooms #7143

merged 10 commits into from
Jul 28, 2023

Conversation

chevdor
Copy link
Contributor

@chevdor chevdor commented Apr 27, 2023 •
edited
Loading

This PR switches rooms/accounts from the legacy matrix server to the new one.
This PR introduces also matrices to allow sending notifications to several rooms as required.

Todo

  • Switch to the new token(s)
    - [ ] ensure POLKADOT_DEVOPS_MATRIX_ACCESS_TOKEN is set and correct
    - [ ] ensure the bot behind POLKADOT_DEVOPS_MATRIX_ACCESS_TOKEN is in the Help Desk: Infrastructure (DevOps) room
  • ensure RELEASENOTES_MATRIX_V2_ACCESS_TOKEN is set
  • find bot name behind RELEASENOTES_MATRIX_V2_ACCESS_TOKEN
  • ensure the bot behind RELEASENOTES_MATRIX_V2_ACCESS_TOKEN in the following rooms:
    • RelEng: Polkadot Release Coordination
    • #KusamaValidatorLounge:polkadot.builders
    • #kusama-announcements:matrix.parity.io\
    • #polkadotvalidatorlounge:web3.foundation
    • #polkadot-announcements:matrix.parity.io
    • Ledger <> Polkadot Coordination
  • rename MATRIX_V2_ACCESS_TOKEN into RELEASENOTES_MATRIX_V2_ACCESS_TOKEN
  • cleanup WHERE we send notifications
  • add notif to General: Rust, Polkadot, Substrate (#engineering-general:parity.io)

After merge

  • delete any remaining secret such as MATRIX_ACCESS_TOKEN

cc @wpank @lazam @jam206

@chevdor chevdor requested a review from a team April 27, 2023 16:25
@chevdor chevdor added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes labels Apr 27, 2023
@EgorPopelyaev EgorPopelyaev reopened this Apr 28, 2023
@chevdor
Copy link
Contributor Author

chevdor commented May 4, 2023

Notification have failed since last month starting withe 0.9.41:
https://github.com/paritytech/polkadot/actions/workflows/release-99_bot.yml

@chevdor chevdor marked this pull request as ready for review July 25, 2023 08:24
@chevdor chevdor requested a review from a team as a code owner July 25, 2023 08:24
@chevdor chevdor changed the title Fix matrix rooms [DNM] Fix matrix rooms Jul 25, 2023
@chevdor chevdor mentioned this pull request Jul 25, 2023
Merged
7 tasks
@chevdor chevdor requested review from lazam and wabkebab July 26, 2023 13:23
@chevdor chevdor changed the title [DNM] Fix matrix rooms Fix matrix rooms Jul 26, 2023
@chevdor chevdor added the A0-please_review Pull request needs code review. label Jul 26, 2023
@chevdor
Copy link
Contributor Author

chevdor commented Jul 26, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@chevdor
Copy link
Contributor Author

chevdor commented Jul 28, 2023

bot merge

@paritytech-processbot paritytech-processbot bot merged commit e54e2e6 into master Jul 28, 2023
4 checks passed
@paritytech-processbot paritytech-processbot bot deleted the wk-230427-fix-matrix branch July 28, 2023 13:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@altaua altaua altaua approved these changes

@coderobe coderobe coderobe approved these changes

@lazam lazam lazam approved these changes

@EgorPopelyaev EgorPopelyaev EgorPopelyaev approved these changes

@wabkebab wabkebab Awaiting requested review from wabkebab

Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@chevdor @EgorPopelyaev @lazam @coderobe @altaua