Skip to content

Commit

Permalink
stackruntime: Pass the apply-time variable values to stackeval
Browse files Browse the repository at this point in the history
This seems to have got lost while resolving some conflicts in
88eec94.

Without this, apply-time input variable values just get ignored entirely
and if any are required the plan is not applyable at all. Now we'll pass
through what the caller provided so stackeval can validate and use it
properly.
  • Loading branch information
apparentlymart committed Jun 19, 2024
1 parent 1714729 commit c57f709
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions internal/stacks/stackruntime/apply.go
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,9 @@ func Apply(ctx context.Context, req *ApplyRequest, resp *ApplyResponse) {
req.Config,
req.RawPlan,
stackeval.ApplyOpts{
ProviderFactories: req.ProviderFactories,
ExperimentsAllowed: req.ExperimentsAllowed,
InputVariableValues: req.InputValues,
ProviderFactories: req.ProviderFactories,
ExperimentsAllowed: req.ExperimentsAllowed,
},
outp,
)
Expand Down

0 comments on commit c57f709

Please sign in to comment.