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.

Sign up for GitHub

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

Jump to bottom

Unable to update job.workers #533

Closed
erik-stephens opened this issue Sep 8, 2017 · 1 comment
Closed

Unable to update job.workers #533

erik-stephens opened this issue Sep 8, 2017 · 1 comment
Labels
bug priority:high

Comments

@erik-stephens
Copy link
Contributor 

$ grep workers some-job.json
    "workers": 2,
    "recycle_workers": 1000,

$ http PUT gen1:5678/jobs/12977f33-7688-4d3e-bbb8-58e7db3fc9d4 < some-job.json

$ http gen1:5678/jobs/12977f33-7688-4d3e-bbb8-58e7db3fc9d4 | jq '._updated, .workers'
"2017-09-08T23:33:24.962Z"
1
@godber
Copy link
Member

godber commented Sep 9, 2017

I'd think an integration test could be written to test this case.

This was referenced Sep 13, 2017
Merged
Merged
@godber godber closed this as completed in 5909e1c Sep 13, 2017
godber added a commit that referenced this issue Sep 13, 2017
@godber
Merge pull request #537 from jsnoble/api_jobs_put
c599af0 
fixed PUT /jobs/:job_id endpoint resolves #533
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug priority:high
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@godber @kstaken @erik-stephens