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

Import is failing with new profiling code when no volumes present in the cluster #1075

Closed GowthamShanmugam closed 5 years ago

GowthamShanmugam commented 5 years ago

tendrl-bug-id: Tendrl/commons#1074 bugzilla: 1676897

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

GowthamShanmugam commented 5 years ago

@shtripat @nthomas-redhat @anmolsachan please review

codecov[bot] commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@55edf1a). Click here to learn what that means. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1075   +/-   ##
=========================================
  Coverage          ?   81.53%           
=========================================
  Files             ?       90           
  Lines             ?     3753           
  Branches          ?      487           
=========================================
  Hits              ?     3060           
  Misses            ?      595           
  Partials          ?       98
Impacted Files Coverage Δ
...s/objects/cluster/atoms/import_cluster/__init__.py 11.29% <ø> (ø)

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 55edf1a...47454f8. Read the comment docs.

GowthamShanmugam commented 5 years ago

@shtripat @nthomas-redhat need to take this PR

GowthamShanmugam commented 5 years ago

image