Skip to content

IPS-2209 topic updated to slack notification#2692

Merged
Rkotyank merged 4 commits into
mainfrom
IPS-2209-core-build-notification-alerting-stack
Apr 28, 2026
Merged

IPS-2209 topic updated to slack notification#2692
Rkotyank merged 4 commits into
mainfrom
IPS-2209-core-build-notification-alerting-stack

Conversation

@Rkotyank
Copy link
Copy Markdown
Contributor

@Rkotyank Rkotyank commented Mar 24, 2026

Proposed changes

What changed

Replace all custom topics with the slack notification topics.

Why did it change

So we are only using a managed stack that deploys topics.

Issue tracking

@Rkotyank Rkotyank marked this pull request as ready for review April 21, 2026 14:57
@Rkotyank Rkotyank requested review from a team as code owners April 21, 2026 14:57
@sonarqubecloud
Copy link
Copy Markdown

gary-pickup
gary-pickup previously approved these changes Apr 22, 2026
Copy link
Copy Markdown

@gary-pickup gary-pickup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Template has been updated replacing topics with imported slack notifications. Approving.

Comment thread deploy/template.yaml Outdated
@sonarqubecloud
Copy link
Copy Markdown

@Rkotyank Rkotyank merged commit f70c139 into main Apr 28, 2026
8 checks passed
@Rkotyank Rkotyank deleted the IPS-2209-core-build-notification-alerting-stack branch April 28, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants