Created
September 25, 2018 01:58
-
-
Save jim-minter/d1a266c870797a2b94a826b3218fdd7a to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
diff --git a/cmd/createorupdate/createorupdate.go b/cmd/createorupdate/createorupdate.go | |
index 6bd9fde6..b8ad6e50 100644 | |
--- a/cmd/createorupdate/createorupdate.go | |
+++ b/cmd/createorupdate/createorupdate.go | |
@@ -107,7 +107,7 @@ func createOrUpdate(ctx context.Context, oc *v20180930preview.OpenShiftManagedCl | |
} | |
if oldCs != nil { | |
- err = p.Update(ctx, cs, azuredeploy, config) | |
+ err = p.Update(ctx, cs, azuredeploy) | |
if err != nil { | |
return nil, err | |
} | |
diff --git a/pkg/api/plugin.go b/pkg/api/plugin.go | |
index 308554da..e2767d79 100644 | |
--- a/pkg/api/plugin.go | |
+++ b/pkg/api/plugin.go | |
@@ -46,5 +46,5 @@ type Plugin interface { | |
HealthCheck(ctx context.Context, cs *OpenShiftManagedCluster) error | |
- Update(ctx context.Context, cs *OpenShiftManagedCluster, azuredeploy []byte, config PluginConfig) error | |
+ Update(ctx context.Context, cs *OpenShiftManagedCluster, azuredeploy []byte) error | |
} | |
diff --git a/pkg/plugin/plugin.go b/pkg/plugin/plugin.go | |
index 78765ae3..2a95b15b 100644 | |
--- a/pkg/plugin/plugin.go | |
+++ b/pkg/plugin/plugin.go | |
@@ -116,8 +116,8 @@ func (p *plugin) HealthCheck(ctx context.Context, cs *api.OpenShiftManagedCluste | |
return healthChecker.HealthCheck(ctx, cs) | |
} | |
-func (p *plugin) Update(ctx context.Context, cs *api.OpenShiftManagedCluster, azuredeploy []byte, config api.PluginConfig) error { | |
+func (p *plugin) Update(ctx context.Context, cs *api.OpenShiftManagedCluster, azuredeploy []byte) error { | |
log.Info("starting update") | |
upgrader := upgrade.NewSimpleUpgrader(p.entry, p.config) | |
- return upgrader.Update(ctx, cs, azuredeploy, config) | |
+ return upgrader.Update(ctx, cs, azuredeploy, p.config) | |
} |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment