fix(oci): account for OCA and pqrpm GPG key dbs

[src-up]

CIV counted only gpg-pubkey in the default rpmdb and expected 3 keys on all RHEL 9+. Still not sure where that number 3 is from.

On OCI, Oracle Cloud Agent imports 2 Oracle Linux keys at first boot.

• On RHEL 9.7+ the Red Hat keys are imported by pqrpm into /usr/lib/pqrpm/lib/sysimage/rpm, so the default db only has the OCA keys.
• RHEL 10 keeps Red Hat keys in the default db (3 RH + 2 OCA).

So Expect 2 in the default db on OCI 9.7–9.x, 5 on OCI 10, and add test_pqrpm_gpg_keys for the 2 Red Hat keys in the pqrpm db. Other clouds are unchanged.

[sourcery-ai]

Hey - I've left some high level feedback:

• The release = version.parse(host.system_info.release) logic is duplicated across tests; consider centralizing release parsing or using a helper to avoid repetition and reduce the chance of inconsistencies.
• The conditional branches in test_number_gpg_keys are becoming complex and cloud/distribution-specific; consider extracting the key-count logic into a small helper function to make the test easier to read and to update when new combinations are added.
• The hard-coded key counts (e.g., 2 and 5) and paths (e.g., /usr/lib/pqrpm/lib/sysimage/rpm) are encoded as literals; consider defining them as named constants to clarify their meaning and make future adjustments simpler.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- The `release = version.parse(host.system_info.release)` logic is duplicated across tests; consider centralizing release parsing or using a helper to avoid repetition and reduce the chance of inconsistencies.
- The conditional branches in `test_number_gpg_keys` are becoming complex and cloud/distribution-specific; consider extracting the key-count logic into a small helper function to make the test easier to read and to update when new combinations are added.
- The hard-coded key counts (e.g., 2 and 5) and paths (e.g., `/usr/lib/pqrpm/lib/sysimage/rpm`) are encoded as literals; consider defining them as named constants to clarify their meaning and make future adjustments simpler.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[sshmulev]

LGTM +1

[src-up]

@F-X64 do these changes look OK to you

[F-X64]

@src-up Changes look great, I got nothing to add to @sshmulev 's review