Tendrl / commons

Common code usable by all Tendrl components
http://www.tendrl.org
GNU Lesser General Public License v2.1
4 stars 23 forks source link

Brick path replace symbol in collectd configuration file #1071

Closed GowthamShanmugam closed 5 years ago

GowthamShanmugam commented 5 years ago

tendrl-bug-id: Tendrl/commons#1070 bugzilla: 1660779

Signed-off-by: GowthamShanmugasundaram gshanmug@redhat.com

GowthamShanmugam commented 5 years ago

@shtripat @nthomas-redhat please review

GowthamShanmugam commented 5 years ago

@shtripat @nthomas-redhat i have verified the changes, it works fine

codecov[bot] commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4887997). Click here to learn what that means. The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1071   +/-   ##
=========================================
  Coverage          ?   82.35%           
=========================================
  Files             ?       90           
  Lines             ?     3712           
  Branches          ?      478           
=========================================
  Hits              ?     3057           
  Misses            ?      556           
  Partials          ?       99
Impacted Files Coverage Δ
...cts/cluster/atoms/configure_monitoring/__init__.py 78.57% <33.33%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4887997...8eede6f. Read the comment docs.

GowthamShanmugam commented 5 years ago

@shtripat changes done