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

Observability pattern for cost monitoring #127

Closed
wants to merge 2 commits into from

Conversation

Howlla
Copy link
Contributor

@Howlla Howlla commented Nov 1, 2023

*Issue #, if available:*#18

Description of changes: This is a placeholder PR to share the work. currently i am facing an issue to get the queryURL from AMP. Kubecost needs this value like https://aps-workspaces.<region>.amazonaws.com/workspaces/<WS-ID>/api/v1/query
One workaround could be adding the WS-ID value from env variables but looking for inputs.

Right now i have to first run this deployment. Then identify the Workspace ID and update cdk code, then deploy again. Not ideal definitely

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@elamaran11
Copy link
Contributor

@Howlla I see the PR is incomplete. Still documentation and few other stuff before i can review. Can you close the loop.

@elamaran11
Copy link
Contributor

@Howlla Any updates on this PR.

@elamaran11 elamaran11 linked an issue Jan 4, 2024 that may be closed by this pull request
@elamaran11
Copy link
Contributor

@Howlla Can you close this documentation and this PR?

@Howlla Howlla closed this Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Observability Pattern for Cost Monitoring with EKS
2 participants