Project

General

Profile

Actions

Feature #29822

open

Consolidate checks/ and hooks that act as checks as a mechanism that works like hooks

Added by Eric Helms over 4 years ago. Updated about 2 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

Current state is there is a Kafo feature called checks/ to provide "system" checks before the installation runs. There are also hooks/ that provide similar functionality, e.g.

https://github.com/theforeman/foreman-installer/blob/develop/katello/hooks/pre/15-check_java.rb
https://github.com/theforeman/foreman-installer/blob/develop/katello/hooks/pre/15-check-disk.rb
https://github.com/theforeman/foreman-installer/blob/develop/katello/hooks/pre_validations/10-check_foreman_proxy_pulp.rb

The hooks concept provides more access to tools, and related data such as input parameters and the answers file than does the checks/ directory. These two should be re-conciled into hooks.

Possible ideas:

  • create a dedicated checks/ hook phase to make it obvious

Related issues 1 (1 open0 closed)

Related to Kafo - Feature #29459: Ability to skip invididual checksNewActions
Actions #1

Updated by Eric Helms over 4 years ago

  • Blocks Feature #29802: Replace --skip-check-i-know-better with --disable-system-checks from katello/hooks added
Actions #2

Updated by Ewoud Kohl van Wijngaarden over 4 years ago

Actions #3

Updated by Ewoud Kohl van Wijngaarden over 4 years ago

It would also be nice if you could skip some, like CPU or DNS rather than wholesale everything. Linked it to #29459.

Actions #4

Updated by Eric Helms about 2 years ago

  • Parent task deleted (#29746)
Actions #5

Updated by Ewoud Kohl van Wijngaarden about 2 years ago

  • Blocks deleted (Feature #29802: Replace --skip-check-i-know-better with --disable-system-checks from katello/hooks)
Actions

Also available in: Atom PDF