Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

fix: Alertmanager cloudevent payload #353

Merged
merged 5 commits into from
Aug 9, 2022

Conversation

TannerGabriel
Copy link
Member

This PR

  • Fix bug where the remediation-service is unable to find action for problem type default
  • Changes remediation.triggered cloud event payload to match the remediation service

…n-service

Signed-off-by: TannerGabriel <gabrieltanner.code@gmail.com>
@github-actions
Copy link
Contributor

github-actions bot commented Aug 9, 2022

Unit Test Results

12 tests   11 ✔️  0s ⏱️
  6 suites    1 💤
  1 files      0

Results for commit 5fbbd21.

♻️ This comment has been updated with latest results.

Signed-off-by: TannerGabriel <gabrieltanner.code@gmail.com>
Signed-off-by: TannerGabriel <gabrieltanner.code@gmail.com>
Signed-off-by: TannerGabriel <gabrieltanner.code@gmail.com>
@TannerGabriel TannerGabriel marked this pull request as ready for review August 9, 2022 13:39
Signed-off-by: TannerGabriel <gabrieltanner.code@gmail.com>
@TannerGabriel TannerGabriel requested a review from Raffy23 August 9, 2022 14:01
Copy link
Member

@Raffy23 Raffy23 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link
Contributor

github-actions bot commented Aug 9, 2022

The following Docker Images have been built:
keptncontrib/prometheus-service:0.8.5-dev-PR-353,keptncontrib/prometheus-service:0.8.5-dev-PR-353.202208091401

@TannerGabriel TannerGabriel merged commit 55caac4 into master Aug 9, 2022
@TannerGabriel TannerGabriel deleted the fix/alertmanager-cloudevent-payload branch August 9, 2022 14:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants