Skip to content
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

Group alerts by owner #1808

Open
szaffarano opened this issue Feb 10, 2025 · 0 comments
Open

Group alerts by owner #1808

szaffarano opened this issue Feb 10, 2025 · 0 comments

Comments

@szaffarano
Copy link

szaffarano commented Feb 10, 2025

Is your feature request related to a problem? Please describe.

Our DWH has many teams developing models, and it would be nice to group alerts by owners so that each team is tagged once and has a summary of the alerts. Then, they can click on the report link to have a more detailed overview. Grouping by model is not enough since they are tagged many times, so they have to go through each message individually.

Describe the solution you'd like

The solution would be adding a new grouping option to the edr monitor command, --group-by owner, e.g.:

edr monitor \
  --slack-token ... \
  --slack-channel-name ... \
  --profile-target ... \
  --report-url ...
  --group-by owner \
  --test true

Describe alternatives you've considered

AFAIK, the current implementation does not allow what we want. Another option would be to add a new edr parameter allowing some sort of message templating so that the messages can be built outside edr, but this also needs to be added to edr, and I think it's a more complex solution.

Additional context

Proposed message:

Image

Where the View tests runs link opens the report selecting the tests filtering by owner. And See more shows the detail, grouped by model:

Image

Would you be willing to contribute this feature?

Absolutely!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant