Skip to content

Commit 95cb9ca

Browse files
authored
Merge pull request #683 from stgraber/main
incus-osd/systemd/networkd: Only trigger network devices
2 parents d5c7f68 + 3c6f5af commit 95cb9ca

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

incus-osd/internal/systemd/networkd.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -152,11 +152,14 @@ func ApplyNetworkConfiguration(ctx context.Context, s *state.State, networkCfg *
152152
// Refresh registration, delaying by 30 seconds if needed to allow the provider to become available,
153153
// such as when IncusOS is self-hosting Operations Center.
154154
if refresh != nil {
155-
go func() {
155+
go func() { //nolint:contextcheck
156156
if delayRefreshCheck {
157157
time.Sleep(30 * time.Second)
158158
}
159159

160+
ctx, ctxCancel := context.WithTimeout(context.Background(), 30*time.Second)
161+
defer ctxCancel()
162+
160163
err := refresh(ctx, s)
161164
if err != nil {
162165
slog.WarnContext(ctx, "Failed to refresh provider registration", "err", err)
@@ -689,7 +692,7 @@ func waitForUdevInterfaceRename(ctx context.Context, expectedInterfaces []string
689692
}
690693

691694
// Trigger udev rule update to pickup device names.
692-
_, err := subprocess.RunCommandContext(ctx, "udevadm", "trigger", "--action=add")
695+
_, err := subprocess.RunCommandContext(ctx, "udevadm", "trigger", "--action=add", "--subsystem-match=net")
693696
if err != nil {
694697
return err
695698
}

0 commit comments

Comments
 (0)