-
Notifications
You must be signed in to change notification settings - Fork 893
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
Introduce feature gates for the scheduler component #805
Conversation
d8be386
to
b71d37a
Compare
To achieve this, I made a few adjustments to make the entire |
That's interesting. I'll look at it later. Thanks! |
59d8f64
to
1852cd5
Compare
Hi @iawia002 I added this PR to the next meeting agenda, hope we can have a chat there. |
Sure, I will be there. |
1852cd5
to
f31ce38
Compare
4f38bbe
to
2f372c2
Compare
Signed-off-by: iawia002 <[email protected]> Signed-off-by: Xinzhao Xu <[email protected]>
2f372c2
to
98ee6ce
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: RainbowMango 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 |
Signed-off-by: iawia002 [email protected]
What type of PR is this?
/kind feature
What this PR does / why we need it:
Introduce the feature gates package to allow users to control whether to enable/disable some global features, this can also be used in other components in the future.
Which issue(s) this PR fixes:
Fixes #701
Special notes for your reviewer:
I directly removed the
--failover
flag. I don’t know if we need to ensure compatibility in the current version.Does this PR introduce a user-facing change?:
/cc @RainbowMango