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

[Dockerfile.test] explicitly COPY .git/ #2

Open
wants to merge 21 commits into
base: github-app
Choose a base branch
from

Conversation

jpopelka
Copy link

Tests in user-cont#237 are failing because CI can't build the test image (from Dockerfile.test).

10:48:09  Step 13/15 : RUN cd /tmp/tmp/ && pip3 install . && rm -rf /tmp/tmp/
...
10:48:13        File "/tmp/pip-req-build-b3d672cb/.eggs/setuptools_scm-3.3.3-py3.7.egg/setuptools_scm/__init__.py", line 113, in _do_parse
10:48:13          "use git+https://github.com/user/proj.git#egg=proj" % config.absolute_root
10:48:13      LookupError: setuptools-scm was unable to detect version for '/tmp/pip-req-build-b3d672cb'.
10:48:13      
10:48:13      Make sure you're either building from a fully intact git repository or PyPI tarballs. Most other sources (such as GitHub's tarballs, a git checkout without the .git folder) don't contain the necessary metadata and will not work.

It seems that there's no .git/ folder.
make image-test works on my machine and I don't see any problem in Dockerfile.test.
But let's try to explicitly copy the .git/

@jpopelka
Copy link
Author

damn, the CI is not run at all here, so I guess you'd have to merge this first to see how it goes :-D

@jpopelka
Copy link
Author

Now I remember we saw this in packit-service too. The Jenkinsfile change should fix it.

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.

3 participants