> On May 21, 2014, 4:52 p.m., Vinod Kone wrote: > > src/master/master.cpp, line 994 > > <https://reviews.apache.org/r/21787/diff/1/?file=586808#file586808line994> > > > > Maybe this should be a WARNING to nudge people to set the principal? > > > > Also, how about the scheduler driver automatically setting > > FramworkInfo.principal when authentication is being used? The one issue is > > that, as we move to native clients, we dont want too much logic in the > > driver.
There are two cases: 1) The user doesn't set FramworkInfo.principal 2) The user sets a FramworkInfo.principal that's different from Credential.principal. We can correct both in the scheduler driver but my concern is that this silent change to FrameworkInfo surprises users. We already accommodate frameworks without FramworkInfo.principal so this should help with the rollout without requiring changes to FrameworkInfo immediately so I think it's sufficient and the least surprise. > On May 21, 2014, 4:52 p.m., Vinod Kone wrote: > > src/tests/mesos.hpp, line 269 > > <https://reviews.apache.org/r/21787/diff/1/?file=586810#file586810line269> > > > > Can we get the principal from DEFAULT_CREDENTIAL here? Apparently yes. - Jiang Yan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21787/#review43672 ----------------------------------------------------------- On May 21, 2014, 3:01 p.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21787/ > ----------------------------------------------------------- > > (Updated May 21, 2014, 3:01 p.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-1373 > https://issues.apache.org/jira/browse/MESOS-1373 > > > Repository: mesos-git > > > Description > ------- > > This will be used to 'Add "per-framework-principal" counters for all messages > from a scheduler on Master': https://issues.apache.org/jira/browse/MESOS-1339 > > > Diffs > ----- > > include/mesos/mesos.proto 8012873f853fdee47f8cb2f5d9dbb7cd8ccc6028 > src/master/master.hpp 5e0d712de997bd10079655df9b07099284f8257f > src/master/master.cpp 075755cad5c50a57c92d7d82f2466b467796f673 > src/tests/authentication_tests.cpp b60e595216c2b854adf048002a0d88b548e1d92e > src/tests/mesos.hpp a1a7d7991c4cdb7a994218602fe4e5d76c056456 > > Diff: https://reviews.apache.org/r/21787/diff/ > > > Testing > ------- > > Added tests. Make check. > > > Thanks, > > Jiang Yan Xu > >
