The inherits stuff doesn't really work as expected in Foreman, and we have been moving to not use it like this anymore.  Basically, all it does is cause a weird default value in the UI if we forget to supply a default override in seeds.rb.  In general, I would prefer to see reasonable defaults right in the pupae manifests.  We can still _use_ quickstack::params internal to the classes if there are cross-module values (as we do with shared params for quickstack::pacemaker::params), or more standard puppet things we want, like determining the value bases on OS family/version.  However, I would rather not see us add any new classes that have this old pattern from the original stand-alone host groups. It should have no functional impact, and may actually allow you to remove at least some of the defaults you probably adde din seeds.rb.  Note that anything in seeds will still override these values inside foreman as you would expect, so there is no loss of functionality by this switch.