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

[teleport-update] Adjust download progress log output #49845

Merged
merged 3 commits into from
Dec 5, 2024

Conversation

sclevine
Copy link
Member

@sclevine sclevine commented Dec 5, 2024

#49805 adding logging for download progress and stats when teleport-update downloads Teleport tgz files.

I realized that I copied the log formatting line used by progressLogger from lineLogger, and I forgot to go back and structure the log before opening the PR. This change correctly structures the log lines.

Old:

2024-12-05T03:22:17Z INFO [UPDATER]   teleport-ent-v16.4.3-linux-arm64-bin.tar.gz - progress: 40% agent/logger.go:50

New:

2024-12-05T03:22:17Z INFO [UPDATER]   Downloading file=teleport-ent-v16.4.3-linux-arm64-bin.tar.gz progress=40% agent/logger.go:50

The teleport-update binary will be used to enable, disable, and trigger automatic Teleport agent updates. The new auto-updates system manages a local installation of the cluster-specified version of Teleport stored in /opt/teleport.

RFD: #47126
Goal (internal): https://github.com/gravitational/cloud/issues/10289

@sclevine sclevine added the no-changelog Indicates that a PR does not require a changelog entry label Dec 5, 2024
@sclevine sclevine added this pull request to the merge queue Dec 5, 2024
Merged via the queue into master with commit 0131f45 Dec 5, 2024
40 of 41 checks passed
@sclevine sclevine deleted the sclevine/teleport-update-progress-tweak branch December 5, 2024 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

4 participants