Update future feature flags default table #1091
Conversation
|
Thanks to the pointer @afrittoli ! 🙏 |
tekton-robot
added
the
size/S
tekton-robot
added
the
approved
|
@Yongxuanzhang: changing LGTM is restricted to collaborators In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
| | Feature stability level | Default | | ||
| | ----------------------- | -------- | | ||
| | Stable | Enabled; Cannot be disabled once the flag is removed (after deprecation) | | ||
| | Beta | Disabled | |
There was a problem hiding this comment.
Current docs says beta feature are enabled by default, we are going to change it right?
https://github.com/tektoncd/pipeline/blob/8123d6332245008983aabe1dc910a63bd2c6f261/api_compatibility_policy.md?plain=1#L120
There was a problem hiding this comment.
I think that we are not. Some of the discussion at: tektoncd/pipeline#6948 (comment). The table is for new features only. And the existing beta features guarded by enable-api-fields will still be turned on by default.
I intend to clarify so at the doc updates via tektoncd/pipeline#7090 (which is currently blocked by tektoncd/pipeline#6941).
JeromeJu
force-pushed
the
per-feature-flag
branch
from
2d0f8bc
to
caa01e1
Compare
tekton-robot
added
size/M
This commit updates the table of new feature flags and their default values at each stability level.
JeromeJu
force-pushed
the
per-feature-flag
branch
from
caa01e1
to
1343953
Compare
|
@QuanZhang-William: changing LGTM is restricted to collaborators In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: afrittoli, dibyom, vdemeester, Yongxuanzhang 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 |
|
Wondering why this is not merged yet? |
Thanks for the reminder! And yes this shall be ready to go. |
|
/lgtm |
This commit updates the table of new feature flags and their default values at each stability level.