Project

General

Custom queries

Profile

Actions
Copy link

User story #7815

closed

Rename /opt/rudder/etc/{force_technique_reload,force_inventory} to use dashes instead of underscores

Added by Jonathan CLARKE about 9 years ago. Updated about 3 years ago.

Status:
Rejected
Priority:
N/A
Assignee:
-
Category:
System integration
Target version:
Ideas (not version specific)
Pull Request:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

Our convention (and general UNIX convention) is clear that file names should use dashes (-) rather than underscores (_) where possible. See http://www.rudder-project.org/foswiki/Development/GeneralGuidelines.

However, two of our files used on a regular basis use underscores for no apparent reason:

  • /opt/rudder/etc/force_technique_reload - force the webapp to reload techniques (checked periodically), should be renamed to /opt/rudder/etc/force-technique-reload
  • /opt/rudder/etc/force_inventory - forces an inventory on the next agent run, should be renamed to /opt/rudder/etc/force-inventory

Obviously, for script and user backwards compatibility, we should support both for a while, and warn about using the deprecated version.

Actions
Copy link
 #21

Updated by Alexis Mousset about 3 years ago

  • Status changed from New to Rejected

Too risky for the consistency benefits (we tried to do it for policy_generation flag and ended up reverting it due to problems it introduced).

Actions
Copy link

Also available in: Atom PDF