Merge pull request #122996 from Huang-Wei/cleanup-dra-postfilter
DRA: always returns Unschedulable in PostFilter
This commit is contained in:
		| @@ -789,7 +789,7 @@ func (pl *dynamicResources) PostFilter(ctx context.Context, cs *framework.CycleS | |||||||
| 			if _, err := pl.clientset.ResourceV1alpha2().ResourceClaims(claim.Namespace).UpdateStatus(ctx, claim, metav1.UpdateOptions{}); err != nil { | 			if _, err := pl.clientset.ResourceV1alpha2().ResourceClaims(claim.Namespace).UpdateStatus(ctx, claim, metav1.UpdateOptions{}); err != nil { | ||||||
| 				return nil, statusError(logger, err) | 				return nil, statusError(logger, err) | ||||||
| 			} | 			} | ||||||
| 			return nil, nil | 			return nil, framework.NewStatus(framework.Unschedulable, "deallocation of ResourceClaim completed") | ||||||
| 		} | 		} | ||||||
| 	} | 	} | ||||||
| 	return nil, framework.NewStatus(framework.Unschedulable, "still not schedulable") | 	return nil, framework.NewStatus(framework.Unschedulable, "still not schedulable") | ||||||
|   | |||||||
| @@ -473,6 +473,7 @@ func TestPlugin(t *testing.T) { | |||||||
| 								Obj() | 								Obj() | ||||||
| 						}, | 						}, | ||||||
| 					}, | 					}, | ||||||
|  | 					status: framework.NewStatus(framework.Unschedulable, `deallocation of ResourceClaim completed`), | ||||||
| 				}, | 				}, | ||||||
| 			}, | 			}, | ||||||
| 		}, | 		}, | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 Kubernetes Prow Robot
					Kubernetes Prow Robot