Bug #35364
closedWhen installing errata via katello-agent, content_action_finish_timeout is ignored and tasks don't wait for client status to finish
Description
Description of problem:
When installing errata via katello-agent, task ends with success for any task that is fetched by gofer and don't end within content_action_accept_timeout.
So, for example a task to install errata on a host will start and if it doesn't fail in 20 seconds (default value for content_action_accept_timeout), the task will end with success. If the task fails on the client side after 20 seconds, satellite would not know.
Version-Release number of selected component (if applicable):
Satellite 6.10.7
Probably also affects 6.11, but didn't test it.
Steps to Reproduce:
1. Setup a Satellite 6.10 having katello-agent capabilities
2. Start an errata installation task which will take more than 20 seconds to finish (or modify the content_action_accept_timeout to a lower value).
3. Observe that in 20 seconds, while the task is still running on the client, on Satellite it will be already finished with success.
Extra test: start a task that will take more than 20 seconds to FAIL. In 20 seconds, the task on satellite will be marked as success independently of what really happened to it on the client.
Extra info: if the task fails, on the host, in less than 20 seconds then Satellite will get the correct status.
Actual results:
Tasks end with success in 20 seconds (only condition for success is the host to get the task from its queue) independently of the actual result on the host.
Expected results:
Task should wait content_action_finish_timeout for the host to send the actual output of the task.
Additional info:
Updated by The Foreman Bot over 2 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/10233 added
Updated by Chris Roberts over 2 years ago
- Category set to Tooling
- Assignee set to Joniel Pasqualetto
- Target version set to Katello 4.6.0
- Triaged changed from No to Yes
Updated by Jeremy Lenz about 2 years ago
- Target version changed from Katello 4.6.0 to Katello 4.7.0
- Triaged changed from No to Yes
Updated by The Foreman Bot about 2 years ago
- Pull request https://github.com/Katello/katello/pull/10278 added
Updated by Samir Jha about 2 years ago
- Pull request deleted (
https://github.com/Katello/katello/pull/10233)
Updated by The Foreman Bot about 2 years ago
- Fixed in Releases Katello 4.7.0 added
Updated by Samir Jha about 2 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset katello|fe90a5c6bd6ad5f2bcc4101f406c23ec41616e88.