-
Notifications
You must be signed in to change notification settings - Fork 222
🐛 Throw error and exit on netplan parse exceptions (LP: #2104373) #551
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
base: main
Are you sure you want to change the base?
Conversation
Still in WIP. Working on executing local tests. |
Exceptions related to netplan parse or validate operations are supposed to throw error and return proper error code. Resolves https://bugs.launchpad.net/netplan/+bug/2104373 Signed-off-by: Ondrej Vasko <ondrej.vaskoo@gmail.com>
d4ca613
to
ddcd795
Compare
Signed-off-by: Ondrej Vasko <ondrej.vaskoo@gmail.com>
Signed-off-by: Ondrej Vasko <ondrej.vaskoo@gmail.com>
- With virtualenv pyflakes and coverage have different names - Explicitly name pyflakes directories to check. Alternative could be using flake8 - Skip codestyle check for .venv and build Signed-off-by: Ondrej Vasko <ondrej.vaskoo@gmail.com>
Legacy tests in my setup are failing like this. Is this my problem of my environment, or long-term issue?
|
Signed-off-by: Ondrej Vasko <ondrej.vaskoo@gmail.com>
Memory sanitizer is failing
Again I don't think there is change in this MR that could cause this. Pipeline in main branch passed fine and this PR is rebased on main. |
Hello, I need help with this PR. Either something is wrong with CI, or adding Some pipelines fail due to unrelated errors (see #551 (comment)). Unit tests are passing well locally. Not in CI. I'm new netplan contributor. The setup for this repository is quite unhandy. For example - why does the project lack List of packages that were needed to install are:
I did unrelated change to update meson to exclude some files and allow to find virtualenv python packages for testing (coverage, ...). I can open new PR for that if that is cleaner. |
Signed-off-by: Ondrej Vasko <ondrej.vaskoo@gmail.com>
Exceptions related to netplan parse or validate operations are supposed to throw error and return proper error code.
Resolves/Closes https://bugs.launchpad.net/netplan/+bug/2104373
Description
Described in launchpad issue.
Checklist
make check
successfully. --> Not successfulmake check-coverage
). --> Not successful