[backport] PR #7308 to 4.x

[elastic-jasper]

Backport PR #7308

Commit 1:
[build] Commit install/remove scripts instead of generating, stop service before removing

• Original sha: 08c9897
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-03-28T02:06:16Z

Commit 2:
[build] template user in install/remove scripts

• Original sha: 5387167
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-02T15:57:16Z

Commit 3:
[build] Run as group kibana

• Original sha: d9a84b2
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-02T16:27:58Z

Commit 4:
[build] Recursive chown optimize folder

• Original sha: 07618c3
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-03T15:12:04Z

Commit 5:
[build] Try service kibana stop before init script

• Original sha: 83c5802
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-03T15:20:16Z

Commit 6:
[build] Attempt to stop service pre-install

• Original sha: 6c6e4d3
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-09T19:40:05Z

Commit 7:
[build] Fail on errors

• Original sha: f9033a1
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-10T15:47:07Z

Commit 8:
[build] Cleanup postrm

• Original sha: bab73b8
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-10T18:16:51Z

Commit 9:
[build] Add comment in postrm explaining empty cases

• Original sha: 4336f2b
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-13T15:03:22Z

Commit 10:
[build] move install to /usr/share, config to /etc/kibana

• Original sha: 6d293af
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-25T14:12:08Z

Commit 11:
[build] consolidate package config

• Original sha: 8f0be48
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-25T19:21:36Z

Commit 12:
[build] Cleanup packages task

• Original sha: db5ac1e
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-25T20:15:09Z

Commit 13:
[build] Style cleanup

• Original sha: 6189c19
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-25T20:19:16Z

Commit 14:
[build] template chown optimize dir

• Original sha: 9213e61
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-05-25T21:27:20Z

Commit 15:
[build] Add name and description to init scripts

• Original sha: e1cb593
• Authored by Jonathan Budzenski jbudz@users.noreply.github.com on 2016-06-07T19:48:57Z

[jbudz]

@epixa this went mostly as expected. old config file stays behind, default one is written to the new location.

[jbudz]

The downside with not merging this is we lose the centralized config, several of the other backports use this. It's probably more work fix all the backports than to merge this and update paths to the old locations.

[epixa]

Sorry, I didn't follow that. So what do you propose?

[jbudz]

Three options:

• Leave it and document changes. Old config files will have to be moved or they won't be automatically used.
• Add a commit on top of this that sets paths back to what they were. Makes backports easier and gives us the centralized config of this PR. Prepare a config upgrade from 4.x -> 5
• Close PR. Backports [build] Set init description to kibana #7431, [packages] Set publish prefix kibana/major_version.x #7387, [build] Cleanup dirs on uninstall #7422 will have conflicts.

I like the second option, safest option is probably third.

[epixa]

By "centralized config" are you referring to the info in config/packages.js?

[jbudz]

Yeah

[epixa]

Cool. Let's do the second one then.

[jbudz]

Closing, will open a new PR for second choice.