Project

General

Custom queries

Profile

Actions
Copy link

Tracker #29746

closed

Merge hooks/ and katello/hooks into a single directory structure

Added by Eric Helms almost 5 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
foreman-installer script
Target version:
Foreman - 3.4.0
% Done:

100%

Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Foreman - 3.4.0
Found in Releases:
Red Hat JIRA:

Description

As part of this effort, hooks that have a similar naming scheme or functionality should be reduced to a single parameter.

The current state is having two sets of hooks directories: hooks/ and katello/hooks/. There are some hooks that have similar functionality and some that are only relevant if Katellos is enabled. The proposal here is to merge the hooks directories and combine some hooks into one. See [Hooks Cleanup RFC](https://community.theforeman.org/t/rfc-cleaning-up-installer-hooks/17089) for more details.

Subtasks 10 (0 open — 10 closed)

Feature #29797: Merge --reset-foreman-db and --reset into --reset-dataClosedWilliam ClarkActions
Refactor #29798: Move --lock-package-versions to hooks/ClosedWilliam ClarkActions
Refactor #29799: Add --clear-pulp-content to hooks/ and scope on katello module being enabledClosedWilliam ClarkActions
Feature #29800: Remove --clear-puppet-environments due to Pulp 3ClosedWilliam ClarkActions
Refactor #29801: Move --tuning option to hooks/ClosedWilliam ClarkActions
Refactor #29803: Add all certs related hooks to hooks/ and scope them on Katello module being enabledClosedWilliam ClarkActions
Feature #29804: Drop upgrade-mongo-storage-engine from hooksDuplicateActions
Feature #29805: Drop --disable-resolve-mismatchesRejectedActions
Feature #30006: Remove MongoDB Storage Engine Migration HookClosedWilliam ClarkActions
Refactor #35208: Move puppet certs exist hook to hooks/ClosedEric HelmsActions

Related issues 4 (2 open — 2 closed)

Related to Installer - Feature #29820: Add ability to have a "hook" that is dedicated to defining new parameters to the installerNewActions
Related to Installer - Feature #35138: Drop umask check from Katello hooksClosedEric HelmsActions
Related to Installer - Refactor #27371: Move Installer hook helpers to Kafo where they make senseNewActions
Blocked by Installer - Bug #29296: Drop --upgrade flag from katello hooksClosedWilliam ClarkActions
Actions
Copy link
 #8

Updated by Ewoud Kohl van Wijngaarden over 2 years ago

  • Category set to foreman-installer script
  • Target version set to 3.4.0
  • Triaged changed from No to Yes

This is now resolved. Not sure why I can't close it.

Actions
Copy link

Also available in: Atom PDF