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

Add setup script for 'apm-server'#82

Open
fatmcgav wants to merge 4 commits into
masterfrom
fix_apm-server_setup
Open

Add setup script for 'apm-server'#82
fatmcgav wants to merge 4 commits into
masterfrom
fix_apm-server_setup

Conversation

@fatmcgav

@fatmcgav fatmcgav commented Feb 12, 2019

Copy link
Copy Markdown
Contributor

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

@fatmcgav fatmcgav requested review from Conky5 and fxdgear February 12, 2019 17:11
@fxdgear

fxdgear commented Feb 12, 2019

Copy link
Copy Markdown
Contributor

@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

fatmcgav commented Feb 12, 2019

Copy link
Copy Markdown
Contributor Author

@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
Copy Markdown
Contributor Author

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

@fxdgear

fxdgear commented Feb 12, 2019

Copy link
Copy Markdown
Contributor

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