Skip to content

fix: Adding delete timestamp check #4078

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.

Sign up for GitHub

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

Merged
merged 9 commits into from
Oct 22, 2025
+77 −43
Merged

fix: Adding delete timestamp check #4078

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ab1235e
Adding delete timestamp check
shreyashastantram Oct 14, 2025
5ccb46e
Adding UT and refactoring getmtpnc
shreyashastantram Oct 15, 2025
6274f99
Update cns/middlewares/k8sSwiftV2_linux_test.go
shreyashastantram Oct 15, 2025
c2bbd84
Changing the MTPNC error message
shreyashastantram Oct 17, 2025
9b6e7fc
Removing unused fields
shreyashastantram Oct 20, 2025
1fd081f
Revert manifest changes for CRD files
shreyashastantram Oct 20, 2025
232bca3
Fixing error message and adding comments
shreyashastantram Oct 21, 2025
4e95dfd
Revert CRD API files to master state
shreyashastantram Oct 21, 2025
f0f45aa
fix: correcting comment assertion
shreyashastantram Oct 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions cns/middlewares/k8sSwiftV2.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ var (
errGetMTPNC = errors.New(NetworkNotReadyErrorMsg + " - failed to get MTPNC")
errInvalidSWIFTv2NICType = errors.New("invalid NIC type for SWIFT v2 scenario")
errInvalidMTPNCPrefixLength = errors.New("invalid prefix length for MTPNC primaryIP, must be 32")
errDeleteTimestampSet = errors.New("MTPNC deletion timestamp is set")
)

type K8sSWIFTv2Middleware struct {
Expand Down Expand Up @@ -193,6 +194,9 @@ func (k *K8sSWIFTv2Middleware) getMTPNC(ctx context.Context, podInfo cns.PodInfo
if err := k.Cli.Get(ctx, mtpncNamespacedName, &mtpnc); err != nil {
return v1alpha1.MultitenantPodNetworkConfig{}, types.UnexpectedError, errors.Wrap(err, errGetMTPNC.Error()).Error()
}
if mtpnc.DeletionTimestamp != nil {
return v1alpha1.MultitenantPodNetworkConfig{}, types.UnexpectedError, errDeleteTimestampSet.Error()
}
// Check if the MTPNC CRD is ready. If one of the fields is empty, return error
if !mtpnc.IsReady() {
return v1alpha1.MultitenantPodNetworkConfig{}, types.UnexpectedError, errMTPNCNotReady.Error()
Expand Down
Loading