35162 Commits (add-missing-variable-loading)

 

All Branches   

Search

Author	SHA1	Message	Date
Tommy O'Keefe	71e86ce563	temporarily require manager approval for go.mod changes	8 months ago
Daniel Schmidt	5da4dc1715	actions: ensure config is known during apply	8 months ago
Daniel Schmidt	ff43fc0227	reorganize action plan tests	8 months ago
Tommy O'Keefe	aa5c8d56dd	temporarily require manager approval for go.mod changes	8 months ago
Liam Cervante	20574d413c	validation: don't strip marks from variables during validation (#37595) ... * validation: don't strip marks from variables during validation * add regression test	8 months ago
Samsondeen	8097374c3d	Fix panic when generating json config for tf query (#37596)	8 months ago
Sarah French	90809dc7e8	Skip TestTest_Parallel due to flakiness (#37594)	8 months ago
hc-github-team-tf-core	098bbe7171	Cleanup after 1.14.0-alpha20250911 release (#37592)	8 months ago
Liam Cervante	49cc60531d	actions: fix cycle when multiple actions are triggered across multiple events (#37591)	8 months ago
hc-github-team-tf-core	31291c551e	Prepare before 1.14.0-alpha20250911 release (#37590)	8 months ago
Kristin Laemmert	31a982ec0f	moduletest: fix error return (#37589)	8 months ago
Liam Cervante	551ba2e525	Implement controlling destroy functionality within Terraform Test (#37359) ... * Add ability to parse backend blocks present in a test file's run blocks, validate configuration (#36541) * Add ability to parse backend blocks from a run block * Add validation to avoid multiple backend blocks across run blocks that use the same internal state file. Update tests. * Add validation to avoid multiple backend blocks within a single run block. Update tests. * Remove use of quotes in diagnostic messages * Add validation to avoid backend blocks being used in plan run blocks. Update tests. * Correct local backend blocks in new test fixtures * Add test to show that different test files can use same backend block for same state key. * Add validation to enforce state-storage backend types are used * Remove TODO comment We only need to consider one file at a time when checking if a state_key already has a backend associated with it; parallelism in `terraform test` is scoped down to individual files. * Add validation to assert that the backend block must be in the first apply command for an internal state * Consolidate backend block validation inside a single if statement * Add initial version of validation that ensures a backend isn't re-used within a file * Explicitly set the state_key at the point of parsing the config TODO: What should be done with method (moduletest.Run).GetStateKey? * Update test fixture now that reusing backend configs has been made invalid * Add automated test showing validation of reused configuration blocks * Skip test due to flakiness, minor change to test config naming * Update test so it tolerates non-deterministic order run blocks are evaluated in * Remove unnecessary value assignment to r.StateKey * Replace use of GetStateKey() with accessing the state key that's now set during test config parsing * Fix bug so that run blocks using child modules get the correct state key set at parsing time * Update acceptance test to also cover scenario where root and child module state keys are in use * Update test name * Add newline to regex * Ensure consistent place where repeat backend error is raised from * Write leftover test state(s) to file (#36614) * Add additional validation that the backend used in a run is a supported type (#36648) * Prevent test run when leftover state data is present (#36685) * `test`: Set the initial state for a state files from a backend, allow the run that defines a backend to write state to the backend (#36646) * Allow use of backend block to set initial state for a state key * Note about alternative place to keep 'backend factories' * Allow the run block defining the backend to write state to it * Fix rebase * Change to accessing backend init functions via ContextOpts * Add tests demonstrating how runs containing backend blocks use and update persisted state * Fix test fixture * Address test failure due to trouble opening the state file This problem doesn't happen on MacOS, so I assume is due to the Linux environment of GitHub runners. * Fix issue with paths properly I hope * Fix defect in test assertion * Pivot back to approach introduced in 4afc3d7 * Let failing tests write to persistent state, add test case covering that. I split the acceptance tests into happy/unhappy paths for this, which required some of the helper functions' declarations to be raised up to package-level. * Change how we update internal state files, so that information about the associated backend is never lost * Fix UpdateStateFile * Ensure that the states map set by TestStateTransformer associates a backend with the correct run. * Misc spelling fixes in comments and a log * Replace state get/set functions with existing helpers (#36747) * Replace state get/set functions with existing helpers * Compare to string representation of state * Compare to string representation of state * Terraform Test: Allow skipping cleanup of entire test file or individual run blocks (#36729) * Add validation to enforce skip_cleanup=false cannot be used with backend blocks (#36857) * Integrate use of backend blocks in tests with skip_cleanup feature (#36848) * Fix nil pointer error, update test to not be table-driven * Make using a backend block implicitly set skip_cleanup to true * Stop state artefacts being created when a backend is in use and no cleanup errors have occurred * Return diagnostics so calling code knows if cleanup experienced issues or not * Update tests to show that when cleanup fails a state artefact is created * Add comment about why diag not returned * Bug fix - actually pull in the state from the state manager! * Split and simplify (?) tests to show the backend block can create and/or reuse prior state * Update test to use new fixtures, assert about state artefact. Fix nil pointer * Update test fixture in use, add guardrail for flakiness of forced error during cleanup * Refactor so resource ID set in only one place * Add documentation for using a `backend` block during `test` (#36832) * Add backend as a documented block in a run block * Add documentation about backend blocks in run blocks. * Make the relationship between backends and state keys more clear, other improvements * More test documentation (#36838) * Terraform Test: cleanup command (#36847) * Allow cleanup of states that depend on prior runs outputs (#36902) * terraform test: refactor graph edge calculation * create fake run block nodes during cleanup operation * tidy up TODOs * fix tests * remove old changes * Update internal/moduletest/graph/node_state_cleanup.go Co-authored-by: Samsondeen <40821565+dsa0x@users.noreply.github.com> * Improve diagnostics around skip_cleanup conflicts (#37385) * Improve diagnostics around skip_cleanup conflicts * remove unused dynamic node * terraform test: refactor manifest file for simplicity (#37412) * test: refactor apply and plan functions so no run block is needed * terraform test: write and load state manifest files * Terraform Test: Allow skipping cleanup of entire test file or individual run blocks (#36729) * terraform test: add support for skip_cleanup attr * terraform test: add cleanup command * terraform test: add backend blocks * pause * fix tests * remove commented code * terraform test: make controlling destroy functionality experimental (#37419) * address comments * Update internal/moduletest/graph/node_state_cleanup.go Co-authored-by: Samsondeen <40821565+dsa0x@users.noreply.github.com> --------- Co-authored-by: Samsondeen <40821565+dsa0x@users.noreply.github.com> * add experimental changelog entries --------- Co-authored-by: Sarah French <15078782+SarahFrench@users.noreply.github.com> Co-authored-by: Samsondeen <40821565+dsa0x@users.noreply.github.com> Co-authored-by: Samsondeen Dare <samsondeen.dare@hashicorp.com>	8 months ago
Kristin Laemmert	e315a07c71	remove destroy_ events from list (#37586) ... I expect these will come back, but since we don't have a firm roadmap just yet it's better not to make future promises.	8 months ago
Samsondeen	9b015c6c28	Allow duplicate list names for different types (#37578)	8 months ago
Liam Cervante	00d680d819	apply: include sensitive metadata when comparing changed input values (#37582) ... * apply: include sensitive metadata when comparing changed input values * correct changelog entry	8 months ago
Daniel Schmidt	003edcf5a1	disallow using the same event in the same action_trigger multiple times	8 months ago
Daniel Schmidt	207f0400fe	plan multiple actions invocations if multiple events are triggered in the same action trigger	8 months ago
Samsondeen	af8cc7e315	test: create all edges to leaf cleanup nodes (#37564)	8 months ago
Kristin Laemmert	bbb8af989c	deps: bump xz to address bug in previous version (#37573) ... * deps: bump xz, bug in previous version	8 months ago
Dirk Avery	74b0925633	Merge pull request #37567 from hashicorp/dep-bump-go-getter ... bump: go-getter to 1.8.0	8 months ago
Kristin Laemmert	cad076409c	Fix e2e tests on windows (#37572) ... * close file explicitly (for windows) * drop the config file in the tf.Path root	8 months ago
Sarah French	67eb3c9210	s3: Fix tests after changing the interface to return diagnostics (#37571) ... * Fix tests after changing the interface to return diagnostics * Fix curly brace position	8 months ago
Kristin Laemmert	cf9c491d9f	fix minor regression in tests (#37570) ... At one point, these tests were explicitly passing an empty cli config filepath so that terraform would ignore any existing cli config file (only relevant if you are running these tests locally), but the behavior changed over time such that it was no longer working for these tests (now if the env var file path is empty, we fall back to the default). The actual behavior is reasonably correct (I've depended on passing in blank config file paths before, so I don't love it, but I don't see a need to break this either) so I've added a small method that drops a blank file in place to fix the behavior and avoid file not found errors.	8 months ago
Kristin Laemmert	f574c9cac9	actions: validate that action referenced in action_trigger exists in config during transform (#37559)	8 months ago
Dirk Avery	92bf04ae16	bump: go-getter to 1.8.0	8 months ago
James Bardin	1cb7d1859f	Merge pull request #37515 from hashicorp/jbardin/generate-config ... provider GenerateResourceConfig	8 months ago
Kristin Laemmert	00da430883	very minor fixups for test convenience (#37555) ... * very minor fixups for test convenience * cleanup mock provider's actions	8 months ago
Sarah French	508234892c	Update `remote.Client` interface's methods to return diagnostics instead of primitive errors (#37502) ... * Update remote package's Client interface to use diagnostics instead of errors * Update all implementations of Client interface to match changes, update calling code (incl. tests) to use diags.	8 months ago
Sarah French	8503c45013	Update `backend.Backend`'s `StateMgr` method to return diagnostics instead of primitive errors (#37496) ... * Fix S3 backend test affected by making the Workspaces method return errors via diagnostics * Address diagnostics comparison issues in test by ensuring expected diagnostics are defined in the context of the config they're triggered by * Fix failing test case `TestBackendConfig_EC2MetadataEndpoint/envvar_invalid_mode` by making `diagnosticBase` struct comparable * Add compile-time checks that diagnostic types fulfil interfaces * Stop diagnosticBase implementing ComparableDiagnostic, re-add S3-specific comparer code to s3 package * Update tests to use the S3-specific comparer again * Fix test case missed in refactoring * Update the backend.Backend interface to use diagnostics as return value from StateMgr method * Fix calls to `Fatalf`	9 months ago
Sarah French	61a7914dd9	Add test helper to create mock provider that contains a state store (#37556) ... * Add a helper for mocking a provider with a state store * Update tests to use the new helper * Update other test fixtures for consistency The related tests pass before and after this change * Update testing helper to return the underlying type, not the interface	9 months ago
Amartya Majumdar	54af16eab9	[IPL-7725] Updating Calls to the Remote HCP Terraform API to Retrieve All Workspace Variables (#37241) ... * [IPL-7725] Resolving merge conflicts * [IPL-7725] Updating go.sum files --------- Co-authored-by: Amartya Majumdar <a.majumdar1041@gmail.com>	9 months ago
hc-github-team-tf-core	6c692b6294	Cleanup after 1.14.0-alpha20250903 release	9 months ago
hc-github-team-tf-core	7b650d109b	Prepare before 1.14.0-alpha20250903 release	9 months ago
zeshan	1baef44036	syncdeps: update x/net to v0.41.0	9 months ago
zeshan	da1a28783a	Fix #36897: Resolve NO_PROXY recognition issue in oss remote backend	9 months ago
Daniel Schmidt	7314a9319d	fix race condition in deferred action test case	9 months ago
Daniel Schmidt	85717fba7e	actions: add number of invoked actions to plan summary	9 months ago
dependabot[bot]	2236ecd044	Bump the github-actions-backward-compatible group with 2 updates (#37543) ... Bumps the github-actions-backward-compatible group with 2 updates: [hashicorp/actions-go-build](https://github.com/hashicorp/actions-go-build) and [actions/cache](https://github.com/actions/cache). Updates `hashicorp/actions-go-build` from 1.1.0 to 1.1.1 - [Release notes](https://github.com/hashicorp/actions-go-build/releases) - [Changelog](https://github.com/hashicorp/actions-go-build/blob/main/CHANGELOG.md) - [Commits](d2504eb37c...b9e2cfba30) Updates `actions/cache` from 4.2.3 to 4.2.4 - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](5a3ec84eff...0400d5f644) --- updated-dependencies: - dependency-name: hashicorp/actions-go-build dependency-version: 1.1.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions-backward-compatible - dependency-name: actions/cache dependency-version: 4.2.4 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions-backward-compatible ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	9 months ago
Kristin Laemmert	cb3fc7974b	test fix: sort DeferredActionInvocations before making assertions (#37547)	9 months ago
Daniel Schmidt	ad4371342f	track partial expanded action invocations through deferred action invocation ... like we do it for resources just simpler because we don't have layered addrs maps	9 months ago
Daniel Schmidt	68ad276d97	all types of action deferrals should lead to deferred triggering resources	9 months ago
Daniel Schmidt	b2c4766d5e	implement review feedback	9 months ago
Daniel Schmidt	1952f2ed23	re-add test cases lost to rebasing	9 months ago
Daniel Schmidt	dffeeec81e	partially expand triggers as well	9 months ago
Daniel Schmidt	b6a9990354	deferring a dependency resource should defer action invocation	9 months ago
Daniel Schmidt	29b2ab15e2	support partial expansion in actions	9 months ago
Daniel Schmidt	a8324e0e21	actions: ensure splat operations are invalid	9 months ago
Daniel Schmidt	f60da0132a	add oxford comma back in ... Co-authored-by: Liam Cervante <liam.cervante@hashicorp.com>	9 months ago
Daniel Schmidt	c8f88d1a31	fix rebasing errors	9 months ago
Daniel Schmidt	92cda9d8fe	add actions prefix for clarity	9 months ago