Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: compute alerts on systematic failure #4

Merged
merged 7 commits into from
Apr 24, 2023
Merged

Conversation

jsilvela
Copy link
Contributor

@jsilvela jsilvela commented Apr 10, 2023

Improvements:

  • stops showing the ignoreFail as a code error
  • creates a short summary for cases where the full summary would be truncated by GH
  • computes an Alerts section, makes it available standalone for use in Slackops
  • adds unit tests

Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>

review: handle overflow

Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
@NiccoloFei NiccoloFei merged commit d4314a5 into main Apr 24, 2023
@NiccoloFei NiccoloFei deleted the dev/add-alerts branch April 24, 2023 12:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants