Skip to content

cli/interactive_tests: complete the end-to-end test for cockroach connect#63846

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
knz:20210419-test-connect
Apr 20, 2021
Merged

cli/interactive_tests: complete the end-to-end test for cockroach connect#63846
craig[bot] merged 1 commit intocockroachdb:masterfrom
knz:20210419-test-connect

Conversation

@knz
Copy link
Copy Markdown
Contributor

@knz knz commented Apr 19, 2021

The end-to-end test for the new connect command was incomplete,
because of issue #61624 that was blocking the functionality.

Now that #63589 is in, we can add the missing test.

Release note: None

@knz knz requested a review from itsbilal April 19, 2021 14:09
@knz knz requested a review from a team as a code owner April 19, 2021 14:09
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@knz
Copy link
Copy Markdown
Contributor Author

knz commented Apr 19, 2021

Current CI failure due to #63851

…nnect`

The end-to-end test for the new `connect` command was incomplete,
because of issue cockroachdb#61624 that was blocking the functionality.

Now that cockroachdb#63589 is in, we can add the missing test.

Release note: None
@knz knz force-pushed the 20210419-test-connect branch from 15bd009 to 26effee Compare April 19, 2021 18:24
Copy link
Copy Markdown
Contributor

@itsbilal itsbilal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm: thanks!

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @knz)

@knz
Copy link
Copy Markdown
Contributor Author

knz commented Apr 20, 2021

thank you!

bors r=itsbilal

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 20, 2021

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 20, 2021

Build succeeded:

@craig craig bot merged commit 09be4f7 into cockroachdb:master Apr 20, 2021
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