-
Notifications
You must be signed in to change notification settings - Fork 82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
【腾讯犀牛鸟开源课题实战】prometheus插件专项建设(PUSH模式支持等) #175
Changes from all commits
809dfe0
ccfc856
8a1de0e
c879973
e1916f8
b980723
7f42fbd
cd24612
816bc28
3d907d6
2309769
1a5bb30
8cb45d4
a931cfc
3a9fe81
35ef136
f87a954
973bf4b
18dc6d7
919b8b3
c986e4f
12afca9
16b5b8c
078f31d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -44,3 +44,13 @@ cc_library( | |
"@trpc_cpp//trpc/metrics/prometheus:prometheus_metrics_api", | ||
], | ||
) | ||
|
||
cc_binary( | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 不需要push这个文件,去掉与之相关的编译引入 |
||
name = "push", | ||
srcs = ["push.cc"], | ||
deps = [ | ||
"@trpc_cpp//trpc/metrics/prometheus:prometheus_metrics_api", | ||
"@trpc_cpp//trpc/log:trpc_log", | ||
|
||
], | ||
) |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -77,6 +77,12 @@ ::trpc::Status ForwardServiceImpl::Route(::trpc::ServerContextPtr context, | |
"counter_name", "counter_desc", {{"const_counter_key", "const_counter_value"}}); | ||
::prometheus::Counter& counter = counter_family->Add({{"counter_key", "counter_value"}}); | ||
counter.Increment(random_num); | ||
|
||
if (::trpc::prometheus::PushMetricsInfo()) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 为啥这里还需要手动调用呢?不能配置一下yaml文件就生效吗? |
||
TRPC_FMT_INFO("Successfully pushed metrics to Pushgateway"); | ||
} else { | ||
TRPC_FMT_ERROR("Failed to push metrics to Pushgateway"); | ||
} | ||
#endif | ||
|
||
auto client_context = ::trpc::MakeClientContext(context, greeter_proxy_); | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
#include <chrono> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 这个文件和框架无关,没必要增加,用法放在文档就好了 |
||
#include <thread> | ||
#include "trpc/metrics/prometheus/prometheus_metrics_api.h" | ||
#include "trpc/log/trpc_log.h" | ||
|
||
|
||
|
||
int main(int argc, char** argv) { | ||
|
||
while (true) { | ||
if (::trpc::prometheus::PushMetricsInfo()) | ||
{ | ||
std::cout << "Successfully pushed metrics to Pushgateway" << std::endl; | ||
} else { | ||
std::cerr << "Failed to push metrics to Pushgateway" << std::endl; | ||
} | ||
|
||
std::this_thread::sleep_for(std::chrono::seconds(5)); // 每60秒推送一次 | ||
} | ||
|
||
return 0; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -44,6 +44,16 @@ plugins: | |
const_labels: | ||
const_key1: const_value1 | ||
const_key2: const_value2 | ||
push_mode: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 例子需要演示pull模式和push模式,应该给出2个文件配置 |
||
enabled: true | ||
gateway_url: "http://pushgateway:9091" | ||
job_name: "test_job" | ||
push_interval_seconds: 2 | ||
auth_cfg: | ||
iss: admin | ||
sub: prometheus-pull | ||
aud: trpc-server | ||
secret: test | ||
log: | ||
default: | ||
- name: default | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
package( | ||
default_visibility = ["//visibility:public"], | ||
) | ||
|
||
cc_library( | ||
name = "jwt-cpp", | ||
hdrs = glob(["**/*.h"]), | ||
deps = [ | ||
"@com_github_openssl_openssl//:libcrypto", | ||
"@com_github_openssl_openssl//:libssl", | ||
], | ||
) |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,10 +20,87 @@ namespace trpc::admin { | |
|
||
PrometheusHandler::PrometheusHandler() { description_ = "[GET /metrics] get prometheus metrics"; } | ||
|
||
void PrometheusHandler::Init() { | ||
PrometheusConfig prometheus_conf; | ||
bool ret = TrpcConfig::GetInstance()->GetPluginConfig<PrometheusConfig>( | ||
"metrics", trpc::prometheus::kPrometheusMetricsName, prometheus_conf); | ||
if (!ret) { | ||
TRPC_LOG_WARN( | ||
"Failed to obtain Prometheus plugin configuration from the framework configuration file. Default configuration " | ||
"will be used."); | ||
} | ||
auth_cfg_ = prometheus_conf.auth_cfg; | ||
} | ||
|
||
bool PrometheusHandler::CheckTokenAuth(std::string bearer_token) { | ||
auto splited = Split(bearer_token, ' '); | ||
if (splited.size() != 2) { | ||
TRPC_FMT_ERROR("error token: {}", bearer_token); | ||
return false; | ||
} | ||
auto method = splited[0]; | ||
if (method != "Bearer") { | ||
TRPC_FMT_ERROR("error auth method: {}", method); | ||
return false; | ||
} | ||
std::string token = std::string(splited[1]); | ||
if (!Jwt::isValid(token, auth_cfg_)) { | ||
TRPC_FMT_ERROR("error token: {}", token); | ||
return false; | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 构造函数做了太复杂的事情,可以定义一个Init函数,把这部分逻辑放在Init函数里 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 已修改。 |
||
return true; | ||
} | ||
|
||
bool PrometheusHandler::CheckBasicAuth(std::string token) { | ||
auto splited = Split(token, ' '); | ||
if (splited.size() != 2) { | ||
TRPC_FMT_ERROR("error token: {}", token); | ||
return false; | ||
} | ||
if (splited[0] != "Basic") { | ||
TRPC_FMT_ERROR("error token: {}", token); | ||
return false; | ||
} | ||
|
||
std::string username_pwd = http::Base64Decode(std::begin(splited[1]), std::end(splited[1])); | ||
auto sp = Split(username_pwd, ':'); | ||
if (sp.size() != 2) { | ||
TRPC_FMT_ERROR("error token: {}", token); | ||
return false; | ||
} | ||
|
||
auto username = sp[0], pwd = sp[1]; | ||
if (username != auth_cfg_["username"] || pwd != auth_cfg_["password"]) { | ||
TRPC_FMT_ERROR("error username or password: username: {}, password: {}", username, pwd); | ||
return false; | ||
} | ||
return true; | ||
} | ||
|
||
void PrometheusHandler::CommandHandle(http::HttpRequestPtr req, rapidjson::Value& result, | ||
rapidjson::Document::AllocatorType& alloc) { | ||
static std::unique_ptr<::prometheus::Serializer> serializer = std::make_unique<::prometheus::TextSerializer>(); | ||
|
||
std::string token = req->GetHeader("authorization"); | ||
|
||
if (!auth_cfg_.empty()) { | ||
if (auth_cfg_.count("username") && auth_cfg_.count("password")) { | ||
// push mode | ||
// use the basic auth if already config the username and password. | ||
if (!CheckBasicAuth(token)) { | ||
result.AddMember("message", "wrong request without right username or password", alloc); | ||
return; | ||
} | ||
} else { | ||
// pull mode | ||
// use the json web token auth. | ||
if (!CheckTokenAuth(token)) { | ||
result.AddMember("message", "wrong request without right token", alloc); | ||
return; | ||
} | ||
} | ||
} | ||
|
||
std::string prometheus_str = serializer->Serialize(trpc::prometheus::Collect()); | ||
result.AddMember(rapidjson::StringRef("trpc-html"), rapidjson::Value(prometheus_str, alloc).Move(), alloc); | ||
} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,7 +15,14 @@ | |
#pragma once | ||
|
||
#include "trpc/admin/admin_handler.h" | ||
#include "trpc/common/config/trpc_config.h" | ||
#include "trpc/log/trpc_log.h" | ||
#include "trpc/metrics/prometheus/prometheus_metrics.h" | ||
#include "trpc/util/http/base64.h" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 用clang-format格式化一下,头文件顺序需要按照字母序顺序排列 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 已修改。 |
||
#include "trpc/util/jwt.h" | ||
#include "trpc/util/prometheus.h" | ||
#include "trpc/util/string/string_helper.h" | ||
#include "trpc/util/time.h" | ||
|
||
namespace trpc::admin { | ||
|
||
|
@@ -24,8 +31,17 @@ class PrometheusHandler : public AdminHandlerBase { | |
public: | ||
PrometheusHandler(); | ||
|
||
void Init(); | ||
|
||
void CommandHandle(http::HttpRequestPtr req, rapidjson::Value& result, | ||
rapidjson::Document::AllocatorType& alloc) override; | ||
|
||
private: | ||
bool CheckTokenAuth(std::string token); | ||
|
||
bool CheckBasicAuth(std::string token); | ||
|
||
std::map<std::string, std::string> auth_cfg_; | ||
}; | ||
|
||
} // namespace trpc::admin | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -71,7 +71,7 @@ class Plugin : public RefCounted<Plugin> { | |
|
||
/// @brief Stop the runtime environment of the plugin | ||
virtual void Stop() noexcept {} | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 记得用clang-format把所有代码文件都格式化一遍(使用项目根目录的.clang-format配置的格式化规范) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 这里出现了不必要的空格 |
||
/// @brief destroy plugin internal resources | ||
virtual void Destroy() noexcept {} | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
默认关闭prometheus,这行可以删掉