Skip to content

Commit

Permalink
[github-actions] fix NCP border routing setup (#2622)
Browse files Browse the repository at this point in the history
The commit fixes issue in the github action for testing NCP border
routing.

Without `sudo` the option cannot be set successfully, and border
routing cannot work in the test. Currently the expect script isn't
strong enough to detect this problem. I'll enhance the script soon.
  • Loading branch information
Irving-cl authored Nov 26, 2024
1 parent 094f9c1 commit 17c2b1a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tests/scripts/ncp_mode
Original file line number Diff line number Diff line change
Expand Up @@ -172,8 +172,8 @@ setup_infraif()
else
echo "backbone1 already exists."
fi
sysctl -w net.ipv6.conf.backbone1.accept_ra=2
sysctl -w net.ipv6.conf.backbone1.accept_ra_rt_info_max_plen=64
sudo sysctl -w net.ipv6.conf.backbone1.accept_ra=2
sudo sysctl -w net.ipv6.conf.backbone1.accept_ra_rt_info_max_plen=64
}

test_setup()
Expand Down

0 comments on commit 17c2b1a

Please sign in to comment.