Feat(cv_device_v3): Add "inventory_mode" to allow ignoring missing devices

[ClausHolbechArista]

Change Summary

Add "inventory_mode" to allow ignoring missing devices

Related Issue(s)

Fixes #579

Component(s) name

arista.cvp.cv_device_v3

Proposed changes

• inventory_mode: Define how missing devices are handled.
  • loose: Ignore missing devices.
  • strict (default): Fail on any missing device.

How to test

Tested manually.

With default inventory_mode: strict:

fatal: [CV_AVD_FABRIC]: FAILED! => {"changed": false, "msg": "Error - the following devices do not exist in CVP ['spine2'] but are defined in the playbook.                 \nMake sure that the devices are provisioned and defined with the full fqdn name                 (including the domain name) if needed."}

With inventory_mode: loose:

ok: [CV_AVD_FABRIC]

Checklist

User Checklist

• N/A

Repository Checklist

• [x] My code has been rebased from devel before I start
• [x] I have read the CONTRIBUTING document.
• [x] My change requires a change to the documentation and documentation have been updated accordingly. (check the box if not applicable)
• [x] I have updated molecule CI testing accordingly. (check the box if not applicable)

[github-actions[bot]]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

[github-actions[bot]]

Conflicts have been resolved. A maintainer will review the pull request shortly.

[github-actions[bot]]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

[github-actions[bot]]

Conflicts have been resolved. A maintainer will review the pull request shortly.

[sugetha24]

@ClausHolbechArista Question:

1. when we have inventory_mode: loose, there is no warning message in the return data structure that warns the user that one of the missing device was ignored. This might be useful to add to give user feedback about what passed and what didn't.

[ClausHolbechArista]

@ClausHolbechArista Question:

1. when we have inventory_mode: loose, there is no warning message in the return data structure that warns the user that one of the missing device was ignored. This might be useful to add to give user feedback about what passed and what didn't.

I agree that we could give the user some information, and I will add that. But I don't think it should be a warning, since the user specifically configured the loose mode (they expect some devices to be missing). So I suggest to add it as info-level output.