+ dev list Thanks! We can iterate on the design on this thread. I noticed that k8s model this in their CRI interface, but not in the top level API: https://github.com/kubernetes/kubernetes/blob/release-1.5/pkg/kubelet/api/v1alpha1/runtime/api.proto#L490
There are some interesting discussion in this thread: https://github.com/kubernetes/kubernetes/issues/5607 I made some suggestion in the following review, but I'd like to hear about what you guys think. https://reviews.apache.org/r/58337/ - Jie On Tue, Apr 18, 2017 at 10:14 AM, haosdent <haosd...@gmail.com> wrote: > Got it, have reverted. Let's discuss the API changes first. > > On Tue, Apr 18, 2017 at 9:02 AM, Jie Yu <j...@mesosphere.io> wrote: > >> Hi, >> >> I noticed this recent commit: >> https://reviews.apache.org/r/58337/ >> >> First of all, thanks for the patch and the review. But I think there is >> some discussion that needs to be carried out at the API level. See my >> comments in the review. >> >> @haosdent, I'd prefer we revert this patch for now so that we can discuss >> what the correct API is going to be. What do you think? >> >> - Jie >> > > > > -- > Best Regards, > Haosdent Huang >
