Project

General

Profile

Refactor #28250

refactor pulp3 repository service classes

Added by Justin Sherrill 5 months ago. Updated 4 months ago.

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

Description

repository.rb is getting big and crufty, we should refactor it to:

make it easier to use the api without specifying a katello repository
move mirror logic to its own class

Associated revisions

Revision 198a8cd2 (diff)
Added by Justin Sherrill 4 months ago

Fixes #28250 - refactory pulp3 service classes

The goal of this was to fix a couple of issues with the
current pulp3 service classes. They mixed api definitions
with katello<->pulp translations around repositories which
made it difficult to work with the api outside the content of
a repository. To remedy this, the api references and instantiation
is moved to a sepearate set of classes.

The second issue was that logic related to mirror repositories
complicated the Repository class considerably, and made it somewhat
confusing in places. To fix this, the mirror logic is moved to a
RepositoryMirror class.

History

#1 Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/Katello/katello/pull/8432 added

#2 Updated by Jonathon Turel 5 months ago

  • Triaged changed from No to Yes

#3 Updated by The Foreman Bot 4 months ago

  • Status changed from New to Ready For Testing

#4 Updated by The Foreman Bot 4 months ago

  • Fixed in Releases Katello 3.15.0 added

#5 Updated by Justin Sherrill 4 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF