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

re-enable free-text searching on alert groups #2232

Open
joeyorlando opened this issue Jun 14, 2023 · 9 comments · May be fixed by #4287
Open

re-enable free-text searching on alert groups #2232

joeyorlando opened this issue Jun 14, 2023 · 9 comments · May be fixed by #4287

Comments

@joeyorlando
Copy link
Collaborator

#2098 and #2101 temporarily disabled it.

We should re-enable searching here, but ensure that the proper indices are added and that it works with large amounts of alert groups.

@github-actions
Copy link
Contributor

The current version of Grafana OnCall, at the time this issue was opened, is v1.2.43. If your issue pertains to an older version of Grafana OnCall, please be sure to list it in the PR description. Thank you 😄!

@fantom0005
Copy link

Hello @joeyorlando @Matvey-Kuk

I hope this message finds you well. Could I kindly inquire if there is an estimated timeline for addressing the issue at hand?

It appears that the search functionality, which seems to be a fundamental feature, has been non-operational for over a year.

I was wondering if, pending the resolution of the performance issues, it might be feasible to introduce a setting that allows those who require the search function to enable it temporarily.

Thank you

@joeyorlando
Copy link
Collaborator Author

this is not on the core team's short term roadmap. I provided a workaround here to have this feature enabled under a feature flag. If you're keen to open a pull request we can certainly review 🙂

@EZ4BRUCE
Copy link

this is not on the core team's short term roadmap. I provided a workaround here to have this feature enabled under a feature flag. If you're keen to open a pull request we can certainly review 🙂

thanks!

@fantom0005
Copy link

this is not on the core team's short term roadmap. I provided a workaround here to have this feature enabled under a feature flag. If you're keen to open a pull request we can certainly review 🙂

@joeyorlando
I tried to include a search for your suggestion in the API. Thanks, this works in the API.
But the plugin itself (frontend) pass the search field only to “stats” API and the list of Incidents are not filtered.

image

@fantom0005
Copy link

The plugin replace the entered value to null for the "search" field in this place.
https://github.com/grafana/oncall/blob/dev/grafana-plugin/src/models/alertgroup/alertgroup.ts#L62

@sreway
Copy link
Contributor

sreway commented Apr 25, 2024

@fantom0005 thanks, I renamed the search field and the search started working

@marganellim
Copy link

@sreway Can you share your solution?

@sreway sreway linked a pull request Apr 26, 2024 that will close this issue
3 tasks
@shev06
Copy link

shev06 commented May 15, 2024

this is not on the core team's short term roadmap. I provided a workaround here to have this feature enabled under a feature flag. If you're keen to open a pull request we can certainly review 🙂

@joeyorlando I tried to include a search for your suggestion in the API. Thanks, this works in the API. But the plugin itself (frontend) pass the search field only to “stats” API and the list of Incidents are not filtered.

image

Same issue

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

Successfully merging a pull request may close this issue.

9 participants