-
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
We need to document our service account and roles story #909
Comments
There is a better option for adding roles: https://github.com/knative/serving/blob/070c82894b8b4ed24447012d7c0f10fc40e611c8/test/config/200-clusterrole.yaml#L24 Role based label selector. |
I think this needs to be more specific. What do you want documented and where? I believe this came from working with #788 (specifically this comment), so the way Broker & Trigger use You are correct that this is a bug that the Broker says it is ready before its constituent pieces say they are ready, that should be fixed. But for documentation, that PR did include documentation about exactly what was needed and why: manual setup. Given that the readiness bug needs to be fixed in addition, where would you like to see the |
Here's a user story and a success test for the contributor persona:
|
/kind feature-request |
Make this part of the required documentation overhaul for 0.6 release |
/milestone v0.10.0 |
* Separate e2e/conf tests Signed-off-by: Matthias Wessendorf <mwessend@redhat.com> * Tracing test............. Signed-off-by: Matthias Wessendorf <mwessend@redhat.com> * Removing the patch to skip those tests Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>
This issue is stale because it has been open for 90 days with no |
I am fairly confused on how we
eventing
plans to use service accounts and roles. This came up because I was unable to get broker to work without creating a service account and role in my namespace but there was already a cluster role for a service account that does not get installed as part of the normal process.We need to give a story how we are managing these and when we make a new one.
The text was updated successfully, but these errors were encountered: