Repository navigation
Conversation
- add bilingual installation and access instructions - document strict defaults and explicit overrides - cover internal images, persistent storage and GitOps - explain upgrade, credentials and startup troubleshooting
- name the HStore Server image in both inventories - link the current topology and rollout constraints - clarify the limits of disk loss recovery guidance
- show PD REST service credentials and Secret usage - separate partition patrol from manual leader balance - explain Store replacement and compression behavior
- retain simple PD credential examples - explain shell history and process argument exposure - link optional private credential and payload handling
- document index-state checks and their coverage limits - explain business assertions and execution credentials - link the full operations example
- add Helm pages to the shared navigation tree - align per-Pod index checks with completed index jobs - update native navigation validation expectations
- preserve upstream deployment and foundation guides - retain Helm routes and shared navigation entries - recalculate native navigation validation expectations
- describe the existing 150-second internal retry window - distinguish the hook deadline from Helm command timeout - keep private HTTP access with an optional PD port-forward path
- require completed index jobs and stable schema during checks - explain persistent HStore cache state and recovery timing - remove the claim that retries settle cached index work
- identify the chart needed for index readiness checks - clarify that older test success does not verify indexes - keep bilingual assertion examples on a valid source link
- preserve the merged ASF documentation baseline - allow the org review to show only the compatibility fix - retain the existing Helm documentation branch unchanged # Conflicts: # content/cn/docs/quickstart/hugegraph/hugegraph-helm-operations.md # content/cn/docs/quickstart/hugegraph/hugegraph-helm.md # content/en/docs/quickstart/hugegraph/hugegraph-helm-operations.md # content/en/docs/quickstart/hugegraph/hugegraph-helm.md
This was referenced Oct 15, 2026
Member
Author
|
Closing per request: Helm updates already merged into master in PR #527; any minor improvements will be folded into existing PRs without opening standalone PRs. |
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Older Helm charts can pass
helm testwithout proving that associated indexes are ready. The bilingual deployment and operations guides now identify the chart required for index-state checks andserver.testGremlin, and the assertion example links point to a source revision that contains the section.Documentation link checks and
git diff --checkpass. The source example URL was checked against GitHub. This is a prose/link correction; no local service or cluster was started.The org review is stacked on
codex/helm-easy-defaultsto show only this follow-up. The ASF PR targets master, where the original guides are already merged.Paired review: hugegraph#51