github/open-cas-linux
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix error logging levels

Browse Source 
Signed-off-by: Jan Musial <jan.musial@intel.com>
This commit is contained in:
Jan Musial 2022-02-08 11:14:04 +01:00
parent 3b42355b82
commit 199993d9a4
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
casadm/cas_main.c
View File

@ -1473,7 +1473,7 @@ int io_class_is_missing() {
/* Option is set, check if this option is allowed */ /* Option is set, check if this option is allowed */
mask = (1 << io_class_params.subcmd); mask = (1 << io_class_params.subcmd);
if (0 == (mask & iter->priv)) { if (0 == (mask & iter->priv)) {
cas_printf(LOG_INFO, "Option '%s' is not allowed\n", option_name); cas_printf(LOG_ERR, "Option '%s' is not allowed\n", option_name);
result = -1; result = -1;
} }
@ -1481,7 +1481,7 @@ int io_class_is_missing() {
/* Option is missing, check if it is required for this sub-command*/ /* Option is missing, check if it is required for this sub-command*/
mask = (1 << io_class_params.subcmd) | (1 << io_class_opt_flag_required); mask = (1 << io_class_params.subcmd) | (1 << io_class_opt_flag_required);
if (mask == (iter->priv & mask)) { if (mask == (iter->priv & mask)) {
cas_printf(LOG_INFO, "Option '%s' is missing\n", option_name); cas_printf(LOG_ERR, "Option '%s' is missing\n", option_name);
result = -1; result = -1;
} }
} }
@ -1741,12 +1741,12 @@ int script_command_is_valid() {
if (option_is_set) { if (option_is_set) {
if (!is_option_allowed(option_id)) { if (!is_option_allowed(option_id)) {
cas_printf(LOG_INFO, "Option '%s' is not allowed\n", option_name); cas_printf(LOG_ERR, "Option '%s' is not allowed\n", option_name);
result = FAILURE; result = FAILURE;
} }
} else { } else {
if (is_option_required(option_id)) { if (is_option_required(option_id)) {
cas_printf(LOG_INFO, "Option '%s' is missing\n", option_name); cas_printf(LOG_ERR, "Option '%s' is missing\n", option_name);
result = FAILURE; result = FAILURE;
} }
} }
@ -2053,7 +2053,7 @@ int standby_is_missing() {
/* Option is set, check if this option is allowed */ /* Option is set, check if this option is allowed */
mask = (1 << standby_params.subcmd); mask = (1 << standby_params.subcmd);
if (0 == (mask & iter->priv)) { if (0 == (mask & iter->priv)) {
cas_printf(LOG_INFO, "Option '%s' is not allowed\n", option_name); cas_printf(LOG_ERR, "Option '%s' is not allowed\n", option_name);
result = -1; result = -1;
} }
@ -2061,7 +2061,7 @@ int standby_is_missing() {
/* Option is missing, check if it is required for this sub-command*/ /* Option is missing, check if it is required for this sub-command*/
mask = (1 << standby_params.subcmd) | (1 << standby_opt_flag_required); mask = (1 << standby_params.subcmd) | (1 << standby_opt_flag_required);
if (mask == (iter->priv & mask)) { if (mask == (iter->priv & mask)) {
cas_printf(LOG_INFO, "Option '%s' is missing\n", option_name); cas_printf(LOG_ERR, "Option '%s' is missing\n", option_name);
result = -1; result = -1;
} }
} }