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

Track processes in the database and on the Dashboard #472

Merged
merged 1 commit into from
Dec 31, 2021

Conversation

bensheldon
Copy link
Owner

@bensheldon bensheldon commented Dec 30, 2021

Connects to #421.

  • Adds a GoodJob::Projects ActiveRecord model
  • Each process has a unique UUID that is updated on fork (when the PID changes)
  • The record is managed within the Notifier using the same long-lived postgres connection that LISTEN/NOTIFY uses.
  • An advisory lock is taken on the Process record (to take the place of a heartbeat/ttl) to detect dangling SIGKILLed processes.

@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 30, 2021 02:42 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 30, 2021 17:29 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 30, 2021 17:42 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 30, 2021 19:01 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 30, 2021 19:09 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 30, 2021 20:17 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 30, 2021 23:46 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 31, 2021 01:14 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 31, 2021 16:45 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 31, 2021 23:03 Inactive
@bensheldon bensheldon temporarily deployed to goodjob-notifier-proces-tapepz December 31, 2021 23:29 Inactive
@bensheldon bensheldon added the enhancement New feature or request label Dec 31, 2021
@bensheldon bensheldon merged commit 208aa52 into main Dec 31, 2021
@bensheldon bensheldon deleted the notifier_process_id branch December 31, 2021 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant