Skip to content

ownerscheck demo - do not merge#7659

Closed
itayd wants to merge 5 commits intoenvoyproxy:masterfrom
itayd:patch-35
Closed

ownerscheck demo - do not merge#7659
itayd wants to merge 5 commits intoenvoyproxy:masterfrom
itayd:patch-35

Conversation

@itayd
Copy link
Copy Markdown
Contributor

@itayd itayd commented Jul 20, 2019

No description provided.

@itayd
Copy link
Copy Markdown
Contributor Author

itayd commented Jul 20, 2019

/rk-bless

@repokitteh-read-only
Copy link
Copy Markdown

This PR has been blessed, meaning its repokitteh changes are in effect in this PR.

😽

Caused by: #7659 was labeled by repokitteh[bot].

see: more, trace.

@repokitteh-read-only
Copy link
Copy Markdown

CC @htuch: FYI only for changes made to api/.
CC @itayd: FYI only for changes made to api/.

😽

Caused by: #7659 was synchronize by itayd.

see: more, trace.

@repokitteh-read-only
Copy link
Copy Markdown

CC @htuch: FYI only for changes made to api/.
CC @itayd: FYI only for changes made to api/.

😽

Caused by: #7659 was synchronize by itayd.

see: more, trace.

@repokitteh-read-only
Copy link
Copy Markdown

CC @htuch: FYI only for changes made to api/.
CC @itayd: FYI only for changes made to api/.
CC @envoyproxy/testers: Your approval is needed for changes made to docs/.

😽

Caused by: #7659 was synchronize by itayd.

see: more, trace.

@repokitteh-read-only
Copy link
Copy Markdown

CC @htuch: Your approval is needed for changes made to api/.
CC @itayd: FYI only for changes made to api/.
CC @envoyproxy/testers: Your approval is needed for changes made to docs/.

😽

Caused by: #7659 was synchronize by itayd.

see: more, trace.

@itayd itayd closed this Jul 27, 2019
htuch pushed a commit that referenced this pull request Jul 29, 2019
Per #7423:

* When a PR contains changes to api/, @envoyproxy/api-shepherds are notified via a CC comment.
* When a PR contains changes to api/udpa, @envoyproxy/udpa-wg is notified via a CC comment.
* An additional GitHub check blocking merge is added that will only be marked as passing once an approval review is received from someone in @envoyproxy/api-shepherds.

Risk Level:
None. But if there is any issue, please revert.

Testing:
In test PRs under different users: #7659

Docs Changes:
None.

Release Notes:
None

Fixes #7423

Signed-off-by: Itay Donanhirsh <itay@bazoo.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants