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

ollama & owui installation update #404

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tbazant
Copy link
Collaborator

@tbazant tbazant commented Dec 17, 2024

No description provided.

@tbazant tbazant self-assigned this Dec 17, 2024
Copy link
Contributor

@dariavladykina dariavladykina left a comment

Choose a reason for hiding this comment

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

LGTM, only minor suggestions. Thanks!

Comment on lines +11 to +14
To override the default values during the &helm; chart installation or update,
you can create an override YAML file with custom values. Then you apply the
values by specifying the path to the override file with the
<option>-f</option> of the <command>helm</command> command.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
To override the default values during the &helm; chart installation or update,
you can create an override YAML file with custom values. Then you apply the
values by specifying the path to the override file with the
<option>-f</option> of the <command>helm</command> command.
To override the default values during the &helm; chart installation or update,
you can create an override YAML file with custom values. Then, apply these
values by specifying the path to the override file with the
<option>-f</option> option of the <command>helm</command> command.

</abstract>
</info>
<table>
<title><filename>values.yaml</filename> options for the &owui; &helm; chart</title>
<title>List of available options for the &owui; &helm; chart</title>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<title>List of available options for the &owui; &helm; chart</title>
<title>Available options for the &owui; &helm; chart</title>

in
<link
<section xml:id="ollama-installing-app-details">
<title>Details about the &ollama;; application</title>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<title>Details about the &ollama;; application</title>
<title>Details about the &ollama; application</title>

Comment on lines +45 to +47
<link xlink:href="https://apps.rancher.io/applications/ollama"/>. It
contains available versions and the link to pull the &ollama; container
image.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<link xlink:href="https://apps.rancher.io/applications/ollama"/>. It
contains available versions and the link to pull the &ollama; container
image.
<link xlink:href="https://apps.rancher.io/applications/ollama"/>. It
contains the available versions and a link to pull the &ollama;
container image.

</listitem>
<listitem>
<para>
The workloads are set to run on &nvidia; enabled nodes as
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The workloads are set to run on &nvidia; enabled nodes as
The workloads are set to run on &nvidia;-enabled nodes as

</itemizedlist>
<para>
If you do not want to use the &nvidia; GPU, remove the
<literal>gpu</literal> section from the
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<literal>gpu</literal> section from the
<literal>gpu</literal> section from

</step>
<step>
<para>
Install the &ollama;; &helm; chart using the
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Install the &ollama;; &helm; chart using the
Install the &ollama; &helm; chart using the

<link
</table> -->
<section xml:id="owui-installing-app-details">
<title>Details about the &owui;; application</title>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<title>Details about the &owui;; application</title>
<title>Details about the &owui; application</title>

all installation options with examples, refer to
Create the <filename>owui_custom_overrides.yaml</filename> file to
override the values of the parent &helm; chart. The file contains URLs
for &milvus; and &ollama;, and specifies whether a stand-alone
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
for &milvus; and &ollama;, and specifies whether a stand-alone
for &milvus; and &ollama; and specifies whether a stand-alone

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.

2 participants