Skip to content
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

build: Run allocator acceptance tests with verbose logging #17736

Merged
merged 1 commit into from
Aug 17, 2017

Conversation

a-robinson
Copy link
Contributor

Without these, the logs don't include any info to help debug failures,
and after the cluster has shut down there's really nothing to go by.

I'm still in the process of testing this out, so I won't merge it until I've verified it works and does what I want.

This will help confirm my hypotheses on #17685, assuming a run fails again soon, and will help with future debugging of these tests.

@a-robinson a-robinson requested review from cuongdo and a team August 17, 2017 18:11
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@cuongdo
Copy link
Contributor

cuongdo commented Aug 17, 2017

:lgtm:


Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks pending.


Comments from Reviewable

@a-robinson
Copy link
Contributor Author

Yup, this works as intended, although the test passed. I'll merge once CI is green.

Without these, the logs don't include any info to help debug failures,
and after the cluster has shut down there's really nothing to go by.
@a-robinson a-robinson merged commit 33a71d4 into cockroachdb:master Aug 17, 2017
@a-robinson a-robinson deleted the accept branch May 18, 2018 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants