fixed zbx_host idempotency with multiple host interfaces of the same …

[D3DeFi]

…type

SUMMARY

Fixes #389

It seems that loop where we compare existing interfaces to user-provided interfaces is checking each existing interface to ALL user-provided interfaces. Even the ones that are already matched are compared.

Additionally, if match is found, loop continues onward to next interface where it will detect difference, because it always compares interfaces that should not match.

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

plugins/modules/zabbix_host.py
tests/integration/targets/test_zabbix_host/

ADDITIONAL INFORMATION

All versions should be affected by this

[codecov]

Codecov Report

Merging #391 (c1fe8d1) into main (d47718d) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #391      +/-   ##
==========================================
- Coverage   76.71%   76.68%   -0.03%     
==========================================
  Files          21       21              
  Lines        2933     2938       +5     
  Branches      752      753       +1     
==========================================
+ Hits         2250     2253       +3     
  Misses        451      451              
- Partials      232      234       +2     

Impacted Files	Coverage Δ
...ns/community/zabbix/plugins/modules/zabbix_host.py	83.33% <0.00%> (-0.25%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fd235ef...c1fe8d1. Read the comment docs.

[D3DeFi]

@sky-joker can you please give this a brief look?

[sky-joker]

Thanks, @D3DeFi for the patch!
LGTM 👍