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

feat(agent): DB persistence and node process disowning #216

Closed
wants to merge 6 commits into from

Conversation

voximity
Copy link
Collaborator

@voximity voximity commented Jun 14, 2024

Motivation

Isolate persisting agent-related data in the same type of database that the control plane uses. Future-proofing and easier to maintain.

End goal is to support zombifying nodes launched by the agent, so the node's process ID can be persisted and tracked, even if the agent is killed and restarted.

Explanation of Changes

TODO

Testing

TODO

Related PRs and Issues

@voximity voximity marked this pull request as draft June 14, 2024 03:55
@voximity
Copy link
Collaborator Author

voximity commented Aug 9, 2024

We should be good to close this now since #231 is merged

@gluax gluax closed this Aug 9, 2024
@voximity
Copy link
Collaborator Author

Wait, I forgot this PR was something else. It has some node process disowning stuff that I'm not sure if we want. We can keep the branch around but the PR closed for now

@gluax
Copy link
Contributor

gluax commented Aug 10, 2024

Wait, I forgot this PR was something else. It has some node process disowning stuff that I'm not sure if we want. We can keep the branch around but the PR closed for now

@voximity @Meshiest and I were also considering just biting the bullet and using systemd since it'd be much easier. Branch might still be a good reference tho.

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

Successfully merging this pull request may close these issues.

2 participants