-
Notifications
You must be signed in to change notification settings - Fork 354
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
add pure #791
add pure #791
Conversation
# Conflicts: # delfin/drivers/pure/storage/rest_handler.py # delfin/drivers/pure/storage/storage.py # delfin/tests/unit/drivers/pure/storage/test_storage.py
pure 初步功能
pure codeview 代码简洁优化
Suggestions for improving Huawei: Simplify and optimize all function names and codes of Pure
pure codeview-recompose test_reset_connection
pure codeview-Delete unnecessary Definitions
pure codeview-Modified the function name
pure codeview-Add files and change class names
pure codeview-Remove unnecessary function wrapping
Pure Modify alert and storage
Pure Modifying field Values
pure amend-remove disk id and controllers id
Codecov Report
@@ Coverage Diff @@
## master #791 +/- ##
==========================================
+ Coverage 70.51% 70.58% +0.06%
==========================================
Files 177 180 +3
Lines 18279 18641 +362
Branches 2695 2749 +54
==========================================
+ Hits 12889 13157 +268
- Misses 4572 4643 +71
- Partials 818 841 +23
|
@@ -0,0 +1,351 @@ | |||
import datetime |
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.
Please add file banner for Copyright in all new files
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.
have already added
copyright information
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
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
What this PR does / why we need it:
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #Special notes for your reviewer:
Release note: