-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Export DirectPV client and admin API package #889
Conversation
4057edb
to
85fc6ed
Compare
55c8fb6
to
fd0d1cd
Compare
fd0d1cd
to
f22f958
Compare
f22f958
to
6b971a6
Compare
b7267b1
to
d28f773
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
9dd9690
to
9cf97b4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
9cf97b4
to
804f9a2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested the basic functions. LGTM 👍
b7e1866
to
ad4b2b8
Compare
Have made some changes to add appropriate return types for the admin functions. PTAL @balamurugana @prakashsvmx @dvaldivia |
ad4b2b8
to
b7190d0
Compare
PTAL @balamurugana |
b7190d0
to
72205f9
Compare
Signed-off-by: Bala.FA <bala@minio.io>
11ac73f
to
531f615
Compare
This will be used to integrate DirectPV externally and provides common usage of functions
For example,
and with the following line in go.mod
after merging, we can use the master's commit id in the go.mod
also checkout the examples in
pkg/admin/examples/