Skip to content

Commit 195185b

Browse files
committed
(SUP-4126) Fix file sync additional metrics
Prior to this commit, the mbeans for the additional file sync storage metrics started with 'puppetserver:name=puppetlabs.puppet'; however, the object name of the mbeans include the hostname of the compiling server. This resulted in the api returning 404s and not collecting the metrics. This commit replaces the 'puppet' part of the name with a glob, which should match correctly.
1 parent 09eede4 commit 195185b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

manifests/service/puppetserver.pp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,17 +25,17 @@
2525
{
2626
'type' => 'read',
2727
'name' => 'file-sync-storage-commit-timer',
28-
'mbean' => 'puppetserver:name=puppetlabs.puppet.file-sync-storage.commit-timer'
28+
'mbean' => 'puppetserver:name=puppetlabs.*.file-sync-storage.commit-timer'
2929
},
3030
{
3131
'type' => 'read',
3232
'name' => 'file-sync-storage-pre-commit-hook-timer',
33-
'mbean' => 'puppetserver:name=puppetlabs.puppet.file-sync-storage.pre-commit-hook-timer'
33+
'mbean' => 'puppetserver:name=puppetlabs.*.file-sync-storage.pre-commit-hook-timer'
3434
},
3535
{
3636
'type' => 'read',
3737
'name' => 'file-sync-storage-commit-add-rm-timer',
38-
'mbean' => 'puppetserver:name=puppetlabs.puppet.file-sync-storage.commit-add-rm-timer'
38+
'mbean' => 'puppetserver:name=puppetlabs.*.file-sync-storage.commit-add-rm-timer'
3939
},
4040
]
4141

0 commit comments

Comments
 (0)