Project

General

Profile

Actions

Bug #4748

closed

errors on import manifest should complete dynflow task and display information to user

Added by Thomas McKay about 10 years ago. Updated over 5 years ago.

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

Description

% hammer -v subscription upload --organization-id megacorp --file test/data/megacorp.zip
....this will work first time....
% hammer -v subscription upload --organization-id megacorp --file test/data/megacorp.zip
ERROR 2014-03-20 11:15:13 Exception ForemanTasks::Lock::LockConflict
Manifest upload failed:
ForemanTasks::Lock::LockConflict

In this case, an error importing the manifest happened ("manifest already imported") and this should be a non-error information message to user.

Other cases (such as trying to import that manifest into an org other than megacorp) will generate real error message which should be displayed to user and dynflow task completed anyway (with rollback if necessary).

Actions #1

Updated by Mike McCune about 10 years ago

  • Triaged set to Yes
Actions #2

Updated by Mike McCune about 10 years ago

  • Triaged deleted (Yes)
Actions #3

Updated by Mike McCune about 10 years ago

  • Assignee set to Ivan Necas
  • Triaged set to Yes
Actions #4

Updated by Mike McCune almost 10 years ago

we should just have the dynflow step get skipped and the task end up in stopped:error state vs paused. Paused causes you to not be able to resolve the issue without having to go to the dynflow console.

Actions #5

Updated by Mike McCune almost 10 years ago

WORKAROUND:

  1. navigate to: https://yourhost/foreman_tasks/dynflow
  2. find the task that is paused with Error
  3. click on the details for it. Find the step that is errored out, click the 'skip' link
  4. Resume the task
  5. Repeat skipping substeps if there are further errors

Once you have the task in the Finished state you can import a manifest

Actions #6

Updated by Justin Sherrill almost 10 years ago

  • translation missing: en.field_release set to 13
Actions #7

Updated by Bryan Kearney almost 10 years ago

  • Bugzilla link set to https://bugzilla.redhat.com/show_bug.cgi?id=1093124
Actions #8

Updated by Ivan Necas almost 10 years ago

  • Status changed from New to Assigned
  • Target version set to 45
Actions #9

Updated by Ivan Necas almost 10 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

Applied in changeset katello|commit:6cb65c11e6743f3aaadabc462f9f33e9d49c8445.

Actions

Also available in: Atom PDF