Project

General

Profile

Actions
Copy link

Architecture #11940

closed

Architecture #10625: Don't merge directive from same technique on generation

Warn if an "uniqueVariable" is found in a technique

Added by François ARMAND over 6 years ago. Updated about 6 years ago.

Status:
Released
Priority:
N/A
Assignee:
François ARMAND
Category:
Web - Config management
Target version:
4.3.0~beta1
Pull Request:
https://github.com/Normation/rudder/p...
Effort required:
Name check:
Fix check:
Regression:

Description

Since unique variable are not supported anymore, we must warn the user if one is used in a technique.

Related issues 1 (0 open — 1 closed)

Related to Rudder - Architecture #11858: Add common pre- and post- agent run action triggered by technique logicReleasedVincent MEMBRÉActions
Actions
Copy link
 #1

Updated by François ARMAND over 6 years ago

Unique variable can be replace with the use of a RudderParameter or node property (unicity of the value on Rudder scope or on node scope), or if the action must be done exactly once, by the bundle sequence pre-/post-hook (see: #11858)

Actions
Copy link
 #2

Updated by François ARMAND over 6 years ago

  • Related to Architecture #11858: Add common pre- and post- agent run action triggered by technique logic added
Actions
Copy link
 #3

Updated by François ARMAND over 6 years ago

  • Subject changed from Warn if an unique variable is found in a technique to Warn if an "uniqueVariable" or "checked" is found in a technique

In that PR, we will also clean all the code still referencing these information (appart for the warning, of course)

Actions
Copy link
 #4

Updated by François ARMAND over 6 years ago

  • Status changed from New to In progress
Actions
Copy link
 #5

Updated by François ARMAND over 6 years ago

  • Subject changed from Warn if an "uniqueVariable" or "checked" is found in a technique to Warn if an "uniqueVariable" is found in a technique

"checked" is actually use. Not sure how/for what, but I will just leave it alone for now.

Actions
Copy link
 #6

Updated by François ARMAND over 6 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/1826
Actions
Copy link
 #7

Updated by Rudder Quality Assistant over 6 years ago

  • Assignee changed from Nicolas CHARLES to François ARMAND
Actions
Copy link
 #8

Updated by François ARMAND over 6 years ago

  • Status changed from Pending technical review to Pending release
Actions
Copy link
 #9

Updated by Vincent MEMBRÉ about 6 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.3.0~beta1 which was released today.

Actions
Copy link

Also available in: Atom PDF