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: Surround dimension names with double quotes #22222

Merged
merged 1 commit into from
Mar 5, 2020

Conversation

jeet-parekh
Copy link
Contributor

What this PR does / why we need it:
Places double quotes around the dimension names in the generated CloudWatch query.

Which issue(s) this PR fixes:
Fixes #22104.

@daniellee daniellee added datasource/CloudWatch pr/external This PR is from external contributor labels Feb 18, 2020
@stale
Copy link

stale bot commented Feb 28, 2020

This pull request has been automatically marked as stale because it has not had activity in the last 2 weeks. It will be closed in 30 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@stale stale bot added the stale Issue with no recent activity label Feb 28, 2020
@jeet-parekh
Copy link
Contributor Author

@sunker, could you review this? Or assign it to somebody?

@stale stale bot removed the stale Issue with no recent activity label Mar 4, 2020
Copy link
Contributor

@sunker sunker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this and it works fine. Great work @jeet-parekh!

@sunker sunker added this to the 6.7 milestone Mar 5, 2020
@sunker sunker changed the title surround CloudWatch dimension names with double quotes CloudWatch: Surround dimension names with double quotes Mar 5, 2020
@sunker sunker merged commit 0f8cfca into grafana:master Mar 5, 2020
bergquist added a commit to bergquist/grafana that referenced this pull request Mar 6, 2020
* master:
  CloudWatch: updated namespaces - Athena, DocDB, and Route53Resolver (grafana#22604)
  Graph: Improve point rendering performance (grafana#22610)
  Alerting: Fix state age test failures (grafana#22606)
  Docs: Update image_rendering.md (grafana#22586)
  UI: Segment improvements (grafana#22601)
  remove section about alias imports (grafana#22585)
  Backend Plugins: Support handling of streaming resource response (grafana#22580)
  Stackdriver: Migrate GCE default project (grafana#22593)
  Toolkit: plugin ci needs to cooperate better with make/mage (grafana#22588)
  surround CloudWatch dimension names with double quotes (grafana#22222)
  Fix: when reloading page make sure that time picker history is converted to dateTime.
  Core: add active users stat (grafana#22563)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CloudWatch metric names with spaces causing queries to fail
3 participants