mesos git commit: Added Benjamin Bannier to the list of committers.

2017-09-14 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 38cb694f5 -> e42d626ab Added Benjamin Bannier to the list of committers. Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/e42d626a Tree:

[2/3] mesos git commit: Added 'id' and 'metadata' fields to 'Resource.DiskInfo.Source'.

2017-09-21 Thread bbannier
Added 'id' and 'metadata' fields to 'Resource.DiskInfo.Source'. IDs will allow to create distinguishable resources, e.g., of RAW or BLOCK type. We also add a metadata field which can be used to expose additional disk information. Review: https://reviews.apache.org/r/58048/ Project:

[3/3] mesos git commit: Used a namespace alias for 'process::http'.

2017-09-21 Thread bbannier
Used a namespace alias for 'process::http'. This patch introduces 'http' as an alias for 'process::http'. Review: https://reviews.apache.org/r/62439/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/8c453362 Tree:

[1/3] mesos git commit: Fixed 'operator==' for 'Resource::DiskInfo::Source'.

2017-09-21 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master c0293a6f7 -> 8c4533623 Fixed 'operator==' for 'Resource::DiskInfo::Source'. When implementing 'operator==' for protobufs as a pattern we typically first check that two 'optional' fields are set for both the left- and right-hand side, and

mesos git commit: Added a `[-s|--skip-hooks]` option when applying reviews.

2017-09-18 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 95406868a -> bcb88aba5 Added a `[-s|--skip-hooks]` option when applying reviews. Review: https://reviews.apache.org/r/61588/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[3/5] mesos git commit: Made 'UpdateSlaveMessage' a union of possible updates.

2017-10-05 Thread bbannier
Made 'UpdateSlaveMessage' a union of possible updates. The 'UpdateSlaveMessage' could either transport an update to an agent's total, or to its oversubscribed resources. In certain scenarios this required the agent to send two messages where before one was sufficient. When talking to a master

[4/5] mesos git commit: Implemented a registrar for resource provider manager state.

2017-10-05 Thread bbannier
ned createStorage(const std::string& path); +}; + + +Owned AgentRegistrarProcess::createStorage(const std::string& path) +{ + // The registrar uses LevelDB as underlying storage. Since LevelDB + // is currently not supported on Windows (see MESOS-5932), we fall + // back to in-memory stora

[5/5] mesos git commit: Rescinded offers possibly affected by updates to agent total resources.

2017-10-05 Thread bbannier
t; *slave; - allocator->recoverResources( - offer->framework_id(), offer->slave_id(), offered, None()); + rescind = true; +} + +// For updates to the agent's total resources all offers are rescinded. +// +// TODO(bbannier): Only rescind offers possibly

[1/5] mesos git commit: Added a master-registry backed resource provider manager registry.

2017-10-05 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 38af943ad -> 46db7e4f2 Added a master-registry backed resource provider manager registry. This patch adds an implementation of the resource provider registrar backed by the master's registrar. With that it becomes possible to persist

[2/5] mesos git commit: Triggered 'UpdateSlaveMessage' when 'ResourceProviderManager' updates.

2017-10-05 Thread bbannier
ff6dab..ff0bc42 100644 --- a/src/tests/slave_tests.cpp +++ b/src/tests/slave_tests.cpp @@ -31,6 +31,8 @@ #include +#include + #include #include #include @@ -41,6 +43,8 @@ #include #include +#include + #include #include #include @@ -8491,6 +8495,121 @@ TEST_P(DefaultContainerDNSFl

[1/2] mesos git commit: Removed unneeded configure step in mesos-tidy Docker image.

2017-10-11 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master de7beab4f -> 090830314 Removed unneeded configure step in mesos-tidy Docker image. The mesos-tidy Docker image uses the cmake build to prepare Mesos for linting. The cmake build does not require running of 'bootstrap'. Review:

mesos git commit: Added documentation to CSI-related source proto fields.

2017-10-13 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master bea361880 -> 7014fbfaa Added documentation to CSI-related source proto fields. This commit adds clarifying documentation to CSI-related proto fields in `Resource.DiskInfo.Source` which were introduced earlier. Review:

