Project

General

Profile

Actions

Bug #24536

open

Rudderc compiles unecessary complex 'class_prefix' at the technique level

Added by Félix DALLIDET 9 months ago. Updated 1 day ago.

Status:
Discussion
Priority:
N/A
Category:
rudderc
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
No

Description

Those are used to define unique prefixes for the classes_noop bundle calls, used when a condition is applied on a method or block and is not verified at runtime, resulting in a report_na report.

As a technique may be use "multiple times", each reporting must be re-evaluated whenever the technique is called, without being influenced by the previous technique calls.
That is why the current implementation uses a class_prefix at the technique level, defined by the concatenation of the directives parameters, this way, a conflict will only occur if two directives are strictly identical.
Still, it over-complexify the cfengine code, leading to the following generated code:

bundle agent call_ntp_technique_d86ce2e5_d5b6_45cc_87e8_c11cca71d907(c_name, c_key, report_id, args, class_prefix, name, version, architecture, provider) {

  methods:
    "d86ce2e5-d5b6-45cc-87e8-c11cca71d907_${report_data.directive_id}" usebundle => _method_reporting_context_v4("${c_name}", "${c_key}", "${report_id}");
    "d86ce2e5-d5b6-45cc-87e8-c11cca71d907_${report_data.directive_id}" usebundle => _classes_noop(canonify("${class_prefix}_package_present_${c_key}"));
    "d86ce2e5-d5b6-45cc-87e8-c11cca71d907_${report_data.directive_id}" usebundle => log_rudder("Skipping method 'Package present' with key parameter 'htop' since condition 'false' is not reached", "htop", canonify("${class_prefix}_package_present_${c_key}"), canonify("${class_prefix}_package_present_${c_key}"), @{args});

}

2 bigs issues here:
  • The resulting conditions of the skipped method are incorrect if wanted, they are unguessable before runtime as they are based on runtime computation of the directive parameters.
    • Note that the documentation never introduce resulting conditions for "skipped" methods, so there should not be any existing user technique using them.
    • The exact defined conditions are: truncate(<concat(techniques_params)>, 1000) + "_" + method_prefix + "_" + component_value + "_noop"
  • Each bundle must take a class_prefix as input which is pretty much useless as its only purpose it to be "a unique internal id per technique call" and is not exposed to the user.
    • As a side effect, the technique will be bugged when one or more of its parameters are empty or undefined (can happen if run manually or with rudderc testing, in a classic run, the webapp is responsible to validate that everything is correct beforehand).
At least two solutions can quite easily be implemented:
  • Dropping the result conditions when a method is skipped: easiest one, as the resulting conditions are not documented, we could just force a result_na report and not define any resulting condition
  • Document and define a resulting condition: for instance method_prefix + "_" + component_value + "_na", and base the generate a force a result_na report.

In the two cases we should be able to work without the need of any unique identified per technique call as the reporting can be hard-coded.
Still, the choice to support or drop the NA resulting conditions must be evaluated as going back afterward will be hard.

Actions #1

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 8.0.7 to 8.0.8
Actions #2

Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 8.0.8 to 8.0.9
Actions #3

Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 8.0.9 to 8.0.10
Actions #4

Updated by Vincent MEMBRÉ 5 months ago

  • Target version changed from 8.0.10 to 1089
Actions #5

Updated by Alexis Mousset 5 months ago

  • Target version changed from 1089 to 8.1.6
Actions #6

Updated by Vincent MEMBRÉ 4 months ago

  • Target version changed from 8.1.6 to 8.1.7
Actions #7

Updated by Vincent MEMBRÉ 2 months ago

  • Target version changed from 8.1.7 to 8.1.8
Actions #8

Updated by Vincent MEMBRÉ 25 days ago

  • Target version changed from 8.1.8 to 8.1.9
Actions #9

Updated by Vincent MEMBRÉ 5 days ago

  • Target version changed from 8.1.9 to 8.1.10
Actions #10

Updated by Vincent MEMBRÉ 1 day ago

  • Target version changed from 8.1.10 to 8.1.11
Actions

Also available in: Atom PDF