Skip to content

bazel: tcmalloc support (#415).#771

Merged
mattklein123 merged 3 commits intoenvoyproxy:masterfrom
htuch:tcmalloc
Apr 17, 2017
Merged

bazel: tcmalloc support (#415).#771
mattklein123 merged 3 commits intoenvoyproxy:masterfrom
htuch:tcmalloc

Conversation

@htuch
Copy link
Copy Markdown
Member

@htuch htuch commented Apr 14, 2017

No description provided.

Comment thread bazel/BUILD Outdated

config_setting(
name = "disable_tcmalloc",
values = {"linkopt": "-fsanitize=address"},
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: Instead of keying off of this specific option, should we actually set some other option when we are doing sanitizing? Should we also allow tcmalloc to be disabled by a generic option? (For example, newer valgrinds are totally broken with tcmalloc).

@mattklein123
Copy link
Copy Markdown
Member

@htuch needs master merge to resolve conflicts.

@mattklein123 mattklein123 merged commit 7406fad into envoyproxy:master Apr 17, 2017
rshriram pushed a commit to rshriram/envoy that referenced this pull request Oct 30, 2018
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Description: Virtual Clusters allow Envoy Mobile to emit stats based on matching criteria. This allows users to single out important traffic for their application. This PR adds an API to the client builder to add virtual cluster configuration and emission of virtual cluster stats.

Note: the client builder API exposed here is not ideal #770 tracks enhancing this.
Risk Level: low
Testing: unit tests, and local stats test. #769 tracks adding CI tests for stats
Docs Changes: added docs for the new API.

Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Description: Virtual Clusters allow Envoy Mobile to emit stats based on matching criteria. This allows users to single out important traffic for their application. This PR adds an API to the client builder to add virtual cluster configuration and emission of virtual cluster stats.

Note: the client builder API exposed here is not ideal #770 tracks enhancing this.
Risk Level: low
Testing: unit tests, and local stats test. #769 tracks adding CI tests for stats
Docs Changes: added docs for the new API.

Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
mathetake added a commit that referenced this pull request Mar 3, 2026
**Description**

This PR adds the proposal for supporting Integration with Endpoint
Picker(GIE)

Related to envoyproxy/ai-gateway#423

---------

Signed-off-by: bitliu <bitliu@tencent.com>
Co-authored-by: Takeshi Yoneda <t.y.mathetake@gmail.com>
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