Skip to content
This repository was archived by the owner on Jun 1, 2022. It is now read-only.

PMM-9968 pmm-admin inventory add agent * --log-level#224

Open
YaroslavPodorvanov wants to merge 18 commits into
mainfrom
PMM-9968-pmm-admin-inventory-with-log-level
Open

PMM-9968 pmm-admin inventory add agent * --log-level#224
YaroslavPodorvanov wants to merge 18 commits into
mainfrom
PMM-9968-pmm-admin-inventory-with-log-level

Conversation

@YaroslavPodorvanov

@YaroslavPodorvanov YaroslavPodorvanov commented May 25, 2022

Copy link
Copy Markdown
Contributor

YaroslavPodorvanov and others added 18 commits April 24, 2022 01:17
# Conflicts:
#	go.mod
#	go.sum
…ry-with-log-level" with updated agents.proto log-level
@YaroslavPodorvanov

Copy link
Copy Markdown
Contributor Author
pmm-admin inventory add agent * pmm-admin exists pmm-admin log-level added pmm-managed exists ui exists
external ✔️ ❌️ ?
mongodb-exporter ✔️ ✔️ ?
mysqld-exporter ✔️ ✔️ ?
node-exporter ✔️ ❌️ ?
pmm-agent ✔️ ❌️ ?
postgres-exporter ✔️ ✔️ ?
proxysql-exporter ✔️ ✔️ ?
qan-mongodb-profiler-agent ✔️ ✔️ ?
qan-mysql-perfschema-agent ✔️ ✔️ ?
qan-mysql-slowlog-agent ✔️ ✔️ ?
qan-postgresql-pgstatements-agent ✔️ ✔️ ?
qan-postgresql-pgstatmonitor-agent ✔️ ✔️ ?
rds-exporter ✔️ ✔️ ?
azure-database-exporter ❌️ ❌️ ?

Command ./bin/pmm-admin inventory add agent azure-database-exporter not exists, but exists rds-exporter and also exists in pmm-managed AgentsService::AddAzureDatabaseExporter

@BupycHuk

BupycHuk commented Jun 1, 2022

Copy link
Copy Markdown
Contributor

@YaroslavPodorvanov why did we skip node_exporter?

@YaroslavPodorvanov

Copy link
Copy Markdown
Contributor Author

why did we skip node_exporter?

@BupycHuk, i will add node_exporter too if you think is required

@BupycHuk

BupycHuk commented Jun 1, 2022

Copy link
Copy Markdown
Contributor

Yeah, I think it's better to include node_exporter as well

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants