Граф коммитов

14 Коммитов

Автор SHA1 Сообщение Дата
Behzad Mirkhanzadeh 4370e6e93c
feat: Adding MACAddress to CNS endpoint State and refactoring stateless CNI code (#2722)
* Adding MACAddress to CNS endpointState and also refactoring statelss cni code

* Addressing the comments.

* Addressing the comments
2024-05-08 00:07:03 +00:00
Behzad Mirkhanzadeh b22ac38f3e
feat: Migration stateless cni (#2470)
* 🌈 feat: adding flags for stateless cni (#2103)

feat: stateless cni

* feat: create stateless cni binary for swift (#2275)

* enabling CNS telemetry

* CNI Telemetry enabled on CNS

* Code changes for Statefull CNI Migration

* Making changes to the CNI state migration code.

* Make code changes for Stateless CNI migrations.

* Make changes to statless CNI migration branch.

* Stateless CNI migration code changes

* resolving migration issue

* remove cni changes

* Applying changes to CNIReonciler

* Addressing the comments.

* Addressing the comments

* addressing the latest comments

* Addressing Evan's comments

* Adding a MigrateSate() function to the cnireconciler

---------

Co-authored-by: Vipul Singh <vipul21sept@gmail.com>
2024-02-08 23:47:56 +00:00
Ramiro c8005ed94b
CNS - consider multiple IPs for a pod in reconciliation after restart (#2079)
* modifying reconcile function to take multiple ncs instead of one. we need this because for reconciliation in dual stack cases both IPs for a pod which can come from distinct ncs must be added at the same time

* adding comments, and renaming functions and types, to make the intent clearer

* adding some dummy values to cns.NewPodInfo invocations in tests, instead of empty strings since we need distinct interface ids

* adding a basic test for dual stack reconciliation

* adding validation for multiple ips for the same ip family on the same pod, which is not allowed

* changing direct use of interface id to pod key, which is better for reconcile flows using information from kubernetes instead of cni

* fixing comments to use host network terminology instead of system pod

* improving error message on duplicate ip from cni found; improving readability of error handling on ip parsing

* only checking for pod ips that are actually set on a pod
2023-08-02 01:49:21 +00:00
Matthew Long cc251ca58a
fix: assume invalid semver CNI has the required dump state command (#2078) 2023-07-26 09:52:27 -07:00
tamilmani1989 d4df70838e
[Fix][CNS] InterfaceID should be set to containerID instead of interfaceName if CNS managing state (#2023)
* InterfaceID should be set to containerID instead of interfaceName if CNS manage endpoint state is enabled

* add log in GetExistingIPConfig function to debug

* Fix UT (set containerid as pod interface id)
lint fix
2023-06-20 11:33:37 -07:00
Quang Nguyen 60e5a26565
feat: CNS manages endpoints state for delegate IPAM use case (#1500)
* rebase

* linting

* rebase

* missing if condition for releaseIPConfig

* update azure-cns.yaml and add UTs

* rebase

* update program iptables changes

* linting

* fix broken tests

* fix podinfoprovider returns error when key is not found

* log when no endpoint state exist when reconcilling

* not remove endpoint state file on failure to read in restserver.restoreState()

* addressed comments

* update acn tag

* go get on acn

* addressed comments

Co-authored-by: Evan Baker <rbtr@users.noreply.github.com>
2022-08-23 16:42:40 -07:00
Paul Johnston 6d208e9762
Cns windows aks (#1059)
* chore: add in some functionality for CNS on windows host process pods
2021-11-22 09:23:01 -08:00
Evan Baker 96bec09d41
chore: appease the linter (3/?), the big gofumpt (#987)
* gofumpt -w -s .

* small addtl cleanups after gofumpt

* rerun after rebase
2021-09-02 16:33:18 -05:00
Evan Baker ac0bede752
feat: crash cns when dupe ips are found during reconcile (#939) 2021-09-01 18:35:19 -05:00
Evan Baker 617f898b12
fix: bump minimum cni to 1.4.7 for reconcile flow (#948)
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
2021-07-20 18:06:35 -05:00
Evan Baker 8bf1a2418d
fix: add addtl log context when cns mutates cni statefile (#935) 2021-07-14 18:49:24 -05:00
Evan Baker e33035520d
write empty json object to empty cni statefile if exists (#933)
* write empty json object to empty cni statefile if exists

* update tests, address review comments
2021-07-14 16:01:35 -07:00
Mathew Merrick 9b24dbd95a
test: [NPM] Use fakeexec for ipsm and iptm tests (#868)
* iptmgr

* more iptm testing

* grep call

* progress

* progress

* ipsm

* ioshim

* update tests

* package restructure

* fix broken test and delint

* reduce scope of ioshim

* reduce scope of ioshim

* ioshim scope

* require no error, retrigger ci

* ut return multiple results

* fix tests from master changes

* unexport ioshim

* update ut

* fix tests

* vendor

* test fix

* go version

* go version

* pipeline fixes

* fix tests
2021-07-14 12:53:45 -07:00
Evan Baker 7d224bf3a6
feat: add flow to initialize CNS from CNI (#890)
* feat: add flow to initialize cns state from cni

* address review comments

* Rename the PodIp map

* fix test

* fix version check

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
Co-authored-by: neaggarw <neaggarw@microsoft.com>
2021-06-29 17:14:11 -05:00