Project

General

Profile

Bug #14286

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 over 1 year ago. Updated 6 months ago.

Status:
Released
Priority:
N/A
Category:
Generic methods
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Priority:
44

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

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

Updated by Nicolas CHARLES over 1 year 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

#2

Updated by Tobias Ell over 1 year ago

Shall I run the agent in debug mode?

#3

Updated by Tobias Ell over 1 year ago

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

#4

Updated by Nicolas CHARLES over 1 year ago

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

#5

Updated by Nicolas CHARLES over 1 year 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
#6

Updated by Nicolas CHARLES over 1 year ago

  • Target version set to 4.3.10
#7

Updated by Nicolas CHARLES over 1 year 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.

#8

Updated by Nicolas CHARLES over 1 year 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

#9

Updated by Tobias Ell over 1 year ago

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

#10

Updated by François ARMAND about 1 year ago

  • Target version changed from 4.3.10 to 4.3.11
#11

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 4.3.11 to 4.3.12
#12

Updated by Benoît PECCATTE about 1 year 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
#13

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 5.0.10 to 5.0.11
#14

Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 5.0.11 to 5.0.12
#15

Updated by Vincent MEMBRÉ 11 months ago

  • Target version changed from 5.0.12 to 5.0.13
#16

Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 5.0.13 to 5.0.14
#17

Updated by François ARMAND 9 months 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.

#18

Updated by Vincent MEMBRÉ 8 months ago

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

Updated by Nicolas CHARLES 6 months 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
#20

Updated by Nicolas CHARLES 6 months ago

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

Updated by Nicolas CHARLES 6 months 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
#22

Updated by Nicolas CHARLES 6 months ago

  • Status changed from Pending technical review to Pending release
#23

Updated by Alexis MOUSSET 6 months 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
#25

Updated by Vincent MEMBRÉ 6 months ago

  • Status changed from Pending release to Released

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

Also available in: Atom PDF