Page MenuHomeClusterLabs Projects
Create Task
Maniphest T808

Deprecate "++" and "+=" XML attribute value expansion
Open, LowPublic
Actions

Description

...unless we find some legitimate use case (CIB epoch etc. being the most likely). Currently it's supported via pcmk__xml_update() -> copy_in_properties() -> expand_plus_plus().

It's a pain to read and maintain, but more importantly CIB replace operations don't treat "<attr>++" and "<attr>+=<int>" specially. They set the values as unexpanded strings.

Ideally, a modify operation and a replace operation should both set the same attribute value in the CIB for the same input attribute value.

https://github.com/ClusterLabs/pacemaker/pull/3413#issuecomment-2035493624

Event Timeline

nrwahl2 triaged this task as Normal priority.Apr 4 2024, 2:41 PM
nrwahl2 created this task.
nrwahl2 created this object with edit policy "Restricted Project (Project)".
nrwahl2 updated the task description. (Show Details)
kgaillot lowered the priority of this task from Normal to Low.Jan 2 2025, 4:21 PM
kgaillot updated the task description. (Show Details)