-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Remove OpenTracing images in Updater #2837
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2837 +/- ##
==========================================
- Coverage 53.12% 53.10% -0.02%
==========================================
Files 58 58
Lines 15625 15625
==========================================
- Hits 8300 8298 -2
- Misses 7048 7050 +2
Partials 277 277
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
@@ -138,17 +138,9 @@ jobs: | |||
platforms: linux/arm,linux/arm64,linux/amd64,linux/ppc64le,linux/s390x | |||
needs-updating: ${{ needs.check.outputs.needs-updating-debian }} | |||
- image: alpine | |||
marker: vsr |
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.
we're loosing a vsr marker? is it a problem? can we add it to any of the images?
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.
Ah good call, I've added them back in other images 👍
Removes OpentTracing images as they are on longer separate images.
To be merged after v2.3.0