Skip to content

Conversation

@span786
Copy link

@span786 span786 commented Dec 2, 2025

Summary

Implement the finish method for class tag inheritance within Puppet::ResourceApi.
This method is being overridden to allow reinjection of default tags (such as the class name) back into the Puppet catalog during compilation.

Additional Context

Add any additional context about the problem here.

  • The Puppet::ResourceApi currently does not handle metaparams like tags effectively.
  • To handle default tags, we needed a point in the catalog-building process where they could be reintroduced. Overriding the finish method provides that opportunity, enabling us to push the tags back into the catalog at the right stage.

Related Issues (if any)

Due to this limitation, modules such as puppetlabs-firewall (version 8.0.0 and later) fail to recognize default tags like the class name. As a result, running puppet apply <manifest.pp> --tags <classname> does nothing because the expected tags are missing.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.

… Puppet::ResourceApi

1. This method is overridden so that we could again inject the default tags (like class name) in puppet catalog.
@span786 span786 requested review from a team as code owners December 2, 2025 03:32
@span786 span786 marked this pull request as draft December 2, 2025 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants