Project

General

Profile

Refactor #29602

Drop repository management parameters from the installer

Added by Ewoud Kohl van Wijngaarden 6 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Category:
Foreman modules
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

Description

Currently we have --foreman-repo, --foreman-gpgcheck, --foreman-configure-epel-repo and --foreman-configure-scl-repo but the installer itself nowadays requires the SCL repo. EPEL is not really used anymore. That only leaves the repo itself and it's better to recommend users to use the foreman-release RPM for that.

There are also similar parameters on the Proxy which is even more confusing because they come from the same repo, but you end up with the same yumrepo/apt files duplicated.

Moving it to a standalone class for Puppet users is a good solution.

Associated revisions

Revision 475afc0a (diff)
Added by Ewoud Kohl van Wijngaarden 6 months ago

Fixes #29602 - Refactor repository handling

This removes the repo parameters from the main class, in favor of a
standalone class that can be included. It uses an anchor because that
can be collected in the main class to keep the correct dependency
chaining while using composition.

Revision 00458202 (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Refs #29602 - Remove dropped repository parameters

Since theforeman-foreman 15.0.0 these parameters no longer exist and by
default no repository setup is done.

History

#1 Updated by The Foreman Bot 6 months ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/puppet-foreman/pull/815 added

#2 Updated by The Foreman Bot 6 months ago

  • Fixed in Releases 2.1.0 added

#3 Updated by Ewoud Kohl van Wijngaarden 6 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot 4 months ago

  • Pull request https://github.com/theforeman/foreman-installer/pull/525 added

Also available in: Atom PDF