This repository was archived by the owner on May 12, 2026. It is now read-only.
[OB-2811] Disable field_caps to improve performance#1
Open
pbasgod7 wants to merge 10 commits into
Open
Conversation
- Disabled getFields() method that calls _field_caps API - Returns empty array to skip autocomplete feature - Fixes performance issues with indices containing many dynamic fields - Related to quickwit-oss/quickwit-datasource#172
- Builds patched Quickwit datasource plugin - Removes signature files for patched version - Builds Grafana Docker image with patched plugin - Pushes to ECR repository: grafana-quickwit - Triggers on push to disable-field-caps-all-fields branch or tags
- Uses gha-runner-ecr-publish runner with existing AWS credentials - Follows Backstage pattern for ECR authentication - No need for AWS_ROLE_ARN secret - Builds on push to branches or PR merge - Manual workflow dispatch available
- Removed 'latest' tag for production safety - Uses git tag if available (e.g., v0.6.0-patched-1) - Falls back to SHA-based tag for untagged commits - Matches best practice for prod/preprod deployments
- Removed unused FieldCapabilitiesResponse import - Removed unused fieldTypeMap import - Both were only needed for field_caps functionality
- Change from gha-runner-ecr-publish to ubuntu-latest - Add AWS OIDC authentication with role assumption - Requires AWS_ROLE_ARN secret to be configured Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This reverts commit 5fe7b04.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The Quickwit Grafana datasource plugin's
_field_capsAPI calls cause severe performance issues on indices with many dynamic fields:Related: quickwit-oss/quickwit-datasource#172
Solution
This PR patches the plugin to disable
_field_capscalls entirely by modifying thegetFields()method to return an empty array.Tradeoff: Field name autocomplete is disabled. Users must manually type field names.
Changes
src/datasource/base.tsto disable field_capsTesting
✅ Local testing confirmed:
_field_capsrequestsCI/CD
gha-runner-ecr-publish(no secrets needed)grafana-quickwit12.4.0-quickwit-0.6.0-patched-<hash>Deployment
ECR repository already created: iterable-infra PR #15204
Related