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

kms_key - Finish deprecation of policy_grant_types and related keys #1344

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jul 11, 2022

SUMMARY

Managing the KMS IAM Policy via policy_mode and policy_grant_types was fragile and previously deprecated.

Complete the deprecation and remove the options in in favour of the policy option.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/kms_key.py

ADDITIONAL INFORMATION

Original deprecation: ansible/ansible#60561

@tremble tremble added the do_not_backport This PR should not be backported to stable- branches unless absolutely necessary label Jul 11, 2022
@github-actions
Copy link

github-actions bot commented Jul 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 integration tests/integration module module plugins plugin (any type) tests tests labels Jul 11, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 56s
✔️ build-ansible-collection SUCCESS in 4m 54s
✔️ ansible-test-sanity-docker-devel SUCCESS in 12m 15s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 17s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 42s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 9m 53s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 48s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 7m 25s
✔️ ansible-test-splitter SUCCESS in 2m 27s
✔️ integration-community.aws-1 SUCCESS in 13m 47s
⚠️ 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

@tremble tremble requested review from jatorcasso, jillr, alinabuzachis and markuman and removed request for jatorcasso July 11, 2022 11:14
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jul 11, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 25s
✔️ build-ansible-collection SUCCESS in 5m 14s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 31s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 13s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 55s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 12s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 48s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 07s
✔️ ansible-test-splitter SUCCESS in 2m 28s
✔️ integration-community.aws-1 SUCCESS in 13m 09s
⚠️ 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

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 9fb435d into ansible-collections:main Jul 11, 2022
@tremble tremble deleted the deprecations/5.0.0/kms_key/policy_grant_types branch September 9, 2022 09:02
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…nsible-collections#1344)

kms_key - Finish deprecation of policy_grant_types and related keys

SUMMARY
Managing the KMS IAM Policy via policy_mode and policy_grant_types was fragile and previously deprecated.
Complete the deprecation and remove the options in in favour of the policy option.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/modules/kms_key.py
ADDITIONAL INFORMATION
Original deprecation: ansible/ansible#60561

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@9fb435d
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
aws_collection_constants

SUMMARY
Adds a simple lookup plugin to retrieve some of the constants in the collection
ISSUE TYPE

New Module Pull Request

COMPONENT NAME
plugins/lookup/aws_collection_constants.py
ADDITIONAL INFORMATION
This means we've one less place we need to update when bumping the supported botocore version.

Reviewed-by: Bikouo Aubin <None>
Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review do_not_backport This PR should not be backported to stable- branches unless absolutely necessary feature This issue/PR relates to a feature request integration tests/integration mergeit Merge the PR (SoftwareFactory) module module plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants