[pkg/ottl] Add ottlfuncs to support time.Second and time.Nanosecond

[r0mdau]

Component(s)

pkg/ottl

Is your feature request related to a problem? Please describe.

Be able to add the current time generated by function Now() as a string in an attribute in a log context.

Example configuration:

  transform:
    error_mode: ignore
    log_statements:
      - context: log
        statements:
          - set(attributes["otelTime"], Format("%04d-%02d-%02dT%02d:%02d:%02d.%09d", [Year(Now()), Month(Now()), Day(Now()), Hour(Now()), Minute(Now()), Second(Now()), Nanosecond(Now())]))

Describe the solution you'd like

Create 2 new OTTL functions:

• Second related to https://pkg.go.dev/time#Time.Second
• Nanosecond related to https://pkg.go.dev/time#Time.Nanosecond

Describe alternatives you've considered

As an alternative, I am using the following configuration

          - set(attributes["eventForwarderTimestampUnixSeconds"], UnixSeconds(Now()))

But it does not help to have a custom string format like needed by some systems I work with.

Additional context

If this change is accepted, I would like to contribute.

[github-actions]

Pinging code owners:

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

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

[edmocosta]

Be able to add the current time generated by function Now() as a string in an attribute in a log context.

Hi @r0mdau, would #36870 solve this issue? It seems the goal is the same, supporting time formatting similar to https://pkg.go.dev/time#Time.Format.

[r0mdau]

Be able to add the current time generated by function Now() as a string in an attribute in a log context.

Hi @r0mdau, would #36870 solve this issue? It seems the goal is the same, supporting time formatting similar to https://pkg.go.dev/time#Time.Format.

Hi @edmocosta yes that would do the trick for my use case. In the other hand, why not implementing those 2 missing converters ?

ps: I have them working in an internal fork Second converter Nanosecond converter

[edmocosta]

I agree with you, I think those converters might be useful. Let's wait for the code owners thoughts.

[TylerHelmuth]

We definitely need to do #36870, but since we already have Minute, Day, Month, and Year we can also add Second and Nanosecond.

[r0mdau]

I am closing the issue as the 2 changes are merged, thanks.