Improve instructions - RMTECTRL about rpi-reporter-mqtt2ha-daemon

[ironsheep]

@bsimmo Sincerely, thank you for all this detail. I'm cleaning this up now.

[ironsheep]

Changes committed and appear in v1.8.5 and later.

[ironsheep]

@bsimmo Please review the updates and let me know what you think.

https://github.com/ironsheep/RPi-Reporter-MQTT2HA-Daemon/blob/master/RMTECTRL.md

[FondledYeti]

@ironsheep, not OP but I'm still having a little trouble with this.

The docs suggest sudo chown daemon RPi-mqtt-daemon-script.sh to update permissions of our reporter script. However there is no script of this name in the repo.

There is an rpi-reporter but when I change ownership for that I still have entries in auth.log that suggest the commands are being run by root and not daemon.

I hope this was the right place to post, sorry if I should have opened a new issue, but it seemed related to this documentation request.

[Andystew94]

^further assistance, please. I am stuck in the same position.

[bsimmo]

I did scan over it and it seemed much better. Just forgot to reply here.

I'll check see what your problem is
Ah. Had a read.
@FondledYeti and @Andystew94 are you looking at S4 section.

I believe that is for adding any of your own commands to run. So in the setup area you create another entry with the command to a script you may want to run (for example an update script). It is then that scrip/file you are altering the permissions on.

You do not need that for the default setup.

If it is not that, which section are you in?

[bsimmo]

So you may add an update script

[Commands]
#update_reporter = /usr/bin/sudo/ place/script/is/update-script.sh 'update request over MQTT'

You would then need to add that through the sections before and then check make it executable as daemon user.

I've not tried it, but that is my take on it.

It will show up as another button in HA.

(as a side note, such an updating script can be found in #87)

None of that is needed for the normal three Commands.
Just check they're in the config file, add to sudoers. Etc.

[ironsheep]

@Andystew94 and @FondledYeti Sheesh... I think I botched the instructions for that Step. I've updated them please review that step (S4) now and see if (1) the instructions now work and (2) they make sense.

https://github.com/ironsheep/RPi-Reporter-MQTT2HA-Daemon/blob/master/RMTECTRL.md#s4-adjusting-the-ownership-of-our-reporter-script

[FondledYeti]

Hi @ironsheep @bsimmo, thanks for your help clearing that up. @ironsheep the updated instructions work nicely. One thing to note: I tried adding a comment to the systemctl restart command to see if it would show up in the logs like the others do and it causes the command to fail (though it does show up in the logs :)). It's not an issue since you haven't suggested doing that, but you may like to include an alternative suggestion for greping the logs when using that command.

Once again, thanks for the help and thanks for publishing the project!