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

feat: add influx semantic commits checker, checks only last commit. #11037

Merged
merged 5 commits into from
May 10, 2022

Conversation

tysonkamp
Copy link
Contributor

Required for all PRs:

  • [ N/A] Updated associated README.md.
  • [ N/A] Wrote appropriate unit tests.
  • Pull request title or commits are in conventional commit format

This change unblocks https://github.com/influxdata/k8s-idpe/issues/5723

Introduce Influxdata semantic commit and PR text checker to allow removal of semantic-pull-requests github app.

@tysonkamp tysonkamp requested a review from reimda April 26, 2022 18:29
@tysonkamp tysonkamp self-assigned this Apr 26, 2022
@telegraf-tiger telegraf-tiger bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Apr 26, 2022
Copy link
Contributor

@reimda reimda left a comment

Choose a reason for hiding this comment

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

Hi @tysonkamp, thanks for working on CI for semantic commits across all the org repos.

We use semantic commit messages on master but we don't enforce them on PR branches, except to make sure that when we squash and merge, the commit to master has a semantic commit.

There are two cases to handle to make sure "squash and merge" makes a semantic commit on master. 1) If there are multiple commits in the branch, GH uses the PR title as the message of the squash commit, so we need to check that the PR title is a valid semantic commit message. 2) if there is only one commit in the PR, there is no squash needed, so when you "squash and merge" it doesn't actually squash. It ignores the PR title and just merges the single commit. In this case the PR commit message must be a semantic commit message.

Our community review process and branch management strategy drive the way we use semantic versioning. We're happy with our processes, so a replacement for our current CI step is going to need to work well with the existing process.

This PR needs some changes in order to work with the existing process. I don't think it does the two steps I described. I heard you and @powersj talked about this yesterday and you are planning to add an option to influxdata/validate-semantic-github-messages that does what we need.

If there's anything I can do to help, please let me know, here or on slack or zoom!

@tysonkamp
Copy link
Contributor Author

@reimda thank you for the clear articulation of the domain. Indeed I talked to @powersj and we distilled the requirement down to this, which I think is identical to your statement. Can you please confirm and assuming we're aligned I will implement this strategy as a configuration option in the semantic validator.

Optional logic for Telegraf team:
If there is one commit, check only the commit message and not the PR title
If there is > 1 commit, check only the PR title

@reimda
Copy link
Contributor

reimda commented Apr 27, 2022

Optional logic for Telegraf team:
If there is one commit, check only the commit message and not the PR title
If there is > 1 commit, check only the PR title

That should do it.

Since this workflow is for PRs that target master, we should add an event filter. We don't want it to run, for example, on release branches. You can filter the pull_request event by using a 'branches' line:

on:
  pull_request:
    branches:
      - master # branch the PR will be merged to

tysonkamp added 2 commits May 9, 2022 09:17
…st here. That's been fixed, running again to validate test.
@powersj
Copy link
Contributor

powersj commented May 9, 2022

@tysonkamp looks like your semantic PR checker is happy with this PR. Are we good to review and merge?

@tysonkamp
Copy link
Contributor Author

tysonkamp commented May 9, 2022

I'd be delighted if it was merged. Let me know if there's anything I should do.

Hang on just a sec though, I did not implement the master branch only logic Dave had mentioned. Checking that in now, give it 5 minutes...

@tysonkamp
Copy link
Contributor Author

Checked in the condition to run when merging to master only. LG2M, semantics checker passed. Others pending.

@reimda reimda added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label May 9, 2022
@tysonkamp tysonkamp requested a review from reimda May 9, 2022 21:24
@reimda reimda merged commit 9c02a47 into master May 10, 2022
@reimda reimda deleted the add-influx-semantic-commits branch May 10, 2022 21:18
jotavalenciano added a commit to dgkanatsios/telegraf that referenced this pull request Jul 1, 2022
* fix: update go.opentelemetry.io/collector/pdata from v0.48.0 to v0.49.0 (influxdata#10984)

* docs: add missing slash in mongodb readme (influxdata#10994)

* chore: update pull request with master when running CI (influxdata#10993)

* chore: add readme linter (influxdata#10916)

* fix: correctly parse various numeric forms (influxdata#10923)

* fix: add mariadb_dialect to address the MariaDB differences in INNODB_METRICS (influxdata#10486)

* docs: correct influx parser type setting in README (influxdata#11004)

* fix: use correct auth token with consul_agent (influxdata#11001)

* docs: add openstack metadata external plugin (influxdata#10999)

* fix: use sprint to cast to strings in gnmi (influxdata#11010)

* chore: Fix readme linter errors for output plugins (influxdata#10951)

* fix: allow Makefile to work on Windows (influxdata#11015)

* fix(cmd): Also allow 0 outputs when using test-wait parameter (influxdata#11013)

* fix: add mutex to gnmi lookup map (influxdata#11008)

* fix: bump github.com/aws/aws-sdk-go-v2/config from 1.13.1 to 1.15.3 (influxdata#10998)

* Update changelog

(cherry picked from commit 234a448)

* fix: remove duplicate influxdb listener writes (influxdata#10976)

* fix: use external xpath parser for gnmi (influxdata#11024)

* chore: Adding influx's semantic commit and PR message checker, so we … (influxdata#11009)

* chore: Adding influx's semantic commit and PR message checker, so we can deprecate semantic-pull-requests

* feat: change commit history for semantic validation to 1 (last commit)

* chore: remove influxdata/validate-semantic-github-messages github workflow (influxdata#11036)

* feat: create and push nightly docker images to quay.io (influxdata#11000)

* fix: reduce log level in disk plugin (influxdata#10925)

* chore: increase timeout for darwin packaging (influxdata#11041)

* chore: enable linting of shell scripts (influxdata#11031)

* feat(outputs.http): Support configuration of `MaxIdleConns` and `MaxIdleConnsPerHost` (influxdata#10954)

* fix: datadog count metrics (influxdata#10979)

* Update changelog

(cherry picked from commit 53863d2)

* style: align plugin renaming (influxdata#10868)

* chore(inputs/disk): add deprecation notice to legacy mountpoints setting (influxdata#10948)

* fix: have telegraf service wait for network up (influxdata#11042)

* feat: add influx semantic commits checker, checks only last commit. (influxdata#11037)

* fix: re-init azure monitor http client on context deadline error (influxdata#11030)

* fix: do not error when closing statsd network connection (influxdata#11043)

* fix: deprecate useless database config option (influxdata#11044)

* fix(inputs.couchbase): Don't assume metrics will all be of the same length (influxdata#11045)

* fix(inputs.couchbase): Don't assume metrics will all be of the same length

* fix: move to one line

* feat(exec, execd): add an option to pass a custom environment to their child process (influxdata#11049)

* refactor: replace strings.Replace with strings.ReplaceAll (influxdata#11079)

* docs: correct copy-and-paste of udp to tcp (influxdata#11080)

* chore: update opentelemetry plugins (influxdata#11085)

* fix(outputs.Wavefront): If no "host" tag is provided, do not add "telegraf.host" tag (influxdata#11078)

Co-authored-by: ffaroo1 <[email protected]>

* fix: bump github.com/showwin/speedtest-go from 1.1.4 to 1.1.5 (influxdata#10722)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: MyaLongmire <[email protected]>

* docs: note about listsnapshots cause zfs parse failures (influxdata#11091)

* feat(inputs.vsphere): Collect resource pools metrics and add resource pool tag in VM metrics (influxdata#10574)

* Collect Resource Pool metrics; Add rpname tag on VM metrics

* Update vSphere readme file

* Update vSphere readme file

* Correct typo in vSphere Readme

* Correct Markdown of metrics.md

* Fix metrics file

* Fix code in endpoint (filter); add some tests

* Update plugins/inputs/vsphere/endpoint.go

That's true I commit this suggestion

Co-authored-by: Sebastian Spaink <[email protected]>

* Removed Context and Endpoint from getResourcePoolName func

Co-authored-by: Simon LAMBERT <[email protected]>
Co-authored-by: Sebastian Spaink <[email protected]>

* feat: add mount option filtering to disk plugin (influxdata#11039)

* Update changelog

(cherry picked from commit c07868f)

* fix: Output erroneous namespace and continue instead of error out (influxdata#11069)

* fix: check net.Listen() error in tests (influxdata#11093)

* docs: fix socket_writer output format link (influxdata#11101)

* feat: Artifactory Webhook Receiver (influxdata#10918)

* test: add test for mysql gatherGlobalVariables using sql-mock (influxdata#10987)

* chore(inputs/file): More clear error messages (influxdata#11104)

* fix: Update gopsutil from v3.22.3 to v3.22.4 to allow for HOST_PROC_MOUNTINFO. (influxdata#11107)

* fix: bump github.com/wavefronthq/wavefront-sdk-go from 0.9.10 to 0.9.11 (influxdata#10970)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: log instance name in skip warnings (influxdata#10995)

* feat(parsers/logfmt): Add tag support (influxdata#11060)

* feat: allow other fluentd metrics apart from retry_count, buffer_queu… (influxdata#11056)

* fix(parsers/nagios): metrics will always return a supported status co… (influxdata#11062)

Co-authored-by: Morten Urban <[email protected]>

* test: remove unused riemann from docker-compose (influxdata#11118)

* fix: elasticsearch output float handling test (influxdata#11120)

* test: remove unecessary flag in sql input (influxdata#11115)

* test: use supported version of elasticsearch (influxdata#11111)

* chore(processors): migrate sample configs into separate files (influxdata#11125)

* chore(aggregators): migrate sample configs into separate files (influxdata#11130)

* chore(outputs): migrate sample configs into separate files (influxdata#11131)

* chore(inputs_m-z): migrate sample configs into separate files (influxdata#11133)

* chore(inputs_a-l): migrate sample configs into separate files (influxdata#11132)

* fix: use readers over closers in http input (influxdata#11083)

* feat: add slab metrics input plugin (influxdata#11075)

Co-authored-by: reimda <[email protected]>
Co-authored-by: Joshua Powers <[email protected]>

* feat: do not error if no nodes found for current config with xpath parser (influxdata#11102)

* feat: [inputs/burrow] fill more http transport parameters (influxdata#6948)

Co-authored-by: Sven Rebhan <[email protected]>

* fix: avoid calling sadc with invalid 0 interval (influxdata#11140)

* fix(inputs.burrow): Move Dialer to variable and run `make fmt` (influxdata#11149)

* fix: doc interval setting for internet speed plugin (influxdata#11150)

* test: switch internet speed to enable file download (influxdata#11146)

* test: update mongodb output to use test containers (influxdata#11137)

* test: force elasticsearch to index batch data (influxdata#11153)

* docs: update review docs (influxdata#11147)

* fix: Improve slab testing without sudo. (influxdata#11151)

* feat: add external huebridge input plugin (influxdata#11159)

* test: add generic test-container code for re-use, migrate mysql (influxdata#11157)

* fix: bump github.com/aws/aws-sdk-go-v2/config from 1.15.3 to 1.15.7 (influxdata#11166)

* test: avoid data race in tcp-listener test (influxdata#11144)

* feat(intel_powerstat): add Max Turbo Frequency and introduce improvements (influxdata#11035)

* chore: add readme linter to CI (influxdata#11020)

* feat(inputs.cpu): Add tags with core id or physical id to cpus (influxdata#11141)

* fix: Remove any content type from prometheus accept header (influxdata#11082)

In influxdata#6745, the prometheus accept header was changed to accept any media
type. However, our prometheus plugin only accepts text. With the release
of newer versions of prometheus, the OpenMetrics type is now available
and could potentially be setup as the exclusive response type. As this
new content type is not supported, Telegraf should not accept it.

The original issue, influxdata#6523, was filed around getting a 406. The issue had
comments from the rabbit-mq maintainers who made changes to their code
to be less regid resolving the issue. The change to telegraf was made
afterwards anyway.

fixes: influxdata#10248

* fix: search services file in /etc/services and fall back to /usr/etc/services (influxdata#11179)

* chore: Embed sample configurations into README for inputs (influxdata#11136)

* test: migrate crate to test-containers code (influxdata#11165)

* test: migrate nats to test-containers (influxdata#11170)

* test: migrate mqtt to test containers (influxdata#11172)

* test: migrate redis to test-containers (influxdata#11174)

* test: migrate memcached to test-containers (influxdata#11176)

* fix: Convert slab plugin to new sample.conf. (influxdata#11181)

* test: migrate aerospike to test-containers (influxdata#11177)

* feat: add field key option to set event partition key (influxdata#11076)

* test: migrate opcua to test-containers (influxdata#11171)

* test: migrate nsq to test containers (influxdata#11173)

* test: migrate openldap tests to test-containers (influxdata#11169)

* feat: Google API Auth (influxdata#11084)

* chore: embed sample configurations into README for outputs (influxdata#11182)

* chore: Embed sample configurations into README for processors (influxdata#11189)

* chore: Embed sample configurations into README for aggregators (influxdata#11190)

* test: remove rabbitmq container not used (influxdata#11175)

* test: migrate pgbouncer to test-containers (influxdata#11186)

* fix(inputs/snmp): Reconnect TCP agents if needed (influxdata#11163)

* fix: update golangci-lint from v1.45.2 to v1.46.2 (influxdata#11191)

* fix: redis plugin goroutine leak triggered by auto reload config mechanism (influxdata#11143)

* feat: Add constant 'algorithm' to the mock plugin (influxdata#11188)

* fix(plugins/amqp): move from `streadway/amqp` to `rabbitmq/amqp091-go` (influxdata#11192)

* test: migrate zookeeper to test-containers (influxdata#11185)

* fix: bump github.com/sensu/sensu-go/api/core/v2 from 2.13.0 to 2.14.0 (influxdata#11021)

* docs: remove recommendation for prometheus metric_version=2 (influxdata#11158)

* chore: update OpenTelmetry plugins (influxdata#11194)

* fix: bump go.opentelemetry.io/otel/metric from 0.28.0 to 0.30.0 (influxdata#11088)

* chore: correctly spell embed (influxdata#11200)

* test: actually skip flaky snmp test (influxdata#11199)

This test had a short skip added to it four years ago and a comment that
the test has random failures. While working on the integration tests,
which run all tests this test started showing up in the errors.

While the two asserts at the end could possibly be updated, it is worth
having someone look deeper into understanding why this change is
required.

In order to get integration tests running, this skips this test always.

* test: refactor testcontainer port lookup (influxdata#11198)

* fix(inputs/snmp): switch new Reconnect method to be a value receiver (influxdata#11197)

* chore: update gopkg.in/yaml.v3 from v3.0.0 to v3.0.1 (influxdata#11213)

* chore: update github.com/opencontainers/runc from v1.0.2 to v1.1.2 (influxdata#11212)

* test: enable logging with testcontainers (influxdata#11211)

* fix: bump github.com/nats-io/nats-server/v2 from 2.7.4 to 2.8.4 (influxdata#11221)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Sebastian Spaink <[email protected]>

* fix: Update sample.conf for prometheus (influxdata#11217)

* fix: (outputs/sql) table existence cache (influxdata#10812)

* fix(inputs/modbus): influxdata#11105 fix requests starting with an omitted field (influxdata#11202)

* feat(inputs.nginx_plus_api): Gather slab metrics (influxdata#10448)

* fix: Upgrade xpath and fix code (influxdata#11241)

* chore: Bump riemann-go-client and remove replacement (influxdata#11236)

* docs: Initial update of integration test docs (influxdata#11210)

* test: migrate kafka to testcontainers (influxdata#11206)

* test: migrate mcrouter to test-containers (influxdata#11208)

* test: migrate postgres to testcontainers (influxdata#11209)

* chore: Bump apcupds and remove the replacement as issue is fixed upstream. (influxdata#11239)

* test: migrate elasticsearch to testcontainers (influxdata#11207)

* feat(inputs.sqlserver): Update query store and latch performance counters (influxdata#11216)

* fix: update moby/ipvs dependency from v1.0.1 to v1.0.2 (influxdata#11242)

* test: remove docker-compose.yml test file (influxdata#11243)

* docs: update test-container docs (influxdata#11244)

* fix: re-add event to splunk serializer (influxdata#11237)

* chore: upgrade windows circleci size (influxdata#11249)

* chore: rename circleci executor (influxdata#11247)

* chore: update go from v1.18.1 to v1.18.3 (influxdata#11248)

* test: harden running of testcontainer integration tests (influxdata#11245)

* test: add circle ci integration testing on PRs (influxdata#11246)

* test: remove restore integration test cache (influxdata#11255)

* feat(intel_powerstat): add uncore frequency metrics (influxdata#11254)

* docs: add missing word <3 (influxdata#11262)

* fix: update modernc.org/sqlite from v1.10.8 to v1.17.3 (influxdata#11260)

* chore: Fix readme linter errors for processor, aggregator, and parser plugins (influxdata#10960)

* fix: bump github.com/tidwall/gjson from 1.10.2 to 1.14.1 (influxdata#11264)

* Bump github.com/tidwall/gjson from v1.10.2 to v1.14.1

* Fix node references which are switched to parent relative now.

* fix: update github.com/containerd/containerd from v1.5.11 to v1.5.13 (influxdata#11266)

* chore: Fix readme linter errors for input plugins A-D (influxdata#10964)

* chore: Fix readme linter errors for input plugins E-L (influxdata#11214)

* fix: remove full access permissions (influxdata#11261)

* fix: add missing build constraints for sqlite (influxdata#11272)

* feat: Migrate xpath parser to new style (influxdata#11218)

* fix: Always build README-embedder for host-architecture (influxdata#11253)

* chore: Fix readme linter errors for input plugins M-Z (influxdata#11274)

* fix(inputs/directory_monitor): Add support for multiline file parsing (influxdata#11234)

* test: add install go for linux, use in integration tests (influxdata#11281)

* Update build version to 1.24.0

* Update changelog for v1.23.0

(cherry picked from commit 7317a81)

* fix: Don't rebase on master when building packages (influxdata#11291)

* fix: Remove all rebase logic from CI (influxdata#11293)

* fix: bump cloud.google.com/go/monitoring from 1.2.0 to 1.5.0 (influxdata#11295)

* feat(inputs.x509_cert): add smtp protocol (influxdata#11271)

Co-authored-by: dreiekk <[email protected]>

* fix(parsers/xpath): Reduce debug messages when empty selection is allowed (influxdata#11302)

* feat: add default appType as config option to groundwork output (influxdata#11300)

* feat: Make the command "config" a subcommand (influxdata#11282)

* feat: make the command "config" a subcommand

* fix: backwards compatible
support telegraf *filters* config

* fix: Prevent concurrent map writes to c.UnusedFields (influxdata#11311)

* docs: explain directly connecting to mongo node (influxdata#11314)

* chore: Remove prefix to use the default `chore(deps)` (influxdata#11315)

* test: update wait for statement for postgres (influxdata#11309)

* test: mark integration tests correctly (influxdata#11317)

* chore(deps): Bump github.com/aws/aws-sdk-go-v2/credentials from 1.12.2 to 1.12.5 (influxdata#11297)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): Bump google.golang.org/grpc from 1.46.2 to 1.47.0 (influxdata#11318)

* chore(deps): Bump k8s.io/client-go from 0.23.3 to 0.24.1 (influxdata#11223)

* chore(deps): Bump github.com/go-logfmt/logfmt from 0.5.0 to 0.5.1 (influxdata#11299)

* chore: make apt-get instructions consistent and add GPG fingerprint (influxdata#11326)

* test: add coveralls coverage to master (influxdata#11256)

* chore(deps): Bump github.com/aws/aws-sdk-go-v2/service/dynamodb (influxdata#11328)

* chore(deps): Bump go.mongodb.org/mongo-driver from 1.9.0 to 1.9.1 (influxdata#11320)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): Bump github.com/gophercloud/gophercloud from 0.24.0 to 0.25.0 (influxdata#11321)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): Bump google.golang.org/api from 0.74.0 to 0.84.0 (influxdata#11338)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: MyaLongmire <[email protected]>

* chore: fix typo (recieve -> receive) (influxdata#11341)

* chore(deps): Bump github.com/fatih/color from 1.10.0 to 1.13.0 (influxdata#11340)

* fix: filter out views in mongodb lookup (influxdata#11280)

* chore(deps): Bump github.com/aws/aws-sdk-go-v2/service/timestreamwrite from 1.3.2 to 1.13.6 (influxdata#11322)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* docs: Update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#11344)

Co-authored-by: Tiger Bot <>

* feat(redis): add Redis 6 ACL auth support (influxdata#9333)

* feat(x509_cert): add proxy support (influxdata#9319)

* chore(deps): Bump github.com/Shopify/sarama from 1.32.0 to 1.34.1 (influxdata#11319)

* docs: Sync sample.conf for recent features (influxdata#11348)

* chore: move agent config into seperate file (influxdata#11337)

* feat: Migrate json parser to new style (influxdata#11226)

* docs: Update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#11351)

Co-authored-by: Tiger Bot <>
Co-authored-by: Sebastian Spaink <[email protected]>

* fix: don't require listeners to be present in overview (influxdata#9315)

* feat: Add CSV serializer (influxdata#11307)

* fix: Restore sample configurations broken during initial migration (influxdata#11276)

* feat: Migrate json_v2 parser to new style (influxdata#11343)

* docs: Update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#11365)

Co-authored-by: Tiger Bot <>

* chore(deps): Bump github.com/dynatrace-oss/dynatrace-metric-utils-go from 0.3.0 to 0.5.0 (influxdata#11342)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): Bump github.com/nats-io/nats.go from 1.15.0 to 1.16.0 (influxdata#11339)

* chore(deps): Bump cloud.google.com/go/pubsub from 1.18.0 to 1.22.2 (influxdata#11349)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* docs: swap bytes sent/recv descriptions in nfsclient (influxdata#11376)

* chore(deps): Bump go.opentelemetry.io/collector/pdata from 0.52.0 to 0.54.0 (influxdata#11369)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): Bump github.com/jackc/pgx/v4 from 4.15.0 to 4.16.1 (influxdata#11346)

* chore(deps): Bump cloud.google.com/go/bigquery from 1.8.0 to 1.33.0 (influxdata#11379)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): Bump github.com/Azure/azure-kusto-go from 0.6.0 to 0.7.0 (influxdata#11378)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: migrate wavefront parser to new style (influxdata#11374)

* feat: allow collecting node-level metrics for Couchbase buckets (influxdata#9717)

* feat: Migrate collectd parser to new style (influxdata#11367)

* chore(deps): Bump cloud.google.com/go/pubsub from 1.22.2 to 1.23.0 (influxdata#11394)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): Bump github.com/aws/aws-sdk-go-v2/service/kinesis (influxdata#11380)

* chore: Remove 'github.com/satori/go.uuid' replacement (influxdata#11240)

* chore: Remove 'github.com/cisco-ie/nx-telemetry-proto' replacement (influxdata#11401)

* chore(deps): Bump github.com/aws/aws-sdk-go-v2/service/ec2 from 1.1.0 to 1.46.0 (influxdata#11382)

* test: add nightly package testing to amd64/linux (influxdata#11377)

* fix: resolve jolokia2 panic on null response (influxdata#11397)

* fix: Sync back sample.confs for inputs.couchbase and outputs.groundwork. (influxdata#11413)

* docs: Update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#11414)

Co-authored-by: Tiger Bot <>

* chore(deps): Bump github.com/golang-jwt/jwt/v4 from 4.4.1 to 4.4.2 (influxdata#11395)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): Bump github.com/vmware/govmomi from 0.27.3 to 0.28.0 (influxdata#11396)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Sven Rebhan <[email protected]>

* chore: move printing of sample config file out of config.go (influxdata#11409)

* feat: migrate form_urlencoded parser to new style (influxdata#11381)

* feat(outputs/wavefront): make maximum http batch size configurable (influxdata#11201)

* docs: Update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#11419)

Co-authored-by: Tiger Bot <>

* feat: migrate value parser to new style (influxdata#11407)

* feat: Migrate graphite parser to new style (influxdata#11405)

* fix: Bring back old xpath section names (influxdata#11335)

* feat: Migrate logfmt parser to new style (influxdata#11366)

* test: exchange confused parameters (expected vs actual) (influxdata#11422)

* chore(deps): Bump github.com/aws/aws-sdk-go-v2/service/cloudwatchlogs from 1.15.4 to 1.15.8 (influxdata#11415)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: Migrate dropwizard parser to new style (influxdata#11371)

* feat: migrate grok to new parser style (influxdata#11408)

* chore(deps): Bump github.com/influxdata/influxdb-observability/otel2influx from 0.2.21 to 0.2.22 (influxdata#11416)

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* test: run check-update on dnf/yum (influxdata#11430)

* feat: adding aws metric streams input plugin (influxdata#11233)

* docs: Update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#11431)

Co-authored-by: Tiger Bot <>

* Remove added modules

* Add again missing modules

* Fix README for compliance

* Reduce lines length

* Fix trailing spaces and indentation

* Fix code warnings

* Update go sum

* Update licenses for dependencies

* Fix race in test

Co-authored-by: Jacob Marble <[email protected]>
Co-authored-by: Adam Zwakenberg <[email protected]>
Co-authored-by: Sebastian Spaink <[email protected]>
Co-authored-by: reimda <[email protected]>
Co-authored-by: Joshua Powers <[email protected]>
Co-authored-by: Marc <[email protected]>
Co-authored-by: Alexander Kapshuna <[email protected]>
Co-authored-by: Adam Rowan <[email protected]>
Co-authored-by: Dmitry Lavrukhin <[email protected]>
Co-authored-by: bewing <[email protected]>
Co-authored-by: Thomas Casteleyn <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Sebastian Spaink <[email protected]>
Co-authored-by: Tyson Kamp <[email protected]>
Co-authored-by: RaviKiran K <[email protected]>
Co-authored-by: Jimmy Rimmer <[email protected]>
Co-authored-by: M.D <[email protected]>
Co-authored-by: Eng Zer Jun <[email protected]>
Co-authored-by: Peter (Stig) Edwards <[email protected]>
Co-authored-by: Farukh Ali <[email protected]>
Co-authored-by: ffaroo1 <[email protected]>
Co-authored-by: MyaLongmire <[email protected]>
Co-authored-by: 6monlambert <[email protected]>
Co-authored-by: Simon LAMBERT <[email protected]>
Co-authored-by: Nathan Ferch <[email protected]>
Co-authored-by: Sergey Lanzman <[email protected]>
Co-authored-by: lambdaq <[email protected]>
Co-authored-by: sammcadams-8451 <[email protected]>
Co-authored-by: Felix Edelmann <[email protected]>
Co-authored-by: Sven Rebhan <[email protected]>
Co-authored-by: Kuldeep Doneriya <[email protected]>
Co-authored-by: Sakerdotes <[email protected]>
Co-authored-by: Morten Urban <[email protected]>
Co-authored-by: Nobuhiro MIKI <[email protected]>
Co-authored-by: Sokolov Yura <[email protected]>
Co-authored-by: Holger <[email protected]>
Co-authored-by: bkotlowski <[email protected]>
Co-authored-by: Martin Molnar <[email protected]>
Co-authored-by: Heiko Schlittermann <[email protected]>
Co-authored-by: crflanigan <[email protected]>
Co-authored-by: zhiyuan-mojie <[email protected]>
Co-authored-by: Ruoshan Huang <[email protected]>
Co-authored-by: TimurDela <[email protected]>
Co-authored-by: glennlod <[email protected]>
Co-authored-by: David Barbarin <[email protected]>
Co-authored-by: Jan-Gerd Tenberge <[email protected]>
Co-authored-by: skillor <[email protected]>
Co-authored-by: dreiekk <[email protected]>
Co-authored-by: Vladislav Senkevich <[email protected]>
Co-authored-by: Jamie Strandboge <[email protected]>
Co-authored-by: Julien Pivotto <[email protected]>
Co-authored-by: telegraf-tiger[bot] <76974415+telegraf-tiger[bot]@users.noreply.github.com>
Co-authored-by: Alexander Krantz <[email protected]>
Co-authored-by: papapiya <[email protected]>
Co-authored-by: Luke Winikates <[email protected]>
Co-authored-by: Cillian McCabe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants