🐛 Throw error and exit on netplan parse exceptions (LP: #2104373)#551
🐛 Throw error and exit on netplan parse exceptions (LP: #2104373)#551Lirt wants to merge 6 commits intocanonical:mainfrom
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 checksuccessfully. --> Not successfulmake check-coverage). --> Not successful