mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Anand Mazumdar <an...@apache.org>
Subject Re: Review Request 57750: Turned off executor authentication in the ExecutorHttpApiTests.
Date Tue, 21 Mar 2017 18:59:27 GMT

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




src/tests/executor_http_api_tests.cpp
Lines 116 (patched)
<https://reviews.apache.org/r/57750/#comment242038>

    Can we add a comment for posterity? Here and else where:
    
    ```cpp
    // Disable executor AuthN on the agent.
    ```


- Anand Mazumdar


On March 21, 2017, 2:53 p.m., Greg Mann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/57750/
> -----------------------------------------------------------
> 
> (Updated March 21, 2017, 2:53 p.m.)
> 
> 
> Review request for mesos, Anand Mazumdar, Alexander Rojas, and Vinod Kone.
> 
> 
> Bugs: MESOS-6999
>     https://issues.apache.org/jira/browse/MESOS-6999
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> This patch turns off executor authentication in the
> ExecutorHttpApiTests, since authentication is not related
> to the functionality they are meant to test. Since executor
> tokens depend on the FrameworkID/ExecutorID/ContainerID of
> the executor, it would be difficult to enable it in these
> tests.
> 
> 
> Diffs
> -----
> 
>   src/tests/executor_http_api_tests.cpp 13d2e1db2ffdad34dadbf2911e91938706fd4fd0 
> 
> 
> Diff: https://reviews.apache.org/r/57750/diff/2/
> 
> 
> Testing
> -------
> 
> `make check` was done to test that all existing tests succeed when executor authentication
is enabled by default in the test cluster code. The ExecutorHttpApiTests are the only tests
in which it has been disabled.
> 
> 
> Thanks,
> 
> Greg Mann
> 
>


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