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

route53_info: fix "key error" for health_check operations #1419

Conversation

mandar242
Copy link
Contributor

SUMMARY

Fixes #1396

This pull request

  1. Add new return key health_check_observations for health check operations, returned when I(query=health_check) and I(health_check_method=status) or I(health_check_method=failure_reason)
  2. Fixes "Key Error" when getting status or failure_reason of a health check.
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

route53_info

ADDITIONAL INFORMATION

@github-actions
Copy link

github-actions bot commented Mar 9, 2023

Docs Build 📝

Thank you for contribution!✨

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

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/3cb6809be9e846f98a972e72a2830f51

✔️ ansible-galaxy-importer SUCCESS in 4m 12s
✔️ build-ansible-collection SUCCESS in 13m 49s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 10m 59s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 11m 06s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 11m 52s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 10m 51s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 11m 40s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 11m 48s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 9m 23s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 12m 33s
✔️ cloud-tox-py3 SUCCESS in 7m 25s
ansible-test-changelog FAILURE in 4m 44s
✔️ ansible-test-splitter SUCCESS in 5m 23s
✔️ integration-amazon.aws-1 SUCCESS in 9m 43s
Skipped 43 jobs

plugins/modules/route53_info.py Show resolved Hide resolved
plugins/modules/route53_info.py Show resolved Hide resolved
plugins/modules/route53_info.py Show resolved Hide resolved
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.

Needs a changelog, and if possible an integration test.

However, I think this is correct, with errors previously being thrown where _observations is now returned.

@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 54s
✔️ build-ansible-collection SUCCESS in 12m 23s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 11m 44s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 11m 46s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 11m 36s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 11m 54s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 9m 30s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 8m 55s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 10m 32s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 8m 49s
✔️ cloud-tox-py3 SUCCESS in 7m 03s
✔️ ansible-test-changelog SUCCESS in 4m 18s
✔️ ansible-test-splitter SUCCESS in 4m 47s
✔️ integration-amazon.aws-1 SUCCESS in 7m 35s
Skipped 43 jobs

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/d8f8d13fb7984249ab5899680fb0f228

✔️ ansible-galaxy-importer SUCCESS in 3m 54s
✔️ build-ansible-collection SUCCESS in 13m 13s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 42s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 8m 32s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 13s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 11m 43s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 9m 35s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 10m 53s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 8m 31s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 8m 29s
✔️ cloud-tox-py3 SUCCESS in 8m 20s
✔️ ansible-test-changelog SUCCESS in 4m 17s
✔️ ansible-test-splitter SUCCESS in 4m 56s
✔️ integration-amazon.aws-1 SUCCESS in 9m 57s
integration-amazon.aws-2 FAILURE in 11m 54s
Skipped 42 jobs

@mandar242
Copy link
Contributor Author

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build failed.
https://ansible.softwarefactory-project.io/zuul/buildset/71cd7801187b4e1fb8ee80ea9c42fe67

✔️ ansible-galaxy-importer SUCCESS in 4m 31s
✔️ build-ansible-collection SUCCESS in 12m 22s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 11m 28s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 8m 41s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 9m 23s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 9m 46s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 10m 23s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 10m 44s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 8m 58s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 9m 55s
✔️ cloud-tox-py3 SUCCESS in 6m 55s
✔️ ansible-test-changelog SUCCESS in 4m 15s
✔️ ansible-test-splitter SUCCESS in 4m 38s
✔️ integration-amazon.aws-1 SUCCESS in 9m 15s
integration-amazon.aws-2 FAILURE in 10m 42s
Skipped 42 jobs

@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 4m 09s
✔️ build-ansible-collection SUCCESS in 12m 28s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 21s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 11m 55s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 8m 27s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 12m 16s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 9m 46s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 11m 25s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 10m 45s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 11m 54s
✔️ cloud-tox-py3 SUCCESS in 7m 20s
✔️ ansible-test-changelog SUCCESS in 4m 12s
✔️ ansible-test-splitter SUCCESS in 4m 52s
✔️ integration-amazon.aws-1 SUCCESS in 7m 11s
✔️ integration-amazon.aws-2 SUCCESS in 9m 50s
Skipped 42 jobs

@tremble
Copy link
Contributor

tremble commented Mar 17, 2023

@mandar242 Issue was the usual AWS problem, things hadn't quite become available "everywhere". Added a delay and it's now working

@tremble tremble added mergeit Merge the PR (SoftwareFactory) backport-5 PR should be backported to the stable-5 branch labels Mar 17, 2023
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 3m 52s
✔️ build-ansible-collection SUCCESS in 12m 50s
✔️ ansible-test-splitter SUCCESS in 5m 00s
✔️ integration-amazon.aws-1 SUCCESS in 8m 14s
✔️ integration-amazon.aws-2 SUCCESS in 8m 42s
Skipped 42 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit a27dfd1 into ansible-collections:main Mar 17, 2023
@patchback

This comment was marked as outdated.

@tremble tremble added backport-5 PR should be backported to the stable-5 branch and removed backport-5 PR should be backported to the stable-5 branch labels Mar 17, 2023
@patchback
Copy link

