Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration of Development Branch to Master branch for Daito RC2 Release #1121

Merged
merged 45 commits into from
Dec 12, 2019

Conversation

skdwriting
Copy link
Collaborator

What this PR does / why we need it:
Integration of Development Branch to Master branch for Daito RC2 Release. Planned release tag is v0.7.0-Daito-RC2

Which issue this PR fixes
-hotpot api updates for basic data provisioning
-host management apis initial version integrated
-updated to openapi 3.0
-hotpot api additional parameters added
-swagger validation added

Release note:
Basic Testing of Hotpot has been completed for all APIs (including host APIs)

wisererik and others added 30 commits November 19, 2019 11:37
Update attachment API based on host management
Fileshare name supports upto 255 characters name length
Add host api validation and filtering
* Add Filters for DurableName for Volume Get API.
* Add suport for ESXI osType for Host create API swagger.
Update GetVolume Filters and Add ESXI osType support for HOST API
* Fix profile api in swagger

* Fix some typo based on comment
Integrate all the "select" function to a single one
Copy link
Collaborator

@kumarashit kumarashit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link

codecov bot commented Dec 12, 2019

Codecov Report

Merging #1121 into master will increase coverage by 0.16%.
The diff coverage is 37.64%.

@@            Coverage Diff             @@
##           master    #1121      +/-   ##
==========================================
+ Coverage   34.73%   34.89%   +0.16%     
==========================================
  Files          95       97       +2     
  Lines       17778    17576     -202     
==========================================
- Hits         6175     6133      -42     
+ Misses      10726    10574     -152     
+ Partials      877      869       -8
Impacted Files Coverage Δ
pkg/dock/discovery/discovery.go 34.07% <ø> (ø) ⬆️
pkg/dock/dock.go 13.01% <ø> (+0.38%) ⬆️
pkg/api/controllers/volume.go 25.8% <ø> (+0.27%) ⬆️
contrib/drivers/huawei/oceanstor/oceanstor.go 0% <0%> (ø) ⬆️
contrib/drivers/lvm/lvm.go 24.21% <0%> (ø) ⬆️
.../drivers/filesharedrivers/oceanstor/oceanclient.go 0% <0%> (ø) ⬆️
...trib/drivers/huawei/fusionstorage/fusionstorage.go 0% <0%> (ø) ⬆️
client/receiver.go 0% <0%> (ø) ⬆️
...ib/drivers/filesharedrivers/oceanstor/oceanstor.go 0% <0%> (ø) ⬆️
contrib/drivers/ceph/ceph.go 0% <0%> (ø) ⬆️
... and 16 more

Copy link
Collaborator

@wisererik wisererik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@skdwriting skdwriting merged commit f3c46cc into master Dec 12, 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.