consul

Commit Graph

Author	SHA1	Message	Date
Daniel Nephin	b9e60c0775	testing: skip slow tests with -short Add a skip condition to all tests slower than 100ms. This change was made using `gotestsum tool slowest` with data from the last 3 CI runs of master. See https://github.com/gotestyourself/gotestsum#finding-and-skipping-slow-tests With this change: ``` $ time go test -count=1 -short ./agent ok github.com/hashicorp/consul/agent 0.743s real 0m4.791s $ time go test -count=1 -short ./agent/consul ok github.com/hashicorp/consul/agent/consul 4.229s real 0m8.769s ```	4 years ago
Daniel Nephin	51efba2c7d	testutil: NewLogBuffer - buffer logs until a test fails Replaces #7559 Running tests in parallel, with background goroutines, results in test output not being associated with the correct test. `go test` does not make any guarantees about output from goroutines being attributed to the correct test case. Attaching log output from background goroutines also cause data races. If the goroutine outlives the test, it will race with the test being marked done. Previously this was noticed as a panic when logging, but with the race detector enabled it is shown as a data race. The previous solution did not address the problem of correct test attribution because test output could still be hidden when it was associated with a test that did not fail. You would have to look at all of the log output to find the relevant lines. It also made debugging test failures more difficult because each log line was very long. This commit attempts a new approach. Instead of printing all the logs, only print when a test fails. This should work well when there are a small number of failures, but may not work well when there are many test failures at the same time. In those cases the failures are unlikely a result of a specific test, and the log output is likely less useful. All of the logs are printed from the test goroutine, so they should be associated with the correct test. Also removes some test helpers that were not used, or only had a single caller. Packages which expose many functions with similar names can be difficult to use correctly. Related: https://github.com/golang/go/issues/38458 (may be fixed in go1.15) https://github.com/golang/go/issues/38382#issuecomment-612940030	4 years ago
Hans Hasselberg	6a18f01b42	agent: ensure node info sync and full sync. (#7189 ) This fixes #7020. There are two problems this PR solves: * if the node info changes it is highly likely to get service and check registration permission errors unless those service tokens have node:write. Hopefully services you register don’t have this permission. * the timer for a full sync gets reset for every partial sync which means that many partial syncs are preventing a full sync from happening Instead of syncing node info last, after services and checks, and possibly saving one RPC because it is included in every service sync, I am syncing node info first. It is only ever going to be a single RPC that we are only doing when node info has changed. This way we are guaranteed to sync node info even when something goes wrong with services or checks which is more likely because there are more syncs happening for them.	5 years ago
Chris Piraino	401221de58	Allow users to configure either unstructured or JSON logging (#7130 ) * hclog Allow users to choose between unstructured and JSON logging	5 years ago
Paul Banks	e812f5516a	Add -sidecar-for and new /agent/service/:service_id endpoint (#4691 ) - A new endpoint `/v1/agent/service/:service_id` which is a generic way to look up the service for a single instance. The primary value here is that it: - supports hash-based blocking and so; - replaces `/agent/connect/proxy/:proxy_id` as the mechanism the built-in proxy uses to read its config. - It's not proxy specific and so works for any service. - It has a temporary shim to call through to the existing endpoint to preserve current managed proxy config defaulting behaviour until that is removed entirely (tested). - The built-in proxy now uses the new endpoint exclusively for it's config - The built-in proxy now has a `-sidecar-for` flag that allows the service ID of the _target_ service to be specified, on the condition that there is exactly one "sidecar" proxy (that is one that has `Proxy.DestinationServiceID` set) for the service registered. - Several fixes for edge cases for SidecarService - A fix for `Alias` checks - when running locally they didn't update their state until some external thing updated the target. If the target service has no checks registered as below, then the alias never made it past critical.	6 years ago
Frank Schroeder	92f136de12	ae: add remaining test cases	7 years ago
Frank Schroeder	622ace2829	ae: refactor StateSyncer to state machine for better testing	7 years ago
Frank Schroeder	1212598ae2	ae: add test that we run a full before a partial sync	7 years ago
Frank Schroeder	3231385089	ae: fix typo in constructor name	7 years ago
Frank Schroeder	51daa96dfe	ae: add test for resume triggering SyncChanges	7 years ago
Frank Schroeder	92088d21e8	ae: add test for ifNotPausedRun	7 years ago
Frank Schroeder	de57b16d99	local state: address review comments * move non-blocking notification mechanism into ae.Trigger * move Pause/Resume into separate type	7 years ago
Frank Schroeder	94ef1041a1	agent: cleanup StateSyncer This patch cleans up the state syncer code by renaming fields, adding helpers and documentation.	7 years ago
Frank Schroeder	29e18c7494	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	58b0e153f9	Revert "agent: decouple anti-entropy from local state" This reverts commit `a842dc9c2b`.	7 years ago
Frank Schroeder	b4e7d0b974	Revert "agent: cleanup StateSyncer" This reverts commit `b7136e100b`.	7 years ago
Frank Schroeder	46641e44d9	Revert "local state: address review comments" This reverts commit `1d315075b1`.	7 years ago
Frank Schroeder	3d202b59bc	Revert "ae: add test for ifNotPausedRun" This reverts commit `f5177ef332`.	7 years ago
Frank Schroeder	e0e96496f1	Revert "ae: add test for resume triggering SyncChanges" This reverts commit `cd0262744d`.	7 years ago
Frank Schroeder	cab3b17292	Revert "ae: fix typo in constructor name" This reverts commit `e88f49e2cc`.	7 years ago
Frank Schroeder	65166cce8e	Revert "ae: add test that we run a full before a partial sync" This reverts commit `8158cec829`.	7 years ago
Frank Schroeder	e78520c3f9	Revert "ae: refactor StateSyncer to state machine for better testing" This reverts commit `8a45365f68`.	7 years ago
Frank Schroeder	4121cafed7	Revert "ae: add remaining test cases" This reverts commit `c32915bb4f`.	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	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	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	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

32 Commits (25826e3ee4f864beb24122aa034b4717dd19aecb)