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

Need for Basic sanity check for yaml files in Travis CI #2719

Open
AYUSHJAIN951 opened this issue Oct 11, 2023 · 2 comments
Open

Need for Basic sanity check for yaml files in Travis CI #2719

AYUSHJAIN951 opened this issue Oct 11, 2023 · 2 comments

Comments

@AYUSHJAIN951
Copy link
Member

With in Travis CI can there be a way to do have a basic sanity check for yaml's?
Like use of "=" should be catched initially itself and asked to update to ":", and other basic checks could be incorporated as well.

For eg in #2654 we can see there is issue with yaml which went in without a check and causes issues like below later.

avocado run --max-parallel-tasks 1 memhotplug.py -m memhotplug.py.data/memhotplug.yaml
JOB HTML : /home/AvocadoTests_src/results/job-2023-10-11T05.23-3e7f92e/results.html
JOB TIME : 434.15 s
JOB ID : a7793f2978f3fa74b9b234b1d51f34f4f43375d6
JOB LOG : /home/AvocadoTests_src/results/job-2023-10-11T05.36-a7793f2/job.log
(1/4) memhotplug.py:MemStress.test_hotplug_loop;run-fb7c: STARTED
Invalid multiplex file 'memhotplug.py.data/memhotplug.yaml': mapping values are not allowed in this context
in "memhotplug.py.data/memhotplug.yaml", line 2, column 10
Make sure !tags and colons are separated by a space (eg. !include :) : memhotplug.py.data/memhotplug.yaml

@AYUSHJAIN951
Copy link
Member Author

@AYUSHJAIN951
Copy link
Member Author

Related PR #2718

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