azure-container-networking/crd/nodenetworkconfig/manifests
Ashish Nair a61940dbde
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 <noreply@github.com>

* 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 <noreply@github.com>
2023-09-22 02:50:29 +00:00
..
acn.azure.com_nodenetworkconfigs.yaml feat: Consume the NCStatus to be able to append subnet is full error to Pod IP reservation failures (#2202) 2023-09-22 02:50:29 +00:00
doc.go fix: correct nnc embed package naming and update apiext version to v1 (#982) 2021-08-30 12:48:10 -05:00