This project is mirrored from https://:*****@github.com/hashicorp/terraform.git.
Pull mirroring failed .
Repository mirroring has been paused due to too many failed attempts. It can be resumed by a project maintainer.
Repository mirroring has been paused due to too many failed attempts. It can be resumed by a project maintainer.
- 20 Oct, 2021 3 commits
-
-
hc-github-team-tf-core authored
-
Martin Atkins authored
-
Martin Atkins authored
We introduced this experiment to gather feedback, and the feedback we saw led to us deciding to do another round of design work before we move forward with something to meet this use-case. In addition to being experimental, this has only been included in alpha releases so far, and so on both counts it is not protected by the Terraform v1.0 Compatibility Promises.
-
- 19 Oct, 2021 4 commits
-
-
Alisdair McDiarmid authored
cli: Restore -lock and -lock-timeout init flags
-
Alisdair McDiarmid authored
cli: Fix flaky init cancel test
-
Alisdair McDiarmid authored
The -lock and -lock-timeout flags were removed prior to the release of 1.0 as they were thought to have no effect. This is not true in the case of state migrations when changing backends. This commit restores these flags, and adds test coverage for locking during backend state migration. Also update the help output describing other boolean flags, showing the argument as the user would type it rather than the default behavior.
-
Alisdair McDiarmid authored
There is a race between the MockSource and ShutdownCh which sometimes causes this test to fail. Add a HangingSource implementation of Source which hangs until the context is cancelled, so that there is always time for a user-initiated shutdown to trigger the cancellation code path under test.
-
- 16 Oct, 2021 1 commit
-
-
Martin Atkins authored
-
- 15 Oct, 2021 1 commit
-
-
Chris Griggs authored
* add new guide doc * update word doc * assign imange files * add fmting changes * formatting * some more changes * Fix Title * Update website/guides/terraform-integration-program.html.md We can remove, I was hoping to have "*Currently, pre-apply..." be in Bbold, but it looks like it doesnt render that way. So we can exclude the asterisk Co-authored-by:
Jeff Escalante <jescalan@users.noreply.github.com> * Fix spacing and remove unused html paragraph * Update website/guides/terraform-integration-program.html.md Good changes, thanks for simplifying it. Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Agreed Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Agreed Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Agreed Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com> * Adding suggested changes from PR. Removing example sentence. (Internal request) * Move note section above badge * Add spacing * Update website/guides/terraform-integration-program.html.md Approved Co-authored-by:
Jeff Escalante <jescalan@users.noreply.github.com> Co-authored-by:
Jeff Escalante <jescalan@users.noreply.github.com> Co-authored-by:
Laura Pacilio <83350965+laurapacilio@users.noreply.github.com>
-
- 14 Oct, 2021 5 commits
-
-
James Bardin authored
-
Martin Atkins authored
We don't use this library anywhere else in Terraform, and this backend was using it only for trivial helpers that are easy to express inline anyway. The new direct code is also type-checkable, whereas these helper functions seem to be written using reflection. This gives us one fewer dependency to worry about and makes the test code for this backend follow a similar assertions style as the rest of this codebase.
-
James Bardin authored
Check for stale plan with no state metadata
-
dependabot[bot] authored
Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.0 to 1.5.2. - [Release notes](https://github.com/golang/protobuf/releases) - [Commits](https://github.com/golang/protobuf/compare/v1.5.0...v1.5.2 ) --- updated-dependencies: - dependency-name: github.com/golang/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by:
dependabot[bot] <support@github.com>
-
Martin Atkins authored
This also transitively upgrades golang.org/x/sys and golang.org/x/net, but there do not seem to be any significant changes compared to the commits we were previously using.
-
- 13 Oct, 2021 9 commits
-
-
James Bardin authored
In order to test applying a plan from an existing state, we need to be able to inject the state meta into the planfile.
-
James Bardin authored
Ensure that we still check for a stale plan even when it was created with no previous state. Create separate errors for incorrect lineage vs incorrect serial. To prevent confusion when applying a first plan multiple times, only report it as a stale plan rather than different lineage.
-
Martin Atkins authored
-
Martin Atkins authored
Previously we would reject attempts to delete a workspace if its state contained any resources at all, even if none of the resources had any resource instance objects associated with it. Nowadays there isn't any situation where the normal Terraform workflow will leave behind resource husks, and so this isn't as problematic as it might've been in the v0.12 era, but nonetheless what we actually care about for this check is whether there might be any remote objects that this state is tracking, and for that it's more precise to look for non-nil resource instance objects, rather than whole resources. This also includes some adjustments to our error messaging to give more information about the problem and to use terminology more consistent with how we currently talk about this situation in our documentation and elsewhere in the UI. We were also using the old State.HasResources method as part of some of our tests. I considered preserving it to avoid changing the behavior of those tests, but the new check seemed close enough to the intent of those tests that it wasn't worth maintaining this method that wouldn't be used in any main code anymore. I've therefore updated those tests to use the new HasResourceInstanceObjects method instead.
-
megan07 authored
update error message for invalid json
-
Megan Bang authored
-
Martin Atkins authored
-
dependabot[bot] authored
Bumps [github.com/xanzy/ssh-agent](https://github.com/xanzy/ssh-agent) from 0.2.1 to 0.3.1. - [Release notes](https://github.com/xanzy/ssh-agent/releases) - [Commits](https://github.com/xanzy/ssh-agent/compare/v0.2.1...v0.3.1 ) --- updated-dependencies: - dependency-name: github.com/xanzy/ssh-agent dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by:
dependabot[bot] <support@github.com>
-
James Bardin authored
fix test fixture with multiple provider instances
-
- 12 Oct, 2021 14 commits
-
-
James Bardin authored
When a test uses multiple instances of the same provider, we may need to have separate objects to prevent overwriting of the MockProvider state. Create a completely new MockProvider in each factory function call rather than re-using the original provider value.
-
Martin Atkins authored
Running the tool this way ensures that we'll always run the version selected by our go.mod file, rather than whatever happened to be available in $GOPATH/bin on the system where we're running this. This change caused some contexts to now be using a newer version of staticcheck with additional checks, and so this commit also includes some changes to quiet the new warnings without any change in overall behavior.
-
Martin Atkins authored
A snapshotDir tracks its current position as part of its state, so we need to use it via pointer rather than value so that Readdirnames can actually update that position, or else we'll just get stuck at position zero. In practice this wasn't hurting anything because we only call Readdir once on our snapshots, to read the whole directory at once. Still nice to fix to avoid a gotcha for future maintenence, though.
-
James Bardin authored
Allow these to share the same backing MockProvider.
-
James Bardin authored
fix test fixture had the instance in the wrong mod
-
James Bardin authored
Make the state match the fixture config. The old test was not technically invalid, but because it caused multiple instances of the provider to be created, they were backed by the same MockProvider value resulting in the `*Called` fields interfering.
-
dependabot[bot] authored
Bumps [github.com/agext/levenshtein](https://github.com/agext/levenshtein) from 1.2.2 to 1.2.3. - [Release notes](https://github.com/agext/levenshtein/releases) - [Commits](https://github.com/agext/levenshtein/compare/v1.2.2...v1.2.3 ) --- updated-dependencies: - dependency-name: github.com/agext/levenshtein dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by:
dependabot[bot] <support@github.com>
-
Martin Atkins authored
CVE-2020-26160 is a high-severity advisory reported against this module. The dgrijalva package is no longer maintained but our legacy etcv2 backend depends on it indirectly, via go.etcd.io/etcd/client. The golang-jwt package is the blessed successor of the original, and has a v3 line which is compatible with the v3 line of dgrijalva, and so through this replace we can get a fix for the advisory without other significant behavior change. We've preserved the etcdv2 backend as-is on a best-effort basis in order to support anyone who is already using it, but recommend that users switch to etcdv3 or to some other backend for ongoing use. We also have future plans to make state storage be a matter for provider plugins rather than built in to Terraform CLI, at which point this backend will either become obsolete or be factored out into its own plugin, at which point we can remove this "replace" directive and the associated dependency altogether.
-
dependabot[bot] authored
Bumps [honnef.co/go/tools](https://github.com/dominikh/go-tools) from 0.0.1-2020.1.4 to 0.3.0-0.dev. - [Release notes](https://github.com/dominikh/go-tools/releases) - [Commits](https://github.com/dominikh/go-tools/compare/v0.0.1-2020.1.4...v0.3.0-0.dev ) --- updated-dependencies: - dependency-name: honnef.co/go/tools dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by:
dependabot[bot] <support@github.com>
-
Martin Atkins authored
This is just a routine upgrade to the latest v1 release.
-
dependabot[bot] authored
Bumps [github.com/mitchellh/copystructure](https://github.com/mitchellh/copystructure) from 1.0.0 to 1.2.0. - [Release notes](https://github.com/mitchellh/copystructure/releases) - [Commits](https://github.com/mitchellh/copystructure/compare/v1.0.0...v1.2.0 ) --- updated-dependencies: - dependency-name: github.com/mitchellh/copystructure dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by:
dependabot[bot] <support@github.com>
-
dependabot[bot] authored
Bumps [github.com/hashicorp/go-plugin](https://github.com/hashicorp/go-plugin) from 1.4.1 to 1.4.3. - [Release notes](https://github.com/hashicorp/go-plugin/releases) - [Commits](https://github.com/hashicorp/go-plugin/compare/v1.4.1...v1.4.3 ) --- updated-dependencies: - dependency-name: github.com/hashicorp/go-plugin dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by:
dependabot[bot] <support@github.com>
-
James Bardin authored
Check for nil change during apply
-
James Bardin authored
Bump google.golang.org/protobuf from 1.25.0 to 1.27.1
-
- 11 Oct, 2021 1 commit
-
-
Martin Atkins authored
-
- 08 Oct, 2021 2 commits
-
-
James Bardin authored
Because NodeAbstractResourceInstance.readDiff can return a nil change, we must check for that in all callers.
-
James Bardin authored
don't refresh deposed instances during the destroy plan
-