With patch this time:

On Fri, 2016-06-03 at 08:59 -0400, Ade Lee wrote:
> commit 9450b5f7695cc827cced6e86281694daa1e5c2c8
> Author: Ade Lee <[email protected]>
> Date:   Thu Jun 2 09:41:35 2016 -0400
> 
>     Add commands to db-server to help with DB related changes
>     
>     Added pki-server kra-db-vlv-add, kra-db-vlv-del, kra-db-vlv
> -reindex
>     Added pki-server db-schema-upgrade
>     
>     If the admin has the directory manager (or equivalent) simple
> creds,
>     then they can enter them as parameters and perform the
> operations.
>     
>     Otherwise, they can specify --generate-ldif to generate LDIF
> files
>     containing the changes that need to be implemented, and implement
>     them using GSSAPI or otherwise.
>     
>     Tickets 2320, 2319
> 
> Please review,
> Thanks,
> Ade
> 
> _______________________________________________
> Pki-devel mailing list
> [email protected]
> https://www.redhat.com/mailman/listinfo/pki-devel
From 9450b5f7695cc827cced6e86281694daa1e5c2c8 Mon Sep 17 00:00:00 2001
From: Ade Lee <[email protected]>
Date: Thu, 2 Jun 2016 09:41:35 -0400
Subject: [PATCH] Add commands to db-server to help with DB related changes

Added pki-server kra-db-vlv-add, kra-db-vlv-del, kra-db-vlv-reindex
Added pki-server db-schema-upgrade

If the admin has the directory manager (or equivalent) simple creds,
then they can enter them as parameters and perform the operations.

Otherwise, they can specify --generate-ldif to generate LDIF files
containing the changes that need to be implemented, and implement
them using GSSAPI or otherwise.

Tickets 2320, 2319
---
 base/server/python/pki/server/__init__.py |  11 +-
 base/server/python/pki/server/cli/db.py   |  89 +++++++-
 base/server/python/pki/server/cli/kra.py  | 352 ++++++++++++++++++++++++++++++
 3 files changed, 449 insertions(+), 3 deletions(-)

diff --git a/base/server/python/pki/server/__init__.py b/base/server/python/pki/server/__init__.py
index 47f6aa5b46af6cf38b231d94a88e6bb02c7d99b1..8347311cfe1692a500f21d86b0ac0c8261e1d752 100644
--- a/base/server/python/pki/server/__init__.py
+++ b/base/server/python/pki/server/__init__.py
@@ -318,7 +318,8 @@ class PKISubsystem(object):
     def disable(self):
         self.instance.undeploy(self.name)
 
-    def open_database(self, name='internaldb'):
+    def open_database(self, name='internaldb', bind_dn=None,
+                      bind_password=None):
 
         # TODO: add LDAPI support
         hostname = self.config['%s.ldapconn.host' % name]
@@ -341,7 +342,13 @@ class PKISubsystem(object):
         connection.set_security_database(self.instance.nssdb_dir)
 
         auth_type = self.config['%s.ldapauth.authtype' % name]
-        if auth_type == 'BasicAuth':
+        if (bind_dn is not None and bind_password is not None):
+            # connect using the provided credentials
+            connection.set_credentials(
+                bind_dn=bind_dn,
+                bind_password=bind_password
+            )
+        elif auth_type == 'BasicAuth':
             connection.set_credentials(
                 bind_dn=self.config['%s.ldapauth.bindDN' % name],
                 bind_password=self.instance.get_password(name)
diff --git a/base/server/python/pki/server/cli/db.py b/base/server/python/pki/server/cli/db.py
index 6cfd6b410eb75394f4b88b087b43a0538d3db6c6..709b39e2c576ffe60cfc04e71ebc9c3f3739bef5 100644
--- a/base/server/python/pki/server/cli/db.py
+++ b/base/server/python/pki/server/cli/db.py
@@ -23,6 +23,7 @@ from __future__ import print_function
 import getopt
 import ldap
 import nss.nss as nss
+import subprocess
 import sys
 
 import pki.cli
@@ -35,6 +36,92 @@ class DBCLI(pki.cli.CLI):
             'db', 'DB management commands')
 
         self.add_module(DBUpgrade())
+        self.add_module(DBSchemaUpgrade())
+
+
+class DBSchemaUpgrade(pki.cli.CLI):
+
+    SCHEMA_PATH = '/usr/share/pki/server/conf/schema.ldif'
+
+    def __init__(self):
+        super(DBSchemaUpgrade, self).__init__(
+            'schema-upgrade', 'Upgrade PKI database schema')
+
+    def usage(self):
+        print('Usage: pki-server db-schema-upgrade [OPTIONS]')
+        print()
+        print('  -i, --instance <instance ID>       Instance ID (default: pki-tomcat).')
+        print('  -D, --bind-dn <Bind DN>            DN to connect to DB (default: cn=Directory Manager).')
+        print('  -w, --bind-password <password>     Password to connect to DB.')
+        print('  -v, --verbose                      Run in verbose mode.')
+        print('      --help                         Show help message.')
+        print()
+
+    def execute(self, args):
+        try:
+            opts, _ = getopt.gnu_getopt(
+                args, 'i:D:w:v', ['instance=', 'bind-dn=', 'bind-password=',
+                                  'verbose', 'help'])
+
+        except getopt.GetoptError as e:
+            print('ERROR: ' + str(e))
+            self.usage()
+            sys.exit(1)
+
+        instance_name = 'pki-tomcat'
+        bind_dn = 'cn=Directory Manager'
+        bind_password = None
+
+        for o, a in opts:
+            if o in ('-i', '--instance'):
+                instance_name = a
+
+            elif o in ('-D', '--bind-dn'):
+                bind_dn = a
+
+            elif o in ('-w', '--bind-password'):
+                bind_password = a
+
+            elif o in ('-v', '--verbose'):
+                self.set_verbose(True)
+
+            elif o == '--help':
+                self.usage()
+                sys.exit()
+
+            else:
+                print('ERROR: unknown option ' + o)
+                self.usage()
+                sys.exit(1)
+
+        instance = pki.server.PKIInstance(instance_name)
+        instance.load()
+        self.update_schema(instance, bind_dn, bind_password)
+
+        self.print_message('Upgrade complete')
+
+    def update_schema(self, instance, bind_dn, bind_password):
+        # TODO(alee) re-implement this using open_database
+        subsystem = instance.subsystems[0]
+        host = subsystem.config['internaldb.ldapconn.host']
+        port = subsystem.config['internaldb.ldapconn.port']
+        secure = subsystem.config['internaldb.ldapconn.secureConn']
+        cmd = ['ldapmodify',
+               '-c',
+               '-D', bind_dn,
+               '-w', bind_password,
+               '-h', host,
+               '-p', port,
+               '-f', self.SCHEMA_PATH
+               ]
+
+        if secure.lower() == "true":
+            cmd.append('-Z')
+
+        try:
+            subprocess.check_output(cmd)
+        except subprocess.CalledProcessError as e:
+            print('ldapmodify returns {}: {}'.format(e.returncode, e.output))
 
 
 class DBUpgrade(pki.cli.CLI):
@@ -70,7 +157,7 @@ class DBUpgrade(pki.cli.CLI):
                 self.set_verbose(True)
 
             elif o == '--help':
-                self.print_help()
+                self.usage()
                 sys.exit()
 
             else:
diff --git a/base/server/python/pki/server/cli/kra.py b/base/server/python/pki/server/cli/kra.py
index cba3e234bb311458d859204bb81335a1591af176..389ea4d215bd7cac694089f25e4bf665602003bb 100644
--- a/base/server/python/pki/server/cli/kra.py
+++ b/base/server/python/pki/server/cli/kra.py
@@ -22,6 +22,8 @@ from __future__ import absolute_import
 from __future__ import print_function
 import getopt
 import io
+import ldap
+import ldif
 import os
 import shutil
 import sys
@@ -29,6 +31,26 @@ import tempfile
 
 import pki.cli
 
+KRA_VLVS = ['allKeys', 'kraAll',
+            'kraArchival', 'kraRecovery',
+            'kraCanceled', 'kraCanceledEnrollment', 'kraCanceledRecovery',
+            'kraRejected', 'kraRejectedEnrollment', 'kraRejectedRecovery',
+            'kraComplete', 'kraCompleteEnrollment', 'kraCompleteRecovery']
+KRA_VLV_PATH = '/usr/share/pki/kra/conf/vlv.ldif'
+KRA_VLV_TASKS_PATH = '/usr/share/pki/kra/conf/vlvtasks.ldif'
+
+
+def create_ldif(instance, subsystem, ldif_path, out_file):
+    subs = {'{instanceId}': instance.name,
+            '{database}': subsystem.config['internaldb.database'],
+            '{rootSuffix}': subsystem.config['internaldb.basedn']}
+
+    with open(ldif_path) as infile, open(out_file, 'w') as outfile:
+        for line in infile:
+            for src, target in subs.items():
+                line = line.replace(src, target)
+            outfile.write(line)
+
 
 class KRACLI(pki.cli.CLI):
 
@@ -37,6 +59,7 @@ class KRACLI(pki.cli.CLI):
             'kra', 'KRA management commands')
 
         self.add_module(KRACloneCLI())
+        self.add_module(KRADBCLI())
 
 
 class KRACloneCLI(pki.cli.CLI):
@@ -141,3 +164,332 @@ class KRAClonePrepareCLI(pki.cli.CLI):
 
         finally:
             shutil.rmtree(tmpdir)
+
+
+class KRADBCLI(pki.cli.CLI):
+
+    def __init__(self):
+        super(KRADBCLI, self).__init__(
+            'db', 'KRA DB management commands')
+
+        self.add_module(KRADBVLVCLI())
+
+
+class KRADBVLVCLI(pki.cli.CLI):
+
+    def __init__(self):
+        super(KRADBVLVCLI, self).__init__(
+            'vlv', 'KRA VLV management commands')
+
+        self.add_module(KRADBVLVAddCLI())
+        self.add_module(KRADBVLVDeleteCLI())
+        self.add_module(KRADBVLVReindexCLI())
+
+
+class KRADBVLVAddCLI(pki.cli.CLI):
+
+    KRA_VLV_PATH = '/usr/share/pki/kra/conf/vlv.ldif'
+
+    def __init__(self):
+        super(KRADBVLVAddCLI, self).__init__(
+            'add', 'Add KRA VLVs')
+        self.out_file = None
+
+    def print_help(self):
+        print('Usage: pki-server kra-db-vlv-add [OPTIONS]')
+        print()
+        print('  -i, --instance <instance ID>       Instance ID (default: pki-tomcat).')
+        print('  -D, --bind-dn <Bind DN>            DN to connect to DB (default: cn=Directory Manager).')
+        print('  -w, --bind-password <password>     Password to connect to DB.')
+        print('  -v, --verbose                      Run in verbose mode.')
+        print('  -g, --generate-ldif <outfile>      Generate LDIF of required changes.')
+        print('      --help                         Show help message.')
+        print()
+
+    def execute(self, args):
+        try:
+            opts, _ = getopt.gnu_getopt(
+                args,
+                'i:D:w:x:g:v',
+                ['instance=', 'bind-dn=', 'bind-password=', 'generate-ldif=',
+                 'verbose', 'help']
+            )
+
+        except getopt.GetoptError as e:
+            print('ERROR: ' + str(e))
+            self.print_help()
+            sys.exit(1)
+
+        instance_name = 'pki-tomcat'
+        bind_dn = 'cn=Directory Manager'
+        bind_password = None
+
+        for o, a in opts:
+            if o in ('-i', '--instance'):
+                instance_name = a
+
+            elif o in ('-D', '--bind-dn'):
+                bind_dn = a
+
+            elif o in ('-w', '--bind-password'):
+                bind_password = a
+
+            elif o in ('-g', '--generate-ldif'):
+                self.out_file = a
+
+            elif o in ('-v', '--verbose'):
+                self.set_verbose(True)
+
+            elif o == '--help':
+                self.print_help()
+                sys.exit()
+
+            else:
+                print('ERROR: unknown option ' + o)
+                self.print_help()
+                sys.exit(1)
+
+        instance = pki.server.PKIInstance(instance_name)
+        instance.load()
+        self.add_vlv(instance, bind_dn, bind_password)
+
+    def add_vlv(self, instance, bind_dn, bind_password):
+        subsystem = instance.get_subsystem('kra')
+        if not subsystem:
+            print('No KRA subsystem available.')
+            return
+
+        if self.out_file:
+            create_ldif(instance, subsystem, self.KRA_VLV_PATH, self.out_file)
+            print('KRA VLVs written to ' + self.out_file)
+            return
+
+        ldif_file = tempfile.NamedTemporaryFile(delete=False)
+        create_ldif(instance, subsystem, self.KRA_VLV_PATH, ldif_file.name)
+
+        conn = subsystem.open_database(bind_dn=bind_dn,
+                                       bind_password=bind_password)
+
+        try:
+            parser = ldif.LDIFRecordList(open(ldif_file.name, "rb"))
+            parser.parse()
+            for dn, entry in parser.all_records:
+                add_modlist = ldap.modlist.addModlist(entry)
+                conn.ldap.add_s(dn, add_modlist)
+        finally:
+            os.unlink(ldif_file.name)
+            conn.close()
+
+        print('KRA VLVs added to the DB for ' + instance.name)
+
+
+class KRADBVLVDeleteCLI(pki.cli.CLI):
+
+    def __init__(self):
+        super(KRADBVLVDeleteCLI, self).__init__(
+            'del', 'Delete KRA VLVs')
+        self.out_file = None
+
+    def print_help(self):
+        print('Usage: pki-server kra-db-vlv-del [OPTIONS]')
+        print()
+        print('  -i, --instance <instance ID>       Instance ID (default: pki-tomcat).')
+        print('  -D, --bind-dn <Bind DN>            DN to connect to DB (default: cn=Directory Manager).')
+        print('  -w, --bind-password <password>     Password to connect to DB.')
+        print('  -g, --generate-ldif <outfile>      Generate LDIF of required changes.')
+        print('  -v, --verbose                      Run in verbose mode.')
+        print('      --help                         Show help message.')
+        print()
+
+    def execute(self, args):
+        try:
+            opts, _ = getopt.gnu_getopt(
+                args,
+                'i:D:w:x:g:v',
+                ['instance=', 'bind-dn=', 'bind-password=', 'generate-ldif=',
+                 'verbose', 'help']
+            )
+
+        except getopt.GetoptError as e:
+            print('ERROR: ' + str(e))
+            self.print_help()
+            sys.exit(1)
+
+        instance_name = 'pki-tomcat'
+        bind_dn = None
+        bind_password = None
+
+        for o, a in opts:
+            if o in ('-i', '--instance'):
+                instance_name = a
+
+            elif o in ('-D', '--bind-dn'):
+                bind_dn = a
+
+            elif o in ('-w', '--bind-password'):
+                bind_password = a
+
+            elif o in ('-g', '--generate-ldif'):
+                self.out_file = a
+
+            elif o in ('-v', '--verbose'):
+                self.set_verbose(True)
+
+            elif o == '--help':
+                self.print_help()
+                sys.exit()
+
+            else:
+                print('ERROR: unknown option ' + o)
+                self.print_help()
+                sys.exit(1)
+
+        instance = pki.server.PKIInstance(instance_name)
+        instance.load()
+        self.delete_vlv(instance, bind_dn, bind_password)
+
+    def delete_vlv(self, instance, bind_dn, bind_password):
+        subsystem = instance.get_subsystem('kra')
+        if not subsystem:
+            if self.verbose:
+                print('modify_kra_vlv: No KRA subsystem available.  '
+                      'Skipping ...')
+                return
+        database = subsystem.config['internaldb.database']
+
+        if self.out_file:
+            with open(self.out_file, "w") as f:
+                for vlv in KRA_VLVS:
+                    dn = ("cn=" + vlv + '-' + instance.name +
+                          ',cn=' + database +
+                          ',cn=ldbm database, cn=plugins, cn=config')
+                    index_dn = ("cn=" + vlv + '-' + instance.name +
+                                "Index," + dn)
+                    f.write('dn: ' + index_dn + '\n')
+                    f.write('changetype: delete' + '\n')
+                    f.write('\n')
+                    f.write('dn: ' + dn + '\n')
+                    f.write('changetype: delete' + '\n')
+                    f.write('\n')
+            print('KRA VLV changes written to ' + self.out_file)
+            return
+
+        conn = subsystem.open_database(bind_dn=bind_dn,
+                                       bind_password=bind_password)
+        try:
+            for vlv in KRA_VLVS:
+                dn = ("cn=" + vlv + '-' + instance.name + ',cn=' + database +
+                      ',cn=ldbm database, cn=plugins, cn=config')
+                index_dn = "cn=" + vlv + '-' + instance.name + "Index," + dn
+
+                try:
+                    conn.ldap.delete_s(index_dn)
+                except ldap.NO_SUCH_OBJECT:
+                    pass
+
+                try:
+                    conn.ldap.delete_s(dn)
+                except ldap.NO_SUCH_OBJECT:
+                    pass
+
+        finally:
+            conn.close()
+
+        print('KRA VLVs deleted from the DB for ' + instance.name)
+
+
+class KRADBVLVReindexCLI(pki.cli.CLI):
+
+    def __init__(self):
+        super(KRADBVLVReindexCLI, self).__init__(
+            'reindex', 'Re-index KRA VLVs')
+        self.out_file = None
+
+    def print_help(self):
+        print('Usage: pki-server kra-db-vlv-reindex [OPTIONS]')
+        print()
+        print('  -i, --instance <instance ID>       Instance ID (default: pki-tomcat).')
+        print('  -D, --bind_dn <Bind DN>            DN to connect to DB (default: cn=Directory Manager).')
+        print('  -w, --bind-password <password>     Password to connect to DB.')
+        print('  -g, --generate-ldif <outfile>      Generate LDIF of required changes.')
+        print('  -v, --verbose                      Run in verbose mode.')
+        print('      --help                         Show help message.')
+        print()
+
+    def execute(self, args):
+        try:
+            opts, _ = getopt.gnu_getopt(
+                args,
+                'i:D:w:x:g:v',
+                ['instance=', 'bind-dn=', 'bind-password=', 'generate-ldif=',
+                 'verbose', 'help']
+            )
+
+        except getopt.GetoptError as e:
+            print('ERROR: ' + str(e))
+            self.print_help()
+            sys.exit(1)
+
+        instance_name = 'pki-tomcat'
+        bind_dn = 'cn=Directory Manager'
+        bind_password = None
+
+        for o, a in opts:
+            if o in ('-i', '--instance'):
+                instance_name = a
+
+            elif o in ('-D', '--bind-dn'):
+                bind_dn = a
+
+            elif o in ('-w', '--bind-password'):
+                bind_password = a
+
+            elif o in ('-g', '--generate-ldif'):
+                self.out_file = a
+
+            elif o in ('-v', '--verbose'):
+                self.set_verbose(True)
+
+            elif o == '--help':
+                self.print_help()
+                sys.exit()
+
+            else:
+                print('ERROR: unknown option ' + o)
+                self.print_help()
+                sys.exit(1)
+
+        instance = pki.server.PKIInstance(instance_name)
+        instance.load()
+        self.reindex_vlv(instance, bind_dn, bind_password)
+
+    def reindex_vlv(self, instance, bind_dn, bind_password):
+        subsystem = instance.get_subsystem('kra')
+        if not subsystem:
+            if self.verbose:
+                print('reindex_vlv: No KRA subsystem available.  '
+                      'Skipping ...')
+                return
+
+        if self.out_file:
+            create_ldif(instance, subsystem, KRA_VLV_TASKS_PATH, self.out_file)
+            print('KRA VLV reindex task written to ' + self.out_file)
+            return
+
+        ldif_file = tempfile.NamedTemporaryFile(delete=False)
+        create_ldif(instance, subsystem, KRA_VLV_TASKS_PATH, ldif_file.name)
+
+        conn = subsystem.open_database(bind_dn=bind_dn,
+                                       bind_password=bind_password)
+
+        try:
+            parser = ldif.LDIFRecordList(open(ldif_file.name, "rb"))
+            parser.parse()
+            for dn, entry in parser.all_records:
+                add_modlist = ldap.modlist.addModlist(entry)
+                conn.ldap.add_s(dn, add_modlist)
+        finally:
+            os.unlink(ldif_file.name)
+            conn.close()
+
+        print('KRA VLV reindex initiated for ' + instance.name)
-- 
2.4.3

_______________________________________________
Pki-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/pki-devel

Reply via email to