[2/2] mesos git commit: Made explicit that we dispatch to a process manager.

2017-09-27 Thread bbannier
Made explicit that we dispatch to a process manager. This member function of 'ProcessManager' was capturing a 'this' pointer when dispatching to itself, but did not properly use 'defer' or 'dispatch'. While this pattern is usually suspect, it was safe here as we can be sure that the process

[1/2] mesos git commit: Fixed usage of 'ATOMIC_FLAG_INIT'.

2017-09-27 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 53b56117b -> 7b0852f79 Fixed usage of 'ATOMIC_FLAG_INIT'. The C++ standard guarantees that the macro 'ATOMIC_FLAG_INIT' is usable for initialization of atomic variables; its use in other context is unspecified. libcxx defines

mesos git commit: Made explicit that a 'switch' covers all branches.

2017-11-27 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 3c9aa7ab5 -> d007c590e Made explicit that a 'switch' covers all branches. Some compilers fail to recognize that the list of cases in this 'switch' statement is exhaustive. Added an explicit 'UNREACHABLE()' to document to the compiler that

[3/4] mesos git commit: Renamed resource provider message UpdateTotalResources to UpdateState.

2017-11-27 Thread bbannier
Renamed resource provider message UpdateTotalResources to UpdateState. This message now captures much more than just resource updates, but instead informs users about the full resource provider state. This change mirrors the message naming used between resource provider manager and resource

[4/4] mesos git commit: Passed operations from resource provider to agent.

2017-11-27 Thread bbannier
Passed operations from resource provider to agent. See summary. Review: https://reviews.apache.org/r/63730/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/3c9aa7ab Tree:

[2/4] mesos git commit: Added a helper to extract consumed resources from offer operations.

2017-11-27 Thread bbannier
} +case Offer::Operation::LAUNCH: +case Offer::Operation::LAUNCH_GROUP: + // TODO(bbannier): Consider adding support for 'LAUNCH' and + // 'LAUNCH_GROUP' operations. +case Offer::Operation::UNKNOWN: + return Error("Unsupported operation"); + } +} + na

[1/4] mesos git commit: Made sure all true allocator agent updates trigger allocations.

2017-11-27 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 3b318dc94 -> 3c9aa7ab5 Made sure all true allocator agent updates trigger allocations. The allocator method 'updateSlave' can be used to modify either an agent's capabilities, its total resources, or both. It is expected that this method

mesos git commit: Added env var to set default flags for parallel test runner.

2017-11-27 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master bf507a208 -> dfd1ee14a Added env var to set default flags for parallel test runner. This patch modifies the parallel test runner to examine the environment variable 'MESOS_GTEST_RUNNER_FLAGS' for a default set of flags to pass. Flags given

[2/2] mesos git commit: Removed stringification for hashmaps of strings.

2017-11-29 Thread bbannier
Removed stringification for hashmaps of strings. More generic hashmap stringification functions for hashmaps of stringifiable types were added to stout in a previous commit. This patch removes more specialized implementations in favor of the more generic one provided by stout itself now. Review:

[1/8] mesos git commit: Made sure only non-terminal operations use resources.

2017-11-30 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master f7858bb3a -> c035e7e87 Made sure only non-terminal operations use resources. The function 'addOfferOperation' is a high-level function which can be used to some extent manage the offer operation life cycle in the master. In addition to

[5/8] mesos git commit: Added a new allocator method to add resources to agents.

2017-11-30 Thread bbannier
Added a new allocator method to add resources to agents. The added method complements 'Allocator::addSlave'. While in 'addSlave' the total agent resources and used resources are passed, the method 'addResourceProvider' added here allows to add additional, potentially used resources to an existing

[6/8] mesos git commit: Introduced an allocator helper function to track used resources.

2017-11-30 Thread bbannier
Introduced an allocator helper function to track used resources. This patch introduces a helper to track allocated resources. It encapsulates all needed updates to the various sorters for reusability. Review: https://reviews.apache.org/r/64136/ Project:

[7/8] mesos git commit: Implemented a test of offer operation reconcilation.

