Skip to content

Improve IT cluster readiness diagnostics#17903

Open
Caideyipi wants to merge 2 commits into
apache:masterfrom
Caideyipi:ci-cluster-readiness-diagnostics
Open

Improve IT cluster readiness diagnostics#17903
Caideyipi wants to merge 2 commits into
apache:masterfrom
Caideyipi:ci-cluster-readiness-diagnostics

Conversation

@Caideyipi

@Caideyipi Caideyipi commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR improves integration-test cluster readiness handling and raises the pipe workflow cluster-ready retry window.

  • make the default cluster ready retry count configurable through integrationTest.clusterReadyRetryCount, while keeping the framework default at 30
  • set -DintegrationTest.clusterReadyRetryCount=90 for all Maven IT invocations in .github/workflows/pipe-it.yml
  • include cluster readiness failure details in the final assertion message, including node sizes, last node status, process status, last exception, and log directories
  • dump node jstacks when cluster status or JDBC readiness checks exhaust retries
  • record per-DataNode JDBC connection failures during readiness checks
  • clear MultiEnvFactory environments before creating a new multi-env set to avoid static list growth across test methods

Tests

  • mvn spotless:apply -pl integration-test -P with-integration-tests -DskipTests
  • mvn -Ddevelocity.off=true spotless:check -pl integration-test -P with-integration-tests -DskipTests
  • git diff --check -- integration-test/src/main/java/org/apache/iotdb/it/env/cluster/env/AbstractEnv.java integration-test/src/main/java/org/apache/iotdb/it/env/MultiEnvFactory.java
  • git diff --check -- .github/workflows/pipe-it.yml
  • mvn -Ddevelocity.off=true compile -pl integration-test -P with-integration-tests -DskipTests reached BUILD SUCCESS; the local JVM then exited with a native memory/pagefile error after build completion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant