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

Ecs cluster facts #50438

Open
wants to merge 3 commits into
base: devel
from

Conversation

Projects
None yet
2 participants
@wimnat
Copy link
Contributor

wimnat commented Jan 2, 2019

SUMMARY

New module for ECS cluster facts

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

ecs_cluster_facts

ADDITIONAL INFORMATION
@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Jan 2, 2019

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

lib/ansible/modules/cloud/amazon/ecs_cluster_facts.py:180:0: trailing-whitespace Trailing whitespace

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

lib/ansible/modules/cloud/amazon/ecs_cluster_facts.py:69:38: W291 trailing whitespace
lib/ansible/modules/cloud/amazon/ecs_cluster_facts.py:180:1: W293 blank line contains whitespace
lib/ansible/modules/cloud/amazon/ecs_cluster_facts.py:196: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/amazon/ecs_cluster_facts.py:0:0: E319 RETURN.cluster_arn.type: not a valid value for dictionary value @ data['cluster_arn']['type']. Got 'string'
lib/ansible/modules/cloud/amazon/ecs_cluster_facts.py:0:0: E319 RETURN.cluster_name.type: not a valid value for dictionary value @ data['cluster_name']['type']. Got 'string'
lib/ansible/modules/cloud/amazon/ecs_cluster_facts.py:0:0: E319 RETURN.instances.type: not a valid value for dictionary value @ data['instances']['type']. Got 'string'
lib/ansible/modules/cloud/amazon/ecs_cluster_facts.py:0:0: E319 RETURN.status.type: not a valid value for dictionary value @ data['status']['type']. Got 'string'

click here for bot help

@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Jan 2, 2019

@wimnat, just so you are aware we have a dedicated Working Group for aws.
You can find other people interested in this in #ansible-aws 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 Jan 3, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Jan 3, 2019

@Constantin007 @Constantin07 @Deepakkothandan @Etherdaemon @Java1Guy @Madhura-CSI @MichaelBaydoun @Sodki @Zeekin @adq @akazakov @alachaum @amir343 @anryko @bekelchik @bpennypacker @brandond @captainkerk @chenl87 @defunctio @dennisconrad @dkhenry @fiunchinho @fivethreeo @flowerysong @garethr @gobins @gunzy83 @gurumaia @hsingh @hyperized @iiibrad @infectsoldier @j-carl @jarv @Java1Guy @jimbydamonk @jmenga @joelthompson @jonhadfield @jonmer85 @joshsouza @jsdalton @jsmartin @kaczynskid @leedm777 @linuxdynasty @loia @lwade @michaeljs1990 @minichate @mjschultz @mmochan @nand0p @naslanidis @nathanwebsterdotme @nerzhul @nickball @orthanc @ozbillwang @piontas @pjodouin @prasadkatti @psykotox @ptux @pwnall @raags @rickmendes @roadmapper @ryansb @ryansydnor @scicoin-project @scottanderson42 @sdubrul @shepdelacreme @silviud @slapula @steynovich @tastychutney @tedder @tgerla @timmahoney @tombamford @tomislacker @tsiganenok @viper233 @whiter @willricardo @wilvk @yaakov-github @zacblazic @zbal @zimbatm

As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add shipit if you would like to see it merged.

click here for bot help

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