[ovirt-users] Building ovirt-engine fails due to failing estCanReadFile(org.ovirt.engine.core.utils.servlet.ServletUtilsTest): We should not be able to read this file.

Yedidyah Bar David didi at redhat.com
Tue May 23 06:37:23 UTC 2017


On Tue, May 23, 2017 at 6:21 AM, Leni Kadali Mutungi
<lenikmutungi at gmail.com> wrote:
> Tried to build ovirt-engine on my computer using the command `make
> clean install-dev`, on Debian Testing.
>
> The full output of the error is as follows:
> https://paste.fedoraproject.org/paste/F6p~LVdNtFlpzQijMbSIYV5M1UNdIGYhyRLivL9gydE=

Did you search the code for relevant parts of the error message?
Searching for 'We should not be able to read this file' finds it in
backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/servlet/ServletUtilsTest.java
twice - once when trying to read '/doesnotexist/iamprettysure', and
the other when trying to read '/etc/securetty'.

The latter file is 0600 on Fedora, RHEL and derivatives. I guess it's
readable for your user on your system, which might not be a good idea
in itself - but that's a different discussion.

Some options for how to continue:

1. Submit a patch for the engine to change this test to test for some
other file that is more likely to be unreadable, e.g. /etc/shadow.

2. Locally set it on your system to be unreadable to you.

3. Assuming it's Debian's default, ask on Debian lists/forums to
change it to be non-world-readable and/or why it's not like that
already.

4. Ignore unit tests for now, 'make BUILD_UT=0 install-dev'. Check
README.adoc for details.

>
> I modified the following variables in the Makefile to suit where the
> required files are:
>
> JS_DEPS_DIR=/home/user/ovirt-js-dependencies
> PYFLAKES=/usr/bin/pyflakes
> JBOSS_HOME=/home/user/wildfly-11.0.0.Alpha1
> WILDFLY_OVERLAY_MODULES=/home/user/wildfly-11.0.0.Alpha1/modules
>
> I have pep8 but was unable to know how to correctly reference it in
> the Makefile, the syntax being:
>
> PEP8=pep8

This should probably be ok, unless your version of pep8 is too old/
too new/has different defaults than what we use in CI (which is the
"common denominator").

Good luck,

>
> Advice on this would be welcome.
>
>
> --
> - Warm regards
> Leni Kadali Mutungi
> _______________________________________________
> Users mailing list
> Users at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/users



-- 
Didi


More information about the Users mailing list