Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upmake docker results in broken image #3109
Comments
This comment has been minimized.
This comment has been minimized.
|
Root cause: prometheus/promu#60 |
brian-brazil
added
kind/bug
priority/P3
labels
Sep 28, 2017
This comment has been minimized.
This comment has been minimized.
taion809
commented
Oct 21, 2017
|
I was able to fix this with a multi-stage docker build. If that is acceptable I can push up my changes for review. |
This comment has been minimized.
This comment has been minimized.
|
@taion809 Oh cool, in general yes! We should just make sure to not diverge too much from how Prometheus is built and it needs to work for cross compilation etc. |
This comment has been minimized.
This comment has been minimized.
taion809
commented
Oct 21, 2017
|
In that case I’ll push a branch up for review. The change made doesn’t
change the makefile or the Use of promu so it should be ok :)
…On Sat, Oct 21, 2017 at 10:45 AM discordianfish ***@***.***> wrote:
@taion809 <https://github.com/taion809> Oh cool, in general yes! We
should just make sure to not diverge too much from how Prometheus is built
and it needs to work for cross compilation etc.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3109 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AE7nIx5Z9G2PQZ41X6vkqbUvfuKRXdQDks5sugN-gaJpZM4PBlrn>
.
|
This comment has been minimized.
This comment has been minimized.
|
@fish IIUC this issue isn't relevant anymore. Do you think it can be closed? |
This comment has been minimized.
This comment has been minimized.
|
@simonpasquier Ah, yah I think this can be closed. Thanks for the ping! |
discordianfish
closed this
Sep 2, 2018
This comment has been minimized.
This comment has been minimized.
lock
bot
commented
Mar 22, 2019
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
discordianfish commentedAug 24, 2017
make dockerrunspromu buildwhich creates a binary with cgo enabled for which the base image doesn't include the necessary shared libs.