backend: make get_package_by_prn work with Pulp domains - #4482
Conversation
See fedora-copr#4472 I tested that PR only locally, where Pulp domains are disabled. On STG and production it fails because it tries to query e.g. this: https://packages.redhat.com/pulp/api/v3/content/rpm/packages/01a05b15-7756-7f24-88f2-f09387e38465/ instead of this: https://packages.redhat.com/api/pulp/public-copr/api/v3/content/rpm/packages/01a05b15-7756-7f24-88f2-f09387e38465/
|
/packit test |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthrough
ChangesPulp package lookup
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to Package lookups now use the domain-aware Pulp API route while preserving the existing GET response behavior. No merge-blocking risk is evident. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
See #4472
I tested that PR only locally, where Pulp domains are disabled. On STG and production it fails because it tries to query e.g. this:
https://packages.redhat.com/pulp/api/v3/content/rpm/packages/01a05b15-7756-7f24-88f2-f09387e38465/
instead of this:
https://packages.redhat.com/api/pulp/public-copr/api/v3/content/rpm/packages/01a05b15-7756-7f24-88f2-f09387e38465/