Skip to content

[LVA] Helm chart update for Live-video-alert-agent app#2333

Open
kumarsjr wants to merge 35 commits intoopen-edge-platform:mainfrom
kumarsjr:live-video-alert-agent
Open

[LVA] Helm chart update for Live-video-alert-agent app#2333
kumarsjr wants to merge 35 commits intoopen-edge-platform:mainfrom
kumarsjr:live-video-alert-agent

Conversation

@kumarsjr
Copy link
Copy Markdown
Contributor

Description

  • Helm chart for Kubernetes deployment with CPU and GPU inference support.
  • Metrics port parameterization.

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes.
  • I have not introduced any 3rd party components incompatible with APACHE-2.0.
  • I have not included any company confidential information, trade secret, password or security token.
  • I have performed a self-review of my code.

- -c
- |
until curl -sf http://{{ printf "%s-ovms" .Release.Name }}:{{ .Values.global.ovmsPort }}/v1/config; do
echo "waiting for OVMS..."; sleep 5;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No escape if OVMS does start?

SPDX-License-Identifier: Apache-2.0
*/}}

{{- define "collector.fullname" -}}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Refer to comments from LVC

Copyright (C) 2025 Intel Corporation
SPDX-License-Identifier: Apache-2.0
*/}}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

refer to comments for LVC helm

@@ -0,0 +1,3 @@
# Copyright (C) 2025 Intel Corporation
# SPDX-License-Identifier: Apache-2.0
{{- include "lva.validate" . -}}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why 000- in filename?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This ensures the validation template runs first before any other resources are created. This file check the required values and fails if something is wrong.

- **Storage Requirement:** The chart creates a **10 Gi** PVC for the VLM model on first run.

See also: [System Requirements](system-requirements.md).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Provide link to this file from top level file.

## Learn More

- [Get Started](docs/user-guide/get-started.md) - Quick deployment guide
- [Deploy with Helm](docs/user-guide/deploy-with-helm.md) - Kubernetes Helm chart deployment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It is not in this file. Recheck in other apps where the helm link is given.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This file can be deleted.

enabled: false
# key: GPU resource label from device plugin (e.g. gpu.intel.com/i915 or gpu.intel.com/xe)
key: ""
device: ""
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

default should be CPU as specified in the override file


### 3. Build Helm Dependencies
```bash
helm dependency build
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is only required if chart is installed from source code. Please add a note.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants