1097 Commits (testing-for-tracing)

Author	SHA1	Message	Date
Daniel Schmidt	d2fe105459	add testing for tracing	2 months ago
Daniel Banck	977a0fb117	Potentially fetch backend variables for most commands ... * modules * providers * providers lock * providers mirror * validate * state mv * state pull * state push * state replace provider * state rm * taint * get	2 months ago
Daniel Banck	5bf90e356f	Allow commands to fetch backend variables ... The new `resolveConstVariables` helper checks if const variables are present in the configuration, but don't have a value yet. In that case we try to fetch them via a backend. This will only work for backends that implement the ConstVariableSupplier interface. (Which the `cloud` backend does) The precedence is the same as for existing commands: a CLI supplied value will override a value from a workspace	2 months ago
Daniel Banck	97534ebd3b	Improve ParseVariableValues signature with a helper ... Instead of passing a "magic" boolean as third parameter, we now have two functions `ParseConstVariableValues` and `ParseVariableValues`.	2 months ago
Daniel Banck	0fa9e5b4da	command: Add vars to get command (and refactor it)	2 months ago
Daniel Banck	cdbd4f17f2	command: Add vars to taint command	2 months ago
Daniel Banck	a836cd610d	command: Add vars to state rm command	2 months ago
Daniel Banck	2115032765	command: Add vars to state replace command	2 months ago
Daniel Banck	36b26535b9	command: Add vars to state push command	2 months ago
Daniel Banck	aac8834600	command: Add vars to state pull command	2 months ago
Daniel Banck	b2a5ce8af1	command: Add vars to state mv command	2 months ago
Daniel Banck	c28c6c6d64	command: Add vars to providers schema command	2 months ago
Daniel Banck	ed2bdf6825	command: Add vars to providers mirror command	2 months ago
Daniel Banck	6c6be5bcfa	command: Add vars to pvoiders lock command	2 months ago
Daniel Banck	1cc8345926	command: Add vars to providers command	2 months ago
Daniel Banck	c9e1b3a47b	command: Add vars to modules command	2 months ago
Daniel Banck	a1c74c7d4e	command: Add vars to graph command	2 months ago
Daniel Banck	4b132a4879	command: Add vars to validate command	2 months ago
Sarah French	d7baa140f8	fix: Add missing or correct values to error diagnostics raise when initialising a state store from backend state file data (#38275)	2 months ago
Daniel Banck	a47b89170f	Fix ParseVariableValues for different command flows ... We want to treat `const` variables as part of the configuration, we always require a value for them when loading configuration. Since most command load configuration via `Meta.loadConfig` or `Meta.loadConfigWithTests`, these flows call `ParseVariableValues` with `constOnly` true. Backend operations like `plan` and `apply` require values for all required variables (as before), so here we set `constOnly` to false, to keep the existing logic.	2 months ago
Roniece	7a9577e310	Lint	2 months ago
Roniece	1084dccefa	Add applied action invocation message type	2 months ago
James Bardin	acd6a7e09a	Merge pull request #38264 from hashicorp/jbardin/path-matcher ... diff: don't panic on invalid keys for relevant attrs	2 months ago
James Bardin	a5077e7ddb	don't panic on invalid keys for relevant attrs ... The PathMatcher used when rendering diffs assumes the caller knows the structure the path was derived from, which is incorrect. RelevantAttributes is derived from references to an object, which may have errors hidden by `try` or `can` functions, or the data may not have been updated to match paths via targeted operations. Even in the case where the data may be incorrect, we can't crash when rendering the data, because the user may not be able to work around the panic with no other information about which resource contains unexpected references.	3 months ago
Daniel Schmidt	e4f91190c9	add tests for dynamic module sources in terraform test	3 months ago
Daniel Banck	f9cfdf1ebe	Refactoring: Modernize graph command to use arguments	3 months ago
Sarah French	ac7206c919	test: Fix E2E tests to use correct errors for assertions, remove use of ioutil in package (#38254)	3 months ago
Daniel Schmidt	a9756b273c	move initwd config loading into terraform ... otherwise we run into cyclic references	3 months ago
Daniel Schmidt	0c796986bb	move load config calls to new mechanism	3 months ago
Daniel Banck	02723fcd79	Add variable arguments to init help text	3 months ago
Daniel Banck	45ba6796ba	Add more dynamic module sources tests	3 months ago
Daniel Banck	b6804e2edd	Use configuration values for jsonconfig ... Instead of outputting the raw values from the ModuleCall, we're now using the evaluated values from the Config. This should keep the JSON output unchanged, we just need to make sure to persist the values in `Config` when loading the configuration.	3 months ago
Daniel Banck	55198eb221	Feedback: Rename remaining static -> const	3 months 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.	3 months ago
Daniel Banck	0043e1ce24	Update module call configuration storage ... This turns the string representation of the source and version attribute of a module call into an expression. This leads to a change in the JSON output: `"source": "./foo"` --> `"source": {"constant_value": "./foo"},`	3 months ago
Daniel Banck	4ef9684188	Rework most of the configuration loading ... We previously used a loader -> BuildConfig flow to load configuration. This commit changes most (but not all yet) flows to use the new graph-based approach. Instead of simply recursively loading the modules, we now need to take a stepped approach: 1. Load the root module 2. Collect the variables and their values 3. Build the configuration with the graph-based approach Because this approach relies on different parts from different packages, it can't easliy be done within the `configload` package. So, now we do most of in the backend or command.	3 months ago
Daniel Banck	fb60c15670	Update init to use new graph-based workflow ... The init command and the module installer are now using the new graph-based workflow to install modules instead of using the recursive BuildConfig.	3 months ago
Radek Simko	7127c76b04	command/views: Bump UI version to v1.3 (#38231)	3 months ago
Radek Simko	07b3e3b800	command/init: Merge duplicated PSS logic back into existing codepath (#38227)	3 months ago
James Bardin	7960f60d21	Merge pull request #38160 from hashicorp/jbardin/inline-type-conversion ... inline type conversions	3 months ago
Sebastian Rivera	305dba5dcc	feat: denote sensitive attribute paths in query start msg ... This new field will be consumed in order to redact sensitive values referenced in a list block's input configuration from the query logs. We simply pipe the config schema and the marked values, and include the paths in the query_start log output.	3 months ago
Kristin Laemmert	0ce1ae7cd6	Merge pull request #38204 from hashicorp/mildwonkey/action-prep ... simplifying actions prep - renaming action-related node types and some internal fields.	3 months ago
Radek Simko	cd9257cd53	PSS: Enable state store configuration change (#38153)	3 months ago
Sarah French	c37cf1c0f5	test: Enable assertions about prompt text shown to users (#38212) ... * test: Update `testInputMap` and `(*UIInput) Input` to allow making assertions about what prompts a user has been shown. Make cleanup happen inside the helper. * test: Update test to assert about prompt contents, showing how changes to `testInputMap` can be used * docs: update godoc for testInputMap	3 months ago
Sarah French	9f5f21f1d8	test: Enable mocking http provider sources in tests (#38206) ... * test: Update`MockSource` to be able to present providers as being installed via HTTP, add `newMockProviderSourceViaHTTP` for direct use in tests. Compare newMockProviderSourceViaHTTP to the existing newMockProviderSource. newMockProviderSource uses FakeInstallablePackageMeta, which creates metadata for providers that says it's being sourced from a local archive. In newMockProviderSourceViaHTTP the metadata reports the provider is downloaded via HTTP. The URL is created using a base URL that should be obtained from a test http server created in the test. * test: Add `newMockProviderSourceUsingTestHttpServer` helper, to abstract away making the http test server. Add test showing it in use. * refactor: Update test helpers to call t.Cleanup internally	3 months ago
Kristin Laemmert	70e074b8aa	chore (actions): rename LifecycleActionTrigger -> ResourceActionTrigger in plan and proto ... We'd previously removed all other references to "lifecycle" actions, which made this reference stand out. ResourceLifecycleActionTrigger is probably the most accurate name, but as this type just needs to be differentiated from InvokeActionTrigger I thought "resource" was enough (and I specifically wanted= to avoid lifecycle at all). I'm not super attached to the name, but I did think it would be clearer if we avoided Lifecycle as much as possible, since that's got some overlap with action subtypes. In this instance, we call it a LifecycleActionTrigger because it's come from the resource's `lifecycle` block. This doesn't directly relate to the concept of LifecycleActions - even if we expand the design to have multiple action types (for example generic and lifecycle actions), both those actions types would use this same Trigger struct.	3 months ago
Daniel Schmidt	cd3fc5ba56	expose deprecation on provider schema	3 months ago
Sarah French	ee3190aab4	fix: Add check that we're able to run the e2e.GoBuild method in `TestPrimary_stateStore_unmanaged_separatePlan` (#38190) ... This change should hopefully address E2E failures that are happening on main. It's hard to test that as the failures are platform specific and neither my laptop nor the runner for tests on PRs matches the failing platforms.	3 months ago
Sarah French	3db7c751a8	PSS: Fix use of reattached providers in init, enable use of reattached providers during plan-apply workflow (#38182) ... * refactor: Check that the state storage provider is present when beginning to initialise a state store for use in a non-init command. Ensure reattached providers can be used. Previously we passed all required providers into backend options to be used within `stateStoreConfig`, which is invoked via (Meta).Backend. The new approach enforces that the provider is present while assembling the backend options passed to (Meta).Backend from (Meta).backend, which is non-init specific. As this code is defending against users running non-init commands before an init, this place feels appropriate and isn't able to impact the init command. * fix: Reattached PSS providers should return early when checking locks, and an empty locks file is only bad if there isn't a reattached PSS provider * test: Assert that running init with reattached PSS provider is ok, via an E2E test that uses the reattach feature. * fix: Allow builtin or reattached providers to be used for state stores when generating a plan file * test: Expand E2E test to show using a reattached provider can be used for a workflow of init, plan with -out, and apply. * chore: Replace 'io/ioutil' and format code in unmanaged e2e tests	3 months ago
Daniel Schmidt	d4d46b38a2	remove unused Meta.variableArgs	3 months ago