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

docs: deploy langfuse for development #347

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

Conversation

friedrichwilken
Copy link
Collaborator

Description

Changes proposed in this pull request:

  • add docs how to deploy langfuse locally and on a k8s cluster for dev work.

Related issue(s)
#49

@friedrichwilken friedrichwilken requested a review from a team as a code owner January 22, 2025 08:28
Copy link

Note(s) for PR Auther:

  • The integration test will be skipped for the PR. You can trigger it manually after adding the label: run-integration-test.
  • The evaluation test will be skipped for the PR. You can trigger it manually after adding the label: evaluation requested.
  • If any changes are made to the evaluation tests data, make sure that the integration tests are working as expected.
  • If any changes are made to how to run the unit tests, make sure to update the steps for unit-tests in the create-release.yml workflow as well.

Note(s) for PR Reviewer(s):

  • Make sure that the integration and evaluation tests are working as expected.

@friedrichwilken friedrichwilken linked an issue Jan 22, 2025 that may be closed by this pull request
5 tasks
@friedrichwilken friedrichwilken changed the title Doc langfuse deployment docs: deploy langfuse for development Jan 22, 2025
docs/langfuse.md Outdated Show resolved Hide resolved
docs/langfuse.md Outdated Show resolved Hide resolved
docs/langfuse.md Outdated Show resolved Hide resolved
docs/langfuse.md Outdated Show resolved Hide resolved
Comment on lines +9 to +12
```json
"REDIS_PASSWORD": "myredissecret",
"REDIS_DB_NUMBER": "1",
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Please indent the code block.

open http://localhost:3000/auth/sign-up
```

Create an account, then hit the `+ New Organization` button. Give your organization a name, hit `Next` in the `Invite Members` page, find a name for you project an hit `Create`.
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
Create an account, then hit the `+ New Organization` button. Give your organization a name, hit `Next` in the `Invite Members` page, find a name for you project an hit `Create`.
4. Create an account, and choose **+ New Organization**. Give your organization a name, and choose **Next**.
5. On the **Invite Members** page, give your project a name, and choose **Create**.

```

Create an account, then hit the `+ New Organization` button. Give your organization a name, hit `Next` in the `Invite Members` page, find a name for you project an hit `Create`.
On the next page, hit the `API Keys` menu point, then hit the `+ Create new API keys` button. Copy the `Secret Key`, `Secret Key` and `Host` and paste them as values in to your `kyma-companion`'s `config/config.json` for your `LANGFUSE_HOST`, `LANGFUSE_SECRET_KEY`, `LANGFUSE_PUBLIC_KEY` keys:
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
On the next page, hit the `API Keys` menu point, then hit the `+ Create new API keys` button. Copy the `Secret Key`, `Secret Key` and `Host` and paste them as values in to your `kyma-companion`'s `config/config.json` for your `LANGFUSE_HOST`, `LANGFUSE_SECRET_KEY`, `LANGFUSE_PUBLIC_KEY` keys:
6. Go to **API Keys**, and choose **+ Create new API keys**. Copy the `Secret Key`, `Public Key`, and `Host`, and paste them as values into your Kyma Companion's `config/config.json` for , **LANGFUSE_SECRET_KEY**, **LANGFUSE_PUBLIC_KEY**, and **LANGFUSE_HOST**:

Comment on lines +50 to +55
```json
...
"LANGFUSE_HOST": "https://localhost:3000",
"LANGFUSE_SECRET_KEY": "<your secret key>",
"LANGFUSE_PUBLIC_KEY": "<your public key>",
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Please indent the code block.

"LANGFUSE_PUBLIC_KEY": "<your public key>",
```

## Deploy on Kubernetes
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
## Deploy on Kubernetes
## Deploy LangFuse on Kubernetes

For the whole section, please create a step-by-step instruction with an ordered list of steps and indented code blocks.


## Deploy on Kubernetes

### For development purpose
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 development purpose

Is there any other purpose?

friedrichwilken and others added 4 commits February 7, 2025 11:17
Co-authored-by: Małgorzata Świeca <malgorzata.swieca@sap.com>
Co-authored-by: Małgorzata Świeca <malgorzata.swieca@sap.com>
Co-authored-by: Małgorzata Świeca <malgorzata.swieca@sap.com>
Co-authored-by: Małgorzata Świeca <malgorzata.swieca@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

blocked: Setup the tracing infrastructure for agents
3 participants