ascii42 / check_vmware_horizon

Icinga/ Nagios check for vmware Horizon
1 stars 0 forks source link

Options documentation improvements #1

Closed abrevers-nttdata closed 1 year ago

abrevers-nttdata commented 1 year ago

The documentation format about options is inconsistent:

-eCS---enable-connectionserver should be -eCS, --enable-connectionserver

-eGW--enable-gateway should be -eGW, --enable-gateway

-eVC--enable-vcenter should be -eVC, --enable-vcenter

-eDB--enable-db should be -eDB, --enable-db

-eAD--enable-ad should be -eAD, --enable-ad

-eRDS--enable-rds should be -eRDS, --enable-rds

-eFA--enable-farms should be -eFA, --enable-farms

-eSA--enable-saml should be -eSA, --enable-saml

-eTS--enable-truesso should be -eTS, --enable-truesso

-ePO--enable-pod should be -ePO, --enable-pod

-A|--enable-all should be -A, --enable-all

abrevers-nttdata commented 1 year ago

Also: -H, --hostname <hostname> seems to be wrong.

-H, --host <hostname> seems to be correct.

(Actually it’s the script that should honor the "--hostname" option, as it’s both aligned with the "--username" option, and also more precise).

ascii42 commented 1 year ago

should be fixed with version 0.1.1