E532 Cache SDK Images As Artifacts by nigthknight · Pull Request #655 · secureCodeBox/secureCodeBox · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@nigthknight
Copy link
Member
@nigthknight nigthknight commented Sep 20, 2021

If applied, this PR (depends on PR #653) will

  • Refactor SDK Matrix To Use Makefiles
  • Upload Images As Artifacts
  • Remove DockerHub upload for hook-sdk and parser-sdk
  • Update CI To Use SDKs From Artifact

@nigthknight nigthknight added ci Changes to the continuous integration setup maintenance labels Sep 20, 2021
@nigthknight nigthknight self-assigned this Sep 20, 2021
@nigthknight
Copy link
Member Author
nigthknight commented Sep 21, 2021

The docker-meta-action seems to be a little flaky at the moment.
Apart from that, this PR should be ready for review
See #659

@nigthknight nigthknight marked this pull request as ready for review September 21, 2021 06:59
@nigthknight nigthknight marked this pull request as draft September 21, 2021 07:17
@nigthknight nigthknight marked this pull request as ready for review September 21, 2021 07:40
@nigthknight nigthknight marked this pull request as draft September 21, 2021 12:56
@nigthknight nigthknight force-pushed the ci/ci-workflow-registryless branch from d2a7007 to ef6b647 Compare September 21, 2021 13:00
@nigthknight nigthknight marked this pull request as ready for review September 21, 2021 13:19
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
I refactored the sdk image builds to use the makefiles and uploaded the
created images as artifacts to be able to remove the Registry secrets
from the workflow.

Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
The default build driver for the used buildx action is `docker-container`.
This means the image is build within a buildkit container with no access
to images loaded into docker.
This is a problem because we need to import the base images for
parser-sdk and hook-sdk to build the images for parsers and hooks
without using a container registry caching these base images.

To fix this I changed the build driver to `docker` to build the image
with the docker daemon.

See:
https://github.com/docker/setup-buildx-action
github.com/docker/buildx/blob/master/docs/reference/buildx_create.md#driver

Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
I changed the number of retention days to one because this pipeline is
used for developement only.
Release Images are build using a special build pipeline and pushed to
docker hub.

Signed-off-by: Yannik Fuhrmeister <yannik.fuhrmeister@iteratec.com>
@nigthknight nigthknight force-pushed the ci/ci-workflow-registryless branch from 8c01bcc to d823f5a Compare September 27, 2021 06:35
@J12934 J12934 merged commit 3bda27c into main Sep 28, 2021
@J12934 J12934 deleted the ci/ci-workflow-registryless branch September 28, 2021 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes to the continuous integration setup maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0