Conversation
m-ildefons
left a comment
There was a problem hiding this comment.
AFAIK, the dependencies of the robot tests are managed through requirements.in/requirements.txt in the harvester_robot_tests directory, not the top-level requirements files.
Aside from that, I'd prefer to pin this with the other dependencies rather than have it as separate install command documented.
| > You need to install `setuptools` version 81 because `pkg_resources` has been removed in version >= 82. See https://github.com/pypa/setuptools/pull/5007. | ||
|
|
||
| ```bash | ||
| pip install setuptools==81.0.0 |
There was a problem hiding this comment.
Why not just add this to test-requirements.in?
The fact that this isn't pinned there is even warned about in test-requirements.txt:
# WARNING: The following packages were not pinned, but pip requires them to be
# pinned when the requirements file includes hashes and the requirement is not
# satisfied by a package already installed. Consider using the --allow-unsafe flag.
# setuptools
There was a problem hiding this comment.
Yeah, if this package is missing in a fresh robot env., we should add it in requirement.in and regenerate requirement.txt too.
There was a problem hiding this comment.
pip-tools is considering setuptools as unsafe. But we can force it with --allow-unsafe. I think that’s fine in our case, as we specifically need this version of setuptools.
There was a problem hiding this comment.
With my first attempt at the PR, I didn’t want to do anything that went against the ‘normal’ behaviour of pip-tools; but I think forcing it is ultimately the better and simpler solution.
de15845 to
6aef8b4
Compare
... which is caused due pypa/setuptools#5007. - Add `setuptools==81.0.0` to forceible install the required package. - Fix crash when using a nightly build cluster which returns a git hash instead a semver. ``` InvalidVersion: Invalid version: 'c6d323ec' ``` Signed-off-by: Volker Theile <vtheile@suse.com>
6aef8b4 to
4001606
Compare
... which is caused due pypa/setuptools#5007.
Additional fixes: