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

Functionbeat keystore is not packaged #13079

Closed
kvch opened this issue Jul 26, 2019 · 0 comments · Fixed by #13332
Closed

Functionbeat keystore is not packaged #13079

kvch opened this issue Jul 26, 2019 · 0 comments · Fixed by #13332

Comments

@kvch
Copy link
Contributor

kvch commented Jul 26, 2019

Version: master

Steps to reproduce:

$ ./functionbeat keystore create
$ ./functionbeat keystore add ES_PWD
$ ./functionbeat package

Keystore file is missing:

$ unzip -l package.zip
Archive:  package.zip
  Length      Date    Time    Name
---------  ---------- -----   ----
     1332  1980-00-00 00:00   functionbeat.yml
 67144240  1980-00-00 00:00   functionbeat
---------                     -------
 67145572                     2 files

The issue has two parts. First, the keystore file cannot be found, thus keystoreRaw returns a zero lenght byte slice. If that's fixed, the path of the file needs to be changed from the hard coded data/functionbeat.keystore to the one configured in functionbeat.yml.

kvch added a commit that referenced this issue Aug 28, 2019
Previously, Functionbeat put the keystore file under `/tmp` due to the overrides of the function. But when loading the keystore file before packaging the Beat, data path was not yet initialized, so it looked for the file under the incorrect folder. Furthermore, the file in the compressed package was hardcoded to `data/functionbeat.keystore`. So there was no way for functionbeat to find the keystore when loaded to the cloud provider.

Closes #13079
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant