Skip to content

[PR #7587/71c6e2c0 backport][3.108] [PULP-1299] Add setting to allow admins to avoid timeout issues with immediate tasks#7596

Merged
pedro-psb merged 1 commit into3.108from
patchback/backports/3.108/71c6e2c0cb5d79db06e1e4cfdc634d9c15246afa/pr-7587
Apr 15, 2026
Merged

[PR #7587/71c6e2c0 backport][3.108] [PULP-1299] Add setting to allow admins to avoid timeout issues with immediate tasks#7596
pedro-psb merged 1 commit into3.108from
patchback/backports/3.108/71c6e2c0cb5d79db06e1e4cfdc634d9c15246afa/pr-7587

Conversation

@patchback
Copy link
Copy Markdown

@patchback patchback Bot commented Apr 15, 2026

This is a backport of PR #7587 as merged into main (71c6e2c).

We can backport new settings right?

I looked at the JIRA and in this case there really isn't any expensive operation going on for RPM or Container distribution updates. There's two synchronous operations: 1. the database update call, 2. the cache invalidation hook. These should both be fairly fast and happen under a second, so not sure how satellite is hitting the 5 second timeout. The only conclusion I can draw is that their third party systems are really slow.

I really don't think we should update the RPM & Container tasks to be deferred since Services never runs into this problem and it would significantly slow down their system if we did so with the thousands of updates they handle daily. I also didn't think we should allow the immediate timeout to be configurable with a setting since the user would be unaware that it would affect the task worker's heartbeat if increased too large and could end up with more cancelled tasks. Open to suggestions, but I think this setting is the simplest solution that can satisfy everyone's needs.

📜 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

@pedro-psb pedro-psb merged commit 0ae8327 into 3.108 Apr 15, 2026
14 checks passed
@pedro-psb pedro-psb deleted the patchback/backports/3.108/71c6e2c0cb5d79db06e1e4cfdc634d9c15246afa/pr-7587 branch April 15, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants