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 Mon, 06 Feb 2017 23:17:42 GMT

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

(Updated Feb. 6, 2017, 11:17 p.m.)


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


Changes
-------

Addressed review comment.


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
will be (dis)allowed to launch on the agent.
If a task or task group cannot be launched due to failed authorization,
a `TASK_FAILED` 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 5049eb783b8ad7b9599f20c3701f7d3d654b4491 
  src/slave/slave.cpp 92564ff8fff06d1cb17192d374d355b4bb7d39d8 

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


Testing
-------

All tests passed.


Thanks,

Anindya Sinha


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