Project

General

Profile

Bug #28530

Move to a single post install hook for Foreman and Katello

Added by Eric Helms 10 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-

Related issues

Related to Installer - Bug #28687: Katello scenario hook directory incorrectly setClosed
Related to Installer - Bug #28732: --disable-system-checks installer option not available in upstream-nightlyClosed

Associated revisions

Revision 31232364 (diff)
Added by Eric Helms 10 months ago

Fixes #28530: Combine post install hooks into a single one

This moves to a single post install hook to handle all scenarios.
This introduces the message helpers to Foreman hooks and starts
to provide the hook extensions Katello uses.

Revision 5a055e9c (diff)
Added by Ewoud Kohl van Wijngaarden 9 months ago

Refs #28530 - Correctly set the hook_dirs

3123236437240575416d5e698815151913bbc6db combined the hook dirs, but
that broke the actual installation where it stayed as ./katello/hooks
instead of /usr/share/foreman-installer/katello/hooks.

0ed385387d953cb1628a7d2e55014cf815f39d98 fixed one part of this, but
left this out.

Revision a997f764 (diff)
Added by Eric Helms 9 months ago

Refs #28530: Ensure hook dir is set using proper installer dir

History

#1 Updated by The Foreman Bot 10 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-installer/pull/436 added

#2 Updated by The Foreman Bot 10 months ago

  • Fixed in Releases 2.0.0 added

#3 Updated by Eric Helms 10 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Ewoud Kohl van Wijngaarden 9 months ago

  • Related to Bug #28687: Katello scenario hook directory incorrectly set added

#5 Updated by Ewoud Kohl van Wijngaarden 9 months ago

  • Related to Bug #28732: --disable-system-checks installer option not available in upstream-nightly added

#6 Updated by The Foreman Bot 9 months ago

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

#7 Updated by The Foreman Bot 9 months ago

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

Also available in: Atom PDF