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

Fix flaky TestInitDatabaseService #47750

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

rosstimothy
Copy link
Contributor

Addresses two separate issues caused by using t.TempDir as the data directory in the test.

  • Session recording is disabled to avoid cleanup problems with a non-empty data dir
  • The debug service is disabled to prevent long unix socket paths from failing tests on macOS

Closes #42963

Addresses two separate issues caused by using t.TempDir as the
data directory in the test.

- Session recording is disabled to avoid cleanup problems with
  a non-empty data dir
- The debug service is disabled to prevent long unix socket
  paths from failing tests on macOS

Closes #42963
@rosstimothy rosstimothy added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v15 backport/branch/v16 labels Oct 21, 2024
@rosstimothy rosstimothy marked this pull request as ready for review October 21, 2024 13:29
@rosstimothy rosstimothy enabled auto-merge October 21, 2024 13:35
@rosstimothy rosstimothy added this pull request to the merge queue Oct 21, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 21, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Oct 21, 2024
Merged via the queue into master with commit cac5a20 Oct 21, 2024
45 checks passed
@rosstimothy rosstimothy deleted the tross/fix_TestInitDatabaseService branch October 21, 2024 15:32
@public-teleport-github-review-bot

@rosstimothy See the table below for backport results.

Branch Result
branch/v15 Failed
branch/v16 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v15 backport/branch/v16 no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TestInitDatabaseService/enabled_invalid_databases flakiness
3 participants