ci: add CNIv2 testing for different proc/OS #2230

jshr-w · 2023-09-20T18:31:31Z

Reason for Change:

Extends CNIv2 overlay load testing to different processors and operating systems, namely: ARM+Ubuntu22, AMD+Mariner, and AMD+Infiniband.

Issue Fixed:

Requirements:

uses conventional commit messages
includes documentation
adds unit tests
relevant PR labels added

Notes:

vipul-21 · 2023-09-20T18:33:11Z

/azp run CNI Load Test

azure-pipelines · 2023-09-20T18:33:24Z

Azure Pipelines successfully started running 1 pipeline(s).

hack/aks/Makefile

.pipelines/cni/pipeline.yaml

hack/aks/Makefile

vipul-21 · 2023-09-27T16:53:44Z

/azp run CNI Load Test

azure-pipelines · 2023-09-27T16:53:56Z

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: jshr-w <[email protected]>

vipul-21 · 2023-09-28T22:35:20Z

.pipelines/cni/load-test-templates/create-cluster-template.yaml

@@ -15,13 +16,15 @@ steps:
      inlineScript: |
        set -ex
        make -C ./hack/aks azcfg AZCLI=az REGION=${{ parameters.region }}
-
+        


nit: white space should be removed. You can setup your IDE to remove those for you on each save of the file.

vipul-21 · 2023-09-28T22:38:12Z

.pipelines/cni/pipeline.yaml

+      name: mariner_linux_overlay
+      clusterType: overlay-byocni-up
+      clusterName: mariner-cniv2-overlay
+      nodeCount: ${LINUX_NODE_COUNT}


I think this will change in accordance with the PR by @jpayne3506: f0021b5#diff-345f445e69479edda267b386cd089556500ef27605c05b5a9d16c4ae0ca61b29R131-R132

vipul-21 · 2023-09-28T22:38:29Z

.pipelines/cni/pipeline.yaml

+      name: arm_linux_overlay
+      clusterType: overlay-byocni-up
+      clusterName: arm-cniv2-overlay
+      nodeCount: ${LINUX_NODE_COUNT}


Same here as well

.pipelines/cni/pipeline.yaml

.pipelines/cni/load-test-templates/create-cluster-template.yaml

jpayne3506

LGTM!

