openstack/cinder
Code Issues Proposed changes

[NetApp] Fix iSCSI CHAP auth issue during volume attach

Browse Source 
Fixes a failure in the NetApp ONTAP driver while trying to
configure CHAP authentication during a volume attach operation, by
making the code more flexible while trying to identify if the
prompted field is available in the buffered message.

Change-Id: I81dd9861b9464244f23967f4f83888d06b50754e
Closes-Bug: #1914639
This commit is contained in:
silvacarloss 2021-02-04 14:08:59 -03:00
parent 355681cd53
commit a5c80032f6
3 changed files with 13 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
cinder/tests/unit/volume/drivers/netapp/dataontap/client/test_api.py
View File

@ -508,9 +508,10 @@ class SSHUtilTests(test.TestCase):
self.sshutil.execute_command, ssh, 'ls') self.sshutil.execute_command, ssh, 'ls')
wait_on_stdout.assert_not_called() wait_on_stdout.assert_not_called()
@ddt.data('Password:', @ddt.data(b'Password:',
'Password: ', b'Password: ',
'Password: \n\n') b'Password: \n\n',
b'Fake response \r\n Password: \n\n')
def test_execute_command_with_prompt(self, response): def test_execute_command_with_prompt(self, response):
ssh = mock.Mock(paramiko.SSHClient) ssh = mock.Mock(paramiko.SSHClient)
stdin, stdout, stderr = self._mock_ssh_channel_files(paramiko.Channel) stdin, stdout, stderr = self._mock_ssh_channel_files(paramiko.Channel)
@ -533,7 +534,7 @@ class SSHUtilTests(test.TestCase):
ssh = mock.Mock(paramiko.SSHClient) ssh = mock.Mock(paramiko.SSHClient)
stdin, stdout, stderr = self._mock_ssh_channel_files(paramiko.Channel) stdin, stdout, stderr = self._mock_ssh_channel_files(paramiko.Channel)
stdout_read = self.mock_object(stdout.channel, 'recv', stdout_read = self.mock_object(stdout.channel, 'recv',
return_value='bad response') return_value=b'bad response')
self.mock_object(ssh, 'exec_command', self.mock_object(ssh, 'exec_command',
return_value=(stdin, stdout, stderr)) return_value=(stdin, stdout, stderr))

2
cinder/volume/drivers/netapp/dataontap/client/api.py
View File

@ -601,7 +601,7 @@ class SSHUtil(object):
stdin, stdout, stderr = client.exec_command(command) stdin, stdout, stderr = client.exec_command(command)
self._wait_on_stdout(stdout, timeout) self._wait_on_stdout(stdout, timeout)
response = stdout.channel.recv(999) response = stdout.channel.recv(999)
if response.strip() != expected_prompt_text: if expected_prompt_text not in response.strip().decode():
msg = _("Unexpected output. Expected [%(expected)s] but " msg = _("Unexpected output. Expected [%(expected)s] but "
"received [%(output)s]") % { "received [%(output)s]") % {
'expected': expected_prompt_text, 'expected': expected_prompt_text,

7
releasenotes/notes/bug-1914639-fix-chap-auth-issue-in-netapp-driver-e92eaa431d6fcbac.yaml Normal file
View File

@ -0,0 +1,7 @@
---
fixes:
- |
Fixed a CHAP authentication issue while trying to attach an iSCSI volume
using the NetApp ONTAP driver. Please refer to the
`Launchpad bug #1914639 <https://bugs.launchpad.net/cinder/+bug/1914639>`_
for more details.