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

The restful handler of storageprovider #21

Merged
merged 8 commits into from
Jun 23, 2018
Merged

The restful handler of storageprovider #21

merged 8 commits into from
Jun 23, 2018

Conversation

hwchiu
Copy link
Contributor

@hwchiu hwchiu commented Jun 21, 2018

The storage-provider is used to setup the storage for system and it should setup all resources we need for kubernetes (storageClass, storage-provisioner).

  • In this patch, we only handle the mongo document.
  • We only support the NFS now and it's easy to add other storage in the field.
    • For NFS, we will setup the deployment and storageClass for nfs-client provisioner.
    • For GlusterFS, we only setup the storageClass for glusterFS provisioner

@codecov-io
Copy link

codecov-io commented Jun 21, 2018

Codecov Report

Merging #21 into develop will increase coverage by 2.89%.
The diff coverage is 90.32%.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop     #21      +/-   ##
==========================================
+ Coverage    57.91%   60.8%   +2.89%     
==========================================
  Files           12      13       +1     
  Lines          316     347      +31     
==========================================
+ Hits           183     211      +28     
- Misses         122     124       +2     
- Partials        11      12       +1
Impacted Files Coverage Δ
src/server/route.go 50% <83.33%> (+9.09%) ⬆️
src/server/handler_storage_provider.go 92% <92%> (ø)

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 949c6be...b4f7cc0. Read the comment docs.

@hwchiu hwchiu changed the title The restful handler of storageprovider [WIP] The restful handler of storageprovider Jun 21, 2018
)

func CreateStorageProvider(ctx *web.Context) {
as, req, resp := ctx.ServiceProvider, ctx.Request, ctx.Response
Copy link
Contributor

Choose a reason for hiding this comment

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

we can use sp sp, req, resp

@hwchiu hwchiu requested review from sufuf3 and WakeupTsai June 22, 2018 09:23
@hwchiu hwchiu changed the title [WIP] The restful handler of storageprovider The restful handler of storageprovider Jun 22, 2018
@hwchiu hwchiu merged commit e0f98bc into develop Jun 23, 2018
@hwchiu hwchiu deleted the hwchiu/VX-119 branch June 23, 2018 05:13
John-Lin pushed a commit that referenced this pull request Jul 25, 2018
The restful handler of storageprovider

Former-commit-id: ca4d5444f048d8044919a23620a0b57da7fa9b22 [formerly e0f98bc]
Former-commit-id: a7b69e7d4e8aff0b786d835c61c831b28377dfb5
John-Lin pushed a commit that referenced this pull request Jul 25, 2018
The restful handler of storageprovider

Former-commit-id: ca4d5444f048d8044919a23620a0b57da7fa9b22 [formerly ca4d5444f048d8044919a23620a0b57da7fa9b22 [formerly e0f98bc]]
Former-commit-id: a7b69e7d4e8aff0b786d835c61c831b28377dfb5
Former-commit-id: 4e3e4d7
John-Lin pushed a commit that referenced this pull request Jul 25, 2018
The restful handler of storageprovider
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.

4 participants