freeipa.git/install/tools/ipa-adtrust-install, branch webui_isolate

Replace exit() calls with exceptions

2017-02-09T13:05:22+00:00

In order to enable correct handling of hard errors from within the
composite installer code, all calls to `sys.exit` have to be replaced
with raising ScriptError.

https://fedorahosted.org/freeipa/ticket/6629

Reviewed-By: Alexander Bokovoy 
Reviewed-By: Martin Basti

Remove unused variables in exception handling

2017-02-09T13:05:22+00:00

https://fedorahosted.org/freeipa/ticket/6629

Reviewed-By: Alexander Bokovoy 
Reviewed-By: Martin Basti

ipa-adtrust-install: format the code for PEP-8 compliance

2017-02-09T13:05:22+00:00

https://fedorahosted.org/freeipa/ticket/6629

Reviewed-By: Alexander Bokovoy 
Reviewed-By: Martin Basti

Set explicit confdir option for global contexts

2016-12-02T08:14:35+00:00

Some API contexts are used to modify global state (e.g. files in /etc
and /var). These contexts do not support confdir overrides. Initialize
the API with an explicit confdir argument to paths.ETC_IPA.

The special contexts are:

* backup
* cli_installer
* installer
* ipctl
* renew
* restore
* server
* updates

The patch also corrects the context of the ipa-httpd-kdcproxy script to
'server'.

https://fedorahosted.org/freeipa/ticket/6389

Signed-off-by: Christian Heimes 
Reviewed-By: Jan Cholasta

ipapython: move certmonger and sysrestore to ipalib.install

2016-11-29T13:50:51+00:00

The certmonger and sysrestore modules depend on ipaplatform.

Move them to ipalib.install as they are used only from installers.

https://fedorahosted.org/freeipa/ticket/6474

Reviewed-By: Stanislav Laznicka

Python3 pylint fixes

2016-11-25T15:18:22+00:00

Sprinkle 'pylint disable' comments over the code base to silence a bunch
of pylint warnings on Python 3. All silenced warnings are harmless and
not bugs.

https://fedorahosted.org/freeipa/ticket/4985

Signed-off-by: Christian Heimes 
Reviewed-By: Martin Basti

services: replace admin_conn with api.Backend.ldap2

2016-11-22T15:17:27+00:00

Since service.admin_conn is only an alias to api.Backend.ldap2,
replace it everywhere with the explicit api.Backend.ldap2 instead.

https://fedorahosted.org/freeipa/ticket/6461

Reviewed-By: Martin Basti

ipa-adtrust-install: ldap conn management

2016-11-07T10:34:03+00:00

Properly close ldap connection.

https://fedorahosted.org/freeipa/ticket/6461

Reviewed-By: Martin Basti 
Reviewed-By: Jan Cholasta

Pylint: remove unused variables from installers and scripts

2016-10-06T08:43:36+00:00

Reviewed-By: Stanislav Laznicka

Pylint: enable check for unused-variables

2016-09-27T11:35:58+00:00

Unused variables may:
* make code less readable
* create dead code
* potentialy hide issues/errors

Enabled check should prevent to leave unused variable in code

Check is locally disabled for modules that fix is not clear or easy or have too many occurences of
unused variables

Reviewed-By: Florence Blanc-Renaud 
Reviewed-By: Stanislav Laznicka