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

Stop accepting value counter passed as --type when using Dogshell cli #833

Open
gdubicki opened this issue Jun 3, 2024 · 1 comment
Open
Labels
kind/bug Bug related issue stale Stale - Bot reminder

Comments

@gdubicki
Copy link

gdubicki commented Jun 3, 2024

Describe the bug

My team has been using dogshell to post metrics with --type counter since at least 2020. I know that this is not documented but it's a pretty obvious different way to write --type count and it works.

The problem is that "counter" seems to actually be a different metric type than "count" in Datadog and while they behave mostly the same, the former seems to be deprecated and it doesn't support some features, like "Metrics without limits".

To Reproduce

Submit a metric --type counter with dogshell.

Expected behavior

  • An error that the type is unsupported should be returned. The command exit code should be non-zero.

or

  • Under the hood the type count should be used and an exit code 0 should be returned. Maybe a warning should be printed that the type counter is not supported and that count has been used?

Environment and Versions (please complete the following information):

$ dog --version
dog 0.49.1

Additional context

The information about the "counter" type not supporting "Metrics without limits" comes from this (private) Datadog support request: https://help.datadoghq.com/hc/en-us/requests/1691412

@gdubicki gdubicki added the kind/bug Bug related issue label Jun 3, 2024
Copy link

github-actions bot commented Jul 4, 2024

Thanks for your contribution!

This issue has been automatically marked as stale because it has not had activity in the last 30 days. Note that the issue will not be automatically closed, but this notification will remind us to investigate why there's been inactivity. Thank you for participating in the Datadog open source community.

If you would like this issue to remain open:

  1. Verify that you can still reproduce the issue in the latest version of this project.

  2. Comment that the issue is still reproducible and include updated details requested in the issue template.

@github-actions github-actions bot added the stale Stale - Bot reminder label Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Bug related issue stale Stale - Bot reminder
Projects
None yet
Development

No branches or pull requests

1 participant