Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@leonwanghui
Copy link
Collaborator

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Release note:

ysxeric and others added 3 commits June 19, 2019 17:25
…t pool (#911)

* nvmeof e2ef.test problems fix

* test should select proper pool for speciifed test
* Modify the description of the cli command help

* Flag is p instead of P
@leonwanghui leonwanghui changed the title Development Merge development into master branch Jun 20, 2019
@leonwanghui leonwanghui self-assigned this Jun 20, 2019
@codecov
Copy link

codecov bot commented Jun 20, 2019

Codecov Report

Merging #918 into master will decrease coverage by <.01%.
The diff coverage is 12.5%.

@@            Coverage Diff             @@
##           master     #918      +/-   ##
==========================================
- Coverage   31.25%   31.25%   -0.01%     
==========================================
  Files          82       82              
  Lines       15189    15191       +2     
==========================================
  Hits         4748     4748              
- Misses       9738     9739       +1     
- Partials      703      704       +1
Impacted Files Coverage Δ
pkg/controller/controller.go 32.39% <0%> (ø) ⬆️
pkg/db/drivers/etcd/etcd.go 27.98% <0%> (-0.03%) ⬇️
osdsctl/cli/filesharesnapshot.go 84.33% <100%> (ø) ⬆️

@leonwanghui leonwanghui merged commit c081859 into master Jun 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants