karaf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Fabian Lange (JIRA)" <j...@apache.org>
Subject [jira] [Created] (KARAF-4475) Performing status check will wipe cache if karaf.clean.all/karaf.clean.cache is set
Date Wed, 06 Apr 2016 16:06:25 GMT
Fabian Lange created KARAF-4475:
-----------------------------------

             Summary: Performing status check will wipe cache if karaf.clean.all/karaf.clean.cache
is set
                 Key: KARAF-4475
                 URL: https://issues.apache.org/jira/browse/KARAF-4475
             Project: Karaf
          Issue Type: Bug
            Reporter: Fabian Lange


Whenever somebody does "new ConfigProperties()" the cache files are deleted:
https://github.com/apache/karaf/blob/master/main/src/main/java/org/apache/karaf/main/ConfigProperties.java#L181

This is problematic, as with a running karaf instance, somebody who will invoke the status
script, will wipe the cache directory of the running instance, resulting in various problems

Both: Status and Stop are guilty of doing that.
https://github.com/apache/karaf/blob/b8c2617c278bb899c0af2ea1a8d2dc987f22f1cb/main/src/main/java/org/apache/karaf/main/Status.java#L38
https://github.com/apache/karaf/blob/b8c2617c278bb899c0af2ea1a8d2dc987f22f1cb/main/src/main/java/org/apache/karaf/main/Stop.java#L42

I propose to move the deletion logic to Main.launch()
https://github.com/apache/karaf/blob/b8c2617c278bb899c0af2ea1a8d2dc987f22f1cb/main/src/main/java/org/apache/karaf/main/Main.java#L230



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message