hope ya don't mind if I grab this one

Closed via cb298b9e. None of the commits from https://github.com/ClusterLabs/pacemaker/pull/3539 were properly linked here, despite their commit messages having the relevant phrases.

(with a little magic to re-allow the old name as a custom name after a CIB has been upgraded).

In T829#12444, @nrwahl2 wrote:

@kgaillot I had added this as a parent task because we can't drop require-all from ordering constraints until after we've dropped support for rolling upgrades from <2.0.0

@kgaillot I had added this as a parent task because we can't drop require-all from ordering constraints until after we've dropped support for rolling upgrades from <2.0.0

I suspect the use case for this feature was being able to set an explicit value in one block of meta-attributes, then have another rule-based block of attributes that resets the value to the default. Simply omitting the meta-attribute from the rule-based block wouldn't cause the explicit value to be removed.

In T800#11838, @nrwahl2 wrote:

<define name="non-empty-string">
   <data type="string">
      <except><value></value></except>
   </data>
</define>

It turns out that due to the same normalization you described, any value that consists of only spaces is rejected. Not just empty strings.

Which might be fine for our purposes, but we need to be aware of it.