2017-11-30 Thread bbannier
Implemented a test of offer operation reconcilation. Whenever a speculated operation fails in a resource provider, we expect the agent to trigger a 'UpdateSlaveMessage' to the master so it can rollback its agent state. This patch adds such a test. Review: https://reviews.apache.org/r/63843/

[3/8] mesos git commit: Allowed removing non-terminal offer operations.

2017-11-30 Thread bbannier
Allowed removing non-terminal offer operations. During reconcilation we might be required to remove non-terminal offer operations from bookkeeping. Review: https://reviews.apache.org/r/63842/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[2/8] mesos git commit: Reconciled offer operations between agent and master.

2017-11-30 Thread bbannier
ECK_SOME(provider.oldTotal); +CHECK_SOME(provider.newTotal); + +Resources oldNonRevocable = + provider.oldTotal->nonRevocable().createStrippedScalarQuantity(); +Resources newNonRevocable = + provider.newTotal->nonRevocable().createStrippedScalarQuantity(); +CHECK_EQ(

[8/8] mesos git commit: Removed currently unneeded 'AWAIT_READY's in 'MockResourceProvider'.

2017-11-30 Thread bbannier
Removed currently unneeded 'AWAIT_READY's in 'MockResourceProvider'. The 'AWAIT_READY's in 'MockResourceProvider' let to the resource provider not being usable with paused clock as we would run into a deadclock in that case. This patch removes the explicit awaits here to make this resource

[4/8] mesos git commit: Reconciled pending resource provider operations in agent.

2017-11-30 Thread bbannier
std::set_difference( +knownUuids.begin(), +knownUuids.end(), +receivedUuids.begin(), +receivedUuids.end(), +std::inserter( +disappearedOperations, disappearedOperations.begin())); + +foreach (const UUID& uuid, disappearedOperations)

mesos git commit: Fixed style violations regarding whitespace around operators.

2017-12-14 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master f6b33da25 -> bf4ac91bd Fixed style violations regarding whitespace around operators. Review: https://reviews.apache.org/r/64603/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

mesos git commit: Fixed a flaky test.

2017-12-14 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master bf4ac91bd -> 7c6b7d207 Fixed a flaky test. Callbacks of already invalidated instances could be called in 'ResubscribeResourceProvider' when simulating a resource provider restart. This has been resolved by not only restarting a resource

[4/6] mesos git commit: Mesos: Used a namespace for UUID.

2017-12-14 Thread bbannier
( knownUuids.begin(), knownUuids.end(), @@ -7163,7 +7164,7 @@ void Slave::handleResourceProviderMessage( std::inserter( disappearedOperations, disappearedOperations.begin())); -foreach (const UUID& uuid, disappearedOperations) { +

[2/6] mesos git commit: Mesos: Used a namespace for UUID.

2017-12-14 Thread bbannier
http://git-wip-us.apache.org/repos/asf/mesos/blob/7467c810/src/tests/slave_tests.cpp -- diff --git a/src/tests/slave_tests.cpp b/src/tests/slave_tests.cpp index 451ad21..122f42d 100644 --- a/src/tests/slave_tests.cpp +++

[5/6] mesos git commit: Mesos: Used a namespace for UUID.

2017-12-14 Thread bbannier
Mesos: Used a namespace for UUID. To avoid conflicts when introducing a UUID type in Mesos' namespace, Stout's UUID type is put in the 'id' namespace. Review: https://reviews.apache.org/r/64381/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[6/6] mesos git commit: Introduced a 'UUID' type.

2017-12-14 Thread bbannier
Introduced a 'UUID' type. Review: https://reviews.apache.org/r/64168/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/605b238d Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/605b238d Diff:

[1/6] mesos git commit: Stout: Used a namespace for UUID.

2017-12-14 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master ae9d1033b -> 605b238d6 Stout: Used a namespace for UUID. To avoid conflicts when introducing a UUID type in Mesos' namespace, Stout's UUID type is put in the 'id' namespace. Review: https://reviews.apache.org/r/64380/ Project:

[3/6] mesos git commit: Mesos: Used a namespace for UUID.

2017-12-14 Thread bbannier
http://git-wip-us.apache.org/repos/asf/mesos/blob/7467c810/src/tests/containerizer/nested_mesos_containerizer_tests.cpp -- diff --git a/src/tests/containerizer/nested_mesos_containerizer_tests.cpp

mesos git commit: Removed unused import in proto file.

2017-12-15 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 784b7f599 -> f036eb611 Removed unused import in proto file. Importing 'mesos/resource_provider/resource_provider.proto' became unnessecary after the cleanups in 'eeb09cbc1'. Review: https://reviews.apache.org/r/64599/ Project:

[2/2] mesos git commit: Made resource provider driver start explicit.

2017-12-15 Thread bbannier
Made resource provider driver start explicit. The driver for HTTP connections of resource providers is implemented as an actor which takes callbacks. It previously started listening for and handling of events on creation. We typically store drivers as member of resource providers. This setup is

mesos git commit: Fixed handling of resource provider updates without resources.

2017-12-15 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master aaef8563f -> 699a5d226 Fixed handling of resource provider updates without resources. On master failover the master might learn about a resource provider without resources. This patch relaxes an assertion in the master permitting such

[8/9] mesos git commit: Added comparison operators for 'ResourceVersionUUID'.

2017-11-08 Thread bbannier
Added comparison operators for 'ResourceVersionUUID'. See summary. Review: https://reviews.apache.org/r/63495/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/06856eca Tree:

[6/9] mesos git commit: Allowed toggling of agent capabilities via command line flags.

2017-11-08 Thread bbannier
Allowed toggling of agent capabilities via command line flags. This patch introduces a new agent command line flags '--agent_features' which can be used to whitelist capabilities to enable. This flag is intended to enable introducing feature flags in the future so experimental features can be

[9/9] mesos git commit: Transmitted agent resource versions in (re)registration.

2017-11-08 Thread bbannier
Transmitted agent resource versions in (re)registration. This commit introduces resource version fields into agent registration and reregistration message. The agent is changed to set these fields when needed; the master in turn stores the versions for use in to be added speculated offer

[7/9] mesos git commit: Synchronized agent resource versions via 'UpdateSlaveMessage'.

2017-11-08 Thread bbannier
+ // We do not update the agent's resource version since + // oversubscribed resources cannot be used for any operations + // but launches. Since oversubscription is run at regular + // intervals updating the version could cause a lot of off

[1/9] mesos git commit: Added tests for agent resource version transmission.

2017-11-08 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 439825f9e -> 72e6cde81 Added tests for agent resource version transmission. This patch introduces separate tests for clock values communicated from resource providers and from agents to masters. Review: https://reviews.apache.org/r/63496/

[5/9] mesos git commit: Added resource version to resource provider UpdateTotalResources call.

2017-11-08 Thread bbannier
Added resource version to resource provider UpdateTotalResources call. This patch surfaces this information to resource provider manager users like the agent. In a later patch we will modify the agent to forward this information to the master. Review: https://reviews.apache.org/r/63491/

[3/9] mesos git commit: Triggered 'UpdateSlaveMessage' when 'ResourceProviderManager' updates.

2017-11-08 Thread bbannier
diff --git a/src/tests/slave_tests.cpp b/src/tests/slave_tests.cpp index f9c2e6b..9928cfc 100644 --- a/src/tests/slave_tests.cpp +++ b/src/tests/slave_tests.cpp @@ -31,6 +31,8 @@ #include +#include + #include #include #include @@ -41,6 +4

[2/9] mesos git commit: Removed unused declaration.

2017-11-08 Thread bbannier
Removed unused declaration. The declaration of this operator was not needed here. It was also done in a surprising namespace as the 'Task' equality operator lives in the 'mesos' namespace while this declaration was in 'mesos::internal'. Review: https://reviews.apache.org/r/63494/ Project:

[4/9] mesos git commit: Added flag protobuf message for agent capabilities.

2017-11-08 Thread bbannier
Added flag protobuf message for agent capabilities. See summary. Review: https://reviews.apache.org/r/63540/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/ca495620 Tree:

mesos git commit: Fixed build dependency for `protobuf_tests.proto`.

2017-12-07 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 54e03f3ce -> 49ca9c7d8 Fixed build dependency for `protobuf_tests.proto`. The protobuf hpp and cpp files for `protobuf_tests.proto` should be built after `BUNDLED_DEPS`, not within it. Review: https://reviews.apache.org/r/62777/

mesos git commit: Added a missing include.

2017-12-08 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master d142d38e3 -> f8771c7f5 Added a missing include. A test case using SSL flags if compiling with SSL-support was added in d142d38, but we did not include the required header. Review: https://reviews.apache.org/r/64448 Project:

[2/4] mesos git commit: Added ResourceVersion to RunTask and RunTaskGroup messages.

2017-12-08 Thread bbannier
Added ResourceVersion to RunTask and RunTaskGroup messages. This commit adds the known agent resource versions to RunTaskMessage and RunTaskGroupMessage. We also update sites where the message is unpacked. In a later commit we will inject versions in the master and evaluate them in the agent.

[1/4] mesos git commit: Passed versions when launching tasks.

2017-12-08 Thread bbannier
ld be set but not both"; @@ -2306,8 +2310,84 @@ void Slave::__run( return; } - LOG(INFO) << "Launching " << taskOrTaskGroup(task, taskGroup) -<< " for framework " << frameworkId; + // Check task invariants. + // + // TODO(bbannier): I

[3/4] mesos git commit: Provided resource provider infos in 'UpdateState' message.

2017-12-08 Thread bbannier
Provided resource provider infos in 'UpdateState' message. To support sending 'ResourceProviderInfo's of all known resource providers to the master as part of 'UpdateSlaveMessage', these information has been added to the 'UpdateState' message that is sent from resource provider to agents. Also,

[3/5] mesos git commit: Explicitly passed resource-provider information in 'UpdateSlaveMessage'.

2017-12-08 Thread bbannier
ferent from the previous // estimate. We also send this whenever we get (re-)registered // (i.e. whenever we transition into the RUNNING state). -if (state == RUNNING && oversubscribedResources != oversubscribed) { +if (state == RUNNING && previousO

[5/5] mesos git commit: Only passed agent's resource version in top-level 'UpdateSlaveMessage'.

2017-12-08 Thread bbannier
Only passed agent's resource version in top-level 'UpdateSlaveMessage'. Since we have moved all resource provider-related information like e.g., resource versions to an explicit field in 'UpdateSlaveMessage', we now only pass agent information in top-level fields. To model that we changed the

[4/5] mesos git commit: Removed 'total' from 'UpdateSlaveMessage'.

2017-12-08 Thread bbannier
s.nonRevocable().filter(agentResources) + +newOversubscribed.getOrElse( +slave->totalResources.revocable().filter(agentResources)) + +newResourceProviderResources; - bool updated = slave->totalResources != newSlaveResources; + // TODO(bbannier): We only need to

[2/5] mesos git commit: Fixed 'getResourceProviderId' for operations without resources.

2017-12-08 Thread bbannier
Fixed 'getResourceProviderId' for operations without resources. The code in 'getResourceProviderId' was written in a way assuming that any operation containing a list resources always contained at least a single resource. This is both incorrect since such operations pass validation, and also

[2/2] mesos git commit: Allowed resubscription of resource providers.

2017-12-01 Thread bbannier
Allowed resubscription of resource providers. A resource provider can resubscribe by including the resource provider ID it got assigned as part of its 'ResourceProviderInfo' in a 'SUBSCRIBE' call. A resubscription is necessary if either the resource provider or the resource provider manager (i.e.

[1/2] mesos git commit: Used helper functions instead of switches for resource extraction.

2017-12-01 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master ac068c130 -> 9ab950d38 Used helper functions instead of switches for resource extraction. Depeding on the type of an offer operation, different resource need to be extracted from an operation. Instead of using a switch, the helper

mesos git commit: Fixed Protobuf header dependency.

2017-10-31 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master fbe7e6447 -> f9b7a76c9 Fixed Protobuf header dependency. While the header files were being created correctly, the explicit dependency in the `CUSTOM_COMMAND` did not exist. Though we depended on `${H}`, the variable was empty as we forgot

[2/3] mesos git commit: Added a realm for resource provider authentication.

2018-05-09 Thread bbannier
Added a realm for resource provider authentication. Review: https://reviews.apache.org/r/66933/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/abc76653 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/abc76653 Diff:

[3/3] mesos git commit: Added token-based authentication to resource provider tests.

2018-05-09 Thread bbannier
Added token-based authentication to resource provider tests. If compiled with SSL support, resource provider tests will use authentication. The 'MockResourceProvider' has been updated to create a JWT when started. Review: https://reviews.apache.org/r/66934/ Project:

[1/3] mesos git commit: Added token-based authentication for resource providers.

2018-05-09 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 86523d315 -> 0f0909c89 Added token-based authentication for resource providers. If a token is provided, it will be used in HTTP requests to the resource provider manager. This allows JWT-based authentication and authorization for resource

[2/3] mesos git commit: Added missing test expectation.

2018-05-04 Thread bbannier
Added missing test expectation. On master failover the scheduler will get disconnected. Add a test expectation for that. This silences a gmock warning. Review: https://reviews.apache.org/r/66849/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[3/3] mesos git commit: Correctly reconciled dropped operation after agent failover.

2018-05-04 Thread bbannier
Correctly reconciled dropped operation after agent failover. When the master receives an `UpdateSlaveMessage` after agent failover it previously did not correctly detect dropped operations (operations known to the master, but unknown to the agent) and did not trigger reconciliation for such

[1/3] mesos git commit: Changed failure check for a call to os::system().

2018-05-04 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 520b72985 -> 351bade6c Changed failure check for a call to os::system(). The previous check for `None()` was missing the case where the program exited with a non-zero exit status. Review: https://reviews.apache.org/r/66776/ Project:

mesos git commit: Updated cpplint from upstream.

2018-05-15 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master e88fe8c31 -> ce7bb6e23 Updated cpplint from upstream. This patch updates the bundled cpplint to upstream '43d512b'. Review: https://reviews.apache.org/r/67127/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

mesos git commit: Updated cpplint to be compatible with Python 3.

2018-05-18 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master a981067d2 -> 89db66e3d Updated cpplint to be compatible with Python 3. This patch keeps the bundled cpplint at upstream `43d512b`. Cpplint now works with Python 2 and 3 by adding part of the content of

mesos git commit: Supported custom error types for `Try` in `AssertSome`.

2018-05-18 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 89db66e3d -> c020a130a Supported custom error types for `Try` in `AssertSome`. Review: https://reviews.apache.org/r/67164/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

mesos git commit: Reformated libprocess example code.

2018-05-24 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 5248cb1c5 -> cb8a30bbd Reformated libprocess example code. Review: https://reviews.apache.org/r/67281/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/cb8a30bb Tree:

[1/3] mesos git commit: Updated cpplint from upstream.

2018-05-15 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 44856413f -> e88fe8c31 http://git-wip-us.apache.org/repos/asf/mesos/blob/7454420c/support/cpplint.py -- diff --git a/support/cpplint.py b/support/cpplint.py index

[2/3] mesos git commit: Updated cpplint from upstream.

2018-05-15 Thread bbannier
Updated cpplint from upstream. This patch updates the bundled cpplint to upstream 'e8ffd7ce6'. We also update the patch file to reflect changes; we include the changes made in '7b1ab032' which where not included in the patch. Review: https://reviews.apache.org/r/64610/ Project:

[3/3] mesos git commit: Added .tox to files excluded by Python linter.

2018-05-15 Thread bbannier
Added .tox to files excluded by Python linter. Review: https://reviews.apache.org/r/67054/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/e88fe8c3 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/e88fe8c3 Diff:

[2/2] mesos git commit: Moved some constant definitions into header file.

2018-06-11 Thread bbannier
Moved some constant definitions into header file. Since we are using C++11 and the declared constants are either POD or `constexpr` types, we can define them in the header as `constexpr`. This also removes the need to declare the constants as `extern` which should make using their headers easier

[1/2] mesos git commit: Made some headers standalone.

2018-06-11 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master e0a77d826 -> b1e88f73b Made some headers standalone. This patch adds includes to two header files in stout so that the headers are standalone. Review: https://reviews.apache.org/r/67521/ Project:

[1/3] mesos git commit: Used `*_SOME` macro for checking `Try` values in libprocess.

2018-05-31 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 0f6ce843b -> 228193754 Used `*_SOME` macro for checking `Try` values in libprocess. Review: https://reviews.apache.org/r/67211/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[3/3] mesos git commit: Used `*_SOME` macro for checking `Try` values.

2018-05-31 Thread bbannier
Used `*_SOME` macro for checking `Try` values. Review: https://reviews.apache.org/r/67212/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/22819375 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/22819375 Diff:

[2/3] mesos git commit: Used `*_SOME` macro for checking `Try` values in stout.

2018-05-31 Thread bbannier
Used `*_SOME` macro for checking `Try` values in stout. Review: https://reviews.apache.org/r/67210/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/4760a777 Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/4760a777

mesos git commit: Changed python3 support check script to print to `stderr`.

2018-05-29 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master a83a25203 -> 30e7429c8 Changed python3 support check script to print to `stderr`. The scripts this might be used in might use `stdout` to report results we do not intend to interfere with, e.g., `support/apply-reviews.py` can print

[2/2] mesos git commit: Fixed compilation issues in libprocess example.

2018-05-30 Thread bbannier
Fixed compilation issues in libprocess example. * Added the mandatory help string argument to calls to 'route()', failing compilation. * The response returned from the '/vars' endpoint had its 'type' field set to 'NONE', causing indefinite hanging when trying to access the endpoint.

[1/2] mesos git commit: Added libprocess example code to build.

2018-05-30 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 04ef101b8 -> dd25f Added libprocess example code to build. Review: https://reviews.apache.org/r/67280/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/dd25

[1/3] mesos git commit: Updated python.m4 to be compatible with Python 3.

2018-05-30 Thread bbannier
Repository: mesos Updated Branches: refs/heads/1.4.x f4c1cdc74 -> c02eccc8f refs/heads/1.5.x 21d8a467e -> f505eacee refs/heads/1.6.x 067f013d7 -> 65a19f50e Updated python.m4 to be compatible with Python 3. This updates `python.m4` to upstream `596e9e1` (commit in automake repo). The Mesos

[3/3] mesos git commit: Updated python.m4 to be compatible with Python 3.

2018-05-30 Thread bbannier
Updated python.m4 to be compatible with Python 3. This updates `python.m4` to upstream `596e9e1` (commit in automake repo). The Mesos patch has been removed as upstream now can e.g., detect python2.6. Review: https://reviews.apache.org/r/67318/ Project:

[2/3] mesos git commit: Updated python.m4 to be compatible with Python 3.

2018-05-30 Thread bbannier
Updated python.m4 to be compatible with Python 3. This updates `python.m4` to upstream `596e9e1` (commit in automake repo). The Mesos patch has been removed as upstream now can e.g., detect python2.6. Review: https://reviews.apache.org/r/67318/ Project:

mesos git commit: Updated python.m4 to be compatible with Python 3.

2018-05-30 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master dd25f -> e287647d7 Updated python.m4 to be compatible with Python 3. This updates `python.m4` to upstream `596e9e1` (commit in automake repo). The Mesos patch has been removed as upstream now can e.g., detect python2.6. Review:

mesos git commit: Removed unused libprocess test tools.

2018-06-04 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 9c8eee35d -> 8a03476eb Removed unused libprocess test tools. Review: https://reviews.apache.org/r/67431/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/8a03476e

mesos git commit: Updated mesos-tidy setup to be based on upstream 6.0 release.

2018-06-04 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 708376ebd -> 01b2d8c22 Updated mesos-tidy setup to be based on upstream 6.0 release. Review: https://reviews.apache.org/r/66035/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[1/2] mesos git commit: Changed default executor tests to not use pipes for synchronization.

2018-06-25 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master d22a3d701 -> 3f60c9857 Changed default executor tests to not use pipes for synchronization. Some tests of nested container functionality used pipes passed to launched tasks to detect whether a task has actually started executing the

[2/2] mesos git commit: Avoided leaking file descriptors in Mesos containerizer.

2018-06-25 Thread bbannier
sary there. +// +// TODO(bbannier): Consider moving this to stout as e.g., `os::lsof`. +#ifndef __WINDOWS__ +static Try> getOpenFileDescriptors() +{ + Try> fds = +#if defined(__linux__) +os::ls("/proc/self/fd"); +#elif defined(__APPLE__) +os::ls("/dev/fd"); +#endif + + if

mesos git commit: Allow for unbundled libevent cmake builds.

2018-06-26 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 4cdb0f376 -> 9407d5468 Allow for unbundled libevent cmake builds. Review: https://reviews.apache.org/r/67606/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/9407d546

mesos git commit: Fixed formatting in libprocess `README`.

2018-06-25 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 3228db4ac -> d22a3d701 Fixed formatting in libprocess `README`. The table columns were misaligned due to missing whitespace. Review: https://reviews.apache.org/r/67719/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[2/3] mesos git commit: Removed `PICOJSON_USE_INT64` from libprocess build system.

2018-06-26 Thread bbannier
Removed `PICOJSON_USE_INT64` from libprocess build system. It is no longer necessary, since the macro is now set directly within `stout/json.hpp`. Review: https://reviews.apache.org/r/67633/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[3/3] mesos git commit: Removed `PICOJSON_USE_INT64` from Mesos build system.

2018-06-26 Thread bbannier
Removed `PICOJSON_USE_INT64` from Mesos build system. It is no longer necessary, since the macro is now defined directly within `stout/json.hpp`. Review: https://reviews.apache.org/r/67634/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

[1/3] mesos git commit: Always defined `PICOJSON_USE_INT64`.

2018-06-26 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 9407d5468 -> 784825b33 Always defined `PICOJSON_USE_INT64`. The macro `PICOJSON_USE_INT64` must always be defined when attempting to use stout's JSON facilities since they unconditionally depend on these features. Previously, we relied on

mesos git commit: Fixed post-reviews for posting reviews from `master` branch.

2018-01-11 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 81937e5e4 -> 164d99e1b Fixed post-reviews for posting reviews from `master` branch. The post-reviews script allows to specify the tracking branch against which to post reviews; we default to creating patches against `master`. The script

mesos git commit: Fixed the broken GRPC build.

2018-01-05 Thread bbannier
Repository: mesos Updated Branches: refs/heads/1.5.x c0e802f52 -> 944d74ad3 Fixed the broken GRPC build. Added `upgradeResources` to the `resources_utils.hpp`, and a silly mistake around using `foreach` over a protobuf mutable repeated field. Review: https://reviews.apache.org/r/64924

mesos git commit: Changed 'ConstantEndpointDetector' to have value semantics.

2018-01-05 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 82a631911 -> 625189058 Changed 'ConstantEndpointDetector' to have value semantics. The class 'ConstantEndpointDetector' was holding a reference to one of its constructor parameters which introduces unnecessary brittleness. In this patch

mesos git commit: Updated `mesos-tidy` docker build setup for benchmarks.

2018-01-18 Thread bbannier
uce them as a side-effect. This +# is pretty hacky for what we want to do, but it's okay for now. +cmake --build 3rdparty/stout/tests --target stout-tests -- -j $(nproc) +cmake --build 3rdparty/libprocess/src/tests --target benchmarks -- -j $(nproc) + # TODO(bbannier): Use a less restrictive `grep` pattern an

mesos git commit: Moved agent response code into 'protobuf_utils.cpp'.

2018-01-17 Thread bbannier
Repository: mesos Updated Branches: refs/heads/master 2c5da1b66 -> f6d7cd6da Moved agent response code into 'protobuf_utils.cpp'. Review: https://reviews.apache.org/r/65043/ Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit:

  1   2   3   4   5   >