[agent-network] Document the provider save-time check - #947
Conversation
Saving a provider now asks the vendor whether its url and credential work, and refuses the record if they do not. That changes what the Connect a Provider steps promise, so step 6 says so and a new subsection covers the rest: which of the two fields a refusal names, that an edit is only re-checked when the url or the key moves, that a url-only change is tested against the stored credential, and that a rejected key never replaces a working one. The list of what is not checked is the part worth getting right — an operator whose provider saved without a word should be able to see why. It covers the providers that publish no listing, a Bedrock record pointed somewhere AWS does not own, and any upstream on a private address, which the agents reach through the tunnel and the management service cannot reach at all.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
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: Organization UI Review profile: CHILL Plan: Pro Plus 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 |
1 participant
Docs for netbirdio/netbird#7301 (backend) and netbirdio/dashboard#772 (UI), which check a provider's upstream URL and credential against the vendor before storing them.
Stacked on #945 — it adds the Load Models from the Provider section this one refers to, and both edit
providers.mdx. Retarget tomainonce #945 merges.What changed
Step 6 of Connect a Provider now says saving is checked, and a new The Save-Time Check subsection sits under it covering what the steps cannot say in one line:
The list of what isn't checked is the part worth getting right: an operator whose provider saved without a word should be able to see why it was exempt rather than assume the check silently passed. Three cases — providers that publish no listing NetBird can ask for, a Bedrock record pointed at a proxy or self-hosted endpoint rather than an AWS runtime host, and any upstream on a private address that the agents reach through the tunnel but the management service cannot reach at all.
A
<Note>covers the vendor-outage case, since "the provider returned an error" is the one message that names neither of the operator's own fields and the right response is to wait rather than to edit anything.Not included
No screenshot. The refusal state is worth one eventually — the modal staying open with the message against the form — but I can't produce it. Happy to merge as-is and follow up.
Validation
npm run lint:mdx— 293 files, no heading hierarchy violations.I did not run
npm run build; flagging that rather than implying otherwise. One anchor was removed rather than guessed at: linking to Custom & Self-hosted Providers needs a slug for a heading containing&, and nothing else in the repo links to such a heading, so there was no precedent to copy. The bullet reads fine without it.Generated by Claude Code