Title: Create multiprocess files with process uuid by ethervoid · Pull Request #694 · prometheus/client_python · GitHub
Open Graph Title: Create multiprocess files with process uuid by ethervoid · Pull Request #694 · prometheus/client_python
X Title: Create multiprocess files with process uuid by ethervoid · Pull Request #694 · prometheus/client_python
Description: This is a draft open to discussion In order to be able to clean-up the process files once it's stopped or destroyed, we need to identify those files with a UUID. If not, those files are going to be processed and collected and will keep growing until we restart the server or master process to perform the cleanup of the metrics directory This change uses the psutil library that gives a hash based on PID+Process creation time in order to differentiate two processes with the same PID but those are different. I have some questions about the feasibility of this change: Is the approach ok? Mainly is respectful in case of exception and even though some files could be left behind is not the same situation as we have now. Also, there is an improvement that I can add and is marking those files I couldn't delete as "type_pid_uuid_defunct" so we can do something to directly remove those files Is it ok to start using an external library? I've seen you're trying to avoid it Tests are failing because the test process is not running and the unittest.mock library is only from 3.3 it's ok to move all the multiprocess tests to required that version to run? This change aims to fix #568 // @csmarchbanks Signed-off-by: Mario de Frutos mario@defrutos.org
Open Graph Description: This is a draft open to discussion In order to be able to clean-up the process files once it's stopped or destroyed, we need to identify those files with a UUID. If not, those files are going t...
X Description: This is a draft open to discussion In order to be able to clean-up the process files once it's stopped or destroyed, we need to identify those files with a UUID. If not, those files are goi...
Opengraph URL: https://github.com/prometheus/client_python/pull/694
X: @github
Domain: patch-diff.githubusercontent.com
| route-pattern | /:user_id/:repository/pull/:id/checks(.:format) |
| route-controller | pull_requests |
| route-action | checks |
| fetch-nonce | v2:54b61634-4fe5-847a-9e80-99c3ba0d4c4d |
| current-catalog-service-hash | 87dc3bc62d9b466312751bfd5f889726f4f1337bdff4e8be7da7c93d6c00a25a |
| request-id | B196:4D0D3:DABBA1:1146FDA:6991527E |
| html-safe-nonce | 71c5422619e820f6f6fbb5016408367845b1e042402a843115d07206196f6282 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCMTk2OjREMEQzOkRBQkJBMToxMTQ2RkRBOjY5OTE1MjdFIiwidmlzaXRvcl9pZCI6IjYxNjUwNTE2NjUzOTExMTI4MzAiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 4f2a9e598d120ee17338c7160b56b439dd055ee7bf919f4f8bfca66244b3d149 |
| hovercard-subject-tag | pull_request:732525180 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,checks,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/prometheus/client_python/pull/694/checks |
| twitter:image | https://avatars.githubusercontent.com/u/741240?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/741240?s=400&v=4 |
| og:image:alt | This is a draft open to discussion In order to be able to clean-up the process files once it's stopped or destroyed, we need to identify those files with a UUID. If not, those files are going t... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 42c603b9d642c4a9065a51770f75e5e27132fef0e858607f5c9cb7e422831a7b |
| turbo-cache-control | no-preview |
| go-import | github.com/prometheus/client_python git https://github.com/prometheus/client_python.git |
| octolytics-dimension-user_id | 3380462 |
| octolytics-dimension-user_login | prometheus |
| octolytics-dimension-repository_id | 30618733 |
| octolytics-dimension-repository_nwo | prometheus/client_python |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 30618733 |
| octolytics-dimension-repository_network_root_nwo | prometheus/client_python |
| turbo-body-classes | logged-out env-production page-responsive full-width full-width-p-0 |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 848bc6032dcc93a9a7301dcc3f379a72ba13b96e |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width