Bug 474996

Summary: extremely slow start with cluster.conf
Product: [Fedora] Fedora Reporter: ritz <rkhadgar>
Component: system-config-lvmAssignee: Marek Grac <mgrac>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 10CC: agk, dwysocha, mbroz, mclasen, prockai
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 1.1.9-1.fc10 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-08-27 02:13:25 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: 516879    
Bug Blocks:    
Attachments:
Description Flags
profile information
none
Proposed patch none

Description ritz 2008-12-06 14:22:15 UTC
Created attachment 325999 [details]
profile information

Description of problem:
system-config-lvm extremely slow

Version-Release number of selected component (if applicable):
system-config-lvm-1.1.4-3.1.fc10.noarch

How reproducible:
always

Steps to Reproduce:
1. run system-config-lvm, takes about 20mins to start up
  
Actual results:
slooooow start-up

Expected results:
fast start-up

Additional info:
s-c-l profile. seems to be an issue with cluster.py, we need most likely need to use cache 

    def __get_info(self):
        try:
            c_conf = minidom.parseString(file('/etc/cluster/cluster.conf').read(10000000)).firstChild
<----- bad mojo
            name = c_conf.getAttribute('name')
            lock = 'dlm'
            nodes_num = 0
            for node in c_conf.childNodes:
                if node.nodeType == xml.dom.Node.ELEMENT_NODE:
                    if node.nodeName == 'cman':
                        lock = 'dlm'
                    elif node.nodeName == 'gulm':
                        lock = 'gulm'
                    elif node.nodeName == 'clusternodes':
                        nodes = node
                        for node in nodes.childNodes:
                            if node.nodeType == xml.dom.Node.ELEMENT_NODE:
                                if node.nodeName == 'clusternode':
                                    nodes_num += 1
            if nodes_num != 0:

Comment 1 Marek Grac 2009-08-12 16:58:37 UTC
Created attachment 357205 [details]
Proposed patch

Patch that adds caching to cluster.conf parsing

Comment 2 Fedora Update System 2009-08-20 13:03:59 UTC
system-config-lvm-1.1.8-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/system-config-lvm-1.1.8-1.fc10

Comment 3 Fedora Update System 2009-08-20 23:00:59 UTC
system-config-lvm-1.1.9-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/system-config-lvm-1.1.9-1.fc10

Comment 4 Fedora Update System 2009-08-22 01:02:15 UTC
system-config-lvm-1.1.9-1.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update system-config-lvm'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-8861

Comment 5 Fedora Update System 2009-08-27 02:13:03 UTC
system-config-lvm-1.1.9-1.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.