search

aws / aws-sdk

Landing page for the AWS SDKs on GitHub
https://aws.amazon.com/tools/
Other
68 stars 12 forks source link

aws docdb describe-db-instances | missing "PerformanceInsightsEnabled" in output #607

Closed mt-spe closed 6 months ago

mt-spe commented 9 months ago

Describe the bug

Output for -

aws docdb describe-db-instances --filters Name=db-cluster-id,Values=adocdb-cluster-cl1 --query 'DBInstances[].PerformanceInsightsEnabled'

is not returning any data for a DocumentDB cluster.

However, results are returned for

aws rds describe-db-instances --filters Name=db-cluster-id,Values=adocdb-cluster-cl1 --query 'DBInstances[].PerformanceInsightsEnabled'

$ aws --version aws-cli/2.8.9 Python/3.9.11 Windows/10 exe/AMD64 prompt/off

Expected Behavior

Output should show value of PerformanceInsightsEnabled - either true or false. For example, this below for a 3 instance cluster.

[ true, true, true ]

Current Behavior

No values are returned. []

Reproduction Steps

aws docdb describe-db-instances --filters Name=db-cluster-id,Values=adocdb-cluster-cl1 --query 'DBInstances[].PerformanceInsightsEnabled'

Possible Solution

aws docdb describe-db-instances output should include the status of PerformanceInsightsEnabled - either true or false.

Additional Information/Context

No response

CLI version used

aws-cli/2.8.9

Environment details (OS name and version, etc.)

Windows/10 exe/AMD64

RyanFitzSimmonsAK commented 9 months ago

Hi @mt-spe, thanks for reaching out. According to the documentation, PerformanceInsightsEnabled is not intended to be returned in the response of aws docdb describe-db-instances. If you'd like me to reach out to the service team about adding this parameter to the response as a feature request, I can do that for you. Otherwise, this can't really be considered a bug. Just let me know how you'd like to proceed here.

mt-spe commented 9 months ago

Hi @RyanFitzSimmonsAK, thank you for reviewing and providing an update.

Since Performance Insights is a new-ish feature for DocumentDB, perhaps it needs to be added to the output for aws docdb describe-db-instances. I agree this cannot be considered a bug. If you would please reach out to the service team for this to be considered a feature-request, it'd be great.

RyanFitzSimmonsAK commented 9 months ago

I've reached out to them and will let you know when I have any updates. In the meantime, I'm going to transfer this issue to our cross-SDK repository.

Ticket # for internal use : V1058494441

RyanFitzSimmonsAK commented 6 months ago

PerformanceInsightsEnabled and PerformanceInsightsKMSKeyId have been added to the DescribeDBInstances response. Thank you for your patience.

github-actions[bot] commented 6 months ago

This issue is now closed.

Comments on closed issues are hard for our team to see. If you need more assistance, please either tag a team member or open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.