-
Notifications
You must be signed in to change notification settings - Fork 31
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
Support old ubuntu netplan #1031
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1031 +/- ##
=======================================
Coverage 16.36% 16.36%
=======================================
Files 109 109
Lines 19628 19628
=======================================
Hits 3212 3212
Misses 16135 16135
Partials 281 281
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: yaacov <[email protected]>
82bc3f6
to
26485bb
Compare
Signed-off-by: yzamir <[email protected]>
Signed-off-by: yzamir <[email protected]>
a735889
to
1043a90
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Signed-off-by: yzamir <[email protected]>
Signed-off-by: yzamir <[email protected]>
Signed-off-by: yzamir <[email protected]>
Quality Gate passedIssues Measures |
Support old ubuntu netplan
Issue:
Ubuntu 18 uses vertion of netplan that does not support the get subcommand
Fix:
a. add a python script that mimics netplan get command
b. check if current version of netplan support get
c. run the python script in cases netplan does not support get