-
Notifications
You must be signed in to change notification settings - Fork 600
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
Implement V1 API for messaging.* resources. #3336
Comments
I'm taking a stab at helping with this one. |
@nlopezgi do you need help, e.g. on doing conversion? |
I think that with #3435 we should be done with this issue? Let me know if something is pending to work on it. |
/close |
@matzew: Closing this issue. 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. |
Problem
Add V1 api for messaging.*
As discussed today, we'll break the tasks down by the api group. This issue tracks the work for that. Also as discussed in the meeting, I offered to jot down some representative PRs that are necessary for the work, and hopefully will help folks doing the work:
Introduce types:
#2407
Defaulting webhook:
#2645
Conversions:
#2611
(but since the functions were renamed, so need to use these names):
#2632
That's a start, I might be missing something but figured I'd share some resources so folks can get started :)
The text was updated successfully, but these errors were encountered: