ci: trim Rocky Linux dependencies and preserve test diagnostics - #6
Open
3for wants to merge 14 commits into
Open
ci: trim Rocky Linux dependencies and preserve test diagnostics#63for wants to merge 14 commits into
3for wants to merge 14 commits into
Conversation
- Remove Development Tools and unnecessary packages - Install only JDK 8, git-core and zstd with weak dependencies disabled - Use C.utf8 to avoid installing glibc-langpack-en - Resolve and validate JAVA_HOME so Gradle no longer requires which
- Collect tron-test.log, rotated logs and JUnit XML across five PR build jobs - Run artifact uploads even when preceding steps fail - Use distinct artifact names per job and matrix configuration - Retain artifacts for 7 days and warn when no files are found
Remove the stale integration-test-multinode.yml reference to avoid unnecessary API requests when cancelling workflows for closed PRs.
- Save build, RocksDB test and coverage output with tee and plain console mode - Use Bash pipefail to preserve failures when capturing stdout and stderr - Include console logs and HTML test reports in diagnostic artifacts - Preserve existing test retry and base coverage failure policies
Remove the test-retry plugin and retry configuration shared by test and testWithRocksDb so test failures fail the task without retrying.
- Clear the thread-local VM snapshot after constant calls in TransferToAccountTest - Use OS-assigned ports for Prometheus tests to avoid conflicts between test JVMs - Ensure BackupServer closes channels bound after shutdown begins - Add regression coverage for VM state cleanup and shutdown during binding
- Limit diagnostic artifacts to **/logs/tron-test.log and ci-logs/*.log - Upload logs only when a preceding step fails - Include base test failures tolerated by continue-on-error - Keep the existing 7-day retention period
- Clear thread-local VM configuration after each RuntimeImplTest case - Stop background block production in ShieldedReceiveTest to preserve synthetic Merkle roots in pending sessions - Keep the producer stopped after the manual block production test
- Set DNF max_parallel_downloads to 10 for Rocky dependency installation - Restore the test-retry plugin and configuration as comments, keeping retries disabled
- Avoid fixed ports in event queue tests and let Jetty allocate its port - Verify ZeroMQ delivery with bounded waits and reliable cleanup - Propagate worker exceptions and validate concurrent toString results - Shut down reward test executors before closing stores - Close HTTP clients and responses and propagate request failures - Correct lite-node history query coverage and assertions
- Restore VM configuration and static rate limiter state after tests - Remove global output redirection and URL handler changes from servlet tests - Verify async task results and service startup and shutdown futures - Close network services, channels and application contexts on failure - Replace timing-sensitive assertions with controlled clocks and signals - Isolate temporary files and avoid fixed listening ports - Fix unbounded waiting in the disabled concurrent benchmark - Add regression coverage for setup and cleanup failures
- Skip the intentional failure fixture during automatic test discovery while preserving explicit VM state restoration checks - Split transaction handler tests into deterministic submission, queue capacity, and invalid transaction scenarios - Preserve unexpected exception details and strengthen assertions - Complete mock dependency setup and executor cleanup
- Isolate block message handler tests with explicit rejection and routing checks - Wait for producer completion before finishing concurrent filter consumption - Bound worker waits, propagate failures, and clean up test resources - Preserve original exception causes in test failures and temporary directory setup
- Use independent event plugin loaders and restore the original singleton - Restore VM configuration and ensure context and Args cleanup on failure - Assert block metric deltas and unchanged error counts against test baselines - Preserve the cause of temporary directory creation failures
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
1 participant
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.
What does this PR do?
Why are these changes required?
The Rocky Linux job installs unnecessary development tools and utilities, increasing dependency downloads and setup work. Test logs and results should remain available for diagnosing failed runs. The cancellation list also contains a stale reference to the removed multinode workflow.
This PR has been tested by: