[sheepdog] [PATCH] configure: fix a stale sheep version output

2013-12-19 Thread Liu Yuan
This fixes a possible stale out of 'sheep -v' Signed-off-by: Liu Yuan --- autogen.sh |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/autogen.sh b/autogen.sh index 83770a1..4bc608d 100755 --- a/autogen.sh +++ b/autogen.sh @@ -2,4 +2,4 @@ # Run this to generate all the init

Re: [sheepdog] [PATCH] configure: fix a stale sheep version output

2013-12-19 Thread Hitoshi Mitake
At Thu, 19 Dec 2013 16:00:24 +0800, Liu Yuan wrote: > > This fixes a possible stale out of 'sheep -v' > > Signed-off-by: Liu Yuan > --- > autogen.sh |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Looks good to me. Reviewed-by: Hitoshi Mitake Thanks, Hitoshi -- sheepdog mailing l

Re: [sheepdog] [PATCH] configure: fix a stale sheep version output

2013-12-19 Thread Liu Yuan
On Thu, Dec 19, 2013 at 04:00:24PM +0800, Liu Yuan wrote: > This fixes a possible stale out of 'sheep -v' > > Signed-off-by: Liu Yuan > --- > autogen.sh |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/autogen.sh b/autogen.sh > index 83770a1..4bc608d 100755 > --- a/au

[sheepdog] [PATCH stable-0.7] configure: fix a stale sheep version output

2013-12-19 Thread Hitoshi Mitake
From: Liu Yuan This fixes a possible stale out of 'sheep -v' Reviewed-by: Hitoshi Mitake Signed-off-by: Liu Yuan --- autogen.sh |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/autogen.sh b/autogen.sh index 83770a1..4bc608d 100755 --- a/autogen.sh +++ b/autogen.sh @@ -2,

[sheepdog] [PATCH stable-0.7] update for version printing

2013-12-19 Thread Hitoshi Mitake
In some cases, current sheep prints an invalid version number when it receives the option '-v'. This behavior is harmful for bug and trouble reporting. This update fixes it. Liu Yuan (1): configure: fix a stale sheep version output autogen.sh |2 +- 1 file changed, 1 insertion(+), 1 deleti

Re: [sheepdog] [sheepdog-users] [PATCH stable-0.7] update for version printing

2013-12-19 Thread Liu Yuan
On Thu, Dec 19, 2013 at 07:02:05PM +0900, Hitoshi Mitake wrote: > In some cases, current sheep prints an invalid version number when it > receives the option '-v'. This behavior is harmful for bug and trouble > reporting. This update fixes it. > > Liu Yuan (1): > configure: fix a stale sheep ver

[sheepdog] [ANNOUNCE] sheepdog stable release v0.7.6-rc2

2013-12-19 Thread Hitoshi Mitake
Hi sheepdog users and developers, I released v0.7.6-rc2 of stable branch. You can download a source archive from these URLs: tar.gz: https://github.com/sheepdog/sheepdog/archive/v0.7.6-rc2.tar.gz zip: https://github.com/sheepdog/sheepdog/archive/v0.7.6-rc2.zip If no one disagrees about this rele

Re: [sheepdog] [PATCH] sheep/http: remove the kv_update_object

2013-12-19 Thread Robin Dong
Reviewed-by: Robin Dong 2013/12/19 Liu Yuan > Both swift and s3 don't have API to update the existing object, it purely > overwrite the whole objects. So no need to maintain this unused code. > > Signed-off-by: Liu Yuan > --- > sheep/http/http.h |2 -- > sheep/http/kv.c | 84 > --

Re: [sheepdog] [PATCH] sheep/http: remove the kv_update_object

2013-12-19 Thread Liu Yuan
On Thu, Dec 19, 2013 at 06:23:13PM +0800, Robin Dong wrote: > Reviewed-by: Robin Dong > > > 2013/12/19 Liu Yuan > > > Both swift and s3 don't have API to update the existing object, it purely > > overwrite the whole objects. So no need to maintain this unused code. > > > > Signed-off-by: Liu Y

[sheepdog] [PATCH] bs_sheepdog.c: add missing initialization of request header

2013-12-19 Thread Ryusuke Konishi
The create_branch function of the sheepdog driver sends a partially uninitialized request to sheepdog. This fixes the defect. Signed-off-by: Ryusuke Konishi Cc: Hitoshi Mitake --- usr/bs_sheepdog.c |1 + 1 file changed, 1 insertion(+) diff --git a/usr/bs_sheepdog.c b/usr/bs_sheepdog.c ind

Re: [sheepdog] [PATCH] bs_sheepdog.c: add missing initialization of request header

2013-12-19 Thread Hitoshi Mitake
At Thu, 19 Dec 2013 20:16:17 +0900, Ryusuke Konishi wrote: > > The create_branch function of the sheepdog driver sends a partially > uninitialized request to sheepdog. This fixes the defect. > > Signed-off-by: Ryusuke Konishi > Cc: Hitoshi Mitake > --- > usr/bs_sheepdog.c |1 + > 1 file c

Re: [sheepdog] [PATCH] bs_sheepdog.c: add missing initialization of request header

2013-12-19 Thread FUJITA Tomonori
On Thu, 19 Dec 2013 20:16:17 +0900 Ryusuke Konishi wrote: > The create_branch function of the sheepdog driver sends a partially > uninitialized request to sheepdog. This fixes the defect. > > Signed-off-by: Ryusuke Konishi > Cc: Hitoshi Mitake > --- > usr/bs_sheepdog.c |1 + > 1 file cha

Re: [sheepdog] [PATCH] bs_sheepdog.c: add missing initialization of read data length

2013-12-19 Thread FUJITA Tomonori
On Thu, 19 Dec 2013 14:46:58 +0900 Ryusuke Konishi wrote: > In the sheepdog driver, the read data length argument of do_req > function is not initialized by some caller functions such as > sd_sync(), sd_close(), and create_branch(). > > This patch adds those missing initializations to prevent un

[sheepdog] [PATCH] dog: let "dog vdi cache" subcommands parse arguments in a correct manner

2013-12-19 Thread Hitoshi Mitake
Current subcommands of "dog vdi cache" can cause segmentation faults because they don't check an index of argv. Signed-off-by: Hitoshi Mitake --- dog/vdi.c | 22 +++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a/dog/vdi.c b/dog/vdi.c index c716589..b6c955c 1

Re: [sheepdog] [PATCH 1/2] sheep: introduce strict mode for write

2013-12-19 Thread Hitoshi Mitake
At Fri, 13 Dec 2013 05:41:03 +0900, MORITA Kazutaka wrote: > > At Thu, 12 Dec 2013 14:37:23 +0800, > Liu Yuan wrote: > > > > > > > > > > Ah, yes. I think SD_FLAG_CMD_WRITE is kind of misleading (couple of > > > > times I > > > > forgot to assigne it while write something and spent time debuggin

Re: [sheepdog] [PATCH] dog: let "dog vdi cache" subcommands parse arguments in a correct manner

2013-12-19 Thread Liu Yuan
On Fri, Dec 20, 2013 at 10:56:34AM +0900, Hitoshi Mitake wrote: > Current subcommands of "dog vdi cache" can cause segmentation faults > because they don't check an index of argv. > I tried every subcommand for 'vdi cache' and don't get the segfault, which exactly commond cause the segfalt? Than

Re: [sheepdog] [PATCH] dog: let "dog vdi cache" subcommands parse arguments in a correct manner

2013-12-19 Thread Hitoshi Mitake
At Fri, 20 Dec 2013 11:47:07 +0800, Liu Yuan wrote: > > On Fri, Dec 20, 2013 at 10:56:34AM +0900, Hitoshi Mitake wrote: > > Current subcommands of "dog vdi cache" can cause segmentation faults > > because they don't check an index of argv. > > > > I tried every subcommand for 'vdi cache' and don

Re: [sheepdog] [PATCH] dog: let "dog vdi cache" subcommands parse arguments in a correct manner

2013-12-19 Thread Liu Yuan
On Fri, Dec 20, 2013 at 01:09:02PM +0900, Hitoshi Mitake wrote: > At Fri, 20 Dec 2013 11:47:07 +0800, > Liu Yuan wrote: > > > > On Fri, Dec 20, 2013 at 10:56:34AM +0900, Hitoshi Mitake wrote: > > > Current subcommands of "dog vdi cache" can cause segmentation faults > > > because they don't check

[sheepdog] [PATCH stable-0.7] dog: let "dog vdi cache" subcommands parse arguments in a correct manner

2013-12-19 Thread Hitoshi Mitake
Current subcommands of "dog vdi cache" can cause segmentation faults because they don't check an index of argv. Signed-off-by: Hitoshi Mitake Signed-off-by: Liu Yuan Conflicts: dog/vdi.c Conflicts were resolved by Hitoshi Mitake. Signed-off-by: Hitoshi Mitake --- dog/vdi.c | 20 +++

[sheepdog] [PATCH stable-0.7] update for segfault of dog command

2013-12-19 Thread Hitoshi Mitake
This update removes segfault in subcommands of "dog vdi cache". Hitoshi Mitake (1): dog: let "dog vdi cache" subcommands parse arguments in a correct manner dog/vdi.c | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) -- 1.7.10.4 -- sheepdog mailing list shee

[sheepdog] [PATCH] tests/func: use rc4 cipher command for openssl

2013-12-19 Thread Liu Yuan
On ubuntu 13.04, _random() always cause segfault because of it triggers a bug for aes-128-cbc cipher command. By using rc4, it achieves the same throughput as aes-128-cbc and workaround the segfault. Signed-off-by: Liu Yuan --- tests/functional/common.rc | 2 +- 1 file changed, 1 insertion(+),

Re: [sheepdog] [PATCH] tests/func: use rc4 cipher command for openssl

