<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 7, 2016 at 11:54 PM, Nir Soffer <span dir="ltr"><<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">broken_on_ci is uses default name="OVIRT_CI", to mark it also for<br>
travis, we need another broken_on_ci with name="TRAVIS_CI".<br>
<br>
Maybe this test should run only if nm is active on the machine?<br></blockquote><div><br></div><div>We need the test to always run when expected.<br></div><div>If NM is not running, the test will not run (silently) and we will never know if there is a problem or not.<br><br></div><div>It is not convenient to mark each CI type as broken, why the test code needs to know we have multiple<br></div><div>CI/s?<br></div><div><br></div><div>Currently, we run on CI tests that are not marked as 'functional'.<br></div><div>Perhaps we need another test type that can be mark not to run on simple CI.<br></div><div>"power-integration", "super-integration"?<br></div><div><br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
On Wed, Dec 7, 2016 at 11:23 PM, Dan Kenigsberg <<a href="mailto:danken@redhat.com">danken@redhat.com</a>> wrote:<br>
> On Wed, Dec 7, 2016 at 2:03 PM, Nir Soffer <<a href="mailto:nsoffer@redhat.com">nsoffer@redhat.com</a>> wrote:<br>
>> Looks like we need @brokentest("reason...", name="TRAVIC_CI") on this:<br>
><br>
> Odd, the code already has<br>
><br>
> @broken_on_ci('NetworkManager should not be started on CI nodes')<br>
><br>
><br>
>><br>
>> See <a href="https://travis-ci.org/oVirt/vdsm/jobs/181933329" rel="noreferrer" target="_blank">https://travis-ci.org/oVirt/<wbr>vdsm/jobs/181933329</a><br>
>><br>
>> ==============================<wbr>==============================<wbr>==========<br>
>><br>
>> ERROR: test suite for <module 'network.nmdbus_test' from<br>
>> '/vdsm/tests/network/nmdbus_<wbr>test.py'><br>
>><br>
>> ------------------------------<wbr>------------------------------<wbr>----------<br>
>><br>
>> Traceback (most recent call last):<br>
>><br>
>> File "/usr/lib/python2.7/site-<wbr>packages/nose/suite.py", line 209, in run<br>
>><br>
>> self.setUp()<br>
>><br>
>> File "/usr/lib/python2.7/site-<wbr>packages/nose/suite.py", line 292, in setUp<br>
>><br>
>> self.setupContext(ancestor)<br>
>><br>
>> File "/usr/lib/python2.7/site-<wbr>packages/nose/suite.py", line 315, in<br>
>> setupContext<br>
>><br>
>> try_run(context, names)<br>
>><br>
>> File "/usr/lib/python2.7/site-<wbr>packages/nose/util.py", line 471, in try_run<br>
>><br>
>> return func()<br>
>><br>
>> File "/vdsm/tests/testValidation.<wbr>py", line 191, in wrapper<br>
>><br>
>> return f(*args, **kwargs)<br>
>><br>
>> File "/vdsm/tests/testValidation.<wbr>py", line 97, in wrapper<br>
>><br>
>> return f(*args, **kwargs)<br>
>><br>
>> File "/vdsm/tests/network/nmdbus_<wbr>test.py", line 48, in setup_module<br>
>><br>
>> NMDbus.init()<br>
>><br>
>> File "/vdsm/lib/vdsm/network/nm/<wbr>nmdbus/__init__.py", line 33, in init<br>
>><br>
>> NMDbus.bus = dbus.SystemBus()<br>
>><br>
>> File "/usr/lib64/python2.7/site-<wbr>packages/dbus/_dbus.py", line 194, in __new__<br>
>><br>
>> private=private)<br>
>><br>
>> File "/usr/lib64/python2.7/site-<wbr>packages/dbus/_dbus.py", line 100, in __new__<br>
>><br>
>> bus = BusConnection.__new__(<wbr>subclass, bus_type, mainloop=mainloop)<br>
>><br>
>> File "/usr/lib64/python2.7/site-<wbr>packages/dbus/bus.py", line 122, in __new__<br>
>><br>
>> bus = cls._new_for_bus(address_or_<wbr>type, mainloop=mainloop)<br>
>><br>
>> DBusException: org.freedesktop.DBus.Error.<wbr>FileNotFound: Failed to<br>
>> connect to socket /var/run/dbus/system_bus_<wbr>socket: No such file or<br>
>> directory<br>
>><br>
>> -------------------- >> begin captured logging << --------------------<br>
>><br>
>> 2016-12-07 11:48:33,458 DEBUG (MainThread) [root] /usr/bin/taskset<br>
>> --cpu-list 0-1 /bin/systemctl status NetworkManager (cwd None)<br>
>> (commands:69)<br>
>><br>
>> 2016-12-07 11:48:33,465 DEBUG (MainThread) [root] FAILED: <err> =<br>
>> 'Failed to get D-Bus connection: Operation not permitted\n'; <rc> = 1<br>
>> (commands:93)<br>
>><br>
>> 2016-12-07 11:48:33,465 DEBUG (MainThread) [root] /usr/bin/taskset<br>
>> --cpu-list 0-1 /bin/systemctl start NetworkManager (cwd None)<br>
>> (commands:69)<br>
>><br>
>> 2016-12-07 11:48:33,470 DEBUG (MainThread) [root] FAILED: <err> =<br>
>> 'Failed to get D-Bus connection: Operation not permitted\n'; <rc> = 1<br>
>> (commands:93)<br>
>><br>
>> --------------------- >> end captured logging << ---------------------<br>
</div></div></blockquote></div><br></div></div>