Skip to content

AAP-58583: SaaS Ansible Chatbot does not find information from RAG DB - #675

Merged
ldjebran merged 1 commit into
mainfrom
romartin/AAP-58583-dynamic-rag-db-id
Dec 22, 2025
Merged

AAP-58583: SaaS Ansible Chatbot does not find information from RAG DB#675
ldjebran merged 1 commit into
mainfrom
romartin/AAP-58583-dynamic-rag-db-id

Conversation

@romartin

Copy link
Copy Markdown
Contributor

Jira Issue: https://issues.redhat.com/browse/AAP-58583

Description

Fixes:

  • Prevents from overriding the chatbot image entry-point
  • Used the python executable reference from the venv, instead of container's /usr/bin

Production deployment

  • This code change is ready for production on its own
  • This PR should be released in 2.4 (cherry-pick should be created)
  • This PR should be released in 2.5 (cherry-pick should be created)
  • This PR should be released in 2.6 (cherry-pick should be created)
  • This code change requires the following considerations before going to production:

@romartin
romartin requested a review from de1987 November 17, 2025 22:01
Comment thread roles/chatbot/templates/chatbot.deployment.yaml.j2
@romartin
romartin marked this pull request as ready for review December 21, 2025 21:51
@romartin
romartin requested a review from ldjebran December 21, 2025 21:52
@romartin

romartin commented Dec 21, 2025

Copy link
Copy Markdown
Contributor Author

@ldjebran this works for upstream, I tested it, BUT I am afraid if it may BREAK downstream once mirroring the repo, as I think the downstream chatbot image does not have the new provider_rag.env file which this PR assumes to be present in the image. I think downstream image actually just exports directly the PROVIDER_VECTOR_DB_ID itself....

@ldjebran

Copy link
Copy Markdown
Contributor

The downstream image has the file

cat /.llama/data/distributions/ansible-chatbot/provider_rag.env 
PROVIDER_VECTOR_DB_ID=vs_cef6882e-00f4-4f26-b130-6a4a1bf15553
bash-5.1$ 

@ldjebran
ldjebran force-pushed the romartin/AAP-58583-dynamic-rag-db-id branch from 32f38d3 to ec0ef17 Compare December 22, 2025 08:59

@ldjebran ldjebran left a comment

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.

LGTM

@ldjebran
ldjebran merged commit 9abf6aa into main Dec 22, 2025
8 checks passed
@ldjebran
ldjebran deleted the romartin/AAP-58583-dynamic-rag-db-id branch December 22, 2025 09:16
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