Katello-backup/restore/service scripts are all moving to foreman-maintain (as discussed in previous threads) and the development is wrapping up with the last two PRs getting close to merging (backup and restore).
This brings up the question of how are we going to handle deprecating the existing backup/restore/service scripts?
My thought is to add deprecation warnings to the existing katello-backup/restore scripts in katello 3.7 since they were included in 3.7 branching. This means we would need to add foreman-maintain as a dependency to katello 3.7, since the new functionality is there. Then the old scripts can be removed in katello 3.8.
I’m curious to hear thoughts on this, I know many people use katello-backup/restore and would like to handle the removal gracefully.
I mostly wonder if katello-service could be replaced by pure systemctl. I suspect most of our scripts can use systemctl leaving katello-service as a tool for users.
There are cases where I highly doubt if it’s needed. One example is our puppet upgrade hook in the installer. Here we stop all services while just stopping puppet and puppetserver is probably enough. systemctl can do that just fine. Another is katello-service start --only postgresql in our regular upgrade hook where systemctl start postgresql should be fine.
The two benefits of katello-service (and foreman-mantain service) is using the service-wait script and ordering the service starting/stopping order. If these aren’t needed, then I don’t see an issue with using systemctl. I think its a case-by-case thing.