Skip to content

wip - bazel 8 #42831

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.

Sign up for GitHub

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

Jump to bottom
Draft
jwendell wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

wip - bazel 8 #42831

jwendell wants to merge 8 commits into from

Conversation

@jwendell
Copy link
Member

@jwendell jwendell commented Jan 2, 2026

No description provided.

@repokitteh-read-only
Copy link

repokitteh-read-only bot commented Jan 2, 2026

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: #42831 was opened by jwendell.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Jan 2, 2026
@repokitteh-read-only
Copy link

repokitteh-read-only bot commented Jan 2, 2026

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 @RyanTheOptimist

🐱

Caused by: #42831 was opened by jwendell.

see: more, trace.

@jwendell
deps: Bump proxy-wasm-cpp-sdk
3bbea53 
The new proxy-wasm-cpp-sdk uses ALLOW_MEMORY_GROWTH=1, allowing WASM
memory to grow dynamically. This caused Buffer::copyTo to fail when
requesting more data than the buffer contains, as allocMemory now
succeeds for large allocations instead of failing. Added explicit bounds
validation before allocating to ensure the requested range is within
the buffer's actual length.

Signed-off-by: Jonh Wendell <[email protected]>
@jwendell
tmp-bazel8
0fd438d 
Signed-off-by: Jonh Wendell <[email protected]>
@jwendell
Patch rules_python for bazel 8
e7146a4 
Signed-off-by: Jonh Wendell <[email protected]>
@jwendell
patch rules_python to default to x86
925a35f 
Signed-off-by: Jonh Wendell <[email protected]>
@jwendell
adapt emsdk patch
1560c7d 
Signed-off-by: Jonh Wendell <[email protected]>
@jwendell
fix yq
d24edb3 
Signed-off-by: Jonh Wendell <[email protected]>
@jwendell
fix tests
e20d44c 
Signed-off-by: Jonh Wendell <[email protected]>
@jwendell
fix mobile flag
14297b8 
Signed-off-by: Jonh Wendell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mattklein123 mattklein123 Awaiting requested review from mattklein123 mattklein123 will be requested when the pull request is marked ready for review mattklein123 is a code owner

@RyanTheOptimist RyanTheOptimist Awaiting requested review from RyanTheOptimist RyanTheOptimist will be requested when the pull request is marked ready for review RyanTheOptimist is a code owner

@abeyad abeyad Awaiting requested review from abeyad abeyad will be requested when the pull request is marked ready for review abeyad is a code owner

@danzh2010 danzh2010 Awaiting requested review from danzh2010 danzh2010 will be requested when the pull request is marked ready for review danzh2010 is a code owner

@kyessenov kyessenov Awaiting requested review from kyessenov kyessenov will be requested when the pull request is marked ready for review kyessenov is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@RyanTheOptimist RyanTheOptimist

Labels

deps Approval required for changes to Envoy's external dependencies

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jwendell @RyanTheOptimist