CASSANDRA-21530 Remove duplicate build scripts, point AGENTS.md to existing project build scripts - #4951
Open
michaelsembwever wants to merge 5 commits into
Open
Conversation
michaelsembwever
force-pushed
the
mck/simplify-agents.md/trunk
branch
2 times, most recently
from
July 20, 2026 21:35
008128d to
5845d28
Compare
michaelsembwever
changed the title
michaelsembwever
force-pushed
the
mck/simplify-agents.md/trunk
branch
2 times, most recently
from
August 22, 2026 16:06
5da0231 to
8ecba31
Compare
michaelsembwever
force-pushed
the
mck/simplify-agents.md/trunk
branch
from
August 22, 2026 17:28
8ecba31 to
a86c75c
Compare
smiklosovic
approved these changes
Aug 24, 2026
Deletes .build/sh/ai-build, .build/sh/ai-ci-test, .build/sh/ci-test and .build/sh/ci-test-loop. AGENTS.md now points to the pre-existing .build/*.sh scripts. Adds --summary to build-jars.sh, check-code.sh and run-tests.sh, and --clean to build-jars.sh. patch by Mick Semb Wever; reviewed by xxx for CASSANDRA-21530 Assisted-by: Claude Code:claude-opus-5
Defines error() before the pre-conditions that call it. Corrects the chunk regexp, as bash does not match \d. Pins generate-test-report to DIST_DIR. The default output directory reports a false pass when DIST_DIR is overridden. Prints a failed line when a microbench target fails, as JMH produces no JUnit xml for the report to read.
Adds .github/workflows/stub-ant-cases.sh, which runs the wrapper scripts against a stub ant, and .github/workflows/test_log_summary.py, which covers both log summarizers. The new build-scripts.yaml workflow runs both suites, shellcheck and dash -n. Makes ant's exit code authoritative in summary mode. A pipeline reported the summarizer's status, and ant can die without printing BUILD FAILED. Keeps errexit inside _main when run-tests.sh runs with --summary, and writes the full output to build/run-tests.log. Extracts run_ant() into .build/sh/_run-ant.sh, shared by build-jars.sh and check-code.sh. Accepts long flags on run-tests.sh and .build/docker/run-tests.sh, and makes the java version argument of _docker_run.sh optional.
…sult
run-ci decided completion from the result field alone. A pipeline latches
its result as soon as one stage publishes a failed test, while the other
stages continue, so the script reported UNSTABLE at 32 minutes on a build
that ran for 82. Completion now needs building to be false as well.
Reports the mid-build latch as a progress line, and warns after ten
consecutive Jenkins API failures. The failure was visible only with DEBUG
set, and an unreachable Jenkins polled in silence.
Reads the retained build info for the final line, rather than fetching it
again. The second call raised KeyError on the exception path.
Corrects the off-by-one in the three download retry loops, which left the
raise unreachable and swallowed a missing artifact. That produced the
misleading "No tests were run (or missing summary file)" line. Extracts
retry_call, shared by all three.
Tests the value and not the key of a queue item's executable field. A
pending item can carry a null, which gave a TypeError.
Interpolates logfile in the Summary stage call to copyToNightlies. Single
quotes sent a literal ${logfile} to the nightlies publisher, so that one
stage log never reached nightlies.apache.org.
michaelsembwever
force-pushed
the
mck/simplify-agents.md/trunk
branch
from
August 24, 2026 12:55
a86c75c to
7717a3e
Compare
…n-tests.sh -a cqlsh-test 11, the form the .build/README.md documents for build scripts) forwards -a cqlsh-test -c 1/1 11 to the inner run-tests.sh, which rejects the trailing 11 as an invalid option.
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
2 participants
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.
Adds summary and clean flags where necessary.