[GitHub] [apisix-dashboard] bzp2010 commented on pull request #2054: fix: perform pid lookup for existing pid file
bzp2010 commented on pull request #2054: URL: https://github.com/apache/apisix-dashboard/pull/2054#issuecomment-981124303 ## Update In the previous PR we have removed the dependency on the built-in service management library and also removed the code for PID file generation. ## So I'll close this PR for now and you can open it again if you need to. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] bzp2010 commented on pull request #2054: fix: perform pid lookup for existing pid file
bzp2010 commented on pull request #2054: URL: https://github.com/apache/apisix-dashboard/pull/2054#issuecomment-923049655 > So we can only provide a service configuration? I don't understand `service configuration`, can you elaborate? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] bzp2010 commented on pull request #2054: fix: perform pid lookup for existing pid file
bzp2010 commented on pull request #2054:
URL: https://github.com/apache/apisix-dashboard/pull/2054#issuecomment-922466372
### Nice jobs!
But, with all due respect, the current management mode of service daemon and
the use of system calls during use will undermine the compatibility of the
operating system.
Therefore, I think our finally goal should be to remove the service daemon
management part from the Manager API and transfer it to the container or system
D, which is compatible with the container and RPM installation. Let
professional software do professional things.
What do you think of this?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
