358 Commits (stacks-variable-validations)

Author	SHA1	Message	Date
sahar-azizighannad	ec5959f80a	improvements	2 months ago
sahar-azizighannad	14116ad820	Merge branch 'main' into stacks-variable-validations	3 months ago
sahar-azizighannad	4333e71d22	make the validation	3 months ago
Radek Simko	0fe906fa8c	make copyrightfix	3 months ago
Daniel Schmidt	a2a8697558	add test case for variable validation	3 months ago
Daniel Schmidt	4b34066cf2	add error to Pre and PostDiff hook ... We need this information for stacks to forward the error to the progress stream as stacks expects everything that reports itself starting to e.g. plan to also finish either in an error or with success.	4 months ago
Daniel Schmidt	f4c22be3c5	stacks: send progress events on failed provider configurations	4 months ago
Daniel Schmidt	b926263c03	stacks: follow component progress cycle for empty destroys ... We expect a component instances to report its plan/apply starting and ending as well as reporting the progress / result. This should also be the case for no-ops like an empty component instance.	4 months ago
Nick Fagerlund	b2aad914f1	stacks: Make path.module and path.root relative, to match documentation (#37982) ... * stacks: Make path.module and path.root relative, to match documentation Previously, we were just handing off the absolute path produced by the sourcebundle struct. But that's no good, because the value doesn't stay consistent between plan and apply when running under tfc-agent. (It uses working directories named after the unique job ID.) It also doesn't match the documentation, which describes these as relative paths. This was preventing people from uploading module-provided files to create aws lambda functions, for example. This commit addresses that by converting the source-bundle provided module path to a relative path (relative to Terraform's working directory). In tfc-agent for stacks runs, that ends up being the directory directly above the sourcebundle directory, and all the paths below that are consistent between plan and apply. * I think this is how the changelog thing works?	5 months ago
Matej Risek	768b7bd0bc	Add local values walk to walk_dynamic ... This helps us catch diagnostics in locals that only happen during evaluation Co-authored-by: Mutahhir Hayat <mutahhir.hayat@hashicorp.com> Co-authored-by: Matej Risek <matej.risek@hashicorp.com>	5 months ago
Liam Cervante	7ac2b7938e	Stacks: temporarily disable ephemeral outputs in Stack components (#37823)	7 months ago
Liam Cervante	411f18e6ac	ephemerals: allow root ephemeral outputs in stacks and test (#37813) ... * ephemerals: allow root ephemeral outputs in stacks and test * remember to set new opt for apply stage	7 months ago
Daniel Schmidt	2714cfde69	standardize on marks.Has	7 months ago
Radek Simko	922fdb2382	Implement LockState & UnlockState provider methods (#37711)	8 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>	8 months ago
Liam Cervante	15a6cd208b	stacks: fix diagnostic referring to tfstacks command (#37639)	8 months ago
Liam Cervante	0b15151513	stacks: remove support for deprecated .tfstack extension (#37617) ... * stacks: remove support for deprecated .tfstack extension * also remove from comments and readme	8 months ago
James Bardin	88d64326d3	stacks stubs	9 months ago
Liam Cervante	abf1c00e03	stacks migrate: make diagnostics more relevant to operation (#37416)	9 months ago
Kristin Laemmert	0374f04424	Add ValidateActionConfig to provider interface, protocol & internal implementations (#37345) ... * Add ValidateActionConfig to provider interface and protocol * Update internal/provider-simple-v6/provider.go Co-authored-by: Daniel Schmidt <danielmschmidt92@gmail.com> --------- Co-authored-by: Daniel Schmidt <danielmschmidt92@gmail.com>	10 months ago
Daniel Schmidt	688a1f9955	add todo	10 months ago
Daniel Schmidt	ac132d1b15	add provider implementation for actions	10 months ago
Liam Cervante	2a1b4c3ed4	Update go-cty and HCL for 1.13 release (#37298)	10 months ago
Sarah French	0f8e77a50f	PSS: Add GetStates and DeleteState RPCs to plugin protocol v6 (#37288) ... * Add GetStates RPC method to Plugin Protocol v6 * Add DeleteState RPC method to Plugin Protocol v6 * Add methods to provider-simple * Fix error messages * Apply suggestions from code review Co-authored-by: Radek Simko <radeksimko@users.noreply.github.com> * Change how GetStates RPC returns the list of state names * Change GetStates implementation to use new data type * `make protobuf` to accommodate new code comments --------- Co-authored-by: Radek Simko <radeksimko@users.noreply.github.com>	10 months ago
Matej Risek	a9b67a6cdc	stacks: Don't require built-in providers to be listed under required providers. (#37234)	11 months ago
Daniel Banck	cb58a15f3d	Implement ListResource RPC (TF-25509) (#37092) ... * Implement ListResource RPC via callbacks * Implement ListResource with iterators * Change `ListResourceResponse` to slice * Turn ListResourceResponse into a struct * Add a limit for ListResource * explicitly cancel stream when listing resources --------- Co-authored-by: Samsondeen Dare <samsondeen.dare@hashicorp.com>	12 months ago
Radek Simko	643266dc90	Add initial state store protocol skeleton (#37197) ... * Add initial state store protocol skeleton Introduce ValidateStateStoreConfig + ConfigureStateStore Update GetProviderSchema + GetMetadata (for mux) * address PR feedback	12 months ago
Daniel Banck	44ae7da18e	Update Plugin Protocol for List and Implement `ValidateListResourceConfig` (#37007) ... * Minor auto-formatting changes * Add list RPC and schema protobuf definitions * make protobuf * make generate * Add ValidateListResourceConfig implementation	1 year ago
Liam Cervante	b6be635748	stacks migrate: allow resource mapping to include nested resources (#37060) ... * stacks migrate: allow resource mapping to include nested resources * make protobuf	1 year ago
Liam Cervante	3d4391865d	stacks: skip resource instance hooks during refresh plans (#37043) ... * stacks: skip resource instance hooks during refresh plans * add copywrite headers	1 year ago
Liam Cervante	a80220dcc9	stacks: rename .tfstack files to .tfcomponent (#37013) ... * stacks: rename .tfstack files to .tfcomponent * fix consistency issues	1 year ago
James Bardin	807e084130	final renaming for function results	1 year ago
James Bardin	a6ec17cb77	rename function results table ... Remove "provider" from the function results fields since it's not going to be exclusively used for providers any longer.	1 year ago
James Bardin	d016070564	move function results hashing to lang ... We need to abstract the function results verification to use internally too, so start by moving it out of the providers code.	1 year ago
Liam Cervante	1bc7d22ac4	stacks: removed blocks should refresh during refresh plans (#36916) ... * stacks: removed blocks should refresh during refresh plans * fix copywrite headers * clarify return values of DefinedByStackInstance	1 year ago
Liam Cervante	dcff9e4d2c	stacks: improve test coverage for removed blocks (#36914) ... * stacks: improve test coverage for removed blocks * remove leftover comment	1 year ago
Liam Cervante	4eaa9d7fa0	stacks: removing embedded stacks should ignore stacks not in state (#36901)	1 year ago
Liam Cervante	063757ff45	stacks: refactor plan, state, and removed tracking with tree structures for efficient lookups (#36850)	1 year ago
Samsondeen	be242c36f4	Stacks migrate: migration workflow improvements (#36882)	1 year ago
Liam Cervante	a06f82746a	stacks: update removed blocks to allow targeting of embedded stacks (#36814) ... * stacks: update removed blocks to allow targeting of embedded stacks * copywrite headers	1 year ago
Liam Cervante	2b5101f734	stacks: include existing components when deferring nested stacks (#36788) ... * stacks: include existing components when deferring nested stacks * improve comments	1 year ago
Liam Cervante	fbd5a7930e	stacks: allow removed blocks to target components in nested stacks (#36787) ... * stacks: allow removed blocks to target components in nested stacks * make copywrite fix * fix comments	1 year ago
Liam Cervante	7fadbe34de	write-only attributes: internal providers should set write-only attributes to null (#36824) ... * write-only attributes: internal providers should set write-only attributes to null * add changelog * fix copywrite headers	1 year ago
Liam Cervante	c8044baf47	stacks: emit state updates for unclaimed components (#36702) ... * stacks: allow multiple removed blocks to target the same component * make linter happy * stacks: emit state updates for unclaimed components	1 year ago
Liam Cervante	9707a27ee6	stacks: rename Removed to RemovedComponent in stackeval (#36782) ... * stacks: provide stack and config on component creation * stacks: rename Removed to RemovedComponent in stackeval * renamed function calls also	1 year ago
Liam Cervante	7bec38bf5f	stacks: provide stack and config on component creation (#36778)	1 year ago
Liam Cervante	ebdf52cd09	stacks: remove all the unnecessary ctx function parameters (#36762) ... * stacks: remove all the unnecessary ctx function parameters * fix missing context	1 year ago
Liam Cervante	f82ae44484	stacks: make promises identify themselves (#36749) ... * stacks: refactor promises so they declare id when starting * stacks: add some missing perEval promises * formatting	1 year ago
Samsondeen	71dbc7d726	migrate command for terraform stacks (#36482)	1 year ago
Samsondeen	8b7e7ad27d	Detect circular references during Validation (#36709)	1 year ago