Skip to content

Commit

Permalink
[cherry-pick]: split workbenches image into 2 params.env file
Browse files Browse the repository at this point in the history
Signed-off-by: Wen Zhou <[email protected]>
  • Loading branch information
zdtsw committed Oct 25, 2023
1 parent 70e398f commit b0266e0
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions components/workbenches/workbenches.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ var (
DependentComponentName = "notebooks"
// manifests for nbc in ODH and downstream + downstream use it for imageparams
notebookControllerPath = deploy.DefaultManifestPath + "/odh-notebook-controller/odh-notebook-controller/base"
// manifests for ODH nbc
// manifests for ODH nbc + downstream use it for imageparams
kfnotebookControllerPath = deploy.DefaultManifestPath + "/odh-notebook-controller/kf-notebook-controller/overlays/openshift"
notebookImagesPath = deploy.DefaultManifestPath + "/notebooks/overlays/additional"
notebookImagesPathSupported = deploy.DefaultManifestPath + "/jupyterhub/notebooks/base"
Expand Down Expand Up @@ -135,10 +135,14 @@ func (w *Workbenches) ReconcileComponent(cli client.Client, owner metav1.Object,
if enabled {
if dscispec.DevFlags.ManifestsUri == "" && len(w.DevFlags.Manifests) == 0 {
if platform == deploy.ManagedRhods || platform == deploy.SelfManagedRhods {
// TODO: fix for nbc new path if want to split into two params.env
// for kf-notebook-controller image
if err := deploy.ApplyParams(notebookControllerPath, w.SetImageParamsMap(imageParamMap), false); err != nil {
return err
}
// for odh-notebook-controller image
if err := deploy.ApplyParams(kfnotebookControllerPath, w.SetImageParamsMap(imageParamMap), false); err != nil {
return err
}
}
}
}
Expand Down

0 comments on commit b0266e0

Please sign in to comment.