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

New module - iam_role #19486

Merged
merged 2 commits into from
Dec 22, 2016
Merged

Conversation

wimnat
Copy link
Contributor

@wimnat wimnat commented Dec 18, 2016

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

iam_role

ANSIBLE VERSION

2.2

SUMMARY

Module to allow managing AWS IAM roles. Similar functionality to current iam module but it was decided that the module should be split in to separate ones i.e. iam_role, iam_user, iam_group.

This module is the first of that split.

Extra functionality includes support for managed policies

@wimnat
Copy link
Contributor Author

wimnat commented Dec 18, 2016

Migrated from ansible/ansible-modules-extras#2201

@ansibot ansibot added affects_2.3 This issue/PR affects Ansible v2.3 aws cloud module This issue/PR relates to a module. new_plugin This PR includes a new plugin. plugin labels Dec 18, 2016
Copy link
Contributor

@gundalow gundalow left a comment

Choose a reason for hiding this comment

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

needs_revision

short_description: Manage AWS IAM roles
description:
- Manage AWS IAM roles
version_added: "2.2"
Copy link
Contributor

Choose a reason for hiding this comment

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

2.3

else:
destroy_role(connection, module)

from ansible.module_utils.basic import *
Copy link
Contributor

Choose a reason for hiding this comment

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

Please import only what you need, rather than *.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is this a new requirement? I've always written my modules like this. Is there doc for likely necessary includes?

Copy link
Contributor

Choose a reason for hiding this comment

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

@ansibot ansibot added needs_triage Needs a first human triage before being processed. and removed triage labels Dec 19, 2016
@gundalow gundalow removed the needs_triage Needs a first human triage before being processed. label Dec 19, 2016
@gundalow
Copy link
Contributor

Given the reviews in ansible/ansible-modules-extras#2201 once my comments have been addressed I think we can merge this

@ansibot ansibot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. test_pull_requests labels Dec 19, 2016
@mattclay
Copy link
Member

@wimnat The file test/integration/targets/win_file/files/foobar/directory fileD is unrelated to this PR and appears to have been added accidentally.

@wimnat wimnat force-pushed the ansible-modules-extras/pull/2201 branch from 5b78095 to 4ad781d Compare December 21, 2016 18:36
@wimnat
Copy link
Contributor Author

wimnat commented Dec 21, 2016

@gundalow all done - just the specific import now
@mattclay - file removed

@ansibot ansibot removed the needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. label Dec 21, 2016
@gundalow gundalow merged commit 69c14bd into ansible:devel Dec 22, 2016
@wimnat wimnat deleted the ansible-modules-extras/pull/2201 branch December 9, 2017 04:34
@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 aws cloud module This issue/PR relates to a module. new_plugin This PR includes a new plugin.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants