Closed gortyy closed 6 years ago
@gortyy Great changes. Can you respond to my comments above? I will approve after.
Also, there is no CICD for this image creation. Do you have credentials to release this new version to Docker Hub? If not, let me know and I will assist.
@cmrust I've been added to appoptics group on dockerhub, I'll build and push the image shortly after approval
Even if they enable APPOPTICS_ENABLE_ELASTICSEARCH, or RabbitMQ, Apache, or MongoDB, etc, they will have to modify the config file somehow to set the hostname/port of the ES, RabbitMQ, Apache, MongoDB server, right? It would be nice to have something similar to MYSQL_HOST
for that, or at least something in the documentation mentioning this.. or maybe I missed it.
@cce I believe most people monitoring specific services will take the Sidecar approach detailed in the README. When you run the agent as a sidecar (second container in pod) they share localhost.
My impression is that some of these plugins (though i'm not familiar with all of them) can find the service without an address or credentials if they can see it at the default port on localhost. I believe this is how the Zookeeper example from the README works, for example.
That said, we do have the single-pod deployment specifically for the Kubernetes plugin that we suggest users deploy. I see no reason we couldn't turn on these plugins individually there and pass them whatever they need for finding their services, to centralize and lower the amount of agents a user has to distribute.
Not trying to discourage improving the docs or adding more ENV vars (it is probably necessary on some of these), just adding details.
@cce I suppose that the intention is to provide configuration by mapping folder /opt/appoptics/etc/plugins.d or tasks.d. then conf/appoptics-init.sh will just enable required config files based on env vars. @gortyy this could be actually added to README. That if customer wants to have custom config (i.e. not using localhost), then he should map/copy config files.
Adding more env variables to cover all configs can result in very long list.
Users can either set APPOPTICS_ENABLE_<plugin>
and use default plugin config or mount config files directly to /opt/appoptics/etc/plugins.d/
without setting the env vars. I'll update README
I've added configuration instructions to README
Cool thanks @cmrust @gortyy OK yes some of the configs do default to localhost — checking them this is true for Apache, Elasticsearch, Mesos, MongoDB, MySQL, RabbitMQ, Zookeeper. Cool!
This one of our few differentiators in our Kubernetes story. It’s not that others doesn’t have it, but that everyone doesn’t.
Sent from my mobile
From: chris erway notifications@github.com Sent: Wednesday, November 7, 2018 12:39:42 PM To: appoptics/appoptics-agent-docker Cc: Subscribed Subject: Re: [appoptics/appoptics-agent-docker] Support for additional Appoptics Host Agent Plugins (AO-9502) (#29)
Cool thanks @cmrusthttps://github.com/cmrust @gortyyhttps://github.com/gortyy OK yes some of the configs do default to localhost — checking them this is true for Apache, Elasticsearch, Mesos, MongoDB, MySQL, RabbitMQ, Zookeeper. Cool!
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/appoptics/appoptics-agent-docker/pull/29#issuecomment-436731950, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHOFENL0gAgsyHfb00bHufT7DEqDEHg9ks5usyjugaJpZM4YOVvG.
https://swicloud.atlassian.net/browse/AO-9502
Host Agent in container now has support for:
Each plugin can be enable by setting
APPOPTICS_ENABLE_<plugin_name>
totrue
. In case of Psutil plugin, the flag isAPPOPTICS_DISABLE_HOSTAGENT
and to turn the plugin, it should be set tofalse
. By default only Docker plugin is turned on.