Skip to content

Fix rpmlint failures in RPM packaging - #5163

Merged
happz merged 3 commits into
mainfrom
tcornell-fix-rpmlint
Sep 24, 2026
Merged

happz merged 3 commits into
mainfrom
tcornell-fix-rpmlint

Conversation

@tcornell-bus

@tcornell-bus tcornell-bus commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Update rpmlintrc filter for:

  • The tmt package rename and jinja2 template files
  • Provisioner spelling warnings and errors
  • no-documentation warnings
  • Drop the redundant libvirt-daemon-config-network Requires already pulled in by python3-testcloud.
    • Details: python3-testcloud (required since Fedora 43 through current Rawhide) lists libvirt-daemon-config-network as one of its own Requires. Since tmt+provision-virtual already requires python3-testcloud >= 0.11.7, the libvirt-daemon-config-network dependency was being pulled in twice — once explicitly by tmt and once transitively through testcloud.

Assisted-by: Cursor AI

Fixes: #3323

Pull Request Checklist

  • implement the feature

@tcornell-bus tcornell-bus added the packaging Changes related to the rpm packaging label Sep 15, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Sep 15, 2026
@tcornell-bus tcornell-bus moved this from backlog to review in planning Sep 15, 2026
@tcornell-bus tcornell-bus added the ci | full test Pull request is ready for the full test execution label Sep 15, 2026
Comment thread packaging/rpm/tmt.rpmlintrc Outdated

@LecrisUT LecrisUT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can also skip the no-documentation issues?

Comment thread packaging/rpm/tmt.rpmlintrc Outdated
Comment thread packaging/rpm/tmt.rpmlintrc Outdated
@psss psss changed the title fix rpmlint failures in RPM packaging Fix rpmlint failures in RPM packaging Sep 22, 2026
@therazix therazix moved this from review to merge in planning Sep 23, 2026
Update rpmlintrc filters for the tmt package rename and provisioner
spelling.
Drop the redundant libvirt-daemon-config-network Requires
already pulled in by python3-testcloud.

Assisted-by: Cursor AI
Catch warnings and errors for provisioner spelling
@LecrisUT
LecrisUT force-pushed the tcornell-fix-rpmlint branch from 577e851 to 604dd12 Compare September 23, 2026 20:50
@happz
happz merged commit 02d5851 into main Sep 24, 2026
32 checks passed
@happz
happz deleted the tcornell-fix-rpmlint branch September 24, 2026 06:59
@therazix therazix added this to the 1.79 milestone Sep 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci | full test Pull request is ready for the full test execution packaging Changes related to the rpm packaging

Projects

Status: done

Development

Successfully merging this pull request may close these issues.

Fix or waive failures reported by rpmlint

5 participants