Project

General

Profile

Actions

Bug #34518

closed

on content import failure for a repository the created version should be cleaned up

Added by Lucy Fu over 2 years ago. Updated over 2 years ago.

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

Description

hammer content-import repository failed and left behind a product and repository with no content.

Actions #1

Updated by Partha Aji over 2 years ago

During content-import the importing server creates a content-view-version and then tries to import the content into pulp. If the pulp import fails the task errors out. The user then has to remove the content view version and try importing again. Generated content views like repo import based content views will not be visible to the user. So it is not possible for the user to delete the version even if they want to.
On import failure we should instead be just deleting the useless content view version automatically.

Steps to Reproduce:
1. Start a content-view version import task
2. make the import task fail unexpectedly (kill the worker importing it, for example)
3. Notice that the task errors out with a warning
4. Go to Content => Content Views - Content View details of the importing content view
Actual:
5. Notice that a new content view version got created even though it has no content
6. Try importing again and you notice an error message along the lines of
"""
Content View Version specified in the metadata - '<my content view> 2.0' already exists. If you wish to replace the existing version, delete '<my content view> 2.0' and try import again.
"""

Expected:
No version created. Since the import did not succeed the generate version should automatically get deleted.

Actions #2

Updated by Partha Aji over 2 years ago

  • Bugzilla link set to 2058905
Actions #3

Updated by The Foreman Bot over 2 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Partha Aji
  • Pull request https://github.com/Katello/katello/pull/9984 added
Actions #4

Updated by Samir Jha over 2 years ago

  • Target version set to Katello 4.5.0
  • Triaged changed from No to Yes
Actions #5

Updated by The Foreman Bot over 2 years ago

  • Fixed in Releases Katello 4.5.0 added
Actions #6

Updated by Partha Aji over 2 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF