Bug 1184895 - [AAA] [extmgr] escape extension name in ENGINE_EXTENSION_ENABLED_ configuration
Summary: [AAA] [extmgr] escape extension name in ENGINE_EXTENSION_ENABLED_ configuration
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-core
Version: 3.5
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 3.5.2
Assignee: Alon Bar-Lev
QA Contact: Ondra Machacek
URL:
Whiteboard: infra
Depends On:
Blocks: oVirt-AAA-rewrite 1193058 1196662 1197441
TreeView+ depends on / blocked
 
Reported: 2015-01-22 12:24 UTC by Alon Bar-Lev
Modified: 2016-02-10 19:34 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-04-29 06:18:05 UTC
oVirt Team: Infra
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 37221 0 master MERGED aaa: normalize extension name within ENGINE_EXTENSION_ENABLED_ Never
oVirt gerrit 37238 0 ovirt-engine-3.5 MERGED aaa: normalize extension name within ENGINE_EXTENSION_ENABLED_ Never

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.
Note You need to log in before you can comment on or make changes to this bug.