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

Add singlepointipam #1093

Merged
merged 1 commit into from
Nov 8, 2021
Merged

Conversation

ljkiraly
Copy link
Contributor

Signed-off-by: Laszlo Kiraly laszlo.kiraly@est.tech

Description

singlepointipam provides an IPAM server chain element. The server provides IPv4/IPv6 addresses from a given address range and updates the IP context with a new destination address, but the source address is generated once.

Issue link

#951

How Has This Been Tested?

  • Added unit testing to cover
  • Tested manually
  • Tested by integration testing
  • Have not tested

Types of changes

  • Bug fix
  • New functionallity
  • Documentation
  • Refactoring
  • CI

Signed-off-by: Laszlo Kiraly <laszlo.kiraly@est.tech>
@denis-tingaikin
Copy link
Member

@glazychev-art Could you please review this when you will have time? Thanks.

if dstSet {
return &connectionInfo{
ipPool: sipam.ipPools[i],
srcAddr: srcAddr.String() + sipam.masks[i],
Copy link
Contributor

Choose a reason for hiding this comment

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

do we also need non /32 mask for srcAddr?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is meant to be used for external communication also. The client pod which getting this address should be connected to a router and should belong to wider subnet, for example /24.

@edwarnicke edwarnicke merged commit 113a36b into networkservicemesh:main Nov 8, 2021
nsmbot pushed a commit to networkservicemesh/cmd-nse-vfio that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-kernel that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-k8s that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-init that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-proxy-dns that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-admission-webhook-k8s that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-memory that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr-proxy that referenced this pull request Nov 8, 2021
…k@main

PR link: networkservicemesh/sdk#1093

Commit: 113a36b
Author: Laszlo Kiraly
Date: 2021-11-08 22:04:42 +0100
Message:
  - Add singlepointipam (#1093)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
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.

4 participants