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

ssm-connector: better debugging and exception in exec_command file_transport_command methods #1017

Closed
1 task done
ryancabrera opened this issue Mar 25, 2022 · 3 comments
Closed
1 task done
Labels
feature This issue/PR relates to a feature request module module needs_triage plugins plugin (any type)

Comments

@ryancabrera
Copy link

Summary

I suspect there might be a bug in the file_transport_command method as any federated user I attempt to use with the SSM connector yields unhandled errors. Specifically, when the script attempt to curl the AnsiballZ.py script from the S3 bucket back to my instance using the presigned URL, the returned object is an XML file with an error (Unsure how to capture the .xml file and save the error)

I'm unsure if the issue is when the connector puts the file into s3 or when it copies the file from s3 to the ec2 instance. I can try to check out the bucket history in CloudTrail to see what is going on there but I believe the script itself should have additional debugging output that shows specifically what is happening with the get/puts in addition to having validation that a get/put is successful.

Issue Type

Feature Idea

Component Name

aws_ssm.py

Additional Information

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@ansibullbot
Copy link

Files identified in the description:

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibullbot ansibullbot added feature This issue/PR relates to a feature request module module needs_triage plugins plugin (any type) labels Mar 25, 2022
@tremble
Copy link
Contributor

tremble commented Feb 6, 2023

The XML-related error is likely due to #1190 and should be fixed in release 5.2.0. Unfortunately because of where the failure you're seeing is there's not much we can do to improve the logging.

@tremble tremble closed this as completed Feb 6, 2023
abikouo pushed a commit to abikouo/community.aws that referenced this issue Oct 24, 2023
…ons#1017)

tests: drop Python2's `u` marker for unicode string

With Python3, all strings are now unicode by default. We can remove
the u markers and clean up the data structure.

Reviewed-by: Jill R <None>
Reviewed-by: Mike Graves <mgraves@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request module module needs_triage plugins plugin (any type)
Projects
None yet
Development

No branches or pull requests

3 participants