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

job number should be on server #43

Open
rcoreilly opened this issue Jul 16, 2021 · 1 comment
Open

job number should be on server #43

rcoreilly opened this issue Jul 16, 2021 · 1 comment

Comments

@rcoreilly
Copy link
Member

the job number is stored locally on the client -- that is a problem! we don't quite have a good place to put it -- currently in ~/gruntdat/projs//nextjob.id could just be in top-level on repo under jobs/nextjob.id -- issue is potential conflict of client vs. server updating -- in general, we try to always have one direction do the updating, otherwise conflicts easily arise. Also it is not really viable to have 2 diff clients active at the same time because they could both submit the same job at the same time.

But, despite all that, it is ind of weird making everything depend on one machine. So maybe we have a specific command to pull the job number down from server, but otherwise the client is always doing the updating?

@rcoreilly
Copy link
Member Author

This is now mostly fixed: jobs/maxjob.id contains the maximum job number for the server, updated after every command, and then grunt ensures that its nextjob.id is always at least greater than that maxjob -- should keep in reasonably in sync without having contention over updating the same file on both sides..

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

No branches or pull requests

1 participant