Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use UTC in es-index-cleaner #3261

Merged
merged 1 commit into from
Sep 13, 2021

Conversation

pavolloffay
Copy link
Member

Resolves #3228

Signed-off-by: Pavol Loffay <p.loffay@gmail.com>
@pavolloffay pavolloffay requested a review from a team as a code owner September 13, 2021 13:01
@pavolloffay
Copy link
Member Author

Do not merge yet!

@codecov
Copy link

codecov bot commented Sep 13, 2021

Codecov Report

Merging #3261 (2fadf49) into master (ef8a559) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3261      +/-   ##
==========================================
- Coverage   95.85%   95.84%   -0.02%     
==========================================
  Files         259      259              
  Lines       15398    15398              
==========================================
- Hits        14760    14758       -2     
- Misses        549      552       +3     
+ Partials       89       88       -1     
Impacted Files Coverage Δ
cmd/collector/app/server/zipkin.go 58.53% <0.00%> (-12.20%) ⬇️
pkg/config/tlscfg/cert_watcher.go 92.63% <0.00%> (-2.11%) ⬇️
plugin/storage/badger/spanstore/reader.go 96.21% <0.00%> (+0.70%) ⬆️
cmd/query/app/static_handler.go 98.80% <0.00%> (+1.19%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ef8a559...2fadf49. Read the comment docs.

@yurishkuro yurishkuro merged commit 7756bf4 into jaegertracing:master Sep 13, 2021
@joe-elliott joe-elliott added this to the v1.27.0 milestone Oct 6, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

es-index-cleaner:latest is not working as expected
4 participants