telegraf icon indicating copy to clipboard operation
telegraf copied to clipboard

Published 20 hours ago • verified publisher icon influxdata

feat(inputs.cisco_telemetry_mdt): Include delete field

Open severindellsperger opened this issue 2 years ago • 1 comments

Required for all PRs

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

resolves #12286

Implement the logic to include the delete field as discussed in #12286 to support Event-Driven Telemetry (EDT).

severindellsperger avatar Dec 06 '22 16:12 severindellsperger

@severindellsperger, did you ask Cisco to host the repo publicly? Have they acknowledged the request?

powersj avatar Jan 10 '23 14:01 powersj

@severindellsperger, did you ask Cisco to host the repo publicly? Have they acknowledged the request? I have no answer yet. I have a meeting next week and will discuss it then.

severindellsperger avatar Jan 11 '23 14:01 severindellsperger

The PR is out: https://github.com/cisco-ie/nx-telemetry-proto/pull/2 Let's hope it will be merged soon so that we can adjust.

severindellsperger avatar Jan 17 '23 14:01 severindellsperger

The PR is out: cisco-ie/nx-telemetry-proto#2 Let's hope it will be merged soon so that we can adjust.

Looks like that got landed. Thank you for doing that work!

powersj avatar Jan 17 '23 18:01 powersj

@powersj @srebhan I made the necessary changes. I think the PR is now ready for the final review.

severindellsperger avatar Jan 18 '23 10:01 severindellsperger

Oh and you should run make docs...

srebhan avatar Jan 18 '23 17:01 srebhan

Download PR build artifacts for linux_amd64.tar.gz, darwin_amd64.tar.gz, and windows_amd64.zip. Downloads for additional architectures and packages are available below.

:relaxed: This pull request doesn't significantly change the Telegraf binary size (less than 1%)

:package: Click here to get additional PR build artifacts

Artifact URLs

DEB RPM TAR GZ ZIP
amd64.deb aarch64.rpm darwin_amd64.tar.gz windows_amd64.zip
arm64.deb armel.rpm darwin_arm64.tar.gz windows_arm64.zip
armel.deb armv6hl.rpm freebsd_amd64.tar.gz windows_i386.zip
armhf.deb i386.rpm freebsd_armv7.tar.gz
i386.deb ppc64le.rpm freebsd_i386.tar.gz
mips.deb riscv64.rpm linux_amd64.tar.gz
mipsel.deb s390x.rpm linux_arm64.tar.gz
ppc64el.deb x86_64.rpm linux_armel.tar.gz
riscv64.deb linux_armhf.tar.gz
s390x.deb linux_i386.tar.gz
linux_mips.tar.gz
linux_mipsel.tar.gz
linux_ppc64le.tar.gz
linux_riscv64.tar.gz
linux_s390x.tar.gz
static_linux_amd64.tar.gz

telegraf-tiger[bot] avatar Jan 19 '23 11:01 telegraf-tiger[bot]