Skip to content

Commit

Permalink
Upgrade conda-store v0.3.10 and simplify specification of image (#1130)
Browse files Browse the repository at this point in the history
* remove conda store image from input_vars

* remove terraform variables

* uprade image to v0.3.10

* reinstate terraform variables with an image default
  • Loading branch information
HarshCasper authored Mar 1, 2022
1 parent e6302f5 commit 06e9402
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 9 deletions.
3 changes: 0 additions & 3 deletions qhub/stages/input_vars.py
Original file line number Diff line number Diff line change
Expand Up @@ -214,9 +214,6 @@ def stage_07_kubernetes_services(stage_outputs, config):
# conda-store
"conda-store-environments": config["environments"],
"conda-store-storage": config["storage"]["conda_store"],
"conda-store-image": _split_docker_image_name(
"quansight/conda-store-server:v0.3.9"
),
# jupyterhub
"cdsdashboards": config["cdsdashboards"],
"jupyterhub-theme": config["theme"]["jupyterhub"],
Expand Down
5 changes: 4 additions & 1 deletion qhub/template/stages/07-kubernetes-services/conda-store.tf
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,12 @@ variable "conda-store-image" {
name = string
tag = string
})
default = {
name = "quansight/conda-store-server"
tag = "v0.3.10"
}
}


# ====================== RESOURCES =======================
module "kubernetes-conda-store-server" {
source = "./modules/kubernetes/services/conda-store"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,6 @@ variable "conda-store-image" {
name = string
tag = string
})
default = {
name = "quansight/conda-store-server"
tag = "v0.3.10"
}
}

variable "external-url" {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@ variable "dashboards" {
]
}


variable "node-group" {
description = "Node key value pair for bound resources"
type = object({
Expand Down

0 comments on commit 06e9402

Please sign in to comment.