Storage plugin updates

[alexandraBara]

Updating storage to also check for % free space.
Both abs and prct args are optional. If neither is provided, prct will default to 10% free.

 min_required_free_space_abs: Optional[str] = None
 min_required_free_space_prct: int = None

min_required_free_space_prct defaults to 10% in storage_analyzer

[hearnsj-amd]

Looks good to me. 10% free is a good value to warn at - for instance CEPH filesystems stop working at 5% free

@landrews-amd I do not fully understand "...supporting a None value for free space percentage would be good ... to continue to support the current behavior that is used internally."
No problem with a None value - just curious.

[alexandraBara]

Looks good to me. 10% free is a good value to warn at - for instance CEPH filesystems stop working at 5% free

@landrews-amd I do not fully understand "...supporting a None value for free space percentage would be good ... to continue to support the current behavior that is used internally." No problem with a None value - just curious.

At first we had hardcoded 10% to be the default prct, but we are now only setting it to 10% when the user doesnt specify abs or prct values. That way, if the user only wants to check an abs value and not care about prct - that still holds. This allows us to keep parity with the current behavior, which the internal tool has.

[landrews-amd]

LGTM