New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix making OpenAPI V2 aggregator lazy #118212
Fix making OpenAPI V2 aggregator lazy #118212
Conversation
3ab8bce
to
51d035a
Compare
abf3d5f
to
684484a
Compare
staging/src/k8s.io/kube-aggregator/pkg/controllers/openapi/aggregator/aggregator.go
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, left two minor comments. LGTM otherwise.
staging/src/k8s.io/kube-aggregator/pkg/controllers/openapi/aggregator/aggregator.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/kube-aggregator/pkg/controllers/openapi/aggregator/aggregator.go
Outdated
Show resolved
Hide resolved
684484a
to
ae29376
Compare
ae29376
to
ea23e13
Compare
Rebased and squashed |
On behalf of the bug triage release team for 1.28, I'd like to inform you about the Code Freeze (01:00 UTC Wednesday 19th July 2023 / 18:00 PDT Tuesday 18th July 2023) that approaches tomorrow. Please ensure this PR goes in by tomorrow to avoid it being pushed out of the milestone also cc PR labeled SIG-leads: |
This is a cleanup/bug fix PR. Also I think the tags are a bit stale, the changes mainly target sig apimachinery. |
staging/src/k8s.io/kube-aggregator/pkg/controllers/openapi/aggregator/aggregator.go
Show resolved
Hide resolved
/lgtm This makes the v2 aggregation a little more sane, and makes it lazy. There is certainly more potential for clarity, but this is a step into the right direction. |
LGTM label has been added. Git tree hash: fb398cccd570350258a97e3b2ef4cc963bee6a5e
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Jefftree, sttts The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Some notes:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: