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

Ansible Tower user and credential module #21020

Merged
merged 8 commits into from Feb 15, 2017
Merged

Ansible Tower user and credential module #21020

merged 8 commits into from Feb 15, 2017

Conversation

wwitzel3
Copy link
Contributor

@wwitzel3 wwitzel3 commented Feb 3, 2017

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

Ansible Tower user module
Ansible Tower credential module

ANSIBLE VERSION
ansible 2.3.0 (tower_module 57b3783869) last updated 2017/02/03 13:47:23 (GMT -400)
  config file =
  configured module search path = Default w/o overrides
SUMMARY

Allows playbook authors to create Ansible Tower users and credentials. There will be a followup to this PR to add more functionality.

Initial module was completed and merged in #20355.

@ansibot ansibot added affects_2.3 This issue/PR affects Ansible v2.3 c:module_utils/ community_review In order to be merged, this PR must follow the community review workflow. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. new_module This PR includes a new module. new_plugin This PR includes a new plugin. labels Feb 3, 2017
@alikins alikins removed the needs_triage Needs a first human triage before being processed. label Feb 6, 2017


EXAMPLES = '''
tower_credential:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a - name: and left align the examples

- name: Add tower credentials
  tower_credential:
    name: Team Name



EXAMPLES = '''
tower_user:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update example

tower_host = dict(),
tower_username = dict(),
tower_password = dict(no_log=True),
tower_verify_ssl = dict(type='bool', default='yes'),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default=True (as it's a bool, not a string)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default=True (as it's a bool, not a string)

- Dis/allow insecure connections to Tower. If C(no), SSL certificates will not be validated.
This should only be used on personally controlled sites using self-signed certificates.
required: False
default: 'yes'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default: True

tower_host = dict(),
tower_username = dict(),
tower_password = dict(no_log=True),
tower_verify_ssl = dict(type='bool', default='yes'),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default=True (as it's a bool, not a string)

- Dis/allow insecure connections to Tower. If C(no), SSL certificates will not be validated.
This should only be used on personally controlled sites using self-signed certificates.
required: False
default: 'yes'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default: True

tower_host = dict(),
tower_username = dict(),
tower_password = dict(no_log=True),
tower_verify_ssl = dict(type='bool', default='yes'),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default=True (as it's a bool, not a string)

@ansibot
Copy link
Contributor

ansibot commented Feb 15, 2017

@ghjm @jlaska @matburt

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

@matburt
Copy link
Member

matburt commented Feb 15, 2017

shipit

1 similar comment
@ghjm
Copy link
Contributor

ghjm commented Feb 15, 2017

shipit

@ansibot ansibot added shipit This PR is ready to be merged by Core and removed community_review In order to be merged, this PR must follow the community review workflow. labels Feb 15, 2017
@gundalow gundalow merged commit f98d41c into ansible:devel Feb 15, 2017
@dagwieers dagwieers added the tower Tower community label Feb 8, 2019
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.3 This issue/PR affects Ansible v2.3 c:module_utils/ module This issue/PR relates to a module. new_module This PR includes a new module. new_plugin This PR includes a new plugin. shipit This PR is ready to be merged by Core tower Tower community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants