-
Notifications
You must be signed in to change notification settings - Fork 355
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
Emc vnx block add host mapping view #807
Emc vnx block add host mapping view #807
Conversation
Add host maping view
Add host maping view
Codecov Report
@@ Coverage Diff @@
## master #807 +/- ##
==========================================
+ Coverage 70.47% 70.50% +0.02%
==========================================
Files 181 181
Lines 19055 19232 +177
Branches 2812 2854 +42
==========================================
+ Hits 13429 13559 +130
- Misses 4756 4782 +26
- Partials 870 891 +21
|
view_model.get('native_masking_view_id'), | ||
host_id, lun_id) | ||
views_list.append(view_model) | ||
elif lun_ids: |
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.
Masking view without native_storage_host_group_id
or native_storage_host_id
is invalid. Please check comment section in function list_masking_views()
in file <>/delfin/drivers/driver.py
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.
Modified
view_model.get('native_masking_view_id'), | ||
lun_id) | ||
views_list.append(view_model) | ||
elif host_names: |
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.
Same as above, masking view require native_volume_group_id
or native_volume_id
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.
Modified
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
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:
1 add host mapping view
Which issue this PR fixes (optional, in fixes #(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note: