Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[build] Include babelcache so it can be chowned to kibana user #7458

Merged
merged 1 commit into from
Jun 15, 2016

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Jun 14, 2016

When optimization is ran as root (or another user) .babelcache.json is written with permissions that may not allow package installs to write to file. This includes an empty cache file in builds and package post install scripts will chown this to the kibana user.

This can be reproduce by installing a kibana package, and before starting the server installing a plugin.

Closes #6730

@@ -0,0 +1,8 @@
module.exports = function (grunt) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

export default here instead of commonjs.

@epixa
Copy link
Contributor

epixa commented Jun 15, 2016

jenkins, test it

@epixa
Copy link
Contributor

epixa commented Jun 15, 2016

LGTM

@jbudz jbudz merged commit e81e1f2 into elastic:master Jun 15, 2016
This was referenced Jun 15, 2016
elastic-jasper added a commit that referenced this pull request Jun 15, 2016
---------

**Commit 1:**
[build] Include babelcache so it can be chowned to kibana user

* Original sha: 83681d9
* Authored by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-06-14T21:48:41Z
elastic-jasper added a commit that referenced this pull request Jun 15, 2016
---------

**Commit 1:**
[build] Include babelcache so it can be chowned to kibana user

* Original sha: 83681d9
* Authored by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-06-14T21:48:41Z
jbudz added a commit that referenced this pull request Jun 15, 2016
jbudz added a commit that referenced this pull request Jun 15, 2016
@epixa epixa added v5.0.0 and removed v5.0.0 labels Jun 28, 2016
@epixa epixa added v4.5.4 and removed v4.5.2 labels Jul 21, 2016
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
[build] Include babelcache so it can be chowned to kibana user

Former-commit-id: e81e1f2
@jbudz jbudz mentioned this pull request Oct 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants