Re: Review Request 52309: Set the user to stderr-logger binary too.

2016-09-30 Thread Sivaram Kannan

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

(Updated Sept. 30, 2016, 3:29 p.m.)


Review request for mesos and Joseph Wu.


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


Repository: mesos


Description
---

Set the user to stderr-logger binary too.


Diffs (updated)
-

  src/slave/container_loggers/lib_logrotate.cpp 
0ca2b3d7dbb57c11c0740aed3914a6b75329af99 

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


Testing
---

1. Run the mesos-logrotate-logger with un-priviledged user and verify whether 
the logs are getting rotated. 
2. Run the mesos-logrotate-logger as root user and verify whether the logs are 
getting rotated.


Thanks,

Sivaram Kannan



Re: Review Request 52309: Set the user to stderr-logger binary too.

2016-09-30 Thread Sivaram Kannan

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

(Updated Sept. 30, 2016, 2:50 p.m.)


Review request for mesos and Joseph Wu.


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


Repository: mesos


Description
---

Set the user to stderr-logger binary too.


Diffs (updated)
-

  src/slave/container_loggers/lib_logrotate.cpp 
0ca2b3d7dbb57c11c0740aed3914a6b75329af99 

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


Testing
---

1. Run the mesos-logrotate-logger with un-priviledged user and verify whether 
the logs are getting rotated. 
2. Run the mesos-logrotate-logger as root user and verify whether the logs are 
getting rotated.


Thanks,

Sivaram Kannan



Re: Review Request 52309: Set the user to stderr-logger binary too.

2016-09-30 Thread Sivaram Kannan

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

(Updated Sept. 30, 2016, 2:38 p.m.)


Review request for mesos and Joseph Wu.


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


Repository: mesos


Description
---

Set the user to stderr-logger binary too.


Diffs
-

  src/slave/container_loggers/lib_logrotate.cpp 
0ca2b3d7dbb57c11c0740aed3914a6b75329af99 

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


Testing
---

1. Run the mesos-logrotate-logger with un-priviledged user and verify whether 
the logs are getting rotated. 
2. Run the mesos-logrotate-logger as root user and verify whether the logs are 
getting rotated.


Thanks,

Sivaram Kannan



Re: Review Request 52309: Set the user to stderr-logger binary too.

2016-09-30 Thread Sivaram Kannan

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

(Updated Sept. 30, 2016, 2:17 p.m.)


Review request for mesos and Joseph Wu.


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


Repository: mesos


Description
---

Set the user to stderr-logger binary too.


Diffs (updated)
-

  src/slave/container_loggers/lib_logrotate.cpp 
0ca2b3d7dbb57c11c0740aed3914a6b75329af99 

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


Testing
---

1. Run the mesos-logrotate-logger with un-priviledged user and verify whether 
the logs are getting rotated. 
2. Run the mesos-logrotate-logger as root user and verify whether the logs are 
getting rotated.


Thanks,

Sivaram Kannan



Re: Review Request 52309: Set the user to stderr-logger binary too.

2016-09-29 Thread Sivaram Kannan

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

(Updated Sept. 29, 2016, 6:47 p.m.)


Review request for mesos and Joseph Wu.


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


Repository: mesos


Description
---

Set the user to stderr-logger binary too.


Diffs
-

  src/slave/container_loggers/lib_logrotate.cpp 
0ca2b3d7dbb57c11c0740aed3914a6b75329af99 

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


Testing
---

1. Run the mesos-logrotate-logger with un-priviledged user and verify whether 
the logs are getting rotated. 
2. Run the mesos-logrotate-logger as root user and verify whether the logs are 
getting rotated.


Thanks,

Sivaram Kannan



Re: Review Request 52309: Set the user to stderr-logger binary too.

2016-09-29 Thread Sivaram Kannan

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

(Updated Sept. 29, 2016, 6:38 p.m.)


Review request for mesos and Joseph Wu.


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


Repository: mesos


Description
---

Set the user to stderr-logger binary too.


Diffs
-

  src/slave/container_loggers/lib_logrotate.cpp 
0ca2b3d7dbb57c11c0740aed3914a6b75329af99 

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


Testing
---

1. Run the mesos-logrotate-logger with un-priviledged user and verify whether 
the logs are getting rotated. 
2. Run the mesos-logrotate-logger as root user and verify whether the logs are 
getting rotated.


Thanks,

Sivaram Kannan



Re: Review Request 52309: Set the user to stderr-logger binary too.

2016-09-29 Thread Sivaram Kannan

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

(Updated Sept. 29, 2016, 3:55 p.m.)


Review request for mesos and Joseph Wu.


Summary (updated)
-

Set the user to stderr-logger binary too.


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


Repository: mesos


Description (updated)
---

Set the user to stderr-logger binary too.


Diffs (updated)
-

  src/slave/container_loggers/lib_logrotate.cpp 
0ca2b3d7dbb57c11c0740aed3914a6b75329af99 

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


Testing
---

1. Run the mesos-logrotate-logger with un-priviledged user and verify whether 
the logs are getting rotated. 
2. Run the mesos-logrotate-logger as root user and verify whether the logs are 
getting rotated.


Thanks,

Sivaram Kannan