Closed shiresky closed 2 years ago
Hi @shiresky. Thanks for your PR.
I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: shiresky
To complete the pull request process, please assign piotrmiskiewicz after the PR has been reviewed.
You can assign the PR to them by writing /assign @piotrmiskiewicz
in a comment when ready.
The full list of commands accepted by this bot can be found here.
/ok-to-test
@shiresky: The following tests failed, say /retest
to rerun all failed tests:
Test name | Commit | Details | Rerun command |
---|---|---|---|
pull-service-catalog-test-e2e-k8s-1-16-9 | c51bbf135632bbc78c2672e8fd342da6024f0c9a | link | /test pull-service-catalog-test-e2e-k8s-1-16-9 |
pull-service-catalog-test-e2e-k8s-1-18-2 | c51bbf135632bbc78c2672e8fd342da6024f0c9a | link | /test pull-service-catalog-test-e2e-k8s-1-18-2 |
pull-service-catalog-test-e2e-k8s-1-17-5 | c51bbf135632bbc78c2672e8fd342da6024f0c9a | link | /test pull-service-catalog-test-e2e-k8s-1-17-5 |
pull-service-catalog-test-migration | c51bbf135632bbc78c2672e8fd342da6024f0c9a | link | /test pull-service-catalog-test-migration |
pull-service-catalog-test-integration | c51bbf135632bbc78c2672e8fd342da6024f0c9a | link | /test pull-service-catalog-test-integration |
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.
I'm affraid that tests would never pass because of
protected groups must have approval annotation "api-approved.kubernetes.io", see https://github.com/kubernetes/enhancements/pull/1111
I have also encountered this communicate on K8s 1.19 but not on K8s 1.18.
Can anybody comment if the issue is possible to resolve or there is Kubernetes code change necessary? @jberkhahn @jhvhs
@shiresky I'm looking into it. Will let you know once I figure it out.
/label api-review
Hi @jhvhs ! Were you able to take a look? Any idea/conclusions?
Hi @jhvhs ! Were you able to take a look? Any idea/conclusions?
I'm preparing this issue for the review process. I'm hoping that it won't take too long.
To unblock this PR, you can add an "api-approved.kubernetes.io":"unapproved, review tracked in https://github.com/kubernetes-sigs/service-catalog/issues/####"
annotation as described in https://github.com/kubernetes/enhancements/blob/master/keps/sig-api-machinery/2337-k8s.io-group-protection/README.md#what-to-do-if-you-accidentally-put-an-unapproved-api-in-a-protected-group
Regarding API review, it looks like there were some interactions with API reviewers in the early days regarding naming, etc, but I don't see an actual review of the schemas themselves:
cc @thockin @smarterclayton as API reviewers that interacted with these types in the past cc @pmorie as early API driver, in case you have pointers to docs or PRs where review was done
Spelunking through API history:
/retitle switch CRDs to v1
There was a lot of early review, but this has been dormant so long and the sig focus was lost and changed (I guess) that I'd have to revisit. The APIs were specifically modelled to make interop with the service broker project - if that focus has changed it's not clear to me that the APIs still accomplish their goal so I think I'd need to see context about "where is service-catalog going" in order to review (which I would be happy to get involved in since the broader service-catalog mission has a lot of value as a generic api).
EDIT: Roughly "I don't think previous review is sufficient in terms of direction to say this meets the newer criteria for V1 APIs and review. I would be happy to dig in and help get understanding of what needs to be done to meet that bar once I have more context for the changes".
I think I'd need to see context about "where is service-catalog going" in order to review (which I would be happy to get involved in since the broader service-catalog mission has a lot of value as a generic api).
@smarterclayton There were no changes to the service-catalog project direction, it is still an OSBAPI implementation. However, the SIG itself is planning to extend the scope within the same area in order to be able to go beyond OSBAPI - https://github.com/kubernetes/community/pull/5746
Hi @jhvhs @smarterclayton
what is the final decision on this PR?
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.
This bot triages issues and PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/remove-lifecycle stale
/lifecycle rotten
/close
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.
This bot triages issues and PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/remove-lifecycle rotten
/close
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle rotten
Hi @jhvhs @smarterclayton @shiresky
Any updates on this PR? and support for K8s v1.22?
is there any thing new about the support for k8s v1.22
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.
This bot triages issues and PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the issue is closedYou can:
/reopen
/remove-lifecycle rotten
Please send feedback to sig-contributor-experience at kubernetes/community.
/close
@k8s-triage-robot: Closed this PR.
This PR is a
What this PR does / why we need it:
service-catalog chart contains CustomResourceDefinition manifests in API group apiextensions.k8s.io/v1beta1 that is deprecated in v1.16 in favor of apiextensions.k8s.io/v1.
Merge Checklist: