karaf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jean-Baptiste Onofré (JIRA) <j...@apache.org>
Subject [jira] [Assigned] (KARAF-5440) No override facility for properties in system.properties
Date Mon, 23 Oct 2017 08:42:00 GMT

     [ https://issues.apache.org/jira/browse/KARAF-5440?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel

Jean-Baptiste Onofré reassigned KARAF-5440:

    Assignee: Jean-Baptiste Onofré

> No override facility for properties in system.properties
> --------------------------------------------------------
>                 Key: KARAF-5440
>                 URL: https://issues.apache.org/jira/browse/KARAF-5440
>             Project: Karaf
>          Issue Type: Bug
>          Components: karaf-core
>            Reporter: Tom Quarendon
>            Assignee: Jean-Baptiste Onofré
>            Priority: Minor
>             Fix For: 4.2.0, 4.1.3
> There are two properties files in karaf that give global properties of various kinds,
system.properties and config.properties.
> The "config.properties" file has a facility to override properties within it by putting
values in the "custom.properties" file. We use this to tweak certain values in out custom
> There's no such facility though for the "system.properties" file.
> At the moment we therefore hold a copy of the whole system.properties file with a couple
of modifications and include that in our distribution. This is fragile though. For example,
recently one of the properties was changes by a commit to karaf and it took me a while to
work out why things suddenly started behaving differently (this isn't a criticism, I'm just
pointing out the fragility, in case it wasn't clear).
> It looks like there wouldn't be any particular reason why this couldn't be done by org.apache.karaf.util.config.PropertiesLoader.
I'm happy to provide a pull request for consideration. I would have suggested having an ${optionals}
line in the system.properties file that would load a file called, say, custom.system.properties.
I would have just replaced the code to load the properties in loadSystemProperties with a
call to loadPropertiesFile.

This message was sent by Atlassian JIRA

View raw message