* [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir
@ 2015-05-13 15:15 Miklos Szeredi
2015-05-13 15:35 ` Vincent Batts
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Miklos Szeredi @ 2015-05-13 15:15 UTC (permalink / raw)
To: David Howells; +Cc: Josh Boyer, Vincent Batts, linux-unionfs
---
tests/rmdir.py | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
--- a/tests/rmdir.py
+++ b/tests/rmdir.py
@@ -203,3 +203,19 @@ from errno import *
ctx.rmdir(sym, err=ENOTDIR)
ctx.rmdir(sym, err=ENOTDIR)
ctx.rmdir(d, err=ENOENT)
+
+# Remove an empty lower directory, recreate, populate and try to remove
+def subtest_16(ctx):
+ """Remove non-empty opaque directory"""
+ d = ctx.empty_dir() + ctx.termslash()
+ f = d + "/b"
+
+ ctx.rmdir(d)
+ ctx.rmdir(d, err=ENOENT)
+ ctx.mkdir(d, 0o755)
+ ctx.open_file(f, wo=1, crt=1, ex=1, write="abcq")
+ ctx.rmdir(d, err=ENOTEMPTY)
+ ctx.unlink(f)
+ ctx.open_file(f, ro=1, err=ENOENT)
+ ctx.unlink(f, err=ENOENT)
+ ctx.rmdir(d)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir
2015-05-13 15:15 [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir Miklos Szeredi
@ 2015-05-13 15:35 ` Vincent Batts
2015-05-14 10:43 ` David Howells
2015-05-14 10:46 ` David Howells
2 siblings, 0 replies; 5+ messages in thread
From: Vincent Batts @ 2015-05-13 15:35 UTC (permalink / raw)
To: Miklos Szeredi; +Cc: David Howells, Josh Boyer, linux-unionfs
On Wed, May 13, 2015 at 11:15 AM, Miklos Szeredi <miklos@szeredi.hu> wrote:
> ---
> tests/rmdir.py | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> --- a/tests/rmdir.py
> +++ b/tests/rmdir.py
> @@ -203,3 +203,19 @@ from errno import *
> ctx.rmdir(sym, err=ENOTDIR)
> ctx.rmdir(sym, err=ENOTDIR)
> ctx.rmdir(d, err=ENOENT)
> +
> +# Remove an empty lower directory, recreate, populate and try to remove
> +def subtest_16(ctx):
> + """Remove non-empty opaque directory"""
> + d = ctx.empty_dir() + ctx.termslash()
> + f = d + "/b"
> +
> + ctx.rmdir(d)
> + ctx.rmdir(d, err=ENOENT)
> + ctx.mkdir(d, 0o755)
> + ctx.open_file(f, wo=1, crt=1, ex=1, write="abcq")
> + ctx.rmdir(d, err=ENOTEMPTY)
> + ctx.unlink(f)
> + ctx.open_file(f, ro=1, err=ENOENT)
> + ctx.unlink(f, err=ENOENT)
> + ctx.rmdir(d)
I'm not familiar with this testsuite. The empty_dir will be on the
lower level, and all the rmdir, mkdir and create are happening on a
mounted union fs? I don't see where that is clear.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir
2015-05-13 15:15 [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir Miklos Szeredi
2015-05-13 15:35 ` Vincent Batts
@ 2015-05-14 10:43 ` David Howells
2015-05-14 10:46 ` David Howells
2 siblings, 0 replies; 5+ messages in thread
From: David Howells @ 2015-05-14 10:43 UTC (permalink / raw)
To: Miklos Szeredi; +Cc: dhowells, Josh Boyer, Vincent Batts, linux-unionfs
Applied and pushed. I've also pushed your overlayfs->overlay patch.
David
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir
2015-05-13 15:15 [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir Miklos Szeredi
2015-05-13 15:35 ` Vincent Batts
2015-05-14 10:43 ` David Howells
@ 2015-05-14 10:46 ` David Howells
2015-05-17 20:45 ` Vincent Batts
2 siblings, 1 reply; 5+ messages in thread
From: David Howells @ 2015-05-14 10:46 UTC (permalink / raw)
To: Vincent Batts; +Cc: dhowells, Miklos Szeredi, Josh Boyer, linux-unionfs
Vincent Batts <vbatts@gmail.com> wrote:
> I'm not familiar with this testsuite. The empty_dir will be on the
> lower level, and all the rmdir, mkdir and create are happening on a
> mounted union fs? I don't see where that is clear.
That's handled by the core of the test infrastructure:
http://git.infradead.org/users/dhowells/unionmount-testsuite.git
Look in the base of the git tree.
The infrastructure creates a load of files and other objects then sets up the
requested union (unionmount, overlayfs or even no union at all) over them and
then runs all the tests.
David
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir
2015-05-14 10:46 ` David Howells
@ 2015-05-17 20:45 ` Vincent Batts
0 siblings, 0 replies; 5+ messages in thread
From: Vincent Batts @ 2015-05-17 20:45 UTC (permalink / raw)
To: David Howells; +Cc: Miklos Szeredi, Josh Boyer, linux-unionfs
On Thu, May 14, 2015 at 6:46 AM, David Howells <dhowells@redhat.com> wrote:
> Vincent Batts <vbatts@gmail.com> wrote:
>
>> I'm not familiar with this testsuite. The empty_dir will be on the
>> lower level, and all the rmdir, mkdir and create are happening on a
>> mounted union fs? I don't see where that is clear.
>
> That's handled by the core of the test infrastructure:
>
> http://git.infradead.org/users/dhowells/unionmount-testsuite.git
>
> Look in the base of the git tree.
>
> The infrastructure creates a load of files and other objects then sets up the
> requested union (unionmount, overlayfs or even no union at all) over them and
> then runs all the tests.
>
> David
RIght on. Thanks.
vb
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-05-17 20:45 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-13 15:15 [PATCH] unionmount-testsuite: test rmdir of non-empty opaque dir Miklos Szeredi
2015-05-13 15:35 ` Vincent Batts
2015-05-14 10:43 ` David Howells
2015-05-14 10:46 ` David Howells
2015-05-17 20:45 ` Vincent Batts
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.