Skip to content

Conversation

@maksim-paskal
Copy link

Users without installed go binnary - can not build this docker image. This PR only add multi-stage feature to build this docker image.

This change do not change any application logic

@it-percona-cla
Copy link

it-percona-cla commented Jun 9, 2021

CLA assistant check
All committers have signed the CLA.

@denisok denisok requested review from a team, askomorokhov and percona-csalguero and removed request for a team June 28, 2021 12:58
@denisok
Copy link

denisok commented Jun 28, 2021

@maksim-paskal maksim-paskal force-pushed the docker-multistage-build branch from a873a10 to 7af5625 Compare June 28, 2021 14:20
askomorokhov
askomorokhov previously approved these changes Jun 29, 2021
Copy link

@askomorokhov askomorokhov left a comment

Choose a reason for hiding this comment

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

Thank you for your contributions!
Looks good to me 👍
Soon we update go version to 1.16

@maksim-paskal
Copy link
Author

@askomorokhov @percona-csalguero - CI is broken - do I need some changes in my PR?

Signed-off-by: Maksim Paskal <paskal.maksim@gmail.com>
@maksim-paskal maksim-paskal force-pushed the docker-multistage-build branch from bf2afb6 to f5cfc2e Compare July 7, 2021 09:45
@maksim-paskal
Copy link
Author

rebased to latest percona:master

@denisok
Copy link

denisok commented Jul 7, 2021

@askomorokhov we have this issue with secrets from forks... could you please test this one from your branch probably and land ?

@askomorokhov
Copy link

@maksim-paskal, thank you for your contribution!
I have merged your Dockerfile in this PR #61

@maksim-paskal maksim-paskal deleted the docker-multistage-build branch July 8, 2021 10:50
@denisok denisok added the award swag prize for the contributors. Please contact Percona if we can't reach you! label Dec 1, 2021
@Aleksa-del
Copy link

@maksim-paskal Hello! We would love to send you a gift from Percona for your contribution. Please, contact us at community-team@percona.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

award swag prize for the contributors. Please contact Percona if we can't reach you! community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants