PDK ignoring config_defaults.yml delete values

[alexjfisher]

Describe the bug
When using a forked pdk template, if I want to delete an arbitrary file, I should be able to via config_defaults.yml

To Reproduce
Add the following to config_defaults.yml

Jenkinsfile:
  delete: true

Expected behavior
When running PDK update on a module using this template, I'd expect any existing Jenkinsfile to be deleted.

Additional context
PDK 1.18.1 on CentOS7 via RPM

This issue is similar to #444
I can workaround the bug, by adding an empty file to moduleroot. IMO. this shouldn't be required.

[github-actions]

This issue has been marked stale because it is open with no recent activity. The PDK Team is actively prioritizing existing bugs and new features, if this issue is still important to you please comment and we will add this to our backlog to complete.

[ardrigh]

This issue is similar to #444 I can workaround the bug, by adding an empty file to moduleroot. IMO. this shouldn't be required.

Workaround required based on the current design which looks at files in the templates directory to then process which ones are unmanaged or deleted.

pdk/lib/pdk/template/renderer/v1/renderer.rb

Line 72 in 6b1aec6

files_in_template(dirs).each do |template_file, template_loc|

I guess the alternative would be parsing config_defaults.yml and .sync.yml (which are meant to merge into a combined set) for header entries. But that approach seems more prone to errors IMO.

[github-actions]

This issue has been marked stale because it is open with no recent activity. The PDK Team is actively prioritizing existing bugs and new features, if this issue is still important to you please comment and we will add this to our backlog to complete.

[cruelsmith]

With the silent drop (without notice in release notes / changelog) of multiple features with puppetlabs/pdk-templates#510 it is more important than ever when maintaining many projects.
All pdk projects that update to pdk 3 will still contain these old feature files that are not maintained anymore. Note that even multiple puppetlabs modules still contain them and they needs manual cleanup without this requested feature.