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
nixosTests.cockroachdb: port to python #73934
nixosTests.cockroachdb: port to python #73934
Conversation
8add180
to
fb124c5
Compare
@GrahamcOfBorg test cockroachdb |
Seeing sort of failure only on x86_64-linux
|
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.
I believe this is what's causing that issue, this doesn't even need to be escaped.
Tested locally.
fb124c5
to
1c9be5c
Compare
Argh, 🤦♂️. Thanks for spotting that! |
e9bd08b
to
2419ecc
Compare
@GrahamcOfBorg test cockroachdb |
For some reason, this still doesn't converge properly here - even though it kinda succeeds in the perl version. 🤔 |
@GrahamcOfBorg test cockroachdb |
4fec4bb
to
0734591
Compare
@GrahamcOfBorg test cockroachdb |
@flokli what's the state of this? |
I couldn't get it to work, and don't really know why it's failing either. It seems to have worked with the perl test, there seems to be a subtle difference with passing parameters, or I'm using it wrongly.
I experimented with some ntp-server-based setups locally, but it seems ntpd really doesn't want to be stratum 0 either ;-)
|
nixos/tests/cockroachdb.nix
Outdated
node.start() | ||
node.wait_for_unit("cockroachdb") | ||
node1.succeed( | ||
"cockroach sql --host=192.168.1.1 --insecure -e 'SHOW ALL CLUSTER SETTINGS' 2>&1" |
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.
IIUIC, this needs ,
between the strings, otherwise they're concatenated.
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.
nice catch! However, I still don't get the servers to sync together. It did eventually succeed!
cockroachdb complained about not enough memory available.
0734591
to
fe6dd71
Compare
I rebased this on latest master. |
@GrahamcOfBorg test cockroachdb |
@tfc this test works now, PTAL. |
Motivation for this change
#72828
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @tfc