-
Notifications
You must be signed in to change notification settings - Fork 463
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
Insights pulling SCA certs - graduation criteria update #947
Insights pulling SCA certs - graduation criteria update #947
Conversation
@@ -138,7 +138,7 @@ This feature is planned as a technical preview in OCP 4.9 and is planned to go G | |||
- this new functionality is documented in the Insights Operator documentation as tech preview | |||
|
|||
#### Tech Preview -> GA | |||
- `Share` resource object is automatically created by the Insights Operator | |||
- ~~`Share` resource object is automatically created by the Insights Operator~~ - `Share` object will still be a tech preview in 4.10. Build team can enhance/create it later |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i don't think it's important to reference version numbers in EPs, the roadmap is intended to define the technical milestones that the feature will go through/requirements at each milestone, not the specific release when we expect to achieve them.
Since this is still the section listing tech preview to GA requirements, are you trying to say that the Share object creation will now not be a GA requirement? If so, just say that (along with why, if you want, and how/when the Share object will be introduced).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes :) OK let me try to update. Thanks
/uncc |
b05f679
to
1439d5e
Compare
/lgtm |
/override ci/prow/markdownlint |
@bparees: Overrode contexts on behalf of bparees: ci/prow/markdownlint 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. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bparees 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 |
No description provided.