Project

General

Profile

Bug #31508

hammer task resume has broken.

Added by Adam Ruzicka almost 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Foreman plugin
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Found in Releases:
Red Hat JIRA:

Description

Description of problem:
  1. hammer task resume command is not resuming all paused tasks as before.

~~~~~~~~~~~~~~~~~~~~~~~~~~
[root@satellite ~]# hammer task --help
Usage:
hammer task [OPTIONS] SUBCOMMAND [ARG] ...

Parameters:
SUBCOMMAND Subcommand
[ARG] ... Subcommand arguments

Subcommands:
info Show task details
list List tasks
progress Show the progress of the task
resume Resume all tasks paused in error state <<=======
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Currently command is raising the error - "Please provide at least one of search or task_ids parameters in the request".

Version-Release number of selected component (if applicable):
Satellite 6.9

Actual results:
[root@satellite ~]# hammer task resume
400 Bad Request
Please provide at least one of search or task_ids parameters in the request

Expected results:
Try to resume all the paused tasks without the need of providing task_ids as before.

Additional info:

This issue seems to happen due to removal of scope search in below commit.

https://github.com/theforeman/foreman-tasks/pull/516/files#diff-d5548a710d9ec9031997c62d4520fe761970aaa226d123b05af0a9949ab2e4dfL96

Associated revisions

Revision 083f5114 (diff)
Added by Adam Ruzicka almost 2 years ago

Fixes #31508 - Restore changed bulk resume api (#599)

  • Fixes #31508 - Restore changed bulk resume api

We changed an existing API to start require parameters it previously didn't
require. This reverts the API to the previous state.

  • Fixes #31508 - Document bulk_* endpoints behavior

History

#1 Updated by The Foreman Bot almost 2 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-tasks/pull/599 added

#2 Updated by The Foreman Bot almost 2 years ago

  • Fixed in Releases foreman-tasks-3.0.3 added

#3 Updated by Adam Ruzicka almost 2 years ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Adam Ruzicka over 1 year ago

  • Fixed in Releases foreman-tasks-4.0.0 added

Also available in: Atom PDF