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

Update playbook to add users to group #1736

Closed
wants to merge 1 commit into from

Conversation

theputta
Copy link
Contributor

@theputta theputta commented Sep 5, 2023

SUMMARY

Added an example to add an IAM user to group

ISSUE TYPE
  • Docs Pull Request
COMPONENT NAME

amazon.aws.iam_user

ADDITIONAL INFORMATION
  • Included my name in author list, please let me know if its okay

@github-actions
Copy link

github-actions bot commented Sep 5, 2023

Docs Build 📝

Thank you for contribution!✨

The docsite for this PR is available for download as an artifact from this run:
https://github.com/ansible-collections/amazon.aws/actions/runs/6090788302

You can compare to the docs for the main branch here:
https://ansible-collections.github.io/amazon.aws/branch/main

File changes:

  • M collections/amazon/aws/iam_user_module.html
Click to see the diff comparison.

NOTE: only file modifications are shown here. New and deleted files are excluded.
See the file list and check the published docs to see those files.

diff --git a/home/runner/work/amazon.aws/amazon.aws/docsbuild/base/collections/amazon/aws/iam_user_module.html b/home/runner/work/amazon.aws/amazon.aws/docsbuild/head/collections/amazon/aws/iam_user_module.html
index bda00d7..b8eb9e1 100644
--- a/home/runner/work/amazon.aws/amazon.aws/docsbuild/base/collections/amazon/aws/iam_user_module.html
+++ b/home/runner/work/amazon.aws/amazon.aws/docsbuild/head/collections/amazon/aws/iam_user_module.html
@@ -511,6 +511,13 @@ see <a class="reference internal" href="#ansible-collections-amazon-aws-iam-user
 <span class="w">    </span><span class="nt">tags</span><span class="p">:</span>
 <span class="w">      </span><span class="nt">Env</span><span class="p">:</span><span class="w"> </span><span class="l l-Scalar l-Scalar-Plain">Prod</span>
 
+<span class="p p-Indicator">-</span><span class="w"> </span><span class="nt">name</span><span class="p">:</span><span class="w"> </span><span class="l l-Scalar l-Scalar-Plain">Add IAM User to Group</span>
+<span class="w">  </span><span class="nt">amazon.aws.iam_user</span><span class="p">:</span>
+<span class="w">    </span><span class="nt">name</span><span class="p">:</span><span class="w"> </span><span class="l l-Scalar l-Scalar-Plain">user-1</span>
+<span class="w">    </span><span class="nt">groups</span><span class="p">:</span>
+<span class="w">      </span><span class="p p-Indicator">-</span><span class="w"> </span><span class="l l-Scalar l-Scalar-Plain">group-1</span>
+
+
 <span class="p p-Indicator">-</span><span class="w"> </span><span class="nt">name</span><span class="p">:</span><span class="w"> </span><span class="l l-Scalar l-Scalar-Plain">Delete the user</span>
 <span class="w">  </span><span class="nt">amazon.aws.iam_user</span><span class="p">:</span>
 <span class="w">    </span><span class="nt">name</span><span class="p">:</span><span class="w"> </span><span class="l l-Scalar l-Scalar-Plain">testuser1</span>

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/fdf787c21e744aa8b6304c1a06c41df7

✔️ ansible-galaxy-importer SUCCESS in 4m 50s
✔️ build-ansible-collection SUCCESS in 12m 19s
✔️ ansible-test-splitter SUCCESS in 4m 43s
✔️ integration-amazon.aws-1 SUCCESS in 8m 43s
✔️ integration-community.aws-1 SUCCESS in 10m 56s
Skipped 42 jobs

Copy link
Contributor

@tremble tremble left a comment

Choose a reason for hiding this comment

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

This module doesn't manage groups. The example isn't valid.

abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ions#1767)

elb_target_group: fix lost property AvailabilityZone

SUMMARY
Closes ansible-collections#1736
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME
elb_target_group

Reviewed-by: Mark Chappell
Reviewed-by: Alina Buzachis
Reviewed-by: Markus Bergholz <git@osuv.de>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ions#1767)

elb_target_group: fix lost property AvailabilityZone

SUMMARY
Closes ansible-collections#1736
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME
elb_target_group

Reviewed-by: Mark Chappell
Reviewed-by: Alina Buzachis
Reviewed-by: Markus Bergholz <git@osuv.de>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…ions#1767)

elb_target_group: fix lost property AvailabilityZone

SUMMARY
Closes ansible-collections#1736
ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME
elb_target_group

Reviewed-by: Mark Chappell
Reviewed-by: Alina Buzachis
Reviewed-by: Markus Bergholz <git@osuv.de>
@tremble tremble added the docs label Nov 30, 2023
@GomathiselviS
Copy link
Collaborator

Closing this PR as the example is not valid

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants