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

Abstract storage backend #205

Merged
merged 3 commits into from
Oct 9, 2019
Merged

Abstract storage backend #205

merged 3 commits into from
Oct 9, 2019

Conversation

cannium
Copy link
Contributor

@cannium cannium commented Sep 27, 2019

Add an interface storage Cluster, for unit tests and new backends. Code is modified from part of #194

backend/backend.go Outdated Show resolved Hide resolved
@cannium cannium force-pushed the backend-interface branch 3 times, most recently from b1bb08d to f3d663b Compare September 30, 2019 08:14
ceph/cluster.go Outdated Show resolved Hide resolved
Copy link
Contributor

@chenji-kael chenji-kael left a comment

Choose a reason for hiding this comment

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

ok

@cxt90730
Copy link
Contributor

cxt90730 commented Oct 8, 2019

resolve conflicts first...

for unit tests and new backends
@cannium cannium force-pushed the backend-interface branch from a84cf67 to 2f443ce Compare October 9, 2019 01:53
@cannium cannium force-pushed the backend-interface branch 2 times, most recently from fe517b2 to 1fae61b Compare October 9, 2019 03:19
@cannium cannium force-pushed the backend-interface branch from 1fae61b to b4c0c80 Compare October 9, 2019 03:29
Copy link
Contributor

@cxt90730 cxt90730 left a comment

Choose a reason for hiding this comment

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

LG

@cxt90730 cxt90730 merged commit 2353cb8 into master Oct 9, 2019
@cannium cannium deleted the backend-interface branch October 9, 2019 04:00
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.

3 participants