Skip to content

Commit

Permalink
fix: issue 579 and error handling when aci provider fail to be initia…
Browse files Browse the repository at this point in the history
…lized (#582)
  • Loading branch information
smritidahal653 committed Nov 28, 2023
1 parent e08ea09 commit bb38fd5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 1 addition & 3 deletions charts/virtual-kubelet/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,13 @@ spec:
replicas: 1
selector:
matchLabels:
app: {{ template "vk.fullname" . }}
app: {{ template "vk.name" . }}
template:
metadata:
{{ include "vk.labels" . | indent 6 }}
component: kubelet
annotations:
checksum/secret: {{ include (print $.Template.BasePath "/secrets.yaml") . | sha256sum }}
labels:
app: {{ template "vk.fullname" . }}
spec:
initContainers:
- name: init-validation
Expand Down
3 changes: 3 additions & 0 deletions cmd/virtual-kubelet/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -222,6 +222,9 @@ func main() {
p, err := azproviderv2.NewACIProvider(ctx, cfgPath, azConfig, azACIAPIs, cfg,
nodeName, operatingSystem, os.Getenv("VKUBELET_POD_IP"),
int32(listenPort), clusterDomain, k8sClient)
if err != nil {
return nil, nil, err
}
p.ConfigureNode(ctx, cfg.Node)
return p, nil, err
},
Expand Down

0 comments on commit bb38fd5

Please sign in to comment.