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
Avoid stopping qubes-network-uplink@eth0.service #442
base: main
Are you sure you want to change the base?
Conversation
This is already started by qubes-network-uplink.service, so stopping it is a bad idea. Since setup-ip is not idempotent, the result is a down network interface. Fixes: QubesOS/qubes-issues#8362 Fixes: QubesOS/qubes-issues#8305
Codecov Report
@@ Coverage Diff @@
## main #442 +/- ##
=======================================
Coverage 70.63% 70.63%
=======================================
Files 3 3
Lines 470 470
=======================================
Hits 332 332
Misses 138 138 |
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.2&build=2023071717-4.2&flavor=pull-requests New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.2&build=2023071104-4.2&flavor=update
Failed tests31 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/77326#dependencies 18 fixed
Unstable tests
|
One of those two PRs break most of dynamic network attach, my bet is on this one. My guess is: |
This is already started by qubes-network-uplink.service, so stopping it is a bad idea. Since setup-ip is not idempotent, the result is a down network interface.
Fixes: QubesOS/qubes-issues#8362
Fixes: QubesOS/qubes-issues#8305