Skip to content

Commit

Permalink
Merge #79622 #79632
Browse files Browse the repository at this point in the history
79622: roachtest: update libpq test r=otan a=rafiss

fixes #79570

This allows the query cancel tests to pass.

Release note: None

79632: cli: cosmetic change in the output of `start` r=dt a=knz

This fixes a cosmetic bug that made its way in the previous change
to `reportServerInfo`. (#74005)

Release note: None

Co-authored-by: Rafi Shamim <[email protected]>
Co-authored-by: Raphael 'kena' Poss <[email protected]>
  • Loading branch information
3 people committed Apr 8, 2022
3 parents dbbe9ed + 17af313 + 440b53b commit 7a97bdd
Show file tree
Hide file tree
Showing 3 changed files with 33 additions and 3 deletions.
2 changes: 1 addition & 1 deletion pkg/cli/start.go
Original file line number Diff line number Diff line change
Expand Up @@ -983,7 +983,7 @@ func reportServerInfo(
buf.Printf("cluster name:\t%s\n", baseCfg.ClusterName)
}
clusterID := serverCfg.BaseConfig.ClusterIDContainer.Get()
if clusterID.Equal(tenantClusterID) {
if tenantClusterID.Equal(uuid.Nil) {
buf.Printf("clusterID:\t%s\n", clusterID)
} else {
buf.Printf("storage clusterID:\t%s\n", clusterID)
Expand Down
2 changes: 1 addition & 1 deletion pkg/cmd/roachtest/tests/libpq.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ import (
)

var libPQReleaseTagRegex = regexp.MustCompile(`^v(?P<major>\d+)\.(?P<minor>\d+)\.(?P<point>\d+)$`)
var libPQSupportedTag = "v1.10.4"
var libPQSupportedTag = "v1.10.5"

func registerLibPQ(r registry.Registry) {
runLibPQ := func(ctx context.Context, t test.Test, c cluster.Cluster) {
Expand Down
32 changes: 31 additions & 1 deletion pkg/cmd/roachtest/tests/libpq_blocklist.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,37 @@ var libPQBlocklists = blocklistsForVersion{
{"v22.1", "libPQBlocklist22_1", libPQBlocklist22_1, "libPQIgnorelist22_1", libPQIgnorelist22_1},
}

var libPQBlocklist22_1 = libPQBlocklist21_2
var libPQBlocklist22_1 = blocklist{
"pq.ExampleConnectorWithNoticeHandler": "unknown",
"pq.TestBinaryByteSliceToInt": "41547",
"pq.TestBinaryByteSlicetoUUID": "41547",
"pq.TestConnListen": "41522",
"pq.TestConnUnlisten": "41522",
"pq.TestConnUnlistenAll": "41522",
"pq.TestConnectorWithNoticeHandler_Simple": "unknown",
"pq.TestConnectorWithNotificationHandler_Simple": "unknown",
"pq.TestCopyFromError": "5807",
"pq.TestCopyInRaiseStmtTrigger": "5807",
"pq.TestCopyInTypes": "5807",
"pq.TestCopyRespLoopConnectionError": "5807",
"pq.TestEncodeAndParseTs": "41563",
"pq.TestInfinityTimestamp": "41564",
"pq.TestIssue186": "41558",
"pq.TestIssue196": "41689",
"pq.TestIssue282": "12137",
"pq.TestListenerFailedQuery": "41522",
"pq.TestListenerListen": "41522",
"pq.TestListenerReconnect": "41522",
"pq.TestListenerUnlisten": "41522",
"pq.TestListenerUnlistenAll": "41522",
"pq.TestNotifyExtra": "41522",
"pq.TestPing": "35897",
"pq.TestQueryRowBugWorkaround": "5807",
"pq.TestReconnect": "35897",
"pq.TestRowsColumnTypes": "41688",
"pq.TestRuntimeParameters": "12137",
"pq.TestStringWithNul": "26366",
}

var libPQBlocklist21_2 = blocklist{
"pq.ExampleConnectorWithNoticeHandler": "unknown",
Expand Down

0 comments on commit 7a97bdd

Please sign in to comment.