jt_pool_cmd should always return error code, even
if it failed to add/remove just one of OSTs from list.
Before this patch it returned latest command result,
ignoring previous failures.
Change-Id: Ife6cefc006f061b47a1b00daf826d0d1d34fd66c
Signed-off-by: Sergey Cheremencev <sergey.cheremencev@hpe.com>
Reviewed-on: https://review.whamcloud.com/38960
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Vladimir Saveliev <c17830@cray.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
for (j = 0; j < array_sz; j++) {
char ostname[MAX_OBD_NAME + 1];
+ int rc2;
snprintf(ostname, MAX_OBD_NAME, format,
array[j]);
ostname[MAX_OBD_NAME] = '\0';
- rc = check_and_complete_ostname(fsname,
+ rc2 = check_and_complete_ostname(fsname,
ostname);
- if (rc) {
+ if (rc2) {
free(array);
free(cmds);
if (ostnames_buf)
free(ostnames_buf);
+ rc = rc ? rc : rc2;
goto out;
}
if (ostnames_buf) {