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

cli/interactive_tests: de-flake test_missing_log_output #53408

Closed
wants to merge 3 commits into from

Conversation

knz
Copy link
Contributor

@knz knz commented Aug 25, 2020

Fixes #50970

This flake was introduced in a recent bump of cockroachdb/errors,
combined with #52201.

Release note: None

Release note (cli change): The command-line flag `--socket` has been
removed. It was deprecated since v20.1. Use `--socket-dir` in
replacement.
@knz knz requested a review from tbg August 25, 2020 08:49
@knz knz requested a review from a team as a code owner August 25, 2020 08:49
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Release note (cli change): The command-line `--insecure` has been
marked as deprecated. See https://go.crdb.dev/issue/53404 for details.
The flag will be removed in a later version in a staged fashion:
first, additional security mechanisms will be added to enable more
flexible deployments which were previously done using `--insecure`;
then the flaf will be removed from server commands, then finally in a
later version also from client commands.
This flake was introduced in a recent bump of `cockroachdb/errors`,
combined with cockroachdb#52201.

Release note: None
@knz
Copy link
Contributor Author

knz commented Aug 25, 2020

bors r=tbg

@craig
Copy link
Contributor

craig bot commented Aug 25, 2020

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Aug 25, 2020

Build failed (retrying...):

@knz
Copy link
Contributor Author

knz commented Aug 25, 2020

bors r-

this did not fix the issue apparently

@craig
Copy link
Contributor

craig bot commented Aug 25, 2020

Canceled.

@tbg tbg added the X-noremind Bots won't notify about PRs with X-noremind label May 6, 2021
@knz
Copy link
Contributor Author

knz commented Dec 28, 2022

We've addressed this since

@knz knz closed this Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
X-noremind Bots won't notify about PRs with X-noremind
Projects
None yet
Development

Successfully merging this pull request may close these issues.

acceptance: TestDockerCLI failed
3 participants