search

Tendrl / gluster-integration

Extracts all data from a Gluster cluster for consumption by Tendrl
GNU Lesser General Public License v2.1
6 stars 20 forks source link

Import cluster is failing when one or more storage nodes does not have at least one lvm #716

Closed GowthamShanmugam closed 5 years ago

GowthamShanmugam commented 5 years ago

bugzilla: 1683603 tendrl-bug-id: Tendrl/gluster-integration#715

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

GowthamShanmugam commented 5 years ago

@shtripat @nthomas-redhat @dahorak please review

shtripat commented 5 years ago

The pep8 issue is related to this PR or generic one?

codecov[bot] commented 5 years ago

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff            @@
##             master    #716   +/-   ##
========================================
  Coverage          ?   38.7%           
========================================
  Files             ?      42           
  Lines             ?    2108           
  Branches          ?     287           
========================================
  Hits              ?     816           
  Misses            ?    1262           
  Partials          ?      30
Impacted Files Coverage Δ
.../gluster_integration/sds_sync/brick_utilization.py 10.84% <0%> (ø)
...rl/gluster_integration/sds_sync/vol_utilization.py 8.88% <0%> (ø)

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 6dd8a8b...823ffa8. Read the comment docs.

GowthamShanmugam commented 5 years ago

@shtripat I verified this fix