vsphere.FilterVMsByX functions have inconsistent return values

[atc0005]

Return values for these functions:

check-vmware/internal/vsphere/vms.go

Line 1059 in f618557

func FilterVMsByID(vms []mo.VirtualMachine, vmID string) (mo.VirtualMachine, int, error) {

check-vmware/internal/vsphere/vms.go

Line 935 in f618557

func FilterVMsByName(vms []mo.VirtualMachine, vmName string) (mo.VirtualMachine, int, error) {

do not match the return values for similar functions:

check-vmware/internal/vsphere/vms.go

Line 1027 in f618557

func FilterVMsByCustomAttributeStatus(vms []VMWithCAs) ([]VMWithCAs, int) {

check-vmware/internal/vsphere/vms.go

Line 978 in f618557

func FilterVMsByCustomAttributeNames(vmsWithCAs []VMWithCAs, caNames []string) ([]VMWithCAs, int) {

I need to consider the use case and adopt a consistent set of return values. Ideally the entire signature would be consistent across the functions to permit creation of a common interface for batch processing.

[atc0005]

Note to self: The problem is unlikely to be limited just to functions that filter VMs as I likely used those functions as templates for other vSphere objects (e.g., datastores, hosts).