Community
OpenEnergyMonitor

Community

Bug? read-write -Set root file-system temporarily to read-write, (default read-only)

emoncms
Tags: #<Tag:0x00007f16b0554650>

(Celso Henriques) #1

Sorry bothering you all with this but i think ( please correct me if i am wrong), if i read it well, the file-system is now set to read-write by default on the emonSD-30Oct18 image.
See: https://community.openenergymonitor.org/t/new-emonsd-dropping-read-only-root-filesystem-requirement/8293
If that so, it need to get fixed the text on the administration tab that says “Set root file-system temporarily to read-write, (default read-only)”.


(Trystan Lea) #2

Good point thanks for spotting @CelsoHenriques, I’ve added this as an issue on the emoncms github to fix https://github.com/emoncms/emoncms/issues/1129


(Glyn Hudson) #3

Correct, we need to detect if we’re running the new image which does not use a RO root FS and remove the button in Emoncms admin for these systems. Thanks for bringing this up.


(Celso Henriques) #4

Is necessary to remove the button? I am no developer and badly understand coding but if the root partition is read only, isnt that more safe, regarding to virus? And fixing just the text for those who want read only. Unless it messes with the system.


(Paul) #5

Since the button’s are dependent on the rpi-rw and rpi-ro commands, a test to see if those commands (files) exist and only display buttons if the’re present would suit all Pi users and images without a hardcoded lookup “version” test.


(Glyn Hudson) #6

True, that’s a good solution. Since currently on the new image since the rpi-rw/ro files are not present the buttons don’t do anyting