Skip to content

chore: upgrade InfluxQL parser#6166

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

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

Conversation

@vlastahajek
Copy link
Contributor

@vlastahajek vlastahajek commented Jan 16, 2026

⚠️
⚠️ 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
Copy link
Contributor

@bednar bednar left a comment

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

@vlastahajek vlastahajek force-pushed the chore/upgrade_influxql branch from 70b2b71 to b9dc3c0 Compare January 20, 2026 13:09
Copy link
Contributor

@bednar bednar left a comment

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
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