-
Notifications
You must be signed in to change notification settings - Fork 129
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
Fix check gossiper generation #151
Conversation
vladzcloudius
commented
Feb 8, 2021
- Remove the dependency on the scylla.yaml contents.
- Check all cluster's nodes at once.
- Add the framework to provide parameters.
The script was pretty bogus as to requirements on the contents of scylla.yaml. Instead of trying to fix the above I simply rewrote it to check all nodes at once. Signed-off-by: Vlad Zolotarov <vladz@scylladb.com>
Signed-off-by: Vlad Zolotarov <vladz@scylladb.com>
* Calculate 'now' once instead of re-doing it in a loop. * Take the 'YEAR' assignment outside the loop too Signed-off-by: Vlad Zolotarov <vladz@scylladb.com>
@tarzanek Could you, please, review? |
|
||
set -e | ||
usage() { | ||
cat $0 | grep "^# !" | cut -d"!" -f2- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wouldn't it be easier just to echo the single line? ;-)
but this works too, so just nitpicking
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, if this functionally works/was tested
then I don't see a problem
logically it makes sense and properly checks the whole ring instead of single IP
merging, thnx Vlad |