Yeah it's intentional. The ordering relationship you mention is already defined by `->` here:

https://github.com/jistr/astapor/blob/0252dde7906dac9fb885fe30c2c7d6f84a76a7d7/puppet/modules/quickstack/manifests/pacemaker/cinder.pp#L113-L117


The aim here is to hook the execution of the cinder-volume related classes into place as if they were part of this block:

https://github.com/jistr/astapor/blob/0252dde7906dac9fb885fe30c2c7d6f84a76a7d7/puppet/modules/quickstack/manifests/pacemaker/cinder.pp#L101-L132

The only reason they're not part of that block is that they have to be executed conditionally.