Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[3006.x][WIP] Handle StrictUndefined for all salt added jinja filters #65004

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 7 commits into
base: 3006.x
Choose a base branch
from
Open

[3006.x][WIP] Handle StrictUndefined for all salt added jinja filters #65004

wants to merge 7 commits into from

Conversation

@cmcmarrow
Copy link
Contributor

@cmcmarrow cmcmarrow commented Aug 17, 2023

What does this PR do?

Handles StrictUndefined for all salt added jinja filters

What issues does this PR fix or reference?

Fixes: #64915

Previous Behavior

{{ dict['missing-key'] | yaml }} -> NULL

New Behavior

{{ dict['missing-key'] | yaml }} -> Rendering SLS error

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

@cmcmarrow cmcmarrow requested a review from a team as a code owner August 17, 2023 00:41
@cmcmarrow cmcmarrow requested review from dwoz and removed request for a team August 17, 2023 00:41
@salt-project-bot-prod-environment salt-project-bot-prod-environment bot changed the title [WIP] Handle StrictUndefined for all salt added jinja filters [3006.x][WIP] Handle StrictUndefined for all salt added jinja filters Aug 17, 2023
@cmcmarrow cmcmarrow added this to the Sulfur v3006.4 milestone Aug 17, 2023
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 00:57 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 00:57 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 00:57 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 00:57 — with GitHub Actions Inactive
@cmcmarrow
Copy link
Contributor Author

cmcmarrow commented Aug 17, 2023

we fail to check nested objects

@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 01:47 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 01:47 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 01:48 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 01:50 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 02:05 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 02:07 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 03:24 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 03:24 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 03:24 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 03:24 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 03:25 — with GitHub Actions Inactive
@cmcmarrow cmcmarrow temporarily deployed to ci August 17, 2023 03:25 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch added the needs-testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases label Aug 17, 2023
@s0undt3ch s0undt3ch temporarily deployed to ci August 17, 2023 07:14 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci August 17, 2023 07:14 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci August 17, 2023 07:14 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci August 17, 2023 07:14 — with GitHub Actions Inactive
@s0undt3ch s0undt3ch temporarily deployed to ci August 17, 2023 07:32 — with GitHub Actions Inactive
@twangboy twangboy self-assigned this May 16, 2025
@twangboy twangboy force-pushed the null_yaml_filter branch from 042c8f6 to f914285 Compare May 16, 2025 18:57
@dwoz dwoz modified the milestones: Sulfur v3006.12, Sulfer v3006.13 Jun 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dwoz dwoz Awaiting requested review from dwoz dwoz was automatically assigned from saltstack/team-core

At least 1 approving review is required to merge this pull request.

Assignees

@twangboy twangboy

Labels

None yet

Projects

None yet

Milestone

Sulfur v3006.17

Development

Successfully merging this pull request may close these issues.

4 participants

@cmcmarrow @s0undt3ch @dwoz @twangboy