Commit 859d5dab authored by Matthias Kaeppler's avatar Matthias Kaeppler

Include container registry in topology pings

We forgot to map this, so registry nodes aren't currently reported.
parent 362fa7b9
......@@ -13,7 +13,8 @@ module Gitlab
'postgres' => 'postgres',
'gitaly' => 'gitaly',
'prometheus' => 'prometheus',
'node' => 'node-exporter'
'node' => 'node-exporter',
'registry' => 'registry'
}.freeze
CollectionFailure = Struct.new(:query, :error) do
......
......@@ -82,6 +82,10 @@ RSpec.describe Gitlab::UsageData::Topology do
process_count: 1,
process_memory_rss: 402
},
{
name: 'registry',
process_count: 1
},
{
name: 'web',
server: 'unicorn'
......@@ -133,6 +137,10 @@ RSpec.describe Gitlab::UsageData::Topology do
{
name: 'redis',
process_count: 1
},
{
name: 'registry',
process_count: 1
}
]
},
......@@ -356,6 +364,10 @@ RSpec.describe Gitlab::UsageData::Topology do
'metric' => { 'instance' => 'instance2:9121', 'job' => 'redis' },
'value' => [1000, '1']
},
{
'metric' => { 'instance' => 'instance2:8080', 'job' => 'registry' },
'value' => [1000, '1']
},
# unknown service => should be stripped out
{
'metric' => { 'instance' => 'instance2:9000', 'job' => 'not-a-gitlab-service' },
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment