Skip to content

Commit

Permalink
add provisioner taints to nodes (#699)
Browse files Browse the repository at this point in the history
* add provisioner taints to nodes

* cloudprovider mock should not add taints
  • Loading branch information
bwagner5 committed Sep 23, 2021
1 parent 1f7933b commit bcc0bbe
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 0 additions & 1 deletion pkg/cloudprovider/fake/cloudprovider.go
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ func (c *CloudProvider) Create(ctx context.Context, constraints *v1alpha4.Constr
},
Spec: v1.NodeSpec{
ProviderID: fmt.Sprintf("fake:///%s/%s", name, zone),
Taints: constraints.Taints,
},
Status: v1.NodeStatus{
NodeInfo: v1.NodeSystemInfo{
Expand Down
1 change: 1 addition & 0 deletions pkg/controllers/allocation/controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ func (c *Controller) Reconcile(ctx context.Context, req reconcile.Request) (reco
map[string]string{v1alpha4.ProvisionerNameLabelKey: provisioner.Name},
packing.Constraints.Labels,
)
node.Spec.Taints = append(node.Spec.Taints, packing.Constraints.Taints...)
return c.Binder.Bind(ctx, node, packing.Pods)
})
})
Expand Down

0 comments on commit bcc0bbe

Please sign in to comment.