LCORE-4167: Merge main into release/0.7 - #2721
Conversation
…dpoint Adds custom OpenTelemetry span instrumentation to `POST /v1/streaming_query/interrupt` so interrupt requests are observable in traces, including their outcome. Each interrupt request emits **one span**: `stream.interrupt`. **Span attributes** - `interrupt.request_id` — the streaming request id targeted by the interrupt - `stream.conversation.id` — conversation the stream belongs to - `interrupt.result` — cancel outcome: `cancelled` / `not_found` / `forbidden` / `already_done` **Span status** The cancel outcome is reflected in the span status: - `OK` — `cancelled` and `already_done` (both are HTTP 200 responses) - `ERROR` (with reason) — `not_found` (404) and `forbidden` (403) **Supporting change: conversation id in the interrupt registry** The interrupt request carries only `request_id`, so the conversation id was not otherwise available at the endpoint. `conversation_id` is now retained on `ActiveStream` and populated at stream registration time (from the response generator context), letting the endpoint surface `stream.conversation.id` on the span. **Privacy** `stream.conversation.id` is recorded **only when the caller owns the stream**. A non-owner (forbidden) or unknown (not-found) request never has a conversation id placed on its span, so a caller cannot surface another user's conversation identifier through a trace. No tokens or secrets are recorded. Signed-off-by: Anik Bhattacharjee <anbhatta@redhat.com>
…d-unused-argument LCORE-3296: Removed unused argument
Targeted doc and CLI fixes from review: restore poc-results under ogx-config-merge with relative links, OgxConfiguration naming in published schema docs, building_applications link, migration accuracy, prompt-guardrails anchor, and stale CLI help. Co-authored-by: Cursor <cursoragent@cursor.com>
…nnotation LCORE-3597: Added missing type annotation
…case-naming LCORE-3297: Snake-case naming
…d-dependencies-2 LCORE-3580: Updated dependencies
…combined LCORE-2547: address OGX rename review findings
The generated schema moved to docs/devel_doc/openapi.json (the generate target and the openapi_spectral workflow already use that path), but the lint-openapi target still linted docs/openapi.json, which no longer exists. make verify therefore failed locally on a missing file while CI, which runs Spectral directly against the new path, stayed green.
…tch-jira output extract_text rendered only strong/code marks, lists, headings and code blocks; every other ADF node was dropped on the floor. Link marks lost their href, inlineCard nodes (the smart links Jira makes out of pasted Jira/GitHub URLs) produced nothing at all, mentions vanished, and table rows vanished — so for the tickets that reference other tickets or PRs, the tool hid exactly the references a reader needs. Render link marks as "text <href>" when the text is not already the URL, inlineCard as "<url>", mentions as their display text, tables as pipe-separated rows, blockquotes with a "> " prefix and rules as "---". The Python sits inside a double-quoted shell string, so the new code sticks to single quotes and string concatenation. Verified on LCORE-3788, whose description carries five smart links that previously did not appear.
Signed-off-by: red-hat-konflux-kflux-prd-rh02 <190377777+red-hat-konflux-kflux-prd-rh02[bot]@users.noreply.github.com>
…ored-if-else-chain LCORE-3819: refactored if-elif chain in tests
LCORE-3298: cleanup
…d-dependencies-2 LCORE-3580: Updated dependencies
…issue LCORE-3298: Fixed issue
…-unified-mode-step-definitions LCORE-2343: implement behave step definitions for unified-mode feature files
…-align-guardrails-spec LCORE-3386: align prompt guardrails spec with the shipped shield-based design
…d-hexagonal-arch LCORE-2441: Updated hexagonal architecture
Signed-off-by: Lucas <lyoon@redhat.com>
LCORE-3599: Added missing type annotation
LCORE-4125: Unnecessary assignment to new_response before return statement
…d-dependencies-2 LCORE-3820: Updated dependencies
LCORE-3991: Enable rule B011 on CI
LCORE-4088: Fix MCP auth header "Bearer" duplication
…references/main Update Konflux references
LCORE-4169: Fix RPM CVEs
…gress fix(OGX): Add pgvector to ogxlibdev deps for remote::pgvector vector_io
LCORE-4171: Update Docs for 0.7.0rc3
…lute-paths RSPEED-3444: fix app_routes_paths to use effective route path including mount prefix
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@jrobertboos do we need to merge all commits into rc3? Wouldn't it be more safe to just cherry-pick relevant commits for RHDH? |
|
@asimurka well won't we have to do it for 0.7.0ga by the end of the month? |
Description
Type of change
pyproject.toml+uv.lock]requirements.*.txtfor Konflux]Tools used to create PR
Identify any AI code assistants used in this PR (for transparency and review context)
Related Tickets & Documents
Checklist before requesting a review
Testing