Project

General

Profile

Actions

Bug #35364

closed

When installing errata via katello-agent, content_action_finish_timeout is ignored and tasks don't wait for client status to finish

Added by Joniel Pasqualetto over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Tooling
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

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:

Actions #1

Updated by The Foreman Bot over 1 year ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/10233 added
Actions #2

Updated by Chris Roberts over 1 year ago

  • Category set to Tooling
  • Assignee set to Joniel Pasqualetto
  • Target version set to Katello 4.6.0
  • Triaged changed from No to Yes
Actions #3

Updated by Samir Jha over 1 year ago

  • Triaged changed from Yes to No
Actions #4

Updated by Jeremy Lenz over 1 year ago

  • Target version changed from Katello 4.6.0 to Katello 4.7.0
  • Triaged changed from No to Yes
Actions #5

Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/katello/pull/10278 added
Actions #6

Updated by Samir Jha over 1 year ago

  • Pull request deleted (https://github.com/Katello/katello/pull/10233)
Actions #7

Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases Katello 4.7.0 added
Actions #8

Updated by Samir Jha over 1 year ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF