Skip to content

Conversation

strideynet
Copy link
Contributor

@strideynet strideynet commented Oct 15, 2025

By default tbot will write internal state to /var/lib/teleport/bot. If a user installs tbot prior to installing Teleport, they'll receive a fairly scary message when they try to use teleport configure:

The data directory %s is not empty and may contain existing cluster state. Running this configuration is likely a mistake. To join a new cluster, specify an alternate --data-dir or clear the %s directory

This PR changes the implementation of teleport configure to ignore the bot subdirectory.

I've manually tested w/ and w/o this change - it doesn't appear we have a good scaffold for automated testing here based on logs.

Closes #60214

@strideynet strideynet added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v16 backport/branch/v17 backport/branch/v18 labels Oct 15, 2025
@github-actions github-actions bot requested a review from gzdunek October 15, 2025 08:28
@github-actions github-actions bot requested a review from Tener October 15, 2025 08:28
@strideynet strideynet added this pull request to the merge queue Oct 15, 2025
@strideynet strideynet removed this pull request from the merge queue due to a manual request Oct 15, 2025
@strideynet strideynet enabled auto-merge October 15, 2025 10:23
@strideynet strideynet added this pull request to the merge queue Oct 15, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 15, 2025
@strideynet strideynet added this pull request to the merge queue Oct 15, 2025
Merged via the queue into master with commit 70b5377 Oct 15, 2025
40 checks passed
@strideynet strideynet deleted the strideynet/fix-spurious-warning-teleport-configure-tbot branch October 15, 2025 11:24
@backport-bot-workflows
Copy link
Contributor

@strideynet See the table below for backport results.

Branch Result
branch/v16 Create PR
branch/v17 Create PR
branch/v18 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

teleport configure existing state check warning should ignore /var/lib/teleport/bot

3 participants