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.
- 01 Apr, 2020 2 commits
-
-
Petros Kolyvas authored
They don't seem to be present on `localhost`
-
Petros Kolyvas authored
-
- 31 Mar, 2020 1 commit
-
-
Petros Kolyvas authored
-
- 27 Mar, 2020 8 commits
-
-
Petros Kolyvas authored
-
Petros Kolyvas authored
-
Petros Kolyvas authored
-
Petros Kolyvas authored
-
Petros Kolyvas authored
-
Alisdair McDiarmid authored
-
Alisdair McDiarmid authored
command/fmt: Include source snippets in errors
-
Alisdair McDiarmid authored
command/format: Fix multi-line diagnostic output
-
- 26 Mar, 2020 6 commits
-
-
Alisdair McDiarmid authored
Previously, if a diagnostic context spanned multiple lines, any lines which did not overlap with the highlight range would be displayed as blank. This commit fixes the bug. The problem was caused by the unconditional use of `PartitionAround` to split the line into before/highlighted/after ranges. When two ranges don't overlap, this method returns empty ranges, which results in a blank line. Instead, we first check if the ranges do overlap, and if not we print the entire line from the context.
-
Alisdair McDiarmid authored
Previously, diagnostic errors would display the filename and line number, along with "(source code not available)". This is because the fmt command directly loads and parses the configuration, instead of using the config loader. This commit registers the manually parsed source as a synthetic configuration file, so that the diagnostic formatter can look up the source for the range with the error and display it.
-
Pam Selle authored
Update ModuleCallOutput and add AbsModuleCallOutput
-
Pam Selle authored
-
James Bardin authored
Module Expansion Activate!
-
James Bardin authored
This name method won't be called in the full graph, and remove it to prevent confusion with the parent node in logs.
-
- 25 Mar, 2020 23 commits
-
-
James Bardin authored
-
James Bardin authored
eval nodes no longer always have a context path
-
James Bardin authored
simplify the test a bit and add a few more combinations to the config
-
James Bardin authored
-
James Bardin authored
The expand logic was separated into nodeExpandRefreshableManagedResource, but the orphan logic wasn't updated.
-
James Bardin authored
-
James Bardin authored
Remove the shims where they aren't necessary from the Init and Close provider nodes. This also removed some provider path checks from the builtin eval context, which cannot be resolved since the context may not be created with a ModuleInstance path.
-
James Bardin authored
We can't get module instances during transformation, so we need to reduce the Dependencies to using `addrs.ConfigResource` for now.
-
James Bardin authored
-
James Bardin authored
-
James Bardin authored
Use the new addrs type here. Also remove the uniqueMap from the config transformer. We enforce uniqueness during config loading, and this is more likely to have false positives due to stringification than anything.
-
James Bardin authored
Resources also need to be expanded during apply, which cannot be done via EvalTree due to the lack of EvalContext.
-
James Bardin authored
While the Expander itself now handles the recursive expansion of modules, Resources themselves still need to be expanded twice, because the evaluation of the Resource, which entails evaluating the for_each or count expressions, is separate from the ResourceInstance expansion. Add a nodeExpandPlannableResource to do handle this expansion to allow all NodePlannableResources to call EvalWriteResourceState with an absolute address.
-
James Bardin authored
As the Graph is walked, the current way to set the context path was to have the walker return a context from EnterPath. This required that every node know it's absolute path, which can no longer be the case during plan when modules have not been expanded. This introduces a new method called WithPath, which returns a copy of the context with the internal path updated to reflect the method argument. Any use of the EvalContext that requires knowing the path will now panic if it wasn't explicitly set to ensure that evaluations always occur in the correct path. Add EvalContext to the GraphWalker interface. EvalContext returns an EvalContext that has not yet set a path. This will allow us to enforce that all context operations requiring a module instance path will require that a path be explicitly set rather than evaluating within the wrong path.
-
Martin Atkins authored
This was incorrectly removing the _source_ entry prior to creating the symlink, therefore ending up with a dangling symlink and no source file. This wasn't obvious before because the test case for LinkFromOtherCache was also incorrectly named and therefore wasn't running. Fixing the name of that test made this problem apparent. The TestLinkFromOtherCache test case now ends up seeing the final resolved directory rather than the symlink target, because of upstream changes to the internal/getproviders filesystem scanning logic to handle symlinks properly.
-
Martin Atkins authored
Previously this was failing to treat symlinks to directories as unpacked layout, because our file info was only an Lstat result, not a full Stat. Now we'll resolve the symlink first, allowing us to handle a symlink to a directory. That's important because our internal/providercache behavior is to symlink from one cache to another where possible.
-
Martin Atkins authored
There's a lot going on in these functions that can be hard to follow from the outside, so we'll add some additional trace logging so that we can more easily understand why things are behaving the way they are.
-
Martin Atkins authored
This was accidentally left out of an earlier commit due to our top-level .gitignore file containing *.exe as an ignore pattern.
-
Pam Selle authored
-
James Bardin authored
Expander.ExpandResource cannot expand all modules
-
Martin Atkins authored
When a provider source produces an HTTP URL location we'll expect it to resolve to a zip file, which we'll first download to a temporary directory and then treat it like a local archive. When a provider source produces a local archive path we'll expect it to be a zip file and extract it into the target directory. This does not yet include an implementation of installing from an already-unpacked local directory. That will follow in a subsequent commit, likely following a similar principle as in Dir.LinkFromOtherCache.
-
Martin Atkins authored
These new functions allow command implementations to get hold of the providercache objects and installation source object derived from the current CLI configuration.
-
Martin Atkins authored
The MultiSource isn't actually properly implemented yet, but this is a minimal implementation just for the case where there are no underlying sources at all, because we use an empty MultiSource as a placeholder when a test in the "command" package fails to explicitly populate a ProviderSource.
-