Bug 576617

Summary:	[apache] ServerName not persisted with Virtual host update
Product:	[Other] RHQ Project	Reporter:	Lukas Krejci <lkrejci>
Component:	Plugins	Assignee:	Filip Drabek <fdrabek>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Corey Welton <cwelton>
Severity:	medium	Docs Contact:
Priority:	high
Version:	3.0.0	CC:	fdrabek
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	2.4	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2010-08-12 16:56:49 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	577313

Description Lukas Krejci 2010-03-24 16:18:46 UTC

Description of problem:

see bug summary

How reproducible:

always

Steps to Reproduce:
1. Make any update throught the UI to some virtual host with ServerName defined in httpd.conf.
2. Save the changes in the UI.
3. Look at the updated httpd.conf. The ServerName directive is missing.
  
Actual results:

Virtual host gets corrupted.

Expected results:

Update should retain the unchanged properties.

Comment 2 Charles Crouch 2010-05-18 14:03:41 UTC

ServerName should be read-only

Comment 3 Filip Drabek 2010-05-18 15:33:20 UTC

commit 09013bf47ff002214dad4f9e21130acacf526234

Comment 4 Corey Welton 2010-05-20 18:15:07 UTC

I was able to edit this and the change was made successfully.  However, per comment #2, should users even be able to populate this field?

Comment 5 Corey Welton 2010-05-20 18:34:39 UTC

I just checked this on a build from a few hours ago.  User is able to change servername.  Unless this was never checked into master, I think we fixed the first part, but per the comment above, I think we shouldn't be able to change it at all.

Will return to dev for now; once we have a resolution one way or another, I can close it out.

Comment 7 Corey Welton 2010-06-04 13:03:40 UTC

QA Verified.

Comment 8 Corey Welton 2010-08-12 16:56:49 UTC

Mass-closure of verified bugs against JON.