Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Print out info on upstream servers for weaveDNS debugging #1571

Closed
bboreham opened this issue Oct 21, 2015 · 5 comments
Closed

Print out info on upstream servers for weaveDNS debugging #1571

bboreham opened this issue Oct 21, 2015 · 5 comments
Assignees
Milestone

Comments

@bboreham
Copy link
Contributor

When remote troubleshooting it can be useful to know what weave thinks its upstream servers are.

@bboreham
Copy link
Contributor Author

And it would be useful to log out that there are zero upstream servers, especially if we get asked to resolve an upstream name.

@rade
Copy link
Member

rade commented Oct 21, 2015

it would be useful to log out that there are zero upstream servers, especially if we get asked to resolve an upstream name.

No. Except perhaps at debug log level.

@awh
Copy link
Contributor

awh commented Oct 21, 2015

weave report all the things

@rade
Copy link
Member

rade commented Oct 21, 2015

I think it's perfectly reasonable to have the upstreams reported in weave status, not just weave report. I am opposed to logging "there are zero upstream servers" every time we attempt to resolve a name upstream.

@rade rade added this to the 1.3.0 milestone Oct 29, 2015
@darabi
Copy link

darabi commented Oct 30, 2015

As a user, I would appreciate a message during startup which clearly states that there are zero upstream servers, making me aware of my mis-configuration. That would have made my life easier and saved @bboreham from my stupid questions on IRC.

In debug mode, I would also appreciate a message like 'cannot resolve name …. in local database and as I don't have upstreams configured, I return a ServFail'.

@inercia inercia self-assigned this Nov 10, 2015
@awh awh closed this as completed in #1662 Nov 25, 2015
awh added a commit that referenced this issue Nov 25, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants