-
Notifications
You must be signed in to change notification settings - Fork 4
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
Specify full path to worker for worker [run|deregister|list] #177
Comments
@srossross I will do this. That's a relatively quick change. |
@srossross are we waiting to merge #165 until this issue #177 ? That makes sense to me. |
@srossross there is already a check in place for existing worker
It doesn't look like there is any ambiguity. The changes in this issue would not be backwards compatible to our VM's documentation. Are you sure we should change? |
Recommendation: go thru next deployment smoothly and then make this interface change (which can cause instability. also needs to address backwards compatability |
@PeterDSteinberg Lets improve on #165. Can you add the ability to specify the exact worker:
e.g.
Running a worker without the full path should raise a deprication warning.
The text was updated successfully, but these errors were encountered: