search

couchbaselabs / sdk-doctor

Application-server-side cluster connection diagnostics.
https://couchbase.com/
Apache License 2.0
18 stars 12 forks source link

Default to localhost #1

Closed ingenthr closed 7 years ago

ingenthr commented 7 years ago

If running sdk-doctor diagnose, it'd be nice if it warned and assumed couchbase://localhost

brett19 commented 7 years ago

The main reason I did not do this is that running diagnostic on the localhost cluster isn't very useful, since the majority of the testing is network related.

ingenthr commented 7 years ago

I agree, but some use cases like the Mac OS Sierra (where query fails to run and just crashes all the time) or Docker (where people forget to map ports) are localhost. I'm okay if you don't want to do this, but I do kind of like the symmetry with the sdks of 'localhost by default if not specified'.

Up to you, close it if you decide not to.

ingenthr commented 7 years ago

Oh, Windows anniv. edition also has that "some services fail to run" thing.