Skip to content

fix: use archive.apache.org for Cassandra downloads#99

Closed
Kubuxu wants to merge 1 commit intomainfrom
fix/cassandra-install
Closed

fix: use archive.apache.org for Cassandra downloads#99
Kubuxu wants to merge 1 commit intomainfrom
fix/cassandra-install

Conversation

@Kubuxu
Copy link
Copy Markdown

@Kubuxu Kubuxu commented Mar 31, 2026

dlcdn.apache.org removes old patch versions when new ones are released,
which broke CI when 5.0.7 replaced 5.0.6. Switch to archive.apache.org
which retains all versions permanently. Bump to 5.0.7 and add SHA256
checksum verification.

dlcdn.apache.org removes old patch versions when new ones are released,
which broke CI when 5.0.7 replaced 5.0.6. Switch to archive.apache.org
which retains all versions permanently. Bump to 5.0.7 and add SHA256
checksum verification.
Copilot AI review requested due to automatic review settings March 31, 2026 12:09
@FilOzzy FilOzzy added this to FOC Mar 31, 2026
@github-project-automation github-project-automation bot moved this to 📌 Triage in FOC Mar 31, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the scenario prerequisites installer to fetch Apache Cassandra from the Apache archive (to avoid disappearing patch releases) and adds integrity verification for the downloaded tarball.

Changes:

  • Switch Cassandra download source from dlcdn.apache.org to archive.apache.org.
  • Bump Cassandra from 5.0.6 to 5.0.7.
  • Add SHA-256 checksum verification of the downloaded Cassandra tarball.

Comment on lines +30 to +35
CASSANDRA_VERSION="5.0.7"
CASSANDRA_SHA256="556be693f1941aeb8ec1538fe6224cbefdca7bc3729f87ff0e24a0052eb98c33"
PYTHON_VERSION="3.11.10"
PYENV_ROOT="${PYENV_ROOT:-$HOME/.pyenv}"
PYTHON_BIN="${PYENV_ROOT}/versions/${PYTHON_VERSION}/bin/python3"
CASSANDRA_URL="https://dlcdn.apache.org/cassandra/${CASSANDRA_VERSION}/apache-cassandra-${CASSANDRA_VERSION}-bin.tar.gz"
CASSANDRA_URL="https://archive.apache.org/dist/cassandra/${CASSANDRA_VERSION}/apache-cassandra-${CASSANDRA_VERSION}-bin.tar.gz"
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CASSANDRA_VERSION was bumped here to 5.0.7, but scenarios/test_caching_subsystem.py still hardcodes CASSANDRA_VERSION = "5.0.6" when constructing CASSANDRA_HOME, so it will fail to find cqlsh after this script installs 5.0.7. Update the scenario to match (or better, derive the version from a single source) so CI/dev runs stay consistent.

Copilot uses AI. Check for mistakes.
Comment on lines 136 to 139
TARBALL="${CASSANDRA_DIR}/apache-cassandra-${CASSANDRA_VERSION}-bin.tar.gz"
curl -fL -o "$TARBALL" "$CASSANDRA_URL"
verify_checksum "$TARBALL" "$CASSANDRA_SHA256"
tar -xzf "$TARBALL" -C "$CASSANDRA_DIR"
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new checksum verification relies on sha256sum, but this script doesn’t check that the command exists (and it’s not available by default on some platforms like macOS). Consider validating the tool up front and/or using a portable fallback (e.g., shasum -a 256 or openssl dgst -sha256) so the script fails with a clear message instead of a generic command-not-found error.

Copilot uses AI. Check for mistakes.
@Kubuxu Kubuxu closed this Mar 31, 2026
@github-project-automation github-project-automation bot moved this from 📌 Triage to 🎉 Done in FOC Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

3 participants