-
Notifications
You must be signed in to change notification settings - Fork 25
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
NETOBSERV-101 R&D: Kube enricher write path for downstream operator #135
Conversation
Skipping CI for Draft Pull Request. |
4fc797b
to
4c62b3b
Compare
/hold |
4c62b3b
to
889ae0f
Compare
889ae0f
to
5ce9179
Compare
c0baf73
to
65b70e4
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 (not tested)
65b70e4
to
def552a
Compare
New changes are detected. LGTM label has been removed. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/unhold |
This PR adds
sendAuthToken
option to configure the following components to send their service account token as Authorization header:It will need skip TLS checks & add TenantID
and flowlogs-pipelineto be merged firstEdit, following @OlivierCazade feedback we can directly set
flowlogs-pipeline
clientConfig.authorization
and keeploki-client-go
as is.