You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Description
When passing complex pillar overrides to salt-ssh state.sls etc., there will be a mismatch during template rendering. The pillar dict will contain the correct data, while calling the pillar.get wrapper module will return data that has not been recursively merged (all wrappers are affected). This is the master-side equivalent of #59802.
Setup
irrelevant
Steps to Reproduce the behavior
# a pillar targeted to the minionmonty: pythonthe_meaning:
of:
life: 42bar: tenderfor: what
Notice the difference of raw and wrapped, the latter is the initial pillar .update()d with the overrides, the former the wanted, recursively merged one.
Versions Report
quite recent checkout of 3006.x branch, nothing has changed in master though
Additional context
Stumbled upon this while writing tests for a PR.
The text was updated successfully, but these errors were encountered:
lkubb
changed the title
[BUG] salt-ssh state.* with pillar overrides: pillar dict and pillar.get exemod out of sync
[BUG] salt-ssh state.* with pillar overrides: pillar dict and pillar.get wrapper out of sync
Oct 29, 2023
lkubb
changed the title
[BUG] salt-ssh state.* with pillar overrides: pillar dict and pillar.get wrapper out of sync
[BUG] salt-ssh state.* with pillar overrides: pillar dict and pillar in wrappers out of sync
Oct 29, 2023
Description
When passing complex pillar overrides to
salt-ssh state.sls
etc., there will be a mismatch during template rendering. Thepillar
dict will contain the correct data, while calling thepillar.get
wrapper module will return data that has not been recursively merged (all wrappers are affected). This is the master-side equivalent of #59802.Setup
irrelevant
Steps to Reproduce the behavior
salt-ssh foo state.apply test pillar='{the_meaning: {of: {life: 2.71, foo: lish}}, btw: turtles}'
Expected behavior
Screenshots
Notice the difference of
raw
andwrapped
, the latter is the initial pillar.update()
d with the overrides, the former the wanted, recursively merged one.Versions Report
quite recent checkout of 3006.x branch, nothing has changed in master though
Additional context
Stumbled upon this while writing tests for a PR.
The text was updated successfully, but these errors were encountered: