Bug 1372803

Summary:	[RFE] Log fusor-initial-setup scriptlets
Product:	Red Hat Quickstart Cloud Installer	Reporter:	Thom Carlin <tcarlin>
Component:	fusor-server	Assignee:	dgao
Status:	CLOSED ERRATA	QA Contact:	James Olin Oden <joden>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	1.0	CC:	dgao, jmontleo, joden, tpapaioa, tsanders
Target Milestone:	---	Keywords:	FutureFeature, Triaged
Target Release:	1.1
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-02-28 01:39:04 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Thom Carlin 2016-09-02 18:17:16 UTC

Description of enhancement:

fusor-initial-setup rpm scripts redirect stdout/stderr to /dev/null.  Please change redirection to a file under /var/log

Version-Release number of selected component (if applicable):

QCI-1.0-RHEL-7-20160901.5

Additional info:

Comment 1 dgao 2016-09-13 20:50:44 UTC

https://gitlab.sat.lab.tlv.redhat.com/rhci/bootiso/merge_requests/16

Comment 2 Tasos Papaioannou 2016-10-05 16:34:31 UTC

The changes have not made it into QCI-1.1-RHEL-7-20161004.t.1:

****

# rpm -q fusor-initial-setup
fusor-initial-setup-1.1.0-1.el7.noarch

# rpm -q fusor-initial-setup --scripts
postinstall scriptlet (using /bin/sh):
if [ $1 -eq 1 ] ; then 
    # Initial installation 
    /bin/systemctl enable fusor-initial-setup-text.service >/dev/null 2>&1 || :
    /bin/systemctl daemon-reload >/dev/null 2>&1 || :
fi
preuninstall scriptlet (using /bin/sh):
if [ $1 -eq 0 ] ; then
    # Package removal, not upgrade
    /bin/systemctl --no-reload disable fusor-initial-setup-text.service > /dev/null 2>&1 || :
    /bin/systemctl stop fusor-initial-setup-text.service > /dev/null 2>&1 || :
fi
postuninstall scriptlet (using /bin/sh):
/bin/systemctl daemon-reload >/dev/null 2>&1 || :
if [ $1 -ge 1 ] ; then
    # Package upgrade, not uninstall
    /bin/systemctl try-restart fusor-initial-setup-text.service >/dev/null 2>&1 || :
fi

****

Comment 4 Jason Montleon 2016-12-05 20:28:31 UTC

https://gitlab.sat.lab.tlv.redhat.com/rhci/bootiso/merge_requests/16 was merged 3 months ago. Looking at the scripts for the installed packages it looks like it's there now; was probably just waiting on a package rebuild.

Comment 5 James Olin Oden 2016-12-07 18:09:54 UTC

Ran:

   rpm -q --scripts fusor-initial-setupos

and saw the pipes to tee /var/log/messages existed.

Verified in QCI-1.1-RHEL-7-20161206.t.0

Comment 8 errata-xmlrpc 2017-02-28 01:39:04 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2017:0335