Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Question] Update syncing to otwld/ollama-helm? #20

Open
malcolmlewis opened this issue May 16, 2024 · 4 comments
Open

[Question] Update syncing to otwld/ollama-helm? #20

malcolmlewis opened this issue May 16, 2024 · 4 comments

Comments

@malcolmlewis
Copy link

Hi, there are a number of Nvidia GPU updates in the latest otwld/ollama-helm chart
App version: 0.1.38
Chart version: 0.29.1

If I do a fresh deployment on the open-webui chart, should these get pulled, or does the chart need updating?

@0xThresh
Copy link
Contributor

Hey @robrakaric, could use your help understanding the behavior for the embedded Ollama chart.

I see that the Chart.yaml file pulls versions of the Ollama chart that are greater than version 0.24.0: https://github.com/open-webui/helm-charts/blob/main/charts/open-webui/Chart.yaml#L27

I also see that we have version v0.25.0 embedded in charts: https://github.com/open-webui/helm-charts/tree/main/charts/open-webui/charts

Do you know offhand whether the OWUI Helm chart pull the latest chart on a new build, or if it pulls the embedded 0.25.0 chart we have in the repo?

@malcolmlewis
Copy link
Author

@0xThresh Hi, I see;

Labels:              app.kubernetes.io/component=ollama                                                                                                                                    │
│                      app.kubernetes.io/instance=open-webui                                                                                                                                 │
│                      app.kubernetes.io/managed-by=Helm                                                                                                                                     │
│                      app.kubernetes.io/version=0.1.34                                                                                                                                      │
│                      helm.sh/chart=ollama-0.25.0                                                                                                                                           │
│                      pod-template-hash=56c55f7c6

@robrakaric
Copy link
Contributor

@0xThresh good question. I'd have to do some testing.

What I'm not sure of is chart-releaser's behavior. The assets in the gh release will be most relevant, and I'm not sure if chart-releaser actually downloads the newest version (helm dependency update), uses the one in the repo, or just respects what's in Chart.lock (helm dependency build).

I'll mess around with in on my fork and see what I can find out.

@0xThresh
Copy link
Contributor

We have a PR open that should get merged whenever Tim takes a look at it, so we can also observe the behavior there once we pull that in. Thanks for taking a look!

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

No branches or pull requests

3 participants