terraform

Commit Graph

Author	SHA1	Message	Date
Daniel Schmidt	e0748b30c4	add interface checks	7 months ago
Daniel Schmidt	7721cd5c50	refactor: make action diff transformer only responsible for attaching action invocations	7 months ago
Daniel Schmidt	327a9922f6	refactor: rename action trigger apply to instance apply	7 months ago
Daniel Schmidt	4322ab20a1	refactor: add abstract action trigger	7 months ago
Daniel Schmidt	0031dcc9ca	refactor: rename targets to action targets	7 months ago
Daniel Schmidt	231b801d4b	refactor: separate action triggers from action invocations through --invoke	7 months ago
Daniel Schmidt	642a27e6c0	refactor: rename action plan to config transformer	7 months ago
Sarah French	d573104221	PSS: Add `stateStoreInitFromConfig` method to `Meta` (#37723 ) * Add SetStateStoreChunkSize to the mock provider for tests * Implement configurable state chunk size * Add chunk size negotiation to `savedStateStore`, update happy path test to assert it's set * Update `savedStateStore` to return diagnostic if a nil factory is passed in, add unhappy path tests * Add test coverage for stateStoreConfig, including use of config overrides with state stores * Add godoc comment for backendInitFromConfig * Avoid nil error in stateStoreConfig when there's no provider factory * Add stateStoreInitFromConfig method and test coverage * Refactor stateStoreInitFromConfig to accept factory directly, instead of accepting all BackendOpts This is to avoid confusion, e.g. unnecessary duplication of config being passed in. * Implement configurable state chunk size * Remove TODO which has since been addressed * Update happy path test for `stateStoreInitFromConfig` to check that chunk size is negotiated * Update chunk size negotiation code in `stateStoreInitFromConfig` to error if the provider doesn't return a chunk size > 0 * Fix test error message --------- Co-authored-by: Radek Simko <radek.simko@gmail.com>	7 months ago
Sarah French	17445f6c5f	PSS: Add chunk size negotiation to `savedStateStore` method, update mock provider for tests. (#37726 ) * Add SetStateStoreChunkSize to the mock provider for tests * Implement configurable state chunk size * Add chunk size negotiation to `savedStateStore`, update happy path test to assert it's set * Update `savedStateStore` to return diagnostic if a nil factory is passed in, add unhappy path tests * Fix test error message * Apply suggestions from code review Co-authored-by: Radek Simko <radeksimko@users.noreply.github.com> * Fix rename --------- Co-authored-by: Radek Simko <radek.simko@gmail.com> Co-authored-by: Radek Simko <radeksimko@users.noreply.github.com>	7 months ago
Daniel Banck	62540da015	search: Drop identity from state if no identity schema is present (#37709 ) * Modernize Go * Check if identity schema is present before decoding state * Test identity decoding from state without schema	7 months ago
Sarah French	aed61af66c	PSS: Implement `Get` and `Put` methods on remote grpc state, using Read/WriteStateBytes RPCs (#37717 ) * Implement Get method on remote gRPC state client, add tests * Implement Put method on remote gRPC state client, add tests	7 months ago
Sarah French	312f296c2d	PSS: Add `savedStateStore` method to `Meta` (#37558 ) * Add test coverage for Meta's `savedBackend` method * Add new Meta `savedStateStore` method and test coverage * Streamline test - remove unneeded assertions and update comments * Remove marks from config before configuring the provider * Remove marks from config before configuring the state store * Add test case for savedStateStore to assert marks aren't passed * Fix call to ConfigureStateStore * Show that tests pass despite not trying to remove marks * Allow Config methods to add marks when reading pluggable state store config from the backend state file * This code is now necessary to let the tests pass * Stop adding marks to PSS-related config when it's parsed from the backend state file * Stop removing marks that aren't there * Remove unnecessary test related to marks	7 months ago
Radek Simko	922fdb2382	Implement LockState & UnlockState provider methods (#37711 )	7 months ago
Akella Umesh	ea20db63fb	actions: add missing actions output for terraform apply cli (#37692 )	7 months ago
davidspielmann	04c95feb8d	Improve tests in internal/states (ProviderAddrs, HasCurrent, Unlock) (#37650 )	7 months ago
James Bardin	221fc59f0e	Merge pull request #37713 from hashicorp/jbardin/list-gen-config get correct schema when generating query config	7 months ago
James Bardin	9c2b2c579a	remove id from generated config turns out the invalid id attributes were captured in the command test, so removing them caused that to fail too.	7 months ago
James Bardin	82889ae987	get correct schema when generating query config During the latest refactor, the wrong schema was fetched when generating configuration from a list resource. Having the list resource schema instead of the managed resource schema prevented the config generation from matching the computed attributes within the state, causing them to show up in the configuration. It turns out the tests for this didn't have computed attributes, so the accidental swap during refactoring didn't cause the tests to fail. A small addition to the tests adding an `id` attribute picks the regression.	7 months ago
Samsondeen	2274026c68	query: add -query flag to validate command (#37671 )	7 months ago
Liam Cervante	898665193c	actions: disable support for deferrals (#37700 )	7 months ago
Sarah French	ffbb62b369	Swap to using `google.golang.org/protobuf/cmd/protoc-gen-go` instead of `github.com/golang/protobuf/protoc-gen-go` (#37685 ) * Reformat Args in steps definitions * Swap to using the new version of protoc-gen-go that doesn't include grpc support (instead it relies on protoc-gen-go-grpc). Update command flags to reflect this change in tooling. See this comment for explanation: https://github.com/golang/protobuf/issues/1070#issuecomment-607293496 * Run `make protobuf` with problem generate steps commented out. * Use `require_unimplemented_servers=false` to minimise changes at the time of upgrading tooling In future we can navigate the consequences of this in its own PR. * WIP * Fix out/opt typo, add remaining flags once terraform1.proto was being found ok * Run `make protobuf` * `go get google.golang.org/grpc/cmd/protoc-gen-go-grpc@v1.3.0` && `go mod tidy` This version includes a feature that allows the copyright comment in the .proto file to be pulled across to generated _grpc.pb.go files. * Run `make protobuf` with new version of protoc-gen-go-grpc that allows copyright comment to be copied to generated output files * Potential fix for `internal/rpcapi/dynrpcserver/generator`	7 months ago
Samsondeen	0217dd2b2a	Revert "temporarily require manager approval for go.mod changes" (#37694 )	7 months ago
Nara Kasbergen Kwon	eae5ac13dd	build: Produce a Windows arm64 binary (#35461 )	7 months ago
Sarah French	c0547e784f	Fixes to protobuf compile script (#37683 ) * Fix lack of protoc-gen-go-grpc This appears to have been removed in https://github.com/hashicorp/terraform/pull/29492/files#diff-0329966ed7ce7286cfaf91161ac003af419cc566dd6420bb07670f9e4a389c81 accidentally following linting? * Fix Args to include all arguments The docs say "Args holds command line arguments, including the command as Args[0]." * Add TODO for swapping to non-deprecated repo for protoc-gen-go	7 months ago
Kristin Laemmert	8ed2f39087	actions: don't panic if nil changes - it might be a refresh (#37687 )	7 months ago
Radek Simko	51fc4c161f	PSS: Implement `ReadStateBytes` + `WriteStateBytes` (#37440 ) * Implement ReadStateBytes + WriteStateBytes * [WIP] - Testing ReadStateBytes and WriteStateBytes (#37464) * Fix nil pointer error * Add WIP test for ReadStateBytes * Move test mock to separate testing file * Update mock to send unexpected EOF when there's a problem returning data and it's not a true EOF * Add test case for when length != expected length * Add test for when trying to read state from a store type that doesn't exist * Change symbol names to lowercase * Add ability to force a diagnostic to be returned from `mockReadStateBytesClient`'s `Recv` method * Add test showing error diagnostics raised by the ReadStateBytes client are returned * Add missing header * Simplify mock by using an embedded type * Rename `mockOpts` to `mockReadStateBytesOpts` * Update existing tests to assert what arguments are passed to the RPC method call * Add mock WriteStateBytesClient which uses `go.uber.org/mock/gomock` to enable assertions about calls to Send * Add a test for WriteStateBytes that makes assertions about calls to the Send method * Update test case to explicitly test writing data smaller than the chunk size * Implement chunking in WriteStateBytes, add test case to assert expected chunking behaviour * Add generated mock for Provider_WriteStateBytesClient in protocol v6 * Update tests to use new `MockProvider_WriteStateBytesClient`, remove handwritten mock * Update code comments in test * Add tests for diagnostics and errors returned during WriteStateBytes * Add generated mock for Provider_ReadStateBytesClient in protocol v6, replace old mock * Add test case for grpc errors in ReadStateBytes, fix how error is returned * Typo in comment * Add missing warning test, rename some test cases * Update proto file definition of Read/WriteStateBytes RPCs (#37529) * Update Read/WriteStateBytes RPCs to match https://github.com/hashicorp/terraform-plugin-go/pull/531 * Run `make protobuf` * Run `make generate` * Update use of `proto.ReadStateBytes_ResponseChunk` in tests * Fix how diagnostics are handled alongside EOF error, update ReadStateBytes test * More fixes - test setup was incorrect I think? I assume that a response would be returned full of zero-values when EOF is encountered. * WIP - avoid crash if chunk is nil * Sarah's updates to radek/pss-read-write (#37642) * Update code to not expect a chunk when EOF encountered * Return early if any grpc errors are encountered during ReadStateBytes * Close the stream with CloseSend once everything's read without error. Add test case about handling grpc errors from CloseSend. * Fix test case about warnings: We would expect to receive a chunk with data alongside the warning and have a normal closing of the stream after EOF * Add log line, remove unneeded type info * Implement configurable state chunk size * handle metadata in WriteStateBytes correctly * validate chunk sizes received from provider * ReadStateBytes: avoid early return on warnings --------- Co-authored-by: Sarah French <15078782+SarahFrench@users.noreply.github.com>	7 months ago
Daniel Schmidt	a4cc769e5b	actions: add extra test cases around conditions referencing the triggering resource	7 months ago
Samsondeen	ab41592fb7	query: propagate graph node removal to descendants (#37664 )	7 months ago
Sarah French	b9522561c8	PSS: Add reusable method for obtaining the provider factory needed for accessing a state store. (#37665 ) * Add method to allow accessing factories from locks that are in memory * Create new getStateStoreProviderFactory method for accessing a factory from config * Update getStateStoreProviderFactory to use in memory locks instead of reading them from the deps lock file. Update calling code to accommodate this. * Add tests for getStateStoreProviderFactory, improve errors returned from method * Update test following schema change in simple providers * Move test case into e2etest package, so we protect against environments where building binaries isn't possible * Fix issues with running test in e2etest package * Update code comments Co-authored-by: Radek Simko <radeksimko@users.noreply.github.com> --------- Co-authored-by: Radek Simko <radeksimko@users.noreply.github.com>	7 months ago
Liam Cervante	1e414491c7	evaluate: return diagnostics instead of unknown for uninitialised locals and resources (#37663 ) * evaluate: return diagnostics instead of unknown for uninitialised locals and resources * changelog * also input variables	7 months ago
Samsondeen	a7504719ef	list: send non-null "config" object to provider when not present in the list block (#37620 )	7 months ago
Kristin Laemmert	f6ad8d3215	actions: fix panic during query plan by not planning actions We don't need to plan any actions during query plan mode. Test stolen from DanielMSchmidt, thank you!	7 months ago
Craig Wright	152ce1df8f	Merge pull request #37638 from Tensho/fix-destroying-docs docs(destroying): fix typos	7 months ago
Andrew Babichev	8097ce9e57	docs(destroying): fix typos	7 months ago
Mark DeCrane	f8b0051bae	Merge pull request #37645 from hashicorp/handle-help-usage-stacks-plugin Handle `-help` usage in stacks plugin	7 months ago
Liam Cervante	e65c562355	actions: allow invoke only on local execution while TFC adds support (#37653 ) * actions: allow invoke only on local execution while TFC adds support * actually return early	7 months ago
Mark DeCrane	7f0ef4c8d0	changelog entry	7 months ago
Mark DeCrane	10f00d437f	Use raw args in -help flag handling	7 months ago
Kristin Laemmert	e241e1ace1	actions: return an error if config is omitted but the schema has required attrs (#37626 ) * actions: improve handling of null config so we can properly report missing required arguments There are various methods in terraform that will let you know if you are missing required attributes - none of which work with nil inputs. This commit changes the handling, passing in an empty object if the config was null, and adding additional context to the error message when the config block is missing.	7 months ago
Sarah French	0dfa115d7d	First step to enable use of TF_REATTACH_PROVIDERS with PSS (#37634 ) * Add test coverage for `parseReattachProviders` * Add unhappy path test cases to parseReattachProviders test * Add `isProviderReattached` function, to help identify when a reattached provider is being used for PSS * Move reattach config-related code into its own package * Make calling code not need to know what the ENV is for reattach providers * Refactor IsProviderReattached to use pre-existing logic * Add test case, make spell check happy with US English * Add headers * Make calling code responsible for reading in the related ENV * Add godoc comment to TF_REATTACH_PROVIDERS const * Update internal/getproviders/reattach/reattach.go	7 months ago
Liam Cervante	15a6cd208b	stacks: fix diagnostic referring to tfstacks command (#37639 )	7 months ago
Samsondeen	719aefdb89	tf query: ready list blocks for beta (#37619 )	7 months ago
Sarah French	5981cd2bf7	Prepare main branch for post-v1.14 development (#37621 ) * Prepare main branch for post-v1.14 development * Update pull_request_template.md to reflect how main == v1.15 now	7 months ago
Daniel Schmidt	332e0386ea	remove feature flag from invoke flag	7 months ago
Daniel Schmidt	2ccc5a2655	change wording	7 months ago
Daniel Schmidt	8f05946b6e	remove ooutdated comment	7 months ago
Daniel Schmidt	e23a282186	improve wording	7 months ago
Daniel Schmidt	d83cd73ecf	fix action reference	7 months ago
Daniel Schmidt	c5f9b060c7	remove mention of before_destroy, we postponed implementing this event type	7 months ago
Daniel Schmidt	b0393f08da	add changelog entry	7 months ago

1 2 3 4 5 ...

34772 Commits (e0748b30c40ba0e8166a1ffd01d427719206005a) All Branches Search

34772 Commits (e0748b30c40ba0e8166a1ffd01d427719206005a)

All Branches