Repository navigation
feat(dcgw+aigw): CGW in AIGW - #7459
cloudjumpercat wants to merge 6 commits into
Conversation
✅ Deploy Preview for kongdeveloper ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com>
Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com>
Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com>
* Add Azure notes Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com> * Apply KSAI feedback Co-authored-by: kong-ksai[bot] <313415923+kong-ksai[bot]@users.noreply.github.com> --------- Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com> Co-authored-by: kong-ksai[bot] <313415923+kong-ksai[bot]@users.noreply.github.com>
ad68292 to
d174793
Compare
* update managed cache and ai index/landing page Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com> * datastore changes, updates to managed cache, more cross links with AIGW pages Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com> * Add include Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com> * Apply batched suggestions from code review Co-authored-by: Angel <Guaris@users.noreply.github.com> Co-authored-by: Diana <75819066+cloudjumpercat@users.noreply.github.com> * fix remaining feedback Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com> --------- Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com> Co-authored-by: Angel <Guaris@users.noreply.github.com>
|
✅ KSAI Review: Finished Run report (federated) · 1 paid run · $0.2664 total
Reviewing cost $0.2664 |
There was a problem hiding this comment.
| Check | Result |
|---|---|
| Scope | Docs change for AI Gateway 2.0 on Dedicated Cloud Gateways: shared note and prereq includes, managed-cache sections, AI Gateway index and landing page, DCGW reference page. |
| Mandate | default-code-reviewer |
| Findings | 3 Low |
| Findings audit | Independent audit completed: three Low findings verified against the tree and upheld; one split suggestion narrowed; no removals or additions. |
Documentation-only change: shared note and prereq includes, a managed-cache section for AI Gateway, a Dedicated Cloud Gateways reference section, plus index and landing-page entries. Links, anchors, and includes resolve. Three convention issues remain, all Low.
Additional Risk pass: no leaked secrets, no security or attack-surface change, and no resource or growth concern. No additional concerns found.
| If you want both {{site.base_gateway}} and {{site.ai_gateway}} to run in a Dedicated Cloud Gateway, you must use separate Dedicated Cloud Gateway control planes for them, instead of combining both on a single control plane. | ||
| Sharing networks is best practice. | ||
|
|
||
| ### Configure an {{site.ai_gateway}} control plane |
There was a problem hiding this comment.
🔵 Low q
This new section adds setup steps and limitations to a reference page that already has 26 headings, 13 of them H2s. The page explains concepts, gives lookup tables, walks through configuration in tabs, and lists limitations. Should the AI Gateway setup steps move to their own page or how-to?
React 👍 if this finding helped, 👎 if it did not.
There was a problem hiding this comment.
How tos will be coming in the future so I can reduce this section then
Signed-off-by: Diana <75819066+cloudjumpercat@users.noreply.github.com>
Description
Fixes #7430
TODO:
Preview Links
Most of the docs were reviewed already in PRs that were merged into this one. The only docs that need a final check are:
For reviewers: You may be wondering, why no how tos? I descoped that work due to the tight time frame with releases this week. I'm planning to update them as fast follows in this ticket: #7789
All links:
Checklist
descriptionentry in frontmatter.