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

Fix multiple host objects/lun conflict issue for netapp_lun_mapping. #52848

Open
wants to merge 2 commits into
base: devel
from

Conversation

Projects
None yet
3 participants
@ndswartz
Copy link
Contributor

ndswartz commented Feb 22, 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.dev0 (lun_mapping_upstream_dev 04cb6453f4) last updated 2019/02/22 11:34:50 (GMT -500)
  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]
@ansibot

This comment has been minimized.

Swartz, Nathan Swartz, Nathan
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.

@ndswartz ndswartz force-pushed the ndswartz:lun_mapping_upstream_dev branch from 04cb645 to ecc836c Feb 22, 2019

@ndswartz

This comment has been minimized.

Copy link
Contributor Author

ndswartz commented Feb 22, 2019

Removed integration tests from the commit. These are currently unsupported and will be introduced at a later time.

@ansibot ansibot added the owner_pr label Feb 22, 2019

@ndswartz ndswartz force-pushed the ndswartz:lun_mapping_upstream_dev branch from 0c43543 to 12735c9 Feb 22, 2019

@ansibot ansibot added the stale_ci label Mar 3, 2019

@ansibot ansibot removed the needs_triage label Mar 10, 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.