Skip to content

Channelable Aggregated Cluster Role#1341

Merged
knative-prow-robot merged 5 commits into
knative:masterfrom
nachocano:channelable-resolver-writer
Jun 5, 2019
Merged

Channelable Aggregated Cluster Role#1341
knative-prow-robot merged 5 commits into
knative:masterfrom
nachocano:channelable-resolver-writer

Conversation

@nachocano
Copy link
Copy Markdown
Contributor

Helps with the channel CRD implementations. E.g., #1217

Proposed Changes

  • Adding an aggregated Channelable cluster role to eventing-controller, so that the subscription controller can read and update specific the subscribers list of different channel implementations (e.g., kafkachannel, inmemorychannel, etc.)
  • Channel CRDs should create their own roles matching the label duck.knative.dev/channelable. This should be stated in the Channels control plane spec.

Release Note

NONE

@googlebot googlebot added the cla: yes Indicates the PR's author has signed the CLA. label Jun 4, 2019
@knative-prow-robot knative-prow-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jun 4, 2019
@nachocano
Copy link
Copy Markdown
Contributor Author

/assign @Harwayne
/assign @akashrv

Comment thread config/200-channelable-clusterrole.yaml Outdated
Comment thread config/200-channelable-clusterrole.yaml Outdated
Comment thread config/channels/in-memory-channel/200-channelable-clusterrole.yaml
@vaikas
Copy link
Copy Markdown
Contributor

vaikas commented Jun 5, 2019

/approve
Leaving out the lgtm so that you can resolve the naming issue.

@knative-prow-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nachocano, vaikas-google

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow-robot knative-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 5, 2019
Copy link
Copy Markdown
Contributor

@Harwayne Harwayne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 5, 2019
@knative-prow-robot knative-prow-robot merged commit 682efbb into knative:master Jun 5, 2019
@Harwayne Harwayne mentioned this pull request Jun 5, 2019
dsimansk pushed a commit to dsimansk/eventing that referenced this pull request Jul 28, 2021
* Execute the tests again

Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>

* For now, we have to disable the ko publish, since we do not need it at all, but its causing issues on our CI

Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants