-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
Documentation feedback on /docs/integrate/etl/influxdb.md #86
Comments
Hi. Trying the Cloud-to-Cloud migration now. Based on https://github.com/daq-tools/influxio/blob/main/influxio/core.py I guess something like this should work:
Getting:
For some other variations also:
|
Analyzing the log output, the CrateDB connectivity information looks reasonable.
Yeah, I mean, that's a hard error. Please create dedicated tickets for missing features, including a detailed repro how |
Will do that
I don't believe so, there's not much customization possible, at least on my free version. Signup->region->deployed. Will create corresponding tickets, thanks. |
{"code":"unauthorized","message":"unauthorized access"} It looks like this is a regular error, not specifically related to InfluxDB2/3, so the root cause is probably on our side. https://community.influxdata.com/search?q=unauthorized%20access |
Using influxio, this pair of commands appear to work. Can you confirm that? influxio copy \
"https://9faaa869a91a3bbb:U379DVLDHD8AJsjzOEluu7TO5nGG6NAca15ksfTa6PqJMWBkBC5haJy7wcWyxsmoo9y_zpk8Gbns9PcLaLa4A==@eu-central-1-1.aws.cloud2.influxdata.com/testdrive/demo" \
"sqlite:///export.sqlite?table=demo" sqlite3 -bail export.sqlite -cmd 'SELECT * FROM demo;' -cmd '.quit' NB: Organization ID and authentication token have been amended, so they are invalid. Please use valid ones instead. |
If that procedure works, and also succeeds on a relevant |
Hi. I am unable to reproduce this. Could you share how it should look with the correct URIs? |
Trying something like this (credentials edited), adapted from your suggestion:
Last line of the error: https://docs.sqlalchemy.org/en/20/errors.html#error-e3q8 |
New versions of relevant packages have been released, including a corresponding fix. You can update to the most recent release, using one of those commands.
Please confirm if that remedies the problem, and, if so, please add a corresponding section about cloud-to-cloud use to the documentation. See also the corresponding Export from Cloud to Cloud documentation of influxio how we present that case concisely by shortening access tokens significantly, in order to reduce line length, and by that, improving readability. |
Hi again. Confirming that
|
Analysis
It might surprise you because you've also worked with influxio, but
Possible UX ImprovementsThoughts ISaying this, there may be room for improvements. For example, influxio could be more lenient, and also accept the Thoughts IINotImplementedError: Importing resource not implemented yet Indeed, the error message could be improved to better convey the application does not understand the Going beyond improving the error message to better convey the reason, we may think about adding additional hints to it, like
|
Ah. That's it, and my bad from deviating from using the right endpoint. But as you said, that kind of error message will be helpful if implemented. Output looks better now, however I'm still having and issue even when using (credentials are correct, confirmed with
|
Observations
In this log output, it appears to me that EvaluationIt is probably the wrong choice, because it says ThoughtsBecause the synthetic protocol identifier is agnostic of plain vs. ssl comms, we will probably also need to synthesize the "I want encryption" flag through a corresponding While we could introduce things like AssessmentThe flaw is in this section, source_url = source_url.replace("influxdb2://", "http://") |
CrateDB Toolkit v0.0.12, released just now, may resolve that problem.
-- https://cratedb-toolkit.readthedocs.io/io/influxdb/loader.html#cloud |
@matkuliak confirmed it works well now. Thank you. |
As an aftermath, will you add a corresponding section about cloud-to-cloud copying to this document, @matkuliak? I think it will be fine to use that miniature section as a blueprint, but make it a bit more verbose/rich, by linking to canonical InfluxDB Cloud and CrateDB Cloud pages, and/or elaborating about how to sign up, or such. -- https://cratedb-toolkit.readthedocs.io/io/influxdb/loader.html#cloud |
Thanks for that patch. |
Documentation feedback
This document uses a Docker-based setup for conveniency reasons, which is excellent.
Other than this, it would be nice to get a rough idea how to run the procedure in a Cloud-to-Cloud scenario.
The text was updated successfully, but these errors were encountered: