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.

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

dns rate limiting hard-coded to 10s, causing havoc when app window is smaller #1128

Closed
rade opened this issue Mar 5, 2016 · 3 comments
Closed

Comments

@rade
Copy link
Member

rade commented Mar 5, 2016

I am running with an app.window of 5s and see a lot of instability in the 'BY DNS NAME' view, with nodes frequently disappearing.

I reckon the cause of that is similar to #1122, i.e. a hard-coded polling interval of 10s, this time at

dnsPollInterval = 10 * time.Second
.

@rade
Copy link
Member Author

rade commented Mar 5, 2016

Hmm. I am seeing this behaviour even with the standard app.window. And it doesn't go away when I lower the hard-coded interval to 1s. So there might be something else going on here.

@tomwilkie
Copy link
Contributor

Yes, the reverse dba resolution is currently not used in the rendering
pipeline (I jut opened a bug about this - #1121). Also, the reverse lookups use a
cache so should be pretty stable.

The problem here is one of Paul's pet peeves and is due to how we merge
nodes in the rendering pipe line. There is already a bug about this.

On Saturday, 5 March 2016, Matthias Radestock [email protected]
wrote:

Hmm. I am seeing this behaviour even with the standard app.window. And it
doesn't go away when I lower the hard-coded interval to 1s. So there might
be something else going on here.


Reply to this email directly or view it on GitHub
#1128 (comment).

@tomwilkie
Copy link
Contributor

Paul + I believe this is actually a dupe of #1130

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants