Support float64 rates for telemetrygen

[damemi]

Component(s)

cmd/telemetrygen

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

Using float values in --rate returns the following error:

Error: invalid argument "0.1" for "--rate" flag: strconv.ParseInt: parsing "0.1": invalid syntax

The --rate flag for telemetrygen is an Int64Var, however this is ultimately parsed to a rate.Limit value, which is an alias for float64.

I propose changing this flag to a Float64Var to allow non-integer rates. My specific use case is to set this value to something like 0.2, to send a metric once every 5 seconds. Currently, the slowest rate I can set is 1, exporting a metric every second (which is an aggressive rate).

Describe the solution you'd like

Support rates <1/second

Describe alternatives you've considered

We could also use rate.Every() to specify something like this, which might be clearer. But this just parses to a rate.Limit, so I think it is just syntactical sugar

Additional context

No response

[github-actions]

Pinging code owners:

• cmd/telemetrygen: @mx-psi @codeboten

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

[damemi]

Opened #33984 as a fix

[crobert-1]

A code owner has approved the PR to resolve this issue, I'm going to remove the needs triage label.