Bug #14286
closedUsing ${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.
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 |
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
Updated by Tobias Ell almost 6 years ago
- File rudder_run.txt rudder_run.txt added
I ran agent in debug, hope I got all output related to the technique.
Updated by Nicolas CHARLES almost 6 years ago
thank you ! we'll investigate why the generic method does not behave as expected
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
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.
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
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.
Updated by François ARMAND over 5 years ago
- Target version changed from 4.3.10 to 4.3.11
Updated by Vincent MEMBRÉ over 5 years ago
- Target version changed from 4.3.11 to 4.3.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
Updated by Vincent MEMBRÉ over 5 years ago
- Target version changed from 5.0.10 to 5.0.11
Updated by Vincent MEMBRÉ over 5 years ago
- Target version changed from 5.0.11 to 5.0.12
Updated by Vincent MEMBRÉ over 5 years ago
- Target version changed from 5.0.12 to 5.0.13
Updated by Vincent MEMBRÉ about 5 years ago
- Target version changed from 5.0.13 to 5.0.14
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.
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
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
Updated by Nicolas CHARLES about 5 years ago
- Status changed from New to In progress
- Assignee set to Nicolas CHARLES
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
Updated by Nicolas CHARLES about 5 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset commit:1cca0dce4599e0751ad412ea43ff414b1b75120f.
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
Updated by Alexis Mousset almost 5 years ago
- Fix check changed from To do to Checked
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.
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