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

[WIP] Adding postgresql configuration module #45072

Open
wants to merge 5 commits into
base: devel
from

Conversation

Projects
None yet
4 participants
@zikalino
Contributor

zikalino commented Sep 2, 2018

SUMMARY

Adding PostgreSQL configuration module

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

azure_rm_postgresqlconfiguration

ANSIBLE VERSION

2.7

ADDITIONAL INFORMATION
@ansibot

This comment has been minimized.

Contributor

ansibot commented Sep 2, 2018

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

test/integration/targets/azure_rm_postgresqlconfiguration/aliases:0:0: missing alias `shippable/azure/group[1-8]` or `unsupported`

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

lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:270:1: E305 expected 2 blank lines after class or function definition, found 1

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

lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E307 version_added should be 2.8. Currently 2.5
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E322 "state" is listed in the argument_spec, but not documented in the module
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E324 Value for "default" from the argument_spec ('present') for "state" does not match the documentation (None)
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E326 Value for "choices" from the argument_spec (['present', 'absent']) for "state" does not match the documentation ([])

click here for bot help

@ansibot

This comment has been minimized.

@ansibot

This comment has been minimized.

Contributor

ansibot commented Sep 3, 2018

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

test/integration/targets/azure_rm_postgresqlconfiguration/aliases:0:0: missing alias `shippable/azure/group[1-8]` or `unsupported`

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

lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:270:1: E305 expected 2 blank lines after class or function definition, found 1

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

lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E307 version_added should be 2.8. Currently 2.5
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E322 "state" is listed in the argument_spec, but not documented in the module
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E324 Value for "default" from the argument_spec ('present') for "state" does not match the documentation (None)
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E326 Value for "choices" from the argument_spec (['present', 'absent']) for "state" does not match the documentation ([])

click here for bot help

@bcoca bcoca removed the needs_triage label Sep 5, 2018

@ansibot ansibot added the stale_ci label Sep 13, 2018

@zikalino zikalino force-pushed the VSChina:postgresqlconfiguration branch from f4e38a1 to eeddeb3 Oct 17, 2018

@zikalino zikalino force-pushed the VSChina:postgresqlconfiguration branch from da65275 to 6ff05af Oct 17, 2018

@zikalino zikalino force-pushed the VSChina:postgresqlconfiguration branch from 6ff05af to e0c91ab Oct 17, 2018

@ansibot ansibot removed the needs_ci label Oct 17, 2018

@ansibot ansibot removed the stale_ci label Oct 17, 2018

@ansibot

This comment has been minimized.

Contributor

ansibot commented Oct 17, 2018

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

lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E322 "parameters" is listed in the argument_spec, but not documented in the module
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E322 "state" is listed in the argument_spec, but not documented in the module
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E324 Value for "default" from the argument_spec ('present') for "state" does not match the documentation (None)
lib/ansible/modules/cloud/azure/azure_rm_postgresqlconfiguration.py:0:0: E326 Value for "choices" from the argument_spec (['present', 'absent']) for "state" does not match the documentation ([])

click here for bot help

@ansibot ansibot added the stale_ci label Oct 25, 2018

@Fred-sun

This comment has been minimized.

Contributor

Fred-sun commented Dec 4, 2018

kindly ping

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