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

20211015 Telegraf - defaults - experimental branch - PR 3 of 3 #432

Conversation

Paraphraser
Copy link

Changes telegraf Dockerfile so that the configuration for sending
metrics to InfluxDB uses the "internal network" URL:

http://influxdb:8086

On a clean install, Telegraf should communicate with InfluxDB
"out of the box".

Acknowledgement: this problem was discovered by
Discord user "tablatronix"

Changes `telegraf` Dockerfile so that the configuration for sending
metrics to InfluxDB uses the "internal network" URL:

```
http://influxdb:8086
```

On a clean install, Telegraf should communicate with InfluxDB
"out of the box".

Acknowledgement: this problem was discovered by
[Discord user "tablatronix"](https://discord.com/channels/638610460567928832/638610461109256194/898349626179076096)
Following on from a [Discord discussion](https://discord.com/channels/638610460567928832/638610461109256194/898730354507137054) with @tablatronix:

1. Add a read-only volume mapping for Docker socket.
2. Restructure the defaults configuration scheme to support "additions",
comprising two files:

	- `inputs.docker.conf` - provided by @tablatronix
	- `inputs.mqtt_consumer.conf` - recovered from the original gcgarner
	default configuration file.

Rather than a ".bak" file, the reference version of the telegraf
configuration file (the one that ships with the DockerHub image) is
provided as:

```
~/IOTstack/volumes/telegraf/telegraf-release.conf
```

Documentation updated on master branch.
@Slyke Slyke merged commit ce0b4e2 into SensorsIot:experimental Oct 17, 2021
@Paraphraser Paraphraser deleted the 20211015-telegraf-influx-container-experimental branch October 17, 2021 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants