Make Unity AI Gateway governance demo more developer-friendly - #21
Merged
Merged
Conversation
Landed fresh off main for #20, which cannot merge as-is: main received this demo via #19 from the same source commit #20 is based on, so README.md and ai_gateway_demo.ipynb conflicted add/add while each side was byte-identical to what the other branch replaced. Content is unchanged from #20; author credit preserved. - README: cut filler, fixed typos, corrected the model-service step numbering, kept the operational gotchas (guardrail block shape, PII post_call trap, denied requests absent from the inference table, per-provider response shapes) - Notebook: dropped repetitive "solution" phrasing and typos, removed a duplicated gw_client assignment - Added pyproject.toml so the README's `uv sync` step works
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
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.
Lands the content of #20. That PR can't be merged as-is and its author is out for a few weeks, so this re-lands the same change as a fresh branch off
main. Nothing is rewritten on #20, and author credit is preserved on the commit (author: Jules Damji, commit5a58d8b).Why a fresh branch
mainreceived this demo in #19 from the same source commit #20 is based on (4543795), soREADME.mdandai_gateway_demo.ipynbconflicted add/add while each side was byte-identical to what the other had replaced. There was no content to reconcile —main's copies are the versions #20 edited. This branch's tree is byte-identical to #20's head (e2dcc5b).What's in it
post_calltrap, denied requests never reaching the inference table, per-provider response shapes.Jail break,mau,Act 7 :), dropped a duplicatedgw_clientassignment.pyproject.tomlso the README'suv syncstep works — the directory had no dependency manifest.Verification
git diff main...dl/uaig-governance-readme-depsis exactly these three files (168 insertions, 128 deletions); nothing else onmainis touched.from <module> import <symbol>references resolve.uv lock --dry-run: resolves 185 packages against the Databricks PyPI proxy, exit 0, no conflicts.[tool.ruff]config, this branch andmain's content report the identical 18 findings (15× E501, W291, W293, I001) — none added, none removed. No CI runs ruff; flagging it so the config isn't read as a passing gate.Follow-up
Once this lands, #20 can be closed as superseded.