Fix check-inodes.sh script.

[jeffmcutter]

Output from running analysis role:

line 36: badinitio: unbound variable

Output testing script manually:

[root@satrhel6-1 ~]# /tmp/check-inodes.sh 
/tmp/check-inodes.sh: line 36: badinitio: unbound variable
[root@satrhel6-1 ~]# bash /tmp/check-inodes.sh 
/tmp/check-inodes.sh: line 36: exit: : numeric argument required

[jeffmcutter]

@djdanielsson Can you look at this one? #202 broke check-inodes.sh (see description above). Thanks!

[djdanielsson]

Hum... I wondered if those changes would break it. I will have to figure out how to make it skip this in sanity check

[djdanielsson]

I'll try and fix this

[swapdisk]

The "unbound variable" error is because $badinitio is unset if the report shortfalls loop doesn't find any problems. It was fine until -u bash option was added. Per bash man page, that option tells bash to treat unset variables as an error...

-u   Treat unset variables and parameters other than the special para‐
     meters "@" and "*", or array variables subscripted  with  "@"  or
     "*",  as an error when performing parameter expansion.  If expan‐
     sion is attempted on an unset variable or  parameter,  the  shell
     prints  an  error  message, and, if not interactive, exits with a
     non-zero status.

Let me recommend we just add badinitio=0 before the loop at line 23 and we should be able to leave everything else as it was and not need to do the sanity test ignores.

[djdanielsson]

yep I figured that out and did just that

[jeffmcutter]

Pulled the latest main and ran analysis successfully on a RHEL 6 box. :-)