Skip to content
This repository has been archived by the owner on Jun 19, 2022. It is now read-only.

Avoid overwriting eventing clusterrole #1993

Merged
merged 3 commits into from
Dec 16, 2020

Conversation

cathyzhyi
Copy link
Contributor

@cathyzhyi cathyzhyi commented Dec 14, 2020

Fixes #1824

Proposed Changes

  • add events-system- prefix to the clusterrole name

Release Note


Docs

@google-cla google-cla bot added the cla: yes (override cla status due to multiple authors bug) label Dec 14, 2020
@AlexandraRoatis
Copy link
Contributor

I think the prefix should be events-system-, since that is what we use to replace cloud-run-events- in #1917 .

@AlexandraRoatis
Copy link
Contributor

/lgtm
/hold in case anyone else wants to review this

Copy link
Member

@yolocs yolocs left a comment

Choose a reason for hiding this comment

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

/lgtm

@AlexandraRoatis
Copy link
Contributor

AlexandraRoatis commented Dec 16, 2020

/unhold

@AlexandraRoatis
Copy link
Contributor

/lgtm

@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AlexandraRoatis, cathyzhyi, yolocs

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:
  • OWNERS [AlexandraRoatis,cathyzhyi,yolocs]

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

@cathyzhyi
Copy link
Contributor Author

/retest

@knative-prow-robot knative-prow-robot merged commit 3fc6d6f into google:master Dec 16, 2020
cathyzhyi added a commit to cathyzhyi/knative-gcp that referenced this pull request Jan 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved cla: yes (override cla status due to multiple authors bug) lgtm size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ClusterRole of knative-eventing and knative-gcp has same name
4 participants