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

removed ServiceName dimension from CPUUtilization & MemoryUtilization… #10

Merged
merged 1 commit into from
Mar 4, 2017

Conversation

AnthonyWC
Copy link
Contributor

Removed "ServiceName": "$service" as default choices for CPUUtilization and MemoryUtilization since they give off error if not chosen.

@jangaraj jangaraj merged commit 211c44d into monitoringartist:master Mar 4, 2017
@jangaraj
Copy link
Member

jangaraj commented Mar 4, 2017

Thank you Anthony.

@AnthonyWC AnthonyWC deleted the fix-aws-ecs branch March 27, 2017 01:39
jangaraj pushed a commit that referenced this pull request May 22, 2019
* Revert "removed ServiceName dimension from CPUUtilization & MemoryUtilization as default (#10)"

This reverts commit 211c44d in order to rollback an unintended consequence
of PR #10. In particular, after that change, the dashboards no longer display per service ECS metrics.

* Create panels for CPU and memory queries not qualified by ServiceName

metrics. The previous commit reverted that change to restore the per-service functionality
and introduces two new panels to support the issue #9 that #10 was meant
to address.

Signed-off-by: Jon Seymour <jon@wildducktheories.com>
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.

None yet

2 participants