theforeman / puppet-certs

Puppet module for dealing with SSL certs across other modules used in Katello
GNU General Public License v3.0
5 stars 39 forks source link

Drop RPMs from being included in tarball #421

Closed ehelms closed 7 months ago

ehelms commented 1 year ago

These were kept in for N-1 support when we first switched away from using RPMs for deployment. Enough release have passed that we can drop this.

ehelms commented 1 year ago

@ekohl could you take a look?

evgeni commented 7 months ago

The RPMs are still built and everything, right? Just not distributed in the certs-tar?

(README needs an update, it says the deployment still uses RPM, but that's unrelated to the changes in this PR)

ehelms commented 7 months ago

The RPMs are still built and everything, right? Just not distributed in the certs-tar?

Correct -- https://github.com/theforeman/puppet-certs/pull/426 is designed to remove all RPM generation and simplify the code quite a bit.