[AI-3] Provision the LLM connection from the environment - #24887
[AI-3] Provision the LLM connection from the environment#24887tangopium wants to merge 1 commit into
Conversation
|
Caution The provided work package version does not match the core version Details:
Please make sure that:
|
|
Caution The Enterprise plan field is not set on the work package Details:
Please make sure that:
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: dcefe085a3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
dcefe08 to
fa61b44
Compare
fa61b44 to
4ca02db
Compare
4ca02db to
b33ce35
Compare
Adds the OPENPROJECT_LLM__CONNECTION setting and an EnvData seeder, so an on-premise deployment can ship the connection alongside its LLM sidecar without touching the UI. When the setting is present the record belongs to the deployment: the form renders read-only with a banner, the destructive actions refuse, and the base contract rejects any write. Seeding must not block on -- or fail because of -- a server that has not finished starting, so the environment path validates with its own contract that skips the reachability probe, and refreshes the catalogue through a background job instead of inline. Part 9 of the AI-3 stack. https://community.openproject.org/work_packages/66020
|
Warning Flaky specs
🤖 Ask Copilot to investigateCopy the prompt below into a new comment on this PR to delegate the investigation to GitHub Copilot. It will look into the flakiness and open a separate pull request with you as reviewer. |
b33ce35 to
786a84f
Compare
Ticket
AI-3
What are you trying to accomplish?
PR 9 of 11 in the AI-3 stack. Adds the
OPENPROJECT_LLM__CONNECTIONsetting and an EnvData seeder, so an on-premise deployment can ship the connection alongside its LLM sidecar without touching the UI. When the setting is present the record belongs to the deployment: the form renders read-only with a banner, the destructive actions refuse, and the contract rejects any write. Seeding must not fail because of a server that has not finished starting, so the environment path skips the reachability probe and refreshes the catalogue in a background job.Merge checklist
llm_connectionfeature flagStacked on #24886.