Bug 1184895

Summary: [AAA] [extmgr] escape extension name in ENGINE_EXTENSION_ENABLED_ configuration
Product: [Retired] oVirt Reporter: Alon Bar-Lev <alonbl>
Component: ovirt-engine-coreAssignee: Alon Bar-Lev <alonbl>
Status: CLOSED CURRENTRELEASE QA Contact: Ondra Machacek <omachace>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.5CC: bugs, ecohen, gklein, iheim, lsurette, rbalakri, yeylon
Target Milestone: ---Keywords: CodeChange
Target Release: 3.5.2   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: infra
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-04-29 06:18:05 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1076964, 1193058, 1196662, 1197441    

Description Alon Bar-Lev 2015-01-22 12:24:53 UTC 
requires to keep configuration file shell compatible.
Comment 1 Eyal Edri 2015-02-26 12:32:37 UTC 
this ovirt bug was fixed during 3.5.1 cycle and is included in the build, and therefore should be verified.
Comment 2 Ondra Machacek 2015-03-18 11:47:41 UTC 
Moving to assigned, as it can't work as designed.
Probably solution would be: ENGINE_EXTENSION_DISABLED = ext1:ext2
Comment 3 Alon Bar-Lev 2015-03-18 11:57:56 UTC 
(In reply to Ondra Machacek from comment #2)
> Moving to assigned, as it can't work as designed.
> Probably solution would be: ENGINE_EXTENSION_DISABLED = ext1:ext2

no, it should be:

ENGINE_EXTENSION_ENABLED_ext1 = false
ENGINE_EXTENSION_ENABLED_ext2 = false
Comment 4 Ondra Machacek 2015-03-18 12:03:26 UTC 
Yes, but what if it is: ENGINE_EXTENSION_ENABLED_redhat.com = false ?
Comment 5 Alon Bar-Lev 2015-03-18 12:06:13 UTC 
(In reply to Ondra Machacek from comment #4)
> Yes, but what if it is: ENGINE_EXTENSION_ENABLED_redhat.com = false ?

ENGINE_EXTENSION_ENABLED_redhat_com = false

all non alphanum or _ are replaced with _
Comment 6 Ondra Machacek 2015-03-18 12:15:51 UTC 
Oh, I see now.
Comment 7 Eyal Edri 2015-04-29 06:18:05 UTC 
ovirt 3.5.2 was GA'd. closing current release.