-
Notifications
You must be signed in to change notification settings - Fork 390
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
add datadog_metric_tag_configuration resource #960
Merged
Merged
Changes from 1 commit
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
8b9782a
initial commit for metric tag configuration resource
efraese 91e8bef
fixed build update to properly check metric_type before setting inclu…
efraese c9bd7ff
Merge branch 'master' into eric.fraese/PA-141
efraese 9108e02
added first pass on tests for tag config resource
efraese efefe5a
added first pass on example for tag config resource
efraese 8bad3d1
changes to upgrade client to master
efraese eead8dc
fmt example
efraese ca7ca3f
fixed tag config schema and use proper id for api calls
efraese 6802863
use proper provider for test
efraese 3c15b70
capitalize test func name
efraese e87647c
enable unstable endpoints, remove prints, use proper id for state, ch…
efraese 295a7bc
Merge branch 'master' into eric.fraese/PA-141
efraese 7962da8
add remaining diff from master merge in
efraese 4e01f32
Merge branch 'master' into eric.fraese/PA-141
efraese 2f6dcbb
cleanup comment and imports
efraese d42b8d7
golint
efraese bcdf74a
fix test to use proper metric_name
efraese 8003a47
fix existing tag config check
efraese 824a457
cassettes
efraese 10c160a
updated cassettes
efraese 4cc3bca
updated cassettes and import test name
efraese f4e886c
removed old cassettes
efraese 53c36f5
cleanup comments and unneeded changes
efraese 013d820
dont include percentiles in example
efraese a7d55f7
added customizeDiff function for include_percentiles option
efraese 727ef6a
removed completed todos
efraese f278291
fix error message
efraese 620916b
fix error message
efraese 7bc12e9
updated example metric name and generated docs
efraese 48c5365
updated cassettes
efraese 94396d3
fix cassettes for import test
efraese 6184cc5
Merge branch 'master' into eric.fraese/PA-141
efraese a6dd240
update docs
efraese d55df6f
update doc comment
efraese a421b46
added examples and updated docs
efraese ac668db
update description, check status code for update
efraese 59089b0
update test check to check status code
efraese 5da5220
update docs
efraese eefa11c
added error test and cassette
efraese c4503ca
updated import cassette
efraese 1174ca2
updated basic cassette
efraese 9fc7712
check status code when importing tag configuration
efraese 6b7dd86
added test invalid import ID, update cassettes
efraese d1faf91
added metrics-agg as codeowner
efraese 41fc2b6
changes from 1st round of PR comments, first pass at handling [] tags
efraese fabe288
update import cassette
efraese 0c388f3
update basic cassette
efraese f09e45f
change regex, change validation to use all function
efraese f5161da
Merge branch 'master' into eric.fraese/PA-141
efraese File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
fix existing tag config check
- Loading branch information
commit 8003a47e8c414e410d86cc3373664ca089b03462
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need that particular check, can't we rely on create failing appropriately?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, the create endpoint will return
409
in this case. I'll remove this check here and just let the create call fail.