You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Qian Zhang <zh...@gmail.com> on 2017/01/03 14:17:30 UTC
Re: Review Request 52379: Added agent flag '--oci_store_dir'.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52379/
-----------------------------------------------------------
(Updated Jan. 3, 2017, 10:17 p.m.)
Review request for mesos, Gilbert Song and Jie Yu.
Changes
-------
Rebased.
Bugs: MESOS-6681
https://issues.apache.org/jira/browse/MESOS-6681
Repository: mesos
Description
-------
Added agent flag '--oci_store_dir'.
Diffs (updated)
-----
docs/configuration.md e4beb2d5a72f1c5f59b2e40f4984cc60b8437d9d
docs/endpoints/slave/state.json.md 0f82c1926404e79b281b2ea5f4d0ca21323aeded
docs/endpoints/slave/state.md b34459e8624f0b29e927ff79be7fc845ac88080b
src/slave/flags.hpp 6ac0d45072157f6741b96266886a326e9337c153
src/slave/flags.cpp 1eccea920338032173be4df6c374ec50dbd2eaf3
src/slave/http.cpp b6e2d4f9190358d113b39140d116b8659ddbb49b
Diff: https://reviews.apache.org/r/52379/diff/
Testing
-------
Thanks,
Qian Zhang
Re: Review Request 52379: Added agent flag '--oci_store_dir'.
Posted by Qian Zhang <zh...@gmail.com>.
> On Jan. 9, 2017, 5:03 p.m., Gilbert Song wrote:
> > src/slave/flags.cpp, line 174
> > <https://reviews.apache.org/r/52379/diff/6/?file=1596001#file1596001line174>
> >
> > Why not consistent with docker/appc store?
This was a suggestion of Avinash when he reviewed the design doc, the reason that we do not choose /tmp/mesos/store/oci is, /tmp is a tmpfs (or ramfs) an in-memory file system, the OCI images can be quite big (as with docker images) and might end up filling this file system very fast. And another reason is /tmp will be automatically cleared upon agent host reboot which is also what we do not want.
- Qian
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52379/#review160873
-----------------------------------------------------------
On Jan. 3, 2017, 10:17 p.m., Qian Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52379/
> -----------------------------------------------------------
>
> (Updated Jan. 3, 2017, 10:17 p.m.)
>
>
> Review request for mesos, Gilbert Song and Jie Yu.
>
>
> Bugs: MESOS-6681
> https://issues.apache.org/jira/browse/MESOS-6681
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Added agent flag '--oci_store_dir'.
>
>
> Diffs
> -----
>
> docs/configuration.md e4beb2d5a72f1c5f59b2e40f4984cc60b8437d9d
> docs/endpoints/slave/state.json.md 0f82c1926404e79b281b2ea5f4d0ca21323aeded
> docs/endpoints/slave/state.md b34459e8624f0b29e927ff79be7fc845ac88080b
> src/slave/flags.hpp 6ac0d45072157f6741b96266886a326e9337c153
> src/slave/flags.cpp 1eccea920338032173be4df6c374ec50dbd2eaf3
> src/slave/http.cpp b6e2d4f9190358d113b39140d116b8659ddbb49b
>
> Diff: https://reviews.apache.org/r/52379/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Qian Zhang
>
>
Re: Review Request 52379: Added agent flag '--oci_store_dir'.
Posted by Gilbert Song <so...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52379/#review160873
-----------------------------------------------------------
src/slave/flags.cpp (line 174)
<https://reviews.apache.org/r/52379/#comment232123>
Why not consistent with docker/appc store?
- Gilbert Song
On Jan. 3, 2017, 6:17 a.m., Qian Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52379/
> -----------------------------------------------------------
>
> (Updated Jan. 3, 2017, 6:17 a.m.)
>
>
> Review request for mesos, Gilbert Song and Jie Yu.
>
>
> Bugs: MESOS-6681
> https://issues.apache.org/jira/browse/MESOS-6681
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Added agent flag '--oci_store_dir'.
>
>
> Diffs
> -----
>
> docs/configuration.md e4beb2d5a72f1c5f59b2e40f4984cc60b8437d9d
> docs/endpoints/slave/state.json.md 0f82c1926404e79b281b2ea5f4d0ca21323aeded
> docs/endpoints/slave/state.md b34459e8624f0b29e927ff79be7fc845ac88080b
> src/slave/flags.hpp 6ac0d45072157f6741b96266886a326e9337c153
> src/slave/flags.cpp 1eccea920338032173be4df6c374ec50dbd2eaf3
> src/slave/http.cpp b6e2d4f9190358d113b39140d116b8659ddbb49b
>
> Diff: https://reviews.apache.org/r/52379/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Qian Zhang
>
>
Re: Review Request 52379: Added agent flag '--oci_store_dir'.
Posted by Qian Zhang <zh...@gmail.com>.
> On Feb. 5, 2017, 12:18 p.m., Jie Yu wrote:
> > src/slave/flags.cpp
> > Lines 173 (patched)
> > <https://reviews.apache.org/r/52379/diff/7/?file=1621012#file1621012line173>
> >
> > Let's make the default /tmp/mesos/store/oci, to be consistent with others.
This was a suggestion of Avinash when he reviewed the design doc, the reason that we do not choose /tmp/mesos/store/oci is, /tmp is a tmpfs (or ramfs) an in-memory file system, the OCI images can be quite big (as with docker images) and might end up filling this file system very fast. And another reason is /tmp will be automatically cleared upon agent host reboot which is also what we do not want.
- Qian
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52379/#review164265
-----------------------------------------------------------
On Feb. 1, 2017, 9:15 a.m., Qian Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52379/
> -----------------------------------------------------------
>
> (Updated Feb. 1, 2017, 9:15 a.m.)
>
>
> Review request for mesos, Gilbert Song and Jie Yu.
>
>
> Bugs: MESOS-6681
> https://issues.apache.org/jira/browse/MESOS-6681
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Added agent flag '--oci_store_dir'.
>
>
> Diffs
> -----
>
> docs/configuration.md 6a30471d325f35e171e81a687c23fd69e9ec97ee
> docs/endpoints/slave/state.json.md 0f82c1926404e79b281b2ea5f4d0ca21323aeded
> docs/endpoints/slave/state.md b34459e8624f0b29e927ff79be7fc845ac88080b
> src/slave/flags.hpp 2c4bd6ae628a272a4c6c2f02670baef011df4505
> src/slave/flags.cpp b48678bae8b3f5bab62710043eaca14ab8370183
> src/slave/http.cpp d68c9adc1db43c2e853c8b2290705fdc7739d45c
>
>
> Diff: https://reviews.apache.org/r/52379/diff/7/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Qian Zhang
>
>
Re: Review Request 52379: Added agent flag '--oci_store_dir'.
Posted by Jie Yu <yu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52379/#review164265
-----------------------------------------------------------
Fix it, then Ship it!
src/slave/flags.cpp (line 173)
<https://reviews.apache.org/r/52379/#comment235933>
Let's make the default /tmp/mesos/store/oci, to be consistent with others.
- Jie Yu
On Feb. 1, 2017, 1:15 a.m., Qian Zhang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52379/
> -----------------------------------------------------------
>
> (Updated Feb. 1, 2017, 1:15 a.m.)
>
>
> Review request for mesos, Gilbert Song and Jie Yu.
>
>
> Bugs: MESOS-6681
> https://issues.apache.org/jira/browse/MESOS-6681
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Added agent flag '--oci_store_dir'.
>
>
> Diffs
> -----
>
> docs/configuration.md 6a30471d325f35e171e81a687c23fd69e9ec97ee
> docs/endpoints/slave/state.json.md 0f82c1926404e79b281b2ea5f4d0ca21323aeded
> docs/endpoints/slave/state.md b34459e8624f0b29e927ff79be7fc845ac88080b
> src/slave/flags.hpp 2c4bd6ae628a272a4c6c2f02670baef011df4505
> src/slave/flags.cpp b48678bae8b3f5bab62710043eaca14ab8370183
> src/slave/http.cpp d68c9adc1db43c2e853c8b2290705fdc7739d45c
>
> Diff: https://reviews.apache.org/r/52379/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Qian Zhang
>
>
Re: Review Request 52379: Added agent flag '--oci_store_dir'.
Posted by Qian Zhang <zh...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52379/
-----------------------------------------------------------
(Updated March 30, 2017, 9:37 a.m.)
Review request for mesos, Gilbert Song and Jie Yu.
Bugs: MESOS-6681
https://issues.apache.org/jira/browse/MESOS-6681
Repository: mesos
Description
-------
Added agent flag '--oci_store_dir'.
Diffs
-----
docs/configuration.md 6a30471d325f35e171e81a687c23fd69e9ec97ee
docs/endpoints/slave/state.json.md 0f82c1926404e79b281b2ea5f4d0ca21323aeded
docs/endpoints/slave/state.md b34459e8624f0b29e927ff79be7fc845ac88080b
src/slave/flags.hpp 2c4bd6ae628a272a4c6c2f02670baef011df4505
src/slave/flags.cpp b48678bae8b3f5bab62710043eaca14ab8370183
src/slave/http.cpp d68c9adc1db43c2e853c8b2290705fdc7739d45c
Diff: https://reviews.apache.org/r/52379/diff/7/
Testing
-------
Thanks,
Qian Zhang
Re: Review Request 52379: Added agent flag '--oci_store_dir'.
Posted by Qian Zhang <zh...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52379/
-----------------------------------------------------------
(Updated Feb. 1, 2017, 9:15 a.m.)
Review request for mesos, Gilbert Song and Jie Yu.
Changes
-------
Rebased.
Bugs: MESOS-6681
https://issues.apache.org/jira/browse/MESOS-6681
Repository: mesos
Description
-------
Added agent flag '--oci_store_dir'.
Diffs (updated)
-----
docs/configuration.md 6a30471d325f35e171e81a687c23fd69e9ec97ee
docs/endpoints/slave/state.json.md 0f82c1926404e79b281b2ea5f4d0ca21323aeded
docs/endpoints/slave/state.md b34459e8624f0b29e927ff79be7fc845ac88080b
src/slave/flags.hpp 2c4bd6ae628a272a4c6c2f02670baef011df4505
src/slave/flags.cpp b48678bae8b3f5bab62710043eaca14ab8370183
src/slave/http.cpp d68c9adc1db43c2e853c8b2290705fdc7739d45c
Diff: https://reviews.apache.org/r/52379/diff/
Testing
-------
Thanks,
Qian Zhang