Skip to content

Fix parsing of java_versions_supported in bin/cassandra.in.sh#803

Open
michaelsembwever wants to merge 1 commit intoapache:trunkfrom
thelastpickle:mck/18831
Open

Fix parsing of java_versions_supported in bin/cassandra.in.sh#803
michaelsembwever wants to merge 1 commit intoapache:trunkfrom
thelastpickle:mck/18831

Conversation

@michaelsembwever
Copy link
Copy Markdown
Member

patch by Mick Semb Wever; reviewed by xxx for CASSANDRA-18831

pull_request:
workflow_dispatch:
schedule:
- cron: '0 0 * * 0'
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

adding weekly build of default (trunk) branch, so we catch such failures a bit faster next time

@michaelsembwever
Copy link
Copy Markdown
Member Author

still hitting an issue with the smoke test on 4.0.19 using python2.17

Add extra debug (both stderr and stdout) from a cluster/node when it fails to start.
Downgrade the python2.7 GHA CI to jdk8 to avoid the cgroup NPE.
And run CI once a week to catch trunk failures earlier.
Remove DSE testing in CI as the downloads are no longer available (without click-through).

 patch by Mick Semb Wever; reviewed by xxx for CASSANDRA-18831
@michaelsembwever
Copy link
Copy Markdown
Member Author

ready for review @driftx

@michaelsembwever michaelsembwever marked this pull request as ready for review March 15, 2026 18:46
Copy link
Copy Markdown
Contributor

@driftx driftx left a comment

Choose a reason for hiding this comment

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

Ticket numbers seems to wrong, but lgtm.

@michaelsembwever
Copy link
Copy Markdown
Member Author

michaelsembwever commented Mar 16, 2026

Ticket numbers seems to wrong,

the jdk21 patch broke the parsing of parsing of java_versions_supported in bin/cassandra.in.sh

and the fix (primarily) in ccmlib/common.py is so small and trivial, it made sense to just use the existing ticket 🤷

@michaelsembwever
Copy link
Copy Markdown
Member Author

merged with dfa4e82

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.

2 participants