-
Notifications
You must be signed in to change notification settings - Fork 421
Docs: Parameter Store aliases documentation incorrect #6213
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
Comments
Thanks for opening your first issue here! We'll come back to you as soon as we can. |
Hey @speshak! Thanks for opening this issue to improve our documentation. You're right, we need to update our documentation to reflect the correct parameter name and avoid customers wasting time with incorrect information. Would you like to send a pull request to update this? I'd really appreciate your contribution here. |
Yeah, I'd be happy to get something sent over. |
Super nice! Please let me know if you need any help. |
|
What were you searching in the docs?
I wanted to look up the layer ARN using a Terraform data resource. When following the example in the documentation I was getting parameter not found errors. I found that when SSM parameter store aliases were introduced in 3.4.0, the release notes state that the python version in the SSM path should include the
.
:The current documentation about using the SSM parameters states
Is this related to an existing documentation section?
https://docs.powertools.aws.dev/lambda/python/latest/#aws-ssm-parameter-store
How can we improve?
I believe the documentation should be updated to reflect that the parameter name includes the period in the python version. When I switched my TF code to include it, the parameter was found as expected.
Got a suggestion in mind?
I believe the documentation should be updated to reflect that the parameter name includes the period in the python version.
Acknowledgment
The text was updated successfully, but these errors were encountered: