imbajin commented on code in PR #3349:
URL: https://github.com/apache/hugegraph/pull/3349#discussion_r4278923436


##########
helm/hugegraph/templates/tests/test-connection.yaml:
##########
@@ -16,6 +16,17 @@
 #
 
 {{ include "hugegraph.resolveValues" . }}
+{{- $smoke := `def ids = [] as Set;
+(graph.schema().getVertexLabels() + graph.schema().getEdgeLabels()).each {
+    ids.addAll(it.indexLabels())
+};
+ids.each {
+    def index = graph.indexLabel(it);
+    if (index.status().name() == 'INVALID') {

Review Comment:
   The hook now requires index.status().ok(), which accepts CREATED only, and 
reports the actual status for every rejected associated index. It still reads 
by ID and keeps the existing per-Pod sweep and retry deadline. The real Groovy 
fixture covers every non-CREATED status for vertex and edge indexes; shell 
fixtures cover stale CREATING and REBUILDING responses. README and operations 
guidance now say to run helm test after index jobs finish. Template, package, 
Kubernetes 1.23 schema, Groovy and shell checks, formatting and compilation 
pass; live stale-cache acceptance remains separate.



-- 
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]

Reply via email to