Skip to content

Revert servicecontrol adapter BUILD change from PR 1987#2153

Merged
istio-merge-robot merged 1 commit intoistio:masterfrom
manlinl:fix
Dec 14, 2017
Merged

Revert servicecontrol adapter BUILD change from PR 1987#2153
istio-merge-robot merged 1 commit intoistio:masterfrom
manlinl:fix

Conversation

@manlinl
Copy link
Copy Markdown
Contributor

@manlinl manlinl commented Dec 14, 2017

What this PR does / why we need it:
Test code shouldn't be compiled into prod code. This reverts change from #1987. It appeared it was an accident.

Release note:

NONE

Test code shouldn't be compiled in prod code
@manlinl manlinl changed the title Revert change from PR 1987 Revert servicecontrol adapter BUILD change from PR 1987 Dec 14, 2017
@manlinl manlinl added the mixer label Dec 14, 2017
Copy link
Copy Markdown
Contributor

@geeknoid geeknoid left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@istio-merge-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geeknoid

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:

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 aa41f6b into istio:master Dec 14, 2017
@manlinl manlinl deleted the fix branch December 14, 2017 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants