Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

constraints are not activate with argument #561

Open
rapidoo opened this issue Sep 28, 2015 · 2 comments
Open

constraints are not activate with argument #561

rapidoo opened this issue Sep 28, 2015 · 2 comments
Labels
Milestone

Comments

@rapidoo
Copy link

rapidoo commented Sep 28, 2015

using Chronos on Ubuntu Ubuntu 12.04.2 LTS

chronos 2.3.4-1.0.183.ubuntu1404 amd64
mesos 0.23.0-1.0.ubuntu1204

When I use constraint without argument, mesos launch the job on the correct server.
But if I add << "arguments": [ "toto"], >> on the same job, constraint are not used, and the job is launch on any slaves.

{
"name": "fred",
"command": "/tmp/toto.sh",
<< "arguments": [ "toto"], >>
"shell": false,
"epsilon": "PT10M",
"executor": "",
"executorFlags": "",
"retries": 2,
"owner": "mail@machin.fr",
"ownerName": "",
"description": "",
"async": false,
"lastError": "",
"cpus": 0.1,
"disk": 256,
"mem": 128,
"disabled": false,
"softError": false,
"dataProcessingJobType": false,
"errorsSinceLastSuccess": 0,
"uris": [],
"environmentVariables": [],
"highPriority": false,
"runAsUser": "root",
"constraints": [
[
"role",
"EQUALS",
"alimentation"
]
],
"schedule": "R/2015-09-17T15:15:00.000Z/PT10S",
"scheduleTimeZone": "Europe/Paris"
}

@rapidoo
Copy link
Author

rapidoo commented Sep 28, 2015

I guess to problem is in JobUtils.scala

def getJobWithArguments(job: BaseJob, arguments: String): BaseJob = {

need to copy constraints too ...

rapidoo pushed a commit to rapidoo/chronos that referenced this issue Sep 28, 2015
@mwasn mwasn added this to the 2.5.0 milestone Sep 29, 2015
@mwasn mwasn added the bug label Sep 29, 2015
@kapil-malik
Copy link
Contributor

This is very important for us. Can anyone kindly confirm when 2.5.0 will be released. The fix at #562 works for us perfectly well, can someone please confirm by when it will atleast be merged in code base?

florianleibert added a commit that referenced this issue Nov 7, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants