----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59000/#review173970 -----------------------------------------------------------
src/slave/containerizer/mesos/isolators/environment_secret.cpp Lines 37-44 (patched) <https://reviews.apache.org/r/59000/#comment247067> Why you need this? Can you make sure to avoid such small nits while copying pasting code? Please remove all unncessary headers, and do a sweep for other reviews as well! src/slave/containerizer/mesos/isolators/environment_secret.cpp Lines 57-58 (patched) <https://reviews.apache.org/r/59000/#comment247069> Do you use those in this file? Ditto my other comments. src/slave/containerizer/mesos/isolators/environment_secret.cpp Lines 100 (patched) <https://reviews.apache.org/r/59000/#comment247068> Please remove std:: prefix here. Please do a sweep to fix all such issues in other reviews! - Jie Yu On May 4, 2017, 8:08 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59000/ > ----------------------------------------------------------- > > (Updated May 4, 2017, 8:08 p.m.) > > > Review request for mesos, Gilbert Song, Greg Mann, Jie Yu, and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > This isolator resolves Secret-type environment variables using a provided > SecretFetcher module. > > > Diffs > ----- > > src/Makefile.am 29da17bee13226e18757e2ad3a7a091427fd35f4 > src/launcher/executor.cpp b05f73e539c22d4d40f07df76168a06373b818d4 > src/slave/containerizer/mesos/containerizer.cpp > b58baed64480e22f640a4852537f85922ed382ae > src/slave/containerizer/mesos/isolators/environment_secret.hpp PRE-CREATION > src/slave/containerizer/mesos/isolators/environment_secret.cpp PRE-CREATION > src/tests/containerizer/environment_secret_isolator_tests.cpp PRE-CREATION > > > Diff: https://reviews.apache.org/r/59000/diff/1/ > > > Testing > ------- > > Added a new test and ran `make check`. > > > Thanks, > > Kapil Arya > >