Skip to content

chore: initialize bzlmod#40692

Draft
mmorel-35 wants to merge 4 commits intoenvoyproxy:mainfrom
mmorel-35:bzlmod
Draft

chore: initialize bzlmod#40692
mmorel-35 wants to merge 4 commits intoenvoyproxy:mainfrom
mmorel-35:bzlmod

Conversation

@mmorel-35
Copy link
Copy Markdown
Contributor

@mmorel-35 mmorel-35 commented Aug 13, 2025

Commit Message: initialize bzlmod
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only
Copy link
Copy Markdown

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #40692 was opened by mmorel-35.

see: more, trace.

@mmorel-35
Copy link
Copy Markdown
Contributor Author

@phlax ,

That's a draft, I want to indrocuce the minimal reviewable changes so the different dependencies can be migrated progressibely

@mmorel-35 mmorel-35 force-pushed the bzlmod branch 2 times, most recently from 380cff7 to b6ef80e Compare August 13, 2025 06:51
@phlax
Copy link
Copy Markdown
Member

phlax commented Aug 13, 2025

brilliant - really nice work - thanks so much @mmorel-35

i started to look at this previously and the pushback was that we need to provide guidance on how to build a filter, as there are a lot of downstreams that currently build against/with the WORKSPACE - ie envoy-filter-example - which also doesnt currently work correctly

i have limited availability atm but will help where i can

@mmorel-35 mmorel-35 force-pushed the bzlmod branch 6 times, most recently from e0bd72f to 1c09e81 Compare August 15, 2025 23:53
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Aug 15, 2025
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @agrawroh

🐱

Caused by: #40692 was synchronize by mmorel-35.

see: more, trace.

@mmorel-35 mmorel-35 force-pushed the bzlmod branch 16 times, most recently from f9bb1af to bb6e33b Compare August 16, 2025 11:46
@mmorel-35 mmorel-35 force-pushed the bzlmod branch 8 times, most recently from eeb1b97 to 284d5e4 Compare August 25, 2025 05:56
@mmorel-35
Copy link
Copy Markdown
Contributor Author

  • Update rules_buf to 0.5.1 minimum
  • Migrate com_github_chrusty_protoc_gen_jsonschema to @envoy instead of @envoy_api

@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 6, 2025

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

mmorel-35 and others added 4 commits November 22, 2025 08:04
…ith automatic detection and patch preservation

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
…tionality for Bazel 8.4.2 with dual-mode support, Option C WORKSPACE.bzlmod separation and BCR migration

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
…, provide expert review, deprecate WORKSPACE mode, and implement CI/CD validation

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deps Approval required for changes to Envoy's external dependencies no stalebot Disables stalebot from closing an issue workflows:untested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants