Issue Description

dsconf repl-agmt create refuses to create a GSSAPI replication agreement because it incorrectly requires --bind-passwd to be set.

Package Version and Platform

Repros on current HEAD: https://pagure.io/389-ds-base/blob/master/f/src/lib389/lib389/cli_conf/replication.py#_666

Steps to reproduce

1. dsconf instance repl-agmt create --suffix dc=foo --host bar.com --port 389 --conn-protocol LDAP --bind-dn uid=ldap/bar.com,dc=foo --bind-method SASL/GSSAPI "GSSAPI Replication to bar.com"

Actual results

"You need to set the bind dn (--bind-dn) and the password (--bind-passwd) for bind method (SASL/GSSAPI)"

Expected results

Replication agreement is successfully created.

The bug and fix are obvious. https://pagure.io/389-ds-base/blob/master/f/src/lib389/lib389/cli_conf/replication.py#_666 has incorrect Python syntax:

    if (bind_method == 'simple' or 'sslclientauth') and (args.bind_dn is None or args.bind_passwd is None):

bind_method == 'simple' or 'sslclientauth' evaluates as (bind_method == 'simple') or 'sslclientauth' in Python, which is always True.

The condition should instead be

    if (bind_method in ('simple', 'sslclientauth')) and (args.bind_dn is None or args.bind_passwd is None):

I confirmed that the command functions properly with that change.