Bug 465737 - virt-manager removes custom options from domain config file
Summary: virt-manager removes custom options from domain config file
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: virt-manager
Version: 5.2
Hardware: All
OS: Linux
low
medium
Target Milestone: rc
: ---
Assignee: Daniel Berrangé
QA Contact: Virtualization Bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-10-06 07:46 UTC by Ales Zelinka
Modified: 2009-12-14 21:16 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-10-06 19:16:28 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Ales Zelinka 2008-10-06 07:46:50 UTC
Description of problem:
When I change domain HW configuration, virt-manager removes all custom (uknown to him) options from the config file

Version-Release number of selected component (if applicable):
virt-manager-0.5.3-8.el5
also tested on virt-manager-0.5.4-4.fc9.i386

How reproducible:
always

Steps to Reproduce:
1. echo 'custom_option = "some value"' >> /etc/xen/domainXYZ
2. change the domain via virt-manager (ie disconnect/connect cdrom image)
3. grep custom_option /etc/xen/domainXYZ
  
Actual results:
custom options are gone

Expected results:
domain config is changed and all custom option are preserved, unchanged and in the same order as before the change

Additional info:

Comment 2 Daniel Berrangé 2008-10-06 19:13:19 UTC
This is a known limitation of the way we manage inactive domains, and is not practical to change. Only parameters that are known to libvirt will be preserved.

Comment 3 Daniel Berrangé 2008-10-06 19:16:28 UTC
Opps, hit submit too soon. Meant to add, that if there are specific config parameters you would like to use, then please file individual BZ tickets for each config parameter, and they can be evaluated for inclusion. Some things are practical to support, some aren't, so we need to consider each config parameter individually as a feature request.

Comment 4 Ales Zelinka 2008-10-07 07:06:09 UTC
OK, I will file those bugs.

Just for the record: I still think this is a bug because:

1) /etc/xen/* are xen config files, not libvirt's or virt-manager's. Upper layer (frontend) should not mess with configs from lower layers. 
2) man xmdomain.cfg neither denys usage of custom options nor it states that only listed ones are valid 
3) xen is perfectly fine with custom options, does not complain when called with custom options nor logs any warnings.

Comment 5 Daniel Berrangé 2008-10-07 07:55:11 UTC
I accept that its a bug - its simply one we cannot fix. The Xen configs files are a horrific mess - they're not actually config files at all, but python scripts. XenD provides no capability for reading or writing these configs, and thus libvirt has to do it itself, and is limited to the config params it understands. This is all dead code, upstream having changed the way XenD manages inactive guests, to get rid of config files. To change the way we deal with this is simply not practical for a stable RHEL-5.x update, hence while this is a bug, it is not one we will fix

Comment 6 Ales Zelinka 2008-10-07 08:06:02 UTC
I see, thanks for the additional explanation.


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