commit ce5c12b Author: John Payne <[email protected]> Date: Thu Oct 19 08:16:35 2023 -0700 ci: Replace manual install of kubectl with ADO KubectlInstaller task (#2307) * ci: remove kubectl install * ci: add KubectlInstaller for kubectl commit 86e5e13 Author: rjdenney <[email protected]> Date: Tue Oct 17 19:41:00 2023 -0400 Adding dualstack overlay support to azure-ipam plugin for Cilium (#2239) * dualstack cilium changes * remove comment * addressing comments and adding unit tests commit 0b45d15 Author: John Payne <[email protected]> Date: Tue Oct 17 08:57:42 2023 -0700 ci: Enable multiple runs with the same commitID (#2292) * ci: add unique value based on pipeline runtime Signed-off-by: John Payne <[email protected]> * ci: adjust cluster names * ci: hange commitID on load-test pipeline --------- Signed-off-by: John Payne <[email protected]> commit e62e44e Author: John Payne <[email protected]> Date: Sat Oct 14 18:51:23 2023 -0700 revert: Cilium load test RBAC (#2291) revert: cilium manifest directories Signed-off-by: John Payne <[email protected]> commit 73d8bfb Author: John Payne <[email protected]> Date: Fri Oct 13 14:06:59 2023 -0700 ci: Add NPM arm64 to CNI Load Test Pipeline (#2289) commit ae37d40 Author: Paul Yu <[email protected]> Date: Fri Oct 13 15:08:58 2023 -0400 [CNI] zap logger for platform package (#2233) * zap logger for platform package commit a3ec127 Author: John Payne <[email protected]> Date: Thu Oct 12 20:22:35 2023 -0700 ci: Update CNI Release Test pipeline (#2283) * ci: update restart node template * ci: parameterize dropgz image commit 6ddc44c Author: John Payne <[email protected]> Date: Thu Oct 12 16:57:44 2023 -0700 ci: Add control through environment variables for CNI Load Test (#2277) * ci: conditional run logic * add: SCENARIO env var control * ci: change RUN to CNI * ci: move SCENARIO to job level * fix: change env vars to unique values * ci: condition logic and description commit 1190646 Author: John Payne <[email protected]> Date: Wed Oct 11 10:31:15 2023 -0700 revert: Latest NPM for integration test (#2263) commit 131cf7a Author: aggarwal0009 <[email protected]> Date: Mon Oct 9 11:43:01 2023 -0700 Fix: Update pni scope to namespaced (#2282) update pni scope to namespaced commit f46a430 Author: John Payne <[email protected]> Date: Mon Oct 9 11:40:53 2023 -0700 ci: Parameterize CNS image repository (#2280) ci: Parameterize CNS image repo commit 5866205 Author: John Payne <[email protected]> Date: Fri Oct 6 14:03:00 2023 -0700 fix: change artifact naming convention (#2278) * fix: matching template artifacts * test: jobName Parameter * ci: capture all test namespaces commit f4dd79c Author: Hunter Gregory <[email protected]> Date: Thu Oct 5 07:20:25 2023 -0700 test(capz): [WIN-NPM] support containerd 1.7 filesystem (#2267) * style: whitespace in all NPM yamls * fix(capz): first try for containerd 1.7 filesystem change * fix(capz): remove kubeconfig arg and add comment commit 728dbb3 Author: Paul Yu <[email protected]> Date: Wed Oct 4 23:07:07 2023 -0400 ci: v4overlay conformance test cases (#2274) v4overlay conformance test cases commit 7a5cb5e Author: Paul Johnston <[email protected]> Date: Wed Oct 4 18:58:14 2023 -0600 chore: install ip-masq-agent as part of overlay cns scenarios (#2273) commit f061370 Author: John Payne <[email protected]> Date: Wed Oct 4 13:44:35 2023 -0700 ci: Agent pool default set at stage level (#2272) commit ae0c08c Author: Paul Yu <[email protected]> Date: Wed Oct 4 14:43:46 2023 -0400 [CNI] zap logger telemetry package (#2266) * zap logger telemetry package commit 64f01b2 Author: John Payne <[email protected]> Date: Tue Oct 3 16:51:12 2023 -0700 ci: Add log template to PR and Load Test Pipeline (#2264) * Initial commit * add: log paths and label to priv. daemonset * add: log template * add: log template to load-test yamls * remove: kubeconfig calls * add: capture failed pods on job failure * add: Linux state files * add: Windows state files * style: change terminal output * add: log template to PR pipeline * fix: rebase * style: add comments to log-template * chore: Addressing Comments * add: sub-directories * ci: Only call log-template on fail for PR commit ad3329a Author: Paul Johnston <[email protected]> Date: Tue Oct 3 12:16:37 2023 -0600 chore: must* functions should panic (#2268) commit 35a6f89 Author: Paul Yu <[email protected]> Date: Mon Oct 2 18:22:42 2023 -0400 ci: dualstack overlay windows test cases (#2262) * dualstack overlay windows test cases commit e874736 Author: John Payne <[email protected]> Date: Fri Sep 29 14:28:43 2023 -0700 ci: Add HNS restart to Windows CNIv1 in Load Test Pipeline (#2261) * add: HNS restart template * add: re-run all tests after HNS restart * style: add provided test coverage to template commit 4423a94 Author: jshr-w <[email protected]> Date: Thu Sep 28 20:17:36 2023 -0700 ci: add CNIv2 testing for different proc/OS (#2230) * ci: proc/os load tests * ci: update deprecated param * ci: update naming * ci: add other proc/os load tests * ci: add other proc/os load tests * remove ubuntu18 test * remove redundant test * fix template call * add infiniband test, arm binary * fix rdma node name, build * update RDMA node count for quota * force pipeline permissions * undo force permissions * remove hardcoding * add os as parameter for cniv1 * set default osSKU * merge variable name change * set default OSSKU * set default OSSKU --------- Signed-off-by: jshr-w <[email protected]> commit 9200af8 Author: Timothy J. Raymond <[email protected]> Date: Thu Sep 28 15:36:15 2023 -0400 Add OpenAPI doucmentation to CNS (#1461) Add initial swagger documentation for CNS This adds documentation for a few of the endpoints in CNS. It's not exhaustive, but it's a place to start. commit 78a577c Author: Paul Yu <[email protected]> Date: Wed Sep 27 19:36:48 2023 -0400 v4overlay windows test cases (#2187) * feat: adding in v4overlay windows tests * chore: address feedback * fix: addressing feedback --------- Co-authored-by: Paul Johnston <[email protected]> commit a3e6682 Author: Evan Baker <[email protected]> Date: Wed Sep 27 18:29:39 2023 -0500 submodule dependency updates (#2242) Signed-off-by: Evan Baker <[email protected]> commit f0021b5 Author: John Payne <[email protected]> Date: Wed Sep 27 13:23:52 2023 -0700 fix: cluster template env vars (#2258) * fix: cluster templates * cherry-pick: 0c8106a * chore: addressing comments commit 982b794 Author: Jaeryn <[email protected]> Date: Wed Sep 27 11:49:43 2023 -0700 feat: update cns network container contract for swift 2 (#2250) Co-authored-by: Jaeryn <[email protected]> Co-authored-by: Quang Nguyen <[email protected]> commit 6251874 Author: Evan Baker <[email protected]> Date: Tue Sep 26 18:18:16 2023 -0500 fix: set cluster auto-upgrade and node auto-upgrade in the hackfile (#2253) Signed-off-by: Evan Baker <[email protected]> commit 0abdc9d Author: Vipul Singh <[email protected]> Date: Tue Sep 26 14:53:56 2023 -0700 ci: removing the submodules pipeline as PR pipeline covers the checks (#2251) commit e4cefac Author: Evan Baker <[email protected]> Date: Tue Sep 26 16:53:18 2023 -0500 Create initial contrbuting guides (#2244) * start creating contrbuting guides Signed-off-by: Evan Baker <[email protected]> * add CLA instructions Signed-off-by: Evan Baker <[email protected]> --------- Signed-off-by: Evan Baker <[email protected]> commit 53d114c Author: Vipul Singh <[email protected]> Date: Tue Sep 26 13:33:58 2023 -0700 fix: skipping the k8s e2e unsupported service conformance test (#2255) commit f28fe96 Author: Hunter Gregory <[email protected]> Date: Mon Sep 25 17:11:52 2023 -0700 test: [NPM] skip cyclonus for test dir (#2247) Signed-off-by: Hunter Gregory <[email protected]> commit beeb66a Author: Paul Johnston <[email protected]> Date: Mon Sep 25 15:40:26 2023 -0600 feat: installing windows cns if windows nodes present (#2246) commit 4943198 Author: Paul Yu <[email protected]> Date: Mon Sep 25 11:59:56 2023 -0400 [CNI] zap logger migration for store package (#2231) * zap logger migration for store package commit 3c6bb62 Author: Evan Baker <[email protected]> Date: Fri Sep 22 17:07:27 2023 -0700 proposal: redesign of CNS IPAM (#2013) proposal: redesign of cns ipam Signed-off-by: Evan Baker <[email protected]> commit 22bc8c7 Author: Rajvi <[email protected]> Date: Fri Sep 22 16:02:57 2023 -0700 fixing config.channelmode to execute go routine for Mellanox (#2240) Co-authored-by: Ashvin Deodhar <[email protected]> commit 5fa9eda Author: rjdenney <[email protected]> Date: Fri Sep 22 17:44:37 2023 -0400 Adding overlay v4/dualstack conflists to Windows Dropgz (#2224) adding overlay v4/dualstack conflists to Windows commit 8760107 Author: John Payne <[email protected]> Date: Fri Sep 22 13:52:31 2023 -0700 ci: Add CNIv1 Linux to Load Test Pipeline (#2241) * refactor: single cniv1 template * Add: handle multiple nodepools * Add: linux cniv1 validate * ci: move NPM|CNI integration * ci: Update subscription used * addressing comments commit ec519a5 Author: Paul Johnston <[email protected]> Date: Fri Sep 22 10:18:30 2023 -0600 chore: refactor load test suite (#2229) commit a61940d Author: Ashish Nair <[email protected]> Date: Thu Sep 21 19:50:29 2023 -0700 feat: Consume the NCStatus to be able to append subnet is full error to Pod IP reservation failures (#2202) * Added Network Conatiner Status to include the latest error code for a Network Container * Updated the crd to have the Status field included into the Network Container * Updated the names and added Status and ErrorText as two fields in NC Status * Fixed the casing and json values for these variables * Propagated the NC Status inside the CNS and IPAM Monitor pool states * Fixed the lint error of missing comma * Saved the updated NC Status into the CNS statefile * Updated the IP assignment to check and error out subnet is Full when there are no more available IPs for CNS to assign * Fixed a minor compilation issue * Fixed lint failures * Fixed lint failures * Removed the reference from the metastate of the ipam monitor * Added Update Success and Update Failed statuses to the NC Status to be able to clearly indicate response status inside the NNC from DNC-RC * Updated the error to use errors pkg instead of fmt * Updating the cns reconcillation logic to skip if there is a failure updating the NC and there are no IPs allocated for the NC * Handled PR comments: * Updated the code to have the NC status be part of the error directly so that it can be consumed by containerD and cx can perform actions on it. * Code update to not use dynamic slices. * Removed the logic which handled 0 IPs allocated to NNC in CNS reconcile Signed-off-by: GitHub <[email protected]> * Addressed the PR comment which helped delete a block of code to store ncIDs and also added more error codes to the NCStatus --------- Signed-off-by: GitHub <[email protected]> commit 1b22180 Author: Camryn Lee <[email protected]> Date: Thu Sep 21 07:45:29 2023 -0700 CNI async delete after ReleaseIPAddress (#2232) add second check for releaseIP failure Co-authored-by: Quang Nguyen <[email protected]> commit 028e162 Author: Evan Baker <[email protected]> Date: Wed Sep 20 14:58:21 2023 -0700 migrate to patch for NNC spec updates (#2188) migrate to patch for nnc spec updates commit 95cc2d6 Author: Evan Baker <[email protected]> Date: Wed Sep 20 14:36:30 2023 -0700 chore: consolidate core dependencies upgrade (#2213) * ci: bump actions/checkout from 3 to 4 Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> * chore: root dependency updates Signed-off-by: Evan Baker <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Evan Baker <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit 2791885 Author: Camryn Lee <[email protected]> Date: Wed Sep 20 10:28:03 2023 -0700 feat: CNS/CNI async delete changes for azure-ipam (#2201) * azure-ipam changes for async delete * cilium cnsconfig change for tests * address comments: update config value and add log line * matching ipam changes to #2183 * include containerID in log msg * update addFile args * return on failure to add file * update go.mod commit c2c59db Author: Paul Johnston <[email protected]> Date: Tue Sep 19 22:05:47 2023 -0600 chore: making installation of CNS daemonset shared (#2227) commit bad286b Author: John Payne <[email protected]> Date: Tue Sep 19 18:40:51 2023 -0700 ci: Enable dropgz-test in PR pipeline (#2116) commit 246fee4 Author: Paul Yu <[email protected]> Date: Tue Sep 19 19:02:28 2023 -0400 [CNI] restruct CNI logger (#2226) * small change for zap log restruct commit 4772008 Author: John Payne <[email protected]> Date: Tue Sep 19 12:37:05 2023 -0700 ci: Add CNIv2 Linux to Load Test pipeline (#2141) * Initial Commit * Add sleep for swift cluster * Change NPM|CNI Integration * Addressing comments * Add: NPM continueOnError * Add: Generate logs for NPM * Change NPM Linux branch - long sleep 10s * refactor: linux validate * fix: rebase * Add: maxSkew for noop deployments * Add: Capture improper node restart * Add: Restart CNS case for Cilium * Addressing Comments * Add: Restart CNS template commit 4fa3bf4 Author: Camryn Lee <[email protected]> Date: Mon Sep 18 21:04:38 2023 -0700 feat: CNS/CNI async pod delete (#2183) * initial changes for cni/cns delete deadlock * add logs and set watcher path * working fswatcher, removing extra debug lines * watcher changes for azure-ipam * remove additional logger from fsnotify and address comments * /deleteIDs directory as part of cnsconfig * add feature flag for async delete * adds some unit test + remove changes for azure-ipam(split pr, dependency conflicts) * update ut * update uts * swift configmap update * fix configmap for test * addressing comments * fix lint * adding cause to connection error struct * connectionerr lint * addressing comments, change watchfs to watcher method * add ctx to releaseIP func * log containerID in failure to add watcher, exit select if context is cancelled * fix logs in network.go after rebase * catch release ip error in invoker_cns.go * retry on failure to release ip * lint fix * rework asyncdelete watcher Signed-off-by: Evan Baker <[email protected]> * include podinterfaceID in file for releaseIP * close file before delete --------- Signed-off-by: Evan Baker <[email protected]> Co-authored-by: Evan Baker <[email protected]> commit 44dc74e Author: Paul Yu <[email protected]> Date: Fri Sep 15 20:14:44 2023 -0400 [CNI] Migrate network and platform package logging to zap (#2209) * network package zap logger * add zaplogger for platform commit c9adf9a Author: rjdenney <[email protected]> Date: Fri Sep 15 18:51:13 2023 -0400 Publishing our Windows dropgz version in linux dropgz manifest (#2218) * publishing windows dropgz * adding 2019 * adding os versions to test * fix arm * removing 2019 cni dropgz test commit 7d479b3 Author: John Payne <[email protected]> Date: Wed Sep 13 11:00:58 2023 -0700 revert: makefile naming for cluster creation (#2214) * revert: cilium naming * Addressing Comments commit d01947f Author: rjdenney <[email protected]> Date: Tue Sep 12 11:49:17 2023 -0400 fix: updating CNI to v1.5.11 for dropgz (#2211) commit c281674 Author: Ramiro <[email protected]> Date: Fri Sep 8 16:15:45 2023 -0700 fix: serviceAccount must be specified at pod spec level in dual stack goldpinger deployments (#2208) service account needs to be specified at the pod spec level, not the container commit 61da686 Author: John Payne <[email protected]> Date: Fri Sep 8 14:27:16 2023 -0700 ci: Increase timeout of datapath tests (#2206) commit 210fe86 Author: Ashish Nair <[email protected]> Date: Fri Sep 8 11:39:17 2023 -0700 Feat: Created a Network Container Status Section to be updated with the latest error code (#2193) * Added Network Conatiner Status to include the latest error code for a Network Container * Updated the crd to have the Status field included into the Network Container * Updated the names and added Status and ErrorText as two fields in NC Status * Fixed the casing and json values for these variables * Added error code to the NC Status and removed the latest prefix from the varibale names * Removed the timestamp variable from the NC Status * Moved the Status object inside the NC Status to be able to accurately define the status of each NC for the node * Changed to having an enum representing the NC Status which DNC-RC will update after inferring the error and CNS can use this field to propagate and NCRequest failures * Made the validation of the new enum optional to keep it backward compatible commit a90a77e Author: Paul Yu <[email protected]> Date: Fri Sep 8 14:36:19 2023 -0400 restruct zap logger for CNI (#2184) * restruct zap logger commit 666f36c Author: Ramiro <[email protected]> Date: Thu Sep 7 20:39:08 2023 -0700 CNS - Ensuring no stale NCs during NNC reconcile (#2192) * ensuring no stale ncs during nnc reconcile * only save state if mutated * ensuring we only remove stale ncs if none of their ips are assigned commit 1360e02 Author: John Payne <[email protected]> Date: Wed Sep 6 18:00:52 2023 -0700 ci: Changing Service Connection for PR pipeline (#2153) * ci: change service connection * add: change ACN PR azureSubscription commit 23e37aa Author: John Payne <[email protected]> Date: Wed Sep 6 11:43:24 2023 -0700 ci: Increase vmSize for dualstack overlay (#2203) fix: Increase vmSize for dualstack overlay Signed-off-by: John Payne <[email protected]> commit e102891 Author: Hunter Gregory <[email protected]> Date: Fri Sep 1 11:25:14 2023 -0700 test(cyclonus): [WIN-NPM] fix consistent failure from not sleeping long enough (#2174) * test(cyclonus): [WIN-NPM] fix consistent failure from not sleeping long enough * build: fix syntax * fix: specify namespace in kubectl * fix: wait timeout=5m Signed-off-by: Hunter Gregory <[email protected]> --------- Signed-off-by: Hunter Gregory <[email protected]> commit 3f313ad Author: Evan Baker <[email protected]> Date: Wed Aug 30 15:54:48 2023 -0500 fix: add OS=windows to manifest-add args (#2194) due to a change in buildx, we need to explicitly pull the windows variant of the windows image that we have previously built. Signed-off-by: Evan Baker <[email protected]> commit 0b743ac Author: Matthew Long <[email protected]> Date: Mon Aug 28 16:11:12 2023 -0700 fix: don't delete an existing conflist (#2115) commit 1116f7e Author: aggarwal0009 <[email protected]> Date: Mon Aug 28 12:46:55 2023 -0700 Update MTPNC scope (#2186) * update mtpnc scope * update mtpnc scope to Namespaced commit 9ace4b7 Author: Evan Baker <[email protected]> Date: Mon Aug 28 14:45:36 2023 -0500 change total IPs and add secondary IP metric (#2172) * change total IPs and add secondary IP metric Updates the Total IPs metrics to include the NC Primary IP in the total. Adds a Secondary IPs metric which holds the value that the Total IPs previously held: NC Secondary IPs known to CNS which could be used by Pods. Signed-off-by: GitHub <[email protected]> * update help wording on IPAM metrics Signed-off-by: Evan Baker <[email protected]> * reword PrimaryIP metric help Signed-off-by: Evan Baker <[email protected]> --------- Signed-off-by: GitHub <[email protected]> Signed-off-by: Evan Baker <[email protected]> commit 1adf24a Author: Evan Baker <[email protected]> Date: Mon Aug 28 13:50:01 2023 -0500 Add MTPNC reconciler for cache population in Swift V2 (#2164) add mtpnc watcher Signed-off-by: Evan Baker <[email protected]> commit c93109a Author: John Payne <[email protected]> Date: Fri Aug 25 17:47:05 2023 -0700 ci: Bypass Dualstack E2E on cluster creation failure (#2185) * test: continue on fail * test: continueOnError * test: succeed cluster creation * revert commented job commit d0559f8 Author: John Payne <[email protected]> Date: Fri Aug 25 10:00:19 2023 -0700 ci: Improve CreateValidator and restartNetworkCmd reliability (#2181) * ci: Improve CreateValidator and restartNetworkCmd reliability * lint fix * Addressing comments commit 2a468c4 Author: rjdenney <[email protected]> Date: Fri Aug 25 12:29:32 2023 -0400 fix: updating CNI to v1.5.10 for dropgz (#2168) change CNI version to v1.5.10 for new dropgz commit 216cc23 Author: aggarwal0009 <[email protected]> Date: Fri Aug 25 09:08:14 2023 -0700 update swiftv2 crds scope (#2176) commit e789e04 Author: John Payne <[email protected]> Date: Thu Aug 24 10:55:23 2023 -0700 ci: update CNS|dropgz version (#2169) * ci: update CNS|dropgz version * Revert cns daemonset to acnpublic commit b1c2508 Author: Paul Yu <[email protected]> Date: Wed Aug 23 21:49:27 2023 -0400 ci: add node and CNS restart test case in dualstack (#2135) * add noderestart test cases commit e5d97bb Author: Diego Becerra <[email protected]> Date: Wed Aug 23 15:53:46 2023 -0700 Add NotFound error handling to NMAgent client (#2163) This PR makes some minor changes to the NMAgent client package as a part of a larger work item. This commit adds a NotFound method to the error returned by the NMAgent client. It also adds special handling to treat a 400 BadRequest as a NotFound when returned by the DeleteNetwork API call, since this case should be interpreted as a NotFound by the caller. commit 35c6833 Author: aggarwal0009 <[email protected]> Date: Wed Aug 23 14:22:58 2023 -0700 Ankaggar/consolidate crds (#2171) * organise related swiftv2 CRDs in same GV package * fix linter error * update crdgen * consolidate external swiftv2 crds * fix manifests * more changes * update crdgen * resolve merge conflicts * fix crdgen check failure * further flatten swiftv2 crds * fix UT failure commit e767b15 Author: Quang Nguyen <[email protected]> Date: Wed Aug 23 16:04:05 2023 -0400 feat: CNS writes SWIFT conflist (#2110) * cns writes swift conflist * gofumpt ed * var naming * add swift scenario to switch commit 06e3877 Author: Evan Baker <[email protected]> Date: Wed Aug 23 10:58:52 2023 -0500 Upgrade controller-runtime (#2162) * upgrade controller-runtime Signed-off-by: Evan Baker <[email protected]> * update build tools and regen crds Signed-off-by: Evan Baker <[email protected]> * fix import conflict Signed-off-by: Evan Baker <[email protected]> --------- Signed-off-by: Evan Baker <[email protected]> commit b5440dd Author: John Payne <[email protected]> Date: Tue Aug 22 16:46:49 2023 -0700 fix: Change cluster name for cilium nightly pipeline (#2167) commit ef30552 Author: John Payne <[email protected]> Date: Mon Aug 21 16:55:35 2023 -0700 fix: Change argument in load-test template (#2136) * fix: Change argument in load-test template * Addressing Comments commit ec9d41e Author: Paul Yu <[email protected]> Date: Mon Aug 21 15:33:10 2023 -0400 hotfix for adding duplicated routes (#2161) commit 1b2a04a Author: Evan Baker <[email protected]> Date: Mon Aug 21 11:56:43 2023 -0500 feat: stub CNS Pod watcher (#2112) * feat: cns watches pods Signed-off-by: Evan Baker <[email protected]> * indirect pod reconcile for more dynamic behavior Signed-off-by: Evan Baker <[email protected]> --------- Signed-off-by: Evan Baker <[email protected]> commit d17079b Author: Evan Baker <[email protected]> Date: Fri Aug 18 16:49:32 2023 -0500 docs: proposal for async pod delete handling (#2138) * docs: proposal for async pod delete handling Signed-off-by: Evan Baker <[email protected]> * Update docs/feature/async-delete/readme.md Co-authored-by: Tyler Lloyd <[email protected]> Signed-off-by: Evan Baker <[email protected]> * reword based on PR feedback Signed-off-by: Evan Baker <[email protected]> --------- Signed-off-by: Evan Baker <[email protected]> Co-authored-by: Tyler Lloyd <[email protected]> commit 626e16c Author: Paul Yu <[email protected]> Date: Fri Aug 18 15:24:55 2023 -0400 ci: add node and CNS restart test case for v4overlay cluster (#2152) * add node restart test case for v4overlay cluster commit 0062d04 Author: tamilmani1989 <[email protected]> Date: Fri Aug 18 09:14:20 2023 -0700 fix: Remove NLM_F_EXCL flag from Netlink Delete call (#2150) * fix: Remove unix.NLM_F_EXCL from Netlink Delete Route api call unix.NLM_F_EXCL is not expected to set in netlink delete route calls. It's no-op in older kernel and didnt return error. From kernel 5.19+, new flag NLM_F_BULK was defined with same value and serves a purpose in delete route call. This changes breaks azure cni and netlink calls fails in 5.19 kernel and onwards. The fix is to remove setting unix.NLM_F_EXCL in netlink delete route request. * fix: Remove unix.NLM_F_EXCL from Netlink Delete Route api call unix.NLM_F_EXCL is not expected to set in netlink delete route calls. It's no-op in older kernel and didnt return error. From kernel 5.19+, new flag NLM_F_BULK was defined with same value and serves a purpose in delete route call. This changes breaks azure cni and netlink calls fails in 5.19 kernel and onwards. The fix is to remove setting unix.NLM_F_EXCL in netlink delete route request. * Add unit tests for netlink add/delete address and add/delete routes commit ba689eb Author: Evan Baker <[email protected]> Date: Thu Aug 17 14:09:56 2023 -0500 add swift v2 config based on node label (#2144) * add swift v2 config based on node label Signed-off-by: Evan Baker <[email protected]> * add tentative swiftv2 label Signed-off-by: Evan Baker <[email protected]> --------- Signed-off-by: Evan Baker <[email protected]> commit 50422dc Author: aggarwal0009 <[email protected]> Date: Thu Aug 17 11:54:18 2023 -0700 [Multitenancy]: Add PodNetwork field to MTPNC spec (#2151) * Add PN to MTPNC spec * add annotation * remove omitempty tag commit 4aecfd6 Author: Vipul Singh <[email protected]> Date: Thu Aug 17 10:31:43 2023 -0700 ci: add packages for submodule trigger (#2154) commit 9a3c50c Author: Behzad Mirkhanzadeh <[email protected]> Date: Wed Aug 16 17:16:55 2023 -0700 perf: 🚀 Increase CNI Lock Timeout to 30 seconds for Linux AKS. (#2101) * Increase Linux Lock Timout to 30 seconds. * addressing the comment by adding a new lock constant for Linux. # Conflicts: # .pipelines/cni/cilium/nightly-release-test.yml

jshr-w added cni Related to CNI. ci Infra or tooling. labels Sep 20, 2023

rbtr requested changes Sep 20, 2023

View reviewed changes

hack/aks/Makefile Outdated Show resolved Hide resolved

jshr-w closed this Sep 25, 2023

rbtr mentioned this pull request Sep 25, 2023

ci: add CNIv2 testing for different proc/OS #2248

Closed

4 tasks

jshr-w changed the title ~~ci: proc/os load tests~~ ci: add CNIv2 testing for different proc/OS Sep 25, 2023

jshr-w reopened this Sep 25, 2023

jshr-w requested review from behzad-mir and jpayne3506 September 25, 2023 21:56

jshr-w self-assigned this Sep 25, 2023

jshr-w marked this pull request as ready for review September 25, 2023 21:58

jshr-w requested a review from a team as a code owner September 25, 2023 21:58

rbtr previously approved these changes Sep 25, 2023

View reviewed changes

vipul-21 reviewed Sep 25, 2023

View reviewed changes

.pipelines/cni/pipeline.yaml Outdated Show resolved Hide resolved

hack/aks/Makefile Outdated Show resolved Hide resolved

jshr-w force-pushed the shjayaraman/osproc branch from 7e2b41a to 3581ac2 Compare September 26, 2023 00:19

jshr-w dismissed rbtr’s stale review via ae84466 September 26, 2023 01:25

vipul-21 previously approved these changes Sep 26, 2023

View reviewed changes

jshr-w dismissed vipul-21’s stale review via d9f2520 September 26, 2023 17:40

jshr-w added 10 commits September 26, 2023 13:36

ci: proc/os load tests

5de45f9

ci: update deprecated param

5efdf94

ci: update naming

d6e6e42

ci: add other proc/os load tests

da1ace4

ci: add other proc/os load tests

0c46340

remove ubuntu18 test

c5fa3e6

remove redundant test

9f47837

fix template call

2496346

add infiniband test, arm binary

7ffec2e

fix rdma node name, build

5ea29c4

jshr-w added 6 commits September 26, 2023 13:36

update RDMA node count for quota

f077ee2

force pipeline permissions

1847dd6

undo force permissions

8aafe04

remove hardcoding

83d0cde

add os as parameter for cniv1

b13d623

set default osSKU

83b8304

jshr-w force-pushed the shjayaraman/osproc branch from ea06ca2 to 83b8304 Compare September 26, 2023 20:36

jshr-w added 2 commits September 28, 2023 09:24

Merge branch 'master' into shjayaraman/osproc

ec5f2df

Signed-off-by: jshr-w <[email protected]>

Merge branch 'master' into shjayaraman/osproc

0069140

jshr-w requested a review from vipul-21 September 28, 2023 21:57

vipul-21 reviewed Sep 28, 2023

View reviewed changes

jpayne3506 requested changes Sep 28, 2023

View reviewed changes

.pipelines/cni/pipeline.yaml Outdated Show resolved Hide resolved

.pipelines/cni/load-test-templates/create-cluster-template.yaml Outdated Show resolved Hide resolved

jshr-w added 3 commits September 28, 2023 15:49

merge variable name change

7206880

set default OSSKU

3d47aab

set default OSSKU

f15be2d

jpayne3506 approved these changes Sep 28, 2023

View reviewed changes

vipul-21 approved these changes Sep 28, 2023

View reviewed changes

jshr-w merged commit 4423a94 into Azure:master Sep 29, 2023
67 of 70 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ci: add CNIv2 testing for different proc/OS #2230

ci: add CNIv2 testing for different proc/OS #2230

jshr-w commented Sep 20, 2023 •

edited

Loading

vipul-21 commented Sep 20, 2023

azure-pipelines bot commented Sep 20, 2023

vipul-21 commented Sep 27, 2023

azure-pipelines bot commented Sep 27, 2023

vipul-21 Sep 28, 2023

vipul-21 Sep 28, 2023

vipul-21 Sep 28, 2023

jpayne3506 left a comment

ci: add CNIv2 testing for different proc/OS #2230

ci: add CNIv2 testing for different proc/OS #2230

Conversation

jshr-w commented Sep 20, 2023 • edited Loading

vipul-21 commented Sep 20, 2023

azure-pipelines bot commented Sep 20, 2023

vipul-21 commented Sep 27, 2023

azure-pipelines bot commented Sep 27, 2023

vipul-21 Sep 28, 2023

Choose a reason for hiding this comment

vipul-21 Sep 28, 2023

Choose a reason for hiding this comment

vipul-21 Sep 28, 2023

Choose a reason for hiding this comment

jpayne3506 left a comment

Choose a reason for hiding this comment

jshr-w commented Sep 20, 2023 •

edited

Loading