-
Notifications
You must be signed in to change notification settings - Fork 16
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
Remove duplicated IPAM code for NSE applications #651
Remove duplicated IPAM code for NSE applications #651
Conversation
a3abb3a
to
1c97fb0
Compare
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.
Let's not modify the copyright holders of other companies ;)
In general, LGTM.
main.go
Outdated
@@ -1,6 +1,8 @@ | |||
// Copyright (c) 2021-2022 Doc.ai and/or its affiliates. | |||
// Copyright (c) 2021-2024 Doc.ai and/or its affiliates. |
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.
// Copyright (c) 2021-2024 Doc.ai and/or its affiliates. | |
// Copyright (c) 2021-2022 Doc.ai and/or its affiliates. |
main.go
Outdated
// | ||
// Copyright (c) 2023 Cisco and/or its affiliates. | ||
// Copyright (c) 2023-2024 Cisco and/or its affiliates. |
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.
// Copyright (c) 2023-2024 Cisco and/or its affiliates. | |
// Copyright (c) 2023 Cisco and/or its affiliates. |
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.
Thank you, I updated it.
NSM issue link: networkservicemesh/sdk#1409 Signed-off-by: Botond Szirtes <[email protected]>
1c97fb0
to
6ccc85c
Compare
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.
Looks good. Thanks!
…d-nse-icmp-responder-vpp@main PR link: networkservicemesh/cmd-nse-icmp-responder-vpp#651 Commit: 6e0c325 Author: Denis Tingaikin Date: 2024-04-16 15:45:37 +0300 Message: - Merge pull request #651 from Nordix/remove-duplicated-ipam-code Signed-off-by: NSMBot <[email protected]>
NSM issue link: networkservicemesh/sdk#1409