Project

General

Profile

Actions

Bug #11600

closed

Result class containing {} are not properly canonized

Added by Janos Mattyasovszky about 7 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Small
Priority:
0
Name check:
Fix check:
Regression:

Description

The following command within a "Command execution result" method:

rpm -q --qf '%{VERSION}-%{RELEASE}' rudder-agent 

Generates resulting classes with following base:

command_execution_result_rpm__q___qf___{VERSION}__{RELEASE}__rudder_agent

=> The {} chars did not get canonized.


Subtasks 3 (0 open3 closed)

Bug #12097: parent patch broke testsReleasedBenoît PECCATTEActions
Bug #12102: Parent patch broke tests (again)ReleasedBenoît PECCATTEActions
Bug #12114: parent patch broke testsReleasedVincent MEMBRÉActions

Related issues 3 (0 open3 closed)

Related to Rudder - Bug #11210: NCF Variable expansion on node properties does not workRejectedActions
Related to Rudder - Bug #10304: Broken variable expression when using node properties in the technique editorReleasedBenoît PECCATTEActions
Is duplicate of Rudder - Bug #8586: Wrong definition of class condition if we use a variable with [ ] in itRejectedActions
Actions #1

Updated by Alexis Mousset about 7 years ago

  • Subject changed from Result class not properly canonized to Result class containing {} are not properly canonized
  • Category set to Technique editor - API
  • Target version set to 0.x
  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Effort required set to Small
  • Priority changed from 0 to 55
Actions #2

Updated by Nicolas CHARLES about 7 years ago

  • Related to Bug #11210: NCF Variable expansion on node properties does not work added
Actions #3

Updated by Nicolas CHARLES about 7 years ago

  • Related to Bug #10304: Broken variable expression when using node properties in the technique editor added
Actions #5

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 0.x to 3.1.25
Actions #6

Updated by Benoît PECCATTE about 7 years ago

  • Priority changed from 55 to 67
Actions #7

Updated by François ARMAND about 7 years ago

  • Assignee set to Vincent MEMBRÉ
Actions #8

Updated by Benoît PECCATTE about 7 years ago

  • Assignee changed from Vincent MEMBRÉ to Benoît PECCATTE
  • Priority changed from 67 to 66
Actions #9

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 3.1.25 to 387
Actions #10

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 387 to 4.1.10
Actions #11

Updated by Benoît PECCATTE almost 7 years ago

  • Status changed from New to In progress
  • Priority changed from 66 to 64
Actions #12

Updated by Benoît PECCATTE almost 7 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Benoît PECCATTE to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/ncf/pull/691
Actions #13

Updated by Benoît PECCATTE almost 7 years ago

  • Is duplicate of Bug #8586: Wrong definition of class condition if we use a variable with [ ] in it added
Actions #14

Updated by Rudder Quality Assistant almost 7 years ago

  • Assignee changed from Vincent MEMBRÉ to Benoît PECCATTE
Actions #15

Updated by Benoît PECCATTE almost 7 years ago

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

Updated by Vincent MEMBRÉ almost 7 years ago

  • Status changed from Pending release to Released
  • Priority changed from 64 to 63

This bug has been fixed in Rudder 4.1.10 and 4.2.4 which were released today.

Actions #17

Updated by Alexis Mousset over 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - API to Web - Technique editor
  • Priority changed from 63 to 0
Actions

Also available in: Atom PDF