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

Improve services start-up mechanism #287

Closed
imobachgs opened this issue Nov 2, 2022 · 1 comment · Fixed by #295
Closed

Improve services start-up mechanism #287

imobachgs opened this issue Nov 2, 2022 · 1 comment · Fixed by #295
Assignees
@mvidner
Labels
bug Something isn't working enhancement New feature or request

Comments

@imobachgs
Copy link
Contributor

The D-Installer backend is composed of several D-Bus services. A service might have dependencies on other services, so they are started sequentially (as we know the dependencies in advance). We decided to use a simplistic approach (waiting 1 second between one service and the next), but it is time to evolve this solution to something more robust. If some service takes more time than expected to start, it might cause the whole start-up sequence to fail.
@imobachgs imobachgs added bug Something isn't working enhancement New feature or request labels Nov 2, 2022
This was referenced Nov 2, 2022
Closed
Closed
@mvidner mvidner mentioned this issue Nov 7, 2022
Merged
5 tasks
@imobachgs imobachgs linked a pull request Nov 14, 2022 that will close this issue
Use D-Bus activation instead of explicit service starting #295
Merged
5 tasks
@imobachgs
Copy link
Contributor Author

#295 fixes this problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mvidner mvidner

Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use D-Bus activation instead of explicit service starting agama-project/agama
2 participants
@imobachgs @mvidner