mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anindya Sinha <anindya_si...@apple.com>
Subject Re: Review Request 55887: Check task user before allowing a task to be launched on the agent.
Date Fri, 03 Mar 2017 20:08:42 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55887/
-----------------------------------------------------------

(Updated March 3, 2017, 8:08 p.m.)


Review request for mesos, Adam B, Alexander Rojas, and Jiang Yan Xu.


Changes
-------

Addressed comments.


Bugs: MESOS-6953
    https://issues.apache.org/jira/browse/MESOS-6953


Repository: mesos


Description
-------

Added support for action `run_tasks` on the agent's flag `acl`. Based on
the ACL configured for `run_tasks`, a task to be launched on the agent
can be (dis)allowed to launch on the agent.
If a task or task group cannot be launched due to failed authorization,
a `TASK_ERROR` Status Update shall be sent with a reason code of
`REASON_TASK_UNAUTHORIZED` or `REASON_TASK_GROUP_UNAUTHORIZED` as
applicable.
Note that in case of a task group, all tasks fail if any of the tasks
within the task group encounter the authorization error.


Diffs (updated)
-----

  src/slave/slave.hpp 857338cc54b341873c338967223c6b0138e9dc3e 
  src/slave/slave.cpp 775f43b449ec53aa51cc5d3de448ddc7c2059bff 


Diff: https://reviews.apache.org/r/55887/diff/7/

Changes: https://reviews.apache.org/r/55887/diff/6-7/


Testing
-------

All tests passed.


Thanks,

Anindya Sinha


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message