Skip to content

Remove pkg resources#14142

Merged
jan-cerny merged 4 commits intoComplianceAsCode:masterfrom
Mab879:remove_pkg_resources
Nov 20, 2025
Merged

Remove pkg resources#14142
jan-cerny merged 4 commits intoComplianceAsCode:masterfrom
Mab879:remove_pkg_resources

Conversation

@Mab879
Copy link
Member

@Mab879 Mab879 commented Nov 19, 2025

Description:

Remove pkg_resources usage from the code base and replace with custom solution.

It might not be as featured as pkg_resources but due limitations on how the python package packaging is distributed this was the easiest to use.

Rationale:

Fixes #13902

Review Hints:

  • Build some products
  • Run the Python test suite

Replaced pkg_resources with a custom RequirementParser.
It implements just enough of pkg_resources.Requirement to work for our
project.

Fixes: ComplianceAsCode#13902
Claude code was used in part in creating this commit.
No longer needed as pkg_resources has been removed from
the code based.
@Mab879 Mab879 added the Infrastructure Our content build system label Nov 19, 2025
@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Used by openshift-ci bot. label Nov 19, 2025
@openshift-ci
Copy link

openshift-ci bot commented Nov 19, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@Mab879
Copy link
Member Author

Mab879 commented Nov 19, 2025

Leaving in draft state until we cut the branch for the 0.1.79 release, no need to make this change right before a release.

@Mab879 Mab879 added this to the 0.1.80 milestone Nov 20, 2025
@Mab879 Mab879 marked this pull request as ready for review November 20, 2025 13:07
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Used by openshift-ci bot. label Nov 20, 2025
@jan-cerny jan-cerny self-assigned this Nov 20, 2025
@openshift-ci
Copy link

openshift-ci bot commented Nov 20, 2025

@Mab879: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-aws-openshift-node-compliance ebdcdff link true /test e2e-aws-openshift-node-compliance

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@jan-cerny
Copy link
Collaborator

/test e2e-aws-openshift-node-compliance

Copy link
Collaborator

@jan-cerny jan-cerny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I haven't found any remaining use of pkg_resources in the project. Python unit tests passed locally.

@jan-cerny jan-cerny merged commit 24dc454 into ComplianceAsCode:master Nov 20, 2025
139 of 140 checks passed
@Mab879 Mab879 deleted the remove_pkg_resources branch November 20, 2025 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Infrastructure Our content build system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bad api call: pkg_resources is deprecated

2 participants