| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
Currently the default logdir is DATADIR /log/gluster-block/
This patch will provide a way to change this default logdir via Env variable
$ export GB_LOGDIR=/var/log/gluster-block-new-path/
Note: make sure to restart the processes (cli & daemon) after you set GB_LOGDIR
Change-Id: Id142e4a4dfe7b6ebc9cf8296b8ceb8bff37691b8
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
$ gluster-block help
gluster-block (0.2.1)
usage:
gluster-block <command> <volname[/blockname]> [<args>] [--json*]
commands:
[...]
delete <volname/blockname> [force]
delete block device.
[...]
Change-Id: I64ac01ec148e2e1d4d0ba0d4c5560df9334d58f5
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: If729b80a8139add989170d3b590e92706128c37e
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In the previous patch,
`commit f2e46779c0175e5063ce07256023f1977b333f2d
Author: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
Date: Fri Jul 7 15:51:38 2017 +0530
tcmu: backstore attribute set cmd_time_out=0
[...]`
unintentionally saveconfig was left inside the loop. Hence fixing this by
pushing it outside.
Problem:
Currently, while populating 'exec' variable the following line after saveconfig
i.e. at line GB_FREE(tmp); the 'tmp' variable is made NULL; Hence in each
loop, condition if(!tmp) is always true, and the 'else' set of statements will
never be executed.
Impact:
On block create with HA > 1, only for Nth(last) node portal gets created for
rest of the nodes portal and their attr will never be created/set. As a
result block is not exported correctly.
Also, remove duplicated DEVNULLPATH redirection with attr setting
Change-Id: I1f77461de6b89af4e4af098e0444ea3526669030
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
currently, runner is not supporting the non-zero cmd timeouts (cmd_time_out !=0)
The tcmu cmd timer just fails the kernel command after cmd_time_out seconds,
leaving the runner command running. When it is zero, it means do not run the
timer, so both the kernel and runner wait for the command to complete.
This patch sets the backstore attribute cmd_time_out=0
Change-Id: Ib7a80c82d0aafe345ff4e79ce88dc92848306f4e
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
$ gluster-block create
Inadequate arguments for create:
gluster-block create <volname/blockname> [ha <count>] [auth<enable|disable>] \
[prealloc <full|no>] <HOST1[,HOST2,...]> <size> [--json*]
Change-Id: Ib089598fd9b386bd3475dd421b41c45ebb01c9d2
Fixes: #33
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
As of now, if an EntryCreate (backend storage create) fails for some reasons
like no space left, we only clean storage, leaving metafile.
This patch will delete metafile along with backend storage from the volume.
Change-Id: I89247b98602c5b991367f671134de496c294f984
Fixes: #32
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
The following warning is reported with Fedora Rawhide (F-27) that comes
with an updated gcc version (7.1.1):
utils.c:217:21: warning: '*' in boolean context, suggest '&&' instead [-Wint-in-bool-context]
if (!tmp && (size * count)) {
~~~~~~^~~~~~~~
Change-Id: If1645e0e1cc1eaa1d8261914918c5a5be13d6dd8
Signed-off-by: Niels de Vos <ndevos@redhat.com>
|
|
|
|
|
| |
Change-Id: If7a232e4a1d550b3912402282c21d0d033aa00d2
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
systemd-unit:
iscsid.service is needed by initiator not target
spec:
update with missing dependencies
Change-Id: I58bd3d8b60fac10eeb2a7d5cd4fc6f827da1d7b4
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch fixes rpcgen file stat issue.
rpcgen, before generating a file, it first stats the file, in case if file
already exists, it do not try to override, instead bails out.
[...]
Making all in rpcl
make[3]: Entering directory
`/builddir/build/BUILD/gluster-block-0.2.1/rpc/rpcl'
rpcgen -h -o ../../rpc/rpcl/block.h block.x
file `../../rpc/rpcl/block.h' already exists and may be overwritten
make[3]: Leaving directory
`/builddir/build/BUILD/gluster-block-0.2.1/rpc/rpcl'
make[3]: *** [block.h] Error 1
[...]
So the fix will be not to package the rpcgen generated file in the
source tarball.
Change-Id: I1d136a25c49aeea2d84e96bf065fe46bf0214a13
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: I65ad54381362280d9a5596e48aa3242f1160fd2d
Signed-off-by: Michael Adam <obnox@redhat.com>
|
|
|
|
|
| |
Change-Id: I4fc56e3fdcbccfde3a70a98f2697744444797e06
Signed-off-by: Vijay Bellur <vbellur@redhat.com>
|
|
|
|
|
| |
Change-Id: I06f20a355453d81955d4308ecad68358fa1f99cc
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
| |
added 'device-mapper-multipath' package in the initiator install dependency list,
which provides 'mpathconf'
Change-Id: I8ff5ed1e10cb3cac93511afa2eec51c9401bd32f
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
| |
currently we allocate sparse files for block backends in the gluster
volume, with 'prealloc = full' option introduced by this patch we should
be able to fully preallocate the backend block file.
Change-Id: Ibf32df5f978f732a3fd248693170463da6d08268
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: I266e081dc318c1ccf29751059992beeadb9d8665
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
| |
check if the child terminated normally before checking for exit status
of the child.
Change-Id: I00357f636f4ee55c10052b4e6071cb08493d32e2
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
| |
In cases when svc_register fails, gluster-blockd will print the
reason and terminate gracefully.
Fixes: #26
Change-Id: Ia243771d99fbaa674f317e7c8e64174f4d8d6234
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also s/BLOCK CONFIG NODE(S)/EXPORTED NODE(S)/ in the info output
$ gluster-block info sample/block
NAME: block
VOLUME: sample
GBID: 6bd70984-be2c-43ac-9e9d-bad04010e42f
SIZE: 1.0 GiB
HA: 1
PASSWORD:
EXPORTED NODE(S): 192.168.0.105
Change-Id: I473b854b939c96b99be8e0b172ac6957b8bc6006
Fixes: #23
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
|
|
| |
Change-Id: Ic6fcb5cc6585fb66099d56175aa80da9275356ae
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
| |
set default log-level environment variable in daemon systemd unit, also
added required comment lines in the sysconfig file.
Change-Id: I9f61842ba95ae278367692eba8e62cc54c1b5cd3
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
| |
one can edit various input options to gluster-blockd daemon via
/etc/sysconfig/gluster-blockd
Change-Id: I720130d6b6599a79d193c5cbfff18921ffb07156
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With this change, we can further narrow down the reason for command
failure on remote node.
We get to know if:
* targetcli is installed
* tcmu-runner is running
* user:glfs is listed
Change-Id: Ib39ec525f951ca510008327b59ab99d8f7645ced
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: Ic8a7d584274f48dcef5ea2356cf317352a3d48ef
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: I7c78ab124705d77c8dc5edbf7ca7829dbf0f07b2
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the asan trace:
==26769==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x602000007b60 at pc 0x7ffff6e9429b bp 0x7ffff1afd800 sp 0x7ffff1afcfa8
WRITE of size 2 at 0x602000007b60 thread T1
#0 0x7ffff6e9429a (/lib64/libasan.so.3+0x5f29a)
#1 0x406b64 in removeDuplicateSubstr /root/gluster-block/rpc/block_svc_routines.c:147
#2 0x41ad4c in blockCreateCliFormatResponse /root/gluster-block/rpc/block_svc_routines.c:1570
#3 0x41ff91 in block_create_cli_1_svc /root/gluster-block/rpc/block_svc_routines.c:1826
#4 0x405e63 in gluster_block_cli_1 /root/gluster-block/rpc/rpcl/block_svc.c:132
#5 0x7ffff57d62a0 in svc_getreq_common (/lib64/libc.so.6+0x13a2a0)
#6 0x7ffff57d63e6 in svc_getreq_poll (/lib64/libc.so.6+0x13a3e6)
#7 0x7ffff57d9d00 in svc_run (/lib64/libc.so.6+0x13dd00)
#8 0x403c61 in glusterBlockCliThreadProc /root/gluster-block/daemon/gluster-blockd.c:130
#9 0x7ffff6c1e6c9 in start_thread (/lib64/libpthread.so.0+0x76c9)
#10 0x7ffff57a3f6e in clone (/lib64/libc.so.6+0x107f6e)
0x602000007b60 is located 0 bytes to the right of 16-byte region [0x602000007b50,0x602000007b60)
allocated by thread T1 here:
#0 0x7ffff6efc020 in calloc (/lib64/libasan.so.3+0xc7020)
#1 0x42d2c3 in gbAllocN /root/gluster-block/utils/utils.c:194
#2 0x406ae9 in removeDuplicateSubstr /root/gluster-block/rpc/block_svc_routines.c:138
#3 0x41ad4c in blockCreateCliFormatResponse /root/gluster-block/rpc/block_svc_routines.c:1570
#4 0x41ff91 in block_create_cli_1_svc /root/gluster-block/rpc/block_svc_routines.c:1826
#5 0x405e63 in gluster_block_cli_1 /root/gluster-block/rpc/rpcl/block_svc.c:132
#6 0x7ffff57d62a0 in svc_getreq_common (/lib64/libc.so.6+0x13a2a0)
#7 0x7ffff5a60a5f (/lib64/libc.so.6+0x3c4a5f)
Thread T1 created by T0 here:
#0 0x7ffff6e66488 in __interceptor_pthread_create (/lib64/libasan.so.3+0x31488)
#1 0x40545f in main /root/gluster-block/daemon/gluster-blockd.c:325
#2 0x7ffff56bc400 in __libc_start_main (/lib64/libc.so.6+0x20400)
Change-Id: Id16fa0c00223f7272c3c977efb268ba5d72bd04b
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
When reply->out is NULL, gluster-blockd is crashing in serializing the response.
Unfortunately the function where we can handle this error is auto-generated, so
I added catch-all errors with a generic-message if errMsg is not set.
Also improved errMsg in somecases where it is not setting by collecting the
error messages from all the config-nodes.
Change-Id: I706d8af9e8e6140d21c0d1268ae57dd5f364aa54
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
|
|
| |
Change-Id: I9dd396737a6719fe3828ec3e9c26bfd8aa1da238
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
|
|
| |
Change-Id: I8c8cf5f179d0cacb2dc9ffbdc6ed6c5bbcc04abb
Signed-off-by: Michael Adam <obnox@redhat.com>
|
|
|
|
|
| |
Change-Id: Ib3e865c171558a1aa81223b7d63c362159cbb53c
Signed-off-by: Michael Adam <obnox@redhat.com>
|
|
|
|
|
| |
Change-Id: I3641174aaff35406cd74eb088267d76091eb0b03
Signed-off-by: Michael Adam <obnox@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before:
-------
Epoch number, eg: [1496312036]
Now:
----
Human readable, eg: [2017-06-01 18:37:46.834377]
Change-Id: Ibcb3de512086f6fa36ad7169f0d9189636361331
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds '--log-level' options
supported log levels:
{
"NONE",
"ERROR",
"WARNING",
"INFO",
"DEBUG",
"TRACE"
}
TRACE being max logging
$ gluster-blockd --help
[...]
--log-level <LOGLEVEL>
Logging severity. Valid options are,
TRACE, DEBUG, INFO, WARNING, ERROR and NONE [default: INFO]
[...]
Change-Id: I5a387eea3e7e7be10a0b56102a61eb81b4ebf2fa
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: I7a2f8804b7d33c58101bbfb58dcff15370239311
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
$ gluster-blockd --help
gluster-blockd (0.2)
usage:
gluster-blockd [--glfs-lru-count <count>]
commands:
--glfs-lru-count <count>
glfs objects cache capacity [max: 512] (default: 5)
--help
show this message and exit.
--version
show version info and exit.
Change-Id: I00a9277690a1c5ace51e223e9e4ed9ce61ae2428
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Problem:
-------
1. Currently, each cli command take ~5 secs for execution. The Maximum
latency is due to initializing a glfs object (glfs_init() and friends).
2. OOM kills due to glfs_fini() leaks (~10MB per object)
Solution:
--------
Caching bipasses glfs_init() calls from the very next command, as in the first
command it goes via the glfs_init route, since there will be cache miss.
Hence with caching cli command on a local machine should take ~1 sec.
ATM, the cache query looks at the volume name only, as the host name will be
localhost in our case and transport will be tcp 24007 always.
The default cache capacity is 5 i.e there can be a max of five glfs entries in
the cache, anything more will lead to release of least recently used object.
This way, if there are <= 5 volume in use for block, there will be no
glfs_fini() calls, hence no leaks, no OOM's.
The next patch will help in making cache capacity configurable.
Change-Id: Ia891451cb92cf09959c1aff85976d78302ec7014
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
[ndevos: correct compiling+linking against libgfapi.so]
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Problem:
When the config node is given as an address that has no route,
connection to the node fails with EHOSTUNREACH(No route to Host).
Since the code doesn't treat it as special errno, the config
ends up updating its status as CONFIGFAIL and as part of cleanup
it attempts to delete it, but fails again because the host is
not reachable. So the block file lingers on in that state forever.
Fix:
Irrespective of errno, unless we don't send an rpc we know that
we don't need to move the status to CONFIGFAIL, so get that info
and act accordingly irrespective of the errno.
Change-Id: Ie3fb9413f089a4875b5612c60ed3d65bc5b5bba1
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
|
|
|
|
| |
Also refactored the code around to reduce number of allocations.
Change-Id: If5431f1504b6716ac4baec3a859189f6f21c80a4
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
status GB_CLEANUP_INPROGRESS means someone attemped a delete, but node was not
in contact/reach. So the configuration on that node is untouched. Hence such
nodes need to be shown in configured list or so called 'BLOCK CONFIG NODE(S):'
as part of info command.
Change-Id: I56f26fdb3fe6e0ac57fae08c58256d8ccb4dc10d
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: I2e0dd64ea1b07d09fa28151a46b703af1d05eed9
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: Icc5d7141338ac81862ecbf1fd20cb28b004fa537
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
==22873==ERROR: AddressSanitizer: heap-buffer-overflow on address
0x6120000224cf at pc 0x7f073b9829b7 bp 0x7f07367fc400 sp 0x7f07367fbba8
WRITE of size 17 at 0x6120000224cf thread T1
#0 0x7f073b9829b6 in strcpy (/lib64/libasan.so.3+0x919b6)
#1 0x424416 in blockStuffMetaInfo /root/gluster-block/rpc/glfs-operations.c:305
#2 0x424bc6 in blockGetMetaInfo /root/gluster-block/rpc/glfs-operations.c:363
Change-Id: Id9da4279d358462dcf1c043b37aeb55dfc38ceb2
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
| |
Signed-off-by: Vijay Bellur <vbellur@redhat.com>
|
|
|
|
|
| |
Change-Id: Ia571462b4d94b3cb3855aa699c1b4303283134ef
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
| |
Also use O_SYNC while creating/truncating block file.
Change-Id: Ic0bcc3f2905c438ca5b5d3e95955bbd5d95d3ef4
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently immediate restarts of gluster-blockd failsi with:
[1495181620] ERROR: bind on port 24006 failed (Address already in use)
Since the closed socket connection is still in TIME_WAIT on 24006
This patch sets socket option SO_REUSEADDR, which allows gluster-blockd to bind
even if socket is in a TIME_WAIT state.
Change-Id: I4e6b3c5d3930e6fca222b11c34b35f883d9be730
Signed-off-by: Vijay Bellur <vbellur@redhat.com>
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: Iac1f2a33bf86cd52ea6e2378cb31a84d40469af9
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
| |
Change-Id: If5f63d7cb711d18739782ebd83495f223f9db0da
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The connect() in daemon takes too long to return for invalid IP addresses,
the delay is much higher that the CLI RPC Timeout, hence the CLI returns
before the daemon returns. Hence SIGPIPE is issued.
The issue can be fixed by increasing the CLI RPC clnt_call() TIMEOUT.
This should actually be handled like:
struct timeval tv;
CLIENT *cl;
cl=clnt_create("somehost", SOMEPROG, SOMEVERS, "tcp");
if (cl=NULL) {
exit(1);
}
tv.tv_sec=60; /* change timeout to 1 minute */
tv.tv_usec=0;
clnt_control(cl, CLSET_TIMEOUT, &tv);
But currently there is a bug in sun RPC which ignores TIMEOUT set using
cln_control. See [1]
Hence using regex to override default TIMEOUT generated in rpc code.
[1] https://lists.gnu.org/archive/html/bug-glibc/2000-10/msg00095.html
Change-Id: I75eceea6b9f362f8bf8c2fcb687155635efc65d7
Fixes: #15
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
|