Project

General

Profile

Actions

Bug #14286

closed

Using ${match.x} in generic method causes an error message in the agent output, and prevents multiple reporting based on this generic method

Added by Tobias Ell almost 6 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Category:
Generic methods
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Priority:
43
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

Hello,

my server is at rudder 5.0.6 and my agent as well.
Although this should be fixed in https://issues.rudder.io/issues/13202
I see "error: The value of variable 'default:_log_v3.class_prefix' contains a reference to itself, '${class_prefix}'" in output of
agent run.

I did upgrade-techniques -o, do I have to build my user techniques anew?


Files

rudder_run.txt (338 KB) rudder_run.txt Tobias Ell, 2019-02-08 12:06

Related issues 1 (0 open1 closed)

Related to Rudder - Bug #13202: error message in output of rudder 4.3 - 'default:_log_v3.class_prefix' contains a reference to itself, '${class_prefix}'ReleasedActions
Actions #1

Updated by Nicolas CHARLES almost 6 years ago

Hi - this issue should be gone, but it seems we missed something

Would it be possible to include the context of this errors (lines befores & after in the agent output) so that we can identify which generic method/technique still need to be corrected ?

Thank you

Actions #2

Updated by Tobias Ell almost 6 years ago

Shall I run the agent in debug mode?

Actions #3

Updated by Tobias Ell almost 6 years ago

I ran agent in debug, hope I got all output related to the technique.

Actions #4

Updated by Nicolas CHARLES almost 6 years ago

thank you ! we'll investigate why the generic method does not behave as expected

Actions #5

Updated by Nicolas CHARLES almost 6 years ago

  • Related to Bug #13202: error message in output of rudder 4.3 - 'default:_log_v3.class_prefix' contains a reference to itself, '${class_prefix}' added
Actions #6

Updated by Nicolas CHARLES almost 6 years ago

  • Target version set to 4.3.10
Actions #7

Updated by Nicolas CHARLES almost 6 years ago

Hi Tobias

Thank oyu very much for this report - it allowed us to understand the issue at stake here.
It's because there is a variable ${match.1} that only has meaning in the sense of string replacement using regexp - but we use the parameters of the generic method as unique identifier for reporting as well
since this variable only has meaning in the context of string replacement, during evaluation of the unique identifier, it is empty - and since cfengine 3.9 an empty variable makes the whole promise ignored. As it is ignored, the identifier doesn't exist - thus the error.

Actions #8

Updated by Nicolas CHARLES almost 6 years ago

i tried to workaround the issue - without much success so far.
Please note that is it only cosmetic, and don't have impact on reporting, unless you are editing several time the same file with variables.

We are looking for a more long term solution

Actions #9

Updated by Tobias Ell almost 6 years ago

OK - I just wanted to be sure that the error will not interfere with checking/reporting.
Thanks yor looking into the issue.

Actions #10

Updated by François ARMAND over 5 years ago

  • Target version changed from 4.3.10 to 4.3.11
Actions #11

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.3.11 to 4.3.12
Actions #12

Updated by Benoît PECCATTE over 5 years ago

  • Tracker changed from Question to Bug
  • Category set to Generic methods
  • Target version changed from 4.3.12 to 5.0.10
  • Priority set to 0
Actions #13

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.10 to 5.0.11
Actions #14

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.11 to 5.0.12
Actions #15

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.12 to 5.0.13
Actions #16

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.13 to 5.0.14
Actions #17

Updated by François ARMAND about 5 years ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Getting started - demo | first install | Technique editor and level 1 Techniques
  • Priority changed from 0 to 46

Nicolas: can you update the description / title with the reach of the problem ? So that other people getting there understand what they shouldn't do.

Actions #18

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.14 to 5.0.15
  • Priority changed from 46 to 45
Actions #19

Updated by Nicolas CHARLES about 5 years ago

  • Subject changed from error in agent run default:_log_v3.class_prefix contains reference to itself to Using ${match.x} in generic method causes an error message in the agent output, and prevent multiple reporting based on this generic method
  • Priority changed from 45 to 44
Actions #20

Updated by Nicolas CHARLES about 5 years ago

  • Status changed from New to In progress
  • Assignee set to Nicolas CHARLES
Actions #21

Updated by Nicolas CHARLES about 5 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Alexis Mousset
  • Pull Request set to https://github.com/Normation/ncf/pull/1111
Actions #22

Updated by Nicolas CHARLES about 5 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:1cca0dce4599e0751ad412ea43ff414b1b75120f.

Actions #23

Updated by Alexis Mousset about 5 years ago

  • Subject changed from Using ${match.x} in generic method causes an error message in the agent output, and prevent multiple reporting based on this generic method to Using ${match.x} in generic method causes an error message in the agent output, and prevents multiple reporting based on this generic method
  • Name check set to Reviewed
  • Fix check set to To do
Actions #24

Updated by Alexis Mousset almost 5 years ago

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

Updated by Vincent MEMBRÉ almost 5 years ago

  • Status changed from Pending release to Released

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

Actions #26

Updated by Alexis Mousset over 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Generic methods to Generic methods
  • Priority changed from 44 to 43
Actions

Also available in: Atom PDF