-
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:Modifying exception types #545
Emc vnx block:Modifying exception types #545
Conversation
Codecov Report
@@ Coverage Diff @@
## master #545 +/- ##
=======================================
Coverage 71.08% 71.08%
=======================================
Files 138 138
Lines 10629 10629
Branches 1191 1191
=======================================
Hits 7556 7556
Misses 2692 2692
Partials 381 381
|
@@ -24,8 +24,8 @@ | |||
EXCEPTION_MAP = {CER_ERR: exception.SSLCertificateFailed, | |||
CALLER_ERR: exception.InvalidUsernameOrPassword, | |||
SECURITY_ERR: exception.InvalidUsernameOrPassword, | |||
TRYING_CONNECT_ERR: exception.ConnectTimeout, | |||
CONNECTION_ERR: exception.ConnectTimeout, | |||
TRYING_CONNECT_ERR: exception.InvalidIpOrPort, |
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.
What is there is some network connectivity issue? It may not be invalid IP or PORT
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.
Page display requirements
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.
Ok hope the message is displayed accordingly
3e0da98
to
d1b4454
Compare
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:
Modifying exception types
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #Modifying exception types
Special notes for your reviewer:
Release note: