Skip to content

Sync ai-rolling-demo-1_9 with latest main#26

Merged
gabemontero merged 11 commits intoredhat-ai-dev:ai-rolling-demo-1_9from
thepetk:ai-rolling-demo-1_9
Jan 30, 2026
Merged

Sync ai-rolling-demo-1_9 with latest main#26
gabemontero merged 11 commits intoredhat-ai-dev:ai-rolling-demo-1_9from
thepetk:ai-rolling-demo-1_9

Conversation

@thepetk
Copy link
Contributor

@thepetk thepetk commented Jan 30, 2026

What does this PR do?:

Syncs the rolling demo branch with the latest main

Which issue(s) this PR fixes:

N/A

PR acceptance criteria:

Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened and linked to this PR, if they are not in the PR scope due to various constraints.

  • Tested and Verified

  • Documentation (READMEs, Product Docs, Blogs, Education Modules, etc.)

How to test changes / Special notes to the reviewer:

maysunfaisal and others added 10 commits January 22, 2026 09:38
* Update logic for RAG Metadata caching

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

* Remove .git suffix from URL

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

* Add tests for RAG metadata regen

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

* Fix ruff linting err

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

* Address PR review and ty linter

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

* Update Llama Stack run config for openai to ollama

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

* Update Ollama Deployment to pull embedding model

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

---------

Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
* Add CI/CD for App, Llama Stack

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

* Update .github/workflows/image.yaml

Co-authored-by: Theofanis Petkos <thepetk@gmail.com>

* Update .github/workflows/image.yaml

Co-authored-by: Theofanis Petkos <thepetk@gmail.com>

* Update .github/workflows/image.yaml

Co-authored-by: Theofanis Petkos <thepetk@gmail.com>

---------

Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
Co-authored-by: Theofanis Petkos <thepetk@gmail.com>
Assisted-by: Claude Opus 4.5

Generated-by: Cursor

Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
…v#20)

discovered during testing of the performance agent that the github add comment tool call
was failing for the k8s top outpu.

investigation confirmed that the output exceeded the max length for a github comment, so
adding some truncating logic to adhere to the size limit

also broke out failure processing for creating and issue from failure processing for adding comments;
we should at least note the github issue creation, with an indicator if any of the comment failed

finally, added a timeout to the comment tool calls, as failure/retry at the openai/LLS level lead to
huge delays from the UI perspective (one instance or 1200 seconds).
Assisted-by: Claude Opus 4.5

Generated-by: Cursor

Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
* Fix Secret ACK checkbox

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

* Default Checkbox to False, and enforce using enum

Assisted-by: Claude Opus 4.5

Generated-by: Cursor
Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>

---------

Signed-off-by: Maysun J Faisal <maysunaneek@gmail.com>
* Remove polling

* Avoid concurrent ingestion

* Update streamlit app

* Add manual refresh

* Add more comments

* Update comment
@gabemontero gabemontero merged commit 7d29878 into redhat-ai-dev:ai-rolling-demo-1_9 Jan 30, 2026
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.

3 participants