Skip to content

feat: rebrand lightspeed to intelligent-assistant - #515

Merged
openshift-merge-bot[bot] merged 2 commits into
redhat-developer:mainfrom
Jdubrick:rename-lightspeed-to-intelligent-assistant
Sep 1, 2026
Merged

feat: rebrand lightspeed to intelligent-assistant#515
openshift-merge-bot[bot] merged 2 commits into
redhat-developer:mainfrom
Jdubrick:rename-lightspeed-to-intelligent-assistant

Conversation

@Jdubrick

@Jdubrick Jdubrick commented Sep 1, 2026

Copy link
Copy Markdown

Description of the change

  • Renames lightspeed to intelligent assistant
  • Updates to use ref:// for plugins and updates to intelligent-assistant plugin
  • Bumps Chart version --> @rm3l I did a major bump since this would be a breaking change. Is this appropriate, or do you have other preferences?

Which issue(s) does this PR fix or relate to

https://redhat.atlassian.net/browse/RHIDP-15900

How to test changes / Special notes to the reviewer

Checklist

  • For each Chart updated, version bumped in the corresponding Chart.yaml according to Semantic Versioning.
  • For each Chart updated, variables are documented in the values.yaml and added to the corresponding README.md. The pre-commit utility can be used to generate the necessary content. Run pre-commit run --all-files to run the hooks and then push any resulting changes. The pre-commit Workflow will enforce this and warn you if needed.
  • JSON Schema template updated and re-generated the raw schema via the pre-commit hook.
  • Tests pass using the Chart Testing tool and the ct lint command.
  • If you updated the orchestrator-infra chart, make sure the versions of the Knative CRDs are aligned with the versions of the CRDs installed by the OpenShift Serverless operators declared in the values.yaml file. See Installing Knative Eventing and Knative Serving CRDs for more details.

Signed-off-by: Jordan Dubrick <jdubrick@redhat.com>
Signed-off-by: Jordan Dubrick <jdubrick@redhat.com>
@Jdubrick
Jdubrick requested review from a team as code owners September 1, 2026 15:27
@sonarqubecloud

sonarqubecloud Bot commented Sep 1, 2026

Copy link
Copy Markdown

@rhdh-qodo-merge

Copy link
Copy Markdown

PR Summary by Qodo

Rebrand RHDH Lightspeed as Intelligent Assistant

✨ Enhancement ⚙️ Configuration changes 📝 Documentation 🧪 Tests 🕐 40+ Minutes

Grey Divider

AI Description

• Rebrands built-in Lightspeed chart configuration as Intelligent Assistant.
• Resolves renamed frontend and backend plugins through catalog ref:// references.
• Migrates templates, vendored configs, CI automation, and documentation; bumps chart to 2.0.0.
Diagram

graph TD
  V["Chart values"] --> H["IA helpers"] --> D["Pod deployment"]
  H --> P["Plugin manifest"] --> D
  H --> C["IA ConfigMaps"] --> D
  W["Sync automation"] --> F["Vendored configs"] --> C
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Provide a deprecated Lightspeed compatibility alias
  • ➕ Allows existing 1.x values files to continue working during migration.
  • ➕ Reduces the risk of previously disabled installations enabling the assistant after upgrade.
  • ➖ Creates precedence and merge ambiguity when both configuration keys are supplied.
  • ➖ Extends obsolete terminology throughout templates, schemas, and support documentation.
  • ➖ Requires additional validation and compatibility tests.
2. Preserve existing ConfigMap names
  • ➕ Avoids replacing configuration objects during Helm upgrades.
  • ➕ Reduces deployment churn for installations using chart-generated ConfigMaps.
  • ➖ Retains obsolete resource names after the public feature rename.
  • ➖ Makes chart state inconsistent with the new configuration API.
  • ➖ Does not address potential suffix collisions for long release names.

Recommendation: The clean breaking rename is appropriate because the chart is moving to 2.0.0, plugin identities also changed, and the documented migration is straightforward. The shortened -ia- ConfigMap naming and explicit external Secret contract are preferable to retaining legacy internals; a compatibility alias should only be added if seamless 1.x values reuse is a release requirement.

Files changed (25) +212 / -210

Enhancement (5) +86 / -77
_helpers.tplResolve and validate Intelligent Assistant settings +27/-18

Resolve and validate Intelligent Assistant settings

• Renames feature helper functions and validation errors for the new values key. ConfigMap names now use a shortened '-ia-' suffix while truncating only the release-name prefix to prevent collisions.

charts/rhdh/templates/_helpers.tpl

deployment.yamlRender deployment resources from Intelligent Assistant values +41/-41

Render deployment resources from Intelligent Assistant values

• Migrates plugin checks, volumes, init container, sidecar, ConfigMap mounts, checksums, and external Secret injection to the new resolved values. Runtime resource names tied to Lightspeed Core remain stable where appropriate.

charts/rhdh/templates/deployment.yaml

dynamic-plugins-configmap.yamlAppend Intelligent Assistant plugins to the manifest +3/-3

Append Intelligent Assistant plugins to the manifest

• Loads enabled plugin entries from 'intelligentAssistant.plugins', including the new catalog reference defaults.

charts/rhdh/templates/dynamic-plugins-configmap.yaml

intelligent-assistant-configmaps.yamlGenerate Intelligent Assistant configuration maps +6/-6

Generate Intelligent Assistant configuration maps

• Uses renamed helpers and the relocated vendored files to render stack, server, and profile ConfigMaps.

charts/rhdh/templates/intelligent-assistant/intelligent-assistant-configmaps.yaml

values.yamlIntroduce Intelligent Assistant chart defaults +9/-9

Introduce Intelligent Assistant chart defaults

• Renames the public values namespace and switches frontend and backend plugin defaults from OCI URLs to 'ref://' references. Documentation comments point users to the relocated Secret example.

charts/rhdh/values.yaml

Tests (6) +9 / -9
action.ymlUse relocated Intelligent Assistant fixtures in chart tests +4/-4

Use relocated Intelligent Assistant fixtures in chart tests

• Updates existing-resource test setup to load Lightspeed Core configuration from the renamed vendored directory.

.github/actions/test-charts/action.yml

test.yamlTest templates with Intelligent Assistant plugins +1/-1

Test templates with Intelligent Assistant plugins

• Updates full-plugin sanity-check overrides to use the renamed chart values key.

.github/workflows/test.yaml

default-values.yamlDisable Intelligent Assistant plugins in default CI values +1/-1

Disable Intelligent Assistant plugins in default CI values

• Moves the empty plugin override under the renamed 'intelligentAssistant' key.

charts/rhdh/ci/default-values.yaml

with-custom-configuration-values.yamlTest custom Intelligent Assistant resources +1/-1

Test custom Intelligent Assistant resources

• Migrates custom ConfigMap and existing Secret test values to the new feature key.

charts/rhdh/ci/with-custom-configuration-values.yaml

with-external-db-values.yamlMigrate external database test values +1/-1

Migrate external database test values

• Uses 'intelligentAssistant.plugins' while exercising the external database chart configuration.

charts/rhdh/ci/with-external-db-values.yaml

with-intelligent-assistant-disabled-values.yamlDisable Intelligent Assistant in CI coverage +1/-1

Disable Intelligent Assistant in CI coverage

• Updates the disabled-feature fixture to use 'intelligentAssistant.enabled'.

charts/rhdh/ci/with-intelligent-assistant-disabled-values.yaml

Documentation (4) +58 / -54
CONTRIBUTING.mdDocument Intelligent Assistant config synchronization +4/-4

Document Intelligent Assistant config synchronization

• Points contributor guidance to the new vendored directory and clarifies that only three Lightspeed Core configuration files are synced. It documents external Secret provisioning rather than chart-generated secret vendoring.

CONTRIBUTING.md

README.mdPublish Intelligent Assistant configuration and migration guidance +41/-39

Publish Intelligent Assistant configuration and migration guidance

• Regenerates chart documentation for version 2.0.0 and the 'intelligentAssistant' values API. It explains ref-based plugins, external secrets, renamed ConfigMaps, and 1.x upgrade requirements.

charts/rhdh/README.md

README.md.gotmplDefine Intelligent Assistant documentation source +10/-8

Define Intelligent Assistant documentation source

• Replaces Lightspeed guidance in the README template with Intelligent Assistant behavior, migration, plugin, Secret, and resource naming details.

charts/rhdh/README.md.gotmpl

secret.example.yamlUpdate the external Secret example +3/-3

Update the external Secret example

• Rebrands provider credential guidance and corrects the values example to use 'intelligentAssistant.existingSecret'.

charts/rhdh/files/intelligent-assistant/secret.example.yaml

Other (10) +59 / -70
CODEOWNERSAssign Intelligent Assistant chart files to the AI team +2/-2

Assign Intelligent Assistant chart files to the AI team

• Renames the ownership section and points RHDH ownership at the relocated 'files/intelligent-assistant' directory.

.github/CODEOWNERS

nightly.yamlInclude Intelligent Assistant plugins in nightly overrides +1/-1

Include Intelligent Assistant plugins in nightly overrides

• Changes generated nightly values to select 'intelligentAssistant.plugins' instead of the removed Lightspeed values key.

.github/workflows/nightly.yaml

sync-lightspeed-configs.yamlMake config synchronization branch and chart aware +30/-12

Make config synchronization branch and chart aware

• Resolves vendored paths and chart metadata by target branch, retaining backstage paths for release 1.10 and using Intelligent Assistant paths elsewhere. It also generates synced-file lists dynamically and updates automated PR messaging.

.github/workflows/sync-lightspeed-configs.yaml

Chart.yamlRelease breaking chart version 2.0.0 +1/-1

Release breaking chart version 2.0.0

• Bumps the RHDH chart from 1.0.2 to 2.0.0 to reflect the incompatible values and resource naming changes.

charts/rhdh/Chart.yaml

config.yamlReference the renamed server configuration override +1/-1

Reference the renamed server configuration override

• Updates the vendored Llama Stack server configuration comment to reference 'intelligentAssistant.config.server'.

charts/rhdh/files/intelligent-assistant/config.yaml

lightspeed-stack.yamlReference the renamed stack configuration override +1/-1

Reference the renamed stack configuration override

• Updates the vendored Lightspeed Core stack comment to reference 'intelligentAssistant.config.stack'.

charts/rhdh/files/intelligent-assistant/lightspeed-stack.yaml

rhdh-profile.pyRelocate the bundled RHDH assistant profile +0/-0

Relocate the bundled RHDH assistant profile

• Moves the unchanged prompt profile into the Intelligent Assistant vendored configuration directory.

charts/rhdh/files/intelligent-assistant/rhdh-profile.py

values.schema.jsonRegenerate the Intelligent Assistant values schema +11/-11

Regenerate the Intelligent Assistant values schema

• Replaces the Lightspeed schema property and plugin defaults with 'intelligentAssistant' and 'ref://' package references. Related descriptions now use the new feature terminology.

charts/rhdh/values.schema.json

values.schema.tmpl.jsonDefine the Intelligent Assistant schema template +8/-8

Define the Intelligent Assistant schema template

• Migrates the schema source to the renamed values object and catalog reference plugin defaults.

charts/rhdh/values.schema.tmpl.json

sync-lightspeed-configs.shSync only Lightspeed Core files into Intelligent Assistant +4/-33

Sync only Lightspeed Core files into Intelligent Assistant

• Changes the destination to the RHDH Intelligent Assistant directory and stops generating a vendored Secret from upstream environment defaults. User-facing output now consistently identifies Lightspeed Core configuration.

hack/sync-lightspeed-configs.sh

@rhdh-qodo-merge

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can describe a rule in plain language on the Rules page and Qodo drafts it for you

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@rhdh-qodo-merge rhdh-qodo-merge Bot added documentation Improvements or additions to documentation enhancement New feature or request Tests labels Sep 1, 2026
@Jdubrick

Jdubrick commented Sep 1, 2026

Copy link
Copy Markdown
Author

/cc @rm3l

@openshift-ci
openshift-ci Bot requested a review from rm3l September 1, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request lgtm Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants