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

create unix socket and qauld-ctl binary #543

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Harshil-Jani
Copy link
Member

Description

With reference to #541. Currently , This PR is about implementing a Unix Socket and creating a basic layout for qauld-ctl binary.

@github-actions
Copy link

github-actions bot commented Mar 26, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Harshil-Jani Harshil-Jani marked this pull request as draft March 26, 2023 20:48
@Harshil-Jani
Copy link
Member Author

I have read the CLA Document and I hereby sign the CLA

@Parinjoy
Copy link

Parinjoy commented Apr 1, 2023

This PR draft fulfills all the objectives of #541 ?

@Harshil-Jani
Copy link
Member Author

This PR draft fulfills all the objectives of #541 ?

Hey This PR is idea about how the objectives metioned can be approached. Feel free to try out and extend this ahead.

@MathJud
Copy link
Member

MathJud commented Apr 11, 2023

Regarding the path to the unix socket, there is a standard path to add the process specific files of a program. qauld should create the following folder on linux, unix & macos:

  • /var/run/qauld/qauld.sock the unix socket file
  • /var/run/qauld/qauld.pid a file containing the qauld process id, in order to shut the process down.

Further future extensions:

  • If somebody would like to run several deamons on the same machine from different users, we could extend the naming convention by adding a subfolder with the user number to the path. e.g. /var/run/qauld/1000/

@MathJud
Copy link
Member

MathJud commented Apr 11, 2023

Windows is now supporting unix sockets too. There is the question on how to make it work there.

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.

3 participants