Skip to content

Retrieving latest release metadata from stats.osctrl.net in osctrl-tls#788

Merged
javuto merged 1 commit intomainfrom
telemetry-osctrl-tls
Mar 30, 2026
Merged

Retrieving latest release metadata from stats.osctrl.net in osctrl-tls#788
javuto merged 1 commit intomainfrom
telemetry-osctrl-tls

Conversation

@javuto
Copy link
Copy Markdown
Collaborator

@javuto javuto commented Mar 30, 2026

Metadata for latest release retrieved from stats.osctrl.net to make sure stable version is used.

@javuto javuto requested a review from Copilot March 30, 2026 21:03
@javuto javuto added ✨ enhancement New feature or request osctrl-tls osctrl-tls related changes labels Mar 30, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds startup-time retrieval of latest release metadata from stats.osctrl.net so osctrl-tls can report whether it’s running the suggested/stable release.

Changes:

  • Add checkLatestRelease() to cmd/tls/main.go to fetch and evaluate latest/suggested release metadata.
  • Run the release check asynchronously during osctrl-tls startup to avoid delaying service initialization.
  • Update the startup comment in cmd/admin/main.go to clarify the goroutine behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
cmd/tls/main.go Adds latest-release retrieval and runs it in a goroutine during startup.
cmd/admin/main.go Comment-only update describing the goroutine-based release check.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cmd/tls/main.go
Comment thread cmd/tls/main.go
@javuto javuto merged commit 7717703 into main Mar 30, 2026
59 checks passed
@javuto javuto deleted the telemetry-osctrl-tls branch March 30, 2026 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ enhancement New feature or request osctrl-tls osctrl-tls related changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants