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.

Sign up for GitHub

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

Jump to bottom

pyats validate testbed: Raise all errors at the same time #11

Open

extsoft opened this issue Nov 1, 2018 · 1 comment

Assignees

Labels

extsoft commented Nov 1, 2018

If there are several errors, the only first error is displayed during the validation.
Please check out the sample below:

(pyats) ~/tmp pyats validate testbed  tb.yaml 
Loading testbed file: tb.yaml
--------------------------------------------------------------------------------

devices.vm: Missing keys: ['connections']

(pyats) ~/tmp cat tb.yaml 
devices:
    vm:
      os: 'linux'
      dd:
        f: a
      type: 'linux'
      ddd: ddd
(pyats) ~/tmp

It will be great if we see all possible errors at the same time.

extsoft changed the title ~~pyats command: Raise all errors at the same time~~ pyats validate testbed: Raise all errors at the same time

simingy added the enhancement label

Contributor

simingy commented Nov 1, 2018

not exactly easy to fix. there's a lot of recursion on these validation logic.

tahigash self-assigned this

tahigash assigned bastell and unassigned tahigash

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment