-
Notifications
You must be signed in to change notification settings - Fork 277
fluent bit: Add support for outbound http proxy #2114
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2114 +/- ##
==========================================
+ Coverage 58.67% 58.69% +0.01%
==========================================
Files 144 144
Lines 5936 5936
==========================================
+ Hits 3483 3484 +1
+ Misses 2450 2449 -1
Partials 3 3
Continue to review full report at Codecov.
|
4821257
to
47e1de7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Signed-off-by: Sanya Kochhar <kochhars@microsoft.com>
Signed-off-by: Sanya Kochhar <kochhars@microsoft.com>
Signed-off-by: Sanya Kochhar <kochhars@microsoft.com>
Signed-off-by: Sanya Kochhar <kochhars@microsoft.com>
Signed-off-by: Sanya Kochhar <kochhars@microsoft.com> Co-authored-by: Rita Zhang <rita.z.zhang@gmail.com>
Signed-off-by: Sanya Kochhar <kochhars@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @SanyaKochhar ! Is there an e2e test scenario that validates the fluentbit integration somewhere?
@michelleN There's this e2e for fluentbit integration https://github.com/openservicemesh/osm/blob/main/tests/e2e/e2e_fluentbit_test.go - but it doesn't test changes made in this PR |
lgtm |
Description: Adds support for outbound proxy with Fluent Bit. Updates Fluent Bit to v1.6.4.
Affected area:
Please answer the following questions with yes/no.
no, no