Skip to content

dist: include Dockerfile in build context#24

Merged
steveej merged 1 commit into
openshift:masterfrom
steveej-forks:fix-docker-on-ci
Dec 5, 2018
Merged

dist: include Dockerfile in build context#24
steveej merged 1 commit into
openshift:masterfrom
steveej-forks:fix-docker-on-ci

Conversation

@steveej
Copy link
Copy Markdown
Contributor

@steveej steveej commented Dec 4, 2018

This seems to be necessary on the CI accordin to the error message:

unable to prepare context: The Dockerfile
(/var/lib/jenkins/workspace/openshift-cincinnati-gh-build-master/dist/Dockerfile)
must be within the build context
(/var/lib/jenkins/workspace/openshift-cincinnati-gh-build-master/dist/..//release-20181204.181943)

@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Dec 4, 2018
Comment thread dist/build_deploy.sh Outdated
Comment thread dist/build_deploy.sh Outdated
This seems to be necessary on the CI accordin to the error message:

```log
unable to prepare context: The Dockerfile
(/var/lib/jenkins/workspace/openshift-cincinnati-gh-build-master/dist/Dockerfile)
must be within the build context
(/var/lib/jenkins/workspace/openshift-cincinnati-gh-build-master/dist/..//release-20181204.181943)
```
@riuvshin
Copy link
Copy Markdown
Contributor

riuvshin commented Dec 4, 2018

/lgtm

@openshift-ci-robot
Copy link
Copy Markdown

@riuvshin: changing LGTM is restricted to assignees, and only openshift/cincinnati repo collaborators may be assigned issues.

Details

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@steveej
Copy link
Copy Markdown
Contributor Author

steveej commented Dec 4, 2018

I'll reproduce the issue and fix in #20 in order to prove it.

@steveej steveej merged commit 521af74 into openshift:master Dec 5, 2018
@steveej steveej deleted the fix-docker-on-ci branch December 6, 2018 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants