136 Commits (096510fdfd085fc74479953cc5914113bbacbfde)

Author	SHA1	Message	Date
Glenn Sarti	1773129832	Add tests for cloud backend taskStage	4 years ago
Glenn Sarti	43371b8293	Add support for pre-apply task results in the cloud backend ... Previously the cloud backend only supported the pre and post plan stages. This commit adds support to display the output of the new pre-apply task stage as well.	4 years ago
Glenn Sarti	18da2f0213	Refactor task stage retrieval in cloud backend ... Previously all of the logic to retrieve Task Stages was in the backend_plan.go file. This commit: * Moves the logic to the backend_taskStages.go file. * Replaces using an array to a map indexed by the Task Stage's stage. This makes it easier to find the relevant stage in the list and means the getTaskStageIDByName function is no longer required.	4 years ago
Megan Bang	72ba8a869e	removes EnableForcePush and other unrelated code	4 years ago
Megan Bang	5eaa4c45c0	fix imports	4 years ago
Megan Bang	de8bd5826f	first part of code review comments	4 years ago
Megan Bang	7e5b7b283e	updates for code consistency	4 years ago
Megan Bang	dbf99f17b1	add test and removed backend state from cloud	4 years ago
Megan Bang	bddf6a9b34	updating to use the latest version of cloud/state.go and just pass schemas along to PersistState in the remote state	4 years ago
Megan Bang	344379f5c7	fix cloud state breaking?	4 years ago
Megan Bang	021f1f69e9	updates to cloud state	4 years ago
Megan Bang	4fab46749a	update persist state	4 years ago
Martin Atkins	783a07d9e8	build: Use Go 1.19 ... Go 1.19's "fmt" has some awareness of the new doc comment formatting conventions and adjusts the presentation of the source comments to make it clearer how godoc would interpret them. Therefore this commit includes various updates made by "go fmt" to acheve that. In line with our usual convention that we make stylistic/grammar/spelling tweaks typically only when we're "in the area" changing something else anyway, I also took this opportunity to review most of the comments that this updated to see if there were any other opportunities to improve them.	4 years ago
Glenn Sarti	8562f8a744	Add Pre-Plan Run Tasks to Terraform CLI ... Prevously the cloud backend would only render post-plan run tasks. Now that pre-plan tasks are in beta, this commit updates the plan phase to render pre-plan run tasks. This commit also moves some common code to the common backend as it will be used by other task stages in the future.	4 years ago
Brandon Croft	50d48c635e	remove unused error and fix output authorization detection	4 years ago
Brandon Croft	0139e75a1a	feature: fall back to reading entire state when detailed-type is not available	4 years ago
Brandon Croft	e1fa690879	style: goimports fixes	4 years ago
Brandon Croft	166d21b20b	refactor: extract cloud state ctyValue conversion function	4 years ago
Brandon Croft	e794efc31e	better errors from terraform output when cloud is configured	4 years ago
Brandon Croft	c33c8b013f	fix: have `terraform output` adhere to authorization w/ cloud ... Normally, `terraform output` refreshes and reads the entire state in the command package before pulling output values out of it. This doesn't give Terraform Cloud the opportunity to apply the read state outputs org permission and instead applies the read state versions permission. I decided to expand the state manager interface to provide a separate GetRootOutputValues function in order to give the cloud backend a more nuanced opportunity to fetch just the outputs. This required moving state Refresh/Read code that was previously in the command into the shared backend state as well as the filesystem state packages.	4 years ago
Brandon Croft	7892cb6b5a	reformat imports using goimports	4 years ago
Brandon Croft	0b9bd2fc36	make cloud backend_state error messages more conventional	4 years ago
Brandon Croft	01d510f3cb	test(cloud): ensure state version is created when saving state	4 years ago
Anna Winkler	d72911a640	Save detailed type outputs to JSONStateOutputs	4 years ago
Luces Huayhuaca	70ddcc7d0d	add url to summary and check for optional values (#30993)	4 years ago
Anna Winkler	22c121df86	Bump compatibility version to 1.3.0 for terraform core release (#30988) ... * Bump compatibility version to 1.3.0 for terraform core release Co-authored-by: Brandon Croft <brandon.croft@gmail.com>	4 years ago
Luces Huayhuaca	ea1f0007ff	fix typo for task.Status (#30978)	4 years ago
Sebastian Rivera	9d7fdbea2d	Handle -input=false in cloud integration ... For non-interactive contexts, Terraform is typically executed with the flag -input=false. However for runs that are not set to auto approve, the cloud integration will prompt a user for approval input even with input being set to false. This commit enables the cloud integration to know the value of the input flag and use it to determine whether or not to ask the user for input. If -input is set to false and the run cannot be auto approved, the cloud integration will throw an error stating run confirmation can no longer be handled in the CLI and that they must do so through the browser.	4 years ago
Sebastian Rivera	5ded48e081	Give token in cloud config higher precedence than CLI config file	4 years ago
Sebastian Rivera	f63ef2b5ef	Rename cloud env vars to use TF_CLOUD prefix	4 years ago
Brandon Croft	42da030090	Merge pull request #30850 from hashicorp/brandonc/cloud_test_revisions ... test(cloud): ensure mocks are used for backend configure tests	4 years ago
Brandon Croft	0dc26a9585	test(cloud): nonexisting org not a valid test when using mocks	4 years ago
Brandon Croft	a38a0ee8a8	test(cloud): ensure mocks are used for backend configure tests ... Also adds a few new tests for cloud configuration using environment variables	4 years ago
Sebastian Rivera	b191faf8a4	Add skip test if missing vars helper	4 years ago
Sebastian Rivera	34114286ff	Cloud e2e tests for configuring with env vars	4 years ago
Sebastian Rivera	8040dfec34	Merge pull request #30787 from hashicorp/sebasslash/tf-workspace-cloud-config ... Add cloud config support for TF_WORKSPACE	4 years ago
Sebastian Rivera	dd864b1bac	Add TF_WORKSPACE validation method	4 years ago
Eng Zer Jun	fedd315275	test: use `T.TempDir` to create temporary test directory (#30803) ... This commit replaces `ioutil.TempDir` with `t.TempDir` in tests. The directory created by `t.TempDir` is automatically removed when the test and all its subtests complete. Prior to this commit, temporary directory created using `ioutil.TempDir` needs to be removed manually by calling `os.RemoveAll`, which is omitted in some tests. The error handling boilerplate e.g. defer func() { if err := os.RemoveAll(dir); err != nil { t.Fatal(err) } } is also tedious, but `t.TempDir` handles this for us nicely. Reference: https://pkg.go.dev/testing#T.TempDir Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>	4 years ago
Sebastian Rivera	995ef15ab9	Add cloud configuration support for TF_WORKSPACE ... TF_WORKSPACE can now be used for your cloud configuration, effectively serving as an alternative to setting the name attribute in your workspaces configuration.	4 years ago
Martin Atkins	49d7c879ac	Fix problems caught by staticcheck v0.3.0 ... This will allow us to upgrade to this version in a later commit without causing the our build checks to fail.	4 years ago
Sebastian Rivera	a52c87cb5a	Fix a flaky test manipulating environment variables ... This test resolves an issue where variables were going unset causing leaks into other test cases and therefore resulting in failures.	4 years ago
Sebastian Rivera	232d48fe00	Add support for TF_HOSTNAME env var	4 years ago
Sebastian Rivera	b1c22d333e	Fix error formatting for missing env and config value ... We previously used to throw an error denoting where in the configuration the attribute was missing or invalid. Considering that organization can be now be omitted from the configuration, our previous error message will be improperly formatted. This commit also updates the message to mention `TF_ORGANIZATION` as a valid substitute if organization is missing or invalid in the configuration.	4 years ago
Sebastian Rivera	45357f5004	Add TF_ORGANIZATION env var support ... TF_ORGANIZATION will serve as a fallback for configuring the organization in the `cloud` block. This is the first step to make it easier for users wanting to configure Terraform programmatically.	4 years ago
Luces Huayhuaca	bdc7d8c0a1	Upgrade go-tfe dependency to use 1.0 version. It contains breaking changes, so we are updating method signatures, method names and the type of optional parameters, as needed. (#30626)	4 years ago
Theo Chupp	d15a2bc024	fix: local variables should not be overridden by remote variables during `terraform import` (#29972) ... * fix: local variables should not be overridden by remote variables during `terraform import` * chore: applied the same fix in the 'internal/cloud' package * backport changes from cloud package to remote package Co-authored-by: Alisdair McDiarmid <alisdair@users.noreply.github.com> Co-authored-by: uturunku1 <luces.huayhuaca@gmail.com>	4 years ago
Sebastian Rivera	52c5f9f6b7	Updated for latest go-tfe run task changes	4 years ago
Sebastian Rivera	126d6df088	Added run task support for post plan run stage, removed pre apply ... This commit stems from the change to make post plan the default run task stage, at the time of this commit's writing! Since pre apply is under internal revision, we have removed the block that polls the pre apply stage until the team decides to re-add support for pre apply run tasks.	4 years ago
uturunku1	383da4893b	use new enum string for task stages	4 years ago
Brandon Croft	aa0dda81b4	Fall back to reading latest run without task_stages ... Older versions of TFE will not allow "task_stages" as an include parameter. In this case, fall back to reading the Run without additional options.	4 years ago