12395 Commits (c9c9e4face9872ad62185f90d7d79efab392d0d7)

 

All Branches   

Search

Author	SHA1	Message	Date
Daniel Nephin	30523c22f5	Merge pull request #8517 from hashicorp/dnephin/remove-defer-os-removeall ... testing: remove all the 'defer os.RemoveAll'	4 years ago
danielehc	96916a770c	Refactor api-docs links to learn (#8488)	4 years ago
danielehc	5483988b5b	Refactor docs links to learn 2 (#8491) ... * Refactor docs links to learn * Apply suggestions from code review Co-authored-by: Derek Strickland <1111455+DerekStrickland@users.noreply.github.com> Co-authored-by: Derek Strickland <1111455+DerekStrickland@users.noreply.github.com>	4 years ago
danielehc	f32b3d2cca	Refactor docs links to learn (#8490)	4 years ago
Daniel Nephin	16217fe9b9	testing: use t.Cleanup in testutil.TempFile ... So that it has the same behaviour as TempDir. Also remove the now unnecessary 'defer os.Remove'	4 years ago
Daniel Nephin	d68edcecf4	testing: Remove all the defer os.Removeall ... Now that testutil uses t.Cleanup to remove the directory the caller no longer has to manage the removal	4 years ago
Daniel Nephin	8a4d292c8e	config: unexport and resolve TODOs in config.Builder ... - unexport testing shims, and document their purpose - resolve a TODO by moving validation to NewBuilder and storing the one field that is used instead of all of Options - create a slice with the correct size to avoid extra allocations	4 years ago
Daniel Nephin	a4b201af36	testing: Improve session_endpoint_test ... While working on another change I caused a bunch of these tests to fail. Unfortunately the failure messages were not super helpful at first. One problem was that the request and response were created outside of the retry. This meant that when the second attempt happened, the request body was empty (because the buffer had been consumed), and so the request was not actually being retried. This was fixed by moving more of the request creation into the retry block. Another problem was that these functions can return errors in two ways, and are not consistent about which way they use. Some errors are returned to the response writer, but the tests were not checking those errors, which was causing a panic later on. This was fixed by adding a check for the response code. Also adds some missing t.Helper(), and has assertIndex use checkIndex so that it is clear these are the same implementation.	4 years ago
Daniel Nephin	587dcebfca	Use t.Helper in testutil/retry	4 years ago
Daniel Nephin	bb6737dc60	Merge pull request #8509 from hashicorp/dnephin/use-t.cleanup-in-testagent ... testing: Use t.cleanup in TestAgent , and fix two flaky tests	4 years ago
Derek Strickland	2bf85fae62	Added link to Secure Consul on K8S Tutorial (#8402)	4 years ago
Daniel Nephin	070e843113	testutil: Add t.Cleanup to TempDir ... TempDir registers a Cleanup so that the directory is always removed. To disable to cleanup, set the TEST_NOCLEANUP env var.	4 years ago
Mike Morris	ac6415f0a8	changelog: update for 1.8.3, 1.7.7 and 1.6.8 (#8508) ... * Update CHANGELOG.md * changelog: update for 1.8.3, 1.7.7 and 1.6.8 Co-authored-by: Sarah Christoff <schristoff@hashicorp.com>	4 years ago
Daniel Nephin	2b920ad199	testing: fix flaky test TestDNS_NonExistentDC_RPC ... I saw this test flake locally, and it was easy to reproduce with -count=10. The failure was: 'TestAgent.dns: rpc error: error=No known Consul servers'. Waiting for the agent seems to fix it.	4 years ago
Daniel Nephin	1912c5ad89	testing: wait until monitor has started before shutdown ... This commit fixes a test that I saw flake locally while running tests. The test output from the monitor started immediately after the line the test was looking for. To fix the problem a channel is closed when the goroutine starts. Shutdown is not called until this channel is closed, which seems to greatly reduce the chance of a flake.	4 years ago
Daniel Nephin	3a4e62836b	testing: Remove TestAgent.Key and change TestAgent.DataDir ... TestAgent.Key was only used by 3 tests. Extracting it from the common helper that is used in hundreds of tests helps keep the shared part small and more focused. This required a second change (which I was planning on making anyway), which was to change the behaviour of DataDir. Now in all cases the TestAgent will use the DataDir, and clean it up once the test is complete.	4 years ago
Iryna Shustava	7f8e723dc2	docs: add docs to use Connect CA providers with Helm (#8464)	4 years ago
Daniel Nephin	b1679508d4	testing: use t.Cleanup in TestAgent for returnPorts	4 years ago
Daniel Nephin	4e8e0de8f0	testing: remove unused fields from TestACLAgent	4 years ago
Derek Strickland	930d7d3158	Learn/link updates derek (#8487) ... * Updated Learn url paths. Co-authored-by: danielehc <40759828+danielehc@users.noreply.github.com>	4 years ago
Iryna Shustava	cd3d6adbf4	docs: consul on k8s doesn't support external servers requiring mTLS (#8484)	4 years ago
Daniel Nephin	2ed33089aa	Merge pull request #8473 from hashicorp/dnephin/unmethod-consul-config ... agent: convert consulConfig method to a function	4 years ago
Daniel Nephin	399c77dfb6	agent: rename vars in newConsulConfig ... 'base' is a bit misleading, since it is the return value. Renamed to cfg.	4 years ago
Daniel Nephin	7b5b170a0d	agent: Move setupKeyring functions to keyring.go ... There are a couple reasons for this change: 1. agent.go is way too big. Smaller files makes code eaasier to read because tools that show usage also include filename which can give a lot more context to someone trying to understand which functions call other functions. 2. these two functions call into a large number of functions already in keyring.go.	4 years ago
Daniel Nephin	9919e5dfa5	agent: unmethod consulConfig ... To allow us to move newConsulConfig out of Agent.	4 years ago
Daniel Nephin	1f7dab07d6	Merge pull request #8482 from hashicorp/dnephin/more-state-store-unmethod ... state: remove unused Store method receiver	4 years ago
Daniel Nephin	8f596f5551	Fix conflict in merged PRs ... One PR renamed the var from config->cfg, and another used the old name config, which caused the build to fail on master.	4 years ago
Daniel Nephin	d677706625	state: remove unused Store method receiver ... And use ReadTxn interface where appropriate.	4 years ago
Daniel Nephin	190fcc14a3	Merge pull request #8463 from hashicorp/dnephin/unmethod-make-node-id ... agent: convert NodeID methods to functions	4 years ago
Daniel Nephin	912aae8624	Merge pull request #8461 from hashicorp/dnephin/remove-notify-shutdown ... agent/consul: Remove NotifyShutdown	4 years ago
Daniel Nephin	5b37efd91b	Merge pull request #8365 from hashicorp/dnephin/fix-service-by-node-meta-flake ... state: speed up tests that use watchLimit	4 years ago
Hans Hasselberg	b86c018a19	Link issue in note template (#8502) ... Issue and PR numbers do not overlap, they are based of the same counter. A PR can be also linked to via issues, if it is a PR, Github will redirect to it. This change has the benefit that one can link to both - issues and PRs.	4 years ago
Luke Kysow	7e5253bf04	Update k8s sync docs (#8452) ... * Update k8s sync docs - remove docs that said for nodeport service we register each instance on a node with its same node name. We instead register each instance onto the k8s-sync node - add docs describing which ports and ips are used	4 years ago
Luke Kysow	0f99b63fc8	Document k8s sidecar resource annotations (#8455)	4 years ago
s-christoff	27a6871434	Update version.js to 1.8.3 (#8505)	4 years ago
Daniel Nephin	3c523eee9e	Merge pull request #8500 from hashicorp/dnephin/auto-config-loader ... auto-config: reduce awareness of config	4 years ago
s-christoff	7301d473ad	Update CHANGELOG.md	4 years ago
Daniel Nephin	37eacf8192	auto-config: reduce awareness of config ... This is a small step to allowing Agent to accept its dependencies instead of creating them in New. There were two fields in autoconfig.Config that were used exclusively to load config. These were replaced with a single function, allowing us to move LoadConfig back to the config package. Also removed the WithX functions for building a Config. Since these were simple assignment, it appeared we were not getting much value from them.	4 years ago
Daniel Nephin	e07554500e	Remove check that hostID is a uuid. ... Immediately afterward we hash the ID, so it does not need to be a uuid anymore.	4 years ago
Daniel Nephin	875d8bde42	agent: convert NodeID methods to functions ... Making these functions allows us to cleanup how an agent is initialized. They only make use of a config and a logger, so they do not need to be agent methods. Also cleanup the testing to use t.Run and require.	4 years ago
Daniel Nephin	0738eb8596	Extract nodeID functions to a different file ... In preparation for turning them into functions. To reduce the scope of Agent, and refactor how Agent is created and started.	4 years ago
R.B. Boyer	839ca03b7c	update changelog snippet	4 years ago
R.B. Boyer	e3cd4a8539	connect: use stronger validation that ingress gateways have compatible protocols defined for their upstreams (#8470) ... Fixes #8466 Since Consul 1.8.0 there was a bug in how ingress gateway protocol compatibility was enforced. At the point in time that an ingress-gateway config entry was modified the discovery chain for each upstream was checked to ensure the ingress gateway protocol matched. Unfortunately future modifications of other config entries were not validated against existing ingress-gateway definitions, such as: 1. create tcp ingress-gateway pointing to 'api' (ok) 2. create service-defaults for 'api' setting protocol=http (worked, but not ok) 3. create service-splitter or service-router for 'api' (worked, but caused an agent panic) If you were to do these in a different order, it would fail without a crash: 1. create service-defaults for 'api' setting protocol=http (ok) 2. create service-splitter or service-router for 'api' (ok) 3. create tcp ingress-gateway pointing to 'api' (fail with message about protocol mismatch) This PR introduces the missing validation. The two new behaviors are: 1. create tcp ingress-gateway pointing to 'api' (ok) 2. (NEW) create service-defaults for 'api' setting protocol=http ("ok" for back compat) 3. (NEW) create service-splitter or service-router for 'api' (fail with message about protocol mismatch) In consideration for any existing users that may be inadvertently be falling into item (2) above, that is now officiall a valid configuration to be in. For anyone falling into item (3) above while you cannot use the API to manufacture that scenario anymore, anyone that has old (now bad) data will still be able to have the agent use them just enough to generate a new agent/proxycfg error message rather than a panic. Unfortunately we just don't have enough information to properly fix the config entries.	4 years ago
Freddy	d72f72dcd5	Notify alias checks when aliased service is [de]registered (#8456)	4 years ago
Daniel Nephin	3d96c5b651	Merge pull request #8469 from hashicorp/dnephin/config-source ... config: make Source an interface to avoid the marshal/unmarshal cycle in auto-config	4 years ago
Mike Morris	ebc9b27cfa	ci: bump Go to v1.14.7 (#8449)	4 years ago
Hans Hasselberg	aacf0fd777	Merge pull request #8471 from hashicorp/local_only ... thread local-only through the layers	4 years ago
Freddy	875816d0d3	Internal endpoint to query intentions associated with a gateway (#8400)	4 years ago
Iryna Shustava	ed0fa4b3b1	docs: update helm chart ref (#8483) ... No longer require servers to be running on k8s when manageSystemACLs is true	4 years ago
Daniel Nephin	36202a12dd	Merge pull request #8453 from hashicorp/dnephin/fix-test-server-timeout ... sdk: mitigate api test timeout	4 years ago