-
Notifications
You must be signed in to change notification settings - Fork 42
feat(): layer2 support for CAPP #787
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
Closed
Closed
Changes from 8 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
05cbc0c
API and CRD changes for adding layer2 support
rahulii 15aaba7
json tag fix
rahulii ebfddac
change ipAddressReservation to address
rahulii 1d2b71e
make changes to the api as per design doc changes
rahulii e1a19b5
minor change
rahulii cdf7155
api changes and user-data implementation
rahulii 4cfb7b8
add IPAddressManagement Service
rahulii f762e5b
merge bootstrap and layer2 cloud configs
rahulii a452289
add unit test cases
rahulii 5f33a86
ipam support
vardhaman-surana aad4223
added rbac
vardhaman-surana 25510bd
minor fix
rahulii 16fdf8d
reconcile port settings for layer2 and bonded port
rahulii b7e1120
minor fix
rahulii fcfcaac
refactor code and add static routes
rahulii b7e871a
bug fix
rahulii ddf37d9
Merge pull request #1 from rahulii/ipam
rahulii d901fe6
delete unused code
rahulii c55ea3f
minor fix
rahulii 4c64c5d
minor fix
rahulii d4d4b25
change failure to failed in state
rahulii 4d30dcc
added ipaddress claim deletion
vardhaman-surana 4f9c8fc
Merge pull request #2 from vardhaman-surana/ipaddressclaim-deletion
rahulii 4efbfd8
add logs
rahulii 07516df
add routes at network file and change user-data to make them persistent
rahulii 6207973
fix a bug where multiple vxlans are involved
rahulii 7e68faf
update design docs
rahulii 475365c
update design docs
rahulii File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you specify the expected format here ? Also add a validation if the format in valid.
Also can a user configure multiple address ranges as below ?
e.g. 10.60.10.2-10.60.10.9, 10.60.10.11-10.60.10.20