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

Bump localai version #113

Merged
merged 2 commits into from
Nov 21, 2024
Merged

Bump localai version #113

merged 2 commits into from
Nov 21, 2024

Conversation

mauromorales
Copy link
Member

No description provided.

@mauromorales mauromorales requested a review from a team November 20, 2024 06:55
LocalAI/run.sh Outdated
@@ -4,7 +4,7 @@ set -ex

K3S_MANIFEST_DIR=${K3S_MANIFEST_DIR:-/var/lib/rancher/k3s/server/manifests/}

VERSION="1.0.2"
VERSION="2.23.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't this be the chart version? (3.4.0 in this case)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jimmykarily ohhh, new topic for me hahaha, so the one that needs to be bumped to 2.23.0 is this one?

https://github.com/go-skynet/helm-charts/blob/274accb100c057b057a9b30c6ced1a8235dd291d/charts/local-ai/Chart.yaml#L2

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes although I don't see it being used anywhere other than a label: https://github.com/search?q=repo%3Ago-skynet%2Fhelm-charts%20appVersion&type=code

the actual version used is controlled by a label: https://github.com/go-skynet/helm-charts/blob/274accb100c057b057a9b30c6ced1a8235dd291d/charts/local-ai/values.yaml#L6

which of course could result in inconsistent labels and actual running versions :D

@mauromorales mauromorales merged commit 63d9226 into main Nov 21, 2024
15 of 16 checks passed
@mauromorales mauromorales deleted the mauromorales-patch-1 branch November 21, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants