On Thu, Dec 16, 2021 at 1:36 PM Milan Zamazal <[email protected]> wrote: > > Hi, > > it seems a recent commit causes a storage test failure on GitHub: > > ================================== FAILURES > =================================== > ________________ test_prepare_connection_without_initiator_name > ________________ > > def test_prepare_connection_without_initiator_name(): > con_def = [{ > "password": "password", > "port": "3260", > "iqn": "iqn.2016-01.com.ovirt:444", > "connection": "192.168.1.2", > "ipv6_enabled": "false", > "id": "994a711a-60f3-411a-aca2-0b60f01e8b8c", > "user": "", > "tpgt": "1", > }] > > conn = storageServer._prepare_connections(sd.ISCSI_DOMAIN, con_def) > E AttributeError: module 'vdsm.storage.storageServer' has no > attribute '_prepare_connections' > > Could you please fix it quickly or disable the test? > > @all: Beware that CI on Vdsm no longer runs the tests and patches must > be tested on GitHub before merging.
Thanks for reporting, fixed in https://gerrit.ovirt.org/c/vdsm/+/118071 I guess this is a side effect of testing only the top patch in github. It is much faster but misses inconsistencies in the series. Once we move to github this should not happen since in github you cannot merge a partial PR. It can still happen if we checkout random patch from a PR that was never tested alone, but this is not very interesting. Nir _______________________________________________ Devel mailing list -- [email protected] To unsubscribe send an email to [email protected] Privacy Statement: https://www.ovirt.org/privacy-policy.html oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/[email protected]/message/TZBXJONKLG53B6AJMLHLMVAGSKIPWKKM/
