#1 New attribute "disabled"
Merged by psss. Opened by psss.
enabled  into  master

Download 1.patch

As discussed on the Best Practices meeting, this pull request is proposing a new attribute to allow disabling individual tests.

@mvadkert, @plautrba, @jheger, @pcahyna, @vdolezal could you please review? Thanks.

rebased onto ae8be78d236b76e2308d39c8ea89cb0e657e4a84

Added default value. @plautrba, thanks for the idea.

Looks good to me :thumbsup:

Purpose, motivation and stories talk about disabling tests, while the attribute is called "enabled". It's a bit strange for me, but I have no real objection.

hmm, maybe I would even say @plautrba is right here, and would politely opt-in for disabled

rebased onto 06e369beb202cf3c0b3842d3d3216b854f97f94e

Makes sense. Sounds better to me as well. I've updated the pull request accordingly.

Pull request created to add support for disabling tests in standard-test-roles:
https://pagure.io/standard-test-roles/pull-request/266

Metadata Update from @psss:
- Request assigned

Pull-Request has been merged by psss

Metadata Update from @psss:
- Pull-request tagged with: test metadata

Metadata