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

roachtest: update pgjdbc blacklist with new expected pass #41389

Merged
merged 1 commit into from
Oct 10, 2019

Conversation

rafiss
Copy link
Collaborator

@rafiss rafiss commented Oct 7, 2019

Tests that used to fail now pass, so we need to update the blacklist.

closes #41366

Release justification: test only change

Release note: None

Tests that used to fail now pass, so we need to update the blacklist.

closes cockroachdb#41366

Release justification: test only change

Release note: None
@rafiss rafiss requested review from jordanlewis and a team October 7, 2019 16:28
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@jordanlewis
Copy link
Member

LGTM

@rafiss
Copy link
Collaborator Author

rafiss commented Oct 8, 2019

bors r+

@craig
Copy link
Contributor

craig bot commented Oct 8, 2019

Build failed (retrying...)

craig bot pushed a commit that referenced this pull request Oct 8, 2019
41031: roachprod: Add support for Microsoft Azure cloud r=bobvawter a=bobvawter

This commit adds support for geodistributed roachprod clusters running on Azure
VMs. Refer to the azure/doc.go package comments for additional information
about how the deployment operates.

Release note: None

41389: roachtest: update pgjdbc blacklist with new expected pass r=rafiss a=rafiss

Tests that used to fail now pass, so we need to update the blacklist.

closes #41366

Release justification: test only change

Release note: None

Co-authored-by: Bob Vawter <bob@cockroachlabs.com>
Co-authored-by: Rafi Shamim <rafi@cockroachlabs.com>
@rafiss
Copy link
Collaborator Author

rafiss commented Oct 9, 2019

Hmmmm well I guess it's lucky that the bors merge failed, because the test is now back to failing, as was expected before.

https://teamcity.cockroachdb.com/viewLog.html?buildId=1525981&buildTypeId=Cockroach_Nightlies_WorkloadNightly&tab=artifacts#%2Fpgjdbc

Leaving this PR open for now in case we need to bring it back again.

@rafiss
Copy link
Collaborator Author

rafiss commented Oct 10, 2019

bors r+

@rafiss
Copy link
Collaborator Author

rafiss commented Oct 10, 2019

re-submitting, seems like these two tests are passing reliably now https://teamcity.cockroachdb.com/viewLog.html?buildId=1531418&tab=artifacts#/pgjdbc

craig bot pushed a commit that referenced this pull request Oct 10, 2019
41389: roachtest: update pgjdbc blacklist with new expected pass r=rafiss a=rafiss

Tests that used to fail now pass, so we need to update the blacklist.

closes #41366

Release justification: test only change

Release note: None

Co-authored-by: Rafi Shamim <rafi@cockroachlabs.com>
@craig
Copy link
Contributor

craig bot commented Oct 10, 2019

Build succeeded

@craig craig bot merged commit 967c8c8 into cockroachdb:master Oct 10, 2019
@rafiss rafiss deleted the pgjdbc-blacklist-oct2019 branch October 14, 2019 18:39
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.

roachtest: pgjdbc failed
3 participants