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

New module to manage IKE crypto profiles on Palo Alto Firewalls #45615

Open
wants to merge 1 commit into
base: devel
from

Conversation

Projects
None yet
3 participants
@ivanbojer
Contributor

ivanbojer commented Sep 13, 2018

SUMMARY

Configures IKE Crypto profile on the firewall with subset of settings

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

panos_ike_crypto_profile

ANSIBLE VERSION
ansible 2.4.2.0
  config file = None
  configured module search path = [u'/Users/ibojer/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /Users/ibojer/.virtualenvs/ansible/lib/python2.7/site-packages/ansible
  executable location = /Users/ibojer/.virtualenvs/ansible/bin/ansible
  python version = 2.7.13 (default, Feb  6 2017, 11:22:10) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-800.0.42.1)]
ADDITIONAL INFORMATION

N/A

@ansibot

This comment has been minimized.

Contributor

ansibot commented Sep 13, 2018

@ansibot

This comment has been minimized.

Contributor

ansibot commented Sep 13, 2018

The test ansible-test sanity --test no-get-exception [explain] failed with 2 errors:

lib/ansible/modules/network/panos/panos_ike_crypto_profile.py:113:40: do not use `get_exception`
lib/ansible/modules/network/panos/panos_ike_crypto_profile.py:224:15: do not use `get_exception`

The test ansible-test sanity --test shebang [explain] failed with 1 error:

lib/ansible/modules/network/panos/panos_ike_crypto_profile.py:1:1: expected module shebang "b'#!/usr/bin/python'" but found: b'#!/usr/bin/env python'

The test ansible-test sanity --test validate-modules [explain] failed with 3 errors:

lib/ansible/modules/network/panos/panos_ike_crypto_profile.py:0:0: E101 Interpreter line is not "#!/usr/bin/python"
lib/ansible/modules/network/panos/panos_ike_crypto_profile.py:0:0: E325 argument_spec for "commit" defines type="bool" but documentation does not
lib/ansible/modules/network/panos/panos_ike_crypto_profile.py:95:29: E311 EXAMPLES is not valid YAML

The test ansible-test sanity --test yamllint [explain] failed with 1 error:

lib/ansible/modules/network/panos/panos_ike_crypto_profile.py:95:29: error EXAMPLES: syntax error: mapping values are not allowed here

click here for bot help

@ansibot

This comment has been minimized.

Contributor

ansibot commented Sep 13, 2018

@ansibot ansibot added the stale_ci label Sep 21, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment