-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Provide self-registering storage system #12978
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
Provide self-registering storage system #12978
Conversation
Signed-off-by: Andrew Thornton <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #12978 +/- ##
==========================================
- Coverage 42.62% 42.61% -0.01%
==========================================
Files 672 673 +1
Lines 73785 73797 +12
==========================================
+ Hits 31448 31452 +4
+ Misses 37247 37244 -3
- Partials 5090 5101 +11
Continue to review full report at Codecov.
|
Signed-off-by: Andrew Thornton <[email protected]>
Signed-off-by: Andrew Thornton <[email protected]>
6d908b3
to
7f2f6fe
Compare
Please resolve conflicts ;) |
conflicts resolved |
@lunny please review |
Signed-off-by: Andrew Thornton <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #12978 +/- ##
==========================================
+ Coverage 42.25% 42.35% +0.10%
==========================================
Files 675 676 +1
Lines 74520 74532 +12
==========================================
+ Hits 31485 31569 +84
+ Misses 37922 37826 -96
- Partials 5113 5137 +24
Continue to review full report at Codecov.
|
failed unrelated |
This PR represents a continuation of some work on #12813.
It provides for a self-registering storage solution and means that we can (mostly) avoid changing code outside of modules/storage if we add a storage system.
I think there is an extension to this to do with the default mapping and further improvements to do with default settings but this gives the first steps for this.
Signed-off-by: Andrew Thornton [email protected]