Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Add setup script for 'apm-server' #82

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

fatmcgav
Copy link
Contributor

@fatmcgav fatmcgav commented Feb 12, 2019

This creates a keystore for apm-server, and adds the ELASTIC_PASSWORD entry to it.
Also copies it into /config/apm-server so that it's available to the server.

Resolves #73

@fxdgear
Copy link
Contributor

fxdgear commented Feb 12, 2019

@fatmcgav this will get resolved in: #76

BUT I'm curious why is the apm-server binary not getting picked up in

We shouldn't need a new setup script.

@fatmcgav
Copy link
Contributor Author

fatmcgav commented Feb 12, 2019

@fxdgear So I went with a new script as there were some subtle differences in path layouts, commands to setup dashboards etc...

Edit: Actually, the file path issue became a non-issue, but the permissions was an issue... apm-server.yml had to be owned by root to be able to progress...

@fatmcgav
Copy link
Contributor Author

Also, when I started looking, having apm-server configured in setup-beat.sh just felt... wrong... to me... apm-server != beat???

@fxdgear
Copy link
Contributor

fxdgear commented Feb 12, 2019

I agree on the name bit.

Perms is kind of annoying, which is why I created my PR referenced above, but haven’t had time to dig into the last issue I ran into. :(

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.

APM server setup fails
2 participants