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

Fix flaky ES span reader test due to date differences #1847

Merged

Conversation

pavolloffay
Copy link
Member

Resolves #1713

Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
@codecov
Copy link

codecov bot commented Oct 9, 2019

Codecov Report

Merging #1847 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1847   +/-   ##
=======================================
  Coverage   98.44%   98.44%           
=======================================
  Files         197      197           
  Lines        9627     9627           
=======================================
  Hits         9477     9477           
  Misses        114      114           
  Partials       36       36

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 1616539...6e77d7a. Read the comment docs.

@yurishkuro yurishkuro merged commit 9a15c1b into jaegertracing:master Oct 9, 2019
@yurishkuro
Copy link
Member

Thanks Pavol

radekg pushed a commit to Klarrio/jaeger that referenced this pull request Oct 28, 2019
…1847)

* Fix flaky ES span reader test due to date differences

Signed-off-by: Pavol Loffay <ploffay@redhat.com>

* fmt

Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Signed-off-by: radekg <radek@gruchalski.com>
backjo pushed a commit to backjo/jaeger that referenced this pull request Dec 19, 2019
…1847)

* Fix flaky ES span reader test due to date differences

Signed-off-by: Pavol Loffay <ploffay@redhat.com>

* fmt

Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Signed-off-by: Jonah Back <jonah@jonahback.com>
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.

Flaky ES test - TestSpanReader_multiRead_followUp_query
2 participants