Expected Behavior
The interfaces (particularly Subscribable) should match the spec from #482, see https://github.com/knative/eventing/blob/master/docs/spec/interfaces.md#subscribable in particular.
Actual Behavior
The implementation (which is currently over in pkg/) defines Subscribable and Channelable separately, and Sinkable and Targetable are not interchangeable.
Additional Info
This will probably also need to update the Subscription Controller and the in-memory Channel controller.
Expected Behavior
The interfaces (particularly Subscribable) should match the spec from #482, see https://github.com/knative/eventing/blob/master/docs/spec/interfaces.md#subscribable in particular.
Actual Behavior
The implementation (which is currently over in
pkg/) defines Subscribable and Channelable separately, and Sinkable and Targetable are not interchangeable.Additional Info
This will probably also need to update the Subscription Controller and the in-memory Channel controller.