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.

Sign up for GitHub

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

Jump to bottom

spanner: many tests failed #2578

Closed
flaky-bot bot opened this issue Jul 9, 2020 · 2 comments
Closed

spanner: many tests failed #2578

flaky-bot bot opened this issue Jul 9, 2020 · 2 comments
Assignees
@skuruppu
Labels
api: spanner Issues related to the Spanner API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@flaky-bot
Copy link

flaky-bot bot commented Jul 9, 2020

Many tests failed at the same time in this package.

  • I will close this issue when there are no more failures in this package and
    there is at least one pass.
  • No new issues will be filed for this package until this issue is closed.
  • If there are already issues for individual test cases, I will close them when
    the corresponding test passes. You can close them earlier, if you prefer, and
    I won't reopen them while this issue is still open.

Here are the tests that failed:

commit: e493799
buildURL: Build Status, Sponge
status: failed
@flaky-bot flaky-bot bot added buildcop: issue priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Jul 9, 2020
@product-auto-label product-auto-label bot added the api: spanner Issues related to the Spanner API. label Jul 9, 2020
@hengfengli
Copy link
Contributor

Error:

panic: cannot handle unexported field at {*status.Error}.e:
	"google.golang.org/grpc/internal/status".Error
consider using a custom Comparer; if you control the implementation of type, you can also consider using an Exporter, AllowUnexported, or cmpopts.IgnoreUnexported [recovered]
	panic: cannot handle unexported field at {*status.Error}.e:
	"google.golang.org/grpc/internal/status".Error
consider using a custom Comparer; if you control the implementation of type, you can also consider using an Exporter, AllowUnexported, or cmpopts.IgnoreUnexported

goroutine 3949 [running]:
testing.tRunner.func1(0xc000258100)
	/usr/local/go/src/testing/testing.go:792 +0x6a7
panic(0xfaab40, 0xc00260d2a0)
	/usr/local/go/src/runtime/panic.go:513 +0x1b9
github.com/google/go-cmp/cmp.validator.apply(0xc001a310e0, 0x10850a0, 0xc001d760b8, 0x1b6, 0x10850a0, 0xc00192e348, 0x1b6)
	/go/pkg/mod/github.com/google/go-cmp@v0.5.0/cmp/options.go:241 +0x62a
github.com/google/go-cmp/cmp.(*state).tryOptions(0xc001a310e0, 0x139dc00, 0x10850a0, 0x10850a0, 0xc001d760b8, 0x1b6, 0x10850a0, 0xc00192e348, 0x1b6, 0x101e1f8)
	/go/pkg/mod/github.com/google/go-cmp@v0.5.0/cmp/compare.go:303 +0x15a
github.com/google/go-cmp/cmp.(*state).compareAny(0xc001a310e0, 0x138ae20, 0xc0004c8900)
	/go/pkg/mod/github.com/google/go-cmp@v0.5.0/cmp/compare.go:258 +0x3c5
github.com/google/go-cmp/cmp.(*state).compareStruct(0xc001a310e0, 0x139dc00, 0x101e1c0, 0x101e1c0, 0xc001d760b8, 0x199, 0x101e1c0, 0xc00192e348, 0x199)
	/go/pkg/mod/github.com/google/go-cmp@v0.5.0/cmp/compare.go:427 +0x594
github.com/google/go-cmp/cmp.(*state).compareAny(0xc001a310e0, 0x138ad60, 0xc001a3ea80)
	/go/pkg/mod/github.com/google/go-cmp@v0.5.0/cmp/compare.go:286 +0x15de
github.com/google/go-cmp/cmp.(*state).comparePtr(0xc001a310e0, 0x139dc00, 0x1016120, 0x1016120, 0xc001d760b8, 0x16, 0x1016120, 0xc00192e348, 0x16)
	/go/pkg/mod/github.com/google/go-cmp@v0.5.0/cmp/compare.go:579 +0x621
github.com/google/go-cmp/cmp.(*state).compareAny(0xc001a310e0, 0x13896a0, 0xc001a3ea40)
	/go/pkg/mod/github.com/google/go-cmp@v0.5.0/cmp/compare.go:292 +0x1469
github.com/google/go-cmp/cmp.Equal(0x1016120, 0xc001d760b8, 0x1016120, 0xc00192e348, 0xc001ba65a0, 0x6, 0x6, 0x4)
	/go/pkg/mod/github.com/google/go-cmp@v0.5.0/cmp/compare.go:94 +0xac
cloud.google.com/go/internal/testutil.Equal(0x1016120, 0xc001d760b8, 0x1016120, 0xc00192e348, 0xc0004f3b00, 0x2, 0x6, 0xc001a89b01)
	/go/pkg/mod/cloud.google.com/go@v0.60.0/internal/testutil/cmp.go:52 +0x16f
cloud.google.com/go/spanner.testEqual(0x1016120, 0xc001d760b8, 0x1016120, 0xc00192e348, 0x1)
	/tmpfs/src/google-cloud-go/spanner/cmp_test.go:28 +0x3e0
cloud.google.com/go/spanner.TestRsdNonblockingStates.func7(0xc000258100)
	/tmpfs/src/google-cloud-go/spanner/read_test.go:873 +0xde9
testing.tRunner(0xc000258100, 0xc001d54c10)
	/usr/local/go/src/testing/testing.go:827 +0x163
created by testing.(*T).Run
	/usr/local/go/src/testing/testing.go:878 +0x65a
FAIL	cloud.google.com/go/spanner	58.205s

This is caused by the upgrade of grpc. #2577 should fix this issue.

@hengfengli
Copy link
Contributor

I would close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skuruppu skuruppu

Labels
api: spanner Issues related to the Spanner API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hengfengli @skuruppu