Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3007.x] Merge forward 3006.x into 3007.x #67002

Open
wants to merge 24 commits into
base: 3007.x
Choose a base branch
from

Conversation

dwoz
Copy link
Contributor

@dwoz dwoz commented Oct 25, 2024

see title

ben-grande and others added 23 commits October 16, 2024 18:08
Add logging and convert to formatted string literal.

Other Yum options uses equal sign as separator but setopt separator is
irregular using space as separator.

Fix: saltstack#66970
Not a valid argument for this function, remove misleading documentation
reference.

Signed-off-by: Georg Pfuetzenreuter <[email protected]>
A pillars dict in a minion node definition was not overridding
correctly the same dict in classes.

* salt/utils/saltclass.py (expanded_dict_from_minion): do not pass a
  reference to minion dict or it will be overridden by classes during
  expansion.

Fixes: saltstack#63933
This was introduced by saltstack#66517

`__invocation_jid` is rewriten to `_State__invocation_id`, making
`cls(**init_kwargs)` fail with a TypeError (unexpected argument
`__invocation_jid` (this behavior is called name mangling).
This ensures dunders like __env__ are defined in states running in
parallel on spawning platforms.

The __running__ dict can contain salt.utils.process.Process instances,
which are still picklable.
@dwoz dwoz requested a review from a team as a code owner October 25, 2024 21:51
@salt-project-bot-prod-environment salt-project-bot-prod-environment bot changed the title Merge forward 3006.x into 3007.x [3007.x] Merge forward 3006.x into 3007.x Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test:full Run the full test suite test:pkg Run the package tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants