Fixes #35496 - Add default pulp_deb config

[quba42]

https://projects.theforeman.org/issues/35496

For Katello users this is the sane/expected default behaviour. It allows
for the synchronization of partial Ubuntu mirrors, a very common use
case in the wild. Users often stumble over the error that is thrown for
such mirrors without this setting.

[quba42]

Should I create a Katello issue to go with this change?

[ekohl]

Should I create a Katello issue to go with this change?

It would be an installer issue. I always appreciate those since it helps with cherry picks.

[ekohl]

Thanks. This is indeed the correct place.

You can also add a test. Either check for the concat fragment, like this:

puppet-pulpcore/spec/classes/plugin_ansible_spec.rb

Lines 26 to 28 in 5858c93

	is_expected.to contain_concat__fragment('plugin-ansible')
	.with_content(%r{^ANSIBLE_API_HOSTNAME = "https://foo.example.com"})
	.with_content(%r{^ANSIBLE_CONTENT_HOSTNAME = "https://foo.example.com/pulp/content"})

Or add .with_config('FORCE... = True') to contain_pulpcore__plugin('deb') here:

puppet-pulpcore/spec/classes/plugin_deb_spec.rb

Lines 19 to 21 in 5858c93

	is_expected.to contain_pulpcore__plugin('deb')
	.that_subscribes_to('Class[Pulpcore::Install]')
	.that_notifies(['Class[Pulpcore::Database]', 'Class[Pulpcore::Service]'])

Looking at it now I'd go with the latter since it's easier and you're testing less for internal details.

[ekohl]

@quba42 we're about to release the module so we're ready for RC1. Any plan to finish this? Ideally speaking we'd release this module this week.

[quba42]

Ugh totally forgot about this. I will have a look if I can quickly finish it now...

[quba42]

@ekohl Added your suggestions and a test (that I think makes sense)...

[ekohl]

Thanks! Now I can do the release for Foreman 3.5 :)