Skip to content

Deps dashboard monitoring - #173

Closed
diracdeltas wants to merge 4 commits into
masterfrom
deps-dashboard-monitoring
Closed

Deps dashboard monitoring#173
diracdeltas wants to merge 4 commits into
masterfrom
deps-dashboard-monitoring

Conversation

@diracdeltas

Copy link
Copy Markdown
Member

Supposedly this does the following:

Add customManagers to renovate.json so Renovate watches the *_VERSION lines in env.sh for tor/zlib/libevent/openssl and lists available upstream releases on the Dependency Dashboard. Configured with dependencyDashboardApproval so it never opens PRs on its own — the brave-tor-client-release Jenkins job remains responsible for that.

Rewrite DEPS_MONITORING.md to lead with the dashboard as the primary signal, keep the upstream RSS/changelog/mailing-list sources as fallback, and document where each tarball's SHA256 hash comes from.

Add customManagers to renovate.json so Renovate watches the *_VERSION
lines in env.sh for tor/zlib/libevent/openssl and lists available
upstream releases on the Dependency Dashboard. Configured with
dependencyDashboardApproval so it never opens PRs on its own — the
brave-tor-client-release Jenkins job remains responsible for that.

Rewrite DEPS_MONITORING.md to lead with the dashboard as the primary
signal, keep the upstream RSS/changelog/mailing-list sources as fallback,
and document where each tarball's SHA256 hash comes from.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR adds Renovate-based monitoring for the pinned C dependency versions in env.sh via the Renovate Dependency Dashboard, and updates the dependency monitoring documentation to make the dashboard the primary signal and clarify how to obtain/verifiy tarball hashes.

Changes:

  • Enable Renovate Dependency Dashboard + approval gating, and add regex customManagers to track *_VERSION variables in env.sh.
  • Add package rules to constrain Tor versions and surface releases immediately on the dashboard.
  • Rewrite DEPS_MONITORING.md to document the dashboard workflow, upstream fallback sources, and the origin of *_HASH values.

Reviewed changes

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

File Description
renovate.json Adds Renovate dashboard configuration and regex managers for TOR_VERSION, ZLIB_VERSION, LIBEVENT_VERSION, OPENSSL_VERSION.
DEPS_MONITORING.md Reorients docs around the Renovate dashboard and documents hash provenance and canonical download URLs.

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

Comment thread DEPS_MONITORING.md
Comment on lines +43 to +47
Canonical URLs (same ones the build downloads from):

```sh
# tor -> hash is published, just read it
curl -fsSL https://dist.torproject.org/tor-$TOR_VERSION.tar.gz.sha256sum
Comment thread DEPS_MONITORING.md Outdated
Comment thread DEPS_MONITORING.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread DEPS_MONITORING.md
`dependencyDashboardApproval`, so it never opens PRs on its own — the dashboard
is purely a notification surface.

When the dashboard shows a newer version, run the `brave-tor-client-release`

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this seems to run every 2 days from my understanding of https://developer.mend.io/github/brave/tor_build_scripts - not sure if it can be configured to be more frequent or if that's acceptable

@diracdeltas

Copy link
Copy Markdown
Member Author

closing for now as the cadence seems too slow

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.

2 participants