Skip to content

Port analytics to InfluxDB v3 write_lp API with no_sync=true - #23619

Closed
SMillerDev with Copilot wants to merge 2 commits into
mainfrom
copilot/port-api-calls-to-influxdb-v3
Closed

Port analytics to InfluxDB v3 write_lp API with no_sync=true#23619
SMillerDev with Copilot wants to merge 2 commits into
mainfrom
copilot/port-api-calls-to-influxdb-v3

Conversation

Copilot AI commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Updates the report_influx method in utils/analytics.rb to use the InfluxDB v3 write API.

Changes

  • Replaces /api/v2/write?bucket=...&precision=s with /api/v3/write_lp?bucket=...&precision=seconds&no_sync=true
  • no_sync=true skips fsync for better write performance
  • Updated the inline comment to reflect both performance considerations

Copilot AI and others added 2 commits August 22, 2026 13:28
Co-authored-by: SMillerDev <1484494+SMillerDev@users.noreply.github.com>
Co-authored-by: SMillerDev <1484494+SMillerDev@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for your pull request. This has been closed because it appears to be missing the pull request template, perhaps because this was written by an AI not a human. We require humans to read and fill in these templates.

Please edit this pull request to fill in the current pull request template. This workflow will reopen this pull request automatically once the template is complete. Do not open a new pull request for this.

@SMillerDev

Copy link
Copy Markdown
Member

Really copilot? Didn't even try to fill out the PR template?

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.

2 participants