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

(cloudwatch): allow period override on LogQueryWidget #32132

Open
2 tasks
straygar opened this issue Nov 14, 2024 · 2 comments
Open
2 tasks

(cloudwatch): allow period override on LogQueryWidget #32132

straygar opened this issue Nov 14, 2024 · 2 comments
Labels
@aws-cdk/aws-cloudwatch Related to Amazon CloudWatch effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2

Comments

@straygar
Copy link

Describe the feature

Allow overriding the default period on a LogQueryWidget.

This is possible on GraphWidget but not on LogQueryWidget. (there are in fact quite a few props other that don't exist on LogQueryWidget).

Use Case

I want to run a Logs Insights query over a large time period, ignoring whatever the global setting is in the dashboard.

Proposed Solution

Extend LogQueryWidget with those of GraphWidget.

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CDK version used

2.166.0

Environment details (OS name and version, etc.)

MacOS

@straygar straygar added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Nov 14, 2024
@github-actions github-actions bot added the @aws-cdk/aws-cloudwatch Related to Amazon CloudWatch label Nov 14, 2024
@khushail khushail added investigating This issue is being investigated and/or work is in progress to resolve the issue. p2 and removed needs-triage This issue or PR still needs to be triaged. labels Nov 14, 2024
@khushail khushail self-assigned this Nov 14, 2024
@khushail
Copy link
Contributor

Hi @straygar , thanks for reaching out.
Yes, the LogQueryWidgetProps does not have the Period property as of now so you can not overrride the period value. Although it would be good to have the missing properties as compared to GraphWidgetPropsI am marking this request as P2 which would mean it won't be immediately addressed by the team but would be on their radar. Also its open for community contribution.

@khushail khushail added effort/small Small work item – less than a day of effort and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Nov 14, 2024
@khushail khushail removed their assignment Nov 14, 2024
@tomoki10
Copy link
Contributor

Looking at the Log Widget Object settings in the CloudWatch Dashboard CloudFormation documentation below, it appears this cannot be achieved since the period parameter cannot be set.

https://docs.aws.amazon.com/AmazonCloudWatch/latest/APIReference/CloudWatch-Dashboard-Body-Structure.html#CloudWatch-Dashboard-Properties-Log-Widget-Object

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-cloudwatch Related to Amazon CloudWatch effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2
Projects
None yet
Development

No branches or pull requests

3 participants