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

Allow Chronos jobs to override taskInfo data #626

Merged
merged 1 commit into from
Mar 3, 2016

Conversation

jamesmulcahy
Copy link
Contributor

  • Add 'taskInfoData' field to Chronos jobs
  • If specified on a job, this data is placed in the
    data field of the taskInfo object, which is passed
    to the mesos executor.
  • If not specified, then default Chronos task data
    is used (as per previous behaviour).

@elingg
Copy link
Member

elingg commented Feb 27, 2016

LGTM

- Add 'taskInfoData' field to Chronos jobs
- If specified on a job, this data is placed in the
  data field of the taskInfo object, which is passed
  to the mesos executor.
- If not specified, then default Chronos task data
  is used (as per previous behaviour).
@elingg
Copy link
Member

elingg commented Mar 1, 2016

James and I discussed and went through reviews and this is looking good. @jamesmulcahy, can you please rebase?

Any objections or comments? If there are no objections, I will go ahead and merge.

elingg added a commit that referenced this pull request Mar 3, 2016
Allow Chronos jobs to override taskInfo data
@elingg elingg merged commit fc2567d into mesos:master Mar 3, 2016
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.

2 participants