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

[BUG] merlin info should generate key #189

Closed
ben-bay opened this issue Mar 23, 2020 · 3 comments · Fixed by #219
Closed

[BUG] merlin info should generate key #189

ben-bay opened this issue Mar 23, 2020 · 3 comments · Fixed by #219
Labels
bug Something isn't working

Comments

@ben-bay
Copy link
Contributor

ben-bay commented Mar 23, 2020

🐛 Bug Report

Describe the bug
Even with the proper results backend password in app.yaml, a run of merlin info shows the results backend as not working because encrypt_data_key has not yet been generated.

To Reproduce
Steps to reproduce the behavior:

  1. Remove encrypt_data_key from merlin home dir.
  2. Run merlin info
  3. See authentication error

Expected behavior
merlin info should generate its own encrypt_data_key if one does not yet exist.

@ben-bay ben-bay added the bug Something isn't working label Mar 23, 2020
@koning
Copy link
Member

koning commented Mar 25, 2020

I do not see an authentication error when I remove the key.

@ben-bay
Copy link
Contributor Author

ben-bay commented Mar 25, 2020

You're right, this behavior isn't as clear-cut as I originally thought.

@koning
Copy link
Member

koning commented May 19, 2020

I think we should add encryption.init_key() to the celery config creation step.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants