avoid overwriting metrics on success, save it in defer (#17780)

This commit is contained in:
Harshavardhana 2023-08-01 22:19:56 -07:00 committed by GitHub
parent 0153f96a20
commit 21cdd2bf5d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -275,9 +275,11 @@ func (p *xlStorageDiskIDCheck) DiskInfo(ctx context.Context, metrics bool) (info
si := p.updateStorageMetrics(storageMetricDiskInfo) si := p.updateStorageMetrics(storageMetricDiskInfo)
defer si(&err) defer si(&err)
defer func() {
if metrics { if metrics {
info.Metrics = p.getMetrics() info.Metrics = p.getMetrics()
} }
}()
if p.health.isFaulty() { if p.health.isFaulty() {
// if disk is already faulty return faulty for 'mc admin info' output and prometheus alerts. // if disk is already faulty return faulty for 'mc admin info' output and prometheus alerts.