Hi,
I've hit an issue after upgrading LAVA on Debian Stretch from 2017.5-6486.59742fe-1 to 2017.5-6509.d71f267-1, trying to add or update device dictionaries using the "lava-tool device-dictionary" command then always failed.
Between these two revisions, the following change was introduced:
commit ae4d4776ca7b7454f5406159226e3c9327dd207f Author: Rémi Duraffort remi.duraffort@linaro.org Date: Tue May 2 15:22:33 2017 +0200
LAVA-757 Move device dictionaries to file system
The device dictionaries are now saved in:
/etc/lava-server/dispatcher-config/devices
which was previously installed with the root user. The LAVA server instance is running under the "lavaserver" user, which didn't have write access to this directory. So I had to manually change the permissions, and then it all worked.
Could you please add a step in the Debian package installation to change the permissions and ownership of the files in /etc/lava-server/ to fix this issue?
Note: It may also be a good idea to propagate the IOError into the xmlrpclib.Fault error message rather than replacing it with a rather mysterious "Unable to store the configuration for %s on disk". I had to do this to find out the root cause of the error; I can send a patch, let me know.
Best wishes, Guillaume
Could you turn this in a bug at bugs.linaro.org so that it's not forgotten.
Thanks
On 23 May 2017 at 16:34, Guillaume Tucker guillaume.tucker@collabora.com wrote:
Hi,
I've hit an issue after upgrading LAVA on Debian Stretch from 2017.5-6486.59742fe-1 to 2017.5-6509.d71f267-1, trying to add or update device dictionaries using the "lava-tool device-dictionary" command then always failed.
Between these two revisions, the following change was introduced:
commit ae4d4776ca7b7454f5406159226e3c9327dd207f Author: Rémi Duraffort remi.duraffort@linaro.org Date: Tue May 2 15:22:33 2017 +0200
LAVA-757 Move device dictionaries to file systemThe device dictionaries are now saved in:
/etc/lava-server/dispatcher-config/devices
which was previously installed with the root user. The LAVA server instance is running under the "lavaserver" user, which didn't have write access to this directory. So I had to manually change the permissions, and then it all worked.
Could you please add a step in the Debian package installation to change the permissions and ownership of the files in /etc/lava-server/ to fix this issue?
Note: It may also be a good idea to propagate the IOError into the xmlrpclib.Fault error message rather than replacing it with a rather mysterious "Unable to store the configuration for %s on disk". I had to do this to find out the root cause of the error; I can send a patch, let me know.
Best wishes, Guillaume _______________________________________________ Lava-users mailing list Lava-users@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lava-users
On 30/05/17 10:12, Matt Hart wrote:
Could you turn this in a bug at bugs.linaro.org so that it's not forgotten.
Here you go:
https://bugs.linaro.org/show_bug.cgi?id=3020
Cheers, Guillaume
On 23 May 2017 at 16:34, Guillaume Tucker guillaume.tucker@collabora.com wrote:
Hi,
I've hit an issue after upgrading LAVA on Debian Stretch from 2017.5-6486.59742fe-1 to 2017.5-6509.d71f267-1, trying to add or update device dictionaries using the "lava-tool device-dictionary" command then always failed.
Between these two revisions, the following change was introduced:
commit ae4d4776ca7b7454f5406159226e3c9327dd207f Author: Rémi Duraffort remi.duraffort@linaro.org Date: Tue May 2 15:22:33 2017 +0200
LAVA-757 Move device dictionaries to file systemThe device dictionaries are now saved in:
/etc/lava-server/dispatcher-config/devices
which was previously installed with the root user. The LAVA server instance is running under the "lavaserver" user, which didn't have write access to this directory. So I had to manually change the permissions, and then it all worked.
Could you please add a step in the Debian package installation to change the permissions and ownership of the files in /etc/lava-server/ to fix this issue?
Note: It may also be a good idea to propagate the IOError into the xmlrpclib.Fault error message rather than replacing it with a rather mysterious "Unable to store the configuration for %s on disk". I had to do this to find out the root cause of the error; I can send a patch, let me know.
Best wishes, Guillaume _______________________________________________ Lava-users mailing list Lava-users@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lava-users
-
Guillaume Tucker -
Matt Hart