Add rules_rs v0.0.27.envoy to bazel-registry with telemetry removal#3708
Draft
Add rules_rs v0.0.27.envoy to bazel-registry with telemetry removal#3708
Conversation
✅ Deploy Preview for nifty-bassi-e26446 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add dzbarsky/rules_rs v0.0.27 to bazel-registry
Add rules_rs v0.0.27.envoy to bazel-registry with telemetry removal
Feb 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds
dzbarsky/rules_rsv0.0.27 to the local bazel-registry. The upstream module includesaspect_tools_telemetrywhich phones home to Aspect Build Inc. The telemetry is loaded but never functionally used, so it's cleanly stripped via patch.Changes
github:dzbarsky/rules_rs0.0.27.envoyand telemetry bazel_dep + extension removedload("@aspect_tools_telemetry_report//:defs.bzl", "TELEMETRY")fromrs/extensions.bzl"@aspect_tools_telemetry_report//:defs.bzl"from bzl_library deps inrs/BUILD.bazel@rules_rs//...Integrity Hashes
sha256-ZKOZJxQPvJ3KJsImxertVYn+6WsZHWsODq6/P2TGrRY=sha256-HwgBe1DXita6BksFKYrWH0lVSuDOEI9NtFw587ZqkWA=Original prompt
Overview
Add
dzbarsky/rules_rsv0.0.27 to the Envoy toolshed bazel-registry with a patch that removes theaspect_tools_telemetrydependency.The upstream
rules_rsmodule (https://github.com/dzbarsky/rules_rs) bundlesaspect_tools_telemetrywhich phones home usage data to Aspect Build Inc. This is unacceptable for Envoy's use. The telemetry is loaded but never functionally used (the load even has# buildifier: disable=loadto suppress unused-load warnings), so it can be cleanly stripped out.What needs to be created
Create the following files under
bazel-registry/modules/rules_rs/:1.
bazel-registry/modules/rules_rs/metadata.json{ "homepage": "https://github.com/dzbarsky/rules_rs", "maintainers": [ { "email": "maintainers@envoyproxy.io", "github": "envoyproxy", "name": "Envoy Proxy Maintainers" } ], "repository": [ "github:dzbarsky/rules_rs" ], "versions": [ "0.0.27.envoy" ], "yanked_versions": {} }2.
bazel-registry/modules/rules_rs/0.0.27.envoy/source.jsonThe upstream release tarball is at:
https://github.com/dzbarsky/rules_rs/releases/download/v0.0.27/rules_rs-v0.0.27.tar.gzYou need to compute the integrity hash. Use
sha256integrity format matching the existing registry pattern (e.g."sha256-..."base64-encoded). You can compute it by downloading the tarball and running:Then base64-encode the hex hash for the
integrityfield.The
source.jsonshould reference the patch file and look like:{ "url": "https://github.com/dzbarsky/rules_rs/releases/download/v0.0.27/rules_rs-v0.0.27.tar.gz", "integrity": "<computed sha256 integrity>", "strip_prefix": "", "patches": { "remove_telemetry.patch": "<computed sha256 integrity of patch file>" }, "patch_strip": 1 }3.
bazel-registry/modules/rules_rs/0.0.27.envoy/MODULE.bazelThis should be a copy of the upstream
MODULE.bazelfrom v0.0.27 (at commitf995a4c17df970158a24c93d586f87f60a4887e8) but with:"0.0.27.envoy"in themodule()callaspect_tools_telemetrybazel_depremovedtel = use_extension(...)anduse_repo(tel, ...)lines removedThe upstream MODULE.bazel content is:
Remove the 3 telemetry-related lines and set version to
"0.0.27.envoy".4.
bazel-registry/modules/rules_rs/0.0.27.envoy/patches/remove_telemetry.patchThis patch must apply to the upstream tarball and make these changes:
a)
rs/extensions.bzl— Remove line 1:b)
rs/BUILD.bazel— Remove the@aspect_tools_telemetry_report//:defs.bzlline from thedepslist of thebzl_librarytarget: