Project

General

Profile

Actions

Bug #34680

closed

A failed CV promote during publish or repo sync causes ISE

Added by Justin Sherrill about 2 years ago. Updated about 2 years ago.

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

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

Actions #1

Updated by The Foreman Bot about 2 years ago

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

Updated by The Foreman Bot about 2 years ago

  • Fixed in Releases Katello 4.5.0 added
Actions #3

Updated by Justin Sherrill about 2 years ago

  • Status changed from Ready For Testing to Closed
Actions #4

Updated by Ryan Verdile about 2 years ago

  • 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
  • Target version set to Katello 4.5.0
  • Triaged changed from No to Yes
Actions

Also available in: Atom PDF