New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Creating new replication agreement crashes the server #2439
Comments
Comment from misko at 2017-09-13 12:15:53 I can provide Ansible playbooks for replicating the problem if necessary. |
Comment from mreynolds (@mreynolds389) at 2017-09-13 15:13:17 Metadata Update from @mreynolds389:
|
Comment from mreynolds (@mreynolds389) at 2017-09-13 15:13:26 The crashing stack
|
Comment from mreynolds (@mreynolds389) at 2017-09-13 15:13:27 Metadata Update from @mreynolds389:
|
Comment from mreynolds (@mreynolds389) at 2017-09-13 15:29:00 |
Comment from mreynolds (@mreynolds389) at 2017-09-13 15:29:01 Metadata Update from @mreynolds389:
|
Comment from tbordaz (@tbordaz) at 2017-09-13 17:29:18 The fix looks good. |
Comment from mreynolds (@mreynolds389) at 2017-09-14 02:34:53
I don't think it can be NULL when we call schedule_set(), but I put the check in anyway. Revised patch: |
Comment from tbordaz (@tbordaz) at 2017-09-14 08:21:59 Thanks Mark, the patch looks good to me. ACK |
Comment from tbordaz (@tbordaz) at 2017-09-14 08:22:00 Metadata Update from @tbordaz:
|
Comment from mreynolds (@mreynolds389) at 2017-09-14 14:33:02 |
Comment from mreynolds (@mreynolds389) at 2017-09-14 14:44:11 Metadata Update from @mreynolds389:
|
Comment from mreynolds (@mreynolds389) at 2017-09-14 17:50:42 Add CI test |
Comment from mreynolds (@mreynolds389) at 2017-09-14 17:50:43 Metadata Update from @mreynolds389:
|
Comment from spichugi (@droideck) at 2017-09-14 18:10:26 LGTM, Like this:
|
Comment from mreynolds (@mreynolds389) at 2017-09-14 20:16:09 Newly revised patch: |
Comment from spichugi (@droideck) at 2017-09-14 22:17:55 Thanks! You have my ack. One small issue though, the commit message body is a bit too long - 93 chars. |
Comment from spichugi (@droideck) at 2017-09-14 22:18:09 Metadata Update from @droideck:
|
Comment from mreynolds (@mreynolds389) at 2017-09-15 15:21:16 CI Test |
Comment from mreynolds (@mreynolds389) at 2017-09-15 15:21:17 Metadata Update from @mreynolds389:
|
Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/49380
Issue Description
When creating new replication agreement, with
nsds5ReplicaEnabled: True
, the server crashes.Package Version and Platform
Fedora 27
389-ds-console-1.2.16-3.fc27.noarch
389-ds-1.2.2-10.fc27.noarch
389-adminutil-1.1.23-4.fc27.x86_64
389-console-1.1.18-3.fc27.noarch
389-ds-base-1.3.7.4-1.fc28.x86_64
389-admin-console-1.1.12-3.fc27.noarch
389-ds-console-doc-1.2.16-3.fc27.noarch
389-dsgw-1.1.11-13.fc27.x86_64
389-ds-base-libs-1.3.7.4-1.fc28.x86_64
389-admin-1.1.46-1.fc27.3.x86_64
389-admin-console-doc-1.1.12-3.fc27.noarch
Steps to reproduce
f-ldap03.sandbox.in.pan-net.eu
, andf-ldap04.sandbox.in.pan-net.eu
cn=test,cn=config
), passwordtest
cn=replica
)f-ldap03.sandbox.in.pan-net.eu
) with this ldif:Please note
nsds5ReplicaEnabled
contains invalid valueTrue
instead ofon
Actual results
(actually, the daemon crashed)
On Debian, the log is even more funny:
Expected results
Not crashing and creating the agreement
The text was updated successfully, but these errors were encountered: