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

Teraslice should validate jobs on start as well #616

Closed
godber opened this issue Nov 15, 2017 · 1 comment
Closed

Teraslice should validate jobs on start as well #616

godber opened this issue Nov 15, 2017 · 1 comment
Labels
bug

Comments

@godber
Copy link
Member

godber commented Nov 15, 2017

When upgrading an existing teraslice cluster it is possible to hit a scenario where code changes to operations add new parameters. Starting stored jobs will fail because validation doesn't happen on start. Validation should happen on start and the defaults and other validation steps will be handled.
@godber godber added the bug label Nov 15, 2017
@godber
Copy link
Member Author

godber commented Nov 16, 2017

Integration tests pass

screen shot 2017-11-16 at 2 50 33 pm

@godber godber closed this as completed in 346944b Nov 20, 2017
godber added a commit that referenced this issue Nov 20, 2017
@godber
Merge pull request #618 from jsnoble/startJob_validation_fix
ebd7563 
added validation on start job as well resolves #616
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@godber