Bug #34680
A failed CV promote during publish or repo sync causes ISE
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
History
#1
Updated by The Foreman Bot 12 months ago
Updated by - 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
Updated by - Status changed from Ready For Testing to Closed
Applied in changeset katello|f0c69a1bf7799807e2238998f7b54e230aa5fa36.
#4
Updated by Ryan Verdile 12 months ago
Updated by - 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
Fixes #34680 - return http 422 on lock conflict