OTTL context inference is incorrect when conditions used with statement

[hisarbalik]

Component(s)

pkg/ottl

What happened?

Description

The OTTL context inference does not work when the statement is combined with conditions.

Steps to Reproduce

Run the following example in https://ottl.run :

metric_statements:
- statements:
  - set(resource.attributes["foo"], "bar")
  conditions:
  - IsMatch(metric.name, "^bar.*")
  - IsMatch(metric.name, "^foo.*")

The example above works when I add a where clause to the statement as a workaround

metric_statements:
- statements:
  - set(resource.attributes["foo"], "bar") where metric.name != nil
  conditions:
  - IsMatch(metric.name, "^bar.*")
  - IsMatch(metric.name, "^foo.*")

Expected Result

Should be no-op for the given example

Actual Result

unable to run metrics statements. Error: invalid config for "transform" processor unable to parse OTTL condition "IsMatch(name, \"^bar.*\")": error while parsing arguments for call to "IsMatch": invalid argument at position 0: missing context name for path "name", possibly valid options are: "resource.name" unable to parse OTTL condition "IsMatch(name, \"^foo.*\")": error while parsing arguments for call to "IsMatch": invalid argument at position 0: missing context name for path "name", possibly valid options are: "resource.name"

Collector version

v0.124.0

Environment information

No response

OpenTelemetry Collector configuration

Log output

Additional context

No response

[github-actions]

Pinging code owners:

• pkg/ottl: @TylerHelmuth @kentquirk @bogdandrutu @evan-bradley @edmocosta

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[edmocosta]

Thanks for reporting @hisarbalik, that is indeed a bug we need to fix. Context-inferred advanced configurations are not taking into consideration the global conditions, and it might infer the wrong context depending on the statements & conditions used.
For now, as an alternative, you can explicitly set the context value as following:

metric_statements:
- context: metric
  statements:
  - set(resource.attributes["foo"], "bar")
  conditions:
  - IsMatch(metric.name, "^bar.*")
  - IsMatch(metric.name, "^foo.*")

[odubajDT]

I can take a look at this

[edmocosta]

I can take a look at this

Hi @odubajDT, I'm working on this issue already. Thank you very much for volunteering :)