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

Disable image build cache #93

Merged
merged 1 commit into from
Jun 3, 2019
Merged

Disable image build cache #93

merged 1 commit into from
Jun 3, 2019

Conversation

mkuron
Copy link
Member

@mkuron mkuron commented May 31, 2019

Temporary workaround for #92‘s corrupted images

@jngrad

Temporary workaround for #92‘s corrupted images
@jngrad
Copy link
Member

jngrad commented Jun 3, 2019

Should we increase the timeout for the build stage?

@mkuron
Copy link
Member Author

mkuron commented Jun 3, 2019

Sure, give it a try. I'm at a conference this week and won't have much time to look into this, so it would be good if you could take care of it.

@jngrad jngrad merged commit da1f5cb into espressomd:master Jun 3, 2019
@jngrad
Copy link
Member

jngrad commented Jun 3, 2019

The master branch just failed because the line

test "$CI_COMMIT_REF_NAME" != "master" || cmd "/kaniko/executor --context $PWD --dockerfile Dockerfile-$docker_tag* --destination $CI_REGISTRY/$CI_PROJECT_PATH/$CI_JOB_NAME"

builds the image again, which causes the same files/folders/symlinks to be created again, leading to name conflicts. This was not an issue before since the cached layers were re-used during the second build.

Before kaniko, docker was used to build once and push to 2 destinations. However I could not find a command line option in kaniko to push an image without building it first, and couldn't experiment with the kaniko image as it doesn't seem to run in interactive mode.

@mkuron
Copy link
Member Author

mkuron commented Jun 3, 2019

Ok, I‘ll try to adapt the build script this evening to only build once.

This was referenced Jun 3, 2019
@mkuron mkuron deleted the patch-8 branch September 25, 2019 10:18
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.

2 participants