Project

General

Profile

Actions

Bug #15989

closed

Nova license path in log is misleading for plugin licenses

Added by François ARMAND over 4 years ago. Updated over 4 years ago.

Status:
Released
Priority:
N/A
Category:
System integration
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Rudder settings | Plugins
Effort required:
Very Small
Priority:
61
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

In the webapp, we have that log at boot:

rudder.dir.licensesFolder="/opt/rudder/etc/licenses" 

But we don't display the license path for plugin. This is extremely misleading. We need to:

- change "rudder.dir.licenseFolder" to something explicit like "rudder.deprecated.cfengine.licenses=/opt/rudder/etc/cfengine-entreprise/licenses" (or not even log it actually, perhaps simpler)
- add the path for plugin licenses so that people can find it.

Actions #1

Updated by François ARMAND over 4 years ago

  • Status changed from New to In progress
Actions #2

Updated by François ARMAND over 4 years ago

I propose to:

- hard-code nova license path to /opt/rudder/etc/cfengine-enterprise-licenses in the code (it is removed in 6.0, so it's just to avoid breaking things in 5.0);
- remove rudder.dir.licensesFolder from properties (and had a special hide to not have it in log even if properties file is migrated from a previous version of rudder)
- add a log info for plugin/license path (since it's not configurable for now, because plugins need to know at compile time where to look)

Actions #3

Updated by François ARMAND over 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/2633
Actions #4

Updated by François ARMAND over 4 years ago

  • Status changed from Pending technical review to Pending release
Actions #5

Updated by Alexis Mousset over 4 years ago

  • Name check changed from To do to Reviewed
Actions #6

Updated by François ARMAND over 4 years ago

  • Fix check changed from To do to Checked
Actions #7

Updated by Vincent MEMBRÉ over 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 5.0.15 which was released today.

Actions

Also available in: Atom PDF