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

Remove use of ignored boto3 parameter #2075

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Apr 9, 2024

SUMMARY

get_ec2_security_group_ids_from_names() has ignored the boto3 parameter since release 4.0.0, drop the use.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/glue_connection.py
plugins/modules/autoscaling_launch_config.py

ADDITIONAL INFORMATION

@tremble tremble added the backport-7 PR should be backported to the stable-7 branch label Apr 9, 2024
@tremble tremble requested a review from alinabuzachis April 9, 2024 09:48
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 3m 56s (non-voting)
✔️ build-ansible-collection SUCCESS in 14m 32s
✔️ ansible-test-splitter SUCCESS in 5m 35s
✔️ integration-community.aws-1 SUCCESS in 6m 09s
✔️ integration-community.aws-2 SUCCESS in 7m 59s
Skipped 20 jobs

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Apr 9, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/2bc6ffaaf8ce43a0a9204769b17b458a

ansible-galaxy-importer FAILURE in 5m 58s (non-voting)
✔️ build-ansible-collection SUCCESS in 14m 15s
✔️ ansible-test-splitter SUCCESS in 5m 20s
✔️ integration-community.aws-1 SUCCESS in 7m 19s
✔️ integration-community.aws-2 SUCCESS in 7m 13s
Skipped 20 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit b5d4ea7 into ansible-collections:main Apr 9, 2024
69 of 105 checks passed
Copy link

patchback bot commented Apr 9, 2024

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/b5d4ea703d861b5becaaead98256535d603bbd96/pr-2075

Backported as #2076

🤖 @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 Apr 9, 2024
Remove use of ignored boto3 parameter

SUMMARY
get_ec2_security_group_ids_from_names() has ignored the boto3 parameter since release 4.0.0, drop the use.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/modules/glue_connection.py
plugins/modules/autoscaling_launch_config.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit b5d4ea7)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Apr 9, 2024
[PR #2075/b5d4ea70 backport][stable-7] Remove use of ignored boto3 parameter

This is a backport of PR #2075 as merged into main (b5d4ea7).
SUMMARY
get_ec2_security_group_ids_from_names() has ignored the boto3 parameter since release 4.0.0, drop the use.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/modules/glue_connection.py
plugins/modules/autoscaling_launch_config.py
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
softwarefactory-project-zuul bot pushed a commit to ansible-collections/amazon.aws that referenced this pull request Apr 15, 2024
Deprecate ignored boto3 parameters

SUMMARY
module_utils.botocore.get_aws_region(), module_utils.botocore.get_aws_connection_info() and module_utils.ec2.get_ec2_security_group_ids_from_names() currently have boto3 parameters which default to None, and do nothing.  This is a leftover from dropping support for the old boto SDK back in release 4.0.0.
Start the cleanup process and deprecate the parameter.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/botocore.py
plugins/module_utils/ec2.py
plugins/module_utils/elbv2.py
plugins/module_utils/modules.py
plugins/modules/ec2_eni.py
ADDITIONAL INFORMATION
(see also ansible-collections/community.aws#2075)

Reviewed-by: Alina Buzachis
patchback bot pushed a commit to ansible-collections/amazon.aws that referenced this pull request Apr 15, 2024
Deprecate ignored boto3 parameters

SUMMARY
module_utils.botocore.get_aws_region(), module_utils.botocore.get_aws_connection_info() and module_utils.ec2.get_ec2_security_group_ids_from_names() currently have boto3 parameters which default to None, and do nothing.  This is a leftover from dropping support for the old boto SDK back in release 4.0.0.
Start the cleanup process and deprecate the parameter.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/botocore.py
plugins/module_utils/ec2.py
plugins/module_utils/elbv2.py
plugins/module_utils/modules.py
plugins/modules/ec2_eni.py
ADDITIONAL INFORMATION
(see also ansible-collections/community.aws#2075)

Reviewed-by: Alina Buzachis
(cherry picked from commit cd455f0)
tremble added a commit to ansible-collections/amazon.aws that referenced this pull request Apr 18, 2024
Deprecate ignored boto3 parameters

SUMMARY
module_utils.botocore.get_aws_region(), module_utils.botocore.get_aws_connection_info() and module_utils.ec2.get_ec2_security_group_ids_from_names() currently have boto3 parameters which default to None, and do nothing.  This is a leftover from dropping support for the old boto SDK back in release 4.0.0.
Start the cleanup process and deprecate the parameter.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/botocore.py
plugins/module_utils/ec2.py
plugins/module_utils/elbv2.py
plugins/module_utils/modules.py
plugins/modules/ec2_eni.py
ADDITIONAL INFORMATION
(see also ansible-collections/community.aws#2075)

Reviewed-by: Alina Buzachis
(cherry picked from commit cd455f0)
tremble added a commit to ansible-collections/amazon.aws that referenced this pull request Apr 18, 2024
Deprecate ignored boto3 parameters

SUMMARY

module_utils.botocore.get_aws_region(), module_utils.botocore.get_aws_connection_info() and module_utils.ec2.get_ec2_security_group_ids_from_names() currently have boto3 parameters which default to None, and do nothing.  This is a leftover from dropping support for the old boto SDK back in release 4.0.0.
Start the cleanup process and deprecate the parameter.

ISSUE TYPE

Feature Pull Request

COMPONENT NAME

plugins/module_utils/botocore.py
plugins/module_utils/ec2.py
plugins/module_utils/elbv2.py
plugins/module_utils/modules.py
plugins/modules/ec2_eni.py

ADDITIONAL INFORMATION

(see also ansible-collections/community.aws#2075)

Reviewed-by: Alina Buzachis
(cherry picked from commit cd455f0)

Co-authored-by: Mark Chappell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-7 PR should be backported to the stable-7 branch mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants