Infra
/
lemur
5
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Merge pull request #2907 from Netflix/fix_write_domains_on_failure_01 

fixing issue where set_domains() is still called when get_all_zones()…
This commit is contained in:
csine-nflx 2020-01-31 13:18:00 -08:00 committed by GitHub
parent 30ab0bcdbe 139769edea
commit 02757359f0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lemur/dns_providers/cli.py
View File

@ -35,7 +35,6 @@ def get_all_zones():
print("[+] Error with DNS Provider {}: {}".format(dns_provider.name, e)) print("[+] Error with DNS Provider {}: {}".format(dns_provider.name, e))
log_data["message"] = f"get all zones failed for {dns_provider} {e}." log_data["message"] = f"get all zones failed for {dns_provider} {e}."
sentry.captureException(extra=log_data) sentry.captureException(extra=log_data)
set_domains(dns_provider, [])
status = SUCCESS_METRIC_STATUS status = SUCCESS_METRIC_STATUS