Skip to content

Conversation

@Takuka0311
Copy link
Collaborator

No description provided.

@Takuka0311 Takuka0311 changed the title Feat: Reporting metrics upon first startup & Fix: Built-in pipeline was unexpectedly uninstalled Feat: Reporting metrics upon first startup Aug 27, 2025
std::this_thread::sleep_for(std::chrono::milliseconds(10));
}
// 启动时需要立刻更新一下cpu和memory指标,因为正常启动时,cpu和memory指标会等30s更新一次
LogtailMonitor::GetInstance()->UpdateCpuMem();
Copy link
Collaborator

Choose a reason for hiding this comment

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

都没加载配置,cpu、内存此时没有什么意义


OnetimeConfigInfoManager::GetInstance()->LoadCheckpointFile();

#ifdef __ENTERPRISE__
Copy link
Collaborator

Choose a reason for hiding this comment

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

开源版调用会有什么影响?尽量少引入特殊的控制。

const string& pipelineName = pipeline.first;
const string& pipleineDetail = pipeline.second;
if (configSet.find(pipelineName) != configSet.end()) {
LOG_WARNING(sLogger,
Copy link
Collaborator

Choose a reason for hiding this comment

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

什么时候会出现

}
}

void SelfMonitorServer::SendStartMetric() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

尽量时序上去保证先后顺序

LOG_INFO(sLogger, ("send start metric", "sent successfully"));
}

void SelfMonitorServer::UpdateAppInfoJson(const PipelineEventGroup& pipelineEventGroup) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

没有什么有意义的信息,就不要李留这些代码

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.

2 participants