Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reconsider 'all'; allow "not" #5

Open
danrue opened this issue Mar 27, 2018 · 1 comment
Open

Reconsider 'all'; allow "not" #5

danrue opened this issue Mar 27, 2018 · 1 comment

Comments

@danrue
Copy link
Contributor

danrue commented Mar 27, 2018

It turns out it is not necessary to have to specify "boards: all", "branches: all", etc. Such behavior should simply be the default, and the fields 'environments', 'boards', and 'branches', should be optional. If they are not specified, "all" is implied.

Once that is true, it can also make sense to implement inverse matching, so that you can specify e.g. "all boards except hikey".

For example, the following skiplist would skip test_verifier on all environments and branches, and on all boards except for hikey.

skiplist:
  - reason: kernel tests baselining
    url: https://projects.linaro.org/projects/CTT/queues/issue/CTT-585
    boards:
      - !hikey
    tests:
      - test_verifier

While we're making things easier, all fields except 'tests' can become optional.

@danrue
Copy link
Contributor Author

danrue commented Mar 27, 2018

The following should probably be an error (what does it mean to say not one board, and then specify another board), which makes me wonder if there is a better approach available.

skiplist:
  - reason: kernel tests baselining
    url: https://projects.linaro.org/projects/CTT/queues/issue/CTT-585
    boards:
      - !hikey
      - db410c
    tests:
      - test_verifier

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant