Bug #2611
closedUse configuration.properties.sample for reference configuration
Description
As for Rudder, we want to have only a configuration.properties.sample file in the Inventory-webapp application, not a configuration.properties one, so that commits error are lowered.
Step one: synchronize configuration.properties and configuration.properties.sample.
Step two: update packaging to use configuration.properties.sample as configuration file.
Updated by François ARMAND over 12 years ago
- Status changed from In progress to Discussion
- Assignee changed from François ARMAND to Nicolas PERRON
- Target version changed from 2.4.0~beta1 to 2.4.0~beta2
Step 1: done.
Now, a packager have to use configuration.properties.sample, and then we could delete configuration.properties.
Updated by Jonathan CLARKE over 12 years ago
- Status changed from Discussion to In progress
- Assignee changed from Nicolas PERRON to Jonathan CLARKE
- Target version changed from 2.4.0~beta2 to 2.4.0~beta1
I'm on it
Updated by Jonathan CLARKE over 12 years ago
Hmmm, the ticket didn't get updated for some reason, but I commited this here: commit:2c73de28c
Updated by Jonathan CLARKE over 12 years ago
- Status changed from In progress to Pending technical review
- % Done changed from 0 to 100
OK, that's http://www.rudder-project.org/redmine/projects/packages/repository/revisions/2c73de2837c940dedc1a3c6aa3f4b48aac72f0b2 with a functional link.
Updated by Jonathan CLARKE over 12 years ago
- Status changed from Pending technical review to Discussion
- Assignee changed from Jonathan CLARKE to François ARMAND
François, I'll leave it to you to actually remove the old file - I don't know what consequences that might have on dev environments.
Updated by François ARMAND over 12 years ago
- Status changed from Discussion to In progress
Updated by François ARMAND over 12 years ago
- Status changed from In progress to Pending technical review
Applied in changeset 9e01b39bacd74ec026f235a39c093a227d55376d.
Updated by Nicolas CHARLES over 12 years ago
- Status changed from Pending technical review to Released
This looks valid, thank you Francois
Updated by Benoît PECCATTE almost 10 years ago
- Project changed from 20 to Rudder
- Category set to Web - Nodes & inventories