[MESOS-4829] Remove `grace_period_seconds` field from Shutdown event v1 protobuf.

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.28.0
• Labels: mesosphere
• Epic Link: Executor API v1
• Sprint: Mesosphere Sprint 30
• Story Points: 3
• Assignee: Anand Mazumdar
• Reporter: Anand Mazumdar
• Shepherd: Vinod Kone
• Created: 01/Mar/16 18:48
• Updated: 26/Nov/18 12:20
• Resolved: 26/Nov/18 12:20

Description

There are two ways in which a shutdown of executor can be triggered:
1. If it receives an explicit `Shutdown` message from the agent.
2. If the recovery timeout period has elapsed, and the executor still hasn’t been able to (re-)connect with the agent.

Currently, the executor library relies on the field `grace_period_seconds` having a default value of 5 seconds to handle the second scenario. https://github.com/apache/mesos/blob/master/src/executor/executor.cpp#L608

The driver used to trigger the grace period via a constant defined in src/slave/constants.cpp. https://github.com/apache/mesos/blob/master/src/exec/exec.cpp#L92

The agent may want to force a shorter shutdown grace period (e.g. oversubscription eviction may have shorter deadline) in the future. For now, we can just read the value via an environment variable.

Issue Links

• relates to: MESOS-1571 Signal escalation timeout is not configurable.