@Koodt thanks for your effort. Could you please add functional tests as well?

coverage: 63.781% (+0.001%) from 63.78%
when pulling 12d6828 on Koodt:add_volumetypes_listopts_field
into 832f6f8 on gophercloud:main.

Unfortunately due to added ExtraSpecs map, the PR can not be backported to v2:

  Incompatible changes:
  - ListOpts: old is comparable, new is not

@mandre @stephenfin do you think we can make an exception?

Unfortunately due to added ExtraSpecs map, the PR can not be backported to v2:

  Incompatible changes:
  - ListOpts: old is comparable, new is not

@mandre @stephenfin do you think we can make an exception?

How about we split the PR in two, with all of the changes except for ExtraSpecs in one PR that we can backport to v2 and another one with just ExtraSpecs that we merge in main only?

I'm always a little bit uncomfortable for making exceptions, because it creates a precedent, and also because we never know how people are using the lib.

Thanks for the PR @Koodt. I recently had the same problem (https://github.com/k-orc/openstack-resource-controller/blob/b8d62d3cb3c4b56078228c4377aa9607161036d5/internal/controllers/volumetype/actuator.go#L75-L87) and didn't realize the API actually supported these fields because they're not documented in https://docs.openstack.org/api-ref/block-storage/v3/#list-all-volume-types but forgot that glance uses additional resource filters (https://docs.openstack.org/api-ref/block-storage/v3/#resource-filters-resource-filters).

Hi! I've prepare the non-breaking changes for back port to v2
extra_specs will come as a separate MR later

Failed to backport PR to v2 branch. See logs for details.