Architecture #2602
closed
Clean-up Rudder configuration.properties
Description
The main Rudder configuration.properties is full of non documented, out-of-date crap that limits a lot the possibility to make insightful choice about what the user should be able to configure.
=> document each option, clean-up no more used properties, etc.
That issue particulary blocks (or at least make even more difficult) #2132
Updated by François ARMAND over 12 years ago
- Status changed from In progress to Pending technical review
- % Done changed from 0 to 100
Applied in changeset 0693e5e358c77174ea67140649c2ef5ff9979a31.
Updated by Nicolas CHARLES over 12 years ago
- Status changed from Pending technical review to Discussion
- % Done changed from 100 to 80
There's an issue with the variable rudder.dir.nodes-promises
The promises deployed on the nodes assume that the promises lies in the /var/rudder/share, and there's nothing (yet) to configure it
However, even if it were configurable, if it changed, nothing would allow the nodes to know that it did, and fetch the new promises at the right place
Updated by François ARMAND over 12 years ago
Actually, there was a parameter for configuring "/var/rudder" part that is also in no use, and the same thing goes for the Rudder System Variable "BASE_FOLDER".
So the cleaned version of properties must take care of that.
Updated by François ARMAND over 12 years ago
- Status changed from Discussion to Pending technical review
- % Done changed from 80 to 100
Applied in changeset 365feee9ab7dac032ef6132fc668b468928788ec.
Updated by François ARMAND over 12 years ago
- Status changed from Pending technical review to In progress
- % Done changed from 100 to 90
There is actually an error in the documentation: inventory history file is actually used for accepted/refused nodes.
Updated by François ARMAND over 12 years ago
- Status changed from In progress to Pending technical review
- % Done changed from 90 to 100
Applied in changeset 85e52eee978ea51efc701cb0d3d455cd636c63ce.
Updated by Nicolas CHARLES over 12 years ago
- Status changed from Pending technical review to Released
This looks correct, thank you Francois. However there might be something related with #2620, but it's not obvious from the code here
Updated by Jonathan CLARKE over 12 years ago
- Target version changed from 47 to 50
Updated by Jonathan CLARKE over 12 years ago
- Target version changed from 50 to 2.4.0~beta3