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 autodetect restarting platform from OpenShift to Kubernetes #1003

Merged
merged 3 commits into from
Apr 5, 2020
Merged

Fix autodetect restarting platform from OpenShift to Kubernetes #1003

merged 3 commits into from
Apr 5, 2020

Conversation

objectiser
Copy link
Contributor

Signed-off-by: Gary Brown gary@brownuk.com

Signed-off-by: Gary Brown <gary@brownuk.com>
@codecov
Copy link

codecov bot commented Apr 4, 2020

Codecov Report

Merging #1003 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1003      +/-   ##
==========================================
- Coverage   64.56%   64.55%   -0.02%     
==========================================
  Files          82       82              
  Lines        6542     6540       -2     
==========================================
- Hits         4224     4222       -2     
  Misses       2177     2177              
  Partials      141      141              
Impacted Files Coverage Δ
pkg/autodetect/main.go 86.70% <100.00%> (-0.17%) ⬇️

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 e79443e...188c860. Read the comment docs.

@objectiser objectiser closed this Apr 4, 2020
@objectiser objectiser reopened this Apr 4, 2020
Signed-off-by: Gary Brown <gary@brownuk.com>
@objectiser
Copy link
Contributor Author

@rubenvp8510 Would you be able to review? It is related to https://issues.redhat.com/browse/TRACING-1117

@objectiser
Copy link
Contributor Author

cc @jpkrohling for future reference :)

@objectiser objectiser closed this Apr 4, 2020
@objectiser objectiser reopened this Apr 4, 2020
@objectiser objectiser closed this Apr 4, 2020
@objectiser objectiser reopened this Apr 4, 2020
Signed-off-by: Gary Brown <gary@brownuk.com>
@objectiser objectiser closed this Apr 4, 2020
@objectiser objectiser reopened this Apr 4, 2020
Copy link
Contributor

@kevinearls kevinearls left a comment

Choose a reason for hiding this comment

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

LGTM. Because of the problems with GH CI I have run the tests on an OCP 4.3.8 cluster and they worked correctly.

@rubenvp8510
Copy link
Collaborator

This looks good.

@objectiser
Copy link
Contributor Author

@kevinearls @rubenvp8510 Thanks - will merge then.

@objectiser objectiser merged commit 404e55f into jaegertracing:master Apr 5, 2020
@objectiser objectiser deleted the detect branch April 5, 2020 15:32
jpkrohling pushed a commit to jpkrohling/jaeger-operator that referenced this pull request Dec 15, 2020
…ertracing#1003)

* Fix autodetect restarting platform from OpenShift to Kubernetes

Signed-off-by: Gary Brown <gary@brownuk.com>

* Add extra check

Signed-off-by: Gary Brown <gary@brownuk.com>

* Further change to trigger tests

Signed-off-by: Gary Brown <gary@brownuk.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.

3 participants