-
Notifications
You must be signed in to change notification settings - Fork 123
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
🐛 Pin setuptools and pin upper-constraints #569
🐛 Pin setuptools and pin upper-constraints #569
Conversation
f7d98d0
to
5f78f9f
Compare
/test metal3-centos-e2e-integration-test-release-1-6 metal3-ubuntu-e2e-integration-test-release-1-6 |
/cc @dtantsur |
Oh we had forgotten to pin the requirements, ouch :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Also mention of upper constrain pinning in the PR title.
Usually the setuptools installed by the system is stable but a bit too old compared to the packages we use for ironic, so pinning a more recent version helps avoiding errors and incompatibilities. Also pin upper-constraints since we forgot! Signed-off-by: Riccardo Pittau <elfosardo@gmail.com>
5f78f9f
to
54d741a
Compare
/test metal3-centos-e2e-integration-test-release-1-6 metal3-ubuntu-e2e-integration-test-release-1-6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
@dtantsur can you please have a look? |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dtantsur The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
OCPBUGS-39017: Bump ironic-lib to fix utf8 decoding issue
Usually the setuptools installed by the system is stable but
a bit too old compared to the packages we use for ironic,
so pinning a more recent version helps avoiding
errors and incompatibilities.
Also pin upper-constraints since we forgot!