bitflicker64 commented on PR #3132: URL: https://github.com/apache/hugegraph/pull/3132#issuecomment-5463700664
Pushed three commits: schema and template fixes (short JWT token guard, test hook resources, optional image.digest), helm-unittest suites (40 tests), and README corrections. The last push adds the ASF license headers the test files were missing, which is what turned the license and build jobs red. Remaining README work I did not fold in, listed so it is not lost. All of it exists in the helm-dev README and can be ported with small wording changes: - Local Kind/minikube section (build images locally, kind load, pullPolicy). Needs one substantive change here: this branch defaults to pullPolicy Always, so `--set <component>.image.pullPolicy=Never` is mandatory with locally loaded images, not optional as the helm-dev text implies. - Pre-install cluster checks: `kubectl config current-context`, `kubectl get nodes`, `kubectl get storageclass`, plus a note that the default topology needs 3 PD and 3 Store PVCs. Prevents the most common first-run failure (PVCs stuck Pending) before it happens. - `--wait --timeout 15m` on the primary install command. - A Hubble row in the Chart Details component table. Two small errors already fixed in the docs commit: the Limitations note said the Disaster Recovery section is below when it is above, and the pinning advice said three component tags when there are four (Hubble also tracks latest). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
