124 Commits (74158a8aa29e74a1887099798b185b9c68078d0f)

Author	SHA1	Message	Date
James Phillips	d12e81860f	Moves Serf helper into lib to fix import cycle in consul-enterprise.	7 years ago
James Phillips	fe36ed6412	Bumps freeport's block size. ... We were seeing some rollover artifacts where something would be shut down so a port could be re-used, but it was still being referenced by some running thing. This gives more time before rolling over.	7 years ago
Alex Dadgar	358e6827cd	Update cluster.go	7 years ago
Alex Dadgar	6d0b9f4dac	Integer division rounding to zero for rate scaling ... This fixes an issue in which integer division was scaling down to zero.	7 years ago
Alex Dadgar	0fccef237d	Initialize freeport lazily to avoid runtime issues ... This PR makes freeport initialize lazily rather than using an init method.	7 years ago
Alex Dadgar	17dcbb1912	Make freeport testing friendly ... This PR allows the caller to decide if they would like to have the calling test fail, have the caller panic on error, or handle the errors themselves.	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
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
James Phillips	b1a15e0c3d	Adds open source side of network segments (feature is Enterprise-only).	7 years ago
Frank Schroeder	3403cd4372	golint: Fix existing comments ... This needs more work.	8 years ago
James Phillips	7c27ca1f77	Adds missing unit tests and cleans up some router bugs.	8 years ago
James Phillips	1091c7314e	Removes remoteConsuls in favor of the new router. ... This has the next wave of RTT integration with the router and also factors some common RTT-related helpers out to lib. While we were in here we also got rid of the coordinate disable config so we don't need to deal with the complexity in the router (there was never a user-visible way to disable coordinates).	8 years ago
James Phillips	bd605e330c	Adds basic support for node IDs.	8 years ago
Sean Chittenden	d695bcaae6	Use a cryptographically secure seed ... `SeededSecurely` is present if someone or something wants to query the way the library was seeded. Obtained from: nomad	9 years ago
Sean Chittenden	da298f527d	Guard against divide by zero in lib.RandomStagger() ... While I'm at it, add a DurationMinusBufferDomain() function to calculate the min/max for a given call to DurationMinusBuffer() in order to keep the implementation details self-contained.	9 years ago
Sean Chittenden	3a6be9cab0	Add a helper function DurationMinusBuffer() to calculate an interval ... used to schedule a TTL check. e.g. d := lib.DurationMinusBuffer(60 * time.Duration, 10 * time.Second, 16) will return a duration between 46.875s and 50s.	9 years ago
Sean Chittenden	4584e70636	Add lib.AbsInt() helper function	9 years ago
Sean Chittenden	88c42f4056	Move lib's tests to lib_test ... This suite of tests is only testing the exported functions	9 years ago
Sean Chittenden	9fb64ab114	Allow adjusting the number of DNS records in a response... ... Based on work done by @fusiondog in #1583, extend the concept to use an integer instead of a boolean. Fixes: #1583 && #1481	9 years ago
Sean Chittenden	4fec6a9608	Guard against very small or negative rates ... Pointed out by: slackpad	9 years ago
Sean Chittenden	81e1e1c05d	Cull unused function and its unit test. ... Pointed out by: @slackpad	9 years ago
Sean Chittenden	6c1bb78d06	Cull unused function and its unit test. ... Pointed out by: @slackpad	9 years ago
Sean Chittenden	4382c1f7aa	Always seed math/rand on consul startup ... Required for jitter calcs. This could be done in consul/agent, but this makes it clear it is done only once process-wide.	9 years ago
Sean Chittenden	7af6a94edb	Factor out duplicate functions into a lib package ... Consolidate code duplication and tests into a single lib package. Most of these functions were from various **/util.go functions that couldn't be imported due to cyclic imports. The consul/lib package is intended to be a terminal node in an import DAG and a place to stash various consul-only helper functions. Pulled in hashicorp/go-uuid instead of consolidating UUID access.	9 years ago