[PATCH] pyauth: Remove the imessaging_ctx parameter

Noel Power nopower at suse.com
Wed Oct 3 19:22:15 UTC 2018


On 03/10/2018 10:37, Noel Power via samba-technical wrote:
> RB+
>
> On 02/10/2018 21:01, Gary Lockyer via samba-technical wrote:
>> Remove the imessaging_ctx parameter from the pyauth constructor. The
>> pyauth code expects this to be a imessaging_context struct.  The
>> imessaging code returns a wrapper structure that contains the
>> imessaging_context and the talloc context it's allocated on.  As no code
>> uses this, and to save others a wasted afternoon I've removed the
>> parameter.  Could I get a review? CI run results
>> https://gitlab.com/catalyst-samba/samba/pipelines/31618803
>>
>> Thanks
>> Gary
>>
>
>
Unfortunately can't get this past autobuild, tried this patch with & 
without Douglas's 'popt_common_creds: actually use the 
ignore_missing_conf flag' Autobuild failed numberous time for me with.  
The failure seems unrelated to the change, I've run the same on CI 
https://gitlab.com/samba-team/devel/samba/pipelines/31918098 and the 
'build_samba_nonenv' tests pass fine

UNEXPECTED(failure): samba.unittests.kerberos.test_krb5_remove_obsolete_keytab_entries_one(none)
REASON: Exception: Exception:"user0 at samba.example.comb"  !="user0 at samba.example.com"
../source4/auth/tests/kerberos.c:64: error: Failure!

FAILED (1 failures, 0 errors and 0 unexpected successes in 0 testsuites)




More information about the samba-technical mailing list