Skip to content
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

Backport/2.8/57096 #57098

Open
wants to merge 3 commits into
base: stable-2.8
from

Conversation

Projects
None yet
4 participants
@ndswartz
Copy link
Contributor

commented May 28, 2019

SUMMARY

This patch fixes an issue of when multiple hosts are created and then
subsequently volume(s) are mapped to them using the same specified number.

Also, multiple tests have been added to the module's integration test.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

netapp_e_lun_mapping

ADDITIONAL INFORMATION

Create multiple host objects and a volume on a NetApp E-Series storage system. Next map the volume to each host using the same LUN number using netapp_e_lun_mapping module.

ansible 2.8.0.post0
  config file = None
  configured module search path = [u'/home/swartzn/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /home/swartzn/ansible-dev/lib/ansible
  executable location = /home/swartzn/ansible-dev/bin/ansible
  python version = 2.7.15rc1 (default, Nov 12 2018, 14:31:15) [GCC 7.3.0]

netapp-eseries added some commits Feb 22, 2019

Fix hosts with same lun number conflict in netapp_e_lun_mapping.
This patch fixes an issue of when multiple hosts are created and then
subsequently volume(s) are mapped to them using the same specified number.
@ansibot

This comment has been minimized.

@abadger

This comment has been minimized.

Copy link
Member

commented May 29, 2019

#57096 has not yet been merged. This cannot be merged until then.

@ansibot ansibot removed the needs_triage label May 29, 2019

@ansibot ansibot added the stale_ci label Jun 6, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.