2013-12-19 Thread Hitoshi Mitake
At Fri, 20 Dec 2013 12:41:08 +0800, Liu Yuan wrote: > > On ubuntu 13.04, _random() always cause segfault because of it triggers a bug > for aes-128-cbc cipher command. > > By using rc4, it achieves the same throughput as aes-128-cbc and workaround > the > segfault. > > Signed-off-by: Liu Yuan

Re: [sheepdog] [PATCH stable-0.7] update for segfault of dog command

2013-12-19 Thread Liu Yuan
On Fri, Dec 20, 2013 at 01:38:27PM +0900, Hitoshi Mitake wrote: > This update removes segfault in subcommands of "dog vdi cache". > > Hitoshi Mitake (1): > dog: let "dog vdi cache" subcommands parse arguments in a correct > manner > > dog/vdi.c | 20 ++-- > 1 file changed

[sheepdog] [PATCH v4 1/4] cluster/local: support lock/unlock

2013-12-19 Thread Liu Yuan
Signed-off-by: Liu Yuan --- sheep/cluster/local.c | 95 +-- 1 file changed, 93 insertions(+), 2 deletions(-) diff --git a/sheep/cluster/local.c b/sheep/cluster/local.c index 6d0af68..3a89f19 100644 --- a/sheep/cluster/local.c +++ b/sheep/cluster/lo

[sheepdog] [PATCH v4 2/4] tests/func: add nginx.conf

2013-12-19 Thread Liu Yuan
This is needed for later test cases to test http services. It is convenient to include it in the tests and make tests self-contained. Signed-off-by: Liu Yuan --- tests/functional/nginx.conf | 97 + 1 file changed, 97 insertions(+) create mode 100644 t

[sheepdog] [PATCH v4 3/4] tests/func: add 081 to test http service

2013-12-19 Thread Liu Yuan
Current master can pass it with local driver, but zk 3.4.5 can pass it too Signed-off-by: Liu Yuan --- tests/functional/081 | 104 + tests/functional/081.out | 69 +++ tests/functional/common.config | 1 + tests/f

[sheepdog] [PATCH v4 0/4] add tests for http services

2013-12-19 Thread Liu Yuan
This patches implement local lock/unlock to assist test http servies and add 2 test cases which stress parallel requests. Local driver can pass it but zk driver will fail both cases, I suspect zk has bugs in its lock implementation. v4: - update comment for tests since zk 3.4.5 can pass it Liu

[sheepdog] [PATCH v4 4/4] tests/func: add 082 to test http services with multiple servers

2013-12-19 Thread Liu Yuan
Current local deiver can pass it and zk 3.4.5 can pass it too. Signed-off-by: Liu Yuan --- tests/functional/082 | 112 +++ tests/functional/082.out | 85 +++ tests/functional/group | 1 + 3 files changed, 198 in

Re: [sheepdog] [PATCH] tests/func: use rc4 cipher command for openssl

2013-12-19 Thread Liu Yuan
On Fri, Dec 20, 2013 at 12:41:08PM +0800, Liu Yuan wrote: > On ubuntu 13.04, _random() always cause segfault because of it triggers a bug > for aes-128-cbc cipher command. > > By using rc4, it achieves the same throughput as aes-128-cbc and workaround > the > segfault. > > Signed-off-by: Liu Yua

[sheepdog] [PATCH stable-0.7] update for test scripts

2013-12-19 Thread Hitoshi Mitake
Some of test scripts can cause segmentation fault because of a bug in openssl on Ubuntu 13.04. This update fixes this problem. Liu Yuan (1): tests/func: use rc4 cipher command for openssl tests/functional/common.rc |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 1.7.10.4 -- she

[sheepdog] [PATCH stable-0.7] tests/func: use rc4 cipher command for openssl

2013-12-19 Thread Hitoshi Mitake
From: Liu Yuan On ubuntu 13.04, _random() always cause segfault because of it triggers a bug for aes-128-cbc cipher command. By using rc4, it achieves the same throughput as aes-128-cbc and workaround the segfault. Reviewed-by: Hitoshi Mitake Signed-off-by: Liu Yuan --- tests/functional/comm

[sheepdog] [ANNOUNCE] sheepdog stable release v0.7.6-rc3

2013-12-19 Thread Hitoshi Mitake
Hi sheepdog users and developers, I released v0.7.6-rc3 of stable branch. You can download a source archive from these URLs: tar.gz: https://github.com/sheepdog/sheepdog/archive/v0.7.6-rc3.tar.gz zip: https://github.com/sheepdog/sheepdog/archive/v0.7.6-rc3.zip This update contains an important f

Re: [sheepdog] [PATCH stable-0.7] update for test scripts

2013-12-19 Thread Liu Yuan
On Fri, Dec 20, 2013 at 02:40:25PM +0900, Hitoshi Mitake wrote: > Some of test scripts can cause segmentation fault because of a bug in > openssl on Ubuntu 13.04. This update fixes this problem. > > Liu Yuan (1): > tests/func: use rc4 cipher command for openssl > > tests/functional/common.rc |