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 LogQueryWidgetProps): (Lack of accountId filed) #26105

Open
1 of 2 tasks
bygo7 opened this issue Jun 23, 2023 · 4 comments · May be fixed by #31512
Open
1 of 2 tasks

(Cloudwatch LogQueryWidgetProps): (Lack of accountId filed) #26105

bygo7 opened this issue Jun 23, 2023 · 4 comments · May be fixed by #31512
Assignees
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. p1

Comments

@bygo7
Copy link

bygo7 commented Jun 23, 2023

Describe the feature

We are trying to utilize the cross-account cross-region visibility through cloudwatch.
I can connect logs from different account to monitoring account, but on the cloudwatch dashboard I need to put "accountId" filed on my LogQueryWidgetProps manually.
Problem is that the LogQueryWidgetProps on CDK does not have "accountId" filed, so that I can not make query without manually going to AWS console and modifying dashboard body json.

Use Case

Constructing LogQueryWidget with CDK, utilizing cross-account cross-region

Proposed Solution

Adding "accountId" filed supported on LogQueryWidgetProps

Other Information

No response

Acknowledgements

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

CDK version used

v2

Environment details (OS name and version, etc.)

@bygo7 bygo7 added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Jun 23, 2023
@github-actions github-actions bot added the @aws-cdk/aws-cloudwatch Related to Amazon CloudWatch label Jun 23, 2023
@peterwoodworth
Copy link
Contributor

LogWidgets don't seem to have an accountId field according to the documentation. This looks to only be limited to metric widgets. Is the documentation inaccurate?

@peterwoodworth peterwoodworth added p1 response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. effort/small Small work item – less than a day of effort and removed needs-triage This issue or PR still needs to be triaged. labels Jun 23, 2023
@bygo7
Copy link
Author

bygo7 commented Jun 23, 2023

@peterwoodworth Yes, but the accountId works if I manually put it in the json dashboard body. Looks like it can query different accounts but it is not documented and not on the cdk. Can't we enable this filed?

@bygo7 bygo7 closed this as completed Jun 23, 2023
@bygo7 bygo7 reopened this Jun 23, 2023
@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

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.

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. label Jun 24, 2023
@JFen9
Copy link

JFen9 commented Jan 12, 2024

I encountered the same issue, but I was able to hack it by providing the ARN of the log group in the source account (which has the account id in it) in place of the log group name when specifying logGroupName property of LogQueryWidgetProps

That said, there indeed is a feature gap between cloudwatch console and cloudformation LogWidget that cloudformation team need to bridge

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. p1
Projects
None yet
4 participants