Skip to content

Commit

Permalink
fix: remove the log field after its parsed into data field TDE-1337 (#…
Browse files Browse the repository at this point in the history
…893)

#### Motivation

The logs are parsed into a `data` field by fluentbit. It is unecessary
duplication to keep the `log` field once it has been merged into `data`.

#### Modification

Tells fluentbit to not keep the `log` field once it's parsed into
`data`.

#### Checklist

_If not applicable, provide explanation of why._

- [ ] Tests updated
- [ ] Docs updated
- [x] Issue linked in Title
  • Loading branch information
paulfouquet authored Dec 5, 2024
1 parent 7590309 commit 61aa899
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions infra/charts/fluentbit.ts
Original file line number Diff line number Diff line change
Expand Up @@ -100,6 +100,12 @@ HC_Period 5
* See: https://github.com/argoproj/argo-workflows/issues/13314
*/
bufferSize: '128k',
/**
* Remove the `log` field from the logs
* as it has been parsed into the `data` field
* so we don't want duplicated logs
*/
keepLog: 'Off',
},
// FIXME: `livenessProbe` and `readinessProbe` deactivated https://github.com/aws/eks-charts/issues/995
livenessProbe: false,
Expand Down

0 comments on commit 61aa899

Please sign in to comment.