Bug 1473501 - Apache configuration files installed by CloudForms packages are overwritten on upgrade
Summary: Apache configuration files installed by CloudForms packages are overwritten o...
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Appliance
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: GA
: cfme-future
Assignee: Gregg Tanzillo
QA Contact: luke couzens
URL:
Whiteboard: black:upgrade:migration
Depends On:
Blocks: 1511957
TreeView+ depends on / blocked
 
Reported: 2017-07-21 03:17 UTC by tachoi
Modified: 2020-12-14 09:11 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-09-15 13:24:58 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description tachoi 2017-07-21 03:17:35 UTC
Description of problem:
Have added a "Rewrite" rule to the file (/etc/httpd/conf.d/manageiq-redirects-ui) to fix the redirect upon user log out with SAML2 configured.  
Subsequent upgrades of CloudForms packages overwrites that file and our change is lost.  Investigation has shown the affected configuration file is not listed in *any* package.  Therefore it is apparent that the manifests for the CloudFroms packages are incomplete (defective). 

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

How reproducible:
NA

Steps to Reproduce:
1.modify /etc/httpd/conf.d/manageiq-redirects-ui
2.Errata upgrade "yum -y cfme-appliance"
3.check /etc/httpd/conf.d/manageiq-redirects-ui

Actual results:


Expected results:
The file /etc/httpd/conf.d/manageiq-redirects-ui should be listed as a "configuration" file in the applicable package so when that package is updated a new file is created with the ".rpmnew" suffix so the modified file is not overwritten and lost.

Additional info:
Currently managed by post config tool like Ansible to fix what CloudForms packaging breaks.

Comment 10 Joe Rafaniello 2017-08-29 22:08:11 UTC
I don't see why you couldn't name the file /etc/httpd/conf.d/custom_saml_integration.conf or something that would make it be included before the other file.

The default httpd.conf in centos/rhel will include all .conf files in the conf.d directory and I'm guessing it's in glob pattern order so alphabetically earlier should get your include line earlier.

#
# Load config files from the config directory "/etc/httpd/conf.d".
#
Include conf.d/*.conf


Note, I haven't tested this but this should work as that's how we load all of our configuration files.

It's still unclear why this is needed and if the workaround used here is something we should ship out of box:  http://talk.manageiq.org/t/keycloak-2-5-1-saml-integration/2134/3


Note You need to log in before you can comment on or make changes to this bug.