-
Notifications
You must be signed in to change notification settings - Fork 566
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
[CORE-2189] Add dynamic log pages #9898
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #9898 +/- ##
==========================================
+ Coverage 58.16% 58.26% +0.09%
==========================================
Files 611 612 +1
Lines 74543 74685 +142
==========================================
+ Hits 43359 43515 +156
+ Misses 30639 30623 -16
- Partials 545 547 +2 ☔ View full report in Codecov by Sentry. |
A quick search through our commands didn’t turn up any existing time flags.
Seems more idiomatic to me.
…-dynamic-log-pages
Due to looking for the previous log dynamically, one will not be found if it does not exist.
Retry invalid batch size once, and use constants rather than magic strings for log direction.
…m/pachyderm into rau/core-2189-dynamic-log-pages
…-2189-dynamic-log-pages
…m/pachyderm into rau/core-2189-dynamic-log-pages
The Apache-2.0–licensed code used in this PR ultimately comes from Loki commit 6e67d1e via Loki commit 3c78579; we have a previously-approved use of Loki commit 6e67d1e via Loki commit a9d85de |
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
This adapts code from Loki’s logcli. Note that this version of logcli was made available under the Apache License 2.0.