yunohost
yunohost copied to clipboard
[enh] Display other system logs
The problem
Some system log are not available in web admin
Solution
This PR add this log in Tools > Log like it was designed during the Log PR.
PR Status
Ready This PR depends of an other on yunohost-admin https://github.com/YunoHost/yunohost-admin/pull/255
How to test
We need to merge this in unstable to test completely, but you can test like this:
- Pull branch on yunohost and yunohost-admin
- use git on yunohost and yunohost-admin
- cp -R yunohost/data/other/logs/* /var/log/yunohost/categories/
- service yunohost-admin
- Try to use the webadmin to see the new log in screenshot
Validation
- [ ] Principle agreement 0/2 :
- [ ] Quick review 0/1 :
- [ ] Simple test 0/1 :
- [ ] Deep review 0/1 :
About this PR. We discuss it and Aleks think it should be transform into conf files store into /etc/yunohost . So it make it quite more difficult to implement.
Closed cf today's meeting:
Aleks: Imho this really feels like some YAGNI ... There's no really any huge gain in terms of support to expose these in the webadmin -> To close ? ljf: yes cause it need to be redone AND because 777 is evil BUT i disagree with you. It could be helpfull to allow user to not need CLI at all to do the operation. However i think important event about those file could be reported by diagnosis system (oom_reaper, a lot of ssh attempt?, ...).