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
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