Skip to content

Commit

Permalink
Remove mixer_client from api_manager. (envoyproxy#72)
Browse files Browse the repository at this point in the history
  • Loading branch information
qiwzhang committed Feb 8, 2017
1 parent 89f0fbf commit 7635536
Show file tree
Hide file tree
Showing 18 changed files with 22 additions and 1,976 deletions.
10 changes: 7 additions & 3 deletions WORKSPACE
Original file line number Diff line number Diff line change
Expand Up @@ -31,16 +31,20 @@ googletest_repositories()
load(
"//contrib/endpoints:repositories.bzl",
"grpc_repositories",
"mixer_client_repositories",
"servicecontrol_client_repositories",
)

grpc_repositories()

mixer_client_repositories()

servicecontrol_client_repositories()

load(
"//src/envoy/mixer:repositories.bzl",
"mixer_client_repositories",
)

mixer_client_repositories()

# Workaround for Bazel > 0.4.0 since it needs newer protobuf.bzl from:
# https://github.com/google/protobuf/pull/2246
# Do not use this git_repository for anything else than protobuf.bzl
Expand Down
12 changes: 0 additions & 12 deletions contrib/endpoints/repositories.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -334,15 +334,3 @@ def servicecontrol_client_repositories(bind=True):
actual = "@servicecontrol_client_git//:service_control_client_lib",
)

def mixer_client_repositories(bind=True):
native.git_repository(
name = "mixerclient_git",
commit = "80e450a5126960e8e6337c3631cf2ef984038eab",
remote = "https://github.com/istio/mixerclient.git",
)

if bind:
native.bind(
name = "mixer_client_lib",
actual = "@mixerclient_git//:mixer_client_lib",
)
1 change: 0 additions & 1 deletion contrib/endpoints/src/api_manager/context/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ cc_library(
"//contrib/endpoints/src/api_manager/auth",
"//contrib/endpoints/src/api_manager/auth:service_account_token",
"//contrib/endpoints/src/api_manager/cloud_trace",
"//contrib/endpoints/src/api_manager/mixer",
"//contrib/endpoints/src/api_manager/service_control",
"//contrib/endpoints/src/api_manager/utils",
"//external:cc_wkt_protos",
Expand Down
15 changes: 4 additions & 11 deletions contrib/endpoints/src/api_manager/context/service_context.cc
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
////////////////////////////////////////////////////////////////////////////////
//
#include "contrib/endpoints/src/api_manager/context/service_context.h"
#include "contrib/endpoints/src/api_manager/mixer/mixer.h"

#include "contrib/endpoints/src/api_manager/service_control/aggregated.h"

Expand Down Expand Up @@ -97,16 +96,10 @@ const std::string& ServiceContext::project_id() const {
}

std::unique_ptr<service_control::Interface> ServiceContext::CreateInterface() {
if (config_->server_config() &&
config_->server_config()->has_mixer_options()) {
return std::unique_ptr<service_control::Interface>(
mixer::Mixer::Create(env_.get(), config_.get()));
} else {
return std::unique_ptr<service_control::Interface>(
service_control::Aggregated::Create(
config_->service(), config_->server_config(), env_.get(),
&service_account_token_));
}
return std::unique_ptr<service_control::Interface>(
service_control::Aggregated::Create(config_->service(),
config_->server_config(), env_.get(),
&service_account_token_));
}

std::unique_ptr<cloud_trace::Aggregator>
Expand Down
42 changes: 0 additions & 42 deletions contrib/endpoints/src/api_manager/mixer/BUILD

This file was deleted.

204 changes: 0 additions & 204 deletions contrib/endpoints/src/api_manager/mixer/mixer.cc

This file was deleted.

Loading

0 comments on commit 7635536

Please sign in to comment.