Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

InfluxDB 2 support #32

Open
jonppe opened this issue Jan 4, 2021 · 2 comments
Open

InfluxDB 2 support #32

jonppe opened this issue Jan 4, 2021 · 2 comments
Assignees
@hagen1778
Labels
enhancement New feature or request

Comments

@jonppe
Copy link

jonppe commented Jan 4, 2021

InfluxDB 2 has been out for a while and I think vmctl does not properly support that.
The result I'm seeing is:

InfluxDB import mode
2021/01/01 10:02:49 failed to create influx client: ping failed: <!doctype html><html><head><meta charset="utf-8"><meta name="viewport" content="width=device-width,initial-scale=1"><title>InfluxDB 2.0</title><base href="/"><link rel="shortcut icon" href="/favicon.ico"></head><body><div id="react-root" data-basepath=""></div><script src="/5e93c5f5aa.js"></script></body></html>

Very briefly looking at the codebase, I suppose vmctl would need to use the new version of the official influxdb client or bring in the changes some other way.

Alternatively, it would be good to mention that >2.0 is not supported.

If anyone else faces the same issue maybe this Python script is good enough alternative
https://github.com/jonppe/influx_to_victoriametrics
@hagen1778
Copy link
Contributor

Hi @jonppe! Thanks for the issue and your own implementation for this!
I'm not familiar with influxdb and everything which happens to it. Migration from v1 was implemented just by reading the docs and trying things for the first time. I wasn't aware there is a v2. Thanks for pointing this out!

I briefly looked at v2 and new go client. This all seems like a totally new piece of software to me and may take additional time to investigate. So far I'll just outline that vmctl does not support influxdb v2.x and will recommend your solution instead- pls see 2d95427

Once again thank you for the issue and alternative implementation. I guess, I'll use it as reference when working on v2 support.

@valyala
Copy link
Contributor

valyala commented Feb 1, 2021

FYI, vmctl has been migrated to main VictoriaMetrics repository. New releases for vmctl will be available at vmutils-*.tar.gz package on the releases page starting from v1.53.0 . It is recommended filing new issues and feature requests for vmctl in main VictoriaMetrics repository.

@valyala valyala added the enhancement New feature or request label Feb 1, 2021
@zekker6 zekker6 mentioned this issue Mar 4, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hagen1778 hagen1778

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@valyala @hagen1778 @jonppe