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

test failure: TestDockerCLI/test_log_config_msg.tcl #39720

Closed
andreimatei opened this issue Aug 16, 2019 · 6 comments · Fixed by #40821
Closed

test failure: TestDockerCLI/test_log_config_msg.tcl #39720

andreimatei opened this issue Aug 16, 2019 · 6 comments · Fixed by #40821
Assignees
Labels
C-investigation Further steps needed to qualify. C-label will change. C-test-failure Broken test (automatically or manually discovered).

Comments

@andreimatei
Copy link
Contributor

Happened to me on CI here:
https://teamcity.cockroachdb.com/viewLog.html?buildId=1442417&buildTypeId=Cockroach_UnitTests

I guess this failure claims that the cluster ID was not printed in the logs... But I'm not sure how to check the get the logs...

TestDockerCLI/test_log_config_msg.tcl/runMode=docker

.190816 17:56:10.919356973 EXPECT TEST: BEGIN START SERVER
?column?
1

.190816 17:56:15.028047904 EXPECT TEST: START SERVER DONE

.190816 17:56:15.034128397 EXPECT TEST: BEGIN STOP SERVER
still waiting

.190816 17:56:16.380400178 EXPECT TEST: END STOP SERVER

.190816 17:56:16.386403861 EXPECT TEST: START TEST: Check that the cluster ID is reported at the start of the first log file.
child process exited abnormally
    while executing
"system "grep -qF '\[config\] clusterID:' logs/db/logs/cockroach.log""
    (file "/go/src/github.com/cockroachdb/cockroach/cli/interactive_tests/test_log_config_msg.tcl" line 12)
non-zero exit code: 1
        cli_test.go:88: non-zero exit code: 1
@andreimatei andreimatei added the C-test-failure Broken test (automatically or manually discovered). label Aug 16, 2019
@knz knz added the C-investigation Further steps needed to qualify. C-label will change. label Aug 27, 2019
@knz
Copy link
Contributor

knz commented Aug 27, 2019

@bobvawter somehow the TC config for acceptance tests does not keep artifacts for TestDockerCLI. It used to (IIRC from my time debugging these tests in 2.1). How do you think I can investigate what changed?

@irfansharif
Copy link
Contributor

Seems this is the current catch-all issue for TestDockerCLI. Here's one of my own from #40737: https://teamcity.cockroachdb.com/viewLog.html?buildId=1484662&buildTypeId=Cockroach_UnitTests

(closer in spirit to #40734.)

@jordanlewis
Copy link
Member

No, I didn't mean to make the other one point to this - I thought it was also about test_log_config_msg.tcl. I've reopened the one you pointed to.

@rohany
Copy link
Contributor

rohany commented Sep 13, 2019

Bumping @knz's question above -- is there a way to get the build output from the failed tests?

@knz
Copy link
Contributor

knz commented Sep 17, 2019

The build output is there but TC is being a bit obtuse about where it is.
The way I find this reliably:

  1. click the test name
  2. click "show build log"
  3. this brings up a page. ignore the build log itself, it's useless. Instead, click the "Artifacts" tab at the top
  4. navigate to artifacts -> TestDockerCLI -> your test
  5. look at the console-output.log and other log files.

I have no idea how to make this navigation easier, I wish that TC was showing the acceptance artifacts at the top level of the main screen.

@knz
Copy link
Contributor

knz commented Sep 17, 2019

@irfansharif regarding this:

Seems this is the current catch-all issue for TestDockerCLI.

No it's not.
This present issue has the following symptom:

child process exited abnormally
    while executing
"system "grep -qF '\[config\] clusterID:' logs/db/logs/cockroach.log"

While your own from #40737 has symptom:

error when contacting licensing server: Get https://register.cockroachdb.com/api/license

They are different issues. Indeed yours is a dup of #40734.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-investigation Further steps needed to qualify. C-label will change. C-test-failure Broken test (automatically or manually discovered).
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants