community.zabbix icon indicating copy to clipboard operation
community.zabbix copied to clipboard

introduce string representations for zabbix_host parameters

Open D3DeFi opened this issue 3 years ago • 1 comments

SUMMARY

tl;dr instead of having

...
- zabbix_host:
    tls_accept: 1
    tls_connect: 1
    interfaces:
      - main: 1
        useip: 0
...

It would be much prettier to have:

...
- zabbix_host:
    tls_accept: certificate
    tls_connect: PSK
    interfaces:
      - main: true
        useip: false
...

But this needs to be introduced in a way that will NOT break backwards compatibility as everyone is using those integers currently.

For example, zabbix_proxy plugin has some values like this already (for example tls_accept).

And there may be more modules that will needs this adjustment.

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

plugins/modules/zabbix_host.py maybe others

D3DeFi avatar Nov 28 '21 13:11 D3DeFi

@D3DeFi In my opinion, We can stop support for non-readable values. Usually, we can use this collection with selecting version(e.g. 2.0.0). People can set non-readable values with old-version if they would like to use non-readable values for parameter.

On the other hand, I think we need to create <resource>_info modules to improve usability.

masa-orca avatar May 05 '23 15:05 masa-orca