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

InfluxDB configuration files #1124

Merged
merged 6 commits into from
Jun 22, 2020
Merged

InfluxDB configuration files #1124

merged 6 commits into from
Jun 22, 2020

Conversation

sanderson
Copy link
Collaborator

Closes #1038
Closes #1113
Closes #1120

Restructured the Configuration options doc and updated the influxd docs to show alternate configuration methods:

  1. CLI flags
  2. Environment variables
  3. A configuration file
  • Tests pass (no build errors)
  • Rebased/mergeable

@sanderson sanderson added this to the beta-13 milestone Jun 18, 2020
| | `--session-renew-disabled` | Disables automatically extending session TTL on request | | `INFLUXD_SESSION_RENEW_DISABLED` |
| | `--store` | Data store for REST resources (bolt or memory) (default `bolt`) | string | `INFLUXD_STORE` |
| | `--tls-cert` | Path to TLS certificate file | string | `INFLUXD_TLS_CERT` |
| | `--tls-key` | Path to TLS private key file | string | `INFLUXD_TLS_KEY` |
Copy link
Contributor

Choose a reason for hiding this comment

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

very nice 👏, I see questions about this in the community every so often 👍

Copy link
Contributor

@jsteenb2 jsteenb2 left a comment

Choose a reason for hiding this comment

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

its just so....
can't beleive it


### InfluxDB configuration file
When `influxd` starts, it checks for a **`config.yaml` in the current working directory**.
To customize the file name or path of the configuration file, set the `INFLUXD_CONFIG_FILE`
Copy link
Contributor

@russorat russorat Jun 18, 2020

Choose a reason for hiding this comment

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

Is there no way to set this via a command line flag? influxd --config /path/to/my/config.yml?

Copy link
Contributor

Choose a reason for hiding this comment

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

the 1.x version of this is INFLUXDB_CONFIG_PATH

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

In talking with @jsteenb2, the presents the chicken and the egg problem.

Copy link
Contributor

Choose a reason for hiding this comment

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

yah its a doozie. @russorat made a card for this. It'll likely require a larger refactor of our cli pkg to make this a reality. bit of a bummer. Was hoping to include this in the initial work but was just to much headache

@sanderson sanderson requested a review from jsteenb2 June 19, 2020 01:28
Copy link
Contributor

@jsteenb2 jsteenb2 left a comment

Choose a reason for hiding this comment

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

this is fantastic 👏. I want to drop this in the #showme channels community/comp when this documentation lands 🙃

Copy link
Contributor

@russorat russorat left a comment

Choose a reason for hiding this comment

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

👍

---

Customize your InfluxDB configuration by using configuration flags to the
Copy link
Contributor

Choose a reason for hiding this comment

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

using >> sending/passing?

content/v2.0/reference/config-options.md Outdated Show resolved Hide resolved
content/v2.0/reference/config-options.md Outdated Show resolved Hide resolved
content/v2.0/reference/config-options.md Show resolved Hide resolved
| | `--e2e-testing` | Add /debug/flush endpoint to clear stores; used for end-to-end tests (default `false`) | | `INFLUXD_E2E_TESTING` |
| | `--engine-path` | Path to persistent engine files (default `~/.influxdbv2/engine`) | string | `INFLUXD_ENGINE_PATH` |
| `-h` | `--help` | Help for the `influxd` command | | |
| | `--http-bind-address` | Bind address for the REST HTTP API (default `:9999`) | string | `INFLUXD_HTTP_BIND_ADDRESSBIND_ADDRESS` |
Copy link
Contributor

Choose a reason for hiding this comment

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

extra "BIND_ADDRESS"?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh, nice catch! Thanks!

@sanderson sanderson merged commit 2b4f7db into beta-13 Jun 22, 2020
@sanderson sanderson deleted the influxd-conf-files branch June 22, 2020 19:31
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.

5 participants