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.

Sign up for GitHub

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

Jump to bottom

Dynatrace Output Plugin: Fixed behaviour when state map is cleared #8251

Merged
merged 66 commits into from
Oct 16, 2020
Merged

Dynatrace Output Plugin: Fixed behaviour when state map is cleared #8251

merged 66 commits into from
Oct 16, 2020

Conversation

thschue
Copy link
Contributor

@thschue thschue commented Oct 9, 2020

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

Thomas Schuetz and others added 30 commits July 2, 2020 11:32
added dynatrace client install instructions
b5f2608
fixed some minor issues
d2c3d57
added idea to gitignore
72dd44e
@wolfgangB33r
changed to DT normalized metric and dimension keys
f49ac2a
@wolfgangB33r
changed to DT normalized metric and dimension keys
62c6054
@wolfgangB33r
changed escaping of values to strconv.Quote
34d5f56
@thschue
Merge pull request #1 from dynatrace-oss-contrib/include-dt-plugin
8091b28 
Add Dynatrace Plugin
added oneagent support
de0b09f
adapted handling for oneagent
9c4f3d7
removed debug output
6ae8014
@thschue
Merge pull request #2 from dynatrace-oss-contrib/add-oneagent
3aa5d94 
Add oneagent handling
fixed normalization for fields, if they start with a digit
02f560c
Merge branch 'master' of github.com:influxdata/telegraf
795e3c5
fixed normalization
fb56fcf
changed package name
0e6a347
added readme information about direct OneAgent ingest endpoint
fc95083
@wolfgangB33r
added a metric name prefix setting
5087a01
Merge branch 'master' of github.com:influxdata/telegraf
c4284f7
fmtted file
9e7dccf
changed doc
91e26d9
@thschue @ssoroka
Update plugins/outputs/dynatrace/dynatrace.go
b783f3a 
Co-authored-by: Steven Soroka <[email protected]>
@thschue @ssoroka
Apply suggestions from code review
38a05fd 
Co-authored-by: Steven Soroka <[email protected]>
added requested changes
a273e62
changed log output
de2728f
changed tls configuration
72b0bc0
adapted tests
f75f254
fmtted file
72c2418
fixed tests
869cda4
@wolfgangB33r
fixed tag order and added encoding checks in all tests
9f16099
@wolfgangB33r
changed formatting
2167b54
mikopp and others added 21 commits August 17, 2020 15:27
@mikopp
Update README.md
231878a
@thschue
Merge pull request #5 from mikopp/patch-4
0687eaf 
Update README.md
@thschue
Update README.md
ceb4221
removed InsecureSkipVerify from struct as it is part of github.com/in…
eeb6bed 
…fluxdata/telegraf/plugins/common/tls
Merge branch 'master' of github.com:influxdata/telegraf
4ae0062
Small Doc change
99b3905
go fmtted file
43b35f2
Added dash to AllowedCharList
6fdc281
Changed Handling of Monotonic Counters
1067ad9
Merge branch 'master' of github.com:influxdata/telegraf
d30a3b0
@thschue @ssoroka
Update plugins/outputs/dynatrace/README.md
f6b3fb3 
Co-authored-by: Steven Soroka <[email protected]>
@wolfgangB33r
added delta value support for monotonic count metric types
a105de5
@wolfgangB33r
deleted out.txt file
30f6cac
Merge branch 'master' of github.com:influxdata/telegraf
65fb0af
@wolfgangB33r
added a global state flush interval to prevent any long running memor…
90a71f7 
…y leak issues
Merge branch 'master' of github.com:influxdata/telegraf
5264a36
@thschue
Fix delta reset (#6)
8964e3c
fixed merge conflicts
d51b77b
removed blank line
6b00413
go fmtted file
18a60d3
Merge branch 'master' of github.com:influxdata/telegraf
3b923ab
@ssoroka
Copy link
Contributor

ssoroka commented Oct 13, 2020

can you try merging the latest master into this branch?

@thschue thschue requested a review from ssoroka October 16, 2020 07:37
@ssoroka ssoroka merged commit c85fb58 into influxdata:master Oct 16, 2020
ssoroka pushed a commit that referenced this pull request Oct 16, 2020
@thschue @ssoroka
Dynatrace Output Plugin: Fixed behaviour when state map is cleared (#…
ccb0e41 
…8251)

(cherry picked from commit c85fb58)
arstercz pushed a commit to arstercz/telegraf that referenced this pull request Mar 5, 2023
@thschue
Dynatrace Output Plugin: Fixed behaviour when state map is cleared (i…
03a7286 
…nfluxdata#8251)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ssoroka ssoroka ssoroka approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@thschue @ssoroka @wolfgangB33r @mikopp