Skip to content

Commit

Permalink
Merge pull request #2852 from lsst-sqre/tickets/DM-42477
Browse files Browse the repository at this point in the history
DM-42477: Switch back to fileservers for user file servers
  • Loading branch information
rra authored Jan 12, 2024
2 parents 2d46989 + 93674cd commit ce86cfc
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
2 changes: 1 addition & 1 deletion applications/nublado/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ JupyterHub and custom spawner for the Rubin Science Platform
| controller.config.fileserver.image.pullPolicy | string | `"IfNotPresent"` | Pull policy for file server image |
| controller.config.fileserver.image.repository | string | `"ghcr.io/lsst-sqre/worblehat"` | File server image to use |
| controller.config.fileserver.image.tag | string | `"0.1.0"` | Tag of file server image to use |
| controller.config.fileserver.namespace | string | `"nublado-fileservers"` | Namespace for user file servers |
| controller.config.fileserver.namespace | string | `"fileservers"` | Namespace for user file servers |
| controller.config.fileserver.nodeSelector | object | `{}` | Node selector rules for user file server pods |
| controller.config.fileserver.pathPrefix | string | `"/files"` | Path prefix for user file servers |
| controller.config.fileserver.resources | object | See `values.yaml` | Resource requests and limits for user file servers |
Expand Down
2 changes: 1 addition & 1 deletion applications/nublado/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ controller:
tag: "0.1.0"

# -- Namespace for user file servers
namespace: "nublado-fileservers"
namespace: "fileservers"

# -- Node selector rules for user file server pods
nodeSelector: {}
Expand Down
9 changes: 7 additions & 2 deletions environments/templates/nublado-fileservers-application.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@
{{/*
The namespace is fileservers even though the Argo CD application is
nublado-fileservers, since otherwise we have a conflict with the
lab namespace for a user with the username fileservers.
*/}}
{{- if .Values.applications.nublado -}}
apiVersion: v1
kind: Namespace
metadata:
name: "nublado-fileservers"
name: "fileservers"
---
apiVersion: argoproj.io/v1alpha1
kind: Application
Expand All @@ -13,7 +18,7 @@ metadata:
- "resources-finalizer.argocd.argoproj.io"
spec:
destination:
namespace: "nublado-fileservers"
namespace: "fileservers"
server: "https://kubernetes.default.svc"
project: "default"
source:
Expand Down

0 comments on commit ce86cfc

Please sign in to comment.