kueue icon indicating copy to clipboard operation
kueue copied to clipboard

Published 20 hours ago • verified publisher icon kubernetes-sigs

Deprecate the `QueueVisibility` feature gate and corresponding API

Open PBundyra opened this issue 1 year ago • 4 comments

What would you like to be cleaned: I suggest coming with a plan to deprecate the QueueVisibility API. I would start with deprecating the QueueVisiblity feature gate and deleting logic behind it, and then remove the visibility field from ClusterQueue's Status when migrating to a next beta version of Kueue API.

Why is this needed: There was the visibility API introduced that covers all business needs that were cover by the QueueVisibility API and more. The visibility API resolves all drawback related to the QueueVisibility API and addresses its limitations. For now I don't see a rationale for maintaining the QueueVisiblity API. Additionally I believe having two different ways (and two feature gates) to monitor pending workloads may lead to a little confusion regarding their pros and cons.

PBundyra avatar May 22 '24 13:05 PBundyra

@alculquicondor @mimowo @mwielgus @tenzen-y WDYT?

PBundyra avatar May 22 '24 13:05 PBundyra

If you agree with this idea I will add this to the v1beta2 wishlist.

PBundyra avatar May 22 '24 13:05 PBundyra

Yeah, I never liked the mechanism, but we may want to seek additional community feedback at the batch-wg meeting / slack.

mimowo avatar May 22 '24 13:05 mimowo

I agree with you, but let's keep it while v1beta1 for the backward compatibility.

tenzen-y avatar May 22 '24 14:05 tenzen-y

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Aug 20 '24 14:08 k8s-triage-robot

/remove-lifecycle stale

tenzen-y avatar Aug 20 '24 14:08 tenzen-y

I agree with you, but let's keep it while v1beta1 for the backward compatibility.

Keeping for backwards compatibility makes sense, but I think we can already deprecate it, so:

  1. update the kube_features to mention it is deprecated, like here
  2. and also mention it is deprecated in the docs.

mimowo avatar Sep 11 '24 17:09 mimowo

Just to x-reference the related effort of promoting the VisibilityOnDemand to beta: https://github.com/kubernetes-sigs/kueue/issues/2973

mimowo avatar Sep 11 '24 17:09 mimowo

@mbobrovskyi would you like to continue with this as a follow up to https://github.com/kubernetes-sigs/kueue/issues/2973?

mimowo avatar Sep 20 '24 09:09 mimowo

/assign

mbobrovskyi avatar Sep 20 '24 10:09 mbobrovskyi