RepoMirrors
/
prometheus
mirror of https://github.com/prometheus/prometheus
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix broken test which relied on DNS resolution #1962 

Switched to testing by way of the static_configs rather than
dns_sd_config parameter.  Verified that the revised test both passes
without network access, and also still catches the bug it's supposed to
cover.
This commit is contained in:
Dan Milstein 2016-09-08 16:59:46 -04:00
parent e5a5dc172d
commit 0cb6b9962e
1 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
retrieval/targetmanager_test.go
View File

@ -42,9 +42,9 @@ func TestTargetSetRecreatesTargetGroupsEveryRun(t *testing.T) {
sOne := `
job_name: "foo"
dns_sd_configs:
- names:
- "srv.name.one.example.org"
static_configs:
- targets: ["foo:9090"]
- targets: ["bar:9090"]
`
if err := yaml.Unmarshal([]byte(sOne), scrapeConfig); err != nil {
t.Fatalf("Unable to load YAML config sOne: %s", err)
@ -57,13 +57,13 @@ dns_sd_configs:
ts.runProviders(context.Background(), providersFromConfig(scrapeConfig))
verifyPresence(ts.tgroups, "dns/0/srv.name.one.example.org", true)
verifyPresence(ts.tgroups, "static/0/0", true)
verifyPresence(ts.tgroups, "static/0/1", true)
sTwo := `
job_name: "foo"
dns_sd_configs:
- names:
- "srv.name.two.example.org"
static_configs:
- targets: ["foo:9090"]
`
if err := yaml.Unmarshal([]byte(sTwo), scrapeConfig); err != nil {
t.Fatalf("Unable to load YAML config sTwo: %s", err)
@ -71,8 +71,8 @@ dns_sd_configs:
ts.runProviders(context.Background(), providersFromConfig(scrapeConfig))
verifyPresence(ts.tgroups, "dns/0/srv.name.one.example.org", false)
verifyPresence(ts.tgroups, "dns/0/srv.name.two.example.org", true)
verifyPresence(ts.tgroups, "static/0/0", true)
verifyPresence(ts.tgroups, "static/0/1", false)
}
func mustNewRegexp(s string) config.Regexp {