{"push_id":749586311,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"abcf220d9f17fb1fd1a574ac3d623ad95455b549","before":"c6acc11b288561d2bf1346e396b5a89c598c5580","commits":[{"sha":"abcf220d9f17fb1fd1a574ac3d623ad95455b549","author":{"email":"60258c486a9660a474d408e6013543d2add80a76@redhat.com","name":"Nir Soffer"},"message":"nettestlib: Add requires_brctl and requries_tc decorators\n\nLot of tests used nettestlib.checkDependencies() in setUp() to skip all\ntests if dependencies are not met. However, this function was checking\nfor two features, but most tests only need one of them, so tests could\nbe skipped for no reason.\n\nNow we new specific requires decorators:\n\n- @requires_brctl   raise SkipTest if running brctl fails\n- @requires_tc      raise SkipTest if creating a bridge and adding a\n                    qdisc fails.\n\nnettestlib.checkDependencies() was creating a bridge and deleting it for\ntesting if we can run brctl. This could leave a stale bridge if deleting\nthe device failed. Now we use safer \"brctl show\".\n\nEach test class is using now the specific requires decorator needed.\n\nChange-Id: I9211cb491b1b6f5e7a8880559ec1e1ca3804e92a\nSigned-off-by: Nir Soffer <nsoffer@redhat.com>\nReviewed-on: https://gerrit.ovirt.org/44218\nReviewed-by: Ido Barkan <ibarkan@redhat.com>\nReviewed-by: Dan Kenigsberg <danken@redhat.com>\nContinuous-Integration: Jenkins CI","distinct":true,"url":"https://api.github.com/repos/oVirt/vdsm/commits/abcf220d9f17fb1fd1a574ac3d623ad95455b549"}]}