93 Commits (dependabot/github_actions/github-actions-backward-compatible-d013ecdca1)

Author	SHA1	Message	Date
Sarah French	2dbb7d9c05	PSS: Remove automatic creation of the default workspace's state file during `init` (#38281) ... * refactor: Stop Terraform creating the default workspace during init when using PSS This is part of reconciling how in the past backends always reported that the default backend existed, even when it didn't. We want state stores to report reality only, so we need to let Terraform handle the discrepancy. Prior to this commit we handled it by making reality match the old lie that the default workspace always exists. After this commit we're just embracing Terraform working with truthful information.	3 weeks ago
Daniel Banck	0fa9e5b4da	command: Add vars to get command (and refactor it)	3 weeks ago
Daniel Banck	cdbd4f17f2	command: Add vars to taint command	3 weeks ago
Daniel Banck	a836cd610d	command: Add vars to state rm command	3 weeks ago
Daniel Banck	2115032765	command: Add vars to state replace command	3 weeks ago
Daniel Banck	36b26535b9	command: Add vars to state push command	3 weeks ago
Daniel Banck	aac8834600	command: Add vars to state pull command	3 weeks ago
Daniel Banck	b2a5ce8af1	command: Add vars to state mv command	3 weeks ago
Daniel Banck	c28c6c6d64	command: Add vars to providers schema command	3 weeks ago
Daniel Banck	ed2bdf6825	command: Add vars to providers mirror command	3 weeks ago
Daniel Banck	6c6be5bcfa	command: Add vars to pvoiders lock command	3 weeks ago
Daniel Banck	1cc8345926	command: Add vars to providers command	3 weeks ago
Daniel Banck	c9e1b3a47b	command: Add vars to modules command	3 weeks ago
Daniel Banck	a1c74c7d4e	command: Add vars to graph command	3 weeks ago
Daniel Banck	4b132a4879	command: Add vars to validate command	3 weeks ago
Daniel Banck	f9cfdf1ebe	Refactoring: Modernize graph command to use arguments	4 weeks ago
Daniel Banck	5a70c424b3	Fix import and show command ... This commit moves some code around to fix configuration loading during the (legacy) import command. And add vars to the show command.	1 month ago
Daniel Schmidt	de068da2ab	refactor addVarsFromFile to use return value	2 months ago
Daniel Schmidt	4e921a8eae	move CollectValuesFromTests to arguments package ... This keeps the CollectValues and CollectValuesFromTests implementations together.	2 months ago
Daniel Schmidt	f6a3f271be	refactor console command to use arguments	2 months ago
Daniel Schmidt	8faccde780	implement centralized variable collection	2 months ago
Daniel Schmidt	026c935961	move UnparsedVariableValue from backendrun to arguments ... This prevents a cyclic dependency and also makes sense semantically. The arguments package will collect the unparsed variable values and the backendrun helpers will work to collect the values and transform them into terraform.InputValue.	2 months ago
Daniel Schmidt	90eae782bf	move vars handling into own file	2 months ago
Daniel Schmidt	81b328f924	update copyright headers	2 months ago
Daniel Schmidt	592d0fe4dd	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	5e51b831be	refactor state-replace-provider command argument parsing	2 months ago
Daniel Schmidt	32fb18572d	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	89bea5de5b	refactor state-rm command argument parsing	2 months ago
Daniel Schmidt	c66d0d72c1	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	442126553b	refactor state-push command argument parsing	2 months ago
Daniel Schmidt	b9f6f14003	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	9f3c282096	refactor state-mv command argument parsing	2 months ago
Daniel Schmidt	52bbc57c62	refactor state-pull command argument parsing	2 months ago
Daniel Schmidt	acb6ed3f22	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	f7cde006ff	refactor state-show command argument parsing	2 months ago
Daniel Schmidt	da74b59ea1	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	3dbfbe5dc9	refactor state-list command argument parsing	2 months ago
Daniel Schmidt	af7783eb62	refactor providers lock command argument parsing	2 months ago
Daniel Schmidt	c7729926fa	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	0066780049	refactor providers mirror command argument parsing	2 months ago
Daniel Schmidt	c15f0409ed	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	0c384e8cd8	refactor providers schema command argument parsing	2 months ago
Daniel Schmidt	6f32f249f7	refactor providers command argument parsing	2 months ago
Daniel Schmidt	3a2686938f	use the arguments package for import command	2 months ago
Daniel Schmidt	61923826f8	don't explicitly specify defaults in argument parsing	2 months ago
Daniel Schmidt	5a9c0196d9	let command handle defaults	2 months ago
Daniel Schmidt	6b83486498	use the arguments package for taint commands	2 months ago
Radek Simko	0fe906fa8c	make copyrightfix	2 months ago
Sarah French	f2818db795	PSS : Add `fs` and `inmem` state storage implementations to the builtin `simplev6` provider, update `grpcwrap` package, use PSS implementation in E2E test (#37790) ... * feat: Implement `inmem` state store in provider-simple-v6 * feat: Add filesystem state store `fs` in provider-simple-v6, no locking implemented * refactor: Move PSS chunking-related constants into the `pluggable` package, so they can be reused. * feat: Implement PSS-related methods in grpcwrap package * test: Add E2E test checking an init and apply (no plan) workflow is usable with both PSS implementations * fix: Ensure state stores are configured with a suggested chunk size from Core --------- Co-authored-by: Radek Simko <radeksimko@users.noreply.github.com>	5 months ago
Sarah French	6b73f710f8	PSS: Implement initialisation of new working directory (or use of `-reconfigure` flag) while using `state_store` (#37732) ... * Minor fixes in diagnostics This can only be done once modules have been parsed and the required providers data is available. There are multiple places where config is parsed, into either Config or Module structs, so this needs to be implemented in multiple places. * Rename test to make it specific to use of backend block in config * Update initBackend to accept whole initArgs collection * Only process --backend-config data, when setting up a `backend`, if that data isn't empty * Simplify how mock provider factories are made in tests * Update mock provider's default logic to track and manage existing workspaces * Add `ProviderSchema` method to `Pluggable` structs. This allows calling code to access the provider schema when using provider configuration data. * Add function for converting a providerreqs.Version to a hashicorp/go-version Version. This is needed for using locks when creating the backend state file. * Implement initial version of init new working directories using `stateStore_C_s`. Default to creating the default workspace if no workspaces exist. * Update test fixtures to match the hashicorp/test mock provider used in PSS tests * Allow tests to obtain locks that include `testingOverrides` providers. The `testingOverrides` field will only be set in tests, so this should not impact end users. * Add tests showing TF can initialize a working directory for the first time (and do the same when forced by -reconfigure flag). Remove replaced tests. * Add -create-default-workspace flag, to be used to disable creating the default workspace by default when -input=false (i.e for use in CI). Refactor creation of default workspace logic. Add tests. * Allow reattached providers to be used during init for PSS * Rename variable to `backendHash` so relation to `backend` is clearer * Allow `(m *Meta) Backend` to return warning diagnostics * Protect against nil testingOverrides in providerFactoriesFromLocks * Add test case seeing what happens if default workspace selected, doesn't exist, but other workspaces do exist. The consequences here are due to using `selectWorkspace` in `stateStore_C_s`, matching what's done in `backend_C_r_s`. * Address code consistency check failure on PR * Refactor use of mock in test that's experiencing EOF error... * Remove test that requires test to supply input for user prompt This test passes when run in isolation but fails when run alongside other tests, even when skipping all other tests using `testStdinPipe`. I don't think the value of this test is great enough to start changing how we test stdin input. * Allow -create-default-workspace to be used regardless of whether input is enabled or disabled * Add TF_SKIP_CREATE_DEFAULT_WORKSPACE environment variable * Responses to feedback, including making testStdinPipe helper log details of errors copying data to stdin. Note: We cannot call t.Fatal from a non-test goroutine. * Use Errorf instead * Allow backend state files to not include version data when a builtin or reattached provider is in use. * Add clarifying comment about re-attached providers when finding the matching entry in required_providers * Report that the default workspace was created to the view * Refactor: use error comparison via `errors.Is` to identify when no workspaces exist. * Move handling of TF_ENABLE_PLUGGABLE_STATE_STORAGE into init's ParseInit func. * Validate that PSS-related flags can only be used when experiments are enabled, enforce coupling of PSS-related flags when in use. * Slight rewording of output message about default workspace * Update test to assert new output about default workspace	6 months ago