[MESOS-10094] Master's agent draining VLOG prints incorrect task counts.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.9.0
• Fix Version/s: 1.9.1, 1.10.0
• Component/s: master
• Target Version/s: 1.9.1
• Epic Link: Agent Draining
• Sprint: Studio 4: RI-23 64
• Story Points: 1
• Assignee: Benjamin Mahler
• Reporter: Benjamin Mahler
• Created: 11/Feb/20 20:58
• Updated: 26/Feb/20 18:03
• Resolved: 26/Feb/20 17:46

Description

This logic is printing the framework counts of these maps rather than the task counts:

https://github.com/apache/mesos/blob/4575c9b452c25f64e6c6cc3eddc12ed3b1f8538b/src/master/master.cpp#L6318-L6319

  // Check if the agent has any tasks running or operations pending.
  if (!slave->pendingTasks.empty() ||
      !slave->tasks.empty() ||
      !slave->operations.empty()) {
    VLOG(1)
      << "DRAINING Agent " << slaveId << " has "
      << slave->pendingTasks.size() << " pending tasks, "
      << slave->tasks.size() << " tasks, and "
      << slave->operations.size() << " operations";
    return;
  }

Since these are hashmap<FrameworkID, hashmap<TaskID, Task>>.