Skip to content

pin setuptools < 82 before installing the pip packages#370

Merged
openshift-merge-bot[bot] merged 2 commits intoopenstack-k8s-operators:antelopefrom
lmiccini:antelope-setuptools-fix
Feb 17, 2026
Merged

pin setuptools < 82 before installing the pip packages#370
openshift-merge-bot[bot] merged 2 commits intoopenstack-k8s-operators:antelopefrom
lmiccini:antelope-setuptools-fix

Conversation

@lmiccini
Copy link
Contributor

This fixes the ModuleNotFoundError for pkg_resources which was removed in setuptools 82+. The fix ensures setuptools is pinned to version < 82 in two places:

  1. In the pre-commit config for pylint to fix CI failures
  2. In the horizontest container build for runtime environment

This fixes the ModuleNotFoundError for pkg_resources which was
removed in setuptools 82+. The fix ensures setuptools is pinned
to version < 82 in two places:

1. In the pre-commit config for pylint to fix CI failures
2. In the horizontest container build for runtime environment

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This ensures the tox test environments also use setuptools < 82
to avoid the pkg_resources import error in ansible-runner.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@karelyatin
Copy link
Contributor

/lgtm

@dciabrin
Copy link
Contributor

/approve

@fmount
Copy link
Contributor

fmount commented Feb 17, 2026

lgtm thanks @lmiccini

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 17, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dciabrin, karelyatin, lmiccini

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit b403f10 into openstack-k8s-operators:antelope Feb 17, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants