Skip to content

Document pr self-approve#356

Merged
istio-merge-robot merged 2 commits intomasterfrom
yutongz-patch-1
Aug 3, 2017
Merged

Document pr self-approve#356
istio-merge-robot merged 2 commits intomasterfrom
yutongz-patch-1

Conversation

@yutongz
Copy link
Copy Markdown
Contributor

@yutongz yutongz commented Aug 1, 2017

Release note:

None

@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Aug 1, 2017
Copy link
Copy Markdown
Member

@gyliu513 gyliu513 left a comment

Choose a reason for hiding this comment

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

/lgtm


> Note: Do not comment "/approve" or add "approved" label unless you are 100% sure you want this change, because after you say that, the pr will be merge any minutes.

> Note: The PR will be self-approved if the PR creator is in OWNERS file and this PR is associated with at lease one issue.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

with at least*

@yutongz
Copy link
Copy Markdown
Contributor Author

yutongz commented Aug 2, 2017

@nlandolfi PTAL

@nlandolfi
Copy link
Copy Markdown
Contributor

/lgtm

@sebastienvas
Copy link
Copy Markdown
Contributor

/approve no-issue

@istio-merge-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gyliu513, nlandolfi, sebastienvas, yutongz

Associated issue requirement bypassed by: sebastienvas

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

Details Needs approval from an approver in each of these OWNERS Files:
  • OWNERS [sebastienvas,yutongz]

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot istio-merge-robot merged commit 37ba847 into master Aug 3, 2017
@yutongz yutongz deleted the yutongz-patch-1 branch August 15, 2017 06:36
soloio-bot pushed a commit to soloio-bot/test-infra that referenced this pull request Jun 22, 2023
Remove testgrid notifications to upstream's oncall group from mention, also update the service account for prow
deployer to use the prow-deployer service account which should allow us to deploy updates via PRs instead of
manually.

Signed-off-by: Daniel Hawton <daniel.hawton@solo.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants