-
Notifications
You must be signed in to change notification settings - Fork 282
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
"host" is repeatedly used as questionable descriptor for full url aka "http://localhost:8086" #5644
Comments
@wolfgangr Thanks for raising this. I agree that the naming of the
Do you have an example in the docs web site that isn't clear about this or are you referring to the Usage docs in the CLI itself? The URL that you listed (https://docs.influxdata.com/influxdb/v2/admin/backup-restore/backup/) doesn't include the use of |
Sure? 🙈 🙊 😁 The superficial answer might be: But we don't spend our time on deciding who is right and wrong, but on assisting other people not to fall in the same trap - OK?
I try to remember:
What could the influx development team do to avoid people get trapped? Envrionment variablesOf course, this is a matter of habit and taste. But those have reasons, too: From https://peps.python.org/pep-0020/#the-zen-of-python
...and the cmd line env is a namespace with quite a laaaarge scope. I know that many projects - Python itself among them - don't care about this wisdom and release uncooperative Software. Sure, you may add your veryveryvery secret token to your .bashrc, if you're on a docker jail or development-dont-care-box. But on a headless production machine, when I administer dozens of services from parallel ssh sessions maybe from different clients, that's not the way to go. Anyway, essence: Misleading error messagesHaving understood my mistake, it was immediately obvious that the "version mismatch" did not come from influxdb <-> influx cmd line, but from apache <-> influx cmd line, sitting at the default http port. The developer may say "so, look, not my fault". Essence: since the web is full of similiar case report, may be it is worth to consider to catch this case (reporting server seems not to be an influxDB) and report a proper error message aka "host may not be a influxd, please check your host URL . format http://hostname:port" Organisation of documentationTo admit, I find the structure of Influx documentation rather confusing. If you think my input may help you in your strive to improve documentation, we may try to dig further. If you think that's OK rihgt now or even I'm stealing your valuable time, simply close the issue. |
just cleaning up and found the files I think yo might think "why didn't he look there": https://docs.influxdata.com/influxdb/v2/reference/cli/influx/backup/ Indeed is refered to by the issue's starting page in a Link quite at the bottom. The option flag enty reads I can remeber having seen this before. For protocol, the guess was right, for port is was wrong. Howerver the message presented didn't provide any clue that I was talking to the wrong server.
It was only when I got the second line of the message with a May be even the first line (presumably added by the backup code?) was even intended to provide assisting information. the other one is |
I understand and apologize for the confusion. I see your point about the language used to describe the InfluxDB default URL in the I'll keep this ticket open with this list of suggested improvements:
I'm sure I missed some finer points, so please list them if you can.
If you have specific suggestions where we can improve, please add them. We have many docs improvements we want to make, but, like you, limited time and resources. This will help us prioritize. FWIW, reliance on environment variables is also my pet peeve and I wish our documentation and tooling didn't rely on or assume them so often. I'm also a JS/UI developer and I get the dangers and annoyances of polluting the namespace. We truly appreciate the patience and the feedback. |
Describe the issue here.
complicated description arising from backup error here:
influxdata/influxdb#25460
simple root cause:
influx ping --host localhost
does not workinflux ping --host "http://localhost:8086"
worksI'd consider it as an documentation issue.
There are loads of web articles from people trapped by this.
"host" in my mental dictionary is just the "host" part of an URL, while in influx world, obviously it refers to a complete url:
"protocol://host:port"
I know that any influx pro is just rofl on this case.
But climbing up the learning curve this are silly issues costing you hours of work.
Relevant URLs
The text was updated successfully, but these errors were encountered: