Skip to content

stdlib::manage: Also permit templates for concat::fragment#1434

Merged
skyamgarp merged 1 commit intopuppetlabs:mainfrom
jcpunk:concat-fragment
Feb 12, 2026
Merged

stdlib::manage: Also permit templates for concat::fragment#1434
skyamgarp merged 1 commit intopuppetlabs:mainfrom
jcpunk:concat-fragment

Conversation

@jcpunk
Copy link
Copy Markdown
Contributor

@jcpunk jcpunk commented Jul 15, 2024

Summary

This adds limited template support for concat::fragment resources under stdlib::manage

Additional Context

Add any additional context about the problem here.

  • Thought process behind the implementation.

It might be nice to divide /etc/motd up via concat and defer some limited elements into trivial yaml.

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

@bwitt
Copy link
Copy Markdown

bwitt commented Oct 15, 2025

Maybe adding some tests would help?

@jcpunk
Copy link
Copy Markdown
Contributor Author

jcpunk commented Oct 15, 2025

Detailed tests added.

@jcpunk
Copy link
Copy Markdown
Contributor Author

jcpunk commented Dec 8, 2025

Checking back in on this...

@skyamgarp
Copy link
Copy Markdown
Contributor

@jcpunk , can you please rebase your changes?

Signed-off-by: Pat Riehecky <riehecky@fnal.gov>
@jcpunk
Copy link
Copy Markdown
Contributor Author

jcpunk commented Feb 10, 2026

Rebased

@skyamgarp
Copy link
Copy Markdown
Contributor

Changes tested with: #1464

@skyamgarp skyamgarp merged commit 75517cc into puppetlabs:main Feb 12, 2026
5 of 6 checks passed
@jcpunk jcpunk deleted the concat-fragment branch February 12, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants