-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Update OpenStack Cloud Provider KEP #1303
Conversation
Welcome @adisky! |
/assign @lingxiankong @chrigl @ramineni @kendallnelson |
@adisky: GitHub didn't allow me to assign the following users: kendallnelson. Note that only kubernetes members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. 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. |
@adisky excellent, can we remove the in-tree in kubernetes 1.17? |
@dims trying hard for removal in v1.17 |
Oh please, yes. CSI migration is the only thing blocking this? @adisky if the code isn't removed (or the storage code stays) we also have to switch to using the utils/mount package rather than the duplicated code we used for the staging patch. |
@hogepodge Still mount package PR is not yet merged in utils package (not updated from long time), keeping an eye on that as well , if removal not completed in time , Ill submit a PR to use new mount package if merged |
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.
Looks good to me, but my opinion definitely shouldn't have the weight that other's do.
removed WIP, some fields are still TODO, we will update them as per our progress. |
/approve Thanks @adisky |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adisky, andrewsykim 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 |
Openstack Cloud Provider has been implemented from long, This PR updated the KEP with current status of provider work. Fixes:#669