Skip to content

Make SLA calculation of a Finding Group use the most severe finding #7102

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

Merged
merged 1 commit into from
Nov 9, 2022

Conversation

Maffooch
Copy link
Contributor

@Maffooch Maffooch commented Nov 9, 2022

When using finding groups within a test, the SLA calculation for groups passes the entire group to the SLA calculation for a single finding. There is an expected mismatch there. This PR takes the most severe finding I the group and passes that finding the SLA calculation function so that the group has the fastest approaching SLA.

When using finding groups within a test, the SLA calculation for groups passes the entire group to the SLA calculation for a single finding. There is an expected mismatch there. This PR takes the most severe finding I the group and passes that finding the SLA calculation function so that the group has the fastest approaching SLA.
@Maffooch Maffooch added the bugfix label Nov 9, 2022
@github-actions github-actions bot added the ui label Nov 9, 2022
Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

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

Approved

@mtesauro
Copy link
Contributor

mtesauro commented Nov 9, 2022

@Maffooch Looks like there's something deprecated in our Helm Lint GH Action setup :-(

@Maffooch
Copy link
Contributor Author

Maffooch commented Nov 9, 2022

Yes, Helm chart unit testing has bee pretty broken for a few months now :/ I change it slightly each release in hopes of finding the right fix, but no dice.

This error happens because the target branch is not dev, and the problematic helm test runs and (expectedly) fails :/

@Maffooch Maffooch merged commit c785aee into release/2.16.1 Nov 9, 2022
@Maffooch Maffooch deleted the Maffooch-patch-2 branch November 9, 2022 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants