Last 3 autobuilds have failed...

classic Classic list List threaded Threaded
11 messages Options
Reply | Threaded
Open this post in threaded view
|

Last 3 autobuilds have failed...

Samba - samba-technical mailing list
With:

[2199(16318)/2230 at 3h40m54s] samba.blackbox.wbinfo(ad_member:local)(ad_member:local)
UNEXPECTED(failure): samba.blackbox.wbinfo(ad_member:local).confirm output of wbinfo -i against ad_member(ad_member:local)
REASON: Exception: Exception: expected 'ADDOMAIN/administrator:*:100000:100004:Administrator:/home/ADDOMAIN/Domain Users/administr
+ator:/bin/false' or 'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/Domain Users/administrator:/bin/false' got 'ADDOMAIN/
+administrator:*:100000:100004::/home/ADDOMAIN/domain users/administrator:/bin/false'

This isn't related to anything I'm pushing (I'm
pushing Justin's 'Add new file information classes'
and the ceph statvfs changes.

Is autobuild working for everyone else right now ?

Jeremy.

Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
On 2018-01-11 at 08:38 -0800 Jeremy Allison via samba-technical sent off:
> [2199(16318)/2230 at 3h40m54s] samba.blackbox.wbinfo(ad_member:local)(ad_member:local)
> UNEXPECTED(failure): samba.blackbox.wbinfo(ad_member:local).confirm output of wbinfo -i against ad_member(ad_member:local)
> REASON: Exception: Exception: expected 'ADDOMAIN/administrator:*:100000:100004:Administrator:/home/ADDOMAIN/Domain Users/administr
> +ator:/bin/false' or 'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/Domain Users/administrator:/bin/false' got 'ADDOMAIN/
> +administrator:*:100000:100004::/home/ADDOMAIN/domain users/administrator:/bin/false'

I also got that error a couple of times and it looks a bit frightening. If the
upper/lower case writing of of nss_winbind is eratic, this is calling for a lot
of trouble. Does anybody know what the error might be here? It wouldn't be good
if we ignore this error and it ends up a 4.8 regression bug finally.

        Björn

Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
On Friday, 12 January 2018 09:04:33 CET Björn JACKE via samba-technical wrote:

> On 2018-01-11 at 08:38 -0800 Jeremy Allison via samba-technical sent off:
> > [2199(16318)/2230 at 3h40m54s]
> > samba.blackbox.wbinfo(ad_member:local)(ad_member:local)
> > UNEXPECTED(failure): samba.blackbox.wbinfo(ad_member:local).confirm
> > output of wbinfo -i against ad_member(ad_member:local) REASON: Exception:
> > Exception: expected
> > 'ADDOMAIN/administrator:*:100000:100004:Administrator:/home/ADDOMAIN/Doma
> > in Users/administr +ator:/bin/false' or
> > 'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/Domain
> > Users/administrator:/bin/false' got 'ADDOMAIN/
> > +administrator:*:100000:100004::/home/ADDOMAIN/domain
> > users/administrator:/bin/false'
> I also got that error a couple of times and it looks a bit frightening. If
> the upper/lower case writing of of nss_winbind is eratic, this is calling
> for a lot of trouble. Does anybody know what the error might be here? It
> wouldn't be good if we ignore this error and it ends up a 4.8 regression
> bug finally.

Uppercase lowercase?

There is "Administrator" missing after the forth colon! Could be related to
the last substitute changes!


        Andreas

--
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             [hidden email]
www.samba.org



Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
On 2018-01-12 at 09:40 +0100 Andreas Schneider sent off:
> Uppercase lowercase?
>
> There is "Administrator" missing after the forth colon! Could be related to
> the last substitute changes!

expected:
'ADDOMAIN/administrator:*:100000:100004:Administrator:/home/ADDOMAIN/Domain Users/administrator:/bin/false'
or
'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/Domain Users/administrator:/bin/false'

got:
'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/domain users/administrator:/bin/false'

the one is missing Administrator, the other is different in case. A bit weird,
that the test has two output options that it expects here.

Björn

Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
In reply to this post by Samba - samba-technical mailing list
On pe, 12 tammi 2018, Andreas Schneider via samba-technical wrote:

> On Friday, 12 January 2018 09:04:33 CET Björn JACKE via samba-technical wrote:
> > On 2018-01-11 at 08:38 -0800 Jeremy Allison via samba-technical sent off:
> > > [2199(16318)/2230 at 3h40m54s]
> > > samba.blackbox.wbinfo(ad_member:local)(ad_member:local)
> > > UNEXPECTED(failure): samba.blackbox.wbinfo(ad_member:local).confirm
> > > output of wbinfo -i against ad_member(ad_member:local) REASON: Exception:
> > > Exception: expected
> > > 'ADDOMAIN/administrator:*:100000:100004:Administrator:/home/ADDOMAIN/Doma
> > > in Users/administr +ator:/bin/false' or
> > > 'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/Domain
> > > Users/administrator:/bin/false' got 'ADDOMAIN/
> > > +administrator:*:100000:100004::/home/ADDOMAIN/domain
> > > users/administrator:/bin/false'
> > I also got that error a couple of times and it looks a bit frightening. If
> > the upper/lower case writing of of nss_winbind is eratic, this is calling
> > for a lot of trouble. Does anybody know what the error might be here? It
> > wouldn't be good if we ignore this error and it ends up a 4.8 regression
> > bug finally.
>
> Uppercase lowercase?
>
> There is "Administrator" missing after the forth colon! Could be related to
> the last substitute changes!
No, GECOS field can be empty -- the test actually says about it as it
expects either existing GECOS or empty one. However, path is including
lowercased 'domain users' group name but the test expects it intact.
--
/ Alexander Bokovoy

Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
On pe, 12 tammi 2018, Alexander Bokovoy via samba-technical wrote:

> On pe, 12 tammi 2018, Andreas Schneider via samba-technical wrote:
> > On Friday, 12 January 2018 09:04:33 CET Björn JACKE via samba-technical wrote:
> > > On 2018-01-11 at 08:38 -0800 Jeremy Allison via samba-technical sent off:
> > > > [2199(16318)/2230 at 3h40m54s]
> > > > samba.blackbox.wbinfo(ad_member:local)(ad_member:local)
> > > > UNEXPECTED(failure): samba.blackbox.wbinfo(ad_member:local).confirm
> > > > output of wbinfo -i against ad_member(ad_member:local) REASON: Exception:
> > > > Exception: expected
> > > > 'ADDOMAIN/administrator:*:100000:100004:Administrator:/home/ADDOMAIN/Doma
> > > > in Users/administr +ator:/bin/false' or
> > > > 'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/Domain
> > > > Users/administrator:/bin/false' got 'ADDOMAIN/
> > > > +administrator:*:100000:100004::/home/ADDOMAIN/domain
> > > > users/administrator:/bin/false'
> > > I also got that error a couple of times and it looks a bit frightening. If
> > > the upper/lower case writing of of nss_winbind is eratic, this is calling
> > > for a lot of trouble. Does anybody know what the error might be here? It
> > > wouldn't be good if we ignore this error and it ends up a 4.8 regression
> > > bug finally.
> >
> > Uppercase lowercase?
> >
> > There is "Administrator" missing after the forth colon! Could be related to
> > the last substitute changes!
> No, GECOS field can be empty -- the test actually says about it as it
> expects either existing GECOS or empty one. However, path is including
> lowercased 'domain users' group name but the test expects it intact.
Attached patch normalizes both tested and expected lines to the same
case before comparison. It still reports the original tested and
expected lines in case of an error.

--
/ Alexander Bokovoy

samba-test_wbinfo.patch (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
On Friday, 12 January 2018 11:02:50 CET Alexander Bokovoy wrote:
> On pe, 12 tammi 2018, Alexander Bokovoy via samba-technical wrote:
> > On pe, 12 tammi 2018, Andreas Schneider via samba-technical wrote:
> > > On Friday, 12 January 2018 09:04:33 CET Björn JACKE via samba-technical
wrote:
> > > > On 2018-01-11 at 08:38 -0800 Jeremy Allison via samba-technical sent
off:

> > > > > [2199(16318)/2230 at 3h40m54s]
> > > > > samba.blackbox.wbinfo(ad_member:local)(ad_member:local)
> > > > > UNEXPECTED(failure): samba.blackbox.wbinfo(ad_member:local).confirm
> > > > > output of wbinfo -i against ad_member(ad_member:local) REASON:
> > > > > Exception:
> > > > > Exception: expected
> > > > > 'ADDOMAIN/administrator:*:100000:100004:Administrator:/home/ADDOMAIN
> > > > > /Doma
> > > > > in Users/administr +ator:/bin/false' or
> > > > > 'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/Domain
> > > > > Users/administrator:/bin/false' got 'ADDOMAIN/
> > > > > +administrator:*:100000:100004::/home/ADDOMAIN/domain
> > > > > users/administrator:/bin/false'
> > > >
> > > > I also got that error a couple of times and it looks a bit
> > > > frightening. If
> > > > the upper/lower case writing of of nss_winbind is eratic, this is
> > > > calling
> > > > for a lot of trouble. Does anybody know what the error might be here?
> > > > It
> > > > wouldn't be good if we ignore this error and it ends up a 4.8
> > > > regression
> > > > bug finally.
> > >
> > > Uppercase lowercase?
> > >
> > > There is "Administrator" missing after the forth colon! Could be related
> > > to
> > > the last substitute changes!
> >
> > No, GECOS field can be empty -- the test actually says about it as it
> > expects either existing GECOS or empty one. However, path is including
> > lowercased 'domain users' group name but the test expects it intact.
>
> Attached patch normalizes both tested and expected lines to the same
> case before comparison. It still reports the original tested and
> expected lines in case of an error.

but if the test is flapping it means the case changes in some runs. So either
it picks up a wrong config file or something else is wrong, isn't it?


        Andreas

--
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             [hidden email]
www.samba.org



Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
On pe, 12 tammi 2018, Andreas Schneider wrote:

> On Friday, 12 January 2018 11:02:50 CET Alexander Bokovoy wrote:
> > On pe, 12 tammi 2018, Alexander Bokovoy via samba-technical wrote:
> > > On pe, 12 tammi 2018, Andreas Schneider via samba-technical wrote:
> > > > On Friday, 12 January 2018 09:04:33 CET Björn JACKE via samba-technical
> wrote:
> > > > > On 2018-01-11 at 08:38 -0800 Jeremy Allison via samba-technical sent
> off:
> > > > > > [2199(16318)/2230 at 3h40m54s]
> > > > > > samba.blackbox.wbinfo(ad_member:local)(ad_member:local)
> > > > > > UNEXPECTED(failure): samba.blackbox.wbinfo(ad_member:local).confirm
> > > > > > output of wbinfo -i against ad_member(ad_member:local) REASON:
> > > > > > Exception:
> > > > > > Exception: expected
> > > > > > 'ADDOMAIN/administrator:*:100000:100004:Administrator:/home/ADDOMAIN
> > > > > > /Doma
> > > > > > in Users/administr +ator:/bin/false' or
> > > > > > 'ADDOMAIN/administrator:*:100000:100004::/home/ADDOMAIN/Domain
> > > > > > Users/administrator:/bin/false' got 'ADDOMAIN/
> > > > > > +administrator:*:100000:100004::/home/ADDOMAIN/domain
> > > > > > users/administrator:/bin/false'
> > > > >
> > > > > I also got that error a couple of times and it looks a bit
> > > > > frightening. If
> > > > > the upper/lower case writing of of nss_winbind is eratic, this is
> > > > > calling
> > > > > for a lot of trouble. Does anybody know what the error might be here?
> > > > > It
> > > > > wouldn't be good if we ignore this error and it ends up a 4.8
> > > > > regression
> > > > > bug finally.
> > > >
> > > > Uppercase lowercase?
> > > >
> > > > There is "Administrator" missing after the forth colon! Could be related
> > > > to
> > > > the last substitute changes!
> > >
> > > No, GECOS field can be empty -- the test actually says about it as it
> > > expects either existing GECOS or empty one. However, path is including
> > > lowercased 'domain users' group name but the test expects it intact.
> >
> > Attached patch normalizes both tested and expected lines to the same
> > case before comparison. It still reports the original tested and
> > expected lines in case of an error.
>
> but if the test is flapping it means the case changes in some runs. So either
> it picks up a wrong config file or something else is wrong, isn't it?
'template homdir' is configured in setup_admember() of Samba3.pm:
       template homedir = /home/%D/%G/%U

%G substitution uses gidtoname() which is using getgrgid() call and
returns gr_name out of it. However, we do map 'domain users' SID to
domusers group, so technically this should be
/home/$DOMAIN/domusers/administrator if things are working properly,
isn't it?

--
/ Alexander Bokovoy

Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
On 2018-01-12 at 13:20 +0200 Alexander Bokovoy sent off:
> 'template homdir' is configured in setup_admember() of Samba3.pm:
>        template homedir = /home/%D/%G/%U
>
> %G substitution uses gidtoname() which is using getgrgid() call and
> returns gr_name out of it. However, we do map 'domain users' SID to
> domusers group, so technically this should be
> /home/$DOMAIN/domusers/administrator if things are working properly,
> isn't it?

well, this reminds me of the idmapping mess why I wrote this patch:

http://samba.2283325.n4.nabble.com/PATCH-s4-provision-don-t-mix-local-uid-numbers-with-domain-mappings-td4727638.html

Unfortunately all the people I know who support the fix didn't speak up in that
thread.

Björn
--
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:[hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
On Fri, Jan 12, 2018 at 01:15:31PM +0100, Björn JACKE via samba-technical wrote:

> On 2018-01-12 at 13:20 +0200 Alexander Bokovoy sent off:
> > 'template homdir' is configured in setup_admember() of Samba3.pm:
> >        template homedir = /home/%D/%G/%U
> >
> > %G substitution uses gidtoname() which is using getgrgid() call and
> > returns gr_name out of it. However, we do map 'domain users' SID to
> > domusers group, so technically this should be
> > /home/$DOMAIN/domusers/administrator if things are working properly,
> > isn't it?
>
> well, this reminds me of the idmapping mess why I wrote this patch:
>
> http://samba.2283325.n4.nabble.com/PATCH-s4-provision-don-t-mix-local-uid-numbers-with-domain-mappings-td4727638.html

Did this ever make it to

https://lists.samba.org/archive/samba-technical/

which I would consider the archive that the Samba Team hosts, not some
cloud-based-service that lives by sending you ads?

If not, I would consider this non-existent.

Volker

--
Besuchen Sie die verinice.XP 2018 in Berlin,
Anwenderkonferenz für Informationssicherheit
vom 21.-23.03.2018 im Sofitel Kurfürstendamm
Info & Anmeldung hier: http://veriniceXP.org

SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:[hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: Last 3 autobuilds have failed...

Samba - samba-technical mailing list
In reply to this post by Samba - samba-technical mailing list
On Fri, 12 Jan 2018 13:15:31 +0100
Björn JACKE via samba-technical <[hidden email]> wrote:

> On 2018-01-12 at 13:20 +0200 Alexander Bokovoy sent off:
> > 'template homdir' is configured in setup_admember() of Samba3.pm:
> >        template homedir = /home/%D/%G/%U
> >
> > %G substitution uses gidtoname() which is using getgrgid() call and
> > returns gr_name out of it. However, we do map 'domain users' SID to
> > domusers group, so technically this should be
> > /home/$DOMAIN/domusers/administrator if things are working properly,
> > isn't it?
>
> well, this reminds me of the idmapping mess why I wrote this patch:
>
> http://samba.2283325.n4.nabble.com/PATCH-s4-provision-don-t-mix-local-uid-numbers-with-domain-mappings-td4727638.html
>
> Unfortunately all the people I know who support the fix didn't speak
> up in that thread.
>
> Björn

I take it this test is an old one and Administrator and Domain Users
have been mapped in idmap.ldb for the last 5 years at least, so
wouldn't Administrator having the UID '0' and Domain Users having the
GID '100' have been a problem for all that time ?

I cannot see that mapping administrator to root on a DC is a problem,
in fact it would probably be a bigger problem if it wasn't.

Rowland