Jenkins build is back to normal : Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME #1769

2013-11-21 Thread Apache Jenkins Server
See https://builds.apache.org/job/Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME/1769/changes

[jira] [Created] (MESOS-834) The Master should reject framework messages from a non-registered driver.

2013-11-21 Thread Benjamin Mahler (JIRA)
Benjamin Mahler created MESOS-834: - Summary: The Master should reject framework messages from a non-registered driver. Key: MESOS-834 URL: https://issues.apache.org/jira/browse/MESOS-834 Project:

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Niklas Nielsen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15745/#review29244 --- Awesome Brenden! Can we write tests that covers the new scenarios?

Re: Review Request 15741: Fixed example tests to use temporary work directory for slaves.

2013-11-21 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15741/ --- (Updated Nov. 21, 2013, 10:52 p.m.) Review request for mesos and Ben Mahler.

Re: Review Request 15741: Fixed example tests to use temporary work directory for slaves.

2013-11-21 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15741/#review29258 --- Ship it! Ship It! - Ben Mahler On Nov. 21, 2013, 10:52 p.m.,

[jira] [Commented] (MESOS-825) Querying Slave Status uses wrong host

2013-11-21 Thread Nikita Vetoshkin (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-825?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13829185#comment-13829185 ] Nikita Vetoshkin commented on MESOS-825: Is there a jira for such a clumsy error

[jira] [Updated] (MESOS-831) script-without-shebang

2013-11-21 Thread Timothy St. Clair (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-831?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Timothy St. Clair updated MESOS-831: Attachment: MESOS-831.patch Simple modification to add change. script-without-shebang

[jira] [Assigned] (MESOS-834) The Master should reject framework messages from a non-registered driver.

2013-11-21 Thread Vinod Kone (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-834?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vinod Kone reassigned MESOS-834: Assignee: Vinod Kone The Master should reject framework messages from a non-registered driver.

[jira] [Created] (MESOS-831) script-without-shebang

2013-11-21 Thread Timothy St. Clair (JIRA)
Timothy St. Clair created MESOS-831: --- Summary: script-without-shebang Key: MESOS-831 URL: https://issues.apache.org/jira/browse/MESOS-831 Project: Mesos Issue Type: Bug

Re: Review Request 15710: Fixed ZooKeeperMasterContenderDetectorTest to use LOOP_FOR to ensure tests are bounded.

2013-11-21 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15710/ --- (Updated Nov. 21, 2013, 11:18 p.m.) Review request for mesos, Benjamin

Review Request 15770: Improved IsolatorTest to use the LOOP_FOR macro.

2013-11-21 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15770/ --- Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone. Bugs:

[jira] [Commented] (MESOS-827) Create LOOP_FOR(duration) macro to guard testing loop from running indefinitely

2013-11-21 Thread Yan Xu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-827?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13829458#comment-13829458 ] Yan Xu commented on MESOS-827: -- https://reviews.apache.org/r/15729/

[jira] [Closed] (MESOS-818) Bump up the minimum number threads libprocess creates to accommodate new tests

2013-11-21 Thread Yan Xu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yan Xu closed MESOS-818. Committed. Bump up the minimum number threads libprocess creates to accommodate new tests

[jira] [Created] (MESOS-835) Reduce the minimum amount of CPUs required to make offers

2013-11-21 Thread Vinod Kone (JIRA)
Vinod Kone created MESOS-835: Summary: Reduce the minimum amount of CPUs required to make offers Key: MESOS-835 URL: https://issues.apache.org/jira/browse/MESOS-835 Project: Mesos Issue Type:

Re: Review Request 15729: Created LOOP_FOR(duration) macro to guard testing loop from running indefinitely.

2013-11-21 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15729/#review29263 --- 3rdparty/libprocess/3rdparty/stout/include/stout/gtest.hpp

Re: Review Request 15729: Created LOOP_FOR(duration) macro to guard testing loop from running indefinitely.

2013-11-21 Thread Jiang Yan Xu
On Nov. 21, 2013, 11:34 p.m., Jie Yu wrote: 3rdparty/libprocess/3rdparty/stout/include/stout/gtest.hpp, line 114 https://reviews.apache.org/r/15729/diff/1/?file=389061#file389061line114 What's the reason to concat the line no. here? Will that be useful some where? It is to

Re: Review Request 15542: Offer an execvp like interface for running tasks.

2013-11-21 Thread Jason Dusek
On Nov. 18, 2013, 8:01 p.m., Vinod Kone wrote: src/launcher/launcher.cpp, lines 188-203 https://reviews.apache.org/r/15542/diff/3/?file=387245#file387245line188 Ditto. Is this async signal safe? This function is called only in src/launcher/main.cpp, where there is no call to

Re: Review Request 15542: Offer an execvp like interface for running tasks.

2013-11-21 Thread Jason Dusek
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15542/ --- (Updated Nov. 21, 2013, 6:51 p.m.) Review request for mesos, Benjamin Hindman,

[jira] [Created] (MESOS-832) Scheduler Driver should reject messages from a non-leading master.

2013-11-21 Thread Benjamin Mahler (JIRA)
Benjamin Mahler created MESOS-832: - Summary: Scheduler Driver should reject messages from a non-leading master. Key: MESOS-832 URL: https://issues.apache.org/jira/browse/MESOS-832 Project: Mesos

[jira] [Resolved] (MESOS-463) Detector ZNode creation failure.

2013-11-21 Thread Yan Xu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-463?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yan Xu resolved MESOS-463. -- Resolution: Fixed Detector ZNode creation failure. Key:

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Brenden Matthews
On Nov. 21, 2013, 7:59 p.m., Niklas Nielsen wrote: src/master/master.cpp, line 1597 https://reviews.apache.org/r/15745/diff/2/?file=389363#file389363line1597 What happens if status.task_id() does not exist in tasks? Is it silently ignored or does it fail? In this case I'm

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Brenden Matthews
On Nov. 21, 2013, 10:03 p.m., Ben Mahler wrote: Slaves continue to retry status updates until an acknowledgement is received (the scheduler driver sends an acknowledgement once Scheduler::statusUpdate was invoked). So in Case 1 you've described, if a task transitions while a

Re: Review Request 15706: Fixed Group to retry when authentication failed due to retryable errors.

2013-11-21 Thread Jiang Yan Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15706/ --- (Updated Nov. 22, 2013, 12:30 a.m.) Review request for mesos, Benjamin

Re: Review Request 15706: Fixed Group to retry when authentication failed due to retryable errors.

2013-11-21 Thread Jiang Yan Xu
On Nov. 20, 2013, 11:23 p.m., Ben Mahler wrote: Looks good, I was looking at how retry() can be initiated from several locations in the code but it appears relatively harmless: we will just do redundant sync() calls on occasion AFAICT. Each call to retry() is guarded by a if

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15745/ --- (Updated Nov. 22, 2013, 12:30 a.m.) Review request for mesos and Niklas

[jira] [Created] (MESOS-833) The Status Update Manager should use a back-off mechanism for retried updates.

2013-11-21 Thread Benjamin Mahler (JIRA)
Benjamin Mahler created MESOS-833: - Summary: The Status Update Manager should use a back-off mechanism for retried updates. Key: MESOS-833 URL: https://issues.apache.org/jira/browse/MESOS-833

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15745/#review29249 --- Slaves continue to retry status updates until an acknowledgement is

[jira] [Created] (MESOS-837) AWAIT_READY should not depend on process::Clock

2013-11-21 Thread Yan Xu (JIRA)
Yan Xu created MESOS-837: Summary: AWAIT_READY should not depend on process::Clock Key: MESOS-837 URL: https://issues.apache.org/jira/browse/MESOS-837 Project: Mesos Issue Type: Bug

Review Request 15773: Fixed master to ignore messages from unregistered framework.

2013-11-21 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15773/ --- Review request for mesos, Benjamin Hindman and Ben Mahler. Bugs: MESOS-834

[jira] [Commented] (MESOS-834) The Master should reject framework messages from a non-registered driver.

2013-11-21 Thread Vinod Kone (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-834?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13829567#comment-13829567 ] Vinod Kone commented on MESOS-834: -- https://reviews.apache.org/r/15773/ The Master

Review Request 15774: Reduced the minimum number of cpus per offer to 0.1.

2013-11-21 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15774/ --- Review request for mesos, Benjamin Hindman, Ben Mahler, and Brian Wickman.

[jira] [Updated] (MESOS-835) Reduce the minimum amount of CPUs required to make offers

2013-11-21 Thread Vinod Kone (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-835?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vinod Kone updated MESOS-835: - Description: Currently we only make offers if a slave has at least 1 cpus. This is likely very high for

Re: Review Request 15706: Fixed Group to retry when authentication failed due to retryable errors.

2013-11-21 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15706/#review29274 --- src/zookeeper/group.cpp

Re: Review Request 15710: Fixed ZooKeeperMasterContenderDetectorTest to use LOOP_FOR to ensure tests are bounded.

2013-11-21 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15710/#review29275 --- src/tests/master_contender_detector_tests.cpp

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Brenden Matthews
On Nov. 21, 2013, 7:59 p.m., Niklas Nielsen wrote: src/master/master.cpp, line 1597 https://reviews.apache.org/r/15745/diff/2/?file=389363#file389363line1597 What happens if status.task_id() does not exist in tasks? Is it silently ignored or does it fail? Brenden Matthews

[jira] [Commented] (MESOS-814) Retry retryable authentication failures

2013-11-21 Thread Yan Xu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-814?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13829216#comment-13829216 ] Yan Xu commented on MESOS-814: -- This ticket further improves the robustness over the fix for

[jira] [Created] (MESOS-838) Consider exporting queue size as a metric from the master

2013-11-21 Thread Chris Lambert (JIRA)
Chris Lambert created MESOS-838: --- Summary: Consider exporting queue size as a metric from the master Key: MESOS-838 URL: https://issues.apache.org/jira/browse/MESOS-838 Project: Mesos Issue

[jira] [Commented] (MESOS-838) Consider exporting queue size as a metric from the master

2013-11-21 Thread Jie Yu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13829688#comment-13829688 ] Jie Yu commented on MESOS-838: -- We thought about this before. For example, we considered

[jira] [Comment Edited] (MESOS-838) Consider exporting queue size as a metric from the master

2013-11-21 Thread Jie Yu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13829688#comment-13829688 ] Jie Yu edited comment on MESOS-838 at 11/22/13 6:09 AM: We thought

Re: Review Request 15319: Propagate discarding in process::dispatch.

2013-11-21 Thread Jie Yu
On Nov. 8, 2013, 12:39 a.m., Benjamin Hindman wrote: 3rdparty/libprocess/include/process/dispatch.hpp, lines 92-93 https://reviews.apache.org/r/15319/diff/1/?file=380494#file380494line92 Why can't we capture this in Promise::associate instead? I think we want these semantics even

[jira] [Commented] (MESOS-838) Consider exporting queue size as a metric from the master

2013-11-21 Thread Chris Lambert (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13829716#comment-13829716 ] Chris Lambert commented on MESOS-838: - Ah ok. Did you decide it was a bad idea? Or is

[jira] [Commented] (MESOS-838) Consider exporting queue size as a metric from the master

2013-11-21 Thread Jie Yu (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-838?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13829727#comment-13829727 ] Jie Yu commented on MESOS-838: -- [~clambert] Not at all. Just thought replicated log fix is