Skip to content
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

Add support for feature gates #556

Closed
Tracked by #523
danielvegamyhre opened this issue May 3, 2024 · 6 comments · Fixed by #557
Closed
Tracked by #523

Add support for feature gates #556

danielvegamyhre opened this issue May 3, 2024 · 6 comments · Fixed by #557
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@danielvegamyhre
Copy link
Contributor

danielvegamyhre commented May 3, 2024

What would you like to be added:

Add support for feature gates in JobSet, like are used in upstream k8s and Kueue here.

Why is this needed:

  • Allow customer to quickly disable a feature if a bug is detected
  • Allow us to have "opt-in" features that are off by default.
@danielvegamyhre danielvegamyhre added the kind/feature Categorizes issue or PR as related to a new feature. label May 3, 2024
@googs1025
Copy link
Member

/assign hi @danielvegamyhre i'll try it

@googs1025
Copy link
Member

Do we need to discuss which feature gates need to be added?

@googs1025
Copy link
Member

/assign

@googs1025
Copy link
Member

I have been thinking, can we include these two items in the feature gate? They can be enabled by default, and if there are more features in the future, they can be added. @danielvegamyhre
Establishment of a headless service
TTL after finished

@danielvegamyhre
Copy link
Contributor Author

I don't think we can (or should) put existing features behind feature gates, it would be weird for the feature gate to exist on some versions but not others. I think we should put new features (not released yet) which are somewhat risky behind a feature gate. Not sure off the top of my head if this aligns with upstream k8s standard practices, @kannon92 do you know?

@googs1025
Copy link
Member

googs1025 commented May 4, 2024

Understood. As far as I understand, the current features of JobSet are all existing ones, and there are no new unstable features yet. In that case, we can provide the feature gate functionality for other developers to use, but without any default configuration. How does that sound? @danielvegamyhre

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants