README: Add instructions to configure #77
Conversation
Without configuring qemu permissions in /etc/libvirt/qemu.conf, avocado-setup.py fails with the following error: ``` ERROR Cannot access storage file '/root/hostos/tests/data/avocado-vt/images/f23-ppc64le.qcow2' (as uid:107, gid:107): Permission denied ``` Signed-off-by: Murilo Opsfelder Araujo <muriloo@linux.vnet.ibm.com>
|
@sathnaga Please review this change also to verify we are not missing anything else concerning permissions config prerequisites. Thanks! |
|
|
||
| user = "root" | ||
| group = "root" | ||
|
|
There was a problem hiding this comment.
@mopsfelder I feel that is a default config option, as it works fine in the hostos installed machines in our env.
checkout, dynamic_ownership = 1, this again default values holds good, if anyone changes to 0 would cause the permission denied issues for image.
There was a problem hiding this comment.
My understanding reading https://github.com/open-power-host-os/tests/blob/master/README#L20 is that it supports CentOS 7.3 as the host.
When I ran avocado-setup.py on a CentOS host, I hit this issue because /etc/libvirt/qemu.conf had #dynamic_ownership = 1 (commented out).
We shall either
(1) update README to say that avocado-setup.py should only be executed on a HostOS host (and remove CentOS as a supported host); or
(2) add instructions in README to properly configure dynamic_ownership.
What do you think, @sathnaga ?
There was a problem hiding this comment.
Good point. The original wording is a bit misleading since the intention was to indicate that the Host tests should be run on the current CentOS-based HostOS (prepping for potential HostOS releases that are based on other OS's). We should just change that to "The latest version of HostOS".
|
I propose that we close this request and merge Pull Request #78 in its place, which clarifies the OS support statement for Host tests. As stated above, the default config for Host OS hosts should not cause the permission issues seen on default CentOS hosts. |
|
We are working on ReadMe wording adjustments in PR #78 to clarify where tests should be run. This adjustment to the ReadMe is no longer needed. |
Without configuring qemu permissions in /etc/libvirt/qemu.conf,
avocado-setup.py fails with the following error:
Signed-off-by: Murilo Opsfelder Araujo muriloo@linux.vnet.ibm.com