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.

Sign up for GitHub

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

Jump to bottom

Analytics SDK fails to recreate app-state folder if analytics config exists #2877

Closed
jleibs opened this issue Jul 31, 2023 · 0 comments · Fixed by #2878
Closed

Analytics SDK fails to recreate app-state folder if analytics config exists #2877

jleibs opened this issue Jul 31, 2023 · 0 comments · Fixed by #2878
Labels
📊 analytics telemetry analytics 🪳 bug Something isn't working

Comments

@jleibs
Copy link
Member

jleibs commented Jul 31, 2023 •
edited
Loading

Describe the bug
If the app-state folder is missing, every time the viewer starts, we get the error:

2023-07-31T12:49:22.917879Z ERROR re_viewer::app: failed to initialize analytics SDK err=No such file or directory (os error 2)

To Reproduce
Delete the analytics app-state folder:

rm -r ~/.local/share/rerun/analytics/

Do NOT delete the analytics config folder from ~/.config/rerun/analytics.json

Expected behavior
The folder should be recreated on-demand as necessary without an Error.

Known workarounds:

  • Delete the analytics config as well: rm ~/.config/rerun/analytics.json
  • Create the analytics directory: mkdir ~/.local/share/rerun/analytics/
@jleibs jleibs added 🪳 bug Something isn't working 📊 analytics telemetry analytics labels Jul 31, 2023
@jleibs jleibs mentioned this issue Jul 31, 2023
Merged
3 tasks
jleibs added a commit that referenced this issue Jul 31, 2023
@jleibs
Recreate the analytics state directory if necessary before creating p…
93df421 
…ipeline (#2878)

Fixes: #2877

### What
It's possible a user has manually deleted their application state
directory for any number of reasons, but we need it to exist or creating
the pipeline will fail.

### Checklist
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)
* [x] I've included a screenshot or gif (if applicable)
* [x] I have tested [demo.rerun.io](https://demo.rerun.io/pr/2878) (if
applicable)

- [PR Build Summary](https://build.rerun.io/pr/2878)
- [Docs
preview](https://rerun.io/preview/pr%3Ajleibs%2Fanalytics_state/docs)
- [Examples
preview](https://rerun.io/preview/pr%3Ajleibs%2Fanalytics_state/examples)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
📊 analytics telemetry analytics 🪳 bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Recreate the analytics state directory if necessary before creating pipeline rerun-io/rerun
1 participant
@jleibs