feat: Remove autoheal functionality from rhsmcertd

[jvlcek]

Card ID: CCT-723

Removed autoheal sub-command from rhsmcertd because autoheal is useless in pure SCA mode..

[jvlcek]

@jirihnidek Here is my draft PR. Please take a look.

I have three questions:

1. How do I test these changes beyond running the pytests?
2. It looks to me like auto_attach_initial_delay , auto_attach_offset, and NEXT_AUTO_ATTACH_UPDATE_FILE can also be removed from src/daemons/rhsmcertd.c. Do you know if that is correct?
3. What should I do about the deprication message from man/rhsmcertd.8 ?
   https://github.com/candlepin/subscription-manager/blob/main/man/rhsmcertd.8#L103-L114

[ptoscano]

It generally seems good, there are few things to fix. Also drop an extra import, see the flake8 failure (it appears also as inline notes in the "files changed" view in github).

It looks to me like auto_attach_initial_delay , auto_attach_offset, and NEXT_AUTO_ATTACH_UPDATE_FILE can also be removed from src/daemons/rhsmcertd.c. Do you know if that is correct?

Yes, it looks so.

What should I do about the deprication message from man/rhsmcertd.8 ?

Edit it to mention only one parameter, according to the changes done here.

That feature looks something to remove as well, and I'd do that in a separate commit/PR.

[jirihnidek]

I have same comments as Pino has.

[jvlcek]

Thank you @ptoscano and @jirihnidek for all of the help and patience.

I believe I have addressed all of the issues you've identified so far.

I will now squash all of the commits I created while resolving yourreview feedback.

[ptoscano]

thanks for the updates, this seems better now; I'll give it a try later on

[jvlcek]

Thank you for the feedback @ptoscano
I will squash my comments are rebase.