438 Commits (cc0499da3de7e751aae7e0720df72f42e8dbd9f2)

Author	SHA1	Message	Date
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	35f4acdddc	local state: remove stale comment	7 years ago
Frank Schroeder	f9267380db	local state: make test more robust	7 years ago
Frank Schroeder	af1243c725	local state: clone check to avoid side effect	7 years ago
Frank Schroeder	39a2d8d25e	local state: use synchronized access to internal maps	7 years ago
Frank Schroeder	bd00814301	ae: do not trigger on Resume while holding the lock	7 years ago
Frank Schroeder	c32915bb4f	ae: add remaining test cases	7 years ago
Frank Schroeder	8a45365f68	ae: refactor StateSyncer to state machine for better testing	7 years ago
Frank Schroeder	8158cec829	ae: add test that we run a full before a partial sync	7 years ago
Frank Schroeder	b9a8b53d52	ae: make control flow more explicit	7 years ago
Frank Schroeder	e88f49e2cc	ae: fix typo in constructor name	7 years ago
Frank Schroeder	cd0262744d	ae: add test for resume triggering SyncChanges	7 years ago
Frank Schroeder	f5177ef332	ae: add test for ifNotPausedRun	7 years ago
Frank Schroeder	066ad01c38	ae: make stagger function pluggable for testing	7 years ago
Frank Schroeder	126046be23	ae: restore previous pause/resume behavior	7 years ago
Frank Schroeder	ffb265dd93	ae: ensure that syncs are blocked when paused	7 years ago
Frank Schroeder	9280841a80	local state: rename Add{Check,Service}State to Set{Check,Service}State	7 years ago
Frank Schroeder	9bc8127728	local state: move Metadata methods together	7 years ago
Frank Schroeder	e86521e637	local state: update documentation of updateSyncState	7 years ago
Frank Schroeder	42188164f8	local state: update comments	7 years ago
Frank Schroeder	1d315075b1	local state: address review comments ... * move non-blocking notification mechanism into ae.Trigger * move Pause/Resume into separate type	7 years ago
Frank Schroeder	c28e23eac8	local state: refactor TestAgentAntiEntropy_EnableTagOverride ... Make intent clearer by being more explicit and adding some comments. Use verify.Values to compare service entries.	7 years ago
Frank Schroeder	86f7ea6013	local state: fix TestAgentAntiEntropy_EnableTagOverride ... The test had a race condition where it relied on the first service to be synced to the remote catalog which sometimes failed.	7 years ago
Frank Schroeder	ff62eaf063	local state: rename tests	7 years ago
Frank Schroeder	2bdba8ab06	local state: drop retry loops from tests ... Since the tests are now using synchronous calls for state syncing we no longer need to use retry loops to wait for the changes to propagate.	7 years ago
Frank Schroeder	ce1f35373e	agent: skip non-sensical TestCatalogRegister ... It is not clear what this test is supposed to verify.	7 years ago
Frank Schroeder	f8e20cd996	local state: fix anti-entropy state tests ... The anti-entropy tests relied on the side-effect of the StartSync() method to perform a full sync instead of a partial sync. This lead to multiple anti-entropy go routines being started unnecessary retry loops. This change changes the behavior to perform synchronous full syncs when necessary removing the need for all of the time.Sleep and most of the retry loops.	7 years ago
Frank Schroeder	e9149f64d9	local state: fix test with updated error message	7 years ago
Frank Schroeder	76682da4a0	local state: fix failing tests	7 years ago
Frank Schroeder	1af52bf7be	local state: tests compile	7 years ago
Frank Schroeder	ccbae7da5b	local state: replace multi-map state with structs ... The state of the service and health check records was spread out over multiple maps guarded by a single lock. Access to the maps has to happen in a coordinated effort and the tests often violated this which made them brittle and racy. This patch replaces the multiple maps with a single one for both checks and services to make the code less fragile. This is also necessary since moving the local state into its own package creates circular dependencies for the tests. To avoid this the tests can no longer access internal data structures which they should not be doing in the first place. The tests still don't compile but this is a ncessary step in that direction.	7 years ago
Frank Schroeder	d447e823c6	local state: move to separate package ... This patch moves the local state to a separate package to further decouple it from the agent code. The code compiles but the tests do not yet.	7 years ago
Frank Schroeder	b5dbad910c	agent: simplify some loops	7 years ago
Frank Schroeder	7a2af206ea	agent: refactor sync loop to linear flow of control	7 years ago
Frank Schroeder	b7136e100b	agent: cleanup StateSyncer ... This patch cleans up the state syncer code by renaming fields, adding helpers and documentation.	7 years ago
Frank Schroeder	a842dc9c2b	agent: decouple anti-entropy from local state ... The anti-entropy code manages background synchronizations of the local state on a regular basis or on demand when either the state has changed or a new consul server has been added. This patch moves the anti-entropy code into its own package and decouples it from the local state code since they are performing two different functions. To simplify code-review this revision does not make any optimizations, renames or refactorings. This will happen in subsequent commits.	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
James Phillips	23d4b24792	Cleans up import sorting.	7 years ago
Hadar Greinsmark	7e1a860978	Implement HTTP Watch handler (#3413) ... Implement HTTP Watch handler	7 years ago
Frank Schroeder	c94751ad43	test: replace porter tool with freeport lib ... This patch removes the porter tool which hands out free ports from a given range with a library which does the same thing. The challenge for acquiring free ports in concurrent go test runs is that go packages are tested concurrently and run in separate processes. There has to be some inter-process synchronization in preventing processes allocating the same ports. freeport allocates blocks of ports from a range expected to be not in heavy use and implements a system-wide mutex by binding to the first port of that block for the lifetime of the application. Ports are then provided sequentially from that block and are tested on localhost before being returned as available.	7 years ago
Frank Schröder	350932161d	dns: return NXDOMAIN if datacenter is invalid (#3200) (#3596) ... Queries to the DNS server can contain an optional datacenter name in the query name. You can query for 'foo.service.consul' or 'foo.service.dc.consul' to get a response for either the default or a specific datacenter. Datacenter names cannot have dots, therefore the datacenter name can refer to only one element in the DNS query name. The DNS server allowed extra labels between the optional datacenter name and the domain and returned a valid response instead of returning NXDOMAIN. For example, if the domain is set to '.consul' then 'foo.service.dc1.extra.consul' should return NXDOMAIN because of 'extra' being between the datacenter name 'dc1' and the domain '.consul'. Fixes #3200	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
James Phillips	53f67c3993	Fixes API client for ScriptArgs and updates documentation. (#3589) ... * Updates the API client to support the current `ScriptArgs` parameter for checks. * Updates docs for checks to explain the `ScriptArgs` parameter issue. * Adds mappings for "args" and "script-args" to give th API parity with config. * Adds checks on return codes. * Removes debug logging that shows empty when args are used.	7 years ago
Ryan Slade	85e4aea9d1	Replace time.Now().Sub(x) with time.Since(x)	7 years ago
James Phillips	f25c66dfd9	Fixes an XSS issue with unescaped node names. (#3578) ... * Fixes an XSS issue with node names in the tomography graph. * Updates built-in static web assets. * Updates the change log.	7 years ago
James Phillips	c943b1b151	Adds a brief wait and poll period to update check status after a timeout. (#3573) ... * Adds a brief wait and poll period to update the check status if we get stucking waiting for the processes to terminate. Fixes #3570 * Jumps out of timeout case and includes script output.	7 years ago
James Phillips	575d70aaa7	Cleans up some drift between the OSS and Enterprise trees.	7 years ago
Kyle Havlovitz	106b8b0b33	Kill check processes after the timeout is reached (#3567) ... * Kill check processes after the timeout is reached Kill the subprocess spawned by a script check once the timeout is reached. Previously Consul just marked the check critical and left the subprocess around. Fixes #3565. * Set err to non-nil when timeout occurs * Fix check timeout test * Kill entire process subtree on check timeout * Add a docs note about windows subprocess termination	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
James Phillips	e6c50f4a71	Removes obsolete segment stub.	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
preetapan	77c972f594	Fixes agent error handling when check definition is invalid. Distingu… (#3560) ... * Fixes agent error handling when check definition is invalid. Distinguishes between empty checks vs invalid checks * Made CheckTypes return Checks from service definition struct rather than a new copy, and other changes from code review. This also errors when json payload contains empty structs * Simplify and improve validate method, and make sure that CheckTypes always returns a new copy of validated check definitions * Tweaks some small style things and error messages. * 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
Preetha Appan	e7dc345cfa	Fix unit test after dns library upgrade to account for correct data length	7 years ago
James Phillips	4dab70cb93	Fixes handling of stop channel and failed barrier attempts. (#3546) ... * Fixes handling of stop channel and failed barrier attempts. There were two issues here. First, we needed to not exit when there was a timeout trying to write the barrier, because Raft might not step down, so we'd be left as the leader but having run all the step down actions. Second, we didn't close over the stopCh correctly, so it was possible to nil that out and have the leaderLoop never exit. We close over it properly AND sequence the nil-ing of it AFTER the leaderLoop exits for good measure, so the code is more robust. Fixes #3545 * Cleans up based on code review feedback. * Tweaks comments. * Renames variables and removes comments.	7 years ago
Victor Boivie	8e361beb7a	Minor typo (boostrap)	7 years ago
James Phillips	3bc6df5f0e	Adds script warning and fixes Docker args recognition.	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
Frank Schröder	ce887a0c45	Provide stable config for agent/self (#3532) ... * config: provide stable config for /v1/agent/self (#3530) This patch adds a stable subset of the previous Config struct to the agent/self response. The actual runtime configuration is moved into DebugConfig and will be documented to change. Fixes #3530 * config: fix tests * doc: update api documentation for /v1/agent/self	7 years ago
James Phillips	4f2dccc2a9	Merge pull request #3531 from hashicorp/pr-3521-slackpad ... ui: Use monospace font for textarea controls.	7 years ago
James Phillips	b34d576193	Updates checked in web assets to pick up CSS change. ... Closes #3521	7 years ago
Preetha Appan	8dcd7e700c	Remove extra newline	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
Preetha Appan	51a04ec87d	Introduces new 'list' permission that applies to KV store recursive reads, and enforced only when opted in.	7 years ago
Frank Schroeder	1944218492	use ports from derived addresses	7 years ago
Frank Schroeder	42f8ff7b3c	config: drop advertise_addrs ... Fixes #3516	7 years ago
Frank Schroeder	abe41d231c	Fix tests after config refactor	7 years ago
Patrick Sodré	7501331d13	Implement encodeKVasRFC1464 function	7 years ago
Patrick Sodré	2cc6ac542c	Add RFC1464 tests	7 years ago
Patrick Sodré	865f087ec9	Turn encodeKVasRFC1464 into a plain function	7 years ago
Patrick Sodré	12fb0bfd5b	Use verify for NodeLookup CNAME, and TXT tests	7 years ago
Patrick Sodré	d5e3b9d843	Refactor formatTxtRecords as encodeKVasRFC1464 ... - Move the logic of rfc1035 out of the encoding function - Left basic version of encodingKV as 'k=v'	7 years ago
Patrick Sodré	655c89ee10	Fix editorial suggestions	7 years ago
Patrick Sodré	afb0c92334	Remove redundant check of Node.Meta size	7 years ago
Patrick Sodré	53e812e759	Return Node.Meta info using the DNS interface	7 years ago
Patrick Sodré	ab90865865	Add test for NoteLookup ANY request	7 years ago
Patrick Sodré	4c6b8022c2	Add test for querying Node.Meta with DNS TXT ... - Lookup TXT records using recursive lookups - Expect TXT record equal to value if key starts with rfc1035- - Expect TXT record in rfc1464 otherwise, i.e. (k=v) ref #2709	7 years ago
Frank Schröder	07dea89f31	fail early when advertise addr is set to ANY (#3507)	7 years ago
Frank Schröder	9a67556bb5	only detect advertise address if derived value is any (#3506) ... * only detect advertise address if derived value is any * determine detect function only when advertise addr is any	7 years ago
James Phillips	98850322c0	Adds a comment about Datacenter and NodeName being stable interfaces ... in the runtime config strucutre.	7 years ago
Frank Schröder	21118cafeb	Recursive sanitize (#3505) ... * vendor: add github.com/sergi/go-diff/diffmatchpatch for diff'ing test output * config: refactor Sanitize to recursively clean runtime config and format complex fields * Removes an extra int cast. * Adds a top-level check test case for sanitization.	7 years ago
James Phillips	0190c4a081	Gets rid of flaky clause in stats fetcher unit test. ... Given how the rutine is coded we can still get data so this wasn't a reliable thing to check.	7 years ago
preetapan	4d9fc638b4	Issue 3452 (#3500) ... * Make sure that id and address are set in member created during reaping of catalog nodes that have been removed from serf * Get address from node table in the state store rather than from service address * Fix incorrect lookup by checkname instead of node name * Make sure that serverlookup is called with the right address format, added unit test. * Address code review comments * Tweaks style stuff.	7 years ago
Frank Schröder	e84c2b2edd	Metrics service prefix (#3498) ... * metrics: replace statsite_prefix with service_prefix The metrics prefix isn't statsite specific and is in fact used for all metrics providers. Since we are deprecating fields anyway we should fix this one as well. Fixes #3293 * Updates docs and sorts telemetry section. * Renames to "metrics_prefix" to disambiguate with Consul services. * Updates the change log.	7 years ago
James Phillips	49058fee11	Merge pull request #3501 from hashicorp/snapshot-test-hang ... Cleans up some edge cases in TestSnapshot_Forward_Leader.	7 years ago
James Phillips	5fa2322e0b	Cleans up some edge cases in TestSnapshot_Forward_Leader. ... These could cause the tests to hang.	7 years ago
Kyle Havlovitz	bfa70a10ca	Fix watch error when http & https are disabled (#3493) ... Remove an error in watch reloading that happens when http and https are both disabled, and use an https address for running watches if no http addresses are present. Fixes #3425.	7 years ago
Preetha Appan	3c4a108769	Move Raft protocol version for list peers end point to server side, fix unit tests. This fixes #3449	7 years ago
Frank Schroeder	56e6439be9	fix data race ... Since state.Checks() returns a shallow copy its elements must not be modified. Copying the elements in the handler does not guarantee consistency since that list is guarded by a different lock. Therefore, the only solution is to have state.Checks() return a deep copy.	7 years ago
Frank Schroeder	7bd85792b2	config: do not clobber multiple check and service definitions ... This patch ensures that multiple files with single 'check' or 'service' definitions result in the combination of them.	7 years ago
James Phillips	a75a779469	Renames `enable_ui` to `ui` to keep compatibility with existing configs.	7 years ago
Frank Schröder	1e461110e6	agent: consolidate handling of 405 Method Not Allowed (#3405) ... * agent: consolidate http method not allowed checks This patch uses the error handling of the http handlers to handle HTTP method not allowed errors across all available endpoints. It also adds a test for testing whether the endpoints respond with the correct status code. * agent: do not panic on metrics tests * agent: drop other tests for MethodNotAllowed * agent: align /agent/join with reality /agent/join uses PUT instead of GET as documented. * agent: align /agent/check/{fail,warn,pass} with reality /agent/check/{fail,warn,pass} uses PUT instead of GET as documented. * fix some tests * Drop more tests for method not allowed * Align TestAgent_RegisterService_InvalidAddress with reality * Changes API client join to use PUT instead of GET. * Fixes agent endpoint verbs and removes obsolete tests. * Updates the change log.	7 years ago
preetapan	73951d8319	Merge pull request #3494 from hashicorp/enforce_json_extension ... Enforce json or hcl extension to Consul config files, updated unit tests	7 years ago
James Phillips	45646ac3f4	Bumps default Raft protocol to version 3. (#3477) ... * Changes default Raft protocol to 3. * Changes numPeers() to report only voters. This should have been there before, but it's more obvious that this is incorrect now that we default the Raft protocol to 3, which puts new servers in a read-only state while Autopilot waits for them to become healthy. * Fixes TestLeader_RollRaftServer. * Fixes TestOperator_RaftRemovePeerByAddress. * Fixes TestServer_*. Relaxed the check for a given number of voter peers and instead do a thorough check that all servers see each other in their Raft configurations. * Fixes TestACL_*. These now just check for Raft replication to be set up, and don't care about the number of voter peers. * Fixes TestOperator_Raft_ListPeers. * Fixes TestAutopilot_CleanupDeadServerPeriodic. * Fixes TestCatalog_ListNodes_ConsistentRead_Fail. * Fixes TestLeader_ChangeServerID and adjusts the conn pool to throw away sockets when it sees io.EOF. * Changes version to 1.0.0 in the options doc. * Makes metrics test more deterministic with autopilot metrics possible.	7 years ago
Preetha Appan	a286ad7533	Enforce json or hcl extension to Consul config files, updated unit tests	7 years ago
James Phillips	f42e85ce22	Removes unused imports in agent_test.go.	7 years ago
Preetha Appan	d7e27e67c1	Introduce Code Policy validation via sentinel, with a noop implementation	7 years ago
Frank Schröder	12216583a1	New config parser, HCL support, multiple bind addrs (#3480) ... * new config parser for agent This patch implements a new config parser for the consul agent which makes the following changes to the previous implementation: * add HCL support * all configuration fragments in tests and for default config are expressed as HCL fragments * HCL fragments can be provided on the command line so that they can eventually replace the command line flags. * HCL/JSON fragments are parsed into a temporary Config structure which can be merged using reflection (all values are pointers). The existing merge logic of overwrite for values and append for slices has been preserved. * A single builder process generates a typed runtime configuration for the agent. The new implementation is more strict and fails in the builder process if no valid runtime configuration can be generated. Therefore, additional validations in other parts of the code should be removed. The builder also pre-computes all required network addresses so that no address/port magic should be required where the configuration is used and should therefore be removed. * Upgrade github.com/hashicorp/hcl to support int64 * improve error messages * fix directory permission test * Fix rtt test * Fix ForceLeave test * Skip performance test for now until we know what to do * Update github.com/hashicorp/memberlist to update log prefix * Make memberlist use the default logger * improve config error handling * do not fail on non-existing data-dir * experiment with non-uniform timeouts to get a handle on stalled leader elections * Run tests for packages separately to eliminate the spurious port conflicts * refactor private address detection and unify approach for ipv4 and ipv6. Fixes #2825 * do not allow unix sockets for DNS * improve bind and advertise addr error handling * go through builder using test coverage * minimal update to the docs * more coverage tests fixed * more tests * fix makefile * cleanup * fix port conflicts with external port server 'porter' * stop test server on error * do not run api test that change global ENV concurrently with the other tests * Run remaining api tests concurrently * no need for retry with the port number service * monkey patch race condition in go-sockaddr until we understand why that fails * monkey patch hcl decoder race condidtion until we understand why that fails * monkey patch spurious errors in strings.EqualFold from here * add test for hcl decoder race condition. Run with go test -parallel 128 * Increase timeout again * cleanup * don't log port allocations by default * use base command arg parsing to format help output properly * handle -dc deprecation case in Build * switch autopilot.max_trailing_logs to int * remove duplicate test case * remove unused methods * remove comments about flag/config value inconsistencies * switch got and want around since the error message was misleading. * Removes a stray debug log. * Removes a stray newline in imports. * Fixes TestACL_Version8. * Runs go fmt. * Adds a default case for unknown address types. * Reoders and reformats some imports. * Adds some comments and fixes typos. * Reorders imports. * add unix socket support for dns later * drop all deprecated flags and arguments * fix wrong field name * remove stray node-id file * drop unnecessary patch section in test * drop duplicate test * add test for LeaveOnTerm and SkipLeaveOnInt in client mode * drop "bla" and add clarifying comment for the test * split up tests to support enterprise/non-enterprise tests * drop raft multiplier and derive values during build phase * sanitize runtime config reflectively and add test * detect invalid config fields * fix tests with invalid config fields * use different values for wan sanitiziation test * drop recursor in favor of recursors * allow dns_config.udp_answer_limit to be zero * make sure tests run on machines with multiple ips * Fix failing tests in a few more places by providing a bind address in the test * Gets rid of skipped TestAgent_CheckPerformanceSettings and adds case for builder. * Add porter to server_test.go to make tests there less flaky * go fmt	7 years ago
James Phillips	d84c0b1a01	Robustifies check in TestCatalog_ListNodes_ConsistentRead_Fail test. ... Fixes #3469	7 years ago
James Phillips	828be5771a	Revert "Manages segments list via a pointer." ... This reverts commit c277a42504.	7 years ago
James Phillips	c277a42504	Manages segments list via a pointer.	7 years ago
James Phillips	96a89a3381	Cleans up formatting.	7 years ago
James Phillips	00605c0214	Shows the segment name in the keyring API and command output.	7 years ago
James Phillips	aa5ef4a098	Populates the segment keyrings based on the LAN keyring.	7 years ago
James Phillips	88a150cee1	Moves reconcile loop into segment stub.	7 years ago
James Phillips	5c03cb571d	Takes the skip out of the client check. ... Without this the merge delegate won't check the segment for non-servers a little below here.	7 years ago
James Phillips	3418c7ff93	Merge pull request #3447 from hashicorp/issue-3070 ... Skips unique node ID check for old versions of Consul.	7 years ago
James Phillips	520060e138	Fixes incorrect comment.	7 years ago
James Phillips	084679ab65	Pulls down some code for the check loop.	7 years ago
James Phillips	3535652595	Uses the Raft configuration for the self-add skip check.	7 years ago
Preetha Appan	5f2e1c9b07	Change member join reconcile step to process joining itself, to handle node IP address changes correctly when number of servers < 3	7 years ago
James Phillips	1333fa57a1	Skips unique node ID check for old versions of Consul. ... Fixes #3070.	7 years ago
James Phillips	67b19ac065	Allow _all for WAN as a no-op.	7 years ago
James Phillips	1a117ba0a8	Makes the all segments query explict, and the default for `consul members`.	7 years ago
James Phillips	9258506dab	Adds simple rate limiting for client agent RPC calls to Consul servers. (#3440) ... * Added rate limiting for agent RPC calls. * Initializes the rate limiter based on the config. * Adds the rate limiter into the snapshot RPC path. * Adds unit tests for the RPC rate limiter. * Groups the RPC limit parameters under "limits" in the config. * Adds some documentation about the RPC limiter. * Sends a 429 response when the rate limiter kicks in. * Adds docs for new telemetry. * Makes snapshot telemetry look like RPC telemetry and cleans up comments.	7 years ago
Kyle Havlovitz	220db48aa7	Merge pull request #3431 from hashicorp/network-segments-oss	7 years ago
Kyle Havlovitz	0e33e2ecab	Pass listeners into setupSegments	7 years ago
Kyle Havlovitz	62102a537e	Organize segments for a cleaner split between enterprise and OSS	7 years ago
Kyle Havlovitz	baa501e0c5	Fill in the segment in the QuerySource for prepared query lookups	7 years ago
Kyle Havlovitz	7e565d7338	Fix some inconsistencies with segment logic and comments	7 years ago
Kyle Havlovitz	16aaf27208	Default bind/advertise for segments to BindAddr/AdvertiseAddr	7 years ago
Preetha Appan	2386214655	Wire server provider for raft layer only on protocol version 3 and above, and update changelog	7 years ago
Kyle Havlovitz	21513b0393	Update coord display in ui to account for segments	7 years ago
Kyle Havlovitz	14b027a3c2	Add segment addr field to tags for LAN flood joiner	7 years ago
Kyle Havlovitz	d129767657	Add agent.segment interpolation to prepared queries	7 years ago
Kyle Havlovitz	2ada0439d4	Add rpc_listener option to segment config	7 years ago
Kyle Havlovitz	a30e7657af	Add segment config validation	7 years ago
James Phillips	b1a15e0c3d	Adds open source side of network segments (feature is Enterprise-only).	7 years ago
Preetha Appan	a231eea0e7	More cleanup from code review	7 years ago
Preetha Appan	c6ee9bfa69	Remove copy pasted duplicate line, update documentation.	7 years ago
Preetha Appan	0f4e24f72c	Consolidate server lookup into one place and replace usages of localConsuls.	7 years ago