Skip to content

Conversation

bwbroersma
Copy link
Collaborator

@bwbroersma bwbroersma commented Jul 18, 2025

grep -rlF '172.43.0.' | xargs sed -r -i 's/172\.43\.0\./172.16.43./g;Te;s@/16@/24@;:e'

@mxsasha mxsasha requested a review from aequitas July 18, 2025 12:19
Copy link
Collaborator

@aequitas aequitas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed /16 -> /24 for the subnets to fix the tests. For the rest looks good.

    grep -rlF '172.43.0.' | xargs sed -r -i 's/172\.43\.0\./172.16.43./g;Te;s@/16@/24@;:e'
@bwbroersma bwbroersma force-pushed the gh1841-use-private-ipv4-space-in-tests branch from 0f755b9 to e165b89 Compare July 18, 2025 13:37
@bwbroersma
Copy link
Collaborator Author

bwbroersma commented Jul 18, 2025

Thanks for the hint, I was working on the commit / sed 3 days ago (when opening the issue), and knew I also needed to fix the subnet, but when I wanted to update the redesign branch today and saw this one open, I totally forgot. Anyway, I added some conditional branch and replacement to sed (;Te;s@/16@/24@;:e) to also fix the subnet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Use private IPv4 address space in tests

2 participants