Yedidyah Bar David has uploaded a new change for review.

Change subject: packaging: setup: Allow dwh on separate host
......................................................................

packaging: setup: Allow dwh on separate host

Change-Id: I06fc960481af258b3954a8968be3439393d3ebdb
Bug-Url: https://bugzilla.redhat.com/1080997
Signed-off-by: Yedidyah Bar David <d...@redhat.com>
---
M ovirt-engine-dwh.spec.in
M packaging/setup/ovirt_engine_setup/dwhconstants.py
A 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/core/__init__.py
A 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/core/misc.py
R 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/__init__.py
R 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/connection.py
R 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/pgpass.py
M packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/core/misc.py
A 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh-common/db/__init__.py
R 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh-common/db/connection.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/__init__.py
M 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/provisioning/postgres.py
12 files changed, 149 insertions(+), 23 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-dwh refs/changes/16/27516/1

diff --git a/ovirt-engine-dwh.spec.in b/ovirt-engine-dwh.spec.in
index e5157b7..a81d16b 100644
--- a/ovirt-engine-dwh.spec.in
+++ b/ovirt-engine-dwh.spec.in
@@ -103,7 +103,6 @@
 Requires:      java-1.7.0-openjdk
 Requires:      jpackage-utils
 Requires:      logrotate
-Requires:      ovirt-engine >= 3.5.0
 Requires:      postgresql-jdbc
 
 %description
@@ -121,12 +120,21 @@
 Summary:       %{product_name} setup
 Group:         Virtualization/Management
 Requires:      ovirt-engine-setup-plugin-ovirt-engine-common
+Requires:      ovirt-engine-setup-plugin-postgresql-server
+Requires:      %{name}-setup-plugin-common
 
 %description setup
 %{product_description} setup package.
 
 %prep
 %setup -c -q
+
+%package setup-plugin-common
+Summary:       %{product_name} Setup and upgrade common plugins
+Group:         Virtualization/Management
+
+%description setup-plugin-common
+%{product_description} Setup and upgrade common plugins
 
 %build
 make %{make_common_opts}
@@ -238,9 +246,12 @@
 %endif
 
 %files setup
-%{_datadir}/ovirt-engine/setup/ovirt_engine_setup/dwh*
 %{_datadir}/ovirt-engine/setup/plugins/*/ovirt-engine-dwh/
 
+%files setup-plugin-common
+%{_datadir}/ovirt-engine/setup/ovirt_engine_setup/dwh*
+%{_datadir}/ovirt-engine/setup/plugins/*/ovirt-engine-dwh-common/
+
 %changelog
 * Sun Mar 30 2014 Yaniv Dary <yd...@redhat.com> - 3.4.1
 Fixed issues:
diff --git a/packaging/setup/ovirt_engine_setup/dwhconstants.py 
b/packaging/setup/ovirt_engine_setup/dwhconstants.py
index 1a92823..398b29e 100644
--- a/packaging/setup/ovirt_engine_setup/dwhconstants.py
+++ b/packaging/setup/ovirt_engine_setup/dwhconstants.py
@@ -221,6 +221,8 @@
     def RESTORE_BACKUP_LATE(self):
         return 'OVESETUP_DWH_DB/restoreBackupLate'
 
+    CREDS_NEEDED = 'OVESETUP_DWH_DB/credsNeeded'
+
 
 @util.export
 @util.codegen
diff --git 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/core/__init__.py
 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/core/__init__.py
new file mode 100644
index 0000000..ca8ccb8
--- /dev/null
+++ 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/core/__init__.py
@@ -0,0 +1,33 @@
+#
+# ovirt-engine-setup -- ovirt engine setup
+# Copyright (C) 2013 Red Hat, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+
+"""ovirt-host-remove core plugin."""
+
+
+from otopi import util
+
+
+from . import misc
+
+
+@util.export
+def createPlugins(context):
+    misc.Plugin(context=context)
+
+
+# vim: expandtab tabstop=4 shiftwidth=4
diff --git 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/core/misc.py
 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/core/misc.py
new file mode 100644
index 0000000..55268f1
--- /dev/null
+++ 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/core/misc.py
@@ -0,0 +1,59 @@
+#
+# ovirt-engine-setup -- ovirt engine setup
+# Copyright (C) 2013 Red Hat, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+
+import gettext
+_ = lambda m: gettext.dgettext(message=m, domain='ovirt-engine-dwh')
+
+
+from otopi import util
+from otopi import plugin
+
+
+from ovirt_engine_setup import constants as osetupcons
+from ovirt_engine_setup import dwhconstants as odwhcons
+
+
+@util.export
+class Plugin(plugin.PluginBase):
+
+    def __init__(self, context):
+        super(Plugin, self).__init__(context=context)
+
+    @plugin.event(
+        stage=plugin.Stages.STAGE_INIT,
+    )
+    def _init(self):
+        self.environment.setdefault(odwhcons.CoreEnv.ENABLE, None)
+
+    @plugin.event(
+        stage=plugin.Stages.STAGE_SETUP,
+    )
+    def _setup(self):
+        self.environment[
+            osetupcons.CoreEnv.REGISTER_UNINSTALL_GROUPS
+        ].createGroup(
+            group='ovirt_dwh_files',
+            description='DWH files',
+            optional=True,
+        )
+        self.environment[
+            osetupcons.CoreEnv.SETUP_ATTRS_MODULES
+        ].append(odwhcons)
+
+
+# vim: expandtab tabstop=4 shiftwidth=4
diff --git 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/__init__.py 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/__init__.py
similarity index 100%
rename from 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/__init__.py
rename to 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/__init__.py
diff --git 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/connection.py
similarity index 100%
rename from 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/connection.py
rename to 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/connection.py
diff --git 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/pgpass.py 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/pgpass.py
similarity index 100%
rename from 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/db/pgpass.py
rename to 
packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh-common/db/pgpass.py
diff --git 
a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/core/misc.py 
b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/core/misc.py
index e9bf710..56ccd95 100644
--- a/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/core/misc.py
+++ b/packaging/setup/plugins/ovirt-engine-common/ovirt-engine-dwh/core/misc.py
@@ -35,25 +35,9 @@
         super(Plugin, self).__init__(context=context)
 
     @plugin.event(
-        stage=plugin.Stages.STAGE_INIT,
-    )
-    def _init(self):
-        self.environment.setdefault(odwhcons.CoreEnv.ENABLE, None)
-
-    @plugin.event(
         stage=plugin.Stages.STAGE_SETUP,
     )
     def _setup(self):
-        self.environment[
-            osetupcons.CoreEnv.REGISTER_UNINSTALL_GROUPS
-        ].createGroup(
-            group='ovirt_dwh_files',
-            description='DWH files',
-            optional=True,
-        )
-        self.environment[
-            osetupcons.CoreEnv.SETUP_ATTRS_MODULES
-        ].append(odwhcons)
         self.logger.debug(
             'dwh version: %s-%s (%s)\n',
             odwhcons.Const.PACKAGE_NAME,
diff --git 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh-common/db/__init__.py
 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh-common/db/__init__.py
new file mode 100644
index 0000000..3ad2872
--- /dev/null
+++ 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh-common/db/__init__.py
@@ -0,0 +1,30 @@
+#
+# ovirt-engine-setup -- ovirt engine setup
+# Copyright (C) 2013 Red Hat, Inc.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+
+from otopi import util
+
+
+from . import connection
+
+
+@util.export
+def createPlugins(context):
+    connection.Plugin(context=context)
+
+
+# vim: expandtab tabstop=4 shiftwidth=4
diff --git 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/connection.py 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh-common/db/connection.py
similarity index 96%
rename from 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/connection.py
rename to 
packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh-common/db/connection.py
index b7333a9..1de5267 100644
--- 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/connection.py
+++ 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh-common/db/connection.py
@@ -93,6 +93,12 @@
         super(Plugin, self).__init__(context=context)
 
     @plugin.event(
+        stage=plugin.Stages.STAGE_INIT,
+    )
+    def _init(self):
+        self.environment.setdefault(odwhcons.DBEnv.CREDS_NEEDED, None)
+
+    @plugin.event(
         stage=plugin.Stages.STAGE_SETUP,
     )
     def _commands(self):
@@ -103,7 +109,10 @@
     @plugin.event(
         stage=plugin.Stages.STAGE_CUSTOMIZATION,
         name=odwhcons.Stages.DB_CONNECTION_CUSTOMIZATION,
-        condition=lambda self: self.environment[odwhcons.CoreEnv.ENABLE],
+        condition=lambda self: (
+            self.environment[odwhcons.CoreEnv.ENABLE] or
+            self.environment[odwhcons.DBEnv.CREDS_NEEDED]
+        ),
         before=(
             osetupcons.Stages.DIALOG_TITLES_E_DATABASE,
         ),
diff --git 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/__init__.py 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/__init__.py
index d37eb09..f6fccb7 100644
--- a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/__init__.py
+++ b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/db/__init__.py
@@ -19,13 +19,11 @@
 from otopi import util
 
 
-from . import connection
 from . import schema
 
 
 @util.export
 def createPlugins(context):
-    connection.Plugin(context=context)
     schema.Plugin(context=context)
 
 
diff --git 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/provisioning/postgres.py
 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/provisioning/postgres.py
index aa004a0..e2045f7 100644
--- 
a/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/provisioning/postgres.py
+++ 
b/packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-dwh/provisioning/postgres.py
@@ -152,14 +152,14 @@
         stage=plugin.Stages.STAGE_CUSTOMIZATION,
         priority=plugin.Stages.PRIORITY_LAST,
         condition=lambda self: self.environment[
-            osetupcons.DBEnv.HOST
+            odwhcons.DBEnv.HOST
         ] == 'localhost',
     )
     def _customization_firewall(self):
         self.environment[osetupcons.NetEnv.FIREWALLD_SERVICES].extend([
             {
                 'name': 'ovirt-postgres',
-                'directory': 'ovirt-engine'
+                'directory': 'postgresql-server'
             },
         ])
 


-- 
To view, visit http://gerrit.ovirt.org/27516
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I06fc960481af258b3954a8968be3439393d3ebdb
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-dwh
Gerrit-Branch: master
Gerrit-Owner: Yedidyah Bar David <d...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to