-
Notifications
You must be signed in to change notification settings - Fork 186
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
Rootless kvm and podman #1415
Rootless kvm and podman #1415
Conversation
Hello @dmach! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2023-09-29 07:55:28 UTC |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1415 +/- ##
=======================================
Coverage 30.38% 30.38%
=======================================
Files 49 49
Lines 17698 17701 +3
=======================================
+ Hits 5377 5379 +2
- Misses 12321 12322 +1
☔ View full report in Codecov by Sentry. |
2ff7aaf
to
eba7f84
Compare
The current conf.get_config() function already handles loading from ENV. Also, always use osc.build.calculate_build_root() instead of making a custom variable substitution.
…vm types To avoid filesystem permission collisions with the builds using su_wrapper, use an alternative buildroot path that appends username to '/var/tmp/build-root' for the rootless builds.
…uild and osc --vm-type option
eba7f84
to
41ce932
Compare
Fixes: #1410
Requires: openSUSE/obs-build#954