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
nixos/cassandra: Port test to python #72897
Conversation
@GrahamcOfBorg test cassandra |
nixos/tests/cassandra.nix
Outdated
name = "cassandra"; | ||
meta = { | ||
maintainers = with lib.maintainers; [ johnazoidberg ]; | ||
} |
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.
} | |
}; |
nixos/tests/cassandra.nix
Outdated
@@ -9,13 +9,16 @@ let | |||
jmxRolesFile = ./cassandra-jmx-roles; | |||
jmxAuthArgs = "-u ${(builtins.elemAt jmxRoles 0).username} -pw ${(builtins.elemAt jmxRoles 0).password}"; | |||
jmxPort = 7200; # Non-standard port so it doesn't accidentally work | |||
jmxPortS = toString jmxPort; |
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.
is this like jmx ports? the case like PortS
seems incorrect.
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.
Yeah okay, I changed it to jmxPortStr
. That should be clearer :)
2b3cadd
to
25c2506
Compare
@GrahamcOfBorg test cassandra |
Motivation for this change
#72828
Things done
nix-build nixos/tests/cassandra.nix
printsNotify maintainers
cc myself