Skip to content
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.

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

Fix crash-on-startup on macOS 12 (#18779) #18864

Merged

Conversation

ggreenway
Copy link
Contributor

gperftools_tcmalloc is not compatible with macOS 12, resulting in a
crash on startup. The crash happens very early, before CLI args are
parsed.

Fixes #17535

Signed-off-by: Greg Greenway [email protected]
(cherry picked from commit b9939c6)

Commit Message:
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:]

ggreenway and others added 2 commits November 2, 2021 08:17
gperftools_tcmalloc is not compatible with macOS 12, resulting in a
crash on startup. The crash happens very early, before CLI args are
parsed.

Fixes envoyproxy#17535

Signed-off-by: Greg Greenway <[email protected]>
(cherry picked from commit b9939c6)
Signed-off-by: Greg Greenway <[email protected]>
@ggreenway
Copy link
Contributor Author

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #18864 (comment) was created by @ggreenway.

see: more, trace.

Copy link
Member

@davinci26 davinci26 left a comment

Choose a reason for hiding this comment

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

LGTM

@ggreenway
Copy link
Contributor Author

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #18864 (comment) was created by @ggreenway.

see: more, trace.

@ggreenway ggreenway enabled auto-merge (squash) November 2, 2021 19:47
@alyssawilk alyssawilk disabled auto-merge November 2, 2021 20:36
@alyssawilk alyssawilk merged commit 4cc4e60 into envoyproxy:release/v1.20 Nov 2, 2021
aidanhahn pushed a commit to datawire/envoy that referenced this pull request Jan 13, 2022
gperftools_tcmalloc is not compatible with macOS 12, resulting in a
crash on startup. The crash happens very early, before CLI args are
parsed.

Fixes envoyproxy#17535
(cherry picked from commit b9939c6)

Signed-off-by: Greg Greenway <[email protected]>
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.

3 participants