plainbox silently ignores dead entries in whitelists

[Zygmunt Krynicki]

This is a composite bug but we basically need all the support required for plainbox to detect dead (or broken, e.g. typos) entries in whitelists. We want to have a way to ensure that our whitelists are correct and preferrably do that earlier than users telling us some tests are being skipped or checkbox-old choking on stale entries.

[Zygmunt Krynicki (zyga)]

We should modify select_jobs() to return annotated qualifier list. It might be challenging to map that back to the source representation but I'm sure we can figure something out. The information should be displayed by `manage.py validate`. It should not be something that operators running the tests should be warned about every time they run pre-existing tests.

[Zygmunt Krynicki (zyga)]

I think this should be handled by the (inexisting) whitelist validator. On command line we can probably provider similar hints for each of the -w / -i options used

[Zygmunt Krynicki (zyga)]

Can we try => future instead the next time. As before, we're not
working on the majority of bugs each two weeks. Let's schedule stuff
to future and re-schedule only those that have a chance of landing.

On Mon, Jul 28, 2014 at 5:30 PM, Sylvain Pineau
<email address hidden> wrote:
> ** Changed in: plainbox
> Milestone: 0.9 => 0.11
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1262251
>
> Title:
> plainbox silently ignores dead entries in whitelists
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/plainbox/+bug/1262251/+subscriptions

[Zygmunt Krynicki (zyga)]

I think this is much, much better now. New style validation captures this issue pretty well. The remaining things that could be fixed to get this better are related to handling validation of command line arguments like 'run -i' but I don't think we need anything more on the whitelist front itself.

Oh, we obviously need to enable the new style validation by default everywhere and get rid of the old code.