From 2f710ec77ddcc08176b53f145974da8fdd80fc51 Mon Sep 17 00:00:00 2001 From: Roman Khavronenko Date: Tue, 4 Jul 2023 13:45:42 +0200 Subject: [PATCH] vmctl: interrupt explore procedure in influx mode if no numeric fields were found (#4576) Signed-off-by: hagen1778 --- app/vmctl/influx/influx.go | 4 ++++ docs/CHANGELOG.md | 1 + 2 files changed, 5 insertions(+) diff --git a/app/vmctl/influx/influx.go b/app/vmctl/influx/influx.go index ab25bcb3f..32b5998c6 100644 --- a/app/vmctl/influx/influx.go +++ b/app/vmctl/influx/influx.go @@ -153,6 +153,10 @@ func (c *Client) Explore() ([]*Series, error) { return nil, fmt.Errorf("failed to get field keys: %s", err) } + if len(mFields) < 1 { + return nil, fmt.Errorf("found no numeric fields for import in database %q", c.database) + } + series, err := c.getSeries() if err != nil { return nil, fmt.Errorf("failed to get series: %s", err) diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md index 7da68a4f6..659250985 100644 --- a/docs/CHANGELOG.md +++ b/docs/CHANGELOG.md @@ -46,6 +46,7 @@ The following tip changes can be tested by building VictoriaMetrics components f Remove redundant limit from [Prometheus api/v1/series](https://docs.victoriametrics.com/Single-server-VictoriaMetrics.html#prometheus-querying-api-usage). See [this issue](https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4339). * BUGFIX: [vmui](https://docs.victoriametrics.com/#vmui): fix application routing issues and problems with manual URL changes. See [this pull request](https://github.com/VictoriaMetrics/VictoriaMetrics/pull/4408). * BUGFIX: add validation for invalid [partial RFC3339 timestamp formats](https://docs.victoriametrics.com/Single-server-VictoriaMetrics.html#timestamp-formats) in query and export APIs. +* BUGFIX: [vmctl](https://docs.victoriametrics.com/vmctl.html): interrupt explore procedure in influx mode if vmctl found no numeric fields. ## [v1.91.3](https://github.com/VictoriaMetrics/VictoriaMetrics/releases/tag/v1.91.3)