Skip to content

Commit f9351ae

Browse files
refactor: update nodeserver.go
add more detailed error logging during kerberos auth phase
1 parent c72a7fe commit f9351ae

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

pkg/nfs/nodeserver.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -158,22 +158,21 @@ func (ns *NodeServer) NodePublishVolume(ctx context.Context, req *csi.NodePublis
158158
cmd := exec.CommandContext(ctx, "ktutil")
159159
cmd.Stdin = bytes.NewBufferString(fmt.Sprintf("addent -p %s -password -k 1 -f\n%s\nwkt /etc/krb5.keytab", krbPrinc, krbPwd))
160160
if err := cmd.Run(); err != nil {
161+
klog.Errorf("error running 'ktutil': %+v", err)
161162
return err
162163
}
163164
}
164165
// obtain kerberos TGT
165166
cmd := exec.CommandContext(ctx, "kinit", krbPrinc)
166167
cmd.Stdin = bytes.NewBufferString(krbPwd + "\n")
167168
if err := cmd.Run(); err != nil {
169+
klog.Errorf("error running 'kinit': %+v", err)
168170
return err
169171
}
170172
// initialize credentials from keytab
171-
cmd = exec.CommandContext(ctx, "kinit", "-k", krbPrinc)
172-
if err != nil {
173-
return err
174-
}
173+
cmd = exec.CommandContext(ctx, "kinit", "-k", krbPrinc)
175174
if err := cmd.Run(); err != nil {
176-
klog.Errorf("%+v", err)
175+
klog.Errorf("error running 'kinit -k': %+v", err)
177176
return err
178177
}
179178
}

0 commit comments

Comments
 (0)