Test suite: Only run the c1,c2 hostname test in CI#29
Conversation
|
Alright, @kleinhenz this is now ready for review. @jbphyswx This should make it easier for people to run the |
| if hosts != ["c1", "c1", "c2", "c2"] | ||
| msg = "Test failed: observed_hosts=$(hosts) does not match expected_hosts=[c1, c1, c2, c2]" | ||
| error(msg) | ||
| const is_ci = parse(Bool, strip(get(ENV, "CI", "false"))) |
There was a problem hiding this comment.
Is this actually present in our CI since we are running inside the docker compose cluster?
There was a problem hiding this comment.
Ah, that's a really good point. No, I don't think it is.
Hmmm. I'll have to figure something else out.
There was a problem hiding this comment.
Alright, I have a different approach now.
If the environment variable is not set (e.g. if you are inside of our Docker Compose cluster), we default to is_ci = true.
There was a problem hiding this comment.
So if you look at the CI logs on this PR (e.g. this CI log), you'll see the following message:
[ Info: This is CI, so we will perform the hostname test
This indicates that the test is being run on our CI, as desired.
2 participants
Depends on:
@asserts to errors, and change bareusing Footousing Foo: name, anothername, ...#28