You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by haosdent huang <ha...@gmail.com> on 2016/01/27 19:11:23 UTC
Review Request 42861: Ensure two Mount Disk resources do not have the
same root path.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/
-----------------------------------------------------------
Review request for mesos, Jie Yu and Joris Van Remoortere.
Bugs: MESOS-4521
https://issues.apache.org/jira/browse/MESOS-4521
Repository: mesos
Description
-------
Ensure two Mount Disk resources do not have the same root path.
Diffs
-----
src/slave/slave.cpp 1f4c8368feb0ce19963577582ce745acfb21aa9f
Diff: https://reviews.apache.org/r/42861/diff/
Testing
-------
Thanks,
haosdent huang
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by Neil Conway <ne...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/#review124576
-----------------------------------------------------------
src/slave/slave.cpp (line 430)
<https://reviews.apache.org/r/42861/#comment187209>
I'm confused by the logic here. Suppose I have two mount volumes at `/mnt/foo` and `/mnt/bar`. It seems this would be rejected -- why?
Similarly, suppose I have three volumes: `/foo`, `/mnt/foo`, and `/mnt/foo/baz`. This probably _should_ be rejected (because `/mnt/foo` and `/mnt/foo/baz` overlap), but AFAICS it wouldn't be rejected by this patch. Checking whether _all_ MOUNT volumes have the same parent / overlap doesn't seem right; don't we want to check whether any two volumes overlap?
src/slave/slave.cpp (line 432)
<https://reviews.apache.org/r/42861/#comment187192>
"directory" is a better term than "folder".
- Neil Conway
On Jan. 28, 2016, 2:12 a.m., haosdent huang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42861/
> -----------------------------------------------------------
>
> (Updated Jan. 28, 2016, 2:12 a.m.)
>
>
> Review request for mesos, Jie Yu and Joris Van Remoortere.
>
>
> Bugs: MESOS-4521
> https://issues.apache.org/jira/browse/MESOS-4521
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Ensure two Mount Disk resources do not have the same root path.
>
>
> Diffs
> -----
>
> src/slave/slave.cpp 1f4c8368feb0ce19963577582ce745acfb21aa9f
>
> Diff: https://reviews.apache.org/r/42861/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> haosdent huang
>
>
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by Mesos ReviewBot <re...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/#review116731
-----------------------------------------------------------
Patch looks great!
Reviews applied: [42860, 42861]
Passed command: export OS=ubuntu:14.04;export CONFIGURATION="--verbose";export COMPILER=gcc; ./support/docker_build.sh
- Mesos ReviewBot
On Jan. 28, 2016, 2:12 a.m., haosdent huang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42861/
> -----------------------------------------------------------
>
> (Updated Jan. 28, 2016, 2:12 a.m.)
>
>
> Review request for mesos, Jie Yu and Joris Van Remoortere.
>
>
> Bugs: MESOS-4521
> https://issues.apache.org/jira/browse/MESOS-4521
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Ensure two Mount Disk resources do not have the same root path.
>
>
> Diffs
> -----
>
> src/slave/slave.cpp 1f4c8368feb0ce19963577582ce745acfb21aa9f
>
> Diff: https://reviews.apache.org/r/42861/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> haosdent huang
>
>
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by Mesos ReviewBot <re...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/#review126630
-----------------------------------------------------------
Patch looks great!
Reviews applied: [42860, 42861]
Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh
- Mesos ReviewBot
On April 1, 2016, 4:43 p.m., haosdent huang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42861/
> -----------------------------------------------------------
>
> (Updated April 1, 2016, 4:43 p.m.)
>
>
> Review request for mesos, Benjamin Bannier, Jie Yu, Joris Van Remoortere, and Neil Conway.
>
>
> Bugs: MESOS-4521
> https://issues.apache.org/jira/browse/MESOS-4521
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Ensure two Mount Disk resources do not have the same root path.
>
>
> Diffs
> -----
>
> src/slave/slave.cpp fc77f594d16a9fb2ca001e089d74e2c0ffeb5baa
>
> Diff: https://reviews.apache.org/r/42861/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> haosdent huang
>
>
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by haosdent huang <ha...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/
-----------------------------------------------------------
(Updated April 1, 2016, 4:43 p.m.)
Review request for mesos, Benjamin Bannier, Jie Yu, Joris Van Remoortere, and Neil Conway.
Changes
-------
Rebase.
Bugs: MESOS-4521
https://issues.apache.org/jira/browse/MESOS-4521
Repository: mesos
Description
-------
Ensure two Mount Disk resources do not have the same root path.
Diffs (updated)
-----
src/slave/slave.cpp fc77f594d16a9fb2ca001e089d74e2c0ffeb5baa
Diff: https://reviews.apache.org/r/42861/diff/
Testing
-------
Thanks,
haosdent huang
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by haosdent huang <ha...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/
-----------------------------------------------------------
(Updated March 29, 2016, 5:33 p.m.)
Review request for mesos, Benjamin Bannier, Jie Yu, Joris Van Remoortere, and Neil Conway.
Changes
-------
Rebase.
Bugs: MESOS-4521
https://issues.apache.org/jira/browse/MESOS-4521
Repository: mesos
Description
-------
Ensure two Mount Disk resources do not have the same root path.
Diffs (updated)
-----
src/slave/slave.cpp fc77f594d16a9fb2ca001e089d74e2c0ffeb5baa
Diff: https://reviews.apache.org/r/42861/diff/
Testing
-------
Thanks,
haosdent huang
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by haosdent huang <ha...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/
-----------------------------------------------------------
(Updated March 29, 2016, 5:29 p.m.)
Review request for mesos, Benjamin Bannier, Jie Yu, Joris Van Remoortere, and Neil Conway.
Changes
-------
Rebase.
Bugs: MESOS-4521
https://issues.apache.org/jira/browse/MESOS-4521
Repository: mesos
Description
-------
Ensure two Mount Disk resources do not have the same root path.
Diffs (updated)
-----
src/slave/slave.cpp fc77f594d16a9fb2ca001e089d74e2c0ffeb5baa
Diff: https://reviews.apache.org/r/42861/diff/
Testing
-------
Thanks,
haosdent huang
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by haosdent huang <ha...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/
-----------------------------------------------------------
(Updated March 29, 2016, 8:56 a.m.)
Review request for mesos, Benjamin Bannier, Jie Yu, Joris Van Remoortere, and Neil Conway.
Changes
-------
Address @neilc comments.
Bugs: MESOS-4521
https://issues.apache.org/jira/browse/MESOS-4521
Repository: mesos
Description
-------
Ensure two Mount Disk resources do not have the same root path.
Diffs (updated)
-----
src/slave/slave.cpp fc77f594d16a9fb2ca001e089d74e2c0ffeb5baa
Diff: https://reviews.apache.org/r/42861/diff/
Testing
-------
Thanks,
haosdent huang
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by Neil Conway <ne...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/#review125802
-----------------------------------------------------------
It would be good to have a unit test for the end-to-end behavior (i.e., we have a test for `path::overlapping` in the other RR, but we should also have a test for trying to create two volumes with overlapping paths).
src/slave/slave.cpp (line 504)
<https://reviews.apache.org/r/42861/#comment188683>
Comment should be updated if we're going to do this even when you're not "verifying against the mount table entries".
src/slave/slave.cpp (line 552)
<https://reviews.apache.org/r/42861/#comment188681>
Error message shouldn't end in punctuation.
- Neil Conway
On March 29, 2016, 2:30 a.m., haosdent huang wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/42861/
> -----------------------------------------------------------
>
> (Updated March 29, 2016, 2:30 a.m.)
>
>
> Review request for mesos, Benjamin Bannier, Jie Yu, Joris Van Remoortere, and Neil Conway.
>
>
> Bugs: MESOS-4521
> https://issues.apache.org/jira/browse/MESOS-4521
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Ensure two Mount Disk resources do not have the same root path.
>
>
> Diffs
> -----
>
> src/slave/slave.cpp f383605a52f31d7b805ad6153adc409dcb40f83a
>
> Diff: https://reviews.apache.org/r/42861/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> haosdent huang
>
>
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by haosdent huang <ha...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/
-----------------------------------------------------------
(Updated March 29, 2016, 2:30 a.m.)
Review request for mesos, Benjamin Bannier, Jie Yu, Joris Van Remoortere, and Neil Conway.
Changes
-------
Address comments.
Bugs: MESOS-4521
https://issues.apache.org/jira/browse/MESOS-4521
Repository: mesos
Description
-------
Ensure two Mount Disk resources do not have the same root path.
Diffs (updated)
-----
src/slave/slave.cpp f383605a52f31d7b805ad6153adc409dcb40f83a
Diff: https://reviews.apache.org/r/42861/diff/
Testing
-------
Thanks,
haosdent huang
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by haosdent huang <ha...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/
-----------------------------------------------------------
(Updated Jan. 28, 2016, 2:12 a.m.)
Review request for mesos, Jie Yu and Joris Van Remoortere.
Bugs: MESOS-4521
https://issues.apache.org/jira/browse/MESOS-4521
Repository: mesos
Description
-------
Ensure two Mount Disk resources do not have the same root path.
Diffs (updated)
-----
src/slave/slave.cpp 1f4c8368feb0ce19963577582ce745acfb21aa9f
Diff: https://reviews.apache.org/r/42861/diff/
Testing
-------
Thanks,
haosdent huang
Re: Review Request 42861: Ensure two Mount Disk resources do not have
the same root path.
Posted by haosdent huang <ha...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42861/
-----------------------------------------------------------
(Updated Jan. 27, 2016, 6:36 p.m.)
Review request for mesos, Jie Yu and Joris Van Remoortere.
Changes
-------
Rebase
Bugs: MESOS-4521
https://issues.apache.org/jira/browse/MESOS-4521
Repository: mesos
Description
-------
Ensure two Mount Disk resources do not have the same root path.
Diffs (updated)
-----
src/slave/slave.cpp 1f4c8368feb0ce19963577582ce745acfb21aa9f
Diff: https://reviews.apache.org/r/42861/diff/
Testing
-------
Thanks,
haosdent huang