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

Fortinet: FortiAnalyzer Plugin and Module Utils and FAZ_DEVICE update. #58882

Open

Conversation

Projects
None yet
4 participants
@ftntcorecse
Copy link
Contributor

commented Jul 9, 2019

SUMMARY

Pull Request to add a connection plugin, module_utils, and an update to existing FAZ_DEVICE module. This opens the door for other modules for FAZ.

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

plugins/httpapi/fortianalyzer.py
modules/network/fortianalyzer/faz_device.py
module_utils/network/fortianalyzer/*.py

ADDITIONAL INFORMATION

@ansibot

This comment has been minimized.

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jul 9, 2019

@ftntcorecse, just so you are aware we have a dedicated Working Group for network.
You can find other people interested in this in #ansible-network on Freenode IRC
For more information about communities, meetings and agendas see https://github.com/ansible/community

click here for bot help

@ansibot ansibot removed the ci_verified label Jul 11, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jul 11, 2019

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

lib/ansible/module_utils/network/fortianalyzer/common.py:34:1: E302 expected 2 blank lines, found 1

click here for bot help

@ansibot ansibot added the ci_verified label Jul 11, 2019

@ansibot ansibot removed the ci_verified label Jul 11, 2019

@ftntcorecse

This comment has been minimized.

Copy link
Contributor Author

commented Jul 12, 2019

Last shippable ended with some obscure errors about OS X that don't regard this code. Sending another minor documentation change push to get another one going.

@ansibot ansibot removed the needs_revision label Jul 12, 2019

@ansibot ansibot added the core_review label Jul 12, 2019

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