luxolus/open-consul
2
0
Fork 0
Code Issues 1 Pull requests Packages Releases Wiki Activity

Merge pull request #3642 from yfouquet/master

Browse source 
[Fix] Service tags not added to health checks
This commit is contained in:
James Phillips 2017-12-14 13:59:39 -08:00 committed by GitHub
parent a06f29998d 6f5aecad76
commit 518ab954bc
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
6 changed files with 13 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
agent/agent.go
View file

@ -1592,6 +1592,7 @@ func (a *Agent) AddService(service *structs.NodeService, chkTypes []*structs.Che
Notes: chkType.Notes,
ServiceID: service.ID,
ServiceName: service.Service,
ServiceTags: service.Tags,
}
if chkType.Status != "" {
check.Status = chkType.Status

5
agent/agent_test.go
View file

@ -290,6 +290,7 @@ func TestAgent_AddService(t *testing.T) {
Notes: "note1",
ServiceID: "svcid1",
ServiceName: "svcname1",
ServiceTags: []string{"tag1"},
},
},
},
@ -329,6 +330,7 @@ func TestAgent_AddService(t *testing.T) {
Notes: "note1",
ServiceID: "svcid2",
ServiceName: "svcname2",
ServiceTags: []string{"tag2"},
},
"check-noname": &structs.HealthCheck{
Node: "node1",
@ -337,6 +339,7 @@ func TestAgent_AddService(t *testing.T) {
Status: "critical",
ServiceID: "svcid2",
ServiceName: "svcname2",
ServiceTags: []string{"tag2"},
},
"service:svcid2:3": &structs.HealthCheck{
Node: "node1",
@ -345,6 +348,7 @@ func TestAgent_AddService(t *testing.T) {
Status: "critical",
ServiceID: "svcid2",
ServiceName: "svcname2",
ServiceTags: []string{"tag2"},
},
"service:svcid2:4": &structs.HealthCheck{
Node: "node1",
@ -353,6 +357,7 @@ func TestAgent_AddService(t *testing.T) {
Status: "critical",
ServiceID: "svcid2",
ServiceName: "svcname2",
ServiceTags: []string{"tag2"},
},
},
},

3
website/source/api/agent/check.html.md
View file

@ -55,7 +55,8 @@ $ curl \
"Notes": "",
"Output": "",
"ServiceID": "redis",
"ServiceName": "redis"
"ServiceName": "redis",
"ServiceTags": ["primary"]
}
}
```

2
website/source/api/agent/service.html.md
View file

@ -51,7 +51,7 @@ $ curl \
"redis": {
"ID": "redis",
"Service": "redis",
"Tags": null,
"Tags": [],
"Address": "",
"Port": 8000
}

6
website/source/api/health.html.md
View file

@ -62,7 +62,7 @@ $ curl \
"Output": "",
"ServiceID": "",
"ServiceName": "",
"ServiceTags": null
"ServiceTags": []
},
{
"ID": "40e4a748-2192-161a-0510-9bf59fe950b5",
@ -239,7 +239,7 @@ $ curl \
"Output": "",
"ServiceID": "",
"ServiceName": "",
"ServiceTags": null
"ServiceTags": []
}
]
}
@ -303,7 +303,7 @@ $ curl \
"Output": "",
"ServiceID": "",
"ServiceName": "",
"ServiceTags": null
"ServiceTags": []
},
{
"Node": "foobar",

2
website/source/intro/getting-started/checks.html.md
View file

@ -75,7 +75,7 @@ can be run on either node):
```text
vagrant@n1:~$ curl http://localhost:8500/v1/health/state/critical
[{"Node":"agent-two","CheckID":"service:web","Name":"Service 'web' check","Status":"critical","Notes":"","ServiceID":"web","ServiceName":"web"}]
[{"Node":"agent-two","CheckID":"service:web","Name":"Service 'web' check","Status":"critical","Notes":"","ServiceID":"web","ServiceName":"web","ServiceTags":["rails"]}]
```
We can see that there is only a single check, our `web` service check, in the