Skip to content

Marimo: new Charts#312

Open
PACordonnier wants to merge 2 commits into
InseeFrLab:mainfrom
adaltas:inseepr
Open

Marimo: new Charts#312
PACordonnier wants to merge 2 commits into
InseeFrLab:mainfrom
adaltas:inseepr

Conversation

@PACordonnier
Copy link
Copy Markdown

Description of the change

The PR aims to add Marimo and all differents flavors to the interactive services catalog.

The PR is pretty huge as I added all the differents flavors for marimo, copying what was done for VSCode:

  • marimo-python
  • marimo-python-gpu
  • marimo-pyspark
  • marimo-tensorflow
  • marimo-tensorflow-gpu
  • marimo-pytorch
  • marimo-pytorch-gpu

For each flavour, I copied the equivalent vscode-* chart. Here's what I did after copying to 'adapt' vscode to marimo

  • in Chart.yaml
    • change name, description and other metadatas, set version to 0.0.1
  • README.md
    • delete its content, to be created by CI
  • values.schema.yaml
    • set the image name from inseefrlab/vscode-(xxx) to pacordonnier/marimo-(xxx)
  • values.yaml
    • set image version
    • set networking.service.port to 2718 (default marimo port)
  • templates folder:

Marimo does not support reading token/password from ENV, so needs to mount the secret-token as volumes

In statefulset.yaml, args are now:

{{- if .Values.git.asCodeServerRoot }}
args: ["{{ .Values.init.standardInitPath }} marimo edit --host 0.0.0.0 --token-password-file ~/.token/PASSWORD /home/{{ .Values.environment.user }}/work/$(basename $GIT_REPOSITORY) .git"]
{{- else }}
args: ["{{ .Values.init.standardInitPath }} marimo edit --host 0.0.0.0 --token-password-file ~/.token/PASSWORD /home/{{ .Values.environment.user }}/work"]
{{- end }}

volume for secret token needs to be declared in statefulset.yaml

spec.template.spec.volumes:
        - name: token
          secret:
            secretName: {{ include "library-chart.secretNameToken" . }}

volumeMounts:

spec.template.spec.containers[0].volumesMounts
- name: token  
  mountPath: /home/{{ .Values.environment.user }}/.token
   readOnly: true

Feel free to add my forked to catalog to your onyxia instances in order to play with Marimo. All the images for marimo has been built and pushed to dockerhub in my own name.

Here's the catalog to add to your onyxia instance:

catalogs:
      - id: ide-adaltas
        name:
          en: Development environments - Adaltas
          fr: Environnements de développement - Adaltas
          de: Entwicklungsumgebungen - Adaltas
          fi: Kehitysympäristöt - Adaltas
          it: Ambienti di sviluppo - Adaltas
          nl: Ontwikkelingsomgevingen - Adaltas
          "no": Utviklingsmiljøer - Adaltas
          zh-CN: 开发环境 - Adaltas
        maintainer: stdin@adaltas.com
        location: https://adaltas.github.io/helm-charts-interactive-services
        status: PROD
        highlightedCharts:
          - marimo-python
          - marimo-python-gpu
          - marimo-tensorflow
          - marimo-tensorflow-gpu
          - marimo-pyspark
          - marimo-pytorch
          - marimo-pytorch-gpu
        type: helm
        skipTlsVerify: false
        caFile:
        allowSharing: false
        visible:
          user: true
          project: true
        username:
        password:
        multipleServicesMode: latest

Checklist

  • Chart version bumped in Chart.yaml
  • Title of the pull request follows this pattern [name_of_the_chart] Descriptive title

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant