Skip to content

Commit

Permalink
fix(pulumi): fix process dependency resolution in plugin command (#5325)
Browse files Browse the repository at this point in the history
Method `resolveDependencies` was unused before this fix.
But, it was implemented to handle the process dependencies.
This commit renames it to override the corresponding method of the parent class,
as it was initially intended.
  • Loading branch information
vvagaytsev committed Oct 31, 2023
1 parent b9c4097 commit d460f3f
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion plugins/pulumi/src/commands.ts
Expand Up @@ -249,7 +249,10 @@ class PulumiPluginCommandTask extends PluginActionTask<PulumiDeploy, PulumiComma
return this.action.longDescription()
}

resolveDependencies() {
/**
* Override the base method to be sure that `garden plugins pulumi preview` happens in dependency order.
*/
override resolveProcessDependencies() {
const pulumiDeployNames = this.graph
.getDeploys()
.filter((d) => d.type === "pulumi")
Expand Down

0 comments on commit d460f3f

Please sign in to comment.