Bug 93400 - Use keytab file specified at the command line during updates
Summary: Use keytab file specified at the command line during updates
Status: RESOLVED FIXED
Alias: None
Product: realmd
Classification: Unclassified
Component: adcli (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Stef Walter
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-16 10:03 UTC by Sumit Bose
Modified: 2015-12-17 08:20 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:

Attachments
update: set login keytab based on command line options (799 bytes, patch)
2015-12-16 10:03 UTC, Sumit Bose 		Details | Splinter Review
View All

Description Sumit Bose 2015-12-16 10:03:28 UTC 
In adcli_tool_computer_update() the keytab file name ktname is currently un-set and as a result always the default keytab is used and the value from the command line is not respected.
Comment 1 Sumit Bose 2015-12-16 10:03:58 UTC 
Created attachment 120541 [details] [review]
update: set login keytab based on command line options
Comment 2 Stef Walter 2015-12-17 08:18:07 UTC 
So this is a regression likely introduced by my changes to:

commit ec1fd29641e9f4638d00e3ccaaa20927d126aa9d
Author: Sumit Bose <sbose@redhat.com>
Date:   Thu Dec 3 18:15:42 2015 +0100

    Allow additional ccache for updates
    
    https://bugs.freedesktop.org/show_bug.cgi?id=92908
    
    Signed-off-by: Stef Walter <stefw@redhat.com>
     * Removed redundant changes for default keytab


Good catch. Will amend the commit message to indicate that.
Comment 3 Stef Walter 2015-12-17 08:20:41 UTC 
Attachment 120541 [details] pushed as 253f37d - update: set login keytab based on command line options

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.