Skip to content

chore(intelligent-assistant): change intelligent assistant (lightspeed) configs repository - #521

Merged
openshift-merge-bot[bot] merged 4 commits into
redhat-developer:mainfrom
michael-valdron:rhdh-intelligent-assistant-configs
Sep 4, 2026
Merged

chore(intelligent-assistant): change intelligent assistant (lightspeed) configs repository#521
openshift-merge-bot[bot] merged 4 commits into
redhat-developer:mainfrom
michael-valdron:rhdh-intelligent-assistant-configs

Conversation

@michael-valdron

Copy link
Copy Markdown
Member

Description of the change

We have migrated redhat-ai-dev/lightspeed-configs to redhat-developer/rhdh-intelligent-assistant-configs for all upcoming feature releases of RHDH. Starting with RHDH 2.1, the intelligent assistant (lightspeed) configs will be fetched from redhat-developer/rhdh-intelligent-assistant-configs.

Note: redhat-ai-dev/lightspeed-configs will be kept active for RHDH 1.10 support, do not backport this change.

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

https://redhat.atlassian.net/browse/RHIDP-14891 (Developer Week)

How to test changes / Special notes to the reviewer

Run bash hack/sync-lightspeed-configs.sh to test syncing the content. Changes from this script are not part of this PR's scope however.

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: Michael Valdron <mvaldron@redhat.com>
@michael-valdron
michael-valdron requested a review from a team as a code owner September 3, 2026 16:21
@rhdh-qodo-merge

Copy link
Copy Markdown

PR Summary by Qodo

Migrate Lightspeed sync to the RHDH intelligent assistant repository

⚙️ Configuration changes 📝 Documentation 🕐 Less than 10 minutes

Grey Divider

AI Description

• Switches default Lightspeed synchronization to the RHDH intelligent assistant configuration
 repository.
• Aligns nightly sync PR messaging and contributor documentation with the migrated source.
• Keeps synchronization behavior and the configurable repository override unchanged.
Diagram

graph TD
  W["Nightly workflow"] -->|"runs"| S["Sync script"] -->|"fetches"| U(["New config repo"])
  S -->|"vendors"| V["Chart configs"]
  D["Contributor guide"] -->|"documents"| S
Loading
High-Level Assessment

Directly replacing the canonical repository in the script and its user-facing references is the simplest appropriate approach. Centralizing the URL across shell, workflow, and Markdown was considered, but would add unnecessary indirection for three distinct contexts while the script already supports explicit repository overrides.

Files changed (3) +3 / -3

Documentation (1) +1 / -1
CONTRIBUTING.mdDocument the new Lightspeed configuration source +1/-1

Document the new Lightspeed configuration source

• Updates contributor guidance to identify the migrated repository as the upstream source for vendored intelligent assistant configuration files.

CONTRIBUTING.md

Other (2) +2 / -2
sync-lightspeed-configs.yamlReference the migrated repository in automated sync pull requests +1/-1

Reference the migrated repository in automated sync pull requests

• Updates the nightly workflow's generated pull request body to identify and link the new intelligent assistant configuration repository.

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

sync-lightspeed-configs.shUse the migrated repository as the default sync source +1/-1

Use the migrated repository as the default sync source

• Changes the script's default GitHub repository to redhat-developer/rhdh-intelligent-assistant-configs. Existing ref selection, check mode, and explicit repository overrides remain unchanged.

hack/sync-lightspeed-configs.sh

@rhdh-qodo-merge

rhdh-qodo-merge Bot commented Sep 3, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 🔗 Cross-repo conflicts (0) 📜 Skill insights (0)

Grey Divider


Remediation recommended

1. Sync PR misstates source ✓ Resolved 🐞 Bug ≡ Correctness
Description
The workflow claims release-1.10 sync changes came from the new intelligent-assistant repository,
even though it checks out release-1.10 before running that branch's legacy sync script. Generated
PRs therefore link reviewers to the wrong upstream changes.
Code

.github/workflows/sync-lightspeed-configs.yaml[151]

+          Automated nightly sync of Lightspeed Core config files from [redhat-developer/rhdh-intelligent-assistant-configs @ ${TARGET_BRANCH}](https://github.com/redhat-developer/rhdh-intelligent-assistant-configs/tree/${TARGET_BRANCH}).
Relevance

●●● Strong

The release-1.10 job runs the legacy sync script, so its generated PR description names the wrong
upstream source.

PR-#480
PR-#200

ⓘ Recommendations generated based on similar findings in past PRs

Evidence
The matrix targets only release-1.10, and the workflow checks out that branch before invoking
hack/sync-lightspeed-configs.sh; consequently, the main-branch default repository changed by this
PR is not the script version executed by this job. The newly hardcoded PR body nevertheless
identifies the new repository as its source.

.github/workflows/sync-lightspeed-configs.yaml[13-18]
.github/workflows/sync-lightspeed-configs.yaml[28-39]
.github/workflows/sync-lightspeed-configs.yaml[80-81]
.github/workflows/sync-lightspeed-configs.yaml[148-156]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The nightly release-1.10 sync PR body incorrectly attributes synced files to `redhat-developer/rhdh-intelligent-assistant-configs`. The job checks out release-1.10 and executes its sync script, so the PR body must identify the legacy `redhat-ai-dev/lightspeed-configs` source.

## Issue Context
The workflow matrix currently contains only `release-1.10`. Restore the legacy repository in the generated body, or derive the displayed repository from the same branch-aware configuration used by the sync operation.

## Fix Focus Areas
- .github/workflows/sync-lightspeed-configs.yaml[151-151]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Context sources
⚠️ Tickets: not configured — ticket URL found in PR but could not be fetched — check ticket provider credentials
✅ Cross-repo context — repo relationships

Grey Divider

Tip of the day
💡 Did you know, you can add REVIEW.md to your repo root and Qodo follows it on every PR

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 labels Sep 3, 2026
…sync workflow if branch target is release-1.10

Signed-off-by: Michael Valdron <mvaldron@redhat.com>
… is the target branch

Signed-off-by: Michael Valdron <mvaldron@redhat.com>
Comment thread .github/workflows/sync-lightspeed-configs.yaml Outdated
Signed-off-by: Michael Valdron <mvaldron@redhat.com>
@sonarqubecloud

sonarqubecloud Bot commented Sep 3, 2026

Copy link
Copy Markdown

Comment thread .github/workflows/sync-lightspeed-configs.yaml
@openshift-ci openshift-ci Bot added the lgtm label Sep 4, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit f5802f8 into redhat-developer:main Sep 4, 2026
10 checks passed
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants