Note: This bug is displayed in read-only format because
the product is no longer active in Red Hat Bugzilla.
Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Description of problem:
With pulp2, Satellite/katello allowed to enable cp_rest and pulp_rest logging to see individual API requests to / resonses from [pulp|candlepin] - see https://access.redhat.com/solutions/3001661 . It is/was a nice feature allowing to 1) understand how katello manipulates with pulp/candlepin objects, 2) simplify a reproducer purely to pulp/candlepin.
Since pulp2to3 migration, this does not further work for pulp. The reason is, katello uses PulpcoreClient module that has class Configuration and :debugging, but no way to enable it.
I *think* the basic fix one-liner:
https://github.com/Katello/katello/blob/db5a820f93dad82348b382570fdad90e36c6f51c/app/models/katello/concerns/smart_proxy_extensions.rb#L190
should be:
config.debugging = ::Foreman::Logging.logger('katello/pulp_rest').debug?
anyway these debugs like:
2023-01-02T22:40:59 [D|kat|6ce9a60c] Calling API: DistributionsRpmApi.partial_update ...
2023-01-02T22:40:59 [D|kat|6ce9a60c] HTTP request body param ~BEGIN~
6ce9a60c | {"content_guard":null,"base_path":"RedHat/Library/custom/zoo_product/ZOO_repo","publication":"/pulp/api/v3/publications/rpm/rpm/2e85692d-4690-4ea1-9092-7927187bf874/"}
6ce9a60c | ~END~
6ce9a60c |
2023-01-02T22:40:59 [D|kat|6ce9a60c] HTTP response body ~BEGIN~
6ce9a60c | {"task":"/pulp/api/v3/tasks/2656ab82-e498-4bdb-9424-08ff56776861/"}
6ce9a60c | ~END~
6ce9a60c |
shall be further bit updated by including URI and method used (like "PATCH /pulp/api/v3/distributions/rpm/rpm/f4adc993-8f2e-40d5-a903-54e45d2ad93f/").
Version-Release number of selected component (if applicable):
Sat 6.10 +
How reproducible:
100%
Steps to Reproduce:
1. Try enabling pulp_rest debugs e.g. per https://access.redhat.com/solutions/3001661
Actual results:
1. no API calls printed to production.log
Expected results:
1. API calls like with pulp2 or candlepin to be printed there also for pulp3.
Additional info:
Pavel, the Katello bits of this are already in 6.14, but the Pulp parts have not been dealt with. Should the Pulp bug be split off so we can close this one as CURRENTRELEASE, or if you want to keep these together, should the component be changed to Pulp?
(In reply to Ian Ballou from comment #5)
> Pavel, the Katello bits of this are already in 6.14, but the Pulp parts have
> not been dealt with. Should the Pulp bug be split off so we can close this
> one as CURRENTRELEASE, or if you want to keep these together, should the
> component be changed to Pulp?
Hello,
glad to see the katello bits are fixed. I dont have strong preference about splitting off or changing the component - whatever works for you better, let chose (I feel changing component requires less administrative but it might confuse some tooling? dunno..)
I thought about it a bit, and I'm going to close this as CURRENT RELEASE since there is benefit here with the Katello change alone. Then I'll make a new BZ for the Pulp issue.