Project

General

Profile

Bug #34680

A failed CV promote during publish or repo sync causes ISE

Added by Justin Sherrill 12 months ago. Updated 12 months ago.

Status:
Closed
Priority:
Normal
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Description of problem:

While running a test, setting up tools repo failed because setup procedure did not wait for CV to be published before attempting to promote it:

ForemanTasks::Lock::LockConflict: Required lock is already taken by other running tasks.
Please inspect their state, fix their errors and resume them.

Conflicts with tasks:
- https://<test_host>.redhat.com/foreman_tasks/tasks/b75e5863-55f3-49b7-9c88-d27d459aef0e

this is understandable, its a side effect of recent changes in task info [1] , setup procedure no longer has a task ID to poll for completion,

but then this error was also seen:

500 Server Error: Internal Server Error for url <redacted>/katello/api/v2/repositories/177/sync

How reproducible:
Test fails every time but not sure about the ISE, I run again on new install to get cleaner logs if required.

Steps to Reproduce:
1. Run a test that use API to set up tools repo, but does not wait for Foreman Task completion between steps.

Actual results:
ISE

Expected results:
Should return a 4XX result

Associated revisions

Revision f0c69a1b (diff)
Added by Justin Sherrill 12 months ago

Fixes #34680 - return http 422 on lock conflict

History

#1 Updated by The Foreman Bot 12 months ago

  • Assignee set to Justin Sherrill
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/10034 added

#2 Updated by The Foreman Bot 12 months ago

  • Fixed in Releases Katello 4.5.0 added

#3 Updated by Justin Sherrill 12 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Ryan Verdile 12 months ago

  • Triaged changed from No to Yes
  • Target version set to Katello 4.5.0
  • Subject changed from A failed CV promote during publish or repo sync causes ISE to A failed CV promote during publish or repo sync causes ISE

Also available in: Atom PDF