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

Bulk migration to fail_json_aws #361

Merged
merged 5 commits into from
Jan 27, 2021

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jan 18, 2021

SUMMARY

Bulk migration from fail_json to fail_json aws when spitting out exceptions (fail_json_aws knows how to interpret and display them)

  • Also move various imports of camel_dict_to_snake_dict to from ansible.module_utils.common.dict_transformations
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/aws_api_gateway.py
plugins/modules/aws_codepipeline.py
plugins/modules/aws_direct_connect_confirm_connection.py
plugins/modules/aws_direct_connect_connection.py
plugins/modules/aws_direct_connect_gateway.py
plugins/modules/aws_direct_connect_link_aggregation_group.py
plugins/modules/aws_direct_connect_virtual_interface.py
plugins/modules/aws_kms_info.py
plugins/modules/aws_s3_bucket_info.py
plugins/modules/cloudfront_info.py
plugins/modules/cloudwatchlogs_log_group.py
plugins/modules/cloudwatchlogs_log_group_info.py
plugins/modules/data_pipeline.py
plugins/modules/dms_endpoint.py
plugins/modules/dms_replication_subnet_group.py
plugins/modules/dynamodb_table.py
plugins/modules/ec2_asg.py
plugins/modules/ec2_elb_info.py
plugins/modules/ec2_lc.py
plugins/modules/ec2_scaling_policy.py
plugins/modules/ec2_snapshot_copy.py
plugins/modules/ec2_vpc_endpoint.py
plugins/modules/ec2_vpc_nacl_info.py
plugins/modules/ec2_vpc_peer.py
plugins/modules/ec2_vpc_vgw.py
plugins/modules/ec2_vpc_vgw_info.py
plugins/modules/ecs_ecr.py
plugins/modules/elasticache.py
plugins/modules/elasticache_parameter_group.py
plugins/modules/elasticache_snapshot.py
plugins/modules/elb_application_lb_info.py
plugins/modules/elb_target.py
plugins/modules/elb_target_group_info.py
plugins/modules/execute_lambda.py
plugins/modules/iam_managed_policy.py
plugins/modules/iam_policy_info.py
plugins/modules/iam_user.py
plugins/modules/lambda.py
plugins/modules/lambda_alias.py
plugins/modules/rds_instance.py
plugins/modules/rds_param_group.py
plugins/modules/redshift.py
plugins/modules/s3_lifecycle.py
plugins/modules/s3_sync.py
plugins/modules/s3_website.py

ADDITIONAL INFORMATION

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request module module needs_triage owner_pr PR created by owner/maintainer plugins plugin (any type) labels Jan 18, 2021
@tremble
Copy link
Contributor Author

tremble commented Jan 27, 2021

Reviewed by Felix, tests all pass. I'm going to get this merged so we can avoid potential bit-rot.

@tremble tremble merged commit 6c88315 into ansible-collections:main Jan 27, 2021
@tremble tremble deleted the cleanup/fail_json_aws branch February 11, 2021 09:12
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 16, 2021
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
* Split imports and sort
* Move camel_dict_to_snake_dict imports to ansible.module_utils.common.dict_transformations
* Cleanup unused imports
* Bulk migration to fail_json_aws
* Changelog

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections@6c88315
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request module module owner_pr PR created by owner/maintainer plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants