-
Notifications
You must be signed in to change notification settings - Fork 272
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
Search doesn't work in Alert Groups page #3030
Comments
+1 |
For our setup, the search function never adapted table with the search value. |
We're going to update this part of the product (including search) soon: #2853 |
Is there any estimate on when the next rev will be released? The broken search is causing alot of headache with a client of ours. |
Search is still not working with version 1.3.44 |
Still looking to see if there's an update with this issue. |
Can confirm, search still broken in v1.3.86. From the code, the relevant search function is implemented here:
However, looking at the MySQL log, no CAST is actually sent as part of the SQL query, which means that maybe the search request parameter is not passed to the API as expected. (Django casts should be done on the DB-side).
|
Bumping this issue. Is there any resolution coming to the search function? |
This was temporarily disabled in #2101 for performance reasons. We are open to pull requests to conditionally enable this functionality under a feature flag! Something like this should get you started:
class AlertGroupView:
...
search_fields = ["=public_primary_key", "=inside_organization_number", "web_title_cache"] if settings.FEATURE_ENABLE_ALERT_GROUP_SEARCH else []
... |
@joeyorlando thanks for sharing the reason. |
@ToonTijtgat2 eventually yes! But in the mean time, we're open to reviewing pull requests 😄 (the code snippet above gets you 90% of the way there) |
closing as duplicate of #2232 |
Does the fact this has been closed as "not planned" indicate you have no plans to have a search function? Are you removing the search feature entirely? |
No plans to remove the search functionality, simply doing some cleanup on our issues and closing this as a duplicate. Feel free to continue the conversation over there! If you really would like to have this re-enabled, I proposed an potential approach for a PR in my comment above, that would allow users to opt-in to having this enabled (disabled by default). If you're willing to create a PR, I will help make sure it gets across the finish-line 😄 |
What went wrong?
What happened:
What did you expect to happen:
How do we reproduce it?
Grafana OnCall Version
1.3.36
Product Area
Alert Flow & Configuration
Grafana OnCall Platform?
Other
User's Browser?
Brave
Anything else to add?
No response
The text was updated successfully, but these errors were encountered: