docs: enable PPL Query Builder by default, remove enablement docs - #387
Conversation
The PPL Query Builder feature flag (explore.logsQueryBuilder.enabled) is now enabled by default across the observability stack setup: - docker-compose OpenSearch Dashboards config - Helm values.yaml and values-anonymous-auth.yaml Since the builder ships on by default, remove the '## Enabling the builder' section and the availability note from the query-builder docs, and drop the 'how to enable it' references from the discover-logs page. Signed-off-by: joshuali925-osdbot <joshuali925-osdbot@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #387 +/- ##
=======================================
Coverage 55.62% 55.62%
=======================================
Files 4 4
Lines 169 169
Branches 47 47
=======================================
Hits 94 94
Misses 74 74
Partials 1 1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
this needs 3.8 or nightly before it would work |
|
Looking into the CI failures now. Both |
This reverts commit 8ecf8e9. Signed-off-by: joshuali925-osdbot <joshuali925-osdbot@users.noreply.github.com>
Fixed — reverted the premature enablement (
|
|
✅ CI is now green on Note this leaves the PR with an empty diff (branch is identical to Recommend either closing this PR and revisiting after the 3.8.0 upgrade, or repurposing it into a coordinated OpenSearch + OSD 3.8.0 version bump if that upgrade is wanted now. Deferring to maintainers on that call. |
Description
Follow-up to #383. Enables the PPL Query Builder feature flag by default across the observability stack setup and removes the now-unnecessary enablement instructions from the docs.
Enable the flag by default — added
explore.logsQueryBuilder.enabled: trueto:docker-compose/opensearch-dashboards/opensearch_dashboards.template.ymlcharts/observability-stack/values.yamlcharts/observability-stack/values-anonymous-auth.yamlThese configs already set
explore.enabled: true, so the builder is now on out of the box.Docs cleanup — since the builder ships enabled by default in this stack:
## Enabling the buildersection fromppl/query-builder.md:::note[Availability]block that told users to flip the flag (and its now-dangling#enabling-the-builderanchor link)investigate/discover-logs.mdTesting
npm run build). The link validator reports the same 41 pre-existing invalid links (16 files) on bothmainand this branch — this change introduces no new broken links, and specifically leaves no dangling#enabling-the-builderanchor.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.