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

put building/obtaining image instructions before trying to run it. #1655

Merged
merged 3 commits into from May 18, 2024

Conversation

Dieterbe
Copy link
Contributor

@Dieterbe Dieterbe commented May 4, 2024

Proposed Changes

  • put building/obtaining image instructions before trying to run it.
  • put buildx instructions
  • syntax fix

Please check that the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Added yourself to AUTHORS.rst

@Dieterbe Dieterbe force-pushed the clarify-where-to-get-image-before-trying-to-run branch from 518bed5 to 461c1a9 Compare May 4, 2024 16:48
Step 4/23 : COPY requirements* .
When using COPY with more than one source file, the destination must be a directory and end with a /
I was getting this warning...:

DEPRECATED: The legacy builder is deprecated and will be removed in a future release.
            Install the buildx component to build images with BuildKit:
            https://docs.docker.com/go/buildx/

...and later on, this failure:

Step 5/23 : RUN --mount=type=tmpfs,target=/root/.cargo     pip3 wheel --no-cache-dir --wheel-dir /wheels -r requirements_docker.txt
the --mount option requires BuildKit. Refer to https://docs.docker.com/go/buildkit/ to learn how to build images with BuildKit enabled
@Dieterbe Dieterbe force-pushed the clarify-where-to-get-image-before-trying-to-run branch from 563a1a5 to 875fe4b Compare May 4, 2024 17:17
@rolandgeider rolandgeider merged commit dd9c9f8 into master May 18, 2024
4 checks passed
@rolandgeider rolandgeider deleted the clarify-where-to-get-image-before-trying-to-run branch May 18, 2024 13:59
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.

None yet

2 participants