-
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
Hpe 3par add host mapping view #798
Conversation
Codecov Report
@@ Coverage Diff @@
## master #798 +/- ##
==========================================
+ Coverage 70.57% 70.80% +0.23%
==========================================
Files 181 181
Lines 19324 19547 +223
Branches 2868 2921 +53
==========================================
+ Hits 13637 13840 +203
- Misses 4793 4795 +2
- Partials 894 912 +18
|
2f6b555
to
a28df8f
Compare
a173acf
to
e4b0270
Compare
3par add host mapping view
3par add host mapping view
3par add host mapping view
'native_port_group_id': native_port_group_id, | ||
"storage_id": storage_id | ||
} | ||
if 'set:' in vv_name: |
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.
We need to validate mandatory fields are set in masking view.
From file <>/delfin/drivers/driver.py
, function list_masking_views()
comments, mandatory fields:
From host side: Mandatorily one of the (native_storage_host_group_id | native_storage_host_id)
From volume side: Mandatorily one of the (native_volume_group_id | native_volume_id)
From port side: Optionally (native_port_group_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
host_name = view.get('hostname') | ||
if vv_name and host_name: | ||
port = view.get('port', '').replace('-', '') | ||
id = view.get('lun') |
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.
id
is a built-in name, please use another name instead.
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
modify test unit
Optimize disk status
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: