Skip to content

feat(http): support aws prometheus#10202

Merged
powersj merged 1 commit intoinfluxdata:masterfrom
baloo:baloo/aws-prometheus
Dec 10, 2021
Merged

feat(http): support aws prometheus#10202
powersj merged 1 commit intoinfluxdata:masterfrom
baloo:baloo/aws-prometheus

Conversation

@baloo
Copy link
Copy Markdown
Contributor

@baloo baloo commented Dec 2, 2021

Required for all PRs:

This adds support to write to Amazon Managed Service for Prometheus.
Writing to AWS Prometheus requires queries to be signed with AWS v4 signatures, this PR adds support for that.

@telegraf-tiger
Copy link
Copy Markdown
Contributor

telegraf-tiger bot commented Dec 2, 2021

Thanks so much for the pull request!
🤝 ✒️ Just a reminder that the CLA has not yet been signed, and we'll need it before merging. Please sign the CLA when you get a chance, then post a comment here saying !signed-cla

@baloo baloo changed the title http: support aws prometheus feat(http): support aws prometheus Dec 2, 2021
@baloo
Copy link
Copy Markdown
Contributor Author

baloo commented Dec 3, 2021

Individual CLA signed, Corporate CLA should have been amended to include me yesterday.

@sspaink
Copy link
Copy Markdown
Contributor

sspaink commented Dec 3, 2021

!signed-cla

@sspaink
Copy link
Copy Markdown
Contributor

sspaink commented Dec 3, 2021

@baloo feature looks good to me, there is a minor linter issue that needs to be resolved before this can be merged.

@sspaink sspaink 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 Dec 3, 2021
@baloo baloo force-pushed the baloo/aws-prometheus branch from 921c330 to ab0941f Compare December 4, 2021 17:10
@baloo
Copy link
Copy Markdown
Contributor Author

baloo commented Dec 5, 2021

@sspaink last lint error is currently in master afaict. Do you want me to fix this here?

@sspaink
Copy link
Copy Markdown
Contributor

sspaink commented Dec 6, 2021

Looks like it has been fixed in master already with this PR: #10159, you will have to rebase this pr on master.

@baloo
feat: support aws managed service for prometheus
ced1c2b 
Aws prometheus needs aws signature support.
@baloo baloo force-pushed the baloo/aws-prometheus branch from ab0941f to ced1c2b Compare December 7, 2021 06:50
@telegraf-tiger
Copy link
Copy Markdown
Contributor

telegraf-tiger bot commented Dec 7, 2021

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

📦 Looks like new artifacts were built from this PR.

Expand this list to get them here ! 🐯

Artifact URLs

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

@powersj powersj merged commit 8976483 into influxdata:master Dec 10, 2021
@baloo baloo deleted the baloo/aws-prometheus branch December 11, 2021 07:28
powersj pushed a commit to powersj/telegraf that referenced this pull request Jan 21, 2022
@baloo @powersj
feat: support aws managed service for prometheus (influxdata#10202)
5650cd3
phemmer added a commit to phemmer/telegraf that referenced this pull request Feb 18, 2022
@phemmer
Merge remote-tracking branch 'origin/master' into postgres
ddb5575 
* origin/master: (133 commits)
  chore: restart service if it is already running and upgraded via RPM (influxdata#9970)
  feat: update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#10237)
  fix: Handle duplicate registration of protocol-buffer files gracefully. (influxdata#10188)
  fix(http_listener_v2): fix panic on close (influxdata#10132)
  feat: add Vault input plugin (influxdata#10198)
  feat: support aws managed service for prometheus (influxdata#10202)
  fix: Make telegraf compile on Windows with golang 1.16.2 (influxdata#10246)
  Update changelog
  feat: Modbus add per-request tags (influxdata#10231)
  fix: Implement NaN and inf handling for elasticsearch output (influxdata#10196)
  feat: add nomad input plugin (influxdata#10106)
  fix: Print loaded plugins and deprecations for once and test (influxdata#10205)
  fix: eliminate MIB dependency for ifname processor (influxdata#10214)
  feat: Optimize locking for SNMP MIBs loading. (influxdata#10206)
  feat: Add SMART plugin concurrency configuration option, nvme-cli v1.14+ support and lint fixes. (influxdata#10150)
  feat: update configs (influxdata#10236)
  fix(inputs/kube_inventory): set TLS server name config properly (influxdata#9975)
  fix: Sudden close of Telegraf caused by OPC UA input plugin (influxdata#10230)
  fix: bump github.com/eclipse/paho.mqtt.golang from 1.3.0 to 1.3.5 (influxdata#9913)
  fix: json_v2 parser timestamp setting (influxdata#10221)
  fix: ensure graylog spec fields not prefixed with '_' (influxdata#10209)
  docs: remove duplicate links in CONTRIBUTING.md (influxdata#10218)
  fix: pool detection and metrics gathering for ZFS >= 2.1.x (influxdata#10099)
  fix: parallelism fix for ifname processor (influxdata#10007)
  chore: Forbids "log" package only for aggregators, inputs, outputs, parsers and processors (influxdata#10191)
  docs: address documentation gap when running telegraf in k8s (influxdata#10215)
  feat: update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#10211)
  fix: mqtt topic extracting no longer requires all three fields (influxdata#10208)
  fix: windows service - graceful shutdown of telegraf (influxdata#9616)
  feat: update etc/telegraf.conf and etc/telegraf_windows.conf (influxdata#10201)
  feat: Modbus support multiple slaves (gateway feature) (influxdata#9279)
  fix: Revert unintented corruption of the Makefile from influxdata#10200. (influxdata#10203)
  chore: remove triggering update-config bot in CI (influxdata#10195)
  Update changelog
  feat: Implement deprecation infrastructure (influxdata#10200)
  fix: extra lock on init for safety (influxdata#10199)
  fix: resolve influxdata#10027 (influxdata#10112)
  fix: register bigquery to output plugins influxdata#10177 (influxdata#10178)
  fix: sysstat use unique temp file vs hard-coded (influxdata#10165)
  refactor: snmp to use gosmi (influxdata#9518)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@sspaink sspaink sspaink approved these changes

@powersj powersj powersj approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@baloo @sspaink @powersj