patchback bot commented Mar 17, 2023

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/a27dfd140c04db72b5c0d5a7b2b7fe498899068f/pr-1419

Backported as #1434

🤖 @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 Mar 17, 2023
route53_info: fix "key error" for health_check operations

SUMMARY

Fixes #1396
This pull request

Add new return key health_check_observations for health check operations, returned when I(query=health_check) and I(health_check_method=status) or I(health_check_method=failure_reason)
Fixes "Key Error" when getting status or failure_reason of a health check.

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

route53_info
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
Reviewed-by: Mandar Kulkarni <mandar242@gmail.com>
Reviewed-by: Alina Buzachis
(cherry picked from commit a27dfd1)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Mar 17, 2023
)

[PR #1419/a27dfd14 backport][stable-5] route53_info: fix "key error" for health_check operations

This is a backport of PR #1419 as merged into main (a27dfd1).
SUMMARY

Fixes #1396
This pull request

Add new return key health_check_observations for health check operations, returned when I(query=health_check) and I(health_check_method=status) or I(health_check_method=failure_reason)
Fixes "Key Error" when getting status or failure_reason of a health check.


ISSUE TYPE


Bugfix Pull Request

COMPONENT NAME

route53_info
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell
@mandar242 mandar242 deleted the route53_info_keyError_fix branch March 17, 2023 22:18
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
Add ResourceType on ec2 snapshot copy

SUMMARY
Fixes this issue that shows up when tags are specified:
An error occurred waiting for the snapshot to become available.: An
error occurred (InvalidParameterValue) when calling the CopySnapshot
operation: Tag specification resource type must have a value

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_snapshot_copy
ADDITIONAL INFORMATION



# before

botocore.exceptions.ClientError: An error occurred (InvalidParameterValue) when calling the CopySnapshot operation: Tag specification resource type must have a value                              
fatal: [localhost]: FAILED! => {                                                                                                                                                                   
    "boto3_version": "1.24.57",                                                                                                                                                                    
    "botocore_version": "1.27.58",                                                                                                                                                                 
    "changed": false,                                                                                                                                                                              
    "error": {                                                                                                                                                                                     
        "code": "InvalidParameterValue",                                                                                                                                                           
        "message": "Tag specification resource type must have a value"                                                                                                                             
    },  
....


# after  (no error message)

changed: [localhost] => {                                                                                                                                                                          
    "changed": true,                                                                                                                                                                               
    "invocation": {                                                                                                                                                                                
        "module_args": {       
.....

Reviewed-by: Mark Chappell <None>
Reviewed-by: Colby Shores <coleshores@gmail.com>
Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
Add ResourceType on ec2 snapshot copy

SUMMARY
Fixes this issue that shows up when tags are specified:
An error occurred waiting for the snapshot to become available.: An
error occurred (InvalidParameterValue) when calling the CopySnapshot
operation: Tag specification resource type must have a value

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_snapshot_copy
ADDITIONAL INFORMATION



# before

botocore.exceptions.ClientError: An error occurred (InvalidParameterValue) when calling the CopySnapshot operation: Tag specification resource type must have a value                              
fatal: [localhost]: FAILED! => {                                                                                                                                                                   
    "boto3_version": "1.24.57",                                                                                                                                                                    
    "botocore_version": "1.27.58",                                                                                                                                                                 
    "changed": false,                                                                                                                                                                              
    "error": {                                                                                                                                                                                     
        "code": "InvalidParameterValue",                                                                                                                                                           
        "message": "Tag specification resource type must have a value"                                                                                                                             
    },  
....


# after  (no error message)

changed: [localhost] => {                                                                                                                                                                          
    "changed": true,                                                                                                                                                                               
    "invocation": {                                                                                                                                                                                
        "module_args": {       
.....

Reviewed-by: Mark Chappell <None>
Reviewed-by: Colby Shores <coleshores@gmail.com>
Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
Add ResourceType on ec2 snapshot copy

SUMMARY
Fixes this issue that shows up when tags are specified:
An error occurred waiting for the snapshot to become available.: An
error occurred (InvalidParameterValue) when calling the CopySnapshot
operation: Tag specification resource type must have a value

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_snapshot_copy
ADDITIONAL INFORMATION



# before

botocore.exceptions.ClientError: An error occurred (InvalidParameterValue) when calling the CopySnapshot operation: Tag specification resource type must have a value                              
fatal: [localhost]: FAILED! => {                                                                                                                                                                   
    "boto3_version": "1.24.57",                                                                                                                                                                    
    "botocore_version": "1.27.58",                                                                                                                                                                 
    "changed": false,                                                                                                                                                                              
    "error": {                                                                                                                                                                                     
        "code": "InvalidParameterValue",                                                                                                                                                           
        "message": "Tag specification resource type must have a value"                                                                                                                             
    },  
....


# after  (no error message)

changed: [localhost] => {                                                                                                                                                                          
    "changed": true,                                                                                                                                                                               
    "invocation": {                                                                                                                                                                                
        "module_args": {       
.....

Reviewed-by: Mark Chappell <None>
Reviewed-by: Colby Shores <coleshores@gmail.com>
Reviewed-by: None <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 mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

route53_info HealthCheck key error
3 participants