{"action":"created","issue":{"url":"https://api.github.com/repos/prometheus/consul_exporter/issues/1","labels_url":"https://api.github.com/repos/prometheus/consul_exporter/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/prometheus/consul_exporter/issues/1/comments","events_url":"https://api.github.com/repos/prometheus/consul_exporter/issues/1/events","html_url":"https://github.com/prometheus/consul_exporter/pull/1","id":62762278,"number":1,"title":"Initial Exporter Implementation","user":{"login":"ewr","id":204350,"avatar_url":"https://avatars.githubusercontent.com/u/204350?v=3","gravatar_id":"","url":"https://api.github.com/users/ewr","html_url":"https://github.com/ewr","followers_url":"https://api.github.com/users/ewr/followers","following_url":"https://api.github.com/users/ewr/following{/other_user}","gists_url":"https://api.github.com/users/ewr/gists{/gist_id}","starred_url":"https://api.github.com/users/ewr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ewr/subscriptions","organizations_url":"https://api.github.com/users/ewr/orgs","repos_url":"https://api.github.com/users/ewr/repos","events_url":"https://api.github.com/users/ewr/events{/privacy}","received_events_url":"https://api.github.com/users/ewr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-18T18:06:29Z","updated_at":"2015-03-18T20:49:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/prometheus/consul_exporter/pulls/1","html_url":"https://github.com/prometheus/consul_exporter/pull/1","diff_url":"https://github.com/prometheus/consul_exporter/pull/1.diff","patch_url":"https://github.com/prometheus/consul_exporter/pull/1.patch"},"body":"Here's my initial implementation.\r\n\r\nCurrently provides service health per node and number of Consul servers in the cluster."},"comment":{"url":"https://api.github.com/repos/prometheus/consul_exporter/issues/comments/83166892","html_url":"https://github.com/prometheus/consul_exporter/pull/1#issuecomment-83166892","issue_url":"https://api.github.com/repos/prometheus/consul_exporter/issues/1","id":83166892,"user":{"login":"ewr","id":204350,"avatar_url":"https://avatars.githubusercontent.com/u/204350?v=3","gravatar_id":"","url":"https://api.github.com/users/ewr","html_url":"https://github.com/ewr","followers_url":"https://api.github.com/users/ewr/followers","following_url":"https://api.github.com/users/ewr/following{/other_user}","gists_url":"https://api.github.com/users/ewr/gists{/gist_id}","starred_url":"https://api.github.com/users/ewr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ewr/subscriptions","organizations_url":"https://api.github.com/users/ewr/orgs","repos_url":"https://api.github.com/users/ewr/repos","events_url":"https://api.github.com/users/ewr/events{/privacy}","received_events_url":"https://api.github.com/users/ewr/received_events","type":"User","site_admin":false},"created_at":"2015-03-18T20:49:30Z","updated_at":"2015-03-18T20:49:30Z","body":"Pulling in your existing Consul stats code sounds great, but it looks like it adds a dependency that the consul binary be available locally. Using the API package, we can connect to a local agent or to a remote server without needing the binary. Maybe we make it optional with a flag?\r\n\r\nIf the `serf_lan_members` in there already equals nodes, then there's no reason to pull that separately. We're only using the count, not the names. We may need to preserve the existing approach to fetch it if we're not pulling info from a local binary.\r\n\r\nI'm not sure on the agent terminology... I think it's a perspective question. In the current implementation, we're querying at one point for all services registered in Consul. `agent_services` would be the services registered on the local agent, whereas `services` is the count of all services known across the DC. I think that (and specifically nodes on services rather than agents) is in keeping with the Consul API around `catalog/services`.\r\n\r\nIn your case with glimpse, are you pulling stats from every agent, or just globally on the cluster?"}}