ci/build: Add .dockerignore to limit build context #13618
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
phlax
force-pushed
the
ci-add-dockerignore
branch
from
a950e98 to
e29661d
Compare
phlax
changed the title
phlax
force-pushed
the
ci-add-dockerignore
branch
3 times, most recently
from
185b647 to
1d54d25
Compare
phlax
force-pushed
the
ci-add-dockerignore
branch
from
35ecde9 to
9628eb7
Compare
phlax
changed the title
phlax
changed the title
phlax
force-pushed
the
ci-add-dockerignore
branch
from
9628eb7 to
6aa1d91
Compare
|
/retest |
|
Retrying Azure Pipelines. |
phlax
force-pushed
the
ci-add-dockerignore
branch
from
6aa1d91 to
257031f
Compare
|
/retest |
|
Retrying Azure Pipelines. |
Signed-off-by: Ryan Northey <ryan@synca.io>
phlax
force-pushed
the
ci-add-dockerignore
branch
from
257031f to
ef75bf9
Compare
mattklein123
approved these changes
Oct 23, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Ryan Northey ryan@synca.io
Commit Message: ci/build: Add .dockerignore to limit build context
Additional Description:
This is a
workaroundfix for #13613 that limits the context that is sent to the Docker daemon when buildingIt doesnt make much difference to CI (windows build may be a little quicker) but it does improve experience for end users building locally
by default it excludes everything and then lists paths to include. I have added the
local/dir as a path to allow adding local artifacts into a build - i will document this as part of #13490 (or once this has landed)Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] Fix #13613
[Optional Deprecated:]