Skip to content

Commit

Permalink
feat: handled review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
Hardikl committed Sep 29, 2023
1 parent 0acad2c commit 4ca9a9d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion cmd/collectors/zapi/plugins/aggregate/aggregate.go
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ func (a *Aggregate) Run(dataMap map[string]*matrix.Matrix) ([]*matrix.Matrix, er
if err != nil {
a.Logger.Error().Err(err).Msg("Failed to update footprint data")
// clean the map in case of the error
aggrFootprintMap = make(map[string]map[string]string)
clear(aggrFootprintMap)
}

// update aggregate instance label with cloud stores info
Expand Down
2 changes: 1 addition & 1 deletion cmd/collectors/zapi/plugins/volume/volume.go
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ func (v *Volume) Run(dataMap map[string]*matrix.Matrix) ([]*matrix.Matrix, error
if err != nil {
v.Logger.Error().Err(err).Msg("Failed to update footprint data")
// clean the map in case of the error
volumeFootprintMap = make(map[string]map[string]string)
clear(volumeFootprintMap)
}

// update volume instance labels
Expand Down

0 comments on commit 4ca9a9d

Please sign in to comment.