Skip to content

Update pulp_rpm main branch CI to use a CS10 based image#4462

Open
dralley wants to merge 4 commits into
pulp:mainfrom
dralley:ci-image
Open

Update pulp_rpm main branch CI to use a CS10 based image#4462
dralley wants to merge 4 commits into
pulp:mainfrom
dralley:ci-image

Conversation

@dralley
Copy link
Copy Markdown
Contributor

@dralley dralley commented May 15, 2026

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

It only worked by accident on EL9.

Assisted-By: claude-opus-4.6
@github-actions github-actions Bot added the multi-commit Added when a PR consists of more than one commit label May 22, 2026
@dralley dralley requested a review from pedro-psb May 22, 2026 03:18
dralley added 2 commits May 21, 2026 23:19
Instead of using the fixture package, which includes a legacy v4
signature, use a package built by rpm-rs with only a v6 signature.

EL10 errors on "rpm --addsign --rpmv6" if the package has a legacy
signature.

Assisted-By: claude-opus-4.6
On RPM >= 4.19 (EL10), `rpm --addsign` refuses to operate on packages
that already contain a legacy v4 signature. Switch the test to use a
signing service backed by `rpmsign --resign`, which replaces the
existing signature and works on both EL9 and EL10.

Also consolidate the three signing script string constants into a single
SIGNING_SCRIPT_TEMPLATE with a _make_signing_script() helper.

Assisted-By: claude-opus-4.6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

multi-commit Added when a PR consists of more than one commit no-changelog no-issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant