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

Added docker buildx support in cico #483

Closed
wants to merge 2 commits into from

Conversation

prabhav-thali
Copy link

Signed-off-by: Prabhav Thali Prabhav.Thali1@ibm.com

What does this PR do?

Adds docker buildx support in cico scripts to build multi-arch images.

@che-bot
Copy link
Contributor

che-bot commented Jun 1, 2020

Can one of the admins verify this patch?

1 similar comment
@che-bot
Copy link
Contributor

che-bot commented Jun 1, 2020

Can one of the admins verify this patch?

Signed-off-by: Prabhav Thali <Prabhav.Thali1@ibm.com>
@ericwill
Copy link
Contributor

@prabhav-thali can you rebase your PR please?

@prabhav-thali
Copy link
Author

@ericwill I have updated the PR.

@ericwill
Copy link
Contributor

Thanks! The PR appears to be a failing the shellcheck -- can you take a look?

@benoitf
Copy link
Contributor

benoitf commented Sep 29, 2020

as ci.centos.org is being shut-downed before end the year I would not add new stuff there.

@prabhav-thali
Copy link
Author

@benoitf Thanks for the clarification. Could you please let us know whats your plan after ci.centos.org sunset and how we can enable s390x support to Che docker images?

@benoitf
Copy link
Contributor

benoitf commented Sep 30, 2020

@prabhav-thali moving to github actions with buildx support

@nickboldt
Copy link
Contributor

nickboldt commented Dec 7, 2020

As the cico scripts are no longer being maintained (centos CI is EOL), this PR will never be merged.

Instead, please submit a new PR with code that calls a PLATFORMS file and runs buildx like this:

https://github.com/eclipse/che-plugin-registry/blob/master/.ci/sidecar-build-publish.sh#L31-L35
or like this:

https://github.com/che-incubator/che-dashboard-next/blob/master/.github/workflows/dockerimage.yml#L35


See also:

PR to move to GH action in plugin registry:

@nickboldt nickboldt closed this Dec 7, 2020
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.

5 participants