Could a config option be added for the puppet agent to require a message when running `puppet agent --disable`? For larger sites with multiple admins, it would be helpful to have a way to force this best practice. I don't expect it to stop `puppet agent --disable ' '` but in that instance the admin knows they are leaving a useless message against the policy.
It'd be fairly easy to add a puppet setting, something like agent_disabled_required_message = true|false to complement agent_disabled_lockfile. If the setting was true and you ran puppet agent --disable then have the agent print an error that a message is required and error, leaving the agent in whatever state it was in previously.
Thank you for filing this issue. We agree it is likely an improvement, but due to other issues demanding precedence, we don’t anticipate being able to address this any time soon. If you are interested in submitting a patch to the repository for this project at https://github.com/puppetlabs/puppet, please open a pull request.