M-Labs/it-infra
M-Labs/it-infra
8
0
Fork 6
Code Issues 11 Pull Requests 1 Releases Wiki Activity

Break cycle dependency of tunnel netdev services on network setup #47

Merged
sb10q merged 1 commits from esavkin/it-infra:34-network-cycle-dependency-fix into master 2024-10-30 17:42:30 +08:00
Conversation 1 Commits 1 Files Changed 1 +8
esavkin commented 2024-10-30 17:24:48 +08:00
Owner
Copy Link

This changes the following chain after nixos-rebuild switch with modified tunnel interfaces:
stop network-setup -> stop TUN-netdev -> stop network-addresses-TUN -> start network-addresses-TUN (fails since it depends on TUN-netdev which is off).

Chain after this change:
stop TUN-netdev -> stop network-setup -> stop network-addresses-TUN -> start TUN-netdev -> start network-addresses-TUN -> start network-setup

Closes #34

This changes the following chain after nixos-rebuild switch with modified tunnel interfaces: stop network-setup -> stop TUN-netdev -> stop network-addresses-TUN -> start network-addresses-TUN (fails since it depends on TUN-netdev which is off). Chain after this change: stop TUN-netdev -> stop network-setup -> stop network-addresses-TUN -> start TUN-netdev -> start network-addresses-TUN -> start network-setup Closes #34
esavkin added 1 commit 2024-10-30 17:24:49 +08:00 
This changes the following chain after nixos-rebuild switch with modified tunnel interfaces:
stop network-setup -> stop TUN-netdev -> stop network-addresses-TUN -> start network-addresses-TUN (fails since it depends on TUN-netdev which is off).

Chain after this change:
stop TUN-netdev -> stop network-setup -> stop network-addresses-TUN -> start TUN-netdev -> start network-addresses-TUN -> start network-setup

Signed-off-by: Egor Savkin <es@m-labs.hk>
sb10q commented 2024-10-30 17:25:22 +08:00
Owner
Copy Link

This is a NixOS bug, please fix it properly and submit a PR upstream.

This is a NixOS bug, please fix it properly and submit a PR upstream.
sb10q merged commit b1779b57cc into master 2024-10-30 17:42:30 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/it-infra#47
No description provided.
Delete Branch "esavkin/it-infra:34-network-cycle-dependency-fix"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?