-
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
Merged
Merged
Changes from 30 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
e4b0270
3par add host mapping view
yuanyu-ghca e1494a8
3par add host mapping view
yuanyu-ghca 431936e
Merge pull request #429 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca e50231f
Merge branch 'master' into hpe_3par_hostmap
yuanyu-ghca 1af34bb
3par add host mapping view
yuanyu-ghca 4b5b8c9
Merge pull request #430 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca a2cff2c
3par add host mapping view
yuanyu-ghca 097fd82
Merge pull request #432 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca 12f6c85
Merge branch 'master' into hpe_3par_hostmap
yuanyu-ghca 48dd99a
Handling view return issues
yuanyu-ghca 868cd2b
Merge pull request #446 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca 727cf34
modify bug
yuanyu-ghca b739071
Merge pull request #449 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca b869067
modify bug
yuanyu-ghca 3f0f900
Merge pull request #451 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca 30ce97e
modify bug
yuanyu-ghca 76a4479
Merge pull request #453 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca 84cafe6
Merge branch 'master' into hpe_3par_hostmap
yuanyu-ghca 900c98d
modify bug
yuanyu-ghca 7d94230
Merge pull request #482 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca e6390af
Merge branch 'master' into hpe_3par_hostmap
yuanyu-ghca cd23dff
modify test unit
yuanyu-ghca 609c793
Merge pull request #484 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca adbf00e
modify bug
yuanyu-ghca f90629f
Merge pull request #487 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca 2a2fe7c
modify bug
yuanyu-ghca ea00d54
Merge pull request #488 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca d9bad9c
Merge branch 'master' into hpe_3par_hostmap
yuanyu-ghca de2010c
Optimize disk status
yuanyu-ghca d1c6554
Merge pull request #512 from yuanyu-ghca/hpe_3par_hostmap
yuanyu-ghca dfe1d35
Merge branch 'master' into hpe_3par_hostmap
tanjiangyu-ghca File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
, functionlist_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