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

Support byte sizes in label filters. #5

Merged
merged 3 commits into from
Oct 12, 2020

Conversation

jeschkies
Copy link

What this PR does / why we need it:
This patch extends the duration label filter with support for byte sizes
such as 1kB and 42MiB.

Checklist

  • Documentation added
  • Tests updated

This patch extends the duration label filter with support for byte sizes
such as `1kB` and `42MiB`.
pkg/logql/expr.y Outdated Show resolved Hide resolved
Copy link
Owner

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

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

LGTM

@cyriltovena cyriltovena merged commit 25dd730 into cyriltovena:logql-parser Oct 12, 2020
@jeschkies jeschkies deleted the karsten/bytes-filter branch October 12, 2020 10:25
jeschkies referenced this pull request in jeschkies/loki Dec 21, 2021
* add TestCreateLokiStack_WhenGetReturnsNotFound_DoesNotError
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.

2 participants