ERROR: 422 on query with binary operation and keep_metric_names about victoriametrics

[zekker6]

Hello @tekert

This is caused by implicit query conversion:

If something other than series selector is passed to rollup function, then a subquery with 1i lookbehind window and 1i step is automatically formed. For example, rate(sum(up)) is automatically converted to rate((sum(default_rollup(up)))[1i:1i]). This behavior can be disabled or logged via cmd-line flags -search.disableImplicitConversion and -search.logImplicitConversion since v1.101.0.

So {__name__=~"node_netstat_Ip(6|)_(Reasm|Frag).*", job="node_exporter"} * 30 is evaluated as a subquery. Also, this will be further transformed into default_rollup({__name__=~"node_netstat_Ip(6|)_(Reasm|Frag).*", job="node_exporter"}) + 30.
In order to make that work as you expect you need to use the following query: rate({__name__=~"node_netstat_Ip(6|)_(Reasm|Frag).*", job="node_exporter"} * 30 keep_metric_names) keep_metric_names.
See this for example.

Also, starting from v1.101.0 it is possible to either enable logging for implicit query conversions or disable this completely by using -search.logImplicitConversion or -search.disableImplicitConversion command-line flags respectively.

[tekert]

Thanks @zekker6 , "other than series selector" mean a subquery that returns multiple metrics in this case?

So to understand correctly if i disable implicit conversion, what would i have to write? if I write the query without * 30 :
rate(default_rollup({__name__=~"node_netstat_Ip(6|)_(Out|In).*"})[30s]) keep_metric_names
Works, and to add a binary operation, seems i have to do it inside the selection like this
rate(default_rollup({__name__=~"node_netstat_Ip(6|)_(Out|In).*"} * 30 keep_metric_names)) keep_metric_names
And not like this:
rate(default_rollup({__name__=~"node_netstat_Ip(6|)_(Out|In).*"}) * 30 keep_metric_names) keep_metric_names

edit: funny if i multiply doing * 30s (s at the end) it works too, wonder what is doing.

[zekker6]

@tekert

"other than series selector" mean a subquery that returns multiple metrics in this case?

No, in this case rate() takes a binary operation expression as an argument(a * b) which leads to using implicit convertion.

So to understand correctly if i disable implicit conversion, what would i have to write? if I write the query without * 30 :
edit: funny if i multiply doing * 30s (s at the end) it works too, wonder what is doing.

The idea is to pass a metric selector to the rollup function, so when you're using rate(a) * b there will be no subquery since rate argument is a metric selector.

[tekert]

I see, thanks @zekker6 closing this.