Skip to content

roachtest: Make ssh debug logging optional.#95762

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
smg260:roachtest_optional_ssh_debug
Jan 25, 2023
Merged

roachtest: Make ssh debug logging optional.#95762
craig[bot] merged 1 commit intocockroachdb:masterfrom
smg260:roachtest_optional_ssh_debug

Conversation

@smg260
Copy link
Copy Markdown
Contributor

@smg260 smg260 commented Jan 24, 2023

Wait commands are issued every 500ms returning a non zero exit code until nodes have started.
This results in a large number of ssh debug logs during cluster creation. Also adopts functional options.

Release note: None
Epic: none

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@smg260 smg260 force-pushed the roachtest_optional_ssh_debug branch 2 times, most recently from e7ea090 to 84f4209 Compare January 24, 2023 16:52
Wait commands are issued every 500ms returning a non zero exit
code until nodes have started. This results in a large number
of ssh debug logs during cluster creation.

Also adopts functional options.

Release note: None
Epic: none
@smg260 smg260 force-pushed the roachtest_optional_ssh_debug branch from 84f4209 to fc0d3a2 Compare January 24, 2023 19:04
@smg260 smg260 marked this pull request as ready for review January 24, 2023 19:06
@smg260 smg260 requested a review from a team as a code owner January 24, 2023 19:06
@smg260 smg260 requested review from renatolabs and srosenberg and removed request for a team January 24, 2023 19:06
Copy link
Copy Markdown

@renatolabs renatolabs left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @srosenberg)

Copy link
Copy Markdown
Member

@srosenberg srosenberg left a comment

Choose a reason for hiding this comment

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

:lgtm:

Thank you, it was getting really noisy! :)

Reviewable status: :shipit: complete! 2 of 0 LGTMs obtained

@smg260
Copy link
Copy Markdown
Contributor Author

smg260 commented Jan 24, 2023

bors r=renatolabs,srosenberg

@craig
Copy link
Copy Markdown
Contributor

craig Bot commented Jan 25, 2023

Build succeeded:

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.

4 participants