Skip to content

chore: upgrade InfluxQL parser - #6166

Merged
bednar merged 37 commits into
masterfrom
chore/upgrade_influxql
Feb 2, 2026
Merged

chore: upgrade InfluxQL parser#6166
bednar merged 37 commits into
masterfrom
chore/upgrade_influxql

Conversation

@vlastahajek

@vlastahajek vlastahajek commented Jan 16, 2026

Copy link
Copy Markdown
Contributor

⚠️
⚠️ Breaking changes:
⚠️ removed release packages
⚠️ - darwin arm64 - cross builder doesn't support building Rust/C code for darwin arm64
⚠️ - linux armhf, armel, i386, static builds

⚠️

The current InfluxQL parser, taken from InfluxDB 1.1.5 doesn't support sub-queries. A parser is needed for V3 support to adjust queries for the backward compatibility endpoint.

Major changes:

  • InfluxQL parser upgraded to the InfluxQL parser library 1.4.1
  • Upgraded Kapacitor to the latest - 1.8.2.
  • Upgraded Flux from 0.65 to 0.199.
  • Upgraded Flux required changes in the build system:
    • using cross builder image
      • removed using custom builder docker image
      • removed the above-mentioned architectures for releasing
    • removed creating and pushing to quay.io Chronograf docker image

Testing requirements:

  • verify the new binaries correctly work on target platforms

  • verify InfluxQL editor works correctly

    • test query with sub-query against InfluxDB 3 Core/Enterprise
  • verify Flux editor works correctly

  • verify Kapacitor management, especially using Teams channel URL

    • On the current release of Chronograf:
      • enable Teams notification endpoint and set Teams channel URL -
      • You can use any HTTP listener as a Webhook endpoint
      • Use Teams notification in a tick script and use custom channel URL- try testing endpoint
    • On the newest Chronograf from the branch:
      • verify Teams endpoint configuration
      • verify Team notification channel URL in the tickscript
  • CHANGELOG.md updated with a link to the PR (not the Issue)

  • Well-formatted commit messages

  • Rebased/mergeable

  • Tests pass

  • swagger.json updated (if modified Go structs or API)

@vlastahajek
vlastahajek marked this pull request as draft January 16, 2026 09:38
@vlastahajek
vlastahajek requested a review from bednar January 19, 2026 07:52
@vlastahajek
vlastahajek marked this pull request as ready for review January 19, 2026 07:52

@bednar bednar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vlastahajek thanks for this improvement! 👍

Please add tests for previously failed InfluxQL from https://influxdata.slack.com/archives/C5BSZ026L/p1767895523973259

Comment thread server/flux.go Outdated
Comment thread ui/package.json Outdated
Comment thread CHANGELOG.md
Comment thread kapacitor.go
@vlastahajek
vlastahajek force-pushed the chore/upgrade_influxql branch from 70b2b71 to b9dc3c0 Compare January 20, 2026 13:09

@bednar bednar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@karel-rehor

Copy link
Copy Markdown
Contributor

Basic tests completed following recommendations in description. Completed plan can be found here.

Looks good to me. 🚴 🏁

@bednar
bednar merged commit 39f8199 into master Feb 2, 2026
4 checks passed
@bednar
bednar deleted the chore/upgrade_influxql branch February 2, 2026 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants