vmware / govmomi

Go library for the VMware vSphere API
Apache License 2.0
2.32k stars 913 forks source link

Re-enable PC updates for indexed properties #3350

Closed brakthehack closed 9 months ago

brakthehack commented 9 months ago

Description

A previous change reverted behavior that ignored indexed property updates in favor of a panic. While we don't implement persistence for partial updates, some consumers rely on this behavior to test property collector updates on specific fields.

This change reverts a03f4735570027d2e6e2bc9cb7584a0af19f9353q to the previous behavior allowing downstream consumers to rely on this field for testing PC updates until full support for indexed fields is implemented.

Closes: #3349

Type of change

Please mark options that are relevant:

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. If applicable, please also list any relevant details for your test configuration.

Checklist: