-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for DSC 2.2 Versions #81
Comments
I've got the same problem. I think simillar issue was reported here: #80 and here: SO: EC2 AMI 2.4 Issue and fixed (temporarily) here: DataStax: AMI 2.4 error. Maybe it will be good to allow to specify custom dsc and cassandra version in AMI params? |
Proposed fix committed to dev-2.5 and dev-2.6. Will test today and release today. |
In response to the comments @mshuler, we already do major.minor detection if a version is passed in. Our current default is to install the "latest" from stable if nothing is passed in (which means things break when the package name of "latest" changes). I'm open to PR's that improve the way we handle the default package versioning... but it's currently moderately messy and I don't have cycles to go back and clean it up. A semi-related open-issue is parsing dependency versions (we currently manually maintain lists of all the versions in our source-code), which is in #77 |
Pushed. Tested with 2.2.0, 2.1.8, and no release supplied: --clustername test1 --clustername test2 --clustername test3 |
It appears the current AMI only attempts to install dsc21 and fails on cassandra version 2.2.0 existing in the "stable" repo.
Better yet, the DSC deb repositories have been split into major.minor repos ("stable" will still be around for a while with all versions):
Perhaps, the major.minor could be used from passed-in --release x.x.x and sources.list prepared accordingly, which will also prevent future unintended major.minor upgrades. A default could be selected as the current "production" major.minor, such as 2.1 today.
The text was updated successfully, but these errors were encountered: