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

eks_cluster: adding tags to eks cluster #1591

Merged
merged 5 commits into from Feb 3, 2023

Conversation

mohgho
Copy link
Contributor

@mohgho mohgho commented Nov 11, 2022

SUMMARY

As described by issue #1513
All the other community aws modules that I use support applying tags to the created resources (where such a feature is supported by aws). Using aws_eks_cluster it is not possible to add tags, although this appears to be a supported feature when viewing a cluster in the aws console.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

aws_eks_cluster

ADDITIONAL INFORMATION

@github-actions
Copy link

github-actions bot commented Nov 11, 2022

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) labels Nov 11, 2022
@goneri goneri closed this Nov 11, 2022
@goneri goneri reopened this Nov 11, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

✔️ ansible-galaxy-importer SUCCESS in 4m 08s
✔️ build-ansible-collection SUCCESS in 5m 28s
ansible-test-sanity-docker-devel FAILURE in 9m 44s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 8m 08s (non-voting)
ansible-test-sanity-docker-stable-2.12 FAILURE in 11m 22s
ansible-test-sanity-docker-stable-2.13 FAILURE in 8m 36s
ansible-test-sanity-docker-stable-2.14 FAILURE in 9m 24s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 25s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 37s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 09s
ansible-test-changelog FAILURE in 2m 23s
✔️ ansible-test-splitter SUCCESS in 2m 27s
integration-community.aws-1 FAILURE in 4m 41s
integration-community.aws-2 FAILURE in 5m 00s
integration-community.aws-3 FAILURE in 6m 54s
integration-community.aws-4 FAILURE in 6m 34s
integration-community.aws-5 FAILURE in 4m 42s
integration-community.aws-6 FAILURE in 5m 09s
integration-community.aws-7 FAILURE in 5m 10s
integration-community.aws-8 FAILURE in 5m 04s
integration-community.aws-9 FAILURE in 4m 56s
integration-community.aws-10 FAILURE in 4m 57s
integration-community.aws-11 FAILURE in 5m 04s
integration-community.aws-12 FAILURE in 5m 02s
integration-community.aws-13 FAILURE in 5m 02s
integration-community.aws-14 FAILURE in 4m 44s
integration-community.aws-15 FAILURE in 5m 06s
integration-community.aws-16 FAILURE in 4m 56s
integration-community.aws-17 FAILURE in 5m 02s
integration-community.aws-18 FAILURE in 8m 16s
integration-community.aws-19 FAILURE in 8m 58s
integration-community.aws-20 FAILURE in 8m 22s
integration-community.aws-21 FAILURE in 4m 55s
integration-community.aws-22 FAILURE in 5m 09s

@mohgho
Copy link
Contributor Author

mohgho commented Nov 30, 2022

@goneri please have look

@goneri
Copy link
Member

goneri commented Nov 30, 2022

Hi @mohgho,

Thank you for the Pull Request. Can you modify this task to also set a tag? https://github.com/ansible-collections/community.aws/blob/main/tests/integration/targets/eks_cluster/tasks/full_test.yml#L77-L83

Also, we will need a new changelog fragment in https://github.com/ansible-collections/community.aws/tree/main/changelogs/fragments to describe the bug fix. This way, your change will be included in the next release changelog.
See: https://docs.ansible.com/ansible/latest/dev_guide/developing_collections_changelogs.html.

plugins/modules/eks_cluster.py Outdated Show resolved Hide resolved
@ansibullbot ansibullbot added integration tests/integration tests tests labels Dec 1, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 44s
✔️ build-ansible-collection SUCCESS in 5m 42s
ansible-test-sanity-docker-devel FAILURE in 7m 26s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 9m 57s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 38s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 12s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 49s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 9m 30s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 26s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 54s
✔️ ansible-test-changelog SUCCESS in 2m 27s
✔️ ansible-test-splitter SUCCESS in 2m 48s
✔️ integration-community.aws-1 SUCCESS in 5m 21s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@mohgho
Copy link
Contributor Author

mohgho commented Dec 1, 2022

Hi @goneri
did the changes.

@mohgho mohgho requested a review from goneri December 1, 2022 12:26
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 48s
✔️ build-ansible-collection SUCCESS in 5m 50s
ansible-test-sanity-docker-devel FAILURE in 7m 51s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 9m 22s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 23s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 04s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 23s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 6m 16s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 11s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 51s
✔️ ansible-test-changelog SUCCESS in 2m 33s
✔️ ansible-test-splitter SUCCESS in 3m 45s
✔️ integration-community.aws-1 SUCCESS in 7m 06s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

Next version is actually 5.2.0, my bad.
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 20s
✔️ build-ansible-collection SUCCESS in 5m 11s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 05s (non-voting)
ansible-test-sanity-docker-milestone FAILURE in 12m 32s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 12m 32s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 50s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 43s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 8m 18s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 02s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 7m 01s
✔️ ansible-test-changelog SUCCESS in 2m 37s
✔️ ansible-test-splitter SUCCESS in 3m 01s
✔️ integration-community.aws-1 SUCCESS in 5m 57s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@tremble
Copy link
Contributor

tremble commented Feb 3, 2023

recheck

changelogs/fragments/1591-eks-add-tags-cluster.yml Outdated Show resolved Hide resolved
plugins/modules/eks_cluster.py Outdated Show resolved Hide resolved
@tremble tremble added the backport-5 PR should be backported to the stable-5 branch label Feb 3, 2023
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 44s
✔️ build-ansible-collection SUCCESS in 5m 40s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 30s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 25s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 45s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 28s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 9m 01s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 5m 46s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 30s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 5m 26s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 9m 08s
✔️ ansible-test-changelog SUCCESS in 2m 22s
✔️ ansible-test-splitter SUCCESS in 2m 25s
✔️ integration-community.aws-1 SUCCESS in 5m 21s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Feb 3, 2023
@tremble
Copy link
Contributor

tremble commented Feb 3, 2023

Hi @mohgho,

Many thanks for taking the time to open this PR. Sorry it took a while to get this merged.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 5m 05s
✔️ build-ansible-collection SUCCESS in 5m 44s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 07s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 04s (non-voting)
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 54s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 10s
✔️ ansible-test-sanity-docker-stable-2.14 SUCCESS in 10m 17s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 7m 11s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 14s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 6m 49s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 5m 56s
✔️ ansible-test-changelog SUCCESS in 2m 30s
✔️ ansible-test-splitter SUCCESS in 2m 56s
✔️ integration-community.aws-1 SUCCESS in 6m 19s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED
⚠️ integration-community.aws-14 SKIPPED
⚠️ integration-community.aws-15 SKIPPED
⚠️ integration-community.aws-16 SKIPPED
⚠️ integration-community.aws-17 SKIPPED
⚠️ integration-community.aws-18 SKIPPED
⚠️ integration-community.aws-19 SKIPPED
⚠️ integration-community.aws-20 SKIPPED
⚠️ integration-community.aws-21 SKIPPED
⚠️ integration-community.aws-22 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 3c1c903 into ansible-collections:main Feb 3, 2023
@patchback
Copy link

patchback bot commented Feb 3, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/3c1c903ac01c4a1e1895c73ab173fa6cc11a542d/pr-1591

Backported as #1699

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Feb 3, 2023
eks_cluster: adding tags to eks cluster

SUMMARY
As described by issue #1513
All the other community aws modules that I use support applying tags to the created resources (where such a feature is supported by aws). Using aws_eks_cluster it is not possible to add tags, although this appears to be a supported feature when viewing a cluster in the aws console.

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

aws_eks_cluster
ADDITIONAL INFORMATION

Reviewed-by: Gonéri Le Bouder <goneri@lebouder.net>
Reviewed-by: Amir <None>
Reviewed-by: Mark Chappell <None>
(cherry picked from commit 3c1c903)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Feb 3, 2023
[PR #1591/3c1c903a backport][stable-5] eks_cluster: adding tags to eks cluster

This is a backport of PR #1591 as merged into main (3c1c903).
SUMMARY
As described by issue #1513
All the other community aws modules that I use support applying tags to the created resources (where such a feature is supported by aws). Using aws_eks_cluster it is not possible to add tags, although this appears to be a supported feature when viewing a cluster in the aws console.

ISSUE TYPE


Feature Pull Request

COMPONENT NAME

aws_eks_cluster
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-5 PR should be backported to the stable-5 branch community_review feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants