nebari-dev / nebari-dev/llm-serving-pack
Document breaking admission change from Access validation
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 3
- Forks
- 3
- Avg merge
- 16h 31m
- Merged PRs (30d)
- 11
Description
Follow-up from #47.
#47 added a webhook rule rejecting LLMModel CRs where Access.Public is not true and Access.Groups is empty. The PR body framed this as a narrow invariant for the new internal authorization block, but the rule fires on every CR regardless of whether the internal endpoint is enabled. Any existing manifest that omitted Access entirely now fails admission.
This is an intentional tightening, but it is a breaking change that is not called out in any user-visible place.
Asks
- Add a section to
docs/design.md(orREADME.md) consolidating the admission rules the webhook enforces: managed-namespace label, subdomain length, subdomain collision, and the new access rule. - Add a CHANGELOG entry (or equivalent release-notes file) for the next release noting the new required field combination and the error message.
- Consider whether to narrow the rule to only fail when
endpoints.internal.enabledis true; keeping it broad is safer but should be a deliberate call.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read issue #47 and inspect the webhook implementation to verify the managed-namespace, subdomain, collision, and access rules and their error message. Update docs/design.md or README.md with the consolidated rules, then add the breaking access requirement to the changelog or release notes. Document the decision about whether the rule applies only when the internal endpoint is enabled.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- documentation, security
- Issue type
- Documentation
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100