Skip to content
This repository has been archived by the owner on Nov 7, 2019. It is now read-only.

Allow PATCH Event Resource #119

Merged
merged 1 commit into from
Oct 16, 2018
Merged

Allow PATCH Event Resource #119

merged 1 commit into from
Oct 16, 2018

Conversation

tantona
Copy link
Contributor

@tantona tantona commented Oct 16, 2018

While testing the master branch I noticed this error

'events "foobarfoobarfoobarfoobarfoobarfoobar.155e2c1afd13d7ce" is forbidden:
User "system:serviceaccount:aws-service-operator:aws-service-operator" cannot
patch events in the namespace "default"' (will not retry!)

I updated the cluster role definition to allow this action

Signed-off-by: Alexander Tanton tantonat@amazon.com

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

While testing the master branch I noticed this error

```
'events "foobarfoobarfoobarfoobarfoobarfoobar.155e2c1afd13d7ce" is forbidden:
User "system:serviceaccount:aws-service-operator:aws-service-operator" cannot
patch events in the namespace "default"' (will not retry!)
```

I updated the cluster role definition to allow this action

Signed-off-by: Alexander Tanton <tantonat@amazon.com>
Copy link
Contributor

@christopherhein christopherhein left a comment

Choose a reason for hiding this comment

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

Nice catch, thanks! :shipit: lgtm

@christopherhein christopherhein added lgtm PR is ready to be merged approved PRs that are approved labels Oct 16, 2018
@tantona tantona merged commit 7665efa into master Oct 16, 2018
@tantona tantona deleted the allow-patch-events branch October 16, 2018 21:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved PRs that are approved lgtm PR is ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants