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

Opt statically linked OpenSSL/BoringSSL applications into TLS tracing by default #1625

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

ddelnano
Copy link
Member

Summary: Opt statically linked OpenSSL/BoringSSL applications into TLS tracing by default

Relevant Issues: #692

Type of change: /kind feature

Test Plan: Feature flag was used for Pixie owned clusters.

… by default

Signed-off-by: Dom Del Nano <ddelnano@pixielabs.ai>
@ddelnano ddelnano marked this pull request as ready for review July 20, 2023 18:11
@ddelnano ddelnano requested a review from a team July 20, 2023 19:52
@aimichelle aimichelle merged commit ea9a357 into pixie-io:main Jul 20, 2023
etep pushed a commit to etep/pixie that referenced this pull request Jul 25, 2023
… by default (pixie-io#1625)

Summary: Opt statically linked OpenSSL/BoringSSL applications into TLS
tracing by default

Relevant Issues: pixie-io#692

Type of change: /kind feature

Test Plan: Feature flag was used for Pixie owned clusters.

Signed-off-by: Dom Del Nano <ddelnano@pixielabs.ai>
vihangm pushed a commit that referenced this pull request Aug 17, 2023
…d announce BoringSSL support (#1678)

Summary: Update static tls (BoringSSL) tracing feature flag default to
true and announce BoringSSL support

Relevant Issues: #692

Type of change: /kind feature

Test Plan: This tls tracing has been enabled since v0.14.3's release
last month (#1625) and the metrics for internal clusters have been
validated

Changelog Message:
```release-notes
Enhance TLS tracing to support statically linked OpenSSL and BoringSSL (OpenSSL API compatible libraries).
```

Signed-off-by: Dom Del Nano <ddelnano@pixielabs.ai>
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