Release v0.2.32#143
Merged
Merged
Conversation
- Wait for pods to be ready (not just deployments) - Add verification that port forwarding is actually working - Check port connectivity before running tests - Add better error messages and logging - Verify port-forward processes stay alive This should fix timeout issues where tests couldn't connect to localhost:8501
… HTML The test was checking page.content() which returns raw HTML before React renders. Now it properly waits for the UI elements to be visible using Playwright's expect().to_be_visible() which is more reliable and matches the pattern used in other tests.
- Combine port forwarding and test steps so processes persist - Add retry logic in test fixture for navigation failures - Add cleanup trap to ensure port forwarding is killed on exit - Increase timeouts for navigation and network idle waits - Better error handling and verification The issue was that port forwarding processes were dying between steps. By running port forwarding in the same step as tests, the processes stay alive throughout all test execution.
…fore interaction - Implement waiting for UI elements instead of relying on raw HTML - Utilize Playwright's expect().to_be_visible() for better synchronization - Align with patterns used in other tests for consistency This change addresses issues with tests failing due to premature interactions with the UI.
- Wait for networkidle after clicking agent mode - Add additional sleep to allow Streamlit to re-render - This should fix the test that was checking raw HTML instead of rendered UI
The agent type selector (Regular vs ReAct) UI feature is commented out in the codebase, so this test was testing a non-existent feature. The test was redundant with test_agent_mode_shows_toolgroups which already verifies agent mode selection and toolgroups visibility. Removed the test to avoid confusion and reduce maintenance burden.
Removed 'asyncio_default_fixture_loop_scope' which is not a valid pytest configuration option and was causing a PytestConfigWarning. This option is not needed as no tests in the codebase use async/await. The warning was harmless but removing it keeps the test output clean.
Removed tests/e2e_ui/test_e2e_minimal.py which contains duplicate tests that already exist in test_chat_ui.py. The TestMaaSIntegration class and test_maas_chat_completion_direct_mode test are already present in test_chat_ui.py, making this file redundant.
Add new drop down to show all available vector databases and when a database is picked, show the documents already uploaded to the database. Ability to delete a document that was already added to a vector database Ability to upload document to the any chosen vector database Create new vector databases by clicking the Create New button
APPENG-4252: Adds the following features
Signed-off-by: Yuval Turgeman <yturgema@redhat.com>
Use oc project instead of namespace
Signed-off-by: Yuval Turgeman <yturgema@redhat.com>
Increase haproxy timeout and llamastack timeout to allow for upload o…
Signed-off-by: Yuval Turgeman <yturgema@redhat.com>
Try to label namespace for rhaoi
Update model readiness in README
fix: Enhance reliability of UI E2E tests
- Split chat.py into agent.py (tool calling), direct.py (manual RAG), chat.py (UI) - Implement agent mode using Responses API with file_search/web_search tools - Implement direct mode using Completions API with manual RAG - Add proper Python logging and fix linting issues - Fix reasoning persistence and display order in UI Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Signed-off-by: Yuval Turgeman <yturgema@redhat.com>
Signed-off-by: Yuval Turgeman <yturgema@redhat.com> Assisted-by: Claude
Signed-off-by: Yuval Turgeman <yturgema@redhat.com>
Switch to responses API
Format search results nicely Signed-off-by: Yuval Turgeman <yturgema@redhat.com> Assisted-by: Claude
Use new ingestion pipeline
# Conflicts: # deploy/helm/rag/Chart.yaml # deploy/helm/rag/values.yaml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release v0.2.32
This PR merges changes from
devtomainfor release v0.2.32.Checklist
After merging, this will automatically: