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

API registration should be delayed until after sensor init #59

Open
oskirby opened this issue Jan 30, 2020 · 1 comment
Open

API registration should be delayed until after sensor init #59

oskirby opened this issue Jan 30, 2020 · 1 comment

Comments

@oskirby
Copy link
Contributor

oskirby commented Jan 30, 2020

It seems that there are a bunch of functions in the API that don't work quite as expected until after the sensor init is performed, this leaves a window of time available where D-Bus calls can be made into the API but they will usually just crash and burn.

Since systemd considers the control daemon to be started as soon as the API is available by D-Bus, we should only register the API after the sensor is ready to deal with these API calls.

@oskirby
Copy link
Contributor Author

oskirby commented Jan 30, 2020

This manifests in the strange exposure values that appear when doing a 'service chronos-control restart' since this typically manages to get camApp booting during this window.

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

No branches or pull requests

1 participant