Comment 10 for bug 1367189

Hi Sergey

Thanks for your confirmation that the issue not just for IBM storwize.

I went through Nova and Cinder code. I think the code change maybe done for both Cinder, and Nova. Cinder does not report the "Part 1" properties to nova but just the information for login "Part 2". (Even usually discovery and login use the same value for user and password value,just different keys)
example:
=====Not report by Cinder but needed by Nova======= Part 1
discovery.sendtargets.auth.authmethod
discovery.sendtargets.auth.username
discovery.sendtargets.auth.password
=============================================

Compared to :

=====Cinder already report for login================Part 2
node.session.auth.authmethod
node.session.auth.username
node.session.auth.password
=============================================

For me, there are two options.
Option 1:
 Cinder report additional discovery properties(part 1) , Nova use these information to do discovery. {both nova and cinder need to change}
Option 2:
Cinder no need change, but nova use "login properties" (part2)to discovery. But there is a risk that, even usually they are the same, but could be different.{only nova need to be change}

Personally, I prefer option 1.