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

Dies if name resolution fails - Request (Tied to issue 68) #133

Open
meyerder opened this issue Jan 3, 2024 · 1 comment
Open

Dies if name resolution fails - Request (Tied to issue 68) #133

meyerder opened this issue Jan 3, 2024 · 1 comment

Comments

@meyerder
Copy link

meyerder commented Jan 3, 2024

I read the following Dies if name resolution fails and the expected is to fast fail.

I understand the aspect of fail fast and to not resolve the names again on a regular basis. The only problem with this is if the Address is cached (which appears to be done on initial load) When you have a Fully Qualified Domain Name that is redundant any change or movement will fail to be tested.

For instance

www.domain.com has a Load Balancer
VIP East = 10.10.10.10 (Primary)
VIP West = 11.11.11.11 (Secondary)

When you start the test domain.com resolves to 10.10.10.10. A hour later East has a issue and it fails to West. Without resolving the www.domain.com on a somewhat regular basis you will never know that all of your end user traffic is actually going to 11.11.11.11

I know the solution is to use IP Addresses only and test BOTH but this could be a reason to implement #68

Ideally the metrics would show error or something verse a exit code on the program.

@baryluk
Copy link

baryluk commented Jan 11, 2024

I would like an option to not die (even on startup), if the resolution fails.

Sometimes one has number of hosts to monitor in smokeping command line, and it is very undersirable to make it fail, and restart, if one of the hosts is gone from DNS forever, or some DNS resolution fails temporarily. Let it run, and mark probes as failed.

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