| Gastón Kleiman |
Re: Review Request 55901: Added support for command health checks to the default executor. |
Mon, 06 Mar, 16:23 |
| Gastón Kleiman |
Review Request 57384: Added `Containerizer::cleanupArtifacts`. |
Tue, 07 Mar, 17:13 |
| Gastón Kleiman |
Review Request 57385: Introduced proto changes needed for the DeleteNestedContainer API call. |
Tue, 07 Mar, 17:13 |
| Gastón Kleiman |
Review Request 57386: Introduced changes to the auth photos needed for DeleteNestedContainer. |
Tue, 07 Mar, 17:14 |
| Gastón Kleiman |
Re: Review Request 57386: Introduced changes to the auth photos needed for DeleteNestedContainer. |
Tue, 07 Mar, 17:14 |
| Gastón Kleiman |
Review Request 57341: Made the usage of C++ namespaces in 'slave/http.cpp' consistent. |
Tue, 07 Mar, 17:15 |
| Gastón Kleiman |
Review Request 57387: Implemented the Agent API call `DeleteNestedContainer`. |
Tue, 07 Mar, 17:16 |
| Gastón Kleiman |
Review Request 57388: Added documentation for the 'DeleteNestedContainer' Agent API call. |
Tue, 07 Mar, 17:17 |
| Gastón Kleiman |
Re: Review Request 57387: Implemented the Agent API call `DeleteNestedContainer`. |
Tue, 07 Mar, 17:17 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::cleanupArtifacts`. |
Wed, 08 Mar, 10:50 |
| Gastón Kleiman |
Re: Review Request 57385: Introduced proto changes needed for the DeleteNestedContainer API call. |
Wed, 08 Mar, 11:07 |
| Gastón Kleiman |
Re: Review Request 57386: Introduced changes to the auth protos needed for DeleteNestedContainer. |
Wed, 08 Mar, 11:08 |
| Gastón Kleiman |
Re: Review Request 57386: Introduced changes to the auth protos needed for DeleteNestedContainer. |
Wed, 08 Mar, 11:10 |
| Gastón Kleiman |
Re: Review Request 57341: Made the usage of C++ namespaces in 'slave/http.cpp' consistent. |
Wed, 08 Mar, 11:11 |
| Gastón Kleiman |
Re: Review Request 57341: Made the usage of C++ namespaces in 'slave/http.cpp' consistent. |
Wed, 08 Mar, 11:12 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::cleanupArtifacts`. |
Wed, 08 Mar, 11:18 |
| Gastón Kleiman |
Re: Review Request 57422: Removed misleading comment from protobuf file. |
Wed, 08 Mar, 16:18 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::cleanupArtifacts`. |
Thu, 09 Mar, 13:46 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::cleanupArtifacts`. |
Thu, 09 Mar, 13:52 |
| Gastón Kleiman |
Re: Review Request 57386: Introduced changes to the auth protos needed for DeleteNestedContainer. |
Thu, 09 Mar, 13:54 |
| Gastón Kleiman |
Re: Review Request 57341: Made the usage of C++ namespaces in 'slave/http.cpp' consistent. |
Thu, 09 Mar, 14:30 |
| Gastón Kleiman |
Re: Review Request 57386: Introduced changes to the authz protos needed for DeleteNestedContainer. |
Thu, 09 Mar, 14:33 |
| Gastón Kleiman |
Re: Review Request 57341: Made the usage of C++ namespaces in 'slave/http.cpp' consistent. |
Fri, 10 Mar, 16:06 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::remove`. |
Fri, 10 Mar, 17:58 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::remove`. |
Fri, 10 Mar, 17:59 |
| Gastón Kleiman |
Re: Review Request 57385: Introduced proto changes needed for the RemoveNestedContainer API call. |
Fri, 10 Mar, 18:00 |
| Gastón Kleiman |
Re: Review Request 57386: Introduced changes to the authz protos needed for RemoveNestedContainer. |
Fri, 10 Mar, 18:02 |
| Gastón Kleiman |
Re: Review Request 57387: Implemented the Agent API call `RemoveNestedContainer`. |
Fri, 10 Mar, 18:04 |
| Gastón Kleiman |
Re: Review Request 57388: Added documentation for the 'RemoveNestedContainer' Agent API call. |
Fri, 10 Mar, 18:05 |
| Gastón Kleiman |
Re: Review Request 57387: Implemented the Agent API call `RemoveNestedContainer`. |
Mon, 13 Mar, 12:18 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::remove`. |
Mon, 13 Mar, 12:32 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::remove`. |
Mon, 13 Mar, 12:33 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::remove`. |
Mon, 13 Mar, 12:33 |
| Gastón Kleiman |
Re: Review Request 57385: Introduced proto changes needed for the RemoveNestedContainer API call. |
Mon, 13 Mar, 12:34 |
| Gastón Kleiman |
Re: Review Request 57387: Implemented the Agent API call `RemoveNestedContainer`. |
Mon, 13 Mar, 12:37 |
| Gastón Kleiman |
Re: Review Request 57388: Added documentation for the 'RemoveNestedContainer' Agent API call. |
Mon, 13 Mar, 12:38 |
| Gastón Kleiman |
Re: Review Request 57384: Added `Containerizer::remove`. |
Mon, 13 Mar, 14:25 |
| Gastón Kleiman |
Re: Review Request 57387: Implemented the Agent API call `RemoveNestedContainer`. |
Mon, 13 Mar, 14:36 |
| Gastón Kleiman |
Re: Review Request 57387: Implemented the Agent API call `RemoveNestedContainer`. |
Mon, 13 Mar, 14:40 |
| Gastón Kleiman |
Review Request 57560: Removed unnecessary curly braces wrapping case statements. |
Mon, 13 Mar, 15:11 |
| Gastón Kleiman |
Re: Review Request 57560: Removed unnecessary curly braces wrapping case statements. |
Mon, 13 Mar, 18:19 |
| Gastón Kleiman |
Re: Review Request 57341: Made the usage of C++ namespaces in 'slave/http.cpp' consistent. |
Mon, 13 Mar, 18:19 |
| Gastón Kleiman |
Re: Review Request 57560: Removed unnecessary curly braces wrapping case statements. |
Tue, 14 Mar, 10:57 |
| Gastón Kleiman |
Review Request 57593: Sorted enum values in `authorizer.proto` by numerical order. |
Tue, 14 Mar, 11:20 |
| Gastón Kleiman |
Re: Review Request 57386: Introduced changes to the authz protos needed for RemoveNestedContainer. |
Tue, 14 Mar, 11:21 |
| Gastón Kleiman |
Re: Review Request 56016: Added a note about task status updates in scheduler and internal API. |
Wed, 15 Mar, 13:14 |
| Gastón Kleiman |
Re: Review Request 57394: Overloaded `<<` for `CheckInfo::Type`. |
Wed, 15 Mar, 13:16 |
| Gastón Kleiman |
Re: Review Request 56017: Added a helper for building a task status from an existing one. |
Wed, 15 Mar, 13:19 |
| Gastón Kleiman |
Re: Review Request 56017: Added a helper for building a task status from an existing one. |
Wed, 15 Mar, 13:24 |
| Gastón Kleiman |
Review Request 57644: Implemented 'ComposingContainerizer::remove(const ContainerID&)'. |
Wed, 15 Mar, 14:58 |
| Gastón Kleiman |
Re: Review Request 56288: Improved the wording of what's logged on command health check timeouts. |
Wed, 15 Mar, 14:58 |
| Gastón Kleiman |
Re: Review Request 55901: Added support for command health checks to the default executor. |
Wed, 15 Mar, 14:59 |
| Gastón Kleiman |
Review Request 57645: Added support for pausing health checks. |
Wed, 15 Mar, 14:59 |
| Gastón Kleiman |
Review Request 57646: Made COMMAND health checks resilient to agent failovers. |
Wed, 15 Mar, 14:59 |
| Gastón Kleiman |
Review Request 57647: Made CMD health checks not reuse the check `ContainerID`. |
Wed, 15 Mar, 14:59 |
| Gastón Kleiman |
Re: Review Request 55901: Added support for command health checks to the default executor. |
Wed, 15 Mar, 15:05 |
| Gastón Kleiman |
Re: Review Request 57644: Implemented 'ComposingContainerizer::remove(const ContainerID&)'. |
Wed, 15 Mar, 16:18 |
| Gastón Kleiman |
Review Request 57648: Improved failure message in `MesosContainerizer::remove()`. |
Wed, 15 Mar, 16:22 |
| Gastón Kleiman |
Re: Review Request 56288: Improved the wording of what's logged on command health check timeouts. |
Wed, 15 Mar, 16:22 |
| Gastón Kleiman |
Re: Review Request 57644: Implemented 'ComposingContainerizer::remove(const ContainerID&)'. |
Wed, 15 Mar, 16:54 |
| Gastón Kleiman |
Re: Review Request 57648: Improved failure message in `MesosContainerizer::remove()`. |
Wed, 15 Mar, 20:15 |
| Gastón Kleiman |
Re: Review Request 57648: Improved failure message in `MesosContainerizer::remove()`. |
Thu, 16 Mar, 13:38 |
| Gastón Kleiman |
Re: Review Request 55901: Added support for command health checks to the default executor. |
Thu, 16 Mar, 14:31 |
| Gastón Kleiman |
Re: Review Request 57773: Improved logging on a fatal condition in command executor. |
Mon, 20 Mar, 15:13 |
| Gastón Kleiman |
Re: Review Request 56449: Moved health checker closer to container in default executor. |
Mon, 20 Mar, 15:23 |
| Gastón Kleiman |
Re: Review Request 57694: Added a warning if default executor gets an unknown acknowledgement. |
Mon, 20 Mar, 15:26 |
| Gastón Kleiman |
Re: Review Request 57694: Added a warning if default executor gets an unknown acknowledgement. |
Mon, 20 Mar, 15:26 |
| Gastón Kleiman |
Re: Review Request 57695: Kept TaskInfo beyond first scheduler ack in default executor. |
Mon, 20 Mar, 16:07 |
| Gastón Kleiman |
Re: Review Request 57775: Ensured check's exit code is interpreted correctly on Windows. |
Mon, 20 Mar, 17:01 |
| Gastón Kleiman |
Re: Review Request 57696: Overloaded `<<` for `CheckStatusInfo`. |
Mon, 20 Mar, 17:12 |
| Gastón Kleiman |
Re: Review Request 57648: Improved failure message in `MesosContainerizer::remove()`. |
Tue, 21 Mar, 13:59 |
| Gastón Kleiman |
Re: Review Request 55901: Added support for command health checks to the default executor. |
Tue, 21 Mar, 14:01 |
| Gastón Kleiman |
Re: Review Request 57646: Made COMMAND health checks resilient to agent failovers. |
Tue, 21 Mar, 14:07 |
| Gastón Kleiman |
Re: Review Request 57646: Made COMMAND health checks resilient to agent failovers. |
Tue, 21 Mar, 14:07 |
| Gastón Kleiman |
Re: Review Request 57647: Made CMD health checks not reuse the check `ContainerID`. |
Tue, 21 Mar, 14:08 |
| Gastón Kleiman |
Re: Review Request 56215: Reused previous task status to generate a new one in default executor. |
Tue, 21 Mar, 16:33 |
| Gastón Kleiman |
Re: Review Request 56217: Added support for general checks to default executor. |
Tue, 21 Mar, 16:40 |
| Gastón Kleiman |
Re: Review Request 57696: Overloaded `<<` for `CheckStatusInfo`. |
Tue, 21 Mar, 16:41 |
| Gastón Kleiman |
Re: Review Request 56213: Added check tests for command executor. |
Tue, 21 Mar, 17:25 |
| Gastón Kleiman |
Re: Review Request 56218: Added a check test for default executor. |
Tue, 21 Mar, 17:27 |
| Gastón Kleiman |
Re: Review Request 57774: Printed a note that docker executor does not support general checks. |
Tue, 21 Mar, 17:29 |
| Gastón Kleiman |
Re: Review Request 57647: Made CMD health checks not reuse the check `ContainerID`. |
Wed, 22 Mar, 13:26 |
| Gastón Kleiman |
Re: Review Request 57647: Made COMMAND health checks not reuse the check `ContainerID`. |
Wed, 22 Mar, 16:24 |
| Gastón Kleiman |
Review Request 57854: Improved log/failure messages in the (health)checker libraries. |
Wed, 22 Mar, 17:58 |
| Gastón Kleiman |
Re: Review Request 55901: Added support for command health checks to the default executor. |
Wed, 22 Mar, 18:00 |
| Gastón Kleiman |
Re: Review Request 55901: Added support for command health checks to the default executor. |
Wed, 22 Mar, 18:00 |
| Gastón Kleiman |
Re: Review Request 57645: Added support for pausing health checks. |
Wed, 22 Mar, 18:01 |
| Gastón Kleiman |
Re: Review Request 57645: Added support for pausing health checks. |
Wed, 22 Mar, 18:01 |
| Gastón Kleiman |
Re: Review Request 57645: Added support for pausing health checks. |
Wed, 22 Mar, 18:01 |
| Gastón Kleiman |
Re: Review Request 55901: Added support for command health checks to the default executor. |
Wed, 22 Mar, 18:19 |
| Gastón Kleiman |
Re: Review Request 57646: Made COMMAND health checks resilient to agent failovers. |
Wed, 22 Mar, 18:19 |
| Gastón Kleiman |
Re: Review Request 57646: Made COMMAND health checks resilient to agent failovers. |
Wed, 22 Mar, 18:21 |
| Gastón Kleiman |
Re: Review Request 57647: Made CMD health checks not reuse the check `ContainerID`. |
Wed, 22 Mar, 18:22 |
| Gastón Kleiman |
Re: Review Request 57647: Made CMD health checks not reuse the check `ContainerID`. |
Wed, 22 Mar, 18:22 |
| Gastón Kleiman |
Re: Review Request 57854: Improved log/failure messages in the (health)checker libraries. |
Wed, 22 Mar, 18:22 |
| Gastón Kleiman |
Re: Review Request 57646: Made COMMAND health checks resilient to agent failovers. |
Wed, 22 Mar, 19:51 |
| Gastón Kleiman |
Re: Review Request 57647: Made COMMAND health checks not reuse the check `ContainerID`. |
Thu, 23 Mar, 12:14 |
| Gastón Kleiman |
Re: Review Request 57774: Printed a note that docker executor does not support general checks. |
Thu, 23 Mar, 12:14 |
| Gastón Kleiman |
Re: Review Request 57775: Ensured check's exit code is interpreted correctly on Windows. |
Thu, 23 Mar, 15:52 |
| Gastón Kleiman |
Re: Review Request 57646: Made COMMAND health checks resilient to agent failovers. |
Thu, 23 Mar, 20:07 |