835 Commits (6289764ea21b3a63242210e54dfb12236cca9574)

Author	SHA1	Message	Date
James Phillips	6289764ea2	Moves sorting up into coordinate endpoint HTTP handlers.	9 years ago
James Phillips	d45fc23abf	Installs a friendly handler for coordinate endpoints when coordinates are disabled.	9 years ago
James Phillips	ce0e9759f8	Fixes config merge fn for disabling coordinates and adds it to JSON.	9 years ago
James Phillips	33e3505aea	Adds endpoints for raw network coordinates.	9 years ago
James Phillips	9c9195746f	Changes ?near=self to a safer ?near=_agent, which is also clearer about what it does.	9 years ago
James Phillips	9caa5b3653	Adds distance sorting to health endpoint. Cleans up unit tests.	9 years ago
James Phillips	e47eea3f3a	Adds a magic "self" node name to distance queries.	9 years ago
James Phillips	54ef97b268	Adds tests for HTTP interface. Removes a stray mark.	9 years ago
James Phillips	89c7203f31	Adds coordinate sorting support to catalog queries for nodes and service nodes.	9 years ago
James Phillips	d734697820	Turns down the coordinate sync rate a bit more.	9 years ago
James Phillips	ad65d953f6	Scales coordinate sends to hit a fixed aggregate rate across the cluster.	9 years ago
James Phillips	66a3d29743	Simplifies the batching function and adds some comments.	9 years ago
James Phillips	5f754c4a87	Does some small cleanups based on PR feedback. ... * Holds coordinate updates in map and gets rid of the update channel. * Cleans up config variables a bit.	9 years ago
James Phillips	7e6d52109b	Hardens Consul from bad coordinates from other nodes.	9 years ago
James Phillips	d12aa2ffab	Moves batching down into the state store and changes it to fail-fast. ... * A batch of updates is done all in a single transaction. * We no longer need to get an update to kick things, there's a periodic flush. * If incoming updates overwhelm the configured flush rate they will be dumped with an error.	9 years ago
James Phillips	b9d5fb0f90	Flips the sense of the coordinate enable option.	9 years ago
James Phillips	92567841d6	Removes one more WAN leftover.	9 years ago
James Phillips	86b112fe31	Does a clean up pass on the Consul side.	9 years ago
James Phillips	01d2452ea3	Merges config changes after rebase.	9 years ago
Derek Chiang	f144d17b1c	Address comments	9 years ago
Derek Chiang	ab9262c656	Add a test case	9 years ago
Derek Chiang	bf5cb7522f	Use IndexedCoordinate instead	9 years ago
Derek Chiang	a1854a7614	Some fixes	9 years ago
Derek Chiang	a338e5efe8	Fix a comment	9 years ago
Derek Chiang	98d87b5dd5	Complete logic for sending coordinates	9 years ago
Derek Chiang	9113b26286	Some fixes	9 years ago
Derek Chiang	a9ea503c69	Adding tests and stuff	9 years ago
Armon Dadgar	6a350d5d19	Merge pull request #1318 from daveadams/f-http-header-token ... Allow specifying Consul token in an HTTP request header	9 years ago
Jeff Mitchell	1e3840b044	Update cleanhttp repo location	9 years ago
Jeff Mitchell	9a5fd5424a	Use cleanhttp to get rid of DefaultTransport	9 years ago
David Adams	b7bcb2a414	Add HTTP request header X-Consul-Token ... Add support for an X-Consul-Token HTTP request header to specify the token with which this request should be fulfilled. The header would have precedence over the responding Agent's default token, but would have lower precedence than a token specified in the query string.	9 years ago
James Phillips	4ee43e90b7	Deletes the old state store and all its accoutrements.	9 years ago
James Phillips	0c90bdc61a	Knocks out the Raft indexes before doing compare.	9 years ago
James Phillips	d57431e300	Gets new structs changes to compile, adds some corner case handling and extra unit tests.	9 years ago
Ryan Uber	d6af59cded	Merge pull request #1309 from hashicorp/f-remove-migrate ... Removes consul-migrate for 0.6	9 years ago
Jeff Mitchell	f49fc095ef	Don't use http.DefaultClient ... Two of the changes are in tests; the one of consequence is in the API. As explained in #1308 this can cause conflicts with downstream programs. Fixes #1308.	9 years ago
Ryan Uber	de287e3efb	agent: consolidates data dir checker	9 years ago
Ryan Uber	10b971df21	agent: test mdb dir protection	9 years ago
Ryan Uber	d901fa6389	agent: remove migrator, refuse to start if mdb dir found	9 years ago
Michael Puncel	a98d25b541	Add http method to log output	9 years ago
James Phillips	0b05dbeb21	Merge pull request #1235 from wuub/master ... fix conflict between handleReload and antiEntropy critical sections	9 years ago
Wojciech Bederski	c4537ed26f	panic when unbalanced localState.Resume() is detected	9 years ago
Dale Wijnand	5a28ebcaa3	Fix a bunch of typos.	9 years ago
James Phillips	2f9ebdb135	Merge pull request #1187 from sfncook/enable_tag_drift_03 ... Enable tag drift 03	9 years ago
Anthony Scalisi	10e028d599	remove various typos	9 years ago
Wojciech Bederski	b014c0f91b	make Pause()/Resume()/isPaused() behave more like a semaphore ... see: https://github.com/hashicorp/consul/issues/1173 #1173 Reasoning: somewhere during consul development Pause()/Resume() and PauseSync()/ResumeSync() were added to protect larger changes to agent's localState. A few of the places that it tries to protect are: - (a *Agent) AddService(...) # part of the method - (c *Command) handleReload(...) # almost the whole method - (l *localState) antiEntropy(...)# isPaused() prevents syncChanges() The main problem is, that in the middle of handleReload(...)'s critical section it indirectly (loadServices()) calls AddService(...). AddService() in turn calls Pause() to protect itself against syncChanges(). At the end of AddService() a defered call to Resume() is made. With the current implementation, this releases isPaused() "lock" in the middle of handleReload() allowing antiEntropy to kick in while configuration reload is still in progress. Specifically almost all services and probably all check are unloaded when syncChanges() is allowed to run. This in turn can causes massive service/check de-/re-registration, and since checks are by default registered in the critical state, a majority of services on a node can be marked as failing. It's made worse with automation, often calling `consul reload` in close proximity on many nodes in the cluster. This change basically turns Pause()/Resume() into P()/V() of a garden-variety semaphore. Allowing Pause() to be called multiple times, and releasing isPaused() only after all matching/defered Resumes() are called as well. TODO/NOTE: as with many semaphore implementations, it might be reasonable to panic() if l.paused ever becomes negative.	9 years ago
Wojciech Bederski	24bc17eaa1	failing test showing that nested Pause()/Resume() release too early ... see: #1173 / https://github.com/hashicorp/consul/issues/1173	9 years ago
Shawn Cook	66fd8fb2a0	Rename EnableTagOverride and update formatting	9 years ago
Shawn Cook	d7ce0b3c6b	Remove debug lines	9 years ago
Shawn Cook	0b3faf6e4a	Merge remote-tracking branch 'hashicorp/master' into enable_tag_drift_03	9 years ago