Skip to content

fix: Vendor infra.leapp.modprobe inside the collection#343

Merged
spetrosi merged 1 commit intoredhat-cop:mainfrom
spetrosi:vendor-modprobe
Jan 21, 2026
Merged

fix: Vendor infra.leapp.modprobe inside the collection#343
spetrosi merged 1 commit intoredhat-cop:mainfrom
spetrosi:vendor-modprobe

Conversation

@spetrosi
Copy link
Collaborator

@spetrosi spetrosi commented Jan 20, 2026

This is required to remove dependency on community.general collection.

I've done this by downloading community-general-11.4.2 from https://galaxy.ansible.com/download/community-general-11.4.2.tar.gz and copying plugins/modules/modprobe.py from it to this collection.
I took version 11.4.2 because this is what rhel-system-roles uses, this version is verified to work with all Python versions that we need to support.
community.general now has a tendency to add features that are not supported on older Python which RHEL 7 uses, so we cannot use latest version.
In the module I've added a warning - "WARNING: Do not use this plugin directly! It is only for role internal use." and replaced community.general. with infra.leapp..
Then had to copy plugins/doc_fragments/attributes.py because it is needed to build module documentation.

@spetrosi spetrosi force-pushed the vendor-modprobe branch 2 times, most recently from fac6395 to 1da2789 Compare January 20, 2026 19:49
@spetrosi
Copy link
Collaborator Author

[citest]

@spetrosi
Copy link
Collaborator Author

@PeterMocary @tomasfratrik FYI

@spetrosi
Copy link
Collaborator Author

leapp-tests_remediations_ tests in each RHEL runs the task manage_one_kernel_module | Load or unload kernel module successfully, so this change is covered by tests and is passing.

This is required to remove dependency on community.general collection
- community.general.redhat_subscription
- community.general.modprobe
- community.general.archive
- community.general.rhsm_repository
Copy link
Collaborator Author

@spetrosi spetrosi Jan 21, 2026

Choose a reason for hiding this comment

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

Removing couple more entries that were replaced in previous PRs.

@spetrosi
Copy link
Collaborator Author

[citest]

@spetrosi spetrosi merged commit 703392e into redhat-cop:main Jan 21, 2026
45 of 47 checks passed
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.

2 participants