Skip to content
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

fix: ips are on the range 172.16.0.0/16 #618

Merged
merged 5 commits into from
May 12, 2023
Merged

Conversation

h4ck3rk3y
Copy link
Contributor

@h4ck3rk3y h4ck3rk3y commented May 12, 2023

Description:

Use Dockers Network Range to fix weirdness we are seeing with AVAX tests

This limits us to 256 APICs (like we had earlier)
This limits us to 256 services per APIC

Is this change user facing?

NO

References (if applicable):

@h4ck3rk3y h4ck3rk3y changed the title change networking logic fix: change networking logic May 12, 2023
@h4ck3rk3y h4ck3rk3y changed the title fix: change networking logic fix: APIC ips range from 172.16.1.0/24 to 172.16.255.0/24 May 12, 2023
@h4ck3rk3y h4ck3rk3y changed the title fix: APIC ips range from 172.16.1.0/24 to 172.16.255.0/24 fix: ips are on the range 172.16.0.0/12 May 12, 2023
@h4ck3rk3y h4ck3rk3y changed the title fix: ips are on the range 172.16.0.0/12 fix: ips are on the range 172.16.0.0/16 May 12, 2023
@h4ck3rk3y h4ck3rk3y merged commit b48cb73 into main May 12, 2023
@h4ck3rk3y h4ck3rk3y deleted the gyani/fix-networking branch May 12, 2023 12:16
h4ck3rk3y pushed a commit that referenced this pull request May 12, 2023
🤖 I have created a release *beep* *boop*
---


##
[0.76.6](0.76.5...0.76.6)
(2023-05-12)


### Bug Fixes

* ips are on the range 172.16.0.0/16
([#618](#618))
([b48cb73](b48cb73))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: kurtosisbot <kurtosisbot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants