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

Eseries base class dev #57074

Open
wants to merge 3 commits into
base: devel
from

Conversation

Projects
None yet
4 participants
@ndswartz
Copy link
Contributor

commented May 28, 2019

SUMMARY

The base class was preventing the use of ports other than 8443 on SANtricity Web Services proxy.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

lib/ansible/module_utils/netapp.py

ADDITIONAL INFORMATION
ansible 2.8.0.dev0
  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 Apr 2, 2019

Add header option to the request method in the netapp eseries base class
Also add content-type check for application/json to the eseries base class request method
Refactor eseries base class to update only embedded api url.
Updated eseries base class and unit tests
@ansibot

This comment has been minimized.

@ndswartz ndswartz force-pushed the ndswartz:eseries_base_class_dev branch from 837c164 to 34b0c99 Jun 7, 2019

@ansibot ansibot added core_review and removed needs_revision labels Jun 7, 2019

@ansibot ansibot added the stale_ci label Jun 15, 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.