a question about ai-statistics plugin

[jujiale]

hello:
I have a question about plugin ai-statistics，the plugin code as below:
in func writeMetric, there is a logic
`

    config.incrementCounter(generateMetricName(route, cluster, model, consumer, InputToken), inputToken)
config.incrementCounter(generateMetricName(route, cluster, model, consumer, OutputToken), outputToken)

`

and in func generateMetricName:
`

func generateMetricName(route, cluster, model, consumer, metricName string) string {
return fmt.Sprintf("route.%s.upstream.%s.model.%s.consumer.%s.metric.%s", route, cluster, model, consumer, metricName)

}

`

last in config.incrementCounter, the code as below:

`

 func (config *AIStatisticsConfig) incrementCounter(metricName string, inc uint64) {
if inc == 0 {
	return
}
counter, ok := config.counterMetrics[metricName]
if !ok {
	counter = proxywasm.DefineCounterMetric(metricName)
	config.counterMetrics[metricName] = counter
}
counter.Increment(inc)

}

`

my question is:
the metrics name should be:route.xxx.upstream.yyy.model.zzz.consumer.aaa.metric.bbb, because it generated by:
`

 func generateMetricName(route, cluster, model, consumer, metricName string) string {
return fmt.Sprintf("route.%s.upstream.%s.model.%s.consumer.%s.metric.%s", route, cluster, model, consumer, metricName)

`

but I find in http://ip:15020/stats/prometheus, the metrics name as below:
`

 route_upstream_model_consumer_metric_input_token{ai_route="test-baiwen-chat",ai_cluster="outbound|80||janus-general.static",ai_model="qwen2.5-72b-instruct-int8-batch",ai_consumer="none"} 148

`

I want to know the metrics name why is route_upstream_model_consumer_metric_input_token rather than route.xxx.upstream.yyy.model.zzz.consumer.aaa.metric.bbb

is there any logic I ignored?

hope your answer. thanks

[jujiale]

and also, if I want to add tags in currrent metrics, what should I do, I tried modify the source code and added some tags, but failed, if have some examples

[cr7258]

I want to know the metrics name why is route_upstream_model_consumer_metric_input_token rather than route.xxx.upstream.yyy.model.zzz.consumer.aaa.metric.bbb

The key and value of the metric label will be extracted from the original metric, and then the value will be removed from the original metric.
Envoy implements Proxy-Wasm ABI, you can find the logic in the Envoy source code.

[jujiale]

The key and value of the metric label will be extracted from the original metric, and then the value will be removed from the original metric. Envoy implements Proxy-Wasm ABI, you can find the logic in the Envoy source code.

thanks for your reply, if I want to add tags in metrics, mustI make a image for envoy and expose the tags like below？
`

 "stats_tags": [
  {
    "tag_name": "wasm_runtime",
    "regex": "^wasm\\.envoy\\.wasm\\.(.*?\\.)plugin"
  },
  {
    "tag_name": "thread_name",
    "regex": "^wasm\\..*?\\.plugin\\..*?\\..*?\\.(\\w+\\.)"
  },
  {
    "tag_name": "plugin_name",
    "regex": "^wasm\\..*?plugin\\.(.*?\\..*?\\.)"
  },
  {
    "tag_name": "ai_route",
    "regex": "^wasmcustom\\.route\\.((.*?)\\.)upstream"
  },
  {
    "tag_name": "ai_cluster",
    "regex": "^wasmcustom\\..*?\\.upstream\\.((.*?)\\.)model"
  },
  {
    "tag_name": "ai_model",
    "regex": "^wasmcustom\\..*?\\.model\\.((.*?)\\.)consumer"
  },
  {
    "tag_name": "ai_consumer",
    "regex": "^wasmcustom\\..*?\\.consumer\\.((.*?)\\.)metric"
  },
  {{- range $a, $tag := .extraStatTags }}
  {
    "regex": "({{ $tag }}=\\.=(.*?);\\.;)",
    "tag_name": "{{ $tag }}"
  },
  {{- end }}
  {
    "tag_name": "response_code_class",
    "regex": "_rq(_(\\dxx))$"
  },
  {
    "tag_name": "listener_address",
    "regex": "^listener\\.(((?:[_.[:digit:]]*|[_\\[\\]aAbBcCdDeEfF[:digit:]]*))\\.)"
  },
  {
    "tag_name": "http_conn_manager_prefix",
    "regex": "^http\\.(((outbound_([0-9]{1,3}\\.{0,1}){4}_\\d{0,5})|([^\\.]+))\\.)"
  },
  {
    "tag_name": "cluster_name",
    "regex": "^cluster\\.((.*?)\\.)(http1\\.|http2\\.|health_check\\.|zone\\.|external\\.|circuit_breakers\\.|[^\\.]+$)"
  }
]

`

if have some other ways, that only through modify higress plugin souce code, achieve this function.

[cr7258]

cc @johnlanni