61 Commits (967a1de1ed3540d62e7939151a8246d9ec81c8e4)

Author	SHA1	Message	Date
James Phillips	98e837167e	Changes maps to merge vs. overwrite when processing configs. ... Fixes #3716	7 years ago
James Phillips	46742a5041	Adds TODOs referencing #3744.	7 years ago
James Phillips	2892f91d0b	Copies the autopilot settings from the runtime config. ... Fixes #3730	7 years ago
James Phillips	44d824a58f	Renames "segments" to "segment" to be consistent with other files.	7 years ago
James Phillips	34c13925d4	Skips files with unknown extensions when not forcing a format. ... Fixes #3685	7 years ago
James Phillips	1e49c157c5	Adds a snapshot agent stub to the config structure. ... Fixes #3678	7 years ago
James Phillips	93f68555d0	Adds enable_agent_tls_for_checks configuration option which allows (#3661) ... HTTP health checks for services requiring 2-way TLS to be checked using the agent's credentials.	7 years ago
Frank Schröder	874e350b2f	config: add -config-format option (#3626) ... * config: refactor ReadPath(s) methods without side-effects Return the sources instead of modifying the state. * config: clean data dir before every test * config: add tests for config-file and config-dir * config: add -config-format option Starting with Consul 1.0 all config files must have a '.json' or '.hcl' extension to make it unambigous how the data should be parsed. Some automation tools generate temporary files by appending a random string to the generated file which obfuscates the extension and prevents the file type detection. This patch adds a -config-format option which can be used to override the auto-detection behavior by forcing all config files or all files within a config directory independent of their extension to be interpreted as of this format. Fixes #3620	7 years ago
Frank Schroeder	2e7ed2fd86	Merge pull request #3585 from hashicorp/document-runtime-config ... Moving the previous `agent/config.go` documentation to `agent/config/runtime.go`.	7 years ago
Frank Schroeder	5bfb2808f9	Merge pull request #3598 from hashicorp/issue-3397-error-with-extra-flags ... The `consul agent` command was ignoring extra command line arguments which can lead to confusion when the user has for example forgotten to add a dash in front of an argument or is not using an `=` when setting boolean flags to `true`. `-bootstrap true` is not the same as `-bootstrap=true`, for example. Since all command line flags are known and we don't expect unparsed arguments we can return an error. However, this may make it slightly more difficult in the future if we ever wanted to have these kinds of arguments. Fixes #3397	7 years ago
Frank Schroeder	b97ab367f4	config: return error on extra command line arguments (#3397) ... The `consul agent` command was ignoring extra command line arguments which can lead to confusion when the user has for example forgotten to add a dash in front of an argument or is not using an `=` when setting boolean flags to `true`. `-bootstrap true` is not the same as `-bootstrap=true`, for example. Since all command line flags are known and we don't expect unparsed arguments we can return an error. However, this may make it slightly more difficult in the future if we ever wanted to have these kinds of arguments. Fixes #3397	7 years ago
Frank Schroeder	1fef7f4b67	config: rename test struct field to args	7 years ago
Frank Schroeder	f6e9ad99ec	config: address review comments	7 years ago
Frank Schroeder	0ed4561f12	config: document remaining config options	7 years ago
Frank Schroeder	9864609201	config: document more config options	7 years ago
Frank Schroeder	bf81cdea6e	config: document more config options	7 years ago
Frank Schroeder	2682ce0b82	config: document more acl options	7 years ago
Frank Schroeder	a38c69d784	config: document config options	7 years ago
Frank Schroeder	7396bd31fd	config: document acl options	7 years ago
Frank Schroeder	7685ef409c	config: document autopilot options	7 years ago
Frank Schroeder	5e57e9273e	config: document dns options	7 years ago
Frank Schroeder	5f59857448	config: document http options	7 years ago
Frank Schroeder	7fede4472d	config: document telemetry options	7 years ago
Frank Schroeder	21a7d399bd	config: address review comments	7 years ago
Frank Schroeder	cf0a571a76	config: document remaining config options	7 years ago
Frank Schroeder	149ab13a13	config: document more config options	7 years ago
Frank Schroeder	a8f709a875	config: document more config options	7 years ago
Frank Schroeder	6c64cf9b5d	config: document more acl options	7 years ago
Frank Schroeder	ffb0f6ec8b	config: document config options	7 years ago
Frank Schroeder	70270d6d98	config: document acl options	7 years ago
Frank Schroeder	7f214b1e1c	config: document autopilot options	7 years ago
Frank Schroeder	cfc891fc25	config: document dns options	7 years ago
Frank Schroeder	d27617c60b	config: document http options	7 years ago
Frank Schroeder	625713dd72	config: document telemetry options	7 years ago
Frank Schroeder	1db8d3cb00	config: do not allow an ANY address as DNS recursor	7 years ago
Frank Schroeder	72bee6284d	config: add support for go-sockaddr templates for DNS recursors ... DNS recursors can be added through go-sockaddr templates. Entries are deduplicated while the order is maintained. Originally proposed by @taylorchu See #2932	7 years ago
Frank Schroeder	8a9c81ed85	config: do not allow an ANY address as DNS recursor	7 years ago
Frank Schroeder	0a0d6af04d	config: add support for go-sockaddr templates for DNS recursors ... DNS recursors can be added through go-sockaddr templates. Entries are deduplicated while the order is maintained. Originally proposed by @taylorchu See #2932	7 years ago
Frank Schroeder	6780964eb8	config: remove redundant code	7 years ago
Frank Schroeder	0faff32c73	config: fix check for segment.port <= 0 and add test	7 years ago
James Phillips	16eb2ef014	Adds check to make sure port is given so we avoid a nil bind address.	7 years ago
Frank Schröder	94f58199b1	agent: add option to discard health output (#3562) ... * agent: add option to discard health output In high volatile environments consul will have checks with "noisy" output which changes every time even though the status does not change. Since the output is stored in the raft log every health check update unblocks a blocking call on health checks since the raft index has changed even though the status of the health checks may not have changed at all. By discarding the output of the health checks the users can choose a different tradeoff. Less visibility on why a check failed in exchange for a reduced change rate on the raft log. * agent: discard output also when adding a check * agent: add test for discard check output * agent: update docs * go vet * Adds discard_check_output to reloadable config table. * Updates the change log.	7 years ago
Frank Schröder	759ef8a1d4	config: add generic method to translate between CamelCase and snake_case (#3557) ... * doc: document discrepancy between id and CheckID * doc: document enable_tag_override change * config: add TranslateKeys helper TranslateKeys makes it easier to map between different representations of internal structures. It allows to recursively map alias keys to canonical keys in structured maps. * config: use TranslateKeys for config file This also adds support for 'enabletagoverride' and removes the need for a separate CheckID alias field. * config: remove dead code * agent: use TranslateKeys for FixupCheckType * agent: translate enable_tag_override during service registration * doc: add '.hcl' as valid extension * config: map ScriptArgs to args * config: add comment for TranslateKeys	7 years ago
James Phillips	bb12368eac	Makes RPC handling more robust when rolling servers. (#3561) ... * Adds client-side retry for no leader errors. This paves over the case where the client was connected to the leader when it loses leadership. * Adds a configurable server RPC drain time and a fail-fast path for RPCs. When a server leaves it gets removed from the Raft configuration, so it will never know who the new leader server ends up being. Without this we'd be doomed to wait out the RPC hold timeout and then fail. This makes things fail a little quicker while a sever is draining, and since we added a client retry AND since the server doing this has already shut down and left the Serf LAN, clients should retry against some other server. * Makes the RPC hold timeout configurable. * Reorders struct members. * Sets the RPC hold timeout default for test servers. * Bumps the leave drain time up to 5 seconds. * Robustifies retries with a simpler client-side RPC hold. * Reverts untended delete.	7 years ago
Victor Boivie	8e361beb7a	Minor typo (boostrap)	7 years ago
Kyle Havlovitz	adf29675f3	Merge pull request #3535 from hashicorp/metric-docs ... Update metric names and add a legacy config flag	7 years ago
Kyle Havlovitz	a3e9ac5840	Add a test for legacy metrics with a whitelist filter	7 years ago
Kyle Havlovitz	198ed6076d	Clean up subprocess handling and make shell use optional (#3509) ... * Clean up handling of subprocesses and make using a shell optional * Update docs for subprocess changes * Fix tests for new subprocess behavior * More cleanup of subprocesses * Minor adjustments and cleanup for subprocess logic * Makes the watch handler reload test use the new path. * Adds check tests for new args path, and updates existing tests to use new path. * Adds support for script args in Docker checks. * Fixes the sanitize unit test. * Adds panic for unknown watch type, and reverts back to Run(). * Adds shell option back to consul lock command. * Adds shell option back to consul exec command. * Adds shell back into consul watch command. * Refactors signal forwarding and makes Windows-friendly. * Adds a clarifying comment. * Changes error wording to a warning. * Scopes signals to interrupt and kill. This avoids us trying to send SIGCHILD to the dead process. * Adds an error for shell=false for consul exec. * Adds notes about the deprecated script and handler fields. * De-nests an if statement.	7 years ago
Kyle Havlovitz	c728564994	Update metric names and add a legacy config flag	7 years ago
Preetha Appan	26accb3b8a	Only allow 'list' policies within 'key' policy definitions. Consolidated two similar tests into one and fixed alignment.	7 years ago