Bug #39305
closed
Some Pulp tasks not tracked due to change in Pulp update APIs
Description
Description of problem:
The following table describes tasks that are no longer tracked properly, along with ones that were already patched. Originally we thought only the remote tasks were affected, but it turns out all update tasks are. Technically Pulpcore 3.85 isn't affected by this issue, but since we use the generated Pulp bindings, and potentially all of the plugins were generated based on core 3.90+, the following task returning APIs are all likely affected.
These tasks typically fire off very quickly, so we have not seen bugs around them yet, but there is potential for race conditions since Dynflow doesn't know to wait on them. Plus, when looking in Dynflow, the task information is missing.
| API | Binding default return type | Monkey-patched? | Task tracked on 3.85? |
|---|---|---|---|
| Remotes (RPM, ULN, Container, PullThrough, Ansible×3, OSTree) | *RemoteResponse ❌ |
✅ Yes | ✅ Yes (patch forces AsyncOperationResponse) |
| Remotes (File, Python) | AsyncOperationResponse ✅ |
N/A | ✅ Yes (correct default) |
| Remotes (Deb) | DebAptRemoteResponse ❌ |
❌ No | ❌ Task silently lost |
| Distributions (RPM, Container×2, Deb, Ansible, OSTree) | *DistributionResponse ❌ |
❌ No | ❌ Task silently lost |
| Distributions (File, Python) | AsyncOperationResponse ✅ |
N/A | ✅ Yes (correct default) |
| Repositories (RPM, Container, Deb, Ansible, OSTree) | *RepositoryResponse ❌ |
❌ No | ❌ Task silently lost |
| Repositories (File, Python) | AsyncOperationResponse ✅ |
N/A | ✅ Yes (correct default) |
| ACS (RPM, Deb) | *AlternateContentSourceResponse ❌ |
❌ No | ❌ Task silently lost |
| Exporters (Pulp) | AsyncOperationResponse ✅ |
N/A | ✅ Yes (correct default) |
How reproducible:
100%
Is this issue a regression from an earlier version:
Yes
Steps to Reproduce:
- Sync a repository and look at the RefreshDistribution action in Dynflow (for one example)
Actual behavior:
Output:
--- pulp_tasks: [] task_groups: []
Expected behavior:
Tasks are populated correctly.
Business Impact / Additional info:
Updated by The Foreman Bot 2 months ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/11735 added
Updated by The Foreman Bot about 2 months ago
- Fixed in Releases Katello 4.21.0 added
Updated by Anonymous about 2 months ago
- Status changed from Ready For Testing to Closed
Applied in changeset katello|7b2c98e5d6ce0a17bc9af99b46a4d6eb46318913.
Updated by 
Updated by