Project

General

Profile

Actions

Bug #6624

closed

UI throws 500 ISE in production.log while installing a package one after another

Added by Brad Buckingham almost 10 years ago. Updated almost 6 years ago.

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

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1105617
Description of problem:
We need to handle the case when required lock is already taken by a task.
UI should throw proper error message. Like I tried to install a package on remote machine via UI and somehow I double clicked the pakage install button.
And production.log thrown a long exception with 500 internal server error. Hpwever I didn't see any error on UI.

Later I tried to install another package when install for one package is already on-going. And this time too I found same error in production.log and no change on UI. Neither the package install button submitted nor any error raised on UI.

log snippet from production.log:

Processing by Katello::Api::V2::SystemPackagesController#install as JSON
Parameters: {"uuid"=>"50415ba8-6cad-4784-8672-565c9dec6b52", "packages"=>["ksh"], "api_version"=>"v2", "system_id"=>"50415ba8-6cad-4784-8672-565c9dec6b52", "system_package"=>{"uuid"=>"50415ba8-6cad-4784-8672-565c9dec6b52", "packages"=>["ksh"]}}
Required lock is already taken by other running tasks.
Please inspect their state, fix their errors nad resume them.

Required lock: read
Conflicts with tasks:
- https://dhcp207-55.lab.eng.pnq.redhat.com/foreman_tasks/tasks/0ef16a7f-e183-44cc-9e74-ba439a4feeae (ForemanTasks::Lock::LockConflict)

Version-Release number of selected component (if applicable):
sat6 beta snap8 (Satellite-6.0.3-RHEL-6-20140604.0).

How reproducible:
always

Steps to Reproduce:
1. register client with sat6 server
2. install package on remote client from UI
3. re-triger install package
4. see production.log

Actual results:

Expected results:
UI should throw proper error on UI when lock is taken by a task and there shouldn't be ISE in poduction.log

Additional info:

Actions #1

Updated by Brad Buckingham almost 10 years ago

  • Assignee set to Brad Buckingham
  • Target version set to 49
  • Triaged changed from No to Yes
Actions #2

Updated by The Foreman Bot almost 10 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/4430 added
  • Pull request deleted ()
Actions #3

Updated by Brad Buckingham almost 10 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #4

Updated by Eric Helms over 9 years ago

  • translation missing: en.field_release set to 13
Actions

Also available in: Atom PDF