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

in_kubernetes_events: update to match consolidated timestamp logic #1277

Merged
merged 1 commit into from Jan 19, 2024

Conversation

ryanohnemus
Copy link
Contributor

Documentation for fluent/fluent-bit#8323

… logic

Signed-off-by: ryanohnemus <ryanohnemus@gmail.com>
@ryanohnemus
Copy link
Contributor Author

@patrick-stephens can you take a look at this one when you get a chance, the matching code was already merged in

@@ -49,3 +48,6 @@ kube_url https://kubernetes.default.svc
name stdout
match *
```

### Event Timestamp
Event timestamp will be created from the first existing field in the following order of precendence: lastTimestamp, firstTimestamp, metadata.creationTimestamp
Copy link
Contributor

Choose a reason for hiding this comment

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

So we can't specify which one we want if all/multiple exist?

@patrick-stephens patrick-stephens merged commit f191428 into fluent:master Jan 19, 2024
4 checks passed
@ryanohnemus ryanohnemus deleted the docs/k8s_ents branch January 20, 2024 19:44
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.

None yet

2 participants