Closed GowthamShanmugam closed 6 years ago
If volume is deleted flag is marked as True then it won't be changed because we are not pushing None value in etcd
Merging #582 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #582 +/- ##
=======================================
Coverage 41.92% 41.92%
=======================================
Files 70 70
Lines 2571 2571
Branches 362 362
=======================================
Hits 1078 1078
Misses 1461 1461
Partials 32 32
Impacted Files | Coverage Δ | |
---|---|---|
...drl/gluster_integration/objects/volume/__init__.py | 94.87% <ø> (ø) |
:arrow_up: |
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 b04c49e...30ead04. Read the comment docs.
Tendrl/monitoring-integration#354
@shtripat @nthomas-redhat @r0h4n please review
@shtripat @r0h4n @nthomas-redhat please review
@GowthamShanmugam I am not sure why you are talking about "volume_alert_counter" in this PR
Also, the commits need to be signed. Please ensure that in future
ah, sorry i wrongly commented here :)
tendrl-bug-id: Tendrl/gluster-integration#575
Signed-off-by: GowthamShanmugam gshanmug@